Skip to content

test(bench): execute SNB Interactive IS1 through live GraphForge - #1084

Merged
DecisionNerd merged 10 commits into
mainfrom
cursor/962-live-snb-interactive-5258
Sep 5, 2026
Merged

test(bench): execute SNB Interactive IS1 through live GraphForge#1084
DecisionNerd merged 10 commits into
mainfrom
cursor/962-live-snb-interactive-5258

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

The SNB Interactive adapter previously validated replayed files without proving public GraphForge execution. Add a bounded live IS1 lane whose Rust runner loads the pinned synthetic fixture through the public facade, warms and executes the parameterized query, and validates its Arrow results against the declared reference.

Live evidence is runner-owned and binds the fixture, job, reference, acquisition, identity, parameters, and execution phases. Preserve column boundaries, empty strings, and whitespace during comparison; an adversarial regression modifies a real engine result to prove merged or shifted fields cannot falsely pass.

IS1 is the live proof. IS2–IS7 and IC1–IC13 remain explicitly labeled static/reference replay coverage. IC14 and IU1–IU8 retain typed unsupported outcomes. The fixture is synthetic engineering data, not official Datagen scale-factor output, and evidence explicitly denies audited certification. This PR does not claim full-suite live execution.

Validation on the integrated tree:

  • make -C benchmarks smoke: passed, including 381 Python tests, all Rust runner tests (14 SNB Interactive tests), benchmark workspace Clippy, formatting, and the Cargo dependency-boundary check.
  • make pre-push-fast and final formatting/diff checks: passed.
  • Live facade execution, reference comparison, field-boundary negatives, asset mutation rejection, and evidence/CLI authority checks passed.
  • Current dependency versions preserved; the lockfile adds only the runner's declared Arrow, facade, and SHA-2 edges.

Closes #962. #952 remains the broader M11 completion gate. Exact-head CI run 33946967997 passed at b5a6f64303c90d059f5fa383a458765db7b00aae, with CI Gate successful, CLEAN merge state, and no unresolved review threads. Squash merged as 88fb033726042cb9b5fec9ce0b5259f3dd73e12f; issue #962 is closed.

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 5ee2f814-298c-4f13-a001-db116bdbf4c1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Warning

Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption.


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.

@github-actions github-actions Bot added documentation Improvements or additions to documentation release:none No release note or version impact labels Sep 2, 2026
cursoragent and others added 10 commits September 5, 2026 05:11
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Caller-provided rows can no longer reach a public live-named API. Trusted
evidence now asserts the complete identity, parameter, and checksum bind.

Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
Co-authored-by: David Spencer <DecisionNerd@users.noreply.github.com>
@DecisionNerd
DecisionNerd force-pushed the cursor/962-live-snb-interactive-5258 branch from 9256492 to b5a6f64 Compare September 5, 2026 05:20
@DecisionNerd
DecisionNerd marked this pull request as ready for review September 5, 2026 05:21
@DecisionNerd
DecisionNerd merged commit 88fb033 into main Sep 5, 2026
21 checks passed
@DecisionNerd
DecisionNerd deleted the cursor/962-live-snb-interactive-5258 branch September 5, 2026 05:33
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 release:none No release note or version impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test(bench): add a separate GDC SNB Interactive suite

3 participants