Skip to content

fix: Standardize TitleCase naming across all Skills - #236

Closed
hjbrandt wants to merge 1 commit into
danielmiessler:mainfrom
hjbrandt:fix/titlecase-naming-consistency
Closed

fix: Standardize TitleCase naming across all Skills#236
hjbrandt wants to merge 1 commit into
danielmiessler:mainfrom
hjbrandt:fix/titlecase-naming-consistency

Conversation

@hjbrandt

Copy link
Copy Markdown

Summary

This PR enforces the TitleCase naming convention specified in SkillSystem.md across all skills. This addresses the inconsistencies reported in #229.

Changes

Directory Renames

Renamed workflows/ to Workflows/ in 6 skills:

  • Art
  • BrightData
  • CreateCLI
  • Createskill
  • Research
  • StoryExplanation

YAML Name Field Fixes

Updated name: in SKILL.md to match directory name (TitleCase):

Skill Before After
AlexHormoziPitch alex-hormozi-pitch AlexHormoziPitch
Art art Art
BrightData brightdata BrightData
CreateCLI system-createcli CreateCLI
Fabric fabric Fabric
Ffuf ffuf-web-fuzzing Ffuf
Prompting prompting Prompting
Research research Research
StoryExplanation story-explanation StoryExplanation

Path Reference Fixes

Updated lowercase path references to TitleCase:

File Fix
Art/SKILL.md + 14 workflows Skills/art/tools/Skills/Art/tools/
BrightData/SKILL.md Skills/brightdata/workflows/Skills/BrightData/Workflows/
CORE/HookSystem.md skills/CORE/SKILL.mdSkills/CORE/SKILL.md
Observability/SKILL.md Skills/observability/Skills/Observability/
Prompting/SKILL.md Skills/prompting/Skills/Prompting/
Research/SKILL.md Skills/research/workflows/Skills/Research/Workflows/
Research/Workflows/Fabric.md Skills/fabric/Skills/Fabric/
Createskill/Workflows/*.md Template paths workflows/Workflows/

Test Plan

  • Verified all YAML frontmatter is valid
  • Verified all USE WHEN triggers present
  • Verified directory names match YAML names
  • Verified all path references use TitleCase
  • No custom/local code included

Notes

  • CONSTITUTION.md contains example/template paths (e.g., Skills/blogging/) that were intentionally left as-is since they are illustrative examples, not actual paths
  • Tested on macOS which required two-step renames due to case-insensitive filesystem

Fixes #229

🤖 Generated with Claude Code

This PR enforces the TitleCase naming convention specified in
SkillSystem.md across all skills.

## Directory Renames
- `workflows/` → `Workflows/` (6 skills: Art, BrightData, CreateCLI,
  Createskill, Research, StoryExplanation)

## YAML Name Field Fixes
Updated `name:` to match directory name in TitleCase:
- AlexHormoziPitch: `alex-hormozi-pitch` → `AlexHormoziPitch`
- Art: `art` → `Art`
- BrightData: `brightdata` → `BrightData`
- CreateCLI: `system-createcli` → `CreateCLI`
- Fabric: `fabric` → `Fabric`
- Ffuf: `ffuf-web-fuzzing` → `Ffuf`
- Prompting: `prompting` → `Prompting`
- Research: `research` → `Research`
- StoryExplanation: `story-explanation` → `StoryExplanation`

## Path Reference Fixes
Updated lowercase path references to TitleCase:
- Art/SKILL.md + 14 workflows: `Skills/art/tools/` → `Skills/Art/tools/`
- BrightData/SKILL.md: `Skills/brightdata/workflows/` → `Skills/BrightData/Workflows/`
- CORE/HookSystem.md: `skills/CORE/SKILL.md` → `Skills/CORE/SKILL.md`
- Observability/SKILL.md: `Skills/observability/` → `Skills/Observability/`
- Prompting/SKILL.md: `Skills/prompting/` → `Skills/Prompting/`
- Research/SKILL.md: `Skills/research/workflows/` → `Skills/Research/Workflows/`
- Research/Workflows/Fabric.md: `Skills/fabric/` → `Skills/Fabric/`
- Createskill/Workflows/*.md: Template paths `workflows/` → `Workflows/`

Fixes danielmiessler#229

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@danielmiessler

Copy link
Copy Markdown
Owner

PAI 2.0 Migration Notice

Thank you for this contribution! We've recently released PAI v2.0, which takes a fundamentally different approach from v1.x.

What changed: PAI v1.x tried to mirror the entire Kai system as an installable template. That approach created too many interconnected dependencies—change one piece, and others broke. v2.0 shifts to modular packs: self-contained, independently installable capabilities that work without understanding the whole system.

This PR: Because the architecture is now fundamentally different, we're closing all open v1.x PRs and issues. This isn't us ignoring your contribution—we genuinely appreciate it! It's that the system has changed so significantly that we need to revisit these within the new context.

Next steps:

  1. Read the updated README (especially "The Journey: PAI v1.x → v2.0")
  2. Check out the new PAI Packs system
  3. If the issue you were addressing still exists in 2.0, please open a new issue or PR with context for the new architecture

We're absolutely happy to address these concerns—just within the 2.0 framework. Thanks for your patience and for contributing to PAI! 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Critical: Workflow references broken due to case mismatch

2 participants