oh-my-customcode 0.68.2 → 0.70.0

package/templates/.claude/skills/professor-triage/SKILL.md

@@ -1,24 +1,24 @@
 ---
 name: professor-triage
-description: Cross-analyze GitHub issues with omc_issue_analyzer comments, verify against codebase, and perform automated triage
+description: Analyze GitHub issues against current codebase and perform automated triage with priority assessment
 scope: harness
-version: 1.0.0
+version: 2.1.0
 user-invocable: true
 effort: high
 context: fork
 argument-hint: "[issue-numbers...] [--label <label>] [--state <state>] [--since <date>]"
 ---
 
-# /professor-triage — Cross-Analysis Issue Triage
+# /professor-triage — Codebase-Driven Issue Triage
 
 ## Purpose
 
-Cross-analyzes GitHub issues containing `omc_issue_analyzer` comments (Architect, Colleague, Professor analyses). Verifies claims against the current codebase, produces a cross-analysis report, and performs automated triage actions.
+Analyzes GitHub issues directly against the current codebase. For each issue, searches relevant code, assesses impact and blast radius, determines whether the issue has already been resolved, and performs automated triage with priority and size estimation. Produces a cross-analysis report and executes low-risk triage actions automatically.
 
 ## Usage
 
 ```
-/professor-triage                          # Default: --label professor --state open
+/professor-triage                          # Default: --state open (excludes verify-done)
 /professor-triage 587 589 590 591 592      # Direct issue numbers
 /professor-triage --label codex-release    # Custom label filter
 /professor-triage --since 2026-03-20       # Date filter
@@ -31,7 +31,7 @@ Cross-analyzes GitHub issues containing `omc_issue_analyzer` comments (Architect
 1. Parse arguments to determine target issues:
    - If issue numbers provided: use those directly
    - If `--label` provided: `gh issue list --label <label> --state <state> --json number`
-   - Default: `gh issue list --label professor --state open --json number`
+   - Default: `gh issue list --state open --json number` + exclude issues with `verify-done` label
    - If `--since` provided: add `--search "created:>YYYY-MM-DD"` filter
 
 2. For each issue, fetch full details:
@@ -41,62 +41,50 @@ gh issue view NNN --json number,title,body,comments,labels,createdAt
 
 3. For batches >20 issues, prefer `gh api graphql` for batch fetching to respect GitHub API rate limits (5000/hour authenticated).
 
-4. If `--label` returns 0 results, check label existence via `gh label list`. Report if label missing.
+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."
 
-### Phase 2: Parse
+### Phase 2: Codebase Analysis
 
-Extract structured data from omc_issue_analyzer comments by matching these headers:
+For each issue, perform direct codebase analysis:
 
-| Header | Section |
-|--------|---------|
-| `## 🏛️ Senior Architect Analysis` | Architecture impact, code-level analysis, strategy, risk assessment |
-| `## 🤝 Project Colleague Review` | Implementation ideas, blind spots, next steps |
-| `## 🎓 Professor Synthesis` | Verification results, consensus, disagreements, priority matrix, missed perspectives, roadmap |
+**2A: Context Extraction** — From issue title and body, extract:
+- File paths mentioned (regex: backtick-wrapped paths, `:\d+` line refs, `(L\d+)`, `(lines \d+-\d+)`)
+- Error messages or stack traces
+- Keywords (function names, class names, config keys, module names)
+- Component areas mentioned (e.g., "auth", "CI", "hooks")
 
-For each issue, extract:
-- **File paths and line references** mentioned in any analysis (regex: backtick-wrapped paths, `(L\d+)`, `(lines \d+-\d+)`, `:\d+` suffixes)
-- **Priority/size assessments** from Professor synthesis
-- **Recommended actions** from all three analyses
-- **Risk levels** and breaking change assessments
-- **Consensus points** where all 3 analyses agree
-- **Disagreement points** where analyses differ
+**2B: Codebase Search** — Delegate to Explore agent(s):
+- Search for extracted keywords using Grep across the codebase
+- Find related files using Glob patterns derived from keywords
+- For explicitly mentioned files, verify existence and read relevant sections
+- For error messages, trace to source location
+- Map import/dependency relationships for affected files
 
-If an issue has no omc_issue_analyzer comments (no matching headers), skip with warning and include in report as "no analysis available".
+**2C: Impact Assessment** — For each relevant file found:
+- Read current state of the code
+- Check recent changes: `git log --since=<issue_created_date> --oneline -- <file>`
+- Determine if the issue has already been addressed by recent commits
+- Assess blast radius (what depends on this code, what does this code depend on)
 
