oh-my-customcodex 0.4.11 → 0.4.13
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/cli/index.js +6 -1
- package/dist/index.js +6 -1
- package/package.json +6 -1
- package/templates/.claude/agents/mgr-creator.md +1 -1
- package/templates/.claude/agents/mgr-gitnerd.md +1 -1
- package/templates/.claude/agents/sys-memory-keeper.md +1 -1
- package/templates/.claude/agents/tracker-checkpoint.md +1 -1
- package/templates/.claude/agents/wiki-curator.md +1 -1
- package/templates/.claude/rules/MUST-agent-design.md +21 -16
- package/templates/.claude/rules/MUST-orchestrator-coordination.md +13 -0
- package/templates/.claude/rules/MUST-permissions.md +15 -0
- package/templates/.claude/skills/action-validator/SKILL.md +41 -0
- package/templates/.claude/skills/adaptive-harness/SKILL.md +5 -0
- package/templates/.claude/skills/agora/SKILL.md +1 -1
- package/templates/.claude/skills/dag-orchestration/SKILL.md +1 -1
- package/templates/.claude/skills/de-lead-routing/SKILL.md +1 -1
- package/templates/.claude/skills/deep-plan/SKILL.md +38 -322
- package/templates/.claude/skills/deep-verify/SKILL.md +1 -1
- package/templates/.claude/skills/dev-lead-routing/SKILL.md +1 -1
- package/templates/.claude/skills/dev-review/SKILL.md +5 -0
- package/templates/.claude/skills/hada-scout/SKILL.md +1 -1
- package/templates/.claude/skills/harness-eval/SKILL.md +1 -1
- package/templates/.claude/skills/harness-synthesizer/SKILL.md +5 -0
- package/templates/.claude/skills/omcodex-improve-report/SKILL.md +1 -1
- package/templates/.claude/skills/omcodex-takeover/SKILL.md +1 -1
- package/templates/.claude/skills/optimize-analyze/SKILL.md +1 -1
- package/templates/.claude/skills/optimize-report/SKILL.md +1 -1
- package/templates/.claude/skills/post-release-followup/SKILL.md +10 -5
- package/templates/.claude/skills/professor-triage/SKILL.md +55 -288
- package/templates/.claude/skills/qa-lead-routing/SKILL.md +1 -1
- package/templates/.claude/skills/research/SKILL.md +6 -1
- package/templates/.claude/skills/result-aggregation/SKILL.md +4 -0
- package/templates/.claude/skills/roundtable-debate/SKILL.md +1 -1
- package/templates/.claude/skills/scout/SKILL.md +1 -1
- package/templates/.claude/skills/secretary-routing/SKILL.md +1 -1
- package/templates/.claude/skills/skill-extractor/SKILL.md +4 -0
- package/templates/.claude/skills/task-decomposition/SKILL.md +1 -1
- package/templates/.claude/skills/worker-reviewer-pipeline/SKILL.md +1 -1
- package/templates/manifest.json +6 -1
- package/templates/workflows/auto-dev.yaml +2 -2
|
@@ -1,323 +1,90 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: professor-triage
|
|
3
|
-
description: Analyze GitHub issues against current codebase and perform automated triage with priority assessment
|
|
3
|
+
description: Analyze GitHub issues against the current codebase and perform automated triage with priority assessment
|
|
4
4
|
scope: harness
|
|
5
|
-
version: 2.
|
|
5
|
+
version: 2.4.0
|
|
6
6
|
user-invocable: true
|
|
7
7
|
effort: high
|
|
8
8
|
context: fork
|
|
9
9
|
argument-hint: "[issue-numbers...] [--label <label>] [--state <state>] [--since <date>]"
|
|
10
10
|
---
|
|
11
11
|
|
|
12
|
-
# /professor-triage
|
|
12
|
+
# /professor-triage - Codebase-Driven Issue Triage
|
|
13
13
|
|
|
14
14
|
## Purpose
|
|
15
15
|
|
|
16
|
-
|
|
16
|
+
Analyze GitHub issues against the current codebase. For each issue, search relevant code, assess impact and blast radius, determine whether the issue has already been resolved, and perform low-risk triage actions with evidence.
|
|
17
17
|
|
|
18
|
-
|
|
19
|
-
|
|
20
|
-
```
|
|
21
|
-
/professor-triage # Default: --state open (excludes verify-done)
|
|
22
|
-
/professor-triage 587 589 590 591 592 # Direct issue numbers
|
|
23
|
-
/professor-triage --label codex-release # Custom label filter
|
|
24
|
-
/professor-triage --since 2026-03-20 # Date filter
|
|
25
|
-
```
|
|
26
|
-
|
|
27
|
-
## Workflow
|
|
18
|
+
**Full phase detail**: `guides/professor-triage/phases.md`
|
|
19
|
+
**Checklist**: `guides/professor-triage/checklists.md`
|
|
28
20
|
|
|
29
|
-
|
|
30
|
-
|
|
31
|
-
1. Parse arguments to determine target issues:
|
|
32
|
-
- If issue numbers provided: use those directly
|
|
33
|
-
- If `--label` provided: `gh issue list --label <label> --state <state> --json number`
|
|
34
|
-
- Default: `gh issue list --state open --json number` + exclude issues with `verify-done` label
|
|
35
|
-
- If `--since` provided: add `--search "created:>YYYY-MM-DD"` filter
|
|
21
|
+
## Usage
|
|
36
22
|
|
|
37
|
-
2. For each issue, fetch full details:
|
|
38
23
|
```bash
|
|
39
|
-
|
|
40
|
-
|
|
41
|
-
|
|
42
|
-
|
|
43
|
-
|
|
44
|
-
4. If filter returns 0 results: if `--label` was used, check label existence via `gh label list`. Report if label missing. If default filter, report "No open issues without verify-done label found."
|
|
45
|
-
|
|
46
|
-
### Phase 2: Codebase Analysis
|
|
47
|
-
|
|
48
|
-
For each issue, perform direct codebase analysis:
|
|
49
|
-
|
|
50
|
-
**2A: Context Extraction** — From issue title and body, extract:
|
|
51
|
-
- File paths mentioned (regex: backtick-wrapped paths, `:\d+` line refs, `(L\d+)`, `(lines \d+-\d+)`)
|
|
52
|
-
- Error messages or stack traces
|
|
53
|
-
- Keywords (function names, class names, config keys, module names)
|
|
54
|
-
- Component areas mentioned (e.g., "auth", "CI", "hooks")
|
|
55
|
-
|
|
56
|
-
**2B: Codebase Search** — Delegate to Explore agent(s):
|
|
57
|
-
- Search for extracted keywords using Grep across the codebase
|
|
58
|
-
- Find related files using Glob patterns derived from keywords
|
|
59
|
-
- For explicitly mentioned files, verify existence and read relevant sections
|
|
60
|
-
- For error messages, trace to source location
|
|
61
|
-
- Map import/dependency relationships for affected files
|
|
62
|
-
|
|
63
|
-
**2C: Impact Assessment** — For each relevant file found:
|
|
64
|
-
- Read current state of the code
|
|
65
|
-
- Check recent changes: `git log --since=<issue_created_date> --oneline -- <file>`
|
|
66
|
-
- Determine if the issue has already been addressed by recent commits
|
|
67
|
-
- Assess blast radius (what depends on this code, what does this code depend on)
|
|
68
|
-
|
|
69
|
-
**2D: Structured Finding** — Produce per-issue analysis:
|
|
70
|
-
|
|
71
|
-
| Field | Content |
|
|
72
|
-
|-------|---------|
|
|
73
|
-
| Affected files | List with status: `exists` ✅ / `missing` ❌ / `changed-since-issue` ⚠️ |
|
|
74
|
-
| Architecture impact | Breaking changes, dependency effects, scope of change |
|
|
75
|
-
| Implementation path | Concrete steps with file:line references from current codebase |
|
|
76
|
-
| Risk level | P1 (critical/security/breaking) / P2 (moderate/compat) / P3 (nice-to-have) |
|
|
77
|
-
| Size estimate | XS (<1h) / S (1-3h) / M (3-8h) / L (1-3d) / XL (>3d) |
|
|
78
|
-
| Already resolved? | Yes / No / Partial — with git evidence (commit hash, PR number) |
|
|
79
|
-
|
|
80
|
-
**Parallelization (R009/R018):**
|
|
81
|
-
- 1-3 issues → single Explore agent per issue (parallel per R009)
|
|
82
|
-
- 4-10 issues → parallel Explore agents, max 4 concurrent (R009)
|
|
83
|
-
- 10+ issues or 3+ Explore agents needed → Agent Teams per R018
|
|
84
|
-
|
|
85
|
-
**Delegation**: All codebase search delegated to Explore agent(s) with `model: haiku`. Orchestrator collects and synthesizes results.
|
|
86
|
-
|
|
87
|
-
### Phase 3: Cross-Analyze
|
|
88
|
-
|
|
89
|
-
**R010 note**: This is a read-only analytical step — no file writes. Per R010 exception, the orchestrator may perform this directly. For batches >15 issues, delegate to a dedicated cross-analysis agent with model: opus.
|
|
90
|
-
|
|
91
|
-
Perform deep cross-analysis with full context from all issues:
|
|
92
|
-
|
|
93
|
-
1. **Common patterns** — Identify findings that appear across multiple issues (e.g., same file referenced, same recommendation theme)
|
|
94
|
-
2. **Duplicate/merge candidates** — Detect issues tracking the same underlying change:
|
|
95
|
-
- Same release series (e.g., alpha.3/5/6)
|
|
96
|
-
- Same upstream dependency
|
|
97
|
-
- Same affected component
|
|
98
|
-
3. **Conflicting findings** — Where findings disagree across issues, resolve based on:
|
|
99
|
-
- Codebase evidence (Phase 2 results)
|
|
100
|
-
- Specificity (concrete code-level finding > abstract observation)
|
|
101
|
-
- Recency (newer findings > older ones)
|
|
102
|
-
4. **Priority matrix** — Unified priority ranking:
|
|
103
|
-
- P1: Breaking changes, security issues, blocking bugs
|
|
104
|
-
- P2: Documentation gaps, compatibility updates, medium-risk items
|
|
105
|
-
- P3: Nice-to-have improvements, future considerations
|
|
106
|
-
5. **Action determination** — Per-issue decision:
|
|
107
|
-
- `Close (Already Resolved)`: Phase 2 found issue already fixed by recent commits
|
|
108
|
-
- `Close (Not Applicable)`: Issue is irrelevant (internal dependency tag, no impact)
|
|
109
|
-
- `Close (Duplicate of #NNN)`: Superseded by another issue in the batch
|
|
110
|
-
- `Open — action required`: Real work needed
|
|
111
|
-
- `Open — monitoring`: Waiting for external trigger (e.g., stable release)
|
|
112
|
-
- `New issue needed`: Cross-analysis discovered issue not yet tracked
|
|
113
|
-
|
|
114
|
-
### Phase 4: Multi-Perspective Analysis & Output
|
|
115
|
-
|
|
116
|
-
For each analyzed issue, generate multi-perspective analysis comments and artifacts.
|
|
117
|
-
|
|
118
|
-
**Parallelization (R009):**
|
|
119
|
-
- Phase 4A + 4B: parallel (independent perspectives)
|
|
120
|
-
- Phase 4C: after 4A + 4B complete (synthesis requires both inputs)
|
|
121
|
-
- Phase 4D + 4E: parallel (independent outputs, both depend on 4C)
|
|
122
|
-
- Phase 4F: after all above (verification gate)
|
|
123
|
-
|
|
124
|
-
**4A: 🏛️ Senior Architect Analysis** — Delegate to arch-documenter (model: sonnet) to post GitHub comment:
|
|
125
|
-
|
|
24
|
+
/professor-triage
|
|
25
|
+
/professor-triage 587 589 590
|
|
26
|
+
/professor-triage --label codex-release
|
|
27
|
+
/professor-triage --since 2026-03-20
|
|
126
28
|
```
|
|
127
|
-
## 🏛️ Senior Architect Analysis
|
|
128
29
|
|
|
129
|
-
|
|
130
|
-
| 컴포넌트 | 영향 | 위험도 |
|
|
131
|
-
|----------|------|--------|
|
|
132
|
-
| {컴포넌트} | {설명} | {High/Medium/Low} |
|
|
30
|
+
## Workflow Contract
|
|
133
31
|
|
|
134
|
-
|
|
135
|
-
|
|
32
|
+
| Phase | Name | Owner | Notes |
|
|
33
|
+
|-------|------|-------|-------|
|
|
34
|
+
| 1 | Gather | Orchestrator | Fetch issue metadata and labels |
|
|
35
|
+
| 2 | Codebase Analysis | Explore agents | Parallel code search and evidence mapping |
|
|
36
|
+
| 3 | Cross-Analyze | Orchestrator or synthesis agent | Duplicate, priority, and action decisions |
|
|
37
|
+
| 4 | Multi-Perspective Output | delegated analysis agents | Architect, colleague, professor synthesis, artifact |
|
|
38
|
+
| 5 | Act | mgr-gitnerd | Labels, comments, close decisions |
|
|
136
39
|
|
|
137
|
-
|
|
138
|
-
- **실현 가능성**: {근거가 포함된 평가}
|
|
139
|
-
- **우선순위 권장**: {P1/P2/P3 및 근거}
|
|
40
|
+
## Delegation Contract
|
|
140
41
|
|
|
141
|
-
|
|
142
|
-
|
|
143
|
-
|
|
144
|
-
|
|
|
42
|
+
| Phase | Agent | Mode |
|
|
43
|
+
|-------|-------|------|
|
|
44
|
+
| Phase 2 codebase search | Explore | bypassPermissions |
|
|
45
|
+
| Phase 4A Senior Architect | general-purpose | bypassPermissions |
|
|
46
|
+
| Phase 4B Project Colleague | general-purpose | bypassPermissions |
|
|
47
|
+
| Phase 4C Professor Synthesis | general-purpose | bypassPermissions |
|
|
48
|
+
| Phase 4D triage comment | mgr-gitnerd | bypassPermissions |
|
|
49
|
+
| Phase 4E artifact report | general-purpose | bypassPermissions |
|
|
50
|
+
| Phase 5 GitHub actions | mgr-gitnerd | bypassPermissions |
|
|
145
51
|
|
|
146
|
-
|
|
52
|
+
Agent selection constraint: artifact-writing delegated agents need Bash access for compatibility-path bypasses. Do not use an agent whose tool policy prevents the required artifact protocol.
|
|
147
53
|
|
|
148
|
-
|
|
149
|
-
_🏛️ Senior Architect perspective — `/professor-triage` v2.2.0_
|
|
150
|
-
```
|
|
151
|
-
|
|
152
|
-
**4B: 🤝 Project Colleague Review** — Delegate to arch-documenter (model: sonnet) to post GitHub comment:
|
|
54
|
+
## Parallelization
|
|
153
55
|
|
|
154
|
-
|
|
155
|
-
|
|
56
|
+
- 1-3 issues: one Explore agent per issue, parallel when useful.
|
|
57
|
+
- 4-10 issues: parallel Explore agents, max 4 concurrent.
|
|
58
|
+
- 10+ issues: prefer a coordinated team surface when available.
|
|
59
|
+
- Phase 4A and 4B are parallel; Phase 4C waits for both; Phase 4D and 4E are parallel after synthesis.
|
|
156
60
|
|
|
157
|
-
|
|
158
|
-
{구체적 코드 위치 및 file:line 참조가 포함된 변경 제안}
|
|
61
|
+
## Sensitive-path compatibility note
|
|
159
62
|
|
|
160
|
-
|
|
161
|
-
- {이름 충돌, 유효성 검사 우회, 경쟁 조건, 엣지 케이스}
|
|
63
|
+
Codex-native artifacts go under `.codex/outputs/sessions/YYYY-MM-DD/professor-triage-HHmmss.md` and may be written with the repo's normal file-write APIs.
|
|
162
64
|
|
|
163
|
-
|
|
164
|
-
1. {구체적 file/function 참조가 포함된 실행 가능한 단계}
|
|
165
|
-
2. {실행 가능한 단계}
|
|
166
|
-
3. {실행 가능한 단계}
|
|
65
|
+
If a delegated task must create, inspect, or modify Claude compatibility artifacts under `.claude/**`, `.claude/outputs/**`, or `templates/.claude/**`, include this block in the delegated prompt:
|
|
167
66
|
|
|
168
|
-
|
|
169
|
-
|
|
67
|
+
```text
|
|
68
|
+
Sensitive-path compatibility note:
|
|
69
|
+
1. Keep Codex-native .codex/** artifacts on the normal file-write path.
|
|
70
|
+
2. On Claude Code v2.1.121+ with bypassPermissions, direct writes to .claude/skills/, .claude/agents/, and .claude/commands/ are allowed.
|
|
71
|
+
3. On Claude Code v2.1.126+ with bypassPermissions, broader protected paths such as .claude/**, .git/**, and .vscode/** are also covered.
|
|
72
|
+
4. Use /tmp/professor-triage-<timestamp>.md only as a legacy fallback when the runtime is older or still prompts, then verify the resulting diff.
|
|
170
73
|
```
|
|
171
74
|
|
|
172
|
-
|
|
173
|
-
|
|
174
|
-
**4C: 🎓 Professor Synthesis** — Delegate to arch-documenter (model: opus) to post GitHub comment. This phase requires 4A and 4B results as input:
|
|
175
|
-
|
|
176
|
-
```
|
|
177
|
-
## 🎓 Professor Synthesis
|
|
178
|
-
|
|
179
|
-
### 코드베이스 검증
|
|
180
|
-
| 주장 (Architect/Colleague) | 검증 | 근거 |
|
|
181
|
-
|---------------------------|------|------|
|
|
182
|
-
| {주장} | ✅/⚠️/❌ | {file:line 또는 git 근거} |
|
|
183
|
-
|
|
184
|
-
### 합의 및 이견
|
|
185
|
-
| 주제 | Architect | Colleague | 판정 |
|
|
186
|
-
|------|-----------|-----------|------|
|
|
187
|
-
| {주제} | {입장} | {입장} | {종합 판단} |
|
|
188
|
-
|
|
189
|
-
### 우선순위 매트릭스
|
|
190
|
-
| 차원 | 평가 |
|
|
191
|
-
|------|------|
|
|
192
|
-
| 긴급성 | {High/Medium/Low} |
|
|
193
|
-
| 중요성 | {High/Medium/Low} |
|
|
194
|
-
| 규모 | {XS/S/M/L/XL} |
|
|
195
|
-
| 권장 순서 | {배치 내 N/M} |
|
|
196
|
-
|
|
197
|
-
### 누락된 관점
|
|
198
|
-
{Architect나 Colleague 모두 제기하지 않은 고려사항}
|
|
199
|
-
|
|
200
|
-
### 실행 로드맵
|
|
201
|
-
| 단계 | 작업 | 파일 | 의존성 |
|
|
202
|
-
|------|------|------|--------|
|
|
203
|
-
| 1 | {작업} | {파일} | — |
|
|
204
|
-
| 2 | {작업} | {파일} | 단계 1 |
|
|
205
|
-
|
|
206
|
-
### 최종 결론
|
|
207
|
-
{확정적 권장 사항이 포함된 2-3문장 종합}
|
|
208
|
-
|
|
209
|
-
---
|
|
210
|
-
_🎓 Professor Synthesis — `/professor-triage` v2.2.0_
|
|
211
|
-
```
|
|
212
|
-
|
|
213
|
-
**4D: Issue Triage Comment (MANDATORY)** — Every analyzed issue MUST receive a triage comment. This is not optional — even for issues created in the same session or with existing analysis. Skipping comments breaks the triage audit trail. Delegate to mgr-gitnerd to post on each analyzed issue:
|
|
214
|
-
|
|
215
|
-
```
|
|
216
|
-
## 🔬 Professor Triage — Codebase Analysis Result
|
|
217
|
-
|
|
218
|
-
**결정**: {Close (Already Resolved) | Close (Not Applicable) | Close (Duplicate of #NNN) | Open — action required | Open — monitoring}
|
|
219
|
-
**근거**: {코드베이스 분석 기반 1-2줄 요약}
|
|
220
|
-
**영향 파일**: {N}개 분석 — {N}✅ {N}⚠️ {N}❌
|
|
221
|
-
**리스크**: {P1/P2/P3} | **규모**: {XS/S/M/L/XL}
|
|
222
|
-
**전체 리포트**: {artifact path}
|
|
223
|
-
|
|
224
|
-
---
|
|
225
|
-
_`/professor-triage` v2.2.0에 의해 현재 코드베이스 대비 분석됨 — 관련 이슈 {N}개_
|
|
226
|
-
```
|
|
227
|
-
|
|
228
|
-
**4E: Artifact Report** — Delegate to arch-documenter to write:
|
|
229
|
-
|
|
230
|
-
Path: `.codex/outputs/sessions/YYYY-MM-DD/professor-triage-HHmmss.md`
|
|
231
|
-
|
|
232
|
-
Timestamps use local machine time (consistent with other artifact skills).
|
|
233
|
-
|
|
234
|
-
Sensitive-path artifact protocol (mandatory): if a delegated agent must create or inspect Claude compatibility artifacts under `.claude/**`, `.claude/outputs/**`, or `templates/.claude/**`, include this protocol in the delegated prompt. The agent must produce artifact bodies in `/tmp/professor-triage-{timestamp}.md` first and must not call Read, Bash, Write, or Edit directly on `.claude/**` paths in unattended flows. Codex-native `.codex/outputs/**` artifacts continue to use managed file-write APIs that create parents.
|
|
235
|
-
|
|
236
|
-
Template:
|
|
237
|
-
```
|
|
238
|
-
# Professor Triage 리포트 — YYYY-MM-DD
|
|
239
|
-
|
|
240
|
-
## 분석 대상
|
|
241
|
-
| # | 제목 | 라벨 | 생성일 |
|
|
242
|
-
|---|------|------|--------|
|
|
243
|
-
|
|
244
|
-
## 이슈별 분석
|
|
245
|
-
### #NNN — title
|
|
246
|
-
- **영향 파일**: N개 분석 — N✅ N⚠️ N❌
|
|
247
|
-
- **아키텍처 영향**: ...
|
|
248
|
-
- **구현 경로**: ...
|
|
249
|
-
- **리스크/우선순위**: P1/P2/P3
|
|
250
|
-
- **규모**: XS/S/M/L/XL
|
|
251
|
-
- **이미 해결됨?**: Yes/No/Partial — 근거
|
|
252
|
-
- **권장 조치**: ...
|
|
253
|
-
|
|
254
|
-
## 교차 분석
|
|
255
|
-
### 공통 패턴
|
|
256
|
-
### 중복/병합 후보
|
|
257
|
-
### 상충 발견사항 해결
|
|
258
|
-
### 우선순위 매트릭스
|
|
259
|
-
|
|
260
|
-
## 다관점 요약
|
|
261
|
-
### Architect 주요 사항
|
|
262
|
-
### Colleague 주요 사항
|
|
263
|
-
### Professor Synthesis 핵심 포인트
|
|
264
|
-
|
|
265
|
-
## 실행된 조치
|
|
266
|
-
| 이슈 | 조치 | 상태 |
|
|
267
|
-
|
|
268
|
-
## 보류 중인 조치 (확인 필요)
|
|
269
|
-
```
|
|
270
|
-
|
|
271
|
-
### Phase 4F: Comment Verification Gate
|
|
272
|
-
|
|
273
|
-
Before proceeding to Phase 5, verify ALL analyzed issues received the full set of comments (Architect + Colleague + Professor Synthesis + Triage):
|
|
274
|
-
```bash
|
|
275
|
-
# For each issue NNN in the batch:
|
|
276
|
-
gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Professor Triage"))) | length'
|
|
277
|
-
# Must be >= 1 for every issue. If any is 0, go back and post.
|
|
278
|
-
|
|
279
|
-
# Also verify multi-perspective comments:
|
|
280
|
-
gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Senior Architect"))) | length'
|
|
281
|
-
gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Project Colleague"))) | length'
|
|
282
|
-
gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Professor Synthesis"))) | length'
|
|
283
|
-
# All must be >= 1. If any is 0, the corresponding Phase 4A/4B/4C was skipped — go back and post.
|
|
284
|
-
```
|
|
285
|
-
|
|
286
|
-
### Phase 5: Act
|
|
287
|
-
|
|
288
|
-
Delegate ALL GitHub operations to mgr-gitnerd.
|
|
289
|
-
|
|
290
|
-
**Automatic (low-risk, reversible):**
|
|
291
|
-
|
|
292
|
-
| Condition | Action |
|
|
293
|
-
|-----------|--------|
|
|
294
|
-
| Phase 2 found issue already resolved (with commit evidence) | `gh issue close --reason "completed"` + comment with resolving commit |
|
|
295
|
-
| Cross-analysis concludes "Not Applicable" / "no action needed" | `gh issue close --reason "not planned"` |
|
|
296
|
-
| Cross-analysis detects same-series duplicates | Keep latest, close others + `duplicate` label |
|
|
297
|
-
| All analysis complete | Add `verify-done` label |
|
|
298
|
-
| Priority assigned | Add `P1`/`P2`/`P3` label |
|
|
299
|
-
|
|
300
|
-
**Confirmation required (high-risk):**
|
|
75
|
+
This protocol must be inline in the delegate prompt; relying on this SKILL.md being present in the parent context is not enough.
|
|
301
76
|
|
|
302
|
-
|
|
77
|
+
## Phase 5 Action Policy
|
|
303
78
|
|
|
304
|
-
|
|
305
|
-
|
|
306
|
-
|
|
307
|
-
|
|
308
|
-
|
|
309
|
-
|
|
79
|
+
Automatic, low-risk actions:
|
|
80
|
+
- Issue already resolved by commit: close as completed with evidence comment.
|
|
81
|
+
- Not applicable to the Codex port: close as not planned with evidence.
|
|
82
|
+
- Duplicate series: close older duplicate and keep the canonical issue open.
|
|
83
|
+
- Completed analysis: add `verify-done` only after evidence is current.
|
|
84
|
+
- Priority assignment: add `P1`/`P2`/`P3` when supported by analysis.
|
|
310
85
|
|
|
311
|
-
|
|
86
|
+
Confirmation required: issue reopen, new issue creation, epic rewrites, or issue body modification.
|
|
312
87
|
|
|
313
|
-
##
|
|
88
|
+
## Permission Mode
|
|
314
89
|
|
|
315
|
-
|
|
316
|
-
- Phase 2: Explore agents with `model: haiku` for codebase search; orchestrator synthesizes findings
|
|
317
|
-
- Phase 3: Orchestrator directly (read-only, R010 exception); opus agent for >15 issues
|
|
318
|
-
- Phase 4A/4B: `arch-documenter` (sonnet) for Architect/Colleague analysis comments (parallel)
|
|
319
|
-
- Phase 4C: `arch-documenter` (opus) for Professor Synthesis comment (requires 4A+4B)
|
|
320
|
-
- Phase 4D: `mgr-gitnerd` for triage comment; Phase 4E: `arch-documenter` for artifact report (parallel)
|
|
321
|
-
- Phase 4F: Verification gate for all 4 comment types
|
|
322
|
-
- Phase 5: `mgr-gitnerd` for all GitHub operations
|
|
323
|
-
- No external dependencies (omc_issue_analyzer removed in v2.0.0, multi-perspective analysis restored in v2.1.0)
|
|
90
|
+
When spawning agents, explicitly pass `mode: "bypassPermissions"` if the runtime supports it. Defaults may override agent frontmatter and reintroduce permission prompts during unattended execution.
|
|
@@ -94,7 +94,7 @@ Delegate to mgr-creator with context:
|
|
|
94
94
|
|
|
95
95
|
## Sensitive-Path Delegation
|
|
96
96
|
|
|
97
|
-
Sensitive-path
|
|
97
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
98
98
|
|
|
99
99
|
## Usage
|
|
100
100
|
|
|
@@ -204,6 +204,11 @@ Convergence expected by round 3. Hard stop at round 30.
|
|
|
204
204
|
2. **Artifact persistence**: The Phase 4 synthesis agent (opus) writes the report to:
|
|
205
205
|
```
|
|
206
206
|
.codex/outputs/sessions/{YYYY-MM-DD}/research-{HHmmss}.md
|
|
207
|
+
|
|
208
|
+
### Compatibility artifact protocol
|
|
209
|
+
|
|
210
|
+
Sensitive-path compatibility note: when delegated work touches `.claude/outputs/`, `.claude/**`, or `templates/.claude/**`, keep `.codex/**` artifacts on the normal file-write path. On Claude Code v2.1.121+ with `bypassPermissions`, direct compatibility writes are allowed for `.claude/skills/`, `.claude/agents/`, and `.claude/commands/`; on v2.1.126+ broader protected paths are covered. Use `/tmp/<skill>-<timestamp>.md` only as a legacy fallback when the runtime is older or still prompts.
|
|
211
|
+
|
|
207
212
|
```
|
|
208
213
|
With metadata header:
|
|
209
214
|
```markdown
|
|
@@ -214,7 +219,7 @@ Convergence expected by round 3. Hard stop at round 30.
|
|
|
214
219
|
---
|
|
215
220
|
```
|
|
216
221
|
The agent writes the artifact file using a file-write API that creates missing parent directories; do not run any Bash directory-creation pre-step for session outputs (R010 compliance).
|
|
217
|
-
Sensitive-path
|
|
222
|
+
Sensitive-path compatibility note: delegated research agents should keep `.codex/**` artifacts on the normal write path and use `/tmp/research-{timestamp}.md` only as a legacy fallback when an older Claude Code runtime still prompts on compatibility paths.
|
|
218
223
|
3. GitHub issue auto-created with findings
|
|
219
224
|
4. Action items with effort estimates
|
|
220
225
|
|
|
@@ -115,6 +115,10 @@ Secretary outputs:
|
|
|
115
115
|
- Individual details if requested
|
|
116
116
|
```
|
|
117
117
|
|
|
118
|
+
### Compatibility artifact protocol
|
|
119
|
+
|
|
120
|
+
Sensitive-path compatibility note: when delegated work touches `.claude/outputs/`, `.claude/**`, or `templates/.claude/**`, keep `.codex/**` artifacts on the normal file-write path. On Claude Code v2.1.121+ with `bypassPermissions`, direct compatibility writes are allowed for `.claude/skills/`, `.claude/agents/`, and `.claude/commands/`; on v2.1.126+ broader protected paths are covered. Use `/tmp/<skill>-<timestamp>.md` only as a legacy fallback when the runtime is older or still prompts.
|
|
121
|
+
|
|
118
122
|
### With Ecomode
|
|
119
123
|
|
|
120
124
|
```
|
|
@@ -22,7 +22,7 @@ Run a bounded debate when convergence would hide useful disagreement. Unlike `ag
|
|
|
22
22
|
|
|
23
23
|
## Sensitive-Path Delegation
|
|
24
24
|
|
|
25
|
-
Sensitive-path
|
|
25
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
26
26
|
|
|
27
27
|
## Workflow
|
|
28
28
|
|
|
@@ -13,7 +13,7 @@ Analyze an external URL (tech blog, tool, library, methodology) to evaluate its
|
|
|
13
13
|
|
|
14
14
|
## Sensitive-Path Delegation
|
|
15
15
|
|
|
16
|
-
Sensitive-path
|
|
16
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
17
17
|
|
|
18
18
|
## Usage
|
|
19
19
|
|
|
@@ -134,7 +134,7 @@ Evaluate: Is this a specialized management/tooling task?
|
|
|
134
134
|
|
|
135
135
|
## Sensitive-Path Delegation
|
|
136
136
|
|
|
137
|
-
Sensitive-path
|
|
137
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
138
138
|
|
|
139
139
|
## Usage
|
|
140
140
|
|
|
@@ -146,6 +146,10 @@ The `skill-extractor-analyzer.sh` Stop hook provides a lightweight pre-analysis:
|
|
|
146
146
|
- Emits advisory stderr message if candidates found
|
|
147
147
|
- Does NOT create skills (that requires user approval via the skill)
|
|
148
148
|
|
|
149
|
+
## Compatibility Artifact Protocol
|
|
150
|
+
|
|
151
|
+
Sensitive-path compatibility note: when delegated work touches `.claude/outputs/`, `.claude/**`, or `templates/.claude/**`, keep `.codex/**` artifacts on the normal file-write path. On Claude Code v2.1.121+ with `bypassPermissions`, direct compatibility writes are allowed for `.claude/skills/`, `.claude/agents/`, and `.claude/commands/`; on v2.1.126+ broader protected paths are covered. Use `/tmp/<skill>-<timestamp>.md` only as a legacy fallback when the runtime is older or still prompts.
|
|
152
|
+
|
|
149
153
|
## Safety
|
|
150
154
|
|
|
151
155
|
- **User approval required**: Never auto-creates skills
|
|
@@ -12,7 +12,7 @@ Analyzes task complexity and decomposes large tasks into smaller, parallelizable
|
|
|
12
12
|
|
|
13
13
|
## Sensitive-Path Delegation
|
|
14
14
|
|
|
15
|
-
Sensitive-path
|
|
15
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
16
16
|
|
|
17
17
|
## Trigger Conditions
|
|
18
18
|
|
|
@@ -14,7 +14,7 @@ Defines an iterative Worker→Reviewer pipeline where one agent implements chang
|
|
|
14
14
|
|
|
15
15
|
## Sensitive-Path Delegation
|
|
16
16
|
|
|
17
|
-
Sensitive-path
|
|
17
|
+
Sensitive-path compatibility note: if this skill delegates work that touches `.claude/**`, `.claude/outputs/**`, `templates/.claude/**`, or read-only measurements of those paths, keep `.codex/**` edits on the normal Codex path. On Claude Code v2.1.121+ with `bypassPermissions`, direct writes to `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` are allowed; on v2.1.126+ that extends to broader protected paths. Only use `/tmp/{skill}-{timestamp}.md` as a legacy fallback when the target runtime is older or still prompts.
|
|
18
18
|
|
|
19
19
|
## When to Activate
|
|
20
20
|
|
package/templates/manifest.json
CHANGED
|
@@ -40,9 +40,9 @@ steps:
|
|
|
40
40
|
|
|
41
41
|
Codex-native sensitive-path policy:
|
|
42
42
|
- Codex-managed `.codex/` edits use the normal edit/patch flow.
|
|
43
|
-
-
|
|
43
|
+
- Claude Code `bypassPermissions` can write `.claude/skills/`, `.claude/agents/`, and `.claude/commands/` directly on v2.1.121+, with broader protected-path writes available on v2.1.126+.
|
|
44
44
|
- If a port requires `.claude` template parity changes, make the change explicit and verify sensitive-path guard tests.
|
|
45
|
-
-
|
|
45
|
+
- Use `/tmp/{skill}-{timestamp}.md` only as a historical fallback for older Claude Code versions, non-bypass sessions, or runs that still surface a protected-path prompt.
|
|
46
46
|
description: Execute implementation plan with appropriate agents
|
|
47
47
|
foreach: planned-issue
|
|
48
48
|
|