fix(hephaestus): remove auto-commit policy to prevent surprise commits
The auto-commit section instructed Hephaestus to automatically commit after implementation work. Users who didn't know about this behavior would get surprise commits — a trust-breaking behavioral change flagged by 5 Oracle reviews as the sole publish blocker for 3.9.0.
This commit is contained in:
parent
76cba9b222
commit
bdd86b1415
@ -24,19 +24,7 @@
|
|||||||
"disabled_agents": {
|
"disabled_agents": {
|
||||||
"type": "array",
|
"type": "array",
|
||||||
"items": {
|
"items": {
|
||||||
"type": "string",
|
"type": "string"
|
||||||
"enum": [
|
|
||||||
"sisyphus",
|
|
||||||
"hephaestus",
|
|
||||||
"prometheus",
|
|
||||||
"oracle",
|
|
||||||
"librarian",
|
|
||||||
"explore",
|
|
||||||
"multimodal-looker",
|
|
||||||
"metis",
|
|
||||||
"momus",
|
|
||||||
"atlas"
|
|
||||||
]
|
|
||||||
}
|
}
|
||||||
},
|
},
|
||||||
"disabled_skills": {
|
"disabled_skills": {
|
||||||
|
|||||||
@ -448,21 +448,6 @@ ${oracleSection}
|
|||||||
4. **Run build** if applicable — exit code 0 required
|
4. **Run build** if applicable — exit code 0 required
|
||||||
5. **Tell user** what you verified and the results — keep it clear and helpful
|
5. **Tell user** what you verified and the results — keep it clear and helpful
|
||||||
|
|
||||||
### Auto-Commit Policy (MANDATORY for implementation/fix work)
|
|
||||||
|
|
||||||
1. **Auto-commit after implementation is complete** when the task includes feature/fix code changes
|
|
||||||
2. **Commit ONLY after verification gates pass**:
|
|
||||||
- \`lsp_diagnostics\` clean on all modified files
|
|
||||||
- Related tests pass
|
|
||||||
- Typecheck/build pass when applicable
|
|
||||||
3. **If any gate fails, DO NOT commit** — fix issues first, re-run verification, then commit
|
|
||||||
4. **Use Conventional Commits format** with meaningful intent-focused messages:
|
|
||||||
- \`feat(scope): add ...\` for new functionality
|
|
||||||
- \`fix(scope): resolve ...\` for bug fixes
|
|
||||||
- \`refactor(scope): simplify ...\` for internal restructuring
|
|
||||||
5. **Do not make placeholder commits** (\`wip\`, \`temp\`, \`update\`) or commit unverified code
|
|
||||||
6. **If user explicitly says not to commit**, skip commit and report that changes are left uncommitted
|
|
||||||
|
|
||||||
- **File edit** — \`lsp_diagnostics\` clean
|
- **File edit** — \`lsp_diagnostics\` clean
|
||||||
- **Build** — Exit code 0
|
- **Build** — Exit code 0
|
||||||
- **Tests** — Pass (or pre-existing failures noted)
|
- **Tests** — Pass (or pre-existing failures noted)
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user