-### Phase 3: Verify
+**2D: Structured Finding** — Produce per-issue analysis:
 
-Verify ALL file paths and line references extracted in Phase 2 against the current codebase.
-
-**Verification levels:**
-
-| Level | Check | Tool |
-|-------|-------|------|
-| Existence | File currently exists | Glob |
-| Content | Referenced line contains expected code | Read |
-| Freshness | Changed since comment was written | `git log --since=<comment_date> -- <file>` |
-
-**Result classification:**
-
-| Status | Meaning | Marker |
-|--------|---------|--------|
-| `verified` | File exists + content matches | ✅ |
-| `stale-still-valid` | File changed but recommendation still applies | ⚠️ |
-| `stale-invalidated` | File changed and recommendation is moot | ⚠️❌ |
-| `missing` | File/path not found | ❌ |
-| `unchecked` | No specific path to verify | ➖ |
-
-**No limits**: All mentioned file paths are verified exhaustively. Quality over token cost.
-
-**Shared verification**: If the same file is mentioned across multiple issues, verify once and share the result.
+| Field | Content |
+|-------|---------|
+| Affected files | List with status: `exists` ✅ / `missing` ❌ / `changed-since-issue` ⚠️ |
+| Architecture impact | Breaking changes, dependency effects, scope of change |
+| Implementation path | Concrete steps with file:line references from current codebase |
+| Risk level | P1 (critical/security/breaking) / P2 (moderate/compat) / P3 (nice-to-have) |
+| Size estimate | XS (<1h) / S (1-3h) / M (3-8h) / L (1-3d) / XL (>3d) |
+| Already resolved? | Yes / No / Partial — with git evidence (commit hash, PR number) |
 
 **Parallelization (R009/R018):**
-- ≤20 unique file references → single Explore agent
-- 21-80 file references → 2-4 parallel Explore agents (R009)
-- 80+ file references or 3+ verification agents needed → Agent Teams per R018
+- 1-3 issues → single Explore agent per issue (parallel per R009)
+- 4-10 issues → parallel Explore agents, max 4 concurrent (R009)
+- 10+ issues or 3+ Explore agents needed → Agent Teams per R018
 
-**Delegation**: Delegate verification to Explore agent(s). Orchestrator collects results.
+**Delegation**: All codebase search delegated to Explore agent(s) with `model: haiku`. Orchestrator collects and synthesizes results.
 
-### Phase 4: Cross-Analyze
+### Phase 3: Cross-Analyze
 
 **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.
 
@@ -107,24 +95,137 @@ Perform deep cross-analysis with full context from all issues:
    - Same release series (e.g., alpha.3/5/6)
    - Same upstream dependency
    - Same affected component
-3. **Conflicting recommendations** — Where analyses disagree across issues, resolve based on:
-   - Codebase verification evidence (Phase 3 results)
-   - Specificity (concrete code-level analysis > abstract strategy)
-   - Recency (newer analyses > older ones)
+3. **Conflicting findings** — Where findings disagree across issues, resolve based on:
+   - Codebase evidence (Phase 2 results)
+   - Specificity (concrete code-level finding > abstract observation)
+   - Recency (newer findings > older ones)
 4. **Priority matrix** — Unified priority ranking:
    - P1: Breaking changes, security issues, blocking bugs
    - P2: Documentation gaps, compatibility updates, medium-risk items
    - P3: Nice-to-have improvements, future considerations
 5. **Action determination** — Per-issue decision:
+   - `Close (Already Resolved)`: Phase 2 found issue already fixed by recent commits
    - `Close (Not Applicable)`: Issue is irrelevant (internal dependency tag, no impact)
    - `Close (Duplicate of #NNN)`: Superseded by another issue in the batch
    - `Open — action required`: Real work needed
    - `Open — monitoring`: Waiting for external trigger (e.g., stable release)
    - `New issue needed`: Cross-analysis discovered issue not yet tracked
 
