Files
harness-tools/docs/workflow/workflow-automation-record.md

7.0 KiB

Risk-based workflow automation record

Status

  • Status: complete
  • Current phase: integrated locally
  • Next safe action: none for this implementation; retain the separate execution-pilot TODO.

Request and scope

  • Request: reduce routine human interaction and redundant testing, especially for replaceable pre-production data.
  • In scope: develop-workflow skill and references/template; code-review skill; ADR and historical pointers; validation evidence and existing TODO alignment.
  • Out of scope: extension runtime/config changes, global instructions, database access, archived engines, checkpoint format changes, replacement of outstanding twelve-scenario/three-pilot validation.

Git identity

  • Source branch: main
  • Base commit: 284fdd638d
  • Feature branch: feature/workflow-automation
  • Worktree: /home/jb/code/harness-tools/.worktrees/workflow-automation
  • Omitted source changes: none; source checkout clean.

Acceptance criteria

  • Routine work proceeds from a recorded plan without an approval pause; substantive choices and protected actions remain gated.
  • Disposable-data policy requires named authorized targets, identity checks, scope and reproducible reset/reseed; pre-production alone grants nothing.
  • Focused tests and review precede one final full verification boundary; valid evidence can be reused and invalidated evidence is rerun.
  • One non-author reviewer covers standards and specification; targeted fixes do not force fresh full reviews.
  • Records remain recoverable with existing heading contract; no initial/per-gate/closure commit ceremony is mandatory.
  • UI direction, real interface validation, worktree isolation, honest results and safe integration remain intact.
  • No new extension, dependency, runtime behavior or user-level policy change.

Approved implementation plan and gates

  • Plan/specification source: user-approved /tmp/pi-present/develop-workflow-automation-review.md and the acceptance criteria above.
  • Exact plan: revise workflow and supporting references, consolidate review policy in code-review, retain checkpoint heading schema, add ADR 0008 and supersession pointers, run focused checks and scenario walkthroughs, obtain one independent review, run final full suite/static checks, commit and integrate locally.
  • Plan approval: user said "approved, let's proceed and implement this plan." on 07OCT2026. This explicitly authorizes the risk-based policy and its use for this revision; no redundant re-approval requested.
  • UI classification and direction approval: documentation-only skill behavior; no product UI change.
  • Risk decisions and scope: no database/environment declared disposable by this change; no dependency, security-posture, production, publishing or deployment changes. Existing external native-hooks default config directory is absent; no installed policy inferred or modified.

Ownership

  • Parent agent owns all edits under skills/develop/develop-workflow/, skills/develop/code-review/SKILL.md, docs/adr/0008-risk-based-development-workflow-07OCT2026.md, supersession pointers in ADR 0006 and the accepted workflow plan, docs/todos.md, and this record.
  • Independent reviewer owns no files and cannot remediate. No mutating child assignments.

Verification matrix

Surface Command or method Exit code Outcome
Checkpoint compatibility node --test skills/develop/develop-workflow/tests/checkpoint-record.test.mjs 0 13/13 passed; existing schema retained
Policy consistency Node validateRecord on template/record, Python local-link validation, git diff --check 0 Both heading contracts valid; 18 links across 9 policy files resolve; whitespace clean. No prose snapshot tests added.
Scenarios independent policy walkthrough: routine feature, disposable/protected data, targeted review fix/shared-code change, non-trivial UI complete Reviewer confirmed all four intended routes. These are policy walkthroughs, not execution pilots; existing validation TODO remains open.
Final repository suite bun run test 0 Entire repository script completed after review, 25 seconds, tested product 0cc0e82 from the isolated worktree. One full-suite run.
Final static check bun run typecheck 0 Passed against product 0cc0e82 after frozen dependency restore. Initial missing-dependency failure retained below.

Review

  • Base and product HEAD: 284fdd638d / 0cc0e82bbc
  • Clean status and tree hash: clean; 2798112031
  • Commit list: 0cc0e82 Make development workflow risk-based and verification proportionate
  • Exact diff command: git diff --binary --full-index --find-renames 284fdd638d1f2c790290491e0cfe2802880cec25...0cc0e82bbc380a3e5f2d8c689098ae9166d00302 --
  • SHA-256: 24ebf984eda2c74298d9865dfe61248502d40c2060e082f0234574c2279b804e
  • Mutator identities: parent agent only
  • Reviewer identity: parent-launched workflow-policy-reviewer using client-import-reviewer-01a0e83c read-only profile; child/session IDs not exposed. No overlap with sole mutator.
  • Verdict: regenerated digest confirmed; Standards approved; Specification approved; all four policy walkthroughs matched requirements.
  • Findings and remediation: no material findings, no remediation cycles.
  • Before/after reviewer Git evidence: clean status, HEAD, tree, and pinned diff digest unchanged, independently rechecked by parent.

Integration and rollback

  • Synchronization and re-verification: git fetch origin completed with exit 0; origin/main and source main both remain 284fdd638d. No rebase delta.
  • Fast-forward result: main advanced cleanly from 284fdd6 to reviewed product 0cc0e82 with --ff-only.
  • Product and evidence commits: 0cc0e82; subsequent record-only evidence commit contains this completed report. No product inputs changed after final verification.
  • Reachability and cleanup: parent proved product reachable from main and both checkouts clean, then removed only .worktrees/workflow-automation and feature/workflow-automation. The unrelated feature-jev-tools worktree was preserved. Final evidence retained directly on main after cleanup; no push or deployment.
  • Rollback strategy: revert the focused policy commit; no data or runtime migration.

Checkpoints and blockers

  • Last committed valid checkpoint: product commit 0cc0e82 contains the initial implementation/review-pending record; this evidence-only update retains final outcomes.
  • Environment recovery: initial typecheck exited 1 because the new worktree lacked package-local dependencies (including effect). bun install --frozen-lockfile restored 590 packages with exit 0 and no tracked changes. Exact Effect 4.0.0-beta.101 and TypeScript 7.0.2 sources already cached under /srv/data/opensrc; no source download or dependency-version change.
  • Blockers: none
  • Exact next safe action: no further implementation action required. Twelve execution scenarios and three real-change pilots remain tracked separately; this revision did not claim them complete.