Skip to content

fix(gateway): harden publish and review recovery#18

Merged
abretonc7s merged 10 commits into
mainfrom
fix/publish-evidence-sidecars
Jun 9, 2026
Merged

fix(gateway): harden publish and review recovery#18
abretonc7s merged 10 commits into
mainfrom
fix/publish-evidence-sidecars

Conversation

@abretonc7s

@abretonc7s abretonc7s commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • keep publish evidence sidecars media-only so non-UI/state evidence can live in coverage/report artifacts without breaking publication
  • ignore stale self-review-fix signals so retry loops do not consume old completion files
  • harden runner prompt delivery verification: queued/visible prompt text is not accepted unless real progress happens
  • launch Claude self-review non-interactively with the task prompt to avoid fragile tmux post-launch input delivery

Validation

  • node scripts/quality/run-tsx-tests.mjs --cwd services/gateway --tsconfig tsconfig.json src/self-review/orchestrator.test.ts src/methods/run/replay-step.test.ts src/runners/registry.test.ts
  • node scripts/quality/run-tsx-tests.mjs --cwd services/gateway --tsconfig tsconfig.json src/runners/registry.test.ts src/self-review/orchestrator.test.ts
  • pre-commit ratchets passed on committed gateway changes

Live recovery evidence

  • TAT-3307 (49b8906f) recovered to human gate after parser/prompt-verification fixes
  • TAT-1043 (3a778bd5) self-review was relaunched with non-interactive Claude after queued-prompt failures

@abretonc7s abretonc7s requested a review from deeeed as a code owner June 9, 2026 02:11
@abretonc7s abretonc7s changed the title fix(gateway): keep publish evidence media-only fix(gateway): harden publish and review recovery Jun 9, 2026
@abretonc7s abretonc7s merged commit 26f2e42 into main Jun 9, 2026
15 checks passed
@abretonc7s abretonc7s deleted the fix/publish-evidence-sidecars branch June 9, 2026 07:07
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.

2 participants