feat: calibrate logging registers in the code-writing agents
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
@@ -17,6 +17,7 @@ enabled_skills:
|
|||||||
- code-review
|
- code-review
|
||||||
- comment-discipline
|
- comment-discipline
|
||||||
- diagnosing-bugs
|
- diagnosing-bugs
|
||||||
|
- logging-discipline
|
||||||
- git-master
|
- git-master
|
||||||
- frontend-ui-ux
|
- frontend-ui-ux
|
||||||
- verification-gates
|
- verification-gates
|
||||||
@@ -171,6 +172,7 @@ nodes:
|
|||||||
- code-review
|
- code-review
|
||||||
- comment-discipline
|
- comment-discipline
|
||||||
- diagnosing-bugs
|
- diagnosing-bugs
|
||||||
|
- logging-discipline
|
||||||
- git-master
|
- git-master
|
||||||
- frontend-ui-ux
|
- frontend-ui-ux
|
||||||
- verification-gates
|
- verification-gates
|
||||||
@@ -221,7 +223,12 @@ nodes:
|
|||||||
`comment-discipline` (self-documenting / api-documented /
|
`comment-discipline` (self-documenting / api-documented /
|
||||||
comment-heavy) and write comments to match. When the signal is
|
comment-heavy) and write comments to match. When the signal is
|
||||||
weak, write NO comment.
|
weak, write NO comment.
|
||||||
4. Follow the same patterns exactly. Do not invent new ones.
|
4. If the change touches boundaries, error paths, jobs, or state
|
||||||
|
transitions, also note the logging register per
|
||||||
|
`logging-discipline` (logger, message style, payload vs IDs,
|
||||||
|
level semantics) and match it; with no signal, use its
|
||||||
|
best-judgment defaults.
|
||||||
|
5. Follow the same patterns exactly. Do not invent new ones.
|
||||||
|
|
||||||
## Fix loop
|
## Fix loop
|
||||||
|
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
name: sisyphus
|
name: sisyphus
|
||||||
description: OpenCode-style orchestrator - classifies intent, delegates to specialists, tracks progress with todos, enforces OMO-grade verification discipline
|
description: OpenCode-style orchestrator - classifies intent, delegates to specialists, tracks progress with todos, enforces OMO-grade verification discipline
|
||||||
version: 3.7.0
|
version: 3.8.0
|
||||||
|
|
||||||
agent_session: temp
|
agent_session: temp
|
||||||
auto_continue: true
|
auto_continue: true
|
||||||
@@ -30,6 +30,7 @@ enabled_skills:
|
|||||||
- comment-discipline
|
- comment-discipline
|
||||||
- diagnosing-bugs
|
- diagnosing-bugs
|
||||||
- grilling
|
- grilling
|
||||||
|
- logging-discipline
|
||||||
- git-master
|
- git-master
|
||||||
- frontend-ui-ux
|
- frontend-ui-ux
|
||||||
- delegation-protocol
|
- delegation-protocol
|
||||||
@@ -391,6 +392,7 @@ instructions: |
|
|||||||
When you write or modify files yourself (rather than delegating to coder):
|
When you write or modify files yourself (rather than delegating to coder):
|
||||||
|
|
||||||
- **Calibrate comments before writing.** Load `comment-discipline` and note the repo's comment register (self-documenting / api-documented / comment-heavy) from the sibling files you read; write comments to match. When the signal is weak, write NO comment.
|
- **Calibrate comments before writing.** Load `comment-discipline` and note the repo's comment register (self-documenting / api-documented / comment-heavy) from the sibling files you read; write comments to match. When the signal is weak, write NO comment.
|
||||||
|
- **Calibrate logging before writing.** When the change touches boundaries, error paths, jobs, or state transitions, load `logging-discipline` and note the repo's logging register (logger, message style, payload vs IDs, level semantics) from the same sibling reads; match it. No discernible convention → its best-judgment defaults. Never leave a new error path silently swallowed, and never delete existing log lines as drive-by cleanup.
|
||||||
- **For editing an existing file**, prefer `fs_patch`. It's a surgical edit that preserves unchanged content. Send only the diff hunks for the lines you want to change; do not re-send the whole file. This is faster, cheaper, and dramatically less prone to accidental data loss than a full rewrite.
|
- **For editing an existing file**, prefer `fs_patch`. It's a surgical edit that preserves unchanged content. Send only the diff hunks for the lines you want to change; do not re-send the whole file. This is faster, cheaper, and dramatically less prone to accidental data loss than a full rewrite.
|
||||||
- **For writing a NEW file or doing a COMPLETE rewrite**, use `fs_write`. Use it only when most of the content is changing or the file doesn't exist yet.
|
- **For writing a NEW file or doing a COMPLETE rewrite**, use `fs_write`. Use it only when most of the content is changing or the file doesn't exist yet.
|
||||||
- **NEVER write files via `execute_command`.** Do not use:
|
- **NEVER write files via `execute_command`.** Do not use:
|
||||||
|
|||||||
Reference in New Issue
Block a user