Skip to content

docs(benchmarks): known-answer import scoping — c-CRAB/CR-Bench TS/JS premise falsified - #307

Merged
norvalbv merged 2 commits into
mainfrom
docs/known-answer-import-scoping
Aug 2, 2026
Merged

docs(benchmarks): known-answer import scoping — c-CRAB/CR-Bench TS/JS premise falsified#307
norvalbv merged 2 commits into
mainfrom
docs/known-answer-import-scoping

Conversation

@norvalbv

@norvalbv norvalbv commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Finding

The queued absolute-recall plan — "c-CRAB (arXiv 2603.23448) + CR-Bench (arXiv 2603.11078), filtered to TS/JS" — was scoped against the actual papers and released artifacts and is not buildable as specified:

  • Both are Python-only. c-CRAB builds on SWE-CARE (Python projects benchmark; its 184 instances / 234 test oracles all come from Python repos — python-telegram-bot, posthog, pytorch-lightning in the worked examples). CR-Bench transforms SWE-Bench (12 Python repos). There is no TS/JS slice in either to filter.
  • c-CRAB's artifact (github.com/c-CRAB-Benchmark/dataset) is the curation pipeline + results, with no license.
  • CR-Bench has released no artifact at all (checked GitHub; nothing public).

Runbook change

Item 2 of the measurement-rules section now records the scoping (dated 2026-08-02) and proposes two unratified replacements for the same goal:

  1. GHSA/npm advisory mining with fix commits — public known-answer facts, re-expressed as anonymized fixtures like all other rows; natural source for the api-security / frontend-security suites. SecBench.js catalogs ~600 such JS vulns but is itself unlicensed — usable as an index, not a source.
  2. CR-Bench's transformation recipe (Alg. 1) applied to SWE-Bench Multimodal's JS/TS repos — correctness-suite known-answer rows; a build project.

Absolute-recall numbers stay blocked until one is ratified and built. This is the second handover premise falsified by verification this cycle (after the rebutted-drop distribution in #306) — both now recorded where the next session will read them.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated the absolute-recall benchmarking plan to remove unsuitable dataset import options.
    • Documented two alternative approaches for JavaScript and TypeScript benchmark coverage.
    • Clarified that absolute-recall measurement remains pending approval and implementation of an alternative.

@coderabbitai

coderabbitai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@norvalbv, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 17 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

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.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: ab220886-972f-4904-b088-4e43c1b6df4f

📥 Commits

Reviewing files that changed from the base of the PR and between 920f574 and 36d202d.

📒 Files selected for processing (1)
  • docs/benchmarks/corpus-growth.md
📝 Walkthrough

Walkthrough

The benchmark documentation removes planned c-CRAB and CR-Bench TS/JS imports. It records GHSA/npm advisory mining and SWE-Bench Multimodal transformation as proposed alternatives. Absolute-recall measurement remains pending approval and implementation.

Changes

Benchmark corpus evaluation

Layer / File(s) Summary
Corpus assessment and alternatives
docs/benchmarks/corpus-growth.md
The document identifies c-CRAB and CR-Bench TS/JS imports as unsuitable. It records two unratified alternatives and keeps absolute-recall figures unavailable pending implementation.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • norvalbv/devkit#275: Both PRs update benchmark corpus evaluation and minimum-corpus documentation.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the documented scope change and the rejected c-CRAB/CR-Bench TS/JS premise.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/known-answer-import-scoping

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 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 `@docs/benchmarks/corpus-growth.md`:
- Around line 95-96: Update item 5 in the Pending work list to remove the
rejected c-CRAB/CR-Bench import request and instead require ratification and
construction of one proposed alternative, keeping the runbook consistent with
the section’s viability decision.
- Around line 88-89: Update the CR-Bench statement in the artifact comparison
paragraph to say it “has no publicly released artifact” instead of “has
published no artifact at all,” preserving the intended scoped claim without
changing the surrounding text.
🪄 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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: e98c059d-14ed-422c-9453-d2af8039704a

📥 Commits

Reviewing files that changed from the base of the PR and between 9db5075 and 920f574.

📒 Files selected for processing (1)
  • docs/benchmarks/corpus-growth.md

Comment thread docs/benchmarks/corpus-growth.md Outdated
Comment thread docs/benchmarks/corpus-growth.md
norvalbv added a commit that referenced this pull request Aug 2, 2026
… fix pending-list conflicts

Review round on #307: 'published no artifact at all' overclaimed (verified
absence of a PUBLIC artifact only) — now 'no publicly released artifact'.
Pending-work item 5 still requested the rejected c-CRAB/CR-Bench import,
contradicting measurement-rule 2's viability finding — now points at ratifying
one of the proposed replacements. Also marked pending item 2 (κ + noise floor)
DONE per #304, same staleness class.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
norvalbv and others added 2 commits August 2, 2026 13:38
… premise falsified

Scoped the queued absolute-recall import against the actual artifacts:
c-CRAB builds on SWE-CARE and CR-Bench transforms SWE-Bench — both Python-only,
so the planned "filtered to TS/JS" slice does not exist in either. c-CRAB's
dataset repo additionally has no license; CR-Bench has released no artifact.

Runbook item 2 now records the finding and proposes (unratified) replacements:
GHSA/npm-advisory mining with fix commits for the security suites, and applying
CR-Bench's transformation recipe to SWE-Bench Multimodal's JS/TS repos for
correctness. Absolute recall stays blocked until one is ratified.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… fix pending-list conflicts

Review round on #307: 'published no artifact at all' overclaimed (verified
absence of a PUBLIC artifact only) — now 'no publicly released artifact'.
Pending-work item 5 still requested the rejected c-CRAB/CR-Bench import,
contradicting measurement-rule 2's viability finding — now points at ratifying
one of the proposed replacements. Also marked pending item 2 (κ + noise floor)
DONE per #304, same staleness class.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@norvalbv
norvalbv force-pushed the docs/known-answer-import-scoping branch from c58d59a to 36d202d Compare August 2, 2026 12:53
@norvalbv
norvalbv merged commit 570097d into main Aug 2, 2026
1 of 2 checks passed
norvalbv added a commit that referenced this pull request Aug 2, 2026
…clone-gate ruling (#310)

benchmarks-grow-from-telemetry gains its convergence record before the release
that ships it: capture loop closed end-to-end (#295/#302/#303/#309, first 8
pure-telemetry rows, corpus 128), label-trust precondition met (#304: κ 0.735
post-triage, 4.2% noise floor; cleanlab floor still pending bench pred_probs),
and the Target's c-CRAB/CR-Bench known-answer path recorded as falsified
(#307) with the replacement candidates awaiting ratification.

New axis clone-gate-non-import-code ([VALIDATED]): clones are measured over
non-import code, excluded at the jscpd tokenizer — with the six-hole failure
of post-hoc fragment classification recorded as the rejected road so a future
simplifier can't silently re-vacuous the gate (#305/#308).

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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