Skip to content

FAQ

No. This is the Astro documentation companion. Application code lives in jajera/pr-readiness-coach.

Does CI block merges when the coach says NOT READY?

Section titled “Does CI block merges when the coach says NOT READY?”

No. PR analysis and Kiro hooks are warn-only — they always exit 0 and coach via comments/reports. Hard merge gates are out of scope for v1.

Machine clients (CLI / GitHub Actions) use an API key on POST /analyze. The browser SPA uses Cognito JWTs on /runs and /ui/analyze. Never bake the API key into Vite.

Why zip-deploy Amplify instead of a Git connection?

Section titled “Why zip-deploy Amplify instead of a Git connection?”

Avoids a GitHub token on Amplify and keeps SPA failures in a separate job after CDK. Vite env is bake-time, so the zip job must run after stack outputs exist.

Yes — fixtures/demo.sh and --local --path exercise heuristics only. Requires Node.js 24+.

Defaults target ap-southeast-2 with Nova Lite + Claude Haiku 4.5 AU inference profile. Model availability is regional — smoke Converse in the same region as deploy.