feat: terminal launcher diagnostics and mutation preflight (Closes #556) #580
Closed
jcwalker3
wants to merge 0 commits from
feat/issue-556-terminal-launcher-diagnose into master
pull from: feat/issue-556-terminal-launcher-diagnose
merge into: :master
:master
:fix/issue-709-decision-lock-cross-profile
:fix/issue-695-native-transport-quarantine
:fix/issue-698-report-validator-schema
:fix/issue-702-stale-binding-lease-recovery
:fix/issue-699-structured-auth-mcp-errors
:fix/issue-695-native-quarantine-v2
:fix/issue-693-review-decision-lock-recovery
:fix/issue-691-obsolete-reviewer-lease-cleanup
:feat/issue-687-reconciler-branch-delete
:fix/issue-683-workflow-guard-hardening
:chore/issue-681-preserve-review-session-wip
:feat/issue-604-anti-stomp-preflight
:feat/issue-606-sentry-observability
:fix/issue-671-block-stable-branch-push
:fix/issue-675-residual-preflight-remediation
:fix/issue-673-remediate-regressions-part2
:fix/issue-673-remediate-regressions
:feat/issue-603-lifecycle-labels
:fix/issue-627-set-issue-labels-pagination
:feat/issue-601-first-class-leases
:feat/issue-612-incident-bridge
:feat/issue-600-controller-allocator-api
:fix/issue-620-head-scoped-review-locks
:feat/issue-613-allocator-db-substrate
:docs/mcp-stable-control-runtime-policy
:feat/issue-609-prepared-review-verdict-resume
:feat/issue-610-live-remote-parity
:feat/issue-503-reviewer-active-worktree
:feat/issue-470-preflight-contract
:feat/issue-440-lock-recovery
:feat/issue-440-branch-recovery
:feat/issue-458-queue-fail-closed-copy
:feat/issue-400-early-duplicate-work-gate
:feat/issue-308-reconcile-inventory-pagination
:feat/issue-308-reconcile-pagination-proof
:docs/issue-261-agent-temp-artifact-cleanup
:feat/issue-262-map-commit-files
:fix/infra-stop-conflict-marker-false-positive
:feat/issue-139-role-aware-task-routing
:feat/issue-188-continuation-selection-wall
:feat/issue-189-continuation-mode-proofs
:feat/issue-232-refresh-wiki-proof-heads
:feat/issue-210-block-workspace-edits
:feat/issue-204-exact-issue-lock
:docs/issue-80-label-taxonomy
:docs/issue-79-safety-boundary-updates
Labels
Clear labels
allocator
anti-stomp
architecture
bug
chore
codex
concurrency
contamination
control-plane
dashboard
database
design
documentation
enhancement
gitea
glitchtip
important
incident
incident-bridge
integration
jenkins
labels
leases
mcp
mcp-health
mcp-menu
multi-project
mutating
nice-to-have
observability
portability
preflight
protected-branch
queue
read-only
reconnect
recovery
refactor
release
reliability
resumable-review
reviewer
roadmap
safety
security
self-hosted
sentry
stale-runtime
status:blocked
status:in-progress
status:pr-open
status:ready
terminal-lock
testing
tracker
type:bug
type:feature
type:feature
type:guardrail
visibility
workflow
workflow-hardening
workflow-hardening
Controller-owned work allocator
Prevent concurrent LLM session stomping
Architecture / structural design
OpenAI Codex client / workflow session surface
Concurrent session safety
Workflow or session contamination incident
MCP control-plane coordination and allocation authority
MCP operational dashboard/queue view
Internal coordination storage (SQLite/Postgres)
Design / investigation, no implementation
Docs / runbooks
New feature or improvement
Gitea MCP workflow
GlitchTip integration
Operational or process incident requiring durable audit trail
Sentry-to-Gitea incident bridging
Integration testing
Jenkins integration
Label taxonomy management
Lease adopt/release/expire lifecycle
MCP server / tooling
MCP namespace and runtime health
MCP menu surface
Work spanning multiple monitoring projects or Gitea repos
Mutating action; requires gating
Observability, metrics, traces, error reporting
Cross-platform / portability
Shared preflight gates before mutation
Protected branch / stable-branch policy concern
Work queue visibility and allocation
Read-only, no mutation
MCP client reconnect/reload recovery path
Recovery paths for stale/foreign leases
Code refactor / restructure
Release / versioning
Reliability / failure handling
Persist and resume prepared review verdicts across sessions
Reviewer workflow tooling
Roadmap / umbrella issue
Safety rails and fail-closed mutation guards
Security / trust boundary
Self-hosted infrastructure integration
Sentry error monitoring integration
Stale backend daemon / runtime-vs-master parity failures
Issue is blocked
Issue is being worked on
Issue has an open pull request
Issue is ready for work
Terminal review lock (#332) path
Tests / test coverage
Issue tracker hygiene / meta
Bug or defect
Feature or enhancement
Feature or enhancement
Safety gate or guardrail
Workflow state visibility for LLMs/operators
Cross-tool workflow
LLM workflow coordination hardening
LLM workflow coordination hardening
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Scaled-Tech-Consulting/Gitea-Tools#580
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
Addresses #556: when terminal spawn fails (e.g.
os error 2), workflows need categorized diagnostics and a fail-closed preflight instead of opaque retries or unsafe fallbacks.Changes
native_mcp_preference.py—diagnose_terminal_failure/probe_terminal_spawnclassify missing cwd, non-dir cwd, missing executable, missing runtime wrapper (venv/pytest), missing shell, probe timeout, or session launcher failure.gitea_mcp_server.py— read-onlygitea_diagnose_terminaltool; mutation-taskgitea_resolve_task_capabilityprobes the launcher and returnsBLOCKED + DIAGNOSEwithterminal_launcher_unhealthywhen unhealthy.GITEA_FORCE_TERMINAL_PROBE=1under pytest).Validation
Notes
run_terminal_commandfailures remain a platform concern; this PR makes Gitea-Tools detect and stop safely before git/pytest finalization.GITEA_FORCE_TERMINAL_PROBE=1to exercise it in tests.Closes #556
Worktree
branches/feat-issue-556-terminal-launcher-diagnoseBranch:
feat/issue-556-terminal-launcher-diagnoserepo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
223cde1b94target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:34:22Z
expires_at: 2026-07-09T16:34:22Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
223cde1b94target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:39:10Z
expires_at: 2026-07-09T16:39:10Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
223cde1b94target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:40:21Z
expires_at: 2026-07-09T16:40:21Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
223cde1b94target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:44:44Z
expires_at: 2026-07-09T16:44:44Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:46:28Z
expires_at: 2026-07-09T16:46:28Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:46:35Z
expires_at: 2026-07-09T16:46:35Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:47:16Z
expires_at: 2026-07-09T16:47:16Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:47:23Z
expires_at: 2026-07-09T16:47:23Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:49:27Z
expires_at: 2026-07-09T16:49:27Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:49:38Z
expires_at: 2026-07-09T16:49:38Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:50:14Z
expires_at: 2026-07-09T16:50:14Z
blocker: none
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:51:01Z
expires_at: 2026-07-09T16:51:01Z
blocker: none
Canonical PR State
STATE: review-approved
WHO_IS_NEXT: merger
NEXT_ACTION: merge PR #580 after explicit operator authorization
NEXT_PROMPT:
WHAT_HAPPENED: Reviewed PR #580 for issue #556; approved head
f558b80cc8.WHY: Scope is limited to terminal diagnostics + preflight + tests + runbook; validation passed.
ISSUE: #556
HEAD_SHA:
f558b80cc8REVIEW_STATUS: APPROVED
MERGE_READY: yes (awaiting explicit merge authorization)
BLOCKERS: none
VALIDATION: git diff --check clean; pytest tests/test_native_mcp_preference.py tests/test_resolve_task_capability.py → 43 passed
LAST_UPDATED_BY: sysadmin (prgs-reviewer)
Review notes
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-reviewer
session_id: 66242-d5706668e24f
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/review-pr-580-issue-556
phase: claimed
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T14:55:58Z
expires_at: 2026-07-09T16:55:58Z
blocker: none
adopted_at: 2026-07-09T15:36:57Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 66242-d5706668e24f
adopted_from_profile: prgs-reviewer
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 8156
adoption_reason: merger-handoff-approved-head
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 1956-aa22dbda40f7
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/merger-queue
phase: adopted
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T15:36:57Z
expires_at: 2026-07-09T17:36:57Z
blocker: none
adopted_at: 2026-07-09T15:42:57Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 1956-aa22dbda40f7
adopted_from_profile: prgs-merger
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 8199
adoption_reason: merger-handoff-approved-head
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 31616-fe56940d16c2
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/merger-queue
phase: adopted
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T15:42:57Z
expires_at: 2026-07-09T17:42:57Z
blocker: none
adopted_at: 2026-07-09T15:45:01Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 31616-fe56940d16c2
adopted_from_profile: prgs-merger
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 8215
adoption_reason: merger-handoff-approved-head
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 37725-5a80923abd79
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/merger-queue
phase: adopted
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T15:45:01Z
expires_at: 2026-07-09T17:45:01Z
blocker: none
adopted_at: 2026-07-09T15:45:49Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 37725-5a80923abd79
adopted_from_profile: prgs-merger
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 8221
adoption_reason: merger-handoff-approved-head
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 37725-84781e38a2fc
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/merger-queue
phase: adopted
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T15:45:49Z
expires_at: 2026-07-09T17:45:49Z
blocker: none
adopted_at: 2026-07-09T15:47:33Z
adopted_by_identity: sysadmin
adopted_by_profile: prgs-merger
adopted_from_session_id: 37725-84781e38a2fc
adopted_from_profile: prgs-merger
adopted_from_reviewer_identity: sysadmin
adopted_from_comment_id: 8225
adoption_reason: merger-handoff-approved-head
repo: Scaled-Tech-Consulting/Gitea-Tools
pr: #580
issue: #556
reviewer_identity: sysadmin
profile: prgs-merger
session_id: 46120-bc5073e7ae62
worktree: /Users/jasonwalker/Development/Gitea-Tools/branches/merger-queue
phase: adopted
candidate_head:
f558b80cc8target_branch: master
target_branch_sha:
cc4b95839dlast_activity: 2026-07-09T15:47:33Z
expires_at: 2026-07-09T17:47:33Z
blocker: none
Canonical PR State
STATE:
merged
WHO_IS_NEXT:
controller
NEXT_ACTION:
No further PR action; head already on master. Close linked issue #556 if still open (issue.close not available on reconciler profile).
NEXT_PROMPT:
WHAT_HAPPENED:
Already-landed reconciliation closed PR #580. Head
f558b80is an ancestor of master; content already present without a second merge.WHY:
ALREADY_LANDED_RECONCILE_REQUIRED — open PR after head already on target.
ISSUE:
#556
BASE:
master
HEAD:
feat/issue-556-terminal-launcher-diagnose
HEAD_SHA:
f558b80cc8REVIEW_STATUS:
none
VALIDATION:
ancestor_proof=true; eligibility_class=ALREADY_LANDED_RECONCILE_REQUIRED
BLOCKERS:
none
MERGE_READY:
n/a — already landed
LAST_UPDATED_BY:
sysadmin / prgs-reconciler / 2026-07-09
Pull request closed