Skip to content

Fix publish workflow caching for per-package pnpm lockfiles - #46

Merged
chee merged 2 commits into
mainfrom
copilot/fix-publish-job
Aug 6, 2026
Merged

Fix publish workflow caching for per-package pnpm lockfiles#46
chee merged 2 commits into
mainfrom
copilot/fix-publish-job

Conversation

Copilot AI commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

The publish GitHub Actions job was failing because pnpm caching was still configured as if the repo had a root pnpm-lock.yaml. This repo installs and builds packages independently, so the workflow now keys caching off the per-package lockfiles instead.

  • Root cause

    • actions/setup-node was using cache: pnpm without a dependency path override, which made it require /pnpm-lock.yaml at the repository root.
    • That conflicts with this repo’s package layout, where each tool owns its own lockfile.
  • Workflow changes

    • Pin pnpm/action-setup to the repo’s pnpm version (11.15.1) for consistency with other workflows.
    • Update actions/setup-node to use all package lockfiles for pnpm cache invalidation:
      - uses: actions/setup-node@v4
        with:
          node-version: 22
          cache: pnpm
          cache-dependency-path: |
            **/pnpm-lock.yaml
    • Add a separate actions/cache entry for package-local node_modules directories so publish can benefit from per-package install reuse without reintroducing a root lockfile.
  • Behavioral impact

    • The publish workflow no longer assumes a root lockfile exists.
    • Cache keys now track the actual package boundaries in this repository, including the nested tasks/src/in-the-cloud package.

Co-authored-by: chee <178266+chee@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job publish Fix publish workflow caching for per-package pnpm lockfiles Aug 6, 2026
Copilot finished work on behalf of chee August 6, 2026 22:01
Copilot AI requested a review from chee August 6, 2026 22:01
@chee
chee marked this pull request as ready for review August 6, 2026 22:40
@chee
chee merged commit ea72c42 into main Aug 6, 2026
4 checks passed
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