Skip to content

feat(detect): add hidden path allowlist#583

Open
Bichalla wants to merge 1 commit intosafishamsi:v5from
Bichalla:graphifyinclude-hidden-path-allowlist
Open

feat(detect): add hidden path allowlist#583
Bichalla wants to merge 1 commit intosafishamsi:v5from
Bichalla:graphifyinclude-hidden-path-allowlist

Conversation

@Bichalla
Copy link
Copy Markdown

Summary

  • Add .graphifyinclude loading to detect().
  • Keep hidden paths skipped by default, but traverse allowlisted hidden paths needed for curated docs.
  • Keep sensitive-looking filenames hard-skipped even when allowlisted.
  • Add regression tests for .hermes/plans/**/*.md inclusion and sensitive allowlist hard-skip.

Verification

  • uv run --with pytest python -m pytest tests/test_detect.py -q → 30 passed
  • uv run --with pytest python -m pytest -q → 443 passed

Context

This supports SSOT-safe project operating docs: hidden operating folders like .hermes/plans/ can remain canonical while Graphify indexes only explicitly curated Markdown paths.

Summary:\n- Add .graphifyinclude loading for detect().\n- Allow curated hidden paths only when explicitly allowlisted.\n- Keep sensitive-looking filenames hard-skipped even when allowlisted.\n- Add tests for hidden .hermes document inclusion and sensitive allowlist hard-skip.\n\nVerification:\n- uv run --with pytest python -m pytest tests/test_detect.py -q\n- uv run --with pytest python -m pytest -q\n\nNotes:\n- .opencode/ remains untracked and intentionally unstaged.\n- origin is upstream safishamsi/graphify; push requires a user-owned fork/branch.
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