Forge skill-forge-update from B to A via judge findings

#22 · closed · 2 comments

View on GitHub ↗

WrathZA

## Summary Two skill-forge-judge cycles on skill-forge-update surfaced seven targeted improvements which were applied via skill-forge-hitl. The skill moved from B (86%, 103/120) to A (91%, 109/120) on the U+S rubric. Changes: terminated two ambiguous decision branches, upgraded Phase 4 from a one-liner to a structured close-out with project-level default, added a Phase 3 zero-change guard, inlined trigger scenarios into the description, opened the drift check with a thinking-lens question, and added a sixth NEVER protecting hitl as Phase 2's exclusive apply mechanism. ## Acceptance Criteria - [x] `skill-forge-update/SKILL.md` description ends with "Use when an existing SKILL.md needs revision, modification, or improvement." - [x] `## NEVER` section contains six rules (verify with `grep -c "^- \*\*NEVER" skill-forge-update/SKILL.md` → 6) - [x] Phase 4 close-out template contains a `**Judge:**` line - [x] Re-running `/skill-forge-judge update` returns grade A (≥90%) ## Blockers None --- *Created with [gh-weld](https://github.com/WrathZA/github-weld)*

Comments

WrathZA

## Session Summary - **Initial judge: skill-forge-update graded B (86%, 103/120) with 6 numbered improvements** — [line 128](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L128) - **Constraint discovered: skill-forge should default to project-level skills, not user-level symlinks** — [line 403](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L403) - **Phase 3 zero-change guard added (mirrors NEVER #5 structure)** — [line 561](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L561) - **Description trigger inlined for harness-independent activation** — [line 648](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L648) - **NEVER #6 added: hitl is Phase 2's exclusive apply mechanism** — [line 842](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L842) - **Re-judge: grade A (91%, 109/120) — single residual finding** — [line 912](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L912) - **Phase 4 template/promise mismatch: Phase 3 `(a)` annotation had no slot in close-out** — [line 950](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L950) - **Adopted as issue #22 with branch `feat/22-skill-forge-update-judge-findings`** — [line 1278](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018#file-session-export-md-L1278) --- # 📋 Session Export | | | |---|---| | **File** | `eecaabb5-44d0-4e28-8184-e77c8b7d6978.jsonl` | | **Exported** | 2026-04-29T06:16:48Z | | **Exchanges** | 16 | --- #### 🧑 User <sub>2026-04-27T17:36:51.042Z</sub> <command-message>skill-forge-judge</command-message> <command-name>/skill-forge-judge</command-name> <command-args>update</command-args> --- #### 🤖 Assistant <sub>2026-04-27T17:36:58.500Z</sub> <details><summary>🔧 Read: `/home/bm/code/skill-forge/skill-forge-update/SKILL.md`</summary> ```json { "file_path": "/home/bm/code/skill-forge/skill-forge-update/SKILL.md" } ``` </details> --- #### 🤖 Assistant <sub>2026-04-27T17:36:58.796Z</sub> <details><summary>🔧 Bash: `ls /home/bm/code/skill-forge/skill-forge-update/`</summary> ```json { "command": "ls /home/bm/code/skill-forge/skill-forge-update/", "description": "List skill-forge-update directory contents" } ``` </details> --- #### 🤖 Assistant <sub>2026-04-27T17:37:03.770Z</sub> <details><summary>🔧 Read: `/home/bm/code/skill-forge/skill-forge-judge/references/universal-dimensions.md`</summary> ```json { "file_path": "/home/bm/code/skill-forge/skill-forge-judge/references/universal-dimensions.md" } ``` </details> --- [Full session context](https://gist.github.com/WrathZA/4537afdc91123fead8a72eba9fa20018) --- *Created with [gh-weld](https://github.com/WrathZA/github-weld)*

WrathZA

Forged skill-forge-update from B (86%, 103/120) to A (91%, 109/120) by applying seven judge findings across two cycles. **What changed:** - Terminated two ambiguous decision branches with explicit re-entry semantics - Upgraded Phase 4 from a one-line file write to a structured close-out with project-level skills as the default and a symlink check that only fires for user-level installs - Added a Phase 3 zero-change guard mirroring NEVER #5: surface `(a)/(r)/(q)` instead of silently advancing - Inlined "Use when an existing SKILL.md needs revision..." into the description for harness-independent activation - Opened the drift check with a thinking-lens question to shift from symptom-matching to first-principles evaluation - Added NEVER #6 protecting hitl as Phase 2's exclusive apply mechanism - Added a `**Judge:**` line to the close-out template to absorb the new Phase 3 reviewed-not-applied annotation Shipped in PR #23.