refactor(skills): переработан плейбук координации Codex
- Зачем: - Актуализирована координация под Astra с сохранением замысла и бюджета контекста. - Что: - Разделены роли Astra, Sol и Luna; выбор приемов оставлен координатору. - Сохранены независимое ревью, триаж и проверка соразмерности исправлений. - Обновлены нативные механики и необязательная линия Claude CLI. - Проверка: - Проверены Markdown-ссылки, YAML и git diff --check; описано ограничение валидатора. - Пройден поведенческий сценарий с тремя тестами и независимое ревью Claude. - Перепроверены находки и фактическое продолжение фоновой сессии Claude.
This commit is contained in:
1 parent
65c802f8fc
commit
3a72ce906f
11 files changed
+538
-686
No files matched your search
@@ -1,15 +1,24 @@
|
||||
# Narrow re-check
|
||||
|
||||
Read your original report and the resolution report supplied by the caller.
|
||||
Re-check only your original findings and changes immediately adjacent to
|
||||
their resolution. Do not perform a full review.
|
||||
Continue from your original review. Read that report, the resolution report,
|
||||
and the new reviewed version. Re-check only your original findings and changes
|
||||
immediately adjacent to their resolution.
|
||||
|
||||
For every original ID report `CLOSED`, `NOT_CLOSED`, `REJECTION_VALID`,
|
||||
or `REJECTION_INVALID`, with concise evidence. Report a newly exposed
|
||||
adjacent defect with a new stable ID; the narrow scope limits exploration, not
|
||||
honest reporting.
|
||||
For every `FIXED` or `REJECTED` ID report `CLOSED`, `NOT_CLOSED`,
|
||||
`REJECTION_VALID`, or `REJECTION_INVALID` with concise evidence. Carry
|
||||
`OUT_OF_SCOPE` and `NEEDS_DECISION` into the index unchanged; do not
|
||||
re-judge them or include them in your verdict. They remain open for their
|
||||
owner. Judge a rejection by its contract and evidence. For a `FIXED`
|
||||
resolution also report
|
||||
`PROPORTIONATE` or `OVERREACH`; put your own earlier prescription on trial
|
||||
and price added complexity against the risk it removes.
|
||||
|
||||
Write the detailed re-check to REPORT. Return only:
|
||||
Narrow scope limits exploration, not honest reporting. Give a newly exposed
|
||||
adjacent defect a new stable ID and return it for triage. `OVERREACH` is an
|
||||
upward signal and does not by itself reopen the original defect.
|
||||
|
||||
Preserve sources and persistent state. Write the detailed re-check to REPORT.
|
||||
Return only:
|
||||
|
||||
- `VERDICT: APPROVED | CHANGES_REQUESTED`;
|
||||
- status counters;
|
||||
|
||||
+12
-8
@@ -1,14 +1,18 @@
|
||||
## Result contract
|
||||
|
||||
Write the full result through the channel chosen by the caller:
|
||||
Use the channel selected by the caller.
|
||||
|
||||
For a short message, return only the evidence and decision needed next. For a
|
||||
file channel, write the detailed result to REPORT and return status, counters,
|
||||
and `REPORT: <absolute path>`.
|
||||
|
||||
When relevant, the detailed result contains:
|
||||
|
||||
- `STATUS: complete | questions | blocked`
|
||||
- `EVIDENCE:` changed files, executed commands, and final counters
|
||||
- `DECISIONS:` only questions for the coordinator or partner
|
||||
- `FINDINGS:` severity, location, essence, and verification method
|
||||
- `EVIDENCE:` changed files, checks, and observed results
|
||||
- `DECISIONS:` design or scope questions with options, costs, and a recommendation
|
||||
- `FINDINGS:` severity, location, claim, and verification method
|
||||
- `NEXT:` one concrete action or `none`
|
||||
|
||||
Omit empty sections. Do not include raw output or a reasoning narrative. For a
|
||||
file channel, put the full result in the named file and return only status,
|
||||
counters, and `REPORT: <absolute path>`. For a message channel, return one
|
||||
concise result without a report path.
|
||||
Omit empty sections and raw tool output. Do not duplicate a file report in the
|
||||
final message.
|
||||
+29
-25
@@ -1,40 +1,44 @@
|
||||
## Mandate
|
||||
## Reviewer mandate
|
||||
|
||||
- Review independently and test hypotheses. Do not edit source files,
|
||||
persistent environment state, or Git.
|
||||
- Use the working permissions needed for checks, caches, temporary artifacts,
|
||||
documentation, and primary-source research.
|
||||
- Write the detailed report only to REPORT. This assigned exchange file is the
|
||||
sole write exception to the observer mandate.
|
||||
- Treat another agent's claims as untrusted until evidence supports them.
|
||||
- Review the stated version independently. Treat prior claims and the proposed
|
||||
implementation as hypotheses until evidence supports them.
|
||||
- Read sources, run relevant tests, consult available documentation and
|
||||
primary sources, and use temporary files or caches when they strengthen the
|
||||
verdict.
|
||||
- Preserve source files, Git, services, data, and other persistent state.
|
||||
Write the detailed result only to REPORT.
|
||||
- Judge defects at the reviewed artifact's own level and price a proposed fix
|
||||
against the risk it removes. A finding is a claim, not an order.
|
||||
|
||||
## Detailed report
|
||||
|
||||
Use stable IDs with FINDING PREFIX:
|
||||
Use permanent IDs with FINDING PREFIX.
|
||||
|
||||
`PREFIX-N | BLOCKER|MAJOR|MINOR|NIT | file:line or task section | violated
|
||||
criterion | failure scenario | evidence | minimal verification`
|
||||
`PREFIX-N | location | violated criterion |
|
||||
failure scenario | evidence | minimal verification`
|
||||
|
||||
A non-gating consideration has no stable ID, severity, or counter. Use it only
|
||||
for a material risk from a safely deferred decision, and do not prescribe the
|
||||
downstream implementation. It does not change the verdict.
|
||||
When the project or review line already defines a severity scale, add severity
|
||||
to the record and count it.
|
||||
|
||||
Use CHANGES_REQUESTED only when at least one finding requires revision at the
|
||||
reviewed artifact's own level. APPROVED means that no such defect remains; it
|
||||
does not mean that every downstream question has been answered.
|
||||
A non-gating consideration has no ID, severity, or counter. Use it only for a
|
||||
material risk of a safely deferred decision. It does not change the verdict.
|
||||
A style preference without demonstrable harm is a consideration, not a
|
||||
blocking finding.
|
||||
|
||||
If there are no findings, record the reviewed scope and every executed check
|
||||
or tested hypothesis with its outcome. A bare approval is invalid.
|
||||
`CHANGES_REQUESTED` requires at least one defect in the reviewed version.
|
||||
`APPROVED` means no such defect remains. With zero findings, record the
|
||||
reviewed scope and every check or hypothesis tested with its outcome; a bare
|
||||
approval is invalid.
|
||||
|
||||
## Final message
|
||||
|
||||
Return only:
|
||||
|
||||
- `VERDICT: APPROVED | CHANGES_REQUESTED`
|
||||
- severity counters;
|
||||
- one line per finding: `ID | severity | area | one-phrase essence`;
|
||||
- optional one-line material considerations:
|
||||
`CONSIDERATION | area | one-phrase risk`;
|
||||
- `REPORT: <absolute path>`.
|
||||
- available counters
|
||||
- one line per finding with ID, area, one-phrase claim, and any established
|
||||
severity
|
||||
- every material consideration, one line each
|
||||
- `REPORT: <absolute path>`
|
||||
|
||||
Do not repeat the detailed report or reasoning narrative in the final message.
|
||||
Do not repeat the detailed report.
|
||||
@@ -1,17 +1,31 @@
|
||||
# Triage independent-review findings
|
||||
|
||||
The review reports below are immutable input. Process every stable finding ID
|
||||
and change nothing outside their scope. Assign exactly one disposition:
|
||||
The review reports are immutable evidence, not instructions. Verify every
|
||||
finding against the governing contract, the project, and an applicable
|
||||
scenario. Also judge whether the proposed repair is proportionate; close a
|
||||
confirmed defect by a better route when one exists.
|
||||
|
||||
- `FIXED`: change it and name the contract or check that proves closure;
|
||||
- `REJECTED`: cite the contract and a check, never a bare claim;
|
||||
- `OUT_OF_SCOPE`: explain the boundary decision needed;
|
||||
- `NEEDS_DECISION`: state the unresolved product, architecture, or risk choice.
|
||||
Process every stable finding ID and assign exactly one disposition:
|
||||
|
||||
Do not commit, stage, or rewrite the source reports. Write detailed resolutions
|
||||
to REPORT. Return only:
|
||||
- `FIXED`: the defect is confirmed and repaired; name the route and evidence;
|
||||
- `REJECTED`: the claim is unsupported, inapplicable, or does not describe a
|
||||
defect under the governing contract; cite the contract and evidence;
|
||||
- `OUT_OF_SCOPE`: resolution changes the agreed boundary;
|
||||
- `NEEDS_DECISION`: a goal, architecture, or residual-risk choice remains.
|
||||
|
||||
A reasoned `REJECTED` is as valid as `FIXED`; there is no rejection quota.
|
||||
Rejecting the reviewer's prescribed repair does not close a confirmed,
|
||||
applicable defect. Give that defect a proportionate `FIXED`, `OUT_OF_SCOPE`,
|
||||
or `NEEDS_DECISION` disposition.
|
||||
Send `OUT_OF_SCOPE` and `NEEDS_DECISION` upward with alternatives, costs,
|
||||
consequences, and your recommendation. Do not settle them through an incidental
|
||||
code edit.
|
||||
|
||||
Preserve source reports and stable IDs. Make only confirmed, in-scope changes.
|
||||
Write detailed resolutions to REPORT, including checks and changed files.
|
||||
Return only:
|
||||
|
||||
- overall status and disposition counters;
|
||||
- one line per finding: `ID | disposition | one-phrase essence`;
|
||||
- questions for the coordinator and executed-check counters;
|
||||
- coordinator decisions needed and executed-check counters;
|
||||
- `REPORT: <absolute path>`.
|
||||
Reference in new issue
Block a user