build: use pnpm as package management system
Metadata
Current evaluation
Merged PR #63 overhauled craft-actions dev tooling: npm to pnpm, Jest to Vitest, added tsc --noEmit typecheck, updated/deprecated eslint, dropped custom prettier config, fixed test linting, removed redundant pack/all scripts. All CI checks passed.
Resolution: closed resolved
Reason: PR #63 was approved by mr-cal ("LGTM") and merged into main with all CI checks green; the codebase now uses pnpm, vitest and the new tsc/eslint setup. Follow-up restructuring promised in the PR description landed separately in PR #73.
Evaluated by: mr-cal-bot
Issue body
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #63 overhauled craft-actions dev tooling: npm to pnpm, Jest to Vitest, added tsc --noEmit typecheck, updated/deprecated eslint, dropped custom prettier config, fixed test linting, removed redundant pack/all scripts. All CI checks passed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged. craft-actions#63 overhauled dev tooling: npm to pnpm, Jest to Vitest, added tsc --noEmit typecheck, modernized ESLint, dropped the custom Prettier config, fixed test linting, and removed the redundant pack/all scripts. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in canonical/craft-actions#63: replaced npm with pnpm and Jest with Vitest, updated lint/TypeScript tooling, fixed test linting, and removed redundant scripts; file restructuring was deferred. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Replaced npm with pnpm, jest with vitest, and updated eslint and TypeScript configs. Removed redundant scripts and fixed test linting. CI passed and review approved. File structure refactoring is deferred to a separate PR. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-actions#73
(confidence 88%)
Merged follow-up implementing the deferred file-layout refactor (per-action folders, dist/ under each action) explicitly left out of #63.
-
Related To:
canonical/craft-actions#61
(confidence 45%)
Earlier refactor extracting shareable components into common modules, the components #63's TS config change was meant to support.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #112 chore(deps): update pnpm to v12 | craft-actions | merged | Renovate PR canonical/craft-actions#112 bumped pnpm 11.25.0 -> 12.3.4 (major). Approved by 1 reviewer and merged into main (2 files, +42/-140); all rockcraft-pack-action, test-action-code and Canonical CLA checks passed. |