Skip to content

feat: add due date view - #186

Open
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:feat/due-date-view
Open

feat: add due date view#186
vycdev2 wants to merge 1 commit into
vycdev:developfrom
vycdev2:feat/due-date-view

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 9, 2026

Copy link
Copy Markdown

Summary

  • add a Due activity-bar view that groups unfinished @due(...) tasks by local calendar date
  • expand today by default, refresh on Todo edits, file changes, configuration changes, and midnight rollover
  • support existing task actions in the Due view and reject impossible formatted due dates

Verification

  • npm run compile — passed (webpack development build)
  • npm test — passed (138 tests)
  • npm run format:check — passed
  • git diff --check — passed

Risk

  • Medium: adds a new tree view and refresh paths, with extraction/date behavior covered by focused tests and the full suite.

Closes #68

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev vycdev left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Due view currently interprets examples inside inline code as live task metadata.

Comment thread src/utils/due_tasks.ts
todoRegex: RegExp,
finishedRegex: RegExp,
today: Date = new Date(),
soonDays: number = 7

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both this finished check and the @due(...) scan below operate on unmasked text. As a result, ☐ Explain \@done` @Due(2026-08-10)is wrongly omitted, while☐ Explain `@due(2026-08-10)`is wrongly added to the Due view. Please reuse the inline-code masking/status helper now ondevelop` before classifying the status and extracting due tags, with regressions for both cases.

@vycdev2
vycdev2 force-pushed the feat/due-date-view branch from 4f8d0e3 to 666a75e Compare August 16, 2026 01:06
@vycdev2

vycdev2 commented Aug 16, 2026

Copy link
Copy Markdown
Author

Addressed the inline-code review feedback and rebased onto current develop.

  • Due task collection now masks matching inline-code spans before task-status classification and @due(...) extraction while preserving the original display text.
  • Added regressions proving inline `@done` does not hide a pending due task and inline `@due(...)` does not create a Due view entry.

Verification: focused RED reproduced the issue (3 passing, 1 failing); focused GREEN passed (4 passing); npm test passed with 234 tests; npm run compile, npm run format:check, and git diff --check develop...HEAD passed.

Updated commit: 666a75e.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

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.

2 participants