[bompus]

Summary

• Benchmark follow-up table extended with the six landed PRs: resolve: memoize jsDeclarationReachable's candidate-side walk #566 decl-scope memo, docs: bump heads to d9bd974c, add #562–#565 to follow-up merges #567 docs, explore: memoize getSizeStats; batch getContext node reads #568 explore warm-path (getSizeStats stamp-memo, getContext batching, shape cache), resolve: check js_decl_scope_memo before the file/tree loads #569 preamble reorder, resolve: memoize bare_call_receiver + generated-path scan #570 bare_call_receiver + generated-path memos, resolve: key local_declaration_scope memo by node id #571 n.id-keyed scope memo, resolve: apply_language_gate filters in place #572 in-place gate filter.
• Measured head bumped d9bd974c → 645abbb8 in the doc and README.
• pretix resolution settle: 4.73 → ~3.5 s (−26%, median of two CODEGRAPH_RESOLVE_PROFILE index runs, 3.2–3.7 s) across resolve: check js_decl_scope_memo before the file/tree loads #569–resolve: apply_language_gate filters in place #572, with exact graph parity (32,799 nodes / 73,404 edges).
• Residual paragraph rewritten: vendored-JS gate cost is now diffuse across many small gates rather than one dominant one.

Test plan

• Documentation only; figures come from profiled runs recorded in the table.

Summary by CodeRabbit

• Documentation
  • Updated the measured-results comparison to reflect the latest fork revision.
  • Added benchmark follow-up results covering resolver performance and allocation changes.
  • Revised the remaining-loss summary to report lower Pretix resolution-settle time and describe residual vendored JavaScript costs as spread across smaller gates.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: bompus/codegraph
• Branch: docs/perf-followup-572
• File: .coderabbit.yaml
• Commit: 73878c8
• Layers: Repository YAML (base), Organization UI (inherited)
• Review profile: CHILL
• Plan: Advanced
• Run ID: a4abf2a4-f914-49a3-9f36-31d045594c95

Commits

Reviewing files that changed from the base of the PR and between 645abbb and 73878c8.

Files selected for processing: 2

• README.md
• docs/benchmarks/fork-vs-upstream-node-bun-2026-10-27.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The README and benchmark document now identify fork revision 645abbb8. The benchmark document adds results for PRs #566–#572 and updates the remaining-loss measurements.

Changes: Benchmark reporting

Layer / File(s)	Summary
Update benchmark comparison and follow-up results docs/benchmarks/fork-vs-upstream-node-bun-2026-10-27.md, README.md	The benchmark document adds results for PRs #566–#572 and updates the remaining-loss summary. The README updates the compared fork revision.

Possibly related PRs

• bompus/codegraph#566: The benchmark update reports results for this PR’s JavaScript declaration-scope memoization.
• bompus/codegraph#570: The benchmark update reports results for this PR’s bare_call_receiver and generated-path memoization.

Priority: Low

Merge Risk: Minimal · up to 73878

No actionable merge-blocking risk remains.

Pre-merge checks: 6

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the documentation changes: recording follow-up merges #566–#572 and updating the measured head to 645abbb.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Suppressions Explained		The pull request changes only README.md and a benchmark Markdown document. The added lines update revision text, benchmark rows, and result descriptions. No added line introduces a lint, type-check, c…
User-Visible Changes Documented		The pull request changes only README.md and a benchmark documentation page. The diff updates benchmark text, revisions, and follow-up rows. It does not add, remove, or rename a CLI command or flag, MC…

Full details: Suppressions Explained

Explanation

The pull request changes only README.md and a benchmark Markdown document. The added lines update revision text, benchmark rows, and result descriptions. No added line introduces a lint, type-check, compiler suppression directive, or ignore configuration entry.

Full details: User-Visible Changes Documented

Explanation

The pull request changes only README.md and a benchmark documentation page. The diff updates benchmark text, revisions, and follow-up rows. It does not add, remove, or rename a CLI command or flag, MCP tool or argument, supported language or framework, agent target, or config key.

Finishing Touches

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

⚠️ This pull request shows signs of AI-generated slop (description_diff_mismatch). It has been flagged by CodeRabbit slop detection and should be reviewed carefully.

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[bompus]

@coderabbitai review

[coderabbitai]

✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.