Skip to content

Fix skills not detected due to nested directory structure#15

Open
nobita2041 wants to merge 1 commit into
makenotion:mainfrom
nobita2041:fix/skills-directory-structure
Open

Fix skills not detected due to nested directory structure#15
nobita2041 wants to merge 1 commit into
makenotion:mainfrom
nobita2041:fix/skills-directory-structure

Conversation

@nobita2041

Copy link
Copy Markdown

Summary

  • Skills (meeting-intelligence, knowledge-capture, research-documentation, spec-to-implementation) are not recognized by Claude Code after plugin installation
  • Root cause: skills are nested under skills/notion/<skill-name>/SKILL.md but Claude Code expects skills/<skill-name>/SKILL.md (one level deep)
  • This PR removes the intermediate notion/ namespace directory so all four skills are properly detected
  • Commands (commands/) are unaffected and work correctly both before and after this change

Reproduction

  1. Install the plugin following the README instructions
  2. Restart Claude Code
  3. Observe that /Notion:search, /Notion:find, etc. (commands) work
  4. Observe that skills like Notion:meeting-intelligence do not appear in the available skills list

Fix

# Before (broken)
skills/
  notion/
    meeting-intelligence/SKILL.md
    knowledge-capture/SKILL.md
    research-documentation/SKILL.md
    spec-to-implementation/SKILL.md

# After (fixed)
skills/
  meeting-intelligence/SKILL.md
  knowledge-capture/SKILL.md
  research-documentation/SKILL.md
  spec-to-implementation/SKILL.md

Verification

After applying the fix and restarting Claude Code, all four skills appear correctly:

  • Notion:meeting-intelligence
  • Notion:knowledge-capture
  • Notion:research-documentation
  • Notion:spec-to-implementation

Test plan

  • Install plugin with this fix applied
  • Restart Claude Code
  • Verify all four skills appear in the skills list
  • Verify existing commands (/Notion:search, /Notion:find, etc.) still work
  • Invoke each skill and confirm it loads the correct SKILL.md content

🤖 Generated with Claude Code

Move skill directories from skills/notion/<skill-name>/ to skills/<skill-name>/.

Claude Code expects skills at skills/<skill-name>/SKILL.md (one level deep),
but the current structure has an extra notion/ namespace directory that prevents
all four skills (meeting-intelligence, knowledge-capture, research-documentation,
spec-to-implementation) from being recognized.

Commands (in commands/) are unaffected and work correctly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
buysellvtproperty added a commit to buysellvtproperty/claude-code-notion-plugin that referenced this pull request May 23, 2026
Skills under skills/notion/<name>/SKILL.md were not loading because
Claude Code's plugin loader expects skills/<name>/SKILL.md (one level
deep). Moving the 4 Workspace Skills up one level fixes discovery.

Same fix as PR makenotion#15 (nobita2041) against upstream issue makenotion#11.
@buysellvtproperty

Copy link
Copy Markdown

Confirmed this fix works. Applied the identical directory flatten in a fork (buysellvtproperty/claude-code-notion-plugin) — all 4 sub-skills (knowledge-capture, meeting-intelligence, research-documentation, spec-to-implementation) now load correctly under the Notion: namespace in Claude Code. Thanks for the patch!

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.

2 participants