Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 3, 2026

CI was failing with npm error Unsupported URL Type "workspace:": workspace:* because the repository uses pnpm-style workspace protocol while the workflow used npm.

Package manager migration

  • Add pnpm-workspace.yaml defining workspace structure
  • Generate pnpm-lock.yaml, remove package-lock.json
  • Update CI workflow: enable Corepack, cache pnpm, use pnpm install --frozen-lockfile

TypeScript fixes

  • Remove unused imports and parameters (prefix with _ where semantically required)
  • Add nullish coalescing for optional classification.confidence
  • Filter undefined from labels array with type predicate
  • Wrap boolean expression with Boolean() for strict typing
// Before: type error on possibly undefined
if (classification.confidence > 0.7) { ... }

// After: nullish coalescing with default
if ((classification.confidence ?? 0) > 0.7) { ... }

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- Add pnpm-workspace.yaml to define workspace structure
- Generate pnpm-lock.yaml to replace package-lock.json
- Update CI workflow to use pnpm instead of npm
- Enable Corepack for pnpm support
- Fix TypeScript errors in conversations subsystem
- Remove unused import (path) in conversations.ts
- Add nullish coalescing for classification.confidence
- Filter undefined values from labels array
- Prefix unused parameter with underscore in gateNotDuplicate
- Wrap hasLabels && hasPriority with Boolean() for strict typing
- Prefix unused match parameter with underscore in normalizeCodeBlocks

Co-authored-by: kayodebristol <3579196+kayodebristol@users.noreply.github.com>
Copilot AI changed the title [WIP] Update actions/upload-artifact from 4 to 6 Fix CI: migrate from npm to pnpm for workspace:* protocol support Feb 3, 2026
Copilot AI requested a review from kayodebristol February 3, 2026 22:11
@kayodebristol
Copy link
Contributor

Draft PR noted in merge sweep. To get this ready to merge: (1) add/confirm a clear description of intent + testing notes, (2) ensure CI is green, (3) request review from required code owners (if any), (4) mark Ready for review once safe. If you want, I can help triage any failing checks.

@kayodebristol kayodebristol deleted the branch dependabot/github_actions/actions/upload-artifact-6 February 4, 2026 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants