Skip to content

Add onlyReferencedTables configuration option#866

Merged
ebullient merged 2 commits into
mainfrom
claude/admiring-colden
Apr 9, 2026
Merged

Add onlyReferencedTables configuration option#866
ebullient merged 2 commits into
mainfrom
claude/admiring-colden

Conversation

@ebullient
Copy link
Copy Markdown
Owner

Changes

  • ✨ Implement onlyReferencedTables configuration option to filter table notes by reference
  • Tables are now only emitted if they are:
    • Linked from included content via {@table} tags or inline table matching, or
    • Explicitly included via filter rules
  • Add tracking of referenced table keys in Tools5eIndex
  • Record table references during linkification in Tools5eLinkifier
  • Implement two-pass conversion logic in Tools5eMarkdownConverter when filtering is enabled
  • Add configuration schema, documentation, and test sample updates
  • Unreferenced tables are logged with (drop | unreferenced) tag when running with --log

Resolves

ebullient and others added 2 commits April 8, 2026 15:16
Add `onlyReferencedTables` config option. When enabled, table notes are
only written if they are linked from other included content (via `{@table}`
tags or inline table matching) or explicitly targeted by an include rule.

Reference tracking is done at render time in Tools5eLinkifier.createLink()
for table/tableGroup types. Tools5eMarkdownConverter.writeFiles() uses a
two-pass approach when the option is enabled: non-tables render first
(populating the reference set), then only referenced tables are written.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ebullient ebullient force-pushed the claude/admiring-colden branch from 0931d3d to ab6227a Compare April 8, 2026 19:17
@ebullient ebullient merged commit e7b34f5 into main Apr 9, 2026
6 checks passed
@ebullient ebullient deleted the claude/admiring-colden branch April 9, 2026 11:04
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