-### Phase 5: Output
+### Phase 4: Multi-Perspective Analysis & Output
+
+For each analyzed issue, generate multi-perspective analysis comments and artifacts.
+
+**Parallelization (R009):**
+- Phase 4A + 4B: parallel (independent perspectives)
+- Phase 4C: after 4A + 4B complete (synthesis requires both inputs)
+- Phase 4D + 4E: parallel (independent outputs, both depend on 4C)
+- Phase 4F: after all above (verification gate)
+
+**4A: 🏛️ Senior Architect Analysis** — Delegate to arch-documenter (model: sonnet) to post GitHub comment:
+
+```
+## 🏛️ Senior Architect Analysis
+
+### Architecture Impact
+| Component | Impact | Risk |
+|-----------|--------|------|
+| {component} | {description} | {High/Medium/Low} |
+
+### Code-Level Analysis
+{Specific file:line references from Phase 2 codebase analysis}
+
+### Strategic Assessment
+- **Feasibility**: {assessment with evidence}
+- **Priority recommendation**: {P1/P2/P3 with rationale}
+
+### Risk & Considerations
+| Risk | Likelihood | Mitigation |
+|------|-----------|------------|
+| {risk} | {High/Medium/Low} | {mitigation} |
+
+**Estimated effort**: {XS/S/M/L/XL}
+
+---
+_🏛️ Senior Architect perspective — `/professor-triage` v2.1.0_
+```
+
+**4B: 🤝 Project Colleague Review** — Delegate to arch-documenter (model: sonnet) to post GitHub comment:
+
+```
+## 🤝 Project Colleague Review
+
+### Implementation Ideas
+{Concrete code locations and change suggestions with file:line references}
+
+### Easy-to-Miss Details
+- {Name collisions, validation bypasses, race conditions, edge cases}
+
+### Suggested Next Steps
+1. {Actionable step with specific file/function reference}
+2. {Actionable step}
+3. {Actionable step}
+
+---
+_🤝 Project Colleague perspective — `/professor-triage` v2.1.0_
+```
+
+Note: Do NOT include a "First Impressions" (첫인상) section in the Colleague Review — this was explicitly excluded per user feedback.
+
+**4C: 🎓 Professor Synthesis** — Delegate to arch-documenter (model: opus) to post GitHub comment. This phase requires 4A and 4B results as input:
 
-**5A: Artifact report** — Delegate to arch-documenter to write:
+```
+## 🎓 Professor Synthesis
+
+### Codebase Verification
+| Claim (from Architect/Colleague) | Verified | Evidence |
+|----------------------------------|----------|----------|
+| {claim} | ✅/⚠️/❌ | {file:line or git evidence} |
+
+### Consensus & Divergence
+| Topic | Architect | Colleague | Verdict |
+|-------|-----------|-----------|---------|
+| {topic} | {position} | {position} | {synthesized judgment} |
+
+### Priority Matrix
+| Dimension | Assessment |
+|-----------|-----------|
+| Urgency | {High/Medium/Low} |
+| Importance | {High/Medium/Low} |
+| Size | {XS/S/M/L/XL} |
+| Recommended order | {N of M in batch} |
+
+### Missed Perspectives
+{Considerations neither Architect nor Colleague raised}
+
+### Execution Roadmap
+| Phase | Task | Files | Depends on |
+|-------|------|-------|-----------|
+| 1 | {task} | {files} | — |
+| 2 | {task} | {files} | Phase 1 |
+
+### Final Conclusion
+{2-3 sentence synthesis with definitive recommendation}
+
+---
+_🎓 Professor Synthesis — `/professor-triage` v2.1.0_
+```
+
+**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:
+
+```
+## 🔬 Professor Triage — Codebase Analysis Result
+
+**Decision**: {Close (Already Resolved) | Close (Not Applicable) | Close (Duplicate of #NNN) | Open — action required | Open — monitoring}
+**Rationale**: {1-2 line summary based on codebase findings}
+**Affected files**: {N} analyzed — {N}✅ {N}⚠️ {N}❌
+**Risk**: {P1/P2/P3} | **Size**: {XS/S/M/L/XL}
+**Full report**: {artifact path}
+
+---
+_Analyzed by `/professor-triage` v2.1.0 against current codebase with {N} related issues_
+```
+
+**4E: Artifact Report** — Delegate to arch-documenter to write:
 
 Path: `.claude/outputs/sessions/YYYY-MM-DD/professor-triage-HHmmss.md`
 
@@ -135,51 +236,52 @@ Template:
 # Professor Triage Report — YYYY-MM-DD
 
 ## Analysis Target
-| # | Title | Labels | Analysis Comments |
-|---|-------|--------|-------------------|
+| # | Title | Labels | Created |
+|---|-------|--------|---------|
 
-## Per-Issue Summary
+## Per-Issue Analysis
 ### #NNN — title
-- Core conclusion:
-- Verification status: N✅ N⚠️ N❌
-- Recommended action:
+- **Affected files**: N analyzed — N✅ N⚠️ N❌
+- **Architecture impact**: ...
+- **Implementation path**: ...
+- **Risk/Priority**: P1/P2/P3
+- **Size**: XS/S/M/L/XL
+- **Already resolved?**: Yes/No/Partial — evidence
+- **Recommended action**: ...
 
 ## Cross-Analysis
 ### Common Patterns
 ### Duplicate/Merge Candidates
