Skip to content

feat: prepare id_hash_debug storage (refs #3098) - #3105

Merged
mblua merged 1 commit into
mainfrom
feature/3098-id-hash-debug-slots
Oct 18, 2026
Merged

mblua merged 1 commit into
mainfrom
feature/3098-id-hash-debug-slots

Conversation

@mblua

@mblua mblua commented Oct 18, 2026

Copy link
Copy Markdown
Owner

Prepare optional id_hash_debug storage in AgentConfig and ProfileCellConfig. Both fields remain serde-skipped and every explicit constructor initializes None, so this prerequisite preserves serialized settings, existing hashes and runtime behavior.

Validation on 5b07f07:

  • Locked Cargo all-target check and read-only format check passed.
  • All 9 existing layering tests passed; none failed, ignored or filtered out.
  • Clean actual-base PRE and tested-head POST have identical complete cyclic SCC member sets/counts, 1211 unique module arcs with zero additions/removals, and byte-identical 96449-byte arc records matching the committed record. Existing arc/cycle guards passed.
  • Independent code and proof review: CODE_AND_PROOF_REVIEW_PASS, no findings. Report SHA256 3A5102584BED1BB2313F57BD177B04AC97F944D8454E570E74F981AEE2A5FDE5. Lead verified all 56 raw evidence files.

Plan: E3097 P01-v2, SHA256 ACAE64D6AE85CF2D626873246234B3737FE3BBD6431FD8CD3847352EE73543AE; parent plan-v3 SHA256 B86F5DC6F3813AF551FD1055953829DC0493936AD82EAE1BAF05DA8D1520351E. Full exact-set consensus and standing authorization recorded under .ac/plans/3097-id-hash-debug; local artifacts under room-19-ac-dev-team-v5/room-shared/e2432-id-hash/id-hash-debug/execution/P01. The user-authorized temporary base visit returned to this unchanged clean feature HEAD. Fresh main remains actual phase base 970c907; no incoming delta or branch update.

This phase prepares inert fields only. Later E3097 phases enable the schema, derivation/persistence and frontend transport.

Closes #3098

@sonarqubecloud

Copy link
Copy Markdown

@mblua

mblua commented Oct 18, 2026

Copy link
Copy Markdown
Owner Author

Step 9.5 incoming-main triage: UNRELATED_NO_UPDATE for P01.

Actual phase base: 970c907. Fetched main: 1892e47 (five incoming commits from PR #3103 / issue #3093). Tested PR head remains 5b07f07, tree ba71292dcc887eb63e4519e7014398b7a1e48e14.

Incoming paths:

  • src/shared/console-capture.ts
  • src/shared/ipc.transport.test.ts
  • src/shared/ipc.ts
  • src/sidebar/App.tsx
  • src/sidebar/components/WorkgroupGroupRail.test.tsx
  • src/sidebar/components/WorkgroupGroupRail.tsx
  • src/sidebar/rail-diagnostics.test.ts (added)
  • src/sidebar/rail-diagnostics.ts (added)
  • src/sidebar/stores/project.test.ts
  • src/sidebar/stores/project.ts

Independent Rust Grinch review found no overlap with P01's 19 Rust files and no change to Rust build inputs, manifests, locks, scripts, CI or structural baselines. P01's serde-skipped None fields have no wire effect. The incoming diagnostic invoke already exists at the phase base; the additive TypeScript diagnostics do not touch the settings decoders or hash fields. Merge-tree inspection found no conflict. The existing exact-head local checks and current CI remain applicable to this unchanged P01 head; CI must still finish successfully before merge.

Review artifact: room-19-ac-dev-team-v5/room-shared/e2432-id-hash/id-hash-debug/execution/P01/rust-main-delta-1892-review-v1.md, SHA256 3CF0978D2A6A883443A90ECD42A13435D7C0F3702042C0887C79195529B8B13A. Per the agreed unrelated-delta landing rule, keep this branch/head and reuse its green CI when available; strict up-to-date protection does not require a branch update for this unrelated delta under the approved admin-merge path.

This classification covers P01 only. P04 must refresh its IPC source/test baseline against actual main when that phase starts. P04 has not started.

@mblua
mblua merged commit 01456f1 into main Oct 18, 2026
20 checks passed
@mblua
mblua deleted the feature/3098-id-hash-debug-slots branch October 18, 2026 04:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: I3097 P01 inert Rust debug slots

1 participant