Epic: Make author workflows transactional and safe for multiple LLM clients #887
Open
opened 2026-07-24 21:09:41 -05:00 by jcwalker3
·
2 comments
No Branch/Tag Specified
master
fix/issue-987-native-mcp-bootstrap
feat/issue-985-project-scoped-launcher-identity
fix/issue-983-cross-repo-base-ref
feat/issue-980-stale-worker-retirement
fix/issue-975-client-identity-heartbeat
fix/issue-973-cross-repo-canonical-roots
fix/issue-970-safely-resolve-missing-worktrees
fix/issue-969-native-mcp-bootstrap
feat/issue-664-break-glass-restart
feat/issue-708-mcp-namespace-attachment
feat/issue-665-restart-audit
fix/issue-700-durable-walls
fix/issue-704-prevent-env-workspace-bindings
feat/issue-707-cross-project-boundaries
fix/issue-690-review-profile-switch-guard
fix/issue-953-bootstrap-lock-provenance
feat/issue-949-native-fleet-inventory
fix/issue-943-runtime-context-helpers
fix/issue-945-owning-pr-renewal-evidence
fix/issue-941-scope-guard-bootstrap-wiring
docs/issue-930-remote-mcp-coupling-inventory
fix/issue-892-author-bootstrap-deadlock
fix/issue-686-detect-reject-manual-mcp
fix/issue-672-mcp-config-drift
fix/issue-689-deterministic-mcp-namespace
feat/issue-666-concurrent-mcp-restart-tests
feat/issue-659-maintenance-drain-mode
feat/issue-648-notifications-console
fix/issue-670-direct-master-incident
feat/issue-644-console-recovery
feat/issue-650-providers-insights
feat/issue-669-scoped-component-recovery
docs/issue-668-mcp-ha-rolling-restart
feat/issue-667-console-restart-controls
feat/issue-645-linkage-console
feat/issue-643-request-preview-initiate
fix/issue-897-permission-stale-runtime-classification
feat/issue-641-runtime-session-view
feat/issue-663-restart-classes
feat/issue-661-drain-proof-hard-gate
fix/issue-854-semantic-container-exclusion
issue-640
fix/issue-682-starlette-httpx2
v1.1.0
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
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
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
Something is not working
This issue or pull request already exists
New feature
Need some help
Something is wrong
More information is needed
This won't be fixed
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: Scaled-Tech-Consulting/Gitea-Tools#887
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.
Problem
Author workflows are a chain of independently-failing operations with no transaction boundary. Every stage keeps its own state (assignment, workflow lease, keyed lock,
/tmppointer file, generation counter, branch, worktree), and those stores are allowed to contradict one another while the gate layer still reportssafe_next_action: proceed. When any single stage fails, the run leaves durable, contradictory state that the next worker — often a different LLM client — cannot interpret or recover from without operator repair.Observed consequences, all reproduced on live runs:
/tmp/gitea_issue_lock.json, the global current-issue pointer).preflight_order_violationfires whenever any call — including read-only ones — lands between capability resolution and the intended mutation, so correctness depends on prompt-level instructions to emit two adjacent tool calls.commit_filescapability was followed by shellgit add/git commit, bypassing the native path entirely.Scope
Eight children: five P0 (correctness and isolation), three P1 (stability, enforcement, observability).
Child issue numbers are linked in a follow-up comment on this issue.
Epic completion gate
Do not restart unattended looping until all of the following hold:
Non-goals
Duplicate verdict
No existing epic covers this scope. Issue #820 is the control-plane architecture program and is a different scope. Verified against a full scan of open issues at master
ccde9e8f1183bc9a6672393a9923f304086fb874.[THREAD STATE LEDGER] Issue #887 — infrastructure coverage for the three defects required alongside this epic
What is true now:
a4c73766f4, mutation_safe true, restart_required false.What changed:
What is blocked:
Who/what acts next:
Canonical Issue State
STATE:
open — epic children complete; adjacent infrastructure coverage now tracked
WHO_IS_NEXT:
author
NEXT_ACTION:
Implement the three infrastructure trackers independently: #899 loop-disable enforcement, #900 cohort lifecycle supervision, #689 deterministic attachment.
NEXT_PROMPT:
WHAT_HAPPENED:
A focused infrastructure-coverage audit compared the exact bodies, comments, and acceptance criteria of #686, #689, #708, #678, #669, #655, #659, and the #797 worker-management family against the fifteen requirements for loop-disable enforcement, daemon lifecycle supervision, and deterministic namespace attachment. Only one requirement was fully covered. Two trackers were created and one was extended so every remaining requirement has a durable home.
WHY:
The epic's completion gate forbids restarting unattended looping until failures are durably recorded and recoverable. That gate cannot be evaluated while loop-disable is an operator convention, daemon cohorts accumulate unbounded, and attachment to a parity-matching cohort is unverifiable.
RELATED_PRS:
none
BLOCKERS:
none
VALIDATION:
Parity re-checked after every mutation and unchanged at
a4c73766f4. #899 and #900 returned valid workflow label validation with exactly one type label and one status label each. #689 returned verified true with preserved_intact true and an empty mismatch list.LAST_UPDATED_BY:
jcwalker3 / prgs-author / AUTHOR / 2026-07-24
[THREAD STATE LEDGER] Issue #887 — verified children, #948 placement, and the five-client canary
What is true now
aad5c8b42361d380a8eeb07b94b90815e594c2c5. The five-client concurrency canary now has a durable tracker, #963, parented here.aad5c8b42361d380a8eeb07b94b90815e594c2c5; mutation_safe true; live_stale false; restart_required false.Verified children of this epic
#948 placement
#948 is the canonical client/session-aware generation-ownership issue, and it is recorded here as belonging to this epic's scope. It previously carried no documented parent. It owns worker identity distinct from role and profile (AC26 through AC44), atomic lease acquisition with fencing (AC9 through AC16), heartbeat and expiration (AC11, AC14), author and reviewer independence (AC18), exact-head verdict and merge gating (AC19 through AC21), and the identity invariants:
worker_identityunique, role not unique, profile not unique, exclusivity on the active lease target only. It reconciles #686 and #900 explicitly.Verified relationships
aad5c8b42361d380a8eeb07b94b90815e594c2c5, and #953 is in state closed. This removes the prerequisite that previously blocked the P0 chain; #888, #889, #890, and #891 are now implementation-eligible.Dependency order for this epic
What changed
What is blocked
status:blockedlabel pending its listed dependencies, which is its designed state, not a defect.Who/what acts next
aad5c8b42361d380a8eeb07b94b90815e594c2c5.Canonical Issue State
STATE:
open — children verified; #948 placement recorded; five-client canary tracked as #963
WHO_IS_NEXT:
author
NEXT_ACTION:
Implement the P0 children #888, #889, #890, and #891, now unblocked because PR #954 reached state closed with merge_commit_sha
aad5c8b423.NEXT_PROMPT:
WHAT_HAPPENED:
A live roadmap audit mapped every multi-LLM requirement to a canonical owner at master
aad5c8b423. Three requirements had no owner: a five-client simultaneous canary, a negative concurrency suite covering repository crossover, and a certification gate on unattended scheduling. #963 was created under this epic for the first two. #964 was created under umbrella #955 for the third. #948 was confirmed as the canonical generation-ownership issue and its placement under this epic is recorded here.WHY:
This epic's completion gate requires two different LLM clients to complete a supervised author canary. That gate is prose in an epic body, owned by no issue and testable by no command, and the operator runs five scheduled workers rather than two. #948 covers the implementation but its concurrency tests are simulated. Without #963 the gate cannot be evaluated, and without #964 nothing prevents unattended scheduling from starting before it is.
ISSUE: 887
RELATED_PRS:
954 (PR state closed, merge_commit_sha aad5c8b42361d380a8eeb07b94b90815e594c2c5; satisfies #953)
BLOCKERS:
none
VALIDATION:
Parity verified before and after every mutation and unchanged at
aad5c8b423with mutation_safe true, live_stale false, restart_required false. The create_issue capability was resolved immediately before each creation. #963 was read back natively: issue state open, correct title, complete body, eight valid workflow labels with exactly one type label and one status label. The duplicate search was repeated after creation and returned exactly one instance of each new title.LAST_UPDATED_BY:
jcwalker3 / prgs-author / AUTHOR / 2026-07-28