-### Conflicting Recommendations Resolution
+### Conflicting Findings Resolution
 ### Priority Matrix
 
+## Multi-Perspective Summary
+### Architect Highlights
+### Colleague Highlights
+### Professor Synthesis Key Points
+
 ## Executed Actions
 | Issue | Action | Status |
 
 ## Pending Actions (Confirmation Required)
 ```
 
-**5B: Issue comments (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:
-
-```
-## 🔬 Professor Triage — Cross-Analysis Result
+### Phase 4F: Comment Verification Gate
 
-**Decision**: {Close (Not Applicable) | Close (Duplicate of #NNN) | Open — action required}
-**Rationale**: {1-2 line summary}
-**Verification**: {N}✅ {N}⚠️ {N}❌
-**Full report**: {artifact path}
-
----
-_Cross-analyzed by `/professor-triage` with {N} related issues_
-```
-
-### Phase 5C: Comment Verification Gate
-
-Before proceeding to Phase 6, verify ALL analyzed issues received comments:
+Before proceeding to Phase 5, verify ALL analyzed issues received the full set of comments (Architect + Colleague + Professor Synthesis + Triage):
 ```bash
 # For each issue NNN in the batch:
 gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Professor Triage"))) | length'
-# Must be >= 1 for every issue. If any is 0, Phase 5B was skipped — go back and post.
+# Must be >= 1 for every issue. If any is 0, go back and post.
+
+# Also verify multi-perspective comments:
+gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Senior Architect"))) | length'
+gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Project Colleague"))) | length'
+gh issue view NNN --json comments --jq '.comments | map(select(.body | contains("Professor Synthesis"))) | length'
+# All must be >= 1. If any is 0, the corresponding Phase 4A/4B/4C was skipped — go back and post.
 ```
 
-### Phase 6: Act
+### Phase 5: Act
 
 Delegate ALL GitHub operations to mgr-gitnerd.
 
@@ -187,10 +289,11 @@ Delegate ALL GitHub operations to mgr-gitnerd.
 
 | Condition | Action |
 |-----------|--------|
-| Professor synthesis concludes "Not Applicable" / "no action needed" | `gh issue close --reason "not planned"` |
+| Phase 2 found issue already resolved (with commit evidence) | `gh issue close --reason "completed"` + comment with resolving commit |
+| Cross-analysis concludes "Not Applicable" / "no action needed" | `gh issue close --reason "not planned"` |
 | Cross-analysis detects same-series duplicates | Keep latest, close others + `duplicate` label |
 | All analysis complete | Add `verify-done` label |
-| Professor synthesis assigns priority | Add `P1`/`P2`/`P3` label |
+| Priority assigned | Add `P1`/`P2`/`P3` label |
 
 **Confirmation required (high-risk):**
 
@@ -207,8 +310,12 @@ Present to user and wait for approval before executing:
 
 ## Notes
 
-- Phase 2 agents: not needed (orchestrator parses text directly)
-- Phase 3 agents: `Explore` with `model: haiku` for file verification
-- Phase 4: orchestrator directly (read-only, R010 exception)
-- Phase 5: `arch-documenter` for artifact, `mgr-gitnerd` for comments
-- Phase 6: `mgr-gitnerd` for all GitHub ops
+- Phase 1: Orchestrator fetches issues directly (no agent needed)
+- Phase 2: Explore agents with `model: haiku` for codebase search; orchestrator synthesizes findings
+- Phase 3: Orchestrator directly (read-only, R010 exception); opus agent for >15 issues
+- Phase 4A/4B: `arch-documenter` (sonnet) for Architect/Colleague analysis comments (parallel)
+- Phase 4C: `arch-documenter` (opus) for Professor Synthesis comment (requires 4A+4B)
+- Phase 4D: `mgr-gitnerd` for triage comment; Phase 4E: `arch-documenter` for artifact report (parallel)
+- Phase 4F: Verification gate for all 4 comment types
+- Phase 5: `mgr-gitnerd` for all GitHub operations
+- No external dependencies (omc_issue_analyzer removed in v2.0.0, multi-perspective analysis restored in v2.1.0)

package/templates/manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.68.2",
+  "version": "0.70.0",
   "lastUpdated": "2026-03-24T00:00:00.000Z",
   "components": [
     {

package/templates/workflows/auto-dev.yaml

@@ -14,7 +14,7 @@ steps:
 
   - name: triage
     skill: professor-triage
-    description: Cross-analyze verify-done issues with omc_issue_analyzer comments
+    description: Analyze verify-done issues against current codebase and perform automated triage
 
   - name: plan
     skill: release-plan