Skip to content

fix(nav): exclude CLAUDE.md and Attachments from sidebar navigation - #26

Merged
GianRomani merged 1 commit into
mainfrom
feat/exclude-internal-files-from-nav
Aug 18, 2026
Merged

fix(nav): exclude CLAUDE.md and Attachments from sidebar navigation#26
GianRomani merged 1 commit into
mainfrom
feat/exclude-internal-files-from-nav

Conversation

@GianRomani

Copy link
Copy Markdown
Owner

Summary

Excludes internal configuration files (CLAUDE.md, AGENTS.md) from published documentation note pages, and prevents non-content asset directories (Attachments/, Templates/, .skills/) from generating empty navigation sections in the sidebar menu.

Changes

  • Suppress Internal Page Generation: Added exclusions in convert.py for CLAUDE.md and AGENTS.md so they are not parsed as public docs or added to the knowledge graph nodes.
  • Suppress Non-Note Navigation Sections: Folders matching Attachments, Templates, .skills, etc. do not have _index.md generated, removing "Attachments" from the mobile off-canvas drawer and desktop sidebar.
  • Preserve Asset Copying: All image files and attachments inside Attachments/ continue to be copied as static resources, preserving all note image embeds (![[image.png]]).

Local Decision Documentation

  • .notes/decisions/2026-08-18-exclude-internal-files-from-nav.md

@netlify

netlify Bot commented Aug 18, 2026

Copy link
Copy Markdown

Deploy Preview for gianfree-notes ready!

Name Link
🔨 Latest commit 5ee7e9b
🔍 Latest deploy log https://app.netlify.com/projects/gianfree-notes/deploys/6a84969726e10600081bdffc
😎 Deploy Preview https://deploy-preview-26--gianfree-notes.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@GianRomani
GianRomani merged commit 8d43cca into main Aug 18, 2026
8 checks passed
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.

1 participant