lsm / neokai / 24903226462

82%

Build Type

push

github

Committed by web-flow

Commit Message

feat(space): populate postApproval on built-in workflows + enable flag (PR 3/5) (#1623)

* feat(space): add shared post-approval merge-template + hash fingerprint

Introduces PR_MERGE_POST_APPROVAL_INSTRUCTIONS — the single source of truth
for "merge the PR after approval" instructions that every built-in workflow
pointing at the reviewer will reuse verbatim. Extends the workflow template
fingerprint to include postApproval (targetAgent + instructions) so future
edits to the template trigger a re-stamp rather than silently drifting.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* feat(space): rewrite built-in-workflow end nodes + seeder re-stamp

Rewrites the end-node prompts on Coding / Research / Fullstack-QA-Loop /
Review-Only so the task agent issues an explicit handoff signal
(send_message to task-agent with post_approval_action) before calling
approve_task. The QA prompt no longer runs `gh pr merge` or worktree-sync
steps — those move under the reviewer's instructions template. Review-Only
drops its "runtime verifies" boilerplate; Plan-and-Decompose is unchanged.

Populates postApproval.targetAgent='reviewer' with the shared merge-template
on the three PR-producing workflows, and drops QA's completionAutonomyLevel
from 4 → 3 (approval is now needed because the reviewer executes the merge).

Teaches seedBuiltInWorkflows to narrowly re-stamp existing template rows
when the stored templateHash drifts from the current template — updating
only postApproval, completionAutonomyLevel, and templateHash so in-flight
workflow runs keep referencing the same node UUIDs. Exposes the re-stamped
names through SeedBuiltInWorkflowsResult.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

* feat(space): auto-restamp built-in workflows on daemon startup

Adds restampBuiltInWorkflowsOnStartup — iterates every space and invokes
seedBuiltInWorkflows so drifted template rows pick up the current
postApproval/completionAutonomyLevel/templateHash befor... (continued)

Coverage Stats

9101 of 12379 branches covered (73.52%)

Branch coverage included in aggregate %.

210 of 240 new or added lines in 7 files covered. (87.5%)

3287 existing lines in 12 files now uncovered.

66721 of 79159 relevant lines covered (84.29%)

296.25 hits per line