Skip to content

test(performance): reconcile M4 exit evidence and scale claims (#345) - #496

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/345-m4-exit-evidence-c6a4
Aug 10, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
cursor/345-m4-exit-evidence-c6a4

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Aug 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Final M4 exit evidence reconciliation on tree 2b1bfda after #342–#344. No new optimizations. Documents honest scale dispositions and refreshes the #334 evidence emitter + oversize file-backed proof.

Closes #345.

Changes

  • Emit docs/development/m4-exit-evidence.json (graphforge-m4-entry-evidence/1) on the final SHA
  • Add docs/development/m4-exit-evidence.md ledger (child PRs, dispositions, reproduction)
  • Refresh file-backed-oversize-evidence.json on the final SHA
  • Update m4-entry-matrix.json known limitations (CSR/kernels shipped; scale-host blockers; GPU OOS)
  • Soft-correct scale-limits.md, entry baseline docs, and benchmark methodology

Honest dispositions

  • 8M/128M densified public facade: optional scale-host measurement — not a universal product claim (agent hosts keep sparse >2 GiB oversize proof)
  • >200M adjacency RSS: accepted pending scale-host evidence; 134M Arrow boundary no longer governs the public build path
  • GPU: out of scope for M4
  • threads-8: policy-supported; unavailable on hosts under the machine concurrency budget

Test plan

Open in Web Open in Cursor 

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Note

Reconcile M4 exit evidence and scale claims across docs and test contracts

Macroscope summarized d915820.

Summary by CodeRabbit

  • Documentation
    • Updated runtime configuration guidance for 1-worker and 8-worker setups.
    • Clarified executable support and when host-relative configurations are unavailable.
    • Documented current limitations around fresh CSR indexes, private compute resources, optional large-scale reruns, and GPU support.
    • Removed outdated statements regarding CSR expansion and serial algorithm execution.

Freeze final-tree #334 evidence at 2b1bfda, refresh oversize file-backed
proof, and correct stale contract/docs claims for shipped CSR/kernel work
plus honest 8M/128M and >200M dispositions. Closes #345.

Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 10, 2026 •

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 36e586cb-6c50-44de-af0b-346393e38992

📥 Commits

Reviewing files that changed from the base of the PR and between 2b1bfda and d915820.

⛔ Files ignored due to path filters (6)
  • benchmarks/m4_entry_baseline.md is excluded by !**/*.md
  • docs/development/file-backed-oversize-evidence.json is excluded by !**/docs/**
  • docs/development/m4-entry-baseline.md is excluded by !**/*.md, !**/docs/**
  • docs/development/m4-exit-evidence.json is excluded by !**/docs/**
  • docs/development/m4-exit-evidence.md is excluded by !**/*.md, !**/docs/**
  • docs/reference/scale-limits.md is excluded by !**/*.md, !**/docs/**
📒 Files selected for processing (1)
  • tests/contracts/m4-entry-matrix.json

Walkthrough

The M4 entry matrix now documents worker configuration availability and current evidence limitations. It replaces outdated CSR and serial-kernel statements with notes on CSR index misses, compute pools, qualified scale evidence, and GPU scope.

Changes

M4 runtime contract

Layer / File(s) Summary
Runtime notes and known limitations
tests/contracts/m4-entry-matrix.json
The matrix clarifies executable worker entries, host-relative unavailability, current CSR and compute-pool behavior, qualified large-scale evidence, and GPU exclusion.

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

Possibly related issues

Possibly related PRs

Suggested labels: ci-cd, tooling

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cursor/345-m4-exit-evidence-c6a4

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

@github-actions github-actions Bot added testing Test coverage and testing infrastructure documentation Improvements or additions to documentation labels Aug 10, 2026
@DecisionNerd
DecisionNerd marked this pull request as ready for review August 10, 2026 03:43
@cursor
cursor Bot merged commit fbafce8 into main Aug 10, 2026
20 of 21 checks passed
@cursor
cursor Bot deleted the cursor/345-m4-exit-evidence-c6a4 branch August 10, 2026 03:44
This was referenced Aug 10, 2026
This was referenced Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation testing Test coverage and testing infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(performance): reconcile final M4 exit evidence and scale claims

2 participants