Skip to content

Commit dd0de62

Browse files
committed
Adjust final-release-review skill
1 parent e0b243d commit dd0de62

1 file changed

Lines changed: 37 additions & 1 deletion

File tree

  • .codex/skills/final-release-review

.codex/skills/final-release-review/SKILL.md

Lines changed: 37 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,9 @@ Use this skill when validating main for release. It guides you to fetch remote t
3333
- Run the quick-start tag command to ensure you use the latest remote tag. If the tag pattern differs, override the pattern argument (e.g., `'*.*.*'`).
3434
- If the user specifies a base tag, prefer it but still fetch remote tags first.
3535
- Keep the working tree clean to avoid diff noise.
36+
- **Assumptions**
37+
- Assume `main` has already passed `$code-change-verification` in CI unless the user says otherwise.
38+
- Do not block a release solely because you did not run tests locally; focus on concrete behavioral or API risks.
3639
- **Map the diff**
3740
- Use `--stat`, `--dirstat`, and `--name-status` outputs to spot hot directories and file types.
3841
- For suspicious files, prefer `git diff --word-diff BASE...TARGET -- <path>`.
@@ -47,7 +50,40 @@ Use this skill when validating main for release. It guides you to fetch remote t
4750
- List: breaking-change candidates, probable regressions/bugs, improvement opportunities, missing release notes/migrations.
4851
- Recommend ship/block and the exact checks needed to unblock if blocking.
4952

50-
## Resources
53+
## Output format (required)
54+
55+
All output must be in English.
56+
57+
Use the following report structure in every response produced by this skill. Be proactive and decisive: make a clear ship/block call near the top, and assign an explicit risk level (LOW/MODERATE/HIGH) to each finding with a short impact statement. Avoid overly cautious hedging when the risk is low and tests passed.
58+
59+
```
60+
### Release readiness review (<tag> -> TARGET <ref>)
61+
62+
This is a release readiness report done by `$final-release-review` skill.
63+
64+
### Diff
65+
66+
https://github.com/openai/openai-agents-python/compare/<tag>...main
67+
68+
### Release call:
69+
- <GREEN LIGHT TO SHIP | BLOCKED> <one-line rationale>
70+
71+
### Scope summary:
72+
- <N files changed (+A/-D); key areas touched: ...>
73+
74+
### Risk assessment (ordered by impact):
75+
1) <Finding title>
76+
- Risk: <LOW/MODERATE/HIGH>. <Impact statement in one sentence.>
77+
- Files: <path(s)>
78+
2) ...
79+
80+
### Notes:
81+
- <working tree status, tag/target assumptions, or re-run guidance>
82+
```
83+
84+
If no risks are found, include a “No material risks identified” line under Risk assessment and still provide a ship call. If you did not run local verification, do not add a verification status section or use it as a release blocker; note any assumptions briefly in Notes.
85+
86+
### Resources
5187

5288
- `scripts/find_latest_release_tag.sh`: Fetches remote tags and returns the newest tag matching a pattern (default `v*`).
5389
- `references/review-checklist.md`: Detailed signals and commands for spotting breaking changes, regressions, and release polish gaps.

0 commit comments

Comments
 (0)