Skip to content

[codex] fix: escape yaml frontmatter and obsidian tags#360

Open
aa-matsunari wants to merge 1 commit intosafishamsi:v4from
aa-matsunari:codex/harden-yaml-frontmatter
Open

[codex] fix: escape yaml frontmatter and obsidian tags#360
aa-matsunari wants to merge 1 commit intosafishamsi:v4from
aa-matsunari:codex/harden-yaml-frontmatter

Conversation

@aa-matsunari
Copy link
Copy Markdown

Summary

  • write source_nodes as escaped YAML list items
  • escape Obsidian frontmatter values and slug community tags/query strings

Changed files

  • graphify/ingest.py
  • graphify/export.py
  • tests/test_ingest.py
  • tests/test_export.py

Impact

  • prevents malformed YAML/frontmatter and broken Dataview queries when labels contain quotes or newlines

Tests

  • /tmp/graphify-review/.venv/bin/python -m pytest /tmp/graphify-review-pr4/tests/test_ingest.py /tmp/graphify-review-pr4/tests/test_export.py

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