Prep the repo for public visibility#78
Conversation
README rewritten for a public audience: CI badge, architecture diagram,
the taint-gate trace, and the measured injection/BEIR results up front;
the internal-codename framing and rename note are gone.
Internal references scrubbed from tracked files: the private audit/backlog
doc names in .gitignore (now in .git/info/exclude locally), the PHASE0.md
pointer in pnpm-workspace.yaml, plan/audit shorthand in code comments
("Phase 3", "audit M1"), the machine-specific node note in scripts/verify,
a stale follow-up note in db/crypto.ts, and the publish-planning voice in
the bench strategy/results docs (section 4 is now "Headline results and
caveats"). AGENTS.md is retitled to a generic dev-environment doc and
package.json drops "(internal codename)".
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview The README is rewritten: CI badge, mermaid architecture diagram, taint-gate trace, measured injection/BEIR tables up front, and links to
Internal shorthand is cleaned across comments and docs: Reviewed by Cursor Bugbot for commit 5a11e8d. Configure here. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughDocumentation and development guidance were refreshed across Orca. README and security narratives now describe the taint gate and measured results, setup instructions reflect pnpm, Turborepo, Node 22, and uv workflows, and benchmark documentation updates reporting guidance. ChangesDocumentation and platform narrative
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Warning Review ran into problems🔥 ProblemsGit: Failed to clone repository. Please run the Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@README.md`:
- Around line 99-105: Update the README setup instructions to distinguish
runtime and migration credentials: require DATABASE_URL to use a non-owner
NOBYPASSRLS role, and require ADMIN_DATABASE_URL for migrate.ts schema, RLS, and
serving-role setup. Clarify that the admin URL is reserved for migrations and
must not be used as the runtime database connection.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: fc6f0cb5-78a5-4b60-9f81-8ef957346b64
📒 Files selected for processing (12)
.gitignoreAGENTS.mdREADME.mdSECURITY.mdapps/api/app/v1/chat/route.tspackage.jsonpackages/bench/BENCHMARK-STRATEGY-2026.mdpackages/bench/RESULTS.mdpackages/core/src/guardrails/no-advice.recall.test.tspackages/db/src/crypto.tspnpm-workspace.yamlscripts/verify/README.md
💤 Files with no reviewable changes (1)
- .gitignore
Gets the repo ready to be flipped public.
README
Rewritten for a public audience: CI badge, a mermaid architecture diagram, the taint-gate trace, and the measured results (deterministic gate floor, AgentDojo 0/5 action-hijack ASR, live red-team 5/5, the fact-poison boundary stated honestly, BEIR vs published baselines) up front. The internal-codename framing and the rename note are gone.
Internal references removed
.gitignoreno longer names the private audit/backlog docs or local tooling config — those patterns moved to.git/info/exclude(per-clone; re-add on a fresh clone).PHASE0.mdpointer (pnpm-workspace.yaml), "(Phase 3)" (chat route), "(audit M1)" (no-advice eval), "(the current dev config)" (SECURITY.md), the machine-specific node note (scripts/verify), and a stale follow-up note (db/crypto.ts).packages/benchstrategy/results docs reworded from publish-planning voice to neutral methodology — §4 is now "Headline results and caveats"; the numbers and caveats are unchanged.package.jsondescription drops "(internal codename)".Verification
pnpm lint:ciclean, the edited test file passes 4/4,apps/apitypechecks, and a grep over tracked files confirms no remaining internal references. The mermaid diagram was render-checked.Summary by CodeRabbit