summaryrefslogtreecommitdiff
path: root/prompts/skills/agent-task-management/references
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-06-28 23:20:48 +0300
committerPaul Buetow <paul@buetow.org>2026-06-28 23:20:48 +0300
commit5ad399e8de3ce070119d1f4960c3f1a3c9d682c2 (patch)
tree592ab2f01f39a048de964858cf20d32167e19f58 /prompts/skills/agent-task-management/references
parent30f5bb3689e0ae55756159406350399f1c06db6b (diff)
sync skills: add skill-maintenance skill + shared DRY references
- New skill: skill-maintenance (SKILL.md + 4 references: best-practices, dry-across-skills, sub-division, audit-checklist) encoding the Agent Skills spec rules, cross-skill DRY, and the f3s/rocky-vm-setup sub-division pattern. - agent-task-management/references/verification-honesty.md: canonical home for the general verification-honesty discipline (linked from go-best-practices). - blog-writing-style/references/gemtext-conventions.md: shared foo.zone gemtext conventions (linked from compose-blog-post, update-blog-post, blog-writing-style). - blog-writing-style/references/patterns-and-examples.md: foo.zone LLM-pattern catalog + rewrite examples extracted from blog-writing-style SKILL.md. - f3s/references/{console-jetkvm-shutdown,shelly-plug}.md: referenced by f3s/SKILL.md (previously untracked).
Diffstat (limited to 'prompts/skills/agent-task-management/references')
-rw-r--r--prompts/skills/agent-task-management/references/verification-honesty.md57
1 files changed, 57 insertions, 0 deletions
diff --git a/prompts/skills/agent-task-management/references/verification-honesty.md b/prompts/skills/agent-task-management/references/verification-honesty.md
new file mode 100644
index 0000000..e083e92
--- /dev/null
+++ b/prompts/skills/agent-task-management/references/verification-honesty.md
@@ -0,0 +1,57 @@
+# Verification Honesty
+
+General discipline for claiming that work is verified. This is task-lifecycle
+policy, not specific to any language — language skills (e.g. `go-best-practices`)
+link here and add only their toolchain-specific concrete actions.
+
+The core rule: **be explicit about what actually ran.** A *missing or incomplete*
+toolchain (not slow tests) is the most common cause of false "verified" claims.
+
+## Before claiming any verification
+
+Confirm the local toolchain can actually build and run the relevant tests.
+
+- **Preflight first.** Verify the build/test path works before trusting it —
+ run the project's build command, confirm required native headers / CGO
+ dependencies are present, and confirm required external tools are on `PATH`.
+ If preflight fails, do not claim the project builds.
+- **Run the smallest verifying subset that DOES work.** When part of the
+ toolchain is missing, still verify what you can — linters, formatters,
+ `build` on the packages that do not need the missing headers, and the unit
+ tests that do not require the missing tool. Use build tags or explicit
+ package paths to skip the unbuildable parts.
+- **Annotate the blocker explicitly.** Record what is missing and the impact
+ with `ask annotate <id> "<note>"` — name the missing header/tool, what you
+ verified, and what you could not.
+- **Never claim full verification when it did not run.** State precisely what
+ was and was not verified (e.g. "vet + gofmt clean; package X not built —
+ header Y missing; tests for X not run"). Do not imply a green build or
+ passing tests that never executed.
+
+## Long-running / timeout-exceeding test suites
+
+Distinct from a *missing* toolchain: here the toolchain works, but the full
+suite is too slow to finish within the command timeout. Run a focused subset
+rather than nothing, and be explicit that you did so.
+
+- **Run a representative subset within the timeout.** Scope to the package(s)
+ the change touches and skip the slow target. Prefer a "short" flag (have slow
+ tests honor it), build tags, or a run-pattern filter to exclude expensive
+ integration/E2E tests; run the unit subset when the full integration suite
+ cannot complete.
+- **Annotate the intentional skip.** Record with `ask annotate <id> "<note>"`
+ that the full suite was *intentionally* skipped, why (exceeds timeout, not a
+ failure), which subset ran, and the result — e.g. "integration suite skipped
+ (>30m, timeout); ran unit subset for package X → pass".
+- **Acceptance implications.** A focused subset is NOT full verification. Be
+ explicit about residual risk — untested integration paths, packages not
+ touched — so the reviewer/orchestrator can decide whether to accept or run
+ the full suite out-of-band. Never imply the full suite passed when it never ran.
+
+## Summary checklist
+
+- [ ] Toolchain preflight passed before claiming a build
+- [ ] The smallest verifying subset that works was run
+- [ ] Missing headers/tools named explicitly in an `ask annotate` note
+- [ ] No implication that unrun tests passed
+- [ ] Slow suites: subset run, skip annotated, residual risk stated \ No newline at end of file