feat(recipes): evidence chains on high-judgment recipe rows#174
Conversation
Add reason and evidence_json to boundary-violations SQL; golden + golden-queries contract; agent-enrichment-wave tracer doc for plans 1-4.
reason has_callers/no_callers plus bounded evidence_json from calls; golden updated; document name-only callee match caveat.
Rows get reason (no_direct_import | reexport_chain_possible) and evidence_json barrel hops from re_export_chains when a false positive may be explained by barrel re-exports.
Completes Plan 1 evidence chains on recipe rows for PR #B.
Filter deprecated-symbols callers to AST provenance; append truncated marker when evidence subqueries cap at 3; surface re_export_chains.truncated on unimported-exports hops; align recipe docs and changeset for PR #B.
🦋 Changeset detectedLatest commit: 76b5d51 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Warning Review limit reached
More reviews will be available in 49 minutes and 31 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (24)
📝 WalkthroughWalkthroughThis PR extends three high-judgment recipes with ChangesEvidence chains on recipe rows
🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Use json('true') for list-cap markers; exercise >3 caller branch via
ProductCard now() fixture; fix stale unimported-exports action text;
add architecture + skill evidence lines; refresh affected goldens.
1 participant
Summary
reasonandevidence_jsoncolumns toboundary-violations,deprecated-symbols, andunimported-exportsso agents can cite detection path beforeapplyor manual edits (Plan 1 / PR #B).boundary-violations: stablereason=boundary_deny_match+ rule tuple in evidence.deprecated-symbols:has_callers/no_callers+ up to three AST caller hops (E.3 truncation marker when capped).unimported-exports:no_direct_import/reexport_chain_possible+ barrel hops fromre_export_chains(includes chaintruncatedflag).docs/golden-queries.mdcontract; patch changeset.Test plan
bun run test:goldenbun test scripts/query-golden-coverage-matrix.test.mjsbun src/index.ts query --recipe boundary-violations --json --root fixtures/minimalbun src/index.ts query --recipe deprecated-symbols --json --root fixtures/minimalbun src/index.ts query --recipe unimported-exports --json --root fixtures/minimalSummary by CodeRabbit
New Features
Documentation