Skip to content

Repair SI Windows byte preservation and source-index portability - #54

Draft
infotradescout wants to merge 5 commits into
codex/si-harness-runtimefrom
codex/si-windows-byte-preservation-20260916
Draft

infotradescout wants to merge 5 commits into
codex/si-harness-runtimefrom
codex/si-windows-byte-preservation-20260916

Conversation

@infotradescout

@infotradescout infotradescout commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

Installed repair and restart milestone completed — September 17, 2026

SI source head remains 46629c3880151e5bcef090a5a85bbb7f8eeeaff4. Draft. The installed controller used here remains the separately versioned aa787adfb696e493e2656fa853041eb294599ec8; no global replacement or main merge occurred.

The previous native repair/cleanup stopping point is now superseded. The normal installed engine completed the original Infinity repair, and a separate OS process recovered that genuine completion. This is scoped installed-controller evidence, not universal client adoption or an installed upgrade to the current SI source.

New installed execution evidence

The controller resumed the existing session and repair lineage. The first packet request correctly refused to export required context because the native runner contained an embedded synthetic signing key. Its original failure, context diagnostic and usage count were retained; the budget and secret-screening rules were unchanged.

The task controller removed that static fixture credential and used a fresh random in-memory key shared only with the test subprocesses. This limited test-fixture maintenance was performed outside the SI artifact API and is not claimed as a governed production-source write. Source and acceptance-runner preimages/hashes were preserved. AST comparison later proved all nine scenario calls/assertions unchanged from the original failure and equivalent to the formatted source fixture.

With complete safe context, the installed SI engine issued its normal repair packet, applied the bound installed-target configuration, executed the original command, and propagated successful verification to the original task. No status or completion evidence was manually edited.

Session:       si-6197351c45264174a75485af9e7fa264
Checkpoint:    cp-8e4cfc136369
Original task: task-fc16bf631c99 — complete
Repair task:   task-b6917df3394c — complete
Success:       command-06baabafb347
Old failure:   command-bd995ef3f8e2 — retained unchanged
Command:       node --test --test-reporter=tap infinity-installed.native.test.mjs
State:         VERIFIED_COMPLETE

All nine native scenario subtests passed against installed Infinity 1a35f0109c951219b623c98383c7ab9cf1445d9b, native PostgreSQL and real API processes. This includes the previously failing suspended-tenant denial, sixteen concurrent conversion requests producing one evidence row, cross-tenant isolation, revocation and fresh API-process recovery. TAP includes the parent, so its 10 passes are not ten independent scenarios.

At 2026-09-17T19:29:42.898001+00:00, separate OS process 32208 loaded the session through the installed engine and independently asserted both completed tasks, VERIFIED_COMPLETE, the original command, valid completion-to-command linkage and byte-identical preservation of the original failure. It also verified that reading the session did not change its bytes. Session SHA-256: 4a4910950a10de6dfca29f6d3c8f51763ff20fe2281b7cdf971b51584cbb59ff.

Producer metadata remains self-reported; this is not independent model/provider-dispatch attestation.

Cleanup completed

At 2026-09-17T19:30:50.744819+00:00, the exact owned PostgreSQL process/data directory and loopback port were checked and zero other client connections confirmed. The synthetic cluster was stopped normally; its data was retained. Prior PID 6390 is not still-running evidence. No unrelated database or product service was stopped.

Retained SI source evidence

Current source 46629c3 converges #51's byte repair with Windows index portability and prevents all-skipped or expected-failure-only unittest runs from falsely completing repairs. The engine requires the final standard stderr report with evidence of actual successful methods. These controls are not tamper-proof against malicious test code.

That exact clean source passed all five canonical Linux quality stages at 2026-09-17T16:22:28.406460+00:00, snapshot 936efbefbd537a6845cbb6c48fdf619287b37c962a99713e1ee772fc87926821; the generated 59-file adapter passed validation. Separately, all 13 repair-handoff tests passed natively on Windows/Python 3.11.9. Those are retained prior results, not new reruns in this milestone.

Full native Windows acceptance still has the historical symlink-capability boundary. The new source was not installed over the old controller, so do not claim its newer unittest behavior was proved by the old controller's Node acceptance.

Infinity source and proof identity

