Review: Create /validate-ticket SKILL.md (lost from #228)

review-674-2026-03-29 Doc

review block

Project

Verdict: BLOCK

Template Completeness

[x] Type -- Bug
[x] Lineage -- references #228, #209
[x] Repo -- forgejo_admin/claude-custom
[x] What Broke -- describes missing SKILL.md on filesystem
[x] Repro Steps -- 4 concrete steps
[x] Expected Behavior -- clear reference to review-ticket pattern
[x] Environment -- local workstation, commit SHA d217aab
[x] Acceptance Criteria -- 4 items
[x] Related -- project + parent issues listed

All bug template sections present. Template is complete.

Traceability

[x] story:pm-scope label -- PM (Ava) scope management story
[x] story note verified -- found in project-pal-e-agency user-stories section (story:pm-scope maps to PM role)
[x] arch:skills label -- skills architecture component
[ ] arch note MISSING -- [SCOPE] No arch-skills note found in pal-e-docs. Search returned zero results.
[x] Forgejo issue -- forgejo_admin/claude-custom#231, open

File Targets

[x] ~/.claude/skills/validate-ticket/SKILL.md -- verified MISSING from local filesystem as claimed
[x] ~/.claude/skills/review-ticket/SKILL.md -- verified EXISTS as reference pattern

However, investigation reveals the file DOES exist in the Forgejo repo (see Repo Placement).

Repo Placement

FUNDAMENTAL SCOPE ERROR. The issue states the SKILL.md "was never persisted" and was "lost in worktree cleanup." Investigation reveals:

PR #230 (feat: add /validate-ticket skill for post-merge validation) was merged to main on claude-custom (merge commit 6bca6be)
The file skills/validate-ticket/SKILL.md exists in the Forgejo repo on main branch
The local checkout ~/claude-custom is 1 commit behind (d217aab vs 6bca6be)
Board item #656 (original #228 work) is correctly in done
The fix is cd ~/claude-custom && git pull origin main followed by hardlink creation -- NOT writing a new SKILL.md

This is an operational sync issue (stale local checkout), not a missing deliverable. A dev agent dispatched against this ticket would create a duplicate SKILL.md or fail in confusion.

Dependencies

Board item #518 (parent #209, "Right-side validation pipeline") is in backlog -- parent decomposed ticket, not a blocker
Board item #656 (original #228) is in done -- the work WAS completed and merged
No in_progress items block this

Acceptance Criteria

All 4 AC are already satisfied in the repo:

[x] ~/.claude/skills/validate-ticket/SKILL.md exists -- in the repo, just not synced locally
[x] Skill parses board-slug#item-id argument -- verified in repo content
[x] Skill dispatches agent that reads skill-validate-ticket from pal-e-docs -- verified
[x] Pattern matches review-ticket skill structure -- same routing/dispatch/verdict pattern

An agent executing this ticket would produce a no-op or duplicate.

Blast Radius

Checked all 27 skill directories in ~/.claude/skills/ against the 28 in the Forgejo repo. Only validate-ticket is missing locally. No other skills have this desync. The root cause is the local clone not being updated after PR #230 merged.

Decomposition Assessment

No decomposition needed. The ticket should not be executed as written -- the premise is invalid. If rewritten as a sync task, it would be a single operational step (git pull + ln), well under the 5-minute rule.

Recommendation

[SCOPE] This ticket is fundamentally invalid. The SKILL.md deliverable exists in the claude-custom repo (merged via PR #230, commit 6bca6be). The local filesystem is 1 commit behind. This is NOT a code bug requiring a new PR.
[SCOPE] Close Forgejo issue #231 as "not a bug" -- the original board item #656 (#228) was correctly completed.
[SCOPE] The operational fix is: cd ~/claude-custom && git pull origin main then mkdir -p ~/.claude/skills/validate-ticket && ln ~/claude-custom/skills/validate-ticket/SKILL.md ~/.claude/skills/validate-ticket/SKILL.md.
[SCOPE] Create architecture note arch-skills for the skills component (missing from pal-e-docs).