Sync generic engine improvements - #1
Merged
Merged
Conversation
Forward-ports the generic weft features developed privately, excluding all company-specific templates and tooling. Added: - weft doctor / wf-doctor: template drift + plugin clone currency - weft analyze / wf-analyze: per-template durations, loops, friction - scripts/wf.py curses TUI + wf-monitor.py plain-text/JSON viewer - unattended-park at human gates (WEFT_UNATTENDED) + executor steps - richer event model: wf.guard_blocked, wf.stalled, paired wf.step_changed, wf.failed - context.md surfacing (description, next action, suggest, insights) - codex-review template; wf-resume / wf-run-step / wf-template skills Fixed: - insights step field now populated into step state (was read by projections but never carried from the template) Changed: - workflow_id carries second resolution (name-YYYYMMDD-HHMMSS) Excluded (company-specific): amonit-triage + address-pr-feedback templates, command-deck, test_amonit_triage. Sanitized default paths and examples in wf-monitor.py, wf-analyze, README to neutral values. 263 tests pass.
dioptx
added a commit
that referenced
this pull request
Aug 14, 2026
…ing work Integrates the public 'Sync generic engine improvements from private (#1)' commit with the 4 local unpushed commits (resolve-finding template + per-run pre-accept guard allowlist). Clean auto-merge — only core/guard_engine.py overlapped and merged without conflict. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N6DLyhgaWctqCCEj2sRTcg
dioptx
added a commit
that referenced
this pull request
Aug 14, 2026
…branch Salvages the one genuinely-unique, still-valuable piece of the Mini's stale terminal-mode-fixes branch (which otherwise predates and conflicts with the public generic-engine sync). Reconciled against PR #1's newer Next-action system rather than ported wholesale: - cli.py: --help / -h / help / no-args now print full usage to stdout and exit 0 (was: 'Unknown command' to stderr, exit 1 — a wall for a first- time terminal user). Unknown commands now point at --help. USAGE lists the current command set incl. PR #1's run-result/resume/analyze/doctor. - projections.py: added _terminal_mode() (driven by CLAUDECODE, overridable via WEFT_TERMINAL). The existing 'Next action:' line and the command- reference footer now translate /weft:* slash commands to their cli.py equivalents when not running inside Claude Code — those slash commands don't exist in a plain shell. Footer is now running-only (a completed workflow no longer shows a stale 'advance' hint — origin's old bug). Deliberately NOT ported: the 482-line sonnet_terminal_drive.py behavioral harness (adds a live-Sonnet dependency to CI and tests an old cli.py) and the renamed/removed old skill files. Feature covered by 7 focused unit tests instead. 270 tests pass (263 + 7 new). Verified rendered output in both modes. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N6DLyhgaWctqCCEj2sRTcg
dioptx
added a commit
that referenced
this pull request
Aug 15, 2026
Forward-ports the generic weft features developed privately, excluding all company-specific templates and tooling. Added: - weft doctor / wf-doctor: template drift + plugin clone currency - weft analyze / wf-analyze: per-template durations, loops, friction - scripts/wf.py curses TUI + wf-monitor.py plain-text/JSON viewer - unattended-park at human gates (WEFT_UNATTENDED) + executor steps - richer event model: wf.guard_blocked, wf.stalled, paired wf.step_changed, wf.failed - context.md surfacing (description, next action, suggest, insights) - codex-review template; wf-resume / wf-run-step / wf-template skills Fixed: - insights step field now populated into step state (was read by projections but never carried from the template) Changed: - workflow_id carries second resolution (name-YYYYMMDD-HHMMSS) Excluded (company-specific): amonit-triage + address-pr-feedback templates, command-deck, test_amonit_triage. Sanitized default paths and examples in wf-monitor.py, wf-analyze, README to neutral values. 263 tests pass. Co-authored-by: Dennis <dennis@dune.com>
dioptx
added a commit
that referenced
this pull request
Aug 15, 2026
…ing work Integrates the public 'Sync generic engine improvements from private (#1)' commit with the 4 local unpushed commits (resolve-finding template + per-run pre-accept guard allowlist). Clean auto-merge — only core/guard_engine.py overlapped and merged without conflict. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N6DLyhgaWctqCCEj2sRTcg
dioptx
added a commit
that referenced
this pull request
Aug 15, 2026
…branch Salvages the one genuinely-unique, still-valuable piece of the Mini's stale terminal-mode-fixes branch (which otherwise predates and conflicts with the public generic-engine sync). Reconciled against PR #1's newer Next-action system rather than ported wholesale: - cli.py: --help / -h / help / no-args now print full usage to stdout and exit 0 (was: 'Unknown command' to stderr, exit 1 — a wall for a first- time terminal user). Unknown commands now point at --help. USAGE lists the current command set incl. PR #1's run-result/resume/analyze/doctor. - projections.py: added _terminal_mode() (driven by CLAUDECODE, overridable via WEFT_TERMINAL). The existing 'Next action:' line and the command- reference footer now translate /weft:* slash commands to their cli.py equivalents when not running inside Claude Code — those slash commands don't exist in a plain shell. Footer is now running-only (a completed workflow no longer shows a stale 'advance' hint — origin's old bug). Deliberately NOT ported: the 482-line sonnet_terminal_drive.py behavioral harness (adds a live-Sonnet dependency to CI and tests an old cli.py) and the renamed/removed old skill files. Feature covered by 7 focused unit tests instead. 270 tests pass (263 + 7 new). Verified rendered output in both modes. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01N6DLyhgaWctqCCEj2sRTcg
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
2 participants
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.
Summary
Forward-ports generic engine improvements.
weft doctor//wf-doctor— template drift + plugin clone currency (offline)weft analyze//wf-analyze— per-template durations, loop counts, guard blocks, frictionscripts/wf.pycurses TUI +wf-monitor.pyplain-text/JSON viewerWEFT_UNATTENDED) + Workflow executor stepswf.guard_blocked,wf.stalled, pairedwf.step_changed,wf.failedcontext.mdsurfacing: description, next action, suggested commands, insightswf-resume,wf-run-step,wf-template;codex-reviewtemplateFixed
insightsstep field is now populated into step state (was rendered by projections but never carried from the template).Changed
workflow_idcarries second resolution (name-YYYYMMDD-HHMMSS) so same-day runs stay distinct.Tests
263 passing.