Skip to content

Fix daily note weekday completion aliases#437

Open
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/420-1778363271-daily-note-weekdays
Open

Fix daily note weekday completion aliases#437
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/420-1778363271-daily-note-weekdays

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot commented May 9, 2026

Summary

Fixes daily-note completions so tomorrow/yesterday also expose their weekday aliases, matching workspace command behavior:

  • tomorrow now also has a next <weekday> completion alias.
  • yesterday now also has a last <weekday> completion alias.
  • Daily-note completions can now produce multiple display/filter aliases for the same underlying date while inserting the same date ref.
  • Adds unit tests for the tomorrow/next weekday and yesterday/last weekday cases.

Fixes #420

Review & Testing Checklist for Human

  • Verify in an editor completion menu that, on a Friday, both tomorrow and next Saturday appear for the same daily note target.
  • Verify selecting the alternate alias still inserts/labels the intended daily note reference.
  • Check completion ordering/duplicates around existing daily-note files vs generated unresolved daily-note suggestions.

Notes

Local validation run:

  • cargo fmt --check
  • cargo clippy
  • cargo test

Link to Devin session: https://app.devin.ai/sessions/5bac1535c3f94e018015fb068ac6400b
Requested by: @Feel-ix-343


Open in Devin Review

Co-Authored-By: Felix Zeller <felixazeller@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Copy Markdown
Contributor Author

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 3 additional findings.

Open in Devin Review

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.

Inconsistency in daily note references between autocompletes and workspace commands

1 participant