-
Notifications
You must be signed in to change notification settings - Fork 0
FOLLOW-UP: Eliminate double frontmatter parse in ParseFile (from PR #207) #212
Copy link
Copy link
Open
Labels
P3-lowLow: extended features, operational docsLow: extended features, operational docsfollow-upFollow-up task from PR reviewFollow-up task from PR reviewfrom-reviewIssue originated from PR reviewIssue originated from PR review
Description
Context
This follow-up task was identified during the review of PR #207.
Source PR: #207
PR Title: feat: auto-tables — infer interactive UI from markdown tables + YAML sources
Suggested by: @claude[bot]
Task Description
ParseFile in page.go calls extractFrontmatter to get sources for auto-table matching, then ParseMarkdownWithPartials parses the same frontmatter again. This is a minor performance concern.
Fix: Thread the already-parsed frontmatter/sources through the preprocessing pipeline instead of re-parsing. This would require refactoring ParseFile to extract frontmatter once and pass it to both preprocessAutoTables and ParseMarkdownWithPartials.
This issue was automatically created by prmonitor from PR review comments.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P3-lowLow: extended features, operational docsLow: extended features, operational docsfollow-upFollow-up task from PR reviewFollow-up task from PR reviewfrom-reviewIssue originated from PR reviewIssue originated from PR review