Skip to content

fix(ci): add missing @stackpanel/infra dep to apps/api#28

Merged
cooper (czxtm) merged 1 commit intomainfrom
claude/debug-ci-deployment-44sGc
May 2, 2026
Merged

fix(ci): add missing @stackpanel/infra dep to apps/api#28
cooper (czxtm) merged 1 commit intomainfrom
claude/debug-ci-deployment-44sGc

Conversation

@czxtm
Copy link
Copy Markdown
Contributor

Summary

  • apps/api/alchemy.run.ts:23 imports from @stackpanel/infra/lib/deploy, but the workspace dep was never declared in apps/api/package.json. CI's bunx alchemy deploy --stage production consequently failed with ResolveMessage: Cannot find module '@stackpanel/infra/lib/deploy' from '.../apps/api/alchemy.run.ts'.
  • apps/web and apps/docs (which use the same helper) both already declare "@stackpanel/infra": "workspace:*"apps/api was the outlier.
  • Added the dep + corresponding entry in bun.lock. After install, apps/api/node_modules/@stackpanel/infra symlinks to packages/infra, and bun --print 'require.resolve("@stackpanel/infra/lib/deploy")' from apps/api now resolves to packages/infra/src/lib/deploy.ts.

Test plan

  • CI re-runs the alchemy deploy --stage production step and the resolve error no longer appears
  • Spot-check that web/docs deploys still resolve @stackpanel/infra/lib/deploy (no lockfile drift)

https://claude.ai/code/session_01WGbZGB93xMgBpBmoUDk91w


Generated by Claude Code

apps/api/alchemy.run.ts imports from @stackpanel/infra/lib/deploy
but the workspace dep wasn't declared, so bunx alchemy deploy failed
with ResolveMessage when CI ran from a clean install.

https://claude.ai/code/session_01WGbZGB93xMgBpBmoUDk91w
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Preview pr-28 has been destroyed.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Docs preview pr-28 has been destroyed.

@czxtm cooper (czxtm) marked this pull request as ready for review May 2, 2026 13:58
@czxtm cooper (czxtm) merged commit a24b86d into main May 2, 2026
6 of 7 checks passed
@cursor
Copy link
Copy Markdown

cursor Bot commented May 2, 2026

PR Summary

Low Risk
Low risk dependency/lockfile change to fix module resolution during deploy; no runtime logic changes beyond enabling an already-used import to resolve.

Overview
Fixes CI deployment failures by declaring the missing @stackpanel/infra workspace dependency for apps/api, aligning it with other apps that already use the same deploy helper.

Updates bun.lock accordingly so @stackpanel/infra/lib/deploy used by apps/api/alchemy.run.ts resolves during bunx alchemy deploy.

Reviewed by Cursor Bugbot for commit 2c19960. Configure here.

@czxtm cooper (czxtm) deleted the claude/debug-ci-deployment-44sGc branch May 2, 2026 13:59
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON, but it could not run because the branch was deleted or merged before autofix could start.

Reviewed by Cursor Bugbot for commit 2c19960. Configure here.

Comment thread apps/api/package.json
"@stackpanel/api": "workspace:*",
"@stackpanel/auth": "workspace:*",
"@stackpanel/db": "workspace:*",
"@stackpanel/infra": "workspace:*",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Infra dep placed in dependencies instead of devDependencies

Low Severity

@stackpanel/infra is added to dependencies but both apps/web and apps/docs declare it in devDependencies. This package is only used by alchemy.run.ts (a deployment-time script), not at runtime by the API server. Placing it in dependencies is inconsistent with the established pattern and misrepresents its role.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 2c19960. Configure here.

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