Skip to content

fix: repair recent thread, evaluation, and cleanup regressions - #73

Open
ScriptedAlchemy wants to merge 2 commits into
mainfrom
fix/post-merge-audit-20261106
Open

ScriptedAlchemy wants to merge 2 commits into
mainfrom
fix/post-merge-audit-20261106

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Nov 6, 2026 •

Copy link
Copy Markdown
Owner

Cleanup can delete a worktree after treating truncated Git inspection output as complete. The same audit found regressions in citation lifetimes, split-suggestion history, and blind-evaluation confidence.

Changes

The index-shutdown ordering races specific to open #67 are fixed on that PR. This PR contains the fixes applicable to current main.

Verification

  • check:changed: native addon, formatting, lint, both Knip scans, and GraphQL generation checks passed.
  • Selected regression tests: 223 passed, with 1 existing permissions-dependent skip.
  • Selected React compiler checks: 7 passed.
  • All 41 affected typecheck tasks passed with the supported --no-cache option. The default cached launcher cannot initialize its seccomp_unotify supervisor in this executor; diagnostics and repository settings remain enabled.
  • Android FilePreviewModal regression: 3/3 passed after reproducing the original two failures.
  • Additional audit suites passed: 116 tests covering thread loading, link parsing, Codex computer use/instructions, and native plugin/model handling; the evaluation builder/runner suite passed 13 tests.
  • New coverage checks incomplete cleanup inspection, discarded history, omitted context, citation retention, query eligibility, and cached-failure retry.

The same Windows fixture corrections were verified in the full Windows cleanup suite on #67, at commit 6fe77ad584d27bc3d2cc9b4084cfcf893c09919b: 209 passed, 1 existing permissions test skipped, job success. The final three allocation/hidden-file fixture corrections also passed on this PR with file format/lint checks; this follow-up changes no production code. Live UI verification could not run because the repository's test-t3-app skill requires its T3 Browser panel, which is unavailable in this harness.

Model: GPT-6 Astra Pro. Harness: ChatGPT.

@changeset-bot

changeset-bot Bot commented Nov 6, 2026 •

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 7ad853b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Compare report totals with actual filesystem allocation and preserve Git's hidden worktree metadata file while testing invalid pointers.

Model: GPT-6 Astra Pro. Harness: ChatGPT.

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.

1 participant