workflow-task-finish
Skill1 starsChanged 6 months agoMITView on GitHub
Never declare done on red: verify, then record what you learned to the knowledge graph.
In this repository33 other files it gives its agents.
- skill-process-gotchasSkill
- workflow-cache-disciplineSkill
- workflow-codegraphSkill
- workflow-conductorSkill
- workflow-context-condenserSkill
- workflow-context-hygieneSkill
- workflow-criticSkill
- workflow-deadline-disciplineSkill
- workflow-deep-researchSkill
- workflow-done-definitionSkill
- workflow-edit-formatSkill
- workflow-effort-routingSkill
- workflow-escalateSkill
- workflow-execute-from-planSkill
- workflow-filesystem-offloadSkill
- workflow-learn-frameworkSkill
- workflow-long-running-processesSkill
- workflow-lspSkill
- workflow-memory-curationSkill
- workflow-plan-contractSkill
- workflow-plan-firstSkill
- workflow-planSkill
- workflow-quality-barSkill
- workflow-repomapSkill
- workflow-retrieve-before-actSkill
- workflow-spec-drivenSkill
- workflow-stuck-detect-resetSkill
- workflow-stuckSkill
- workflow-subagent-isolationSkill
- workflow-subtask-loopSkill
- workflow-task-startSkill
- workflow-tool-selectionSkill
- workflow-verify-enhancedSkill
The file
We split this skill into the rules it gives an agent
10 rules. About 0 are common, 4 are distinctive, and 6 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈4
Distinctive · estimate
This author's own choices. Worth a look.
6
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- could be automatic: a setting could enforce it
- Distinctive: line 19
Run this before reporting any coding task complete.
- Very specific: line 24
Verify (hard gate): Call verify(path) (mcp-verify) on the code you changed.
- Very specific: line 26
If passed is false, read the per-stage diagnostics, fix the cause, and re-run verify.
- Very specific: line 28
If the same failure persists after a few honest attempts, switch to the workflow-stuck skill instead of thrashing.
- Very specific: line 30
If verify is unavailable (server down), say so explicitly and fall back to running the project's lint/type/test commands yourself — but …
- Distinctive: line 40
Let a skill distill: If the task involved a novel, reusable technique, allow Hermes's skill-creation loop to capture it (don't force it.
- Distinctive: line 61
Do not let yourself off the hook.
- Very specific: line 69
Don't run history-destroying git commands (force-push, hard reset, checkout .
- Very specific: line 71
Create a fresh commit each time.
- Distinctive: line 74
Don't force-push a branch others share.
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
<!-- TRIGGERS WHEN: "Never declare done on red: verify, then record what you learned to the knowledge graph." version: 1.0.0 author: Hermes Max license: MIT platforms: [linux, maco -->Task Finish — the verification gate + knowledge capture
Run this before reporting any coding task complete. You may not declare "done" while verification is red.
Steps
- Verify (hard gate). Call
verify(path)(mcp-verify) on the code you changed. It runs lint → typecheck → unit tests.- If
passedis false, read the per-stage diagnostics, fix the cause, and re-runverify. Repeat until green. Do not report done on red. - If the same failure persists after a few honest attempts, switch to the
workflow-stuckskill instead of thrashing. - If
verifyis unavailable (server down), say so explicitly and fall back to running the project's lint/type/test commands yourself — but still do not claim success unattended without some green signal.
- If
- Record what you learned to mcp-knowledge-graph so the next session starts
ahead:
record_entityfor any new decision (type="decision"), bug (type="bug"), or component (type="file"|"service") — include a shortwhyin props.record_relationto link them, e.g.(decision)-[applies_to]->(file),(bug)-[fixed_in]->(commit),(service)-[depends_on]->(service).
- Let a skill distill. If the task involved a novel, reusable technique, allow Hermes's skill-creation loop to capture it (don't force it; the nudge will prompt you when warranted).
Verification — run it, show it, verdict it
Reading code is not verifying it, and an implementer model's own claim of "green" is not
evidence. For every criterion in the plan's VERIFICATION section, and every step's
DONE-WHEN, EITHER call run_done_when(command, expected_output | expected_exit_code)
(mcp-verify) — it executes the command in the verify sandbox and returns a verdict — OR run
the command yourself and record exactly:
- Check: what you are confirming
- Command: the exact command you ran
- Observed: the real terminal output, pasted verbatim (never summarized)
- Result: PASS, or FAIL with expected-vs-actual
Close each check with one literal line, exactly one of: VERDICT: PASS, VERDICT: FAIL,
VERDICT: PARTIAL. The harness reads that line; a FAIL/PARTIAL — or a missing verdict —
is not done and routes to retry/escalation.
Do not let yourself off the hook. "It looks right", "this is the standard way", and "the tests
the implementer wrote already pass" are not verification — execute the check or call it a skip.
Report honestly: a VERDICT: PARTIAL that names the unverified step is correct; a VERDICT:
PASS you did not actually observe is a failed task. For core logic a green exit is necessary
but not sufficient — also run verify_formal / property_test on the target.
Git safety (if you commit)
- Don't run history-destroying git commands (force-push, hard reset,
checkout ./clean/ branch deletion) unless the operator asked for that exact command. - Create a fresh commit each time; don't
--amend— a failed pre-commit hook means nothing was committed, so amending would overwrite the PREVIOUS (good) commit. - Stage files explicitly by path, not
-A/., so unrelated edits never ride along. - Don't force-push a branch others share.
Why
A gate that cannot be bypassed on broken code makes unattended operation trustworthy — reliability you can leave alone overnight. Recording decisions is what makes the second related task faster than the first.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

