Skip to content

Set CI for one-shot pnpm compose services - #37

Open
uF4No wants to merge 1 commit into
mainfrom
codex/pnpm-ci-for-one-shot-services
Open

Set CI for one-shot pnpm compose services#37
uF4No wants to merge 1 commit into
mainfrom
codex/pnpm-ci-for-one-shot-services

Conversation

@uF4No

@uF4No uF4No commented May 18, 2026

Copy link
Copy Markdown

Summary

  • set CI=true for the compose services that run pnpm install without a TTY
  • cover bridge-funds, sso-contracts-deployer, and sso-permissions-setup
  • avoid ERR_PNPM_ABORTED_REMOVE_MODULES_DIR_NO_TTY when mounted node_modules needs to be recreated

Context

Running docker compose up -d through the demo repo's pnpm local-prividium:start hits sso-contracts-deployer first. That service runs pnpm install in a non-interactive container with a mounted node_modules volume, so pnpm may need to purge and recreate the directory. Without CI=true, pnpm aborts instead of proceeding.

Verification

  • reproduced the original failure in sso-contracts-deployer with ERR_PNPM_ABORTED_REMOVE_MODULES_DIR_NO_TTY
  • reran after this change and confirmed the service moved past pnpm install into the TypeScript deployment script

@Deniallugo
Deniallugo force-pushed the codex/pnpm-ci-for-one-shot-services branch from 675070d to 18e3dc2 Compare June 19, 2026 21:16
@vladbochok
vladbochok force-pushed the codex/pnpm-ci-for-one-shot-services branch from 18e3dc2 to 675070d Compare June 20, 2026 14:16
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