Skip to content

Commit cf18463

Browse files
authored
Merge pull request solvcon#1339 from tigercosmos/worktree-fix-worktree-skill
Add Codex-specific project skills
2 parents 778186e + 5fe8555 commit cf18463

12 files changed

Lines changed: 92 additions & 1 deletion

File tree

.agents/skills

Lines changed: 0 additions & 1 deletion
This file was deleted.
Lines changed: 24 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,24 @@
1+
---
2+
name: code-review
3+
description: Review a diff, branch, path, commit, or pull request for actionable correctness defects. Use when the user invokes $code-review or asks for a code review. Keep the review read-only unless the user explicitly requests fixes or comments.
4+
---
5+
6+
# Code Review
7+
8+
Resolve the exact review target first. For current work, include staged,
9+
unstaged, and untracked changes. For a pull request, inspect its real base,
10+
head, diff, and relevant discussion.
11+
12+
Use code graph tools to trace changed symbols, callers, and invariants. Read the
13+
minimum surrounding code and tests needed to prove each finding. Focus on bugs,
14+
regressions, unsafe behavior, missing validation, and tests that fail to cover
15+
a material risk. Do not report style preferences or speculative concerns.
16+
17+
Rank findings by severity. For every finding, cite exact file and line evidence,
18+
describe the failing scenario, and explain the impact. If no actionable issue
19+
is found, say so explicitly and note any verification gap.
20+
21+
Remain read-only by default. Apply fixes or post review comments only when the
22+
user explicitly asks, then re-review the resulting live diff.
23+
24+
<!-- vim: set ff=unix fenc=utf8 et sw=4 ts=4 sts=4 tw=79: -->

.agents/skills/commit-code

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/commit-code

.agents/skills/cpp-style-review

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/cpp-style-review

.agents/skills/create-pr

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/create-pr

.agents/skills/ide-user-presets

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/ide-user-presets
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/prototype-with-devplan

.agents/skills/python-style-review

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/python-style-review

.agents/skills/serve-docs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
../../.claude/skills/serve-docs

.agents/skills/simplify/SKILL.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
---
2+
name: simplify
3+
description: Review recently changed code for reuse, simplicity, efficiency, and appropriate abstraction, then apply safe behavior-preserving cleanup. Use when the user invokes $simplify or asks to simplify or refine recent changes. Do not use for correctness review.
4+
---
5+
6+
# Simplify
7+
8+
Review the target the user names. Otherwise inspect tracked and untracked
9+
changes with `git status --short --untracked-files=all`, then read the relevant
10+
diffs and new files. If there is no changed code, report that and stop.
11+
12+
Check four things:
13+
14+
1. Reuse existing helpers instead of duplicating logic.
15+
2. Reduce unnecessary control flow, state, indirection, and comments.
16+
3. Remove avoidable work or allocation when the benefit is clear.
17+
4. Keep behavior at the right abstraction level for the surrounding code.
18+
19+
Use code graph tools for relationships and reuse discovery. Read the diff and
20+
the minimum surrounding code needed to validate each cleanup. Do not broaden
21+
the task into bug hunting, feature work, or unrelated refactoring.
22+
23+
Apply only changes that preserve intended behavior and make the code plainly
24+
simpler. Leave uncertain or subjective suggestions unapplied and report them.
25+
Keep edits inside changed code unless a directly reused helper requires a
26+
small adjacent update.
27+
28+
Run focused tests and relevant lint checks after editing. Invoke
29+
`$cpp-style-review` after C++ changes and `$python-style-review` after Python
30+
changes. Review the final diff and revert any cleanup that adds complexity or
31+
changes behavior. Do not commit or open a pull request unless asked.
32+
33+
Report the cleanups applied, verification results, and any suggestions left
34+
for the user.
35+
36+
<!-- vim: set ff=unix fenc=utf8 et sw=4 ts=4 sts=4 tw=79: -->

0 commit comments

Comments
 (0)