Skip to content

agent/ci-workflow - #7

Open
shramee wants to merge 2 commits into
mainfrom
agent/ci-workflow
Open

agent/ci-workflow#7
shramee wants to merge 2 commits into
mainfrom
agent/ci-workflow

Conversation

@shramee

@shramee shramee commented Aug 5, 2026

Copy link
Copy Markdown
Contributor
  • fix: add missing pnpm-workspace.yaml, add ts-jest to config
  • ci: add GitHub Actions workflow and CLAUDE.md

shrameez and others added 2 commits July 6, 2026 19:56
Add CI pipeline at .github/workflows/ci.yml running on push/PR to main:
checkout, Node 20, pnpm via corepack, Go 1.22, install, embed,
build, test, format:check.

Add CLAUDE.md documenting workspace layout and available commands.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
pnpm doesn't honor the 'workspaces' field in package.json (that's an npm/yarn
convention); without pnpm-workspace.yaml the packages were never linked, so
config's devDependencies like tsup never resolved. config's jest.config also
referenced the ts-jest preset without it being a dependency.

Note: core/tests/index.test.ts has a pre-existing failing test unrelated to
this fix ('proving incorrect proof format' throws instead of asserting) —
touches proof/circuit code, left for human review per CLAUDE.md.
@shramee

shramee commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

⚠️ This PR was opened by a bug in sdlc/cli ship — the validation gate swallowed a nonzero exit and pushed anyway. Tests are red on this branch:

tests/index.test.ts › @mistcash/sdk › test proofs › proving incorrect proof format
TypeError: Cannot convert undefined to a BigInt
  at prove_groth16 (src/gnark/index.ts:115:6)
Tests: 1 failed, 12 passed, 13 total

The failure is in prove_groth16 and looks unrelated to what this branch adds (CI workflow, CLAUDE.md, pnpm-workspace.yaml, lockfile) — likely pre-existing, surfaced now that the workspace file makes pnpm -r test actually reach core. Do not merge until it's triaged.

The ship gate itself has been fixed.

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