Skip to content

fix(examples): link cli-empresa-inc to @figtools/cli via workspace:* - #54

Merged
dryor merged 1 commit into
mainfrom
fix/release-lockfile-out-of-sync
Aug 8, 2026
Merged

fix(examples): link cli-empresa-inc to @figtools/cli via workspace:*#54
dryor merged 1 commit into
mainfrom
fix/release-lockfile-out-of-sync

Conversation

@dryor

@dryor dryor commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Diagnoses the failing "Publish Packages" run (https://github.com/dryor/figtools/actions/runs/31269182123/job/93132163808): pnpm install --frozen-lockfile failed because examples/cli-empresa-inc/package.json depends on @figtools/cli with a real semver range (^0.4.0), which changesets bumped when versioning @figtools/cli — but 0.4.0 didn't exist on npm yet, since publishing it is what that same workflow run was trying to do.
  • Switches the example's dependency to workspace:* (matching the pattern packages/cli already uses for @figtools/core), so it resolves locally via a workspace link instead of racing against npm. Regenerated pnpm-lock.yaml to match.
  • Verified by reproducing the exact CI command (pnpm install --frozen-lockfile --ignore-scripts) after a clean node_modules wipe — passes now. Also ran pnpm build and pnpm test, both green.

Test plan

  • pnpm install --frozen-lockfile --ignore-scripts (same command the failing job runs) — passes
  • pnpm build — passes
  • pnpm test — 128/128 passing
  • Re-run the "Publish Packages" workflow after merge to confirm the actual publish succeeds

🤖 Generated with Claude Code

examples/cli-empresa-inc depended on @figtools/cli with a real semver
range (^0.4.0), so when changesets bumped it to track the not-yet-published
next version, `pnpm install --frozen-lockfile` in the Publish Packages
workflow tried to resolve it against npm and failed — the version didn't
exist yet because publishing it is what the same job was trying to do.
Switching to workspace:* (the same pattern packages/cli already uses for
@figtools/core) resolves it locally instead, so version bumps here no
longer race against npm publish.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@dryor
dryor merged commit af7a4cc into main Aug 8, 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.

1 participant