Infinity PR #16 is now published at 05c73f19a9bc6f67d1a6e289b4e5cdbe2aafe673. Its strict index is current/ready, final source is clean, and all 64 source tests passed. The native test used installed 1a35f010...; final comparison matched all 48 compiled API/registry/contracts files. Only the native fixture and generated index differ in the final source. A second installation labeled with the fixture-only final revision was not completed after a preparation append was blocked; the old label is not relabeled as a new installation.

Infinity's separately declared SI consumer remains pinned to d15220aea58553b08d9f577cf527ff3118f42edf, installable=false / installation_unverified. External controller acceptance does not establish installation of that consumer.

Evidence and next actual requirement

Root: D:/si-infinity-continuation-20260917/evidence.

  • si-installed-repair-verification.json and infinity-native-repaired.log: real repair/native result.
  • si-fresh-process-completion.json: independent completed-session recovery.
  • infinity-native-fixture-key-change.json: limited fixture change and preimages.
  • infinity-native-final-content.json: unchanged nine AST scenarios, clean published source and 48 matching runtime files.
  • owned-postgres-shutdown.json: exact completed cleanup.
  • Existing SI Linux/Windows source receipts remain valid within their recorded scopes.

Next is deliberate current-runtime installation/client entrypoint integration and a real cold-start project task using saved state automatically. The completed narrow test/repair loop is no longer the blocker. Universal ChatGPT/Codex adoption, normal all-project governance, full Windows release acceptance, hosted required checks and production release are not established.

Do not redo the native suspension repair, branch convergence, original WSL install, completed database cleanup or generic repository audit without a relevant change. Resume from this verified completion and keep source versions, installed versions and consumer declarations distinct.

@infotradescout infotradescout changed the title Preserve exact staged UTF-8 bytes on Windows Repair SI Windows byte preservation and source-index portability Sep 16, 2026

Copy link
Copy Markdown
Collaborator Author

September 17 continuation — branches converged, clean Linux gate passed, installed action exercised

Published exact head: 3d754ac691ac3487283a2b18e7f6a9d209de8d59. Normal non-force push advanced this existing branch from 3b6f6f30; remote head was read back. The merge retains #51 aa787adfb696e493e2656fa853041eb294599ec8 and #54 3b6f6f30b52e756f6e73cef3a770e40d471e41f8. Main and the installed global skill were not changed.

The overlapping writer changes were the same newline="" repair with different comments. The native permission assertions were equivalent; the merged test also retains #51's independent exact-byte application/receipt regression. The ChatGPT adapter was regenerated canonically, not edited by hand. Delta against previous #54: four files, 14 additions/4 deletions; no source mode changes.

Executed source evidence

  • Canonical final quality gate: all five stages PASS at 2026-09-17T15:45:36.414414+00:00, exact head above, workingTreeDirty=false, source snapshot a9b3e55210a639c502fecc64eb052d621633b970560283eeabfb4d1430988e89, 323 counted source files.
  • Adapter validation: 59 files PASS. Focused product-model transaction suite: 17 PASS. Diff checks passed.
  • Initial full gate correctly rejected four unlisted generated Python cache files. A diagnostic run executed 344 tests with one release-inventory failure; the four untracked .pyc files were preserved outside the checkout and the unchanged canonical gate then passed. No source, release manifest rule, test assertion, or gate was weakened to erase that failure.
  • Evidence: D:/si-infinity-continuation-20260917/evidence/si-converged-final-quality.json, si-cache-isolation-correction.json, archived generated cache, and diagnostic failure logs.

Existing installed runtime: beyond read-only reload, but not completed repair

The original byte-verified 117-file WSL installation at aa787ad was preserved and used. Its saved session si-6197351c45264174a75485af9e7fa264 correctly rejected replacing the old default proposal. The current user's instruction, continue SI and infinity here, was reconciled through the existing runtime into checkpoint cp-8e4cfc136369; it was not recorded as a fabricated literal APPROVE message or production permission.

