Skip to content

fix(scripts): update publish-cli + e2e-publish for colonyq rename#506

Merged
NagyVikt merged 1 commit into
mainfrom
agent/claude/fix-publish-scripts-after-rename-2026-05-10-03-15
May 10, 2026
Merged

fix(scripts): update publish-cli + e2e-publish for colonyq rename#506
NagyVikt merged 1 commit into
mainfrom
agent/claude/fix-publish-scripts-after-rename-2026-05-10-03-15

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Summary

PR #505 renamed @imdeadpool/colony-clicolonyq, but missed two pnpm --filter calls in the publish scripts. Without this fix, pnpm publish:cli fails immediately because the filter doesn't match any workspace package.

File Line Change
scripts/publish-cli.sh 10 pnpm --filter @imdeadpool/colony-cli stage-publishpnpm --filter colonyq stage-publish
scripts/e2e-publish.sh 46 same

Test plan

  • pnpm publish:cli:dry-run → tarball generates as colonyq-0.7.0.tgz (422.5 kB packed, 1.8 MB unpacked, 23 files); registry would publish under colonyq@0.7.0

Why this wasn't caught in #505

The rename PR's verification ran pnpm --filter colonyq build/test/typecheck directly, which proved the package itself works under the new name. But it didn't exercise the wrapper scripts that drive publish — those still hardcoded the old filter. Adding a pnpm publish:cli:dry-run step to CI would catch this class of issue.

🤖 Generated with Claude Code

PR #505 renamed the package to `colonyq` but missed two `pnpm --filter`
calls in the publish scripts. Without this fix, `pnpm publish:cli` fails
because the filter no longer matches a workspace package.

Verified via `pnpm publish:cli:dry-run`: tarball generates correctly as
`colonyq-0.7.0.tgz` (422.5 kB packed).
@NagyVikt NagyVikt merged commit d145ce0 into main May 10, 2026
1 of 3 checks passed
@NagyVikt NagyVikt deleted the agent/claude/fix-publish-scripts-after-rename-2026-05-10-03-15 branch May 10, 2026 13:31
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