Fix stalled workflows and integrate development on devel - #13
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Targets devel. Merging this PR does not publish a release. Merge #13 before #14.
Development and publication flow
Feature PRs now target
devel, with CI on every new PR revision. When a release is wanted, a reviewed same-repositorydevel→releasePR starts the existing automatic patch/tag/publication pipeline. Other feature branches targetingreleaseare rejected by CI and cannot start automatic publication.After stable publication, the publisher commits the versioned README on
release, opens/updates the main promotion PR, and automatically merges the published release head intodevelwithout a synchronization PR. Normal Git merges/pushes preserve newer development commits, retry concurrent updates, and fail visibly on conflicts or denied pushes. Retries reuse the published version and assets.mainremains PR-only.The release pipeline has 18 integration tests, including real Git remotes, protected-main rejection, devel fast-forward/divergence, conflict recovery, denied writes, and a concurrent push. README, release documentation/diagram, installation notes, AGENTS and changelog describe the new flow. The release diagram and all eight runtime diagrams parse successfully.
Problem and behavior
A session timeout or manual interruption left the durable task
blocked/runningforever. Completing that session in the TUI did not notify the dispatcher, so it never verified/published the work and newly discovered issue comments remained queued across service restarts.Recheck stopped sessions, including legacy checkpoints, and return successful sessions to normal validation, configured checks, and publication. Preserve the existing branch/worktree/PR and process pending feedback only after that publication completes.
Add
/restartworkflow,opencode2-automation restartworkflow owner/repo#123, and the corresponding RPC. Recovery persists its intent, reconnects to existing execution, and continues the same stopped session when necessary. It does not duplicate prompts after uncertain responses, interrupt active work, bypass checks/questions/permissions, or clear saved work. Verification/publication failures retry their saved stage.Validation
npm run check: lint, typecheck, 157 tests, and build passed.npm run package:check: archive installation, loaders, SDK entrypoints, TUI, and CLI passed.Earlier server verification (recovery code only)
Installed the tested feature archive at commit
d9eb0d1on the affected host, backed up Git refs and session/queue state, and reloaded both configured owners while sessions were idle. No queue repair or manual publication was used. The dispatcher automatically reconciled the legacy timeout for issue #18, completed verification/publication of round 3, and started round 4 for pending comment5679024617, preservingthirst-for-levels, its saved worktree, and PR #19. Both automation plugins were active and both schedulers reported zero failures. The new implementation round remains in progress.Documentation
Reviewed all eight issue-to-PR diagrams against the implementation and updated their ordering, phase/status transitions, saved worktree behavior, automatic/manual recovery, media failure paths, verification/publication gates, merge polling and tab cleanup. Updated README and architecture/runtime/advanced/configuration/installation references, including command scope and the distinction between queued feedback and execution.
AGENTS.md and the bundled bot prompt now require affected descriptions, examples and diagrams to be updated in the same change, with documentation impact reviewed before completion. This requirement is an agent instruction; it is not an additional runtime gate.
Documentation validation: all eight diagrams passed the Mermaid parser, 66 local links (including section anchors) passed, Markdown diff checks passed, and three existing prompt/branch tests passed. The server verification above used code build
d9eb0d1; this later documentation/instruction commit is included in the PR for release.