Critical: Workflow references broken due to case mismatch

[noxx]

I had Kai do a deep infrastructure audit of PAI and it found several critical issues that break functionality. I've already applied fixes locally but wanted to report them for the public repo:

Issues Found (by Kai autonomously)

1. Workflow File References Use Wrong Case

• Skills/Research/SKILL.md and Skills/Art/SKILL.md reference workflows with kebab-case (workflows/claude-research.md)
• Actual files use TitleCase (workflows/ClaudeResearch.md)
• Impact: Workflow routing completely broken for Research and Art skills

3. Case Mismatch in Hook Paths

• load-dynamic-requirements.ts references ${PAI_DIR}/commands/ (lowercase)
• update-documentation.ts checks for skills/, hooks/, commands/, agents/ (lowercase)
• Actual directories are TitleCase: Skills/, Hooks/, Commands/, Agents/
• Impact: File lookups fail silently

4. Skills Missing Mandatory USE WHEN

• AlexHormoziPitch, Ffuf, and Prompting skills lack USE WHEN in description
• Impact: Claude Code won't auto-activate these skills

Recommended Fixes

All involve updating references to match actual TitleCase file/directory names

Found these issues through automated parallel exploration - Kai launched 5 agents to audit different areas simultaneously and cross-referenced findings.

[sti0]

#226

[hjbrandt]

We independently discovered and fixed these same issues in our local PAI fork. Here's what we found and fixed:

Fixes Applied

1. Directory Renames (TitleCase)

Multiple skills had lowercase workflows/ directories that needed renaming to Workflows/:

• AlexHormoziPitch, Art, BrightData, CreateCLI, Createskill, Ffuf, Observability, Prompting, Research, StoryExplanation

Note: macOS case-insensitive filesystem requires two-step rename:

git mv workflows workflows_tmp && git mv workflows_tmp Workflows

2. Path Reference Fixes

Fixed lowercase path references across skills:

File	Fix
Art/SKILL.md + 14 workflows	Skills/art/tools/ → Skills/Art/tools/
Observability/SKILL.md	Skills/observability/ → Skills/Observability/ (7 occurrences)
BrightData/README.md	Directory tree workflows/ → Workflows/
CORE/ProsodyGuide.md	Removed refs to non-existent files
Createskill/Workflows/*.md	Template paths workflows/ → Workflows/
CORE/HookSystem.md	skills/CORE/SKILL.md → Skills/CORE/SKILL.md (4 occurrences)
CORE/HookSystem.md	${PAI_DIR}/hooks/ → ${PAI_DIR}/Hooks/ (2 occurrences)

3. YAML Frontmatter Fixes

• Fabric/SKILL.md: name: fabric → name: Fabric

Issues Filed

We also filed specific issues for official skills that need upstream fixes:

• AlexHormoziPitch skill references missing Command file #231 - AlexHormoziPitch
• CreateCLI skill: Inconsistent naming and missing workflow files #232 - CreateCLI
• Ffuf skill: TitleCase naming and structural compliance #233 - Ffuf
• Prompting skill: Missing required sections and lowercase path reference #234 - Prompting
• Art skill uses lowercase naming, violates SkillSystem.md TitleCase convention #235 - Art

Validation

All tested skills now pass:

• ✓ Valid YAML frontmatter
• ✓ USE WHEN trigger present
• ✓ Directory name matches YAML name
• ✓ TitleCase naming consistent

Happy to contribute a PR if helpful.