fix(recovery): restore 0pt source and isolate integration writes #539

Merged
admin merged 2 commits from codex/operator-source-recovery-20261110 into main 2026-11-10 02:55:24 +00:00
Owner

The production-documentation merge 585bdd4 left the default branch with only two documents and removed the application, contracts, tests and workflows. This restores exactly the 957 paths present at fb072d47 but missing from that branch. Every restored blob and file mode was independently checked against the baseline. Both current production documents remain byte-identical; no surviving default-branch content is overwritten.

The first commit is the exact source recovery. A separate test-safety commit removes an inherited production ST0RE write path from the integration suite: no opt-in means no HTTP request; an explicit opt-in requires an explicit loopback test origin. The real HTTP client remains in use when enabled, while cross-origin requests and redirect following are blocked. All functional assertions remain. No production application code, scanner policy, branch protection or credential is changed by this safety commit.

Validation: frozen dependency install; typecheck; Next build; text-contrast check; full Vitest suite 1434 passed / 197 explicitly skipped. Forty-one safety-guard cases pass, including zero requests by default and denial before dispatch for production targets. An ephemeral loopback HTTP503 server observed exactly one GET and zero mutations. The credential scanner reported zero blocking findings (89 informational, no allowlist). Final synthetic credential-URI fixture strings are assembled at runtime with identical test values because the native pre-receive scanner correctly rejected their literal spelling; no scanner exception was added.

The canonical contracts/openapi.yaml has SHA-256 1b0bbca6f020dab35c052ae7e5036c396fd7bd8546a81bc355a3fb39d23c01a2 (31532 bytes), matching the existing SIAX parity snapshot. Native CI and protected review must finish before merge. This PR makes no production acceptance claim. Existing baseline whitespace is intentionally preserved by the exact recovery; the separate safety diff passes git diff --check. The primary checkout and its unique branch/untracked work were left untouched.

The production-documentation merge 585bdd4 left the default branch with only two documents and removed the application, contracts, tests and workflows. This restores exactly the 957 paths present at fb072d47 but missing from that branch. Every restored blob and file mode was independently checked against the baseline. Both current production documents remain byte-identical; no surviving default-branch content is overwritten. The first commit is the exact source recovery. A separate test-safety commit removes an inherited production ST0RE write path from the integration suite: no opt-in means no HTTP request; an explicit opt-in requires an explicit loopback test origin. The real HTTP client remains in use when enabled, while cross-origin requests and redirect following are blocked. All functional assertions remain. No production application code, scanner policy, branch protection or credential is changed by this safety commit. Validation: frozen dependency install; typecheck; Next build; text-contrast check; full Vitest suite 1434 passed / 197 explicitly skipped. Forty-one safety-guard cases pass, including zero requests by default and denial before dispatch for production targets. An ephemeral loopback HTTP503 server observed exactly one GET and zero mutations. The credential scanner reported zero blocking findings (89 informational, no allowlist). Final synthetic credential-URI fixture strings are assembled at runtime with identical test values because the native pre-receive scanner correctly rejected their literal spelling; no scanner exception was added. The canonical contracts/openapi.yaml has SHA-256 1b0bbca6f020dab35c052ae7e5036c396fd7bd8546a81bc355a3fb39d23c01a2 (31532 bytes), matching the existing SIAX parity snapshot. Native CI and protected review must finish before merge. This PR makes no production acceptance claim. Existing baseline whitespace is intentionally preserved by the exact recovery; the separate safety diff passes git diff --check. The primary checkout and its unique branch/untracked work were left untouched.
admin added 2 commits 2026-11-10 02:50:40 +00:00
test(safety): require an isolated opt-in target for ST0RE integration
masterplan-lock / check (push) Successful in 6s
CI (SIAX Cloud) / sonar (pull_request) Skipped
masterplan-lock / check (pull_request) Successful in 4s
CI (SIAX Cloud) / Component conformance (blocking) (pull_request) Successful in 9s
CI (SIAX Cloud) / security (pull_request) Successful in 16s
Security Scan (SIAX Cloud) / security (pull_request) Successful in 17s
CI (SIAX Cloud) / quality (pull_request) Successful in 1m13s
CI (SIAX Cloud) / integration (pull_request) Successful in 1m15s
0cac0266e0
siax-bot approved these changes 2026-11-10 02:55:07 +00:00
siax-bot left a comment
Collaborator

Independent source review completed by a separate Codex agent on exact head 0cac0266e0. Independently verified all 957 restored paths against the prior complete canonical tree, including blob IDs and file modes, and preserved every surviving production-document blob. Contract hash is unchanged. Reviewed the separate test-origin guard: explicit opt-in plus an exact loopback origin, no default HTTP, redirects rejected, and writes confined to that origin. Independently reran all41 guard tests; all passed. This is a service-account review record of actual code review, not human attestation. Native CI passed on this head; production credential injection and full SIAX acceptance are not implied.

Independent source review completed by a separate Codex agent on exact head 0cac0266e01d67a4db8ee566e250428675f67986. Independently verified all 957 restored paths against the prior complete canonical tree, including blob IDs and file modes, and preserved every surviving production-document blob. Contract hash is unchanged. Reviewed the separate test-origin guard: explicit opt-in plus an exact loopback origin, no default HTTP, redirects rejected, and writes confined to that origin. Independently reran all41 guard tests; all passed. This is a service-account review record of actual code review, not human attestation. Native CI passed on this head; production credential injection and full SIAX acceptance are not implied.
admin merged commit a6c605ea7f into main 2026-11-10 02:55:24 +00:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/0pt#539