Skip to content

Partial revert of #5806: drop check_harness_server_cleanup.sh + 817 (keep launcher fixes) - #5867

Merged
noahgift merged 1 commit into
mainfrom
revert/5806-guard-only
Oct 21, 2026
Merged

noahgift merged 1 commit into
mainfrom
revert/5806-guard-only

Conversation

@noahgift

Copy link
Copy Markdown
Member

Re-cut of #5863 as ONE commit (same tree as 1a2a168) so it lands cleanly under the rebase merge queue.

Operator 2026-10-21 13:20Z (verbatim, relayed): "Partial revert, not full: revert only scripts/check_harness_server_cleanup.sh + 817-harness-server-cleanup.cmd. Keep the 3 launcher zombie fixes from #5806. Open now, top of the queue. Re-land: guard rewritten in bash (bashrs) or Rust. NO python3 in any image, NO skip-if-missing."

Fixes the main-red: python3 missing in the container (#5844 shard 5) and the apr-stub race (#5854). Supersedes #5863 and #5860 (both draft). Not armed.

🤖 Generated with Claude Code

…mmand; keep #5806's launcher fixes (Refs #5806)

Operator 2026-10-21 13:20Z: partial revert of #5806. The guard needs python3, which the sovereign-ci container lacks (#5844 shard 5), and its apr-stub row races (#5854). Re-land: bash (bashrs) or Rust, no python3, no skip-if-missing.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift

Copy link
Copy Markdown
Member Author

patch-id proof (git patch-id --stable, diff vs merge-base b9cb67a):

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seat (non-author) bound to head 844320d. Receipt: handoff/triage-quorum-5867-844320dec2.md. Opus (executed) + agy unanimous APPROVE. Verified: exactly 2 files deleted (105 deletions), no remaining reference to the guard or fragment 817, check_explicit_test_commands / check_shard_union (117, derived) / check_guard_steps_isolated rc0, stable patch-id equals #58631a2a168 (c22027f073f2), 3 launcher fixes untouched. Non-blocking: nothing covers the guard_server watchers until the operator-ordered bash/Rust re-land; keep that ticket open. Not armed by me.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seat (non-author, lane: Opus via a3 dispatch; not triage, not Sonnet) bound to head 844320d. Diff read: deletes only scripts/check_harness_server_cleanup.sh + ci/explicit-test-commands.d/817; no dangling refs; #5806 launcher fixes intact. VERDICT: APPROVE.

@noahgift
noahgift enabled auto-merge October 21, 2026 13:46
@github-actions

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=5867 head=844320dec29dc16a28bc95c9f5de8ff9883776b1 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

@noahgift
noahgift added this pull request to the merge queue Oct 21, 2026
Merged via the queue into main with commit ce0c882 Oct 21, 2026
45 of 59 checks passed
@noahgift
noahgift deleted the revert/5806-guard-only branch October 21, 2026 21:27
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