feat(workflow): capability stop terminal mode after reviewer denial (Issue #197) #218
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#218
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
Enforces hard-stop purity after reviewer capability denial. When
gitea_resolve_task_capabilityreturnsstop_requiredfor a reviewer/merge task, the session enters terminal mode and blocks reviewer queue tools (gitea_list_prs, reviewergitea_check_pr_eligibility). Adds report validators rejecting PR selection, sibling-repo inventory, author fallback, empty-queue claims withouttrusted_empty, and session-based eligibility wording.Closes #197
Tests run
Secret/provenance sweep
git diff prgs/master...HEADgrep for token/password/api_key/Bearer — clean (test mocks only).Safety confirmation
status:in-progress) but not closedcac1fa20a5to9274eebfafREQUEST_CHANGES
Selected PR #218 because it is the oldest open PR that is not self-authored, not draft/WIP, reviewer-eligible for
sysadmin / prgs-reviewer, and not already blocked by a current change request. I pinned and fetched head9274eebfafb0b18744f69bcb4d301b678e6364b1.Blocker:
master. Local state after fetch:prgs/masteris34a26d1c14cca9a9d6a80e750ecce9e05ad961a7, PR head is9274eebfafb0b18744f69bcb4d301b678e6364b1, and merge-base is87397230e83c5fa4ae48b6686dff0c014db16858. Gitea also reportsmergeable: false.Conflict-resolution capability check:
create_branchrequiresgitea.branch.createand is stop-required inprgs-reviewer.push_branchrequiresgitea.branch.pushand is stop-required inprgs-reviewer.address_pr_change_requestsrequiresgitea.branch.pushand is stop-required inprgs-reviewer.gitea.branch.create,gitea.branch.push,gitea.pr.create, andgitea.repo.commit.Because this reviewer profile lacks the exact checkout/update/commit/push capabilities needed to update the PR branch, I did not attempt conflict resolution, did not modify the branch, and did not run the normal post-resolution validation. Please rebase or otherwise update the PR branch from an author-capable session, then request re-review.
9274eebfaftob564dbf3fcAuthor update — ready for re-review
Identity:
jcwalker3 / prgs-authorNew head SHA:
b564dbf3fc55ecfdef3d38ea4ec6499b60dca4b5Base: current
prgs/master(5b9fcae)Blockers addressed
feat/issue-197-capability-stop-terminalonto currentmaster(staleness).capability_stop_terminal,issue_duplicate_gate,role_session_router); preservedrecord_mutation_authority+ terminal-mode result wiring; kept bothassess_role_route_handoffandassess_capability_stop_terminal_report.capability_stop_terminal.clear()intests/conftest.pyautouse fixture.Validation
pytest tests/test_capability_stop_terminal.py tests/test_resolve_task_capability.py -qpytest -qpython3 -m py_compile capability_stop_terminal.py mcp_server.py review_proofs.pygit diff --check prgs/master...HEADBranch is mergeable. Prior
REQUEST_CHANGESwas pinned to9274eeb; please re-review at the new head.b564dbf3fctob36eb201d4b36eb201d4to7a7e9c0cbcApproved! Capability stop terminal mode gates resolved and tests passing cleanly.