Skip to content

feat(recipes): high-crap-score with graph-estimated coverage#175

Merged
SutuSebastian merged 5 commits into
mainfrom
feat/high-crap-score
Jun 10, 2026
Merged

feat(recipes): high-crap-score with graph-estimated coverage#175
SutuSebastian merged 5 commits into
mainfrom
feat/high-crap-score

Conversation

@SutuSebastian

Copy link
Copy Markdown
Contributor

Summary

  • Add high-crap-score recipe: CRAP ranking with measured coverage when ingested, else graph-estimated 85/40/0% tiers from test-file reachability over dependencies + direct test references/calls.
  • Spike (scripts/spike-crap-reachability.sql + test) locks tier counts on fixtures/minimal (1/4/39 at 85/40/0%).
  • Golden high-crap-score (min_crap=15, labyrinth @ coverage_source: estimated); measured override via scripts/high-crap-score-measured.test.mjs.
  • Cross-link high-complexity-untested.md; agent rule/skill coverage columns; maintainer docs contract.

Test plan

  • bun run test:golden
  • bun test scripts/spike-crap-reachability.test.mjs scripts/high-crap-score-measured.test.mjs
  • bun test scripts/query-golden-coverage-matrix.test.mjs
  • bun src/index.ts query --recipe high-crap-score --json --params min_crap=15 --root fixtures/minimal

Spike locks 85/40/0% reachability tiers on fixtures/minimal; ships
high-crap-score recipe (measured override when coverage ingested),
golden + script tests, and high-complexity-untested cross-link.
Add coverage_source one-liners to served rule/skill, golden-queries and
architecture contracts, roadmap checkbox, wave slice 2.4, and recipe
precedence note (measured 0% beats graph tiers).
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@SutuSebastian, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 24 minutes and 23 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 @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

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 configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 764ba819-4bb1-4bc7-93c0-e2bdd5dee4c8

📥 Commits

Reviewing files that changed from the base of the PR and between a11242e and 1e80ee7.

📒 Files selected for processing (33)
  • .agents/skills/harden-pr/SKILL.md
  • .changeset/high-crap-score.md
  • docs/architecture.md
  • docs/glossary.md
  • docs/golden-queries.md
  • docs/plans/agent-enrichment-wave.md
  • docs/plans/apply-write-safety.md
  • docs/plans/evidence-chains-on-recipe-rows.md
  • docs/plans/graph-estimated-crap.md
  • docs/roadmap.md
  • fixtures/CAPABILITIES.json
  • fixtures/golden/minimal/files-count.json
  • fixtures/golden/minimal/files-hashes.json
  • fixtures/golden/minimal/files-largest.json
  • fixtures/golden/minimal/high-crap-score.json
  • fixtures/golden/minimal/index-summary.json
  • fixtures/golden/minimal/index-table-stats.json
  • fixtures/golden/minimal/source-fts-row-count.json
  • fixtures/golden/minimal/unimported-exports.json
  • fixtures/golden/scenarios.json
  • fixtures/minimal/README.md
  • fixtures/minimal/src/lib/alias-blind-spot.ts
  • scripts/high-crap-score-measured.test.mjs
  • scripts/spike-crap-reachability.sql
  • scripts/spike-crap-reachability.test.mjs
  • templates/agent-content/mcp-instructions.md
  • templates/agent-content/rule/00-full.md
  • templates/agent-content/skill/10-recipes-context.md
  • templates/recipes/high-complexity-untested.md
  • templates/recipes/high-crap-score.md
  • templates/recipes/high-crap-score.sql
  • templates/recipes/unimported-exports.md
  • templates/recipes/unimported-exports.sql
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/high-crap-score

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@changeset-bot

changeset-bot Bot commented Jun 10, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 1e80ee7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@stainless-code/codemap Patch

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

Delete evidence-chains and graph-estimated-crap plan files; lift
contract to golden-queries/roadmap; slim wave doc to plans 3–4;
encode retire-on-merge in wave conventions and harden-pr skill.
unimported-exports third reason + evidence; alias-blind-spot fixture;
MCP/glossary/rule parity for CRAP and evidence; no-defer convention in
wave + harden-pr.


Tighten unresolved_import_blind_spot to name-only matching, extend changeset/CAPABILITIES/fixture README/agent skill, add labyrinth measured-vs-estimated test, and refresh README goldens.
@SutuSebastian SutuSebastian merged commit af86d10 into main Jun 10, 2026
11 checks passed
@SutuSebastian SutuSebastian deleted the feat/high-crap-score branch June 10, 2026 09:50
@github-actions github-actions Bot mentioned this pull request Jun 10, 2026
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.

1 participant