workspace,ci: remove the os_input feature (make it unconditional)

[itamar-starkware]

No description provided.

[cursor]

PR Summary

Medium Risk
Always-on OS-input storage and committer witness paths widen the default production surface (Patricia witness fetch, extra DB tables/files) compared to builds that previously omitted os_input; behavior should match the old feature-enabled builds but any latent #[cfg(feature = "os_input")] leftovers would fail to compile or gate metrics incorrectly.

Overview
Makes OS replay input (accessed keys, Patricia witnesses via ReadPathsAndCommitBlock, initial_reads, compressed state commitment infos, Cende blob fields) always on by deleting the workspace-wide os_input feature and the optional dependency wiring behind it.

Build & CI: Drops os_input from crate Cargo.toml files (batcher, committer, storage, blockifier, integration tests, etc.), promotes former optional deps to normal deps where needed, and removes .github/workflows/apollo_storage_os_input_ci.yml (no more separate --features os_input test matrix).

Runtime behavior (unchanged logic, single path): Batcher always persists accessed keys, trims initial reads, and routes commitment tasks to ReadPathsAndCommitBlock when keys exist; storage always exposes accessed_keys / state_commitment_infos tables and mmap files (MAX_DBS 27). Committer always uses witness-aware forest writes and exposes read_paths_and_commit_block. Batcher types always include GetStateCommitmentInfos and CentralObjects::initial_reads.

Code cleanup: Removes duplicate trait methods, #[cfg(feature = "os_input")] branches, and feature-gated test helpers; consolidates committer ComponentRequestHandler to one impl with ForestStorageWithWitnesses.

Observability: Grafana dev dashboard adds panels for get_state_commitment_infos and read_paths_and_commit_block request metrics.

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

[reviewable-StarkWare]

This change is 

[itamar-starkware]

• workspace,ci: remove the os_input feature (make it unconditional) #14902 : 2 dependent PRs (#14907 , #14909 ) 👈 (View in Graphite)
• main

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

[github-actions]

Artifacts upload workflows:

• Started at 2026-08-01T21:10:56Z
• Started at 2026-08-01T22:07:02Z
• Started at 2026-08-02T08:21:43Z
• Started at 2026-08-02T12:33:48Z
• Started at 2026-08-02T12:55:13Z
• Started at 2026-08-02T20:22:14Z

[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 d52c949. Configure here.}

[cursor]

Leftover os_input feature gates

Medium Severity

The os_input feature was removed from apollo_consensus_orchestrator, but #[cfg(feature = "os_input")] still guards the CENDE_LAST_STATE_COMMITMENT_INFOS_BLOCK_NUMBER import and the update after a successful blob write. That cfg never enables, so the metric stays unset even though witness data is now always produced.

Additional Locations (1)

• crates/apollo_consensus_orchestrator/src/cende/mod.rs#L51-L53

 

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