-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
P4-backlogBacklog: future, nice-to-haveBacklog: future, nice-to-havefollow-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 #145.
Source PR: #145
PR Title: feat: auto-interactive markdown checkboxes
Suggested by: @claude-bot
Priority: Low
Task Description
The frontmatterPattern regex in auto_tasks.go uses \A---\n which requires exactly \n newlines. Files with \r\n line endings (Windows) won't match, causing the frontmatter to be processed as content and potentially breaking task list detection.
Tasks:
- Add unit test with
\r\nline endings to confirm the behavior - If it fails, update the regex to
\A---\r?\n(.+?)\r?\n---\r?\n - Consider if this also affects other regex patterns in the file
Original Comment
The frontmatterPattern uses
\A---\nwhich requires exactly one newline. Files with\r\nline endings (Windows) or---\r\nwon't match.
This issue was automatically created by prmonitor from PR review comments.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P4-backlogBacklog: future, nice-to-haveBacklog: future, nice-to-havefollow-upFollow-up task from PR reviewFollow-up task from PR reviewfrom-reviewIssue originated from PR reviewIssue originated from PR review