apollo_integration_tests: store the commitment infos delta in the dummy recorder

[itamar-starkware]

No description provided.

[cursor]

PR Summary

Low Risk
Changes are confined to apollo_integration_tests test utilities and harness wiring; no production sequencer or recorder code paths are modified.

Overview
The integration-test dummy cende recorder no longer always returns a fixed null commitment-infos height offset. It now parses incoming blob JSON, advances an internal offset from recent_state_commitment_infos block numbers (matching production’s “first height not yet stored” contract), and returns that offset on the GET endpoint so blob senders can avoid resending the full recent-blocks window.

A new RecorderStats type tracks blob receive counts and that offset; spawn_success_recorder_with_stats implements the behavior, with spawn_local_success_recorder returning Arc<RecorderStats> alongside the URL and join handle. FlowSequencerSetup exposes recorder_stats for flow tests; integration test manager ignores the shared recorder stats with _recorder_stats.

The write-blob handler accepts Json<serde_json::Value> for inspection, and the router uses DefaultBodyLimit::disable() so large cende blobs are accepted like before validation.

^{Reviewed by Cursor Bugbot for commit a565ff3. Bugbot is set up for automated code reviews on this repo. Configure here.}

[reviewable-StarkWare]

This change is 

[itamar-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• apollo_integration_tests: verify Patricia storage proofs of persisted witnesses in flow tests #14905
• apollo_integration_tests: assert witness-carrying cende blobs reach the recorder in flow tests #14918
• apollo_integration_tests: store the commitment infos delta in the dummy recorder #14935 👈 (View in Graphite)
• apollo_integration_tests: assert witnesses persisted for committed heights in flow tests #14903 : 1 other dependent PR (#14904 )
• central_systest_blobs: publish a decoded commitment-infos companion beside the blobs #14917
• central_systest_blobs: carry real witness fields in the regression blobs #14910
• central_systest_blobs: generalize the GCS helpers to fetch and upload arbitrary objects #14920
• workspace,ci: remove the os_input feature (make it unconditional) #14902 : 2 other dependent PRs (#14907 , #14909 )
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 3b33448. Configure here.}

[itamar-starkware]

@itamar-starkware made 1 comment.
Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on yoavGrs).

[yoavGrs]

@yoavGrs made 1 comment.
Reviewable status: 0 of 3 files reviewed, 1 unresolved discussion (waiting on itamar-starkware).

---

crates/apollo_integration_tests/src/utils.rs line 598 at r1 (raw file):

#[derive(Debug, Default)]
pub struct RecorderStats {
    pub num_blobs_received: AtomicUsize,

Please keep it simple.

I suggest storing the offset and a map from invalid block_number to reason.
Is there any validation that this does not cover?

[yoavGrs]

@yoavGrs made 1 comment.
Reviewable status: 0 of 3 files reviewed, 2 unresolved discussions (waiting on itamar-starkware).

---

crates/apollo_integration_tests/src/utils.rs line 619 at r2 (raw file):

                        // A blob is prepared before its own height's commitment completes, so
                        // early blobs legitimately carry no commitment infos.
                        if let Some(max_stored_height) = blob["recent_state_commitment_infos"]

I suggested validating the witnesses' heights in the blob against the stored offset.
You can report any issue in a dedicated field.