fix: repair CI failures (version check + E2E setup)#7
Conversation
- ci.yml: add file guard to version check loop so it no-ops on a single-package repo (was failing with jq exit 2 on missing packages/) - ci.yml: replace npm-pack E2E setup step with a direct bun build of the public API (src/index.ts -> dist/index.js CJS) - hook-runner.ts: run binary from repo source via bun instead of the planned native binary distribution; point dist path at repo root - src/index.ts: add public API entry point used by the bun build step Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (5)
📝 WalkthroughWalkthroughThe pull request refactors the E2E testing infrastructure from a packaging-and-installation model using Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
Summary
packages/*/package.jsondoesn't exist in this single-package repo. Added a[ -f "$pkg" ] || continueguard so the loop is a no-op when no subpackages are present.bun run test:npxreferenced a script that didn't exist. Thetest:npxapproach (npm pack → install into.test-npx/→ run against@failproofai/<platform>native binary) has been removed since we're not shipping platform binaries. Replaced with a directbun buildof the public API entry point (src/index.ts → dist/index.js), and E2E tests now invoke the binary from the repo source viabun.Test plan
qualityjob passes (version check no longer fails on missingpackages/)test-e2ejob passes (Build E2E fixturesstep +E2E Hook Testsboth succeed)bun build src/index.ts --outdir dist --target node --format cjs && bun run test:e2epasses locally🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Documentation
Chores