Source
CodeRabbit finding #4 from PR #2.
What
MarkdownDetailView.swift contains 4 distinct types:
MarkdownDetailView (the scroll view + anchor link handler)MarkdownBlock enum + parser extension on MarkdownDocumentMarkdownBlockView (the block renderer)ResizableTableView
This violates the project's "one primary type per file" convention from CLAUDE.md.
Why deferred
The file is ~580 lines and all types are tightly related — they form a single rendering pipeline. Splitting now would add 3 new files and Xcode project entries for a cosmetic improvement. The types are not reused elsewhere, so discoverability isn't impacted.
When to revisit
If the file grows past ~800 lines, or if any of these types need to be reused outside the markdown detail view (e.g., MarkdownBlock parser used in export or search).
Files
app/Parsely/Parsely/Views/Screens/MarkdownDetailView.swift
Source
CodeRabbit finding #4 from PR #2.
What
MarkdownDetailView.swiftcontains 4 distinct types:MarkdownDetailView(the scroll view + anchor link handler)MarkdownBlockenum + parser extension onMarkdownDocumentMarkdownBlockView(the block renderer)ResizableTableViewThis violates the project's "one primary type per file" convention from CLAUDE.md.
Why deferred
The file is ~580 lines and all types are tightly related — they form a single rendering pipeline. Splitting now would add 3 new files and Xcode project entries for a cosmetic improvement. The types are not reused elsewhere, so discoverability isn't impacted.
When to revisit
If the file grows past ~800 lines, or if any of these types need to be reused outside the markdown detail view (e.g.,
MarkdownBlockparser used in export or search).Files
app/Parsely/Parsely/Views/Screens/MarkdownDetailView.swift