The installed engine generated a bounded task packet, actually applied two hashed files in its authorized isolated acceptance workspace, ran the native Node acceptance command, and recorded the real failure. Original task: task-fc16bf631c99. Failure evidence: command-bd995ef3f8e2. Engine-issued repair task: task-b6917df3394c. The test exercised the installed Infinity API and native local PostgreSQL, not memory-store substitutes. Eight of nine scenario subtests passed; suspended-tenant authentication failed with actual HTTP 200 versus expected 401. Parent TAP failure is not a second independent defect.

The controller implemented the Infinity source repair separately in its authorized feature worktree. A tool safety check blocked preparation of the SI repair-run handoff; that operation was not retried through another path. Therefore original task remains repairing and repair task remains ready; no repaired native pass, propagated completion, or fresh-process completed-task proof is claimed. A separate preservation/cleanup script was also blocked before execution; cleanup is not claimed.

Resume / release boundaries

Keep this PR draft. Native Windows full-quality acceptance is NOT newly proved; prior symlink-capability errors remain an explicit boundary. No privileges were enabled, global installation replaced, main merged, hosted required CI cleared, model/provider dispatch attested, or all-project automatic adoption established.

Use existing source and receipts; do not redo convergence, reinstall the original WSL runtime, or re-audit the repositories. Next unproved transition is the native Infinity suspension-repair verification with the original command and authentic SI repair lineage, followed by a fresh-process read of genuine completion evidence. Local fixture database cleanup was not completed in this continuation.

Copy link
Copy Markdown
Collaborator Author

September 17 follow-on — reject zero-success unittest completion

Published head: 46629c3880151e5bcef090a5a85bbb7f8eeeaff4, ordinary non-force advancement from the previously converged 3d754ac6. Remote identity was read back. This is a source/runtime repair on the existing draft; main and the installed WSL runtime were not changed.

Reproduced defect and implemented correction

The existing verify_task path accepted exit code 0 plus any positive Ran N tests match. An actual isolated repair was therefore marked complete when every unittest was skipped, or when its only result was an expected failure. An earlier printed stderr summary also satisfied the old matcher. These were reproduced against the prior runtime source, not inferred from a review.

The canonical engine now requires the final standard unittest report on stderr and a positive lower bound on successful methods after subtracting skipped and expected-failure counts. Fixture stdout and earlier summaries cannot replace that final report. A genuine successful method with an optional skip still passes. Ambiguous subtest skip counts remain unproved rather than inventing success. This is not tamper-proof attestation against arbitrary malicious test code.

The existing repair-handoff suite owns five additional cases. The initial run had three reproduced completion failures plus one test-fixture assumption error: zero collection already received exit 5 from the existing guard, not the fixture's assumed zero. That extra exit-code assumption was removed while retaining the actual zero-test stderr and failure assertions. No existing guard was relaxed. Final focused result: 13/13 passed.

Exact-source acceptance

  • All five canonical Linux quality stages passed at 2026-09-17T16:22:28.406460+00:00 on exact clean head 46629c3.
  • Source snapshot: 936efbefbd537a6845cbb6c48fdf619287b37c962a99713e1ee772fc87926821; counted source files: 323.
  • Canonically regenerated ChatGPT adapter: 59 files validated.
  • Four committed paths: canonical engine, its existing repair-handoff test, generated adapter engine, generated adapter metadata. Diff checks passed.
  • Evidence: D:/si-infinity-continuation-20260917/evidence/si-zero-success-baseline.log, si-success-focused.log, si-success-adapter-test.log, si-success-full-quality.json.

Unchanged installed-client boundary

This follow-on did NOT retry the previously blocked installed Infinity repair handoff, alter its session, claim a native PostgreSQL repaired pass, or install this new source over the existing aa787ad runtime. The native Windows symlink-capability limitation and universal-client-adoption gap remain. No hosted required-CI or production acceptance is claimed.

The original installed session remains at its last recorded checkpoint cp-8e4cfc136369; original task task-fc16bf631c99 was repairing and repair task-b6917df3394c was ready. Those are retained prior observations, not a fresh session inspection in this follow-on. The synthetic PostgreSQL cleanup is also still unconfirmed.

Resume this exact published candidate and the existing repair boundary. Do not repeat branch convergence or these passing source checks unless affected by later changes. Future installed acceptance must distinguish this candidate's new verification semantics from the still-installed earlier runtime.

This branch has not been deployed

No deployments
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.

3 participants