Skip to content

Document the hub-authored freeze against the sidecar probe gate - #527

Closed
TheGreatAxios wants to merge 2 commits into
mainfrom
cl-7273-workflow-freeze-drift
Closed

TheGreatAxios wants to merge 2 commits into
mainfrom
cl-7273-workflow-freeze-drift

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

CL-7273.

Analysis lane. The ticket's premise — that @corbits/workflow-freeze reimplements the probe gate's freeze and will drift — does not survive reading the imports.

Every step composes a native primitive: projectLiveToInert (@intx/workflow), walkCapabilities (@intx/workflow-deploy), createDefaultDirectorRegistry (@intx/agent), computeWireDefinitionHash (@intx/types), createDbFrozenApprovalWriter (@intx/hub-sessions).

The two that matter are shared with the probe path by identity: workflow-probe-gate.ts:454 calls the same createDbFrozenApprovalWriter, and line 40 imports the same computeWireDefinitionHash it recomputes with at line 233. The hash preimage and the all-or-nothing stamp are one implementation.

What differs is the input (already-inert hub-authored JSON, so no probe round-trip exists) and the approval policy (self-approve, which the probe gate's own comments document as the analogue for live-authored definitions).

The drift test the ticket asked for already exists in substance — index.test.ts pins frozen.wireHash to the native function over the inert projection. A cross-path test would assert a function equals itself.

Recommends closing CL-7273 as 'not a reimplementation'. This package is closer to packages/approvals (cited in CL-7257 as the correct pattern) than to the mintRepoGrant drift.

Docs only. No code change. Draft — not for merge.

@TheGreatAxios
TheGreatAxios force-pushed the cl-7273-workflow-freeze-drift branch from e671c64 to e30d7a5 Compare August 31, 2026 03:12
@TheGreatAxios
TheGreatAxios marked this pull request as ready for review August 31, 2026 03:12
@TheGreatAxios
TheGreatAxios force-pushed the cl-7273-workflow-freeze-drift branch from e30d7a5 to 8476241 Compare August 31, 2026 03:18
@TheGreatAxios

Copy link
Copy Markdown
Contributor Author

Consolidated into #531 — four independent docs files merging as one CI cycle rather than four serialized ones under strict branch protection. Content unchanged.

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.

1 participant