stefano
251148c3ff
## Summary - add repository-wide quality tooling and verification scaffolding, including CI workflows, pnpm workspace setup, ESLint/Prettier/markdown checks, and generated-output verification helpers - reorganize skill sources and generation flow by introducing canonical `_source` variants, generator/manifests, reusable helper abstractions, and shared web-automation/browser utilities - clean up and expand documentation so the root README flows into docs and skill docs, with clearer development, reviewer, installer, and workflow guidance ## Notable changes - docs flow and consistency cleanup across `README.md`, `docs/README.md`, and related docs - new scripts for `check`, docs verification, generated-file verification, shell portability, and safe directory replacement - refactors in Atlassian and web-automation skill runtimes to reduce duplication and centralize reusable code - changelog, development documentation, and CI surface updates ## Test Plan - [ ] `pnpm run check` - [ ] review generated/manifests and skill sync outputs - [ ] smoke-check docs flow from `README.md` to `docs/README.md` to skill docs ## Notes - this branch currently includes tracked `skills/web-automation/shared/node_modules` content that should be reviewed carefully as potentially noisy/accidental committed artifacts Co-authored-by: Stefano Fiorini <stefano.fiorini@firsthorizon.com> Reviewed-on: #1
4.5 KiB
4.5 KiB
Task Plan: [Short Title]
Variant guardrail (Claude Code): When generating or updating this file, the agent MUST be out of plan mode. Sub-skills (
brainstorming,test-driven-development,verification-before-completion,finishing-a-development-branch,using-git-worktrees) MUST be invoked through theSkilltool explicitly — no shell wrappers.
Metadata
| Field | Value |
|---|---|
| Created | YYYY-MM-DD |
| Slug | YYYY-MM-DD- |
| Runtime | claude-code |
| Reviewer CLI | codex | claude | cursor | opencode | pi |
| Reviewer Model | |
| MAX_ROUNDS | 10 |
| Branch Strategy | current-branch | worktree |
| Branch Name | <current branch name, or new branch name when worktree is used> |
| Worktree Path | <absolute path to worktree dir; blank when Branch Strategy = current-branch> |
| Status | draft |
Status Enum (authoritative)
| Value | Meaning |
|---|---|
draft |
Newly created; plan review not yet started |
plan-approved |
Plan review loop returned APPROVED |
implementation-in-progress |
Phase 6 executing |
implementation-approved |
Phase 8 review loop returned APPROVED; awaiting commit |
pushed |
Committed + pushed to remote |
local-only |
Committed locally; user declined push |
aborted-plan-review |
MAX_ROUNDS reached in Phase 5; user aborted |
aborted-impl-review |
MAX_ROUNDS reached in Phase 8; user aborted |
aborted-verification |
Phase 7 retries exhausted; user aborted |
failed |
Hard tooling failure |
Prompt
Interpretation
Assumptions
Files
| Action | Path | Why |
|---|---|---|
Approach
TDD Approach
Acceptance Criteria
- <criterion 1>
- <criterion 2>
Verification
Rollback
Runtime State
plan_review_round: 0
implementation_review_round: 0
CODEX_PLAN_SESSION_ID:
CODEX_IMPL_SESSION_ID:
CURSOR_PLAN_SESSION_ID:
CURSOR_IMPL_SESSION_ID:
OPENCODE_PLAN_SESSION_ID:
OPENCODE_IMPL_SESSION_ID:
last_phase_entered:
last_round_ts:
last_scan_outcome_plan:
last_scan_outcome_impl:
verification_attempts: 0
tests_added_count: 0
tdd_used: false
Review History
| Timestamp (ISO-8601) | Loop | Round | Verdict | Summary |
|---|---|---|---|---|
Final Status
Guardrails (do NOT remove)
- This file is the single persistent artifact for
do-task. Do not split it or delete it on success. Statusmust always match one of the 10 enum values.Runtime Stateis updated by the skill, not by the user.- Review History is append-only.
last_scan_outcome_planandlast_scan_outcome_implrecord the most recent secret-scan result for each loop. They are informational; the scan itself runs per-payload with no caching.