Repository navigation
Quick add: every other Saturday keeps its weekday; connectors and start phrases leave the title - #1019
Merged
Conversation
…rt phrases leave the title The rule grammar gains "every N weeks on <weekdays>" (an RRULE FREQ=WEEKLY;INTERVAL=N;BYDAY with the due date as its start), so "take the bins out today and repeat every other Saturday" is due today and moves to the right Saturday when completed. "and repeat", "repeating", "and then" and their equivalents in every bundled language go with the rule, and "starting <date>" / "from <date>" next to a rule set where the series starts. Shared vectors pin the web parser and the Go port. Fixes #1016 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Bkw4TZLFDGeMdZR3vWZF13
The week phase of the new rule and the parser's start phrases, connectors and week-on words load with quick add and every list (0.1 KB over). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Bkw4TZLFDGeMdZR3vWZF13
Collaborator
Author
|
Raised the first-load budget to 687 KB (measured 686.1 KB, 0.1 KB over): the new rule's week phase and the parser's start phrases, connectors and week-on words load with quick add and every list. The earlier Web test failure (objectivesSwitch.test.tsx) is unrelated and passes locally. |
This was referenced Nov 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1016
What changed
every N weeks on <weekdays>(every 2 weeks on saturday,every 3 weeks on mon, fri), ininternal/recurrenceandweb/src/lib/recurrence.ts. The next occurrence is the first of those weekdays after the due date, in a Monday-to-Sunday week that is a multiple of N weeks from the due date's week. That phase holds through catch-up. These are the same dates asFREQ=WEEKLY;INTERVAL=N;BYDAY=…with the due date as DTSTART. A due date on another weekday counts as an occurrence of its own, and the ones after it follow the rule.every week on tueis stored asevery tuesday. CalDAV serves the rule as that RRULE and reads it back the same way. Habits count the same dates. The importer now readsevery other <weekday>.every other <weekday>andevery N weeks on <weekday>(alle 2 Wochen am Samstag,toutes les 2 semaines le samedi…) produce the new rule.and repeat,repeat,repeating,and then,andand their equivalents. They stay in the title when no rule follows ("Repeat the test").starting today/tomorrow/<date>,from <date>,ab,à partir de,a partir de,a partire da,desde… The task is then due on the rule's first date on or after the start date. A date phrase of its own stays the first occurrence, even when it isn't on the rule's weekday.quick-add-weekday-recurrence(archived; it updatesrecurring-tasksandnatural-language-capture). Docs:recurrence.md,quick-add.md,repeat-rules.md,import.md. Changelog fragment included.Results (today = Wed 2026-11-04)
Checks
go test ./internal/quickadd ./internal/recurrence ./internal/importer ./internal/caldav ./internal/vault ./internal/sync ./internal/agenttools ./internal/cli;npx vitest run(full suite;seriesSplit.test.tsxfailed once under parallel load and passes on its own, and it usesevery tuesday, which this change leaves alone);scripts/fuzz.sh 30s ./internal/quickadd,scripts/fuzz.sh 20s ./internal/recurrence;make openspec-validate,make docs-check,make repo-check.Out of scope:
every 2 Mondayskeeps its literal reading. The week phase does not follow the week-start preference.🤖 Generated with Claude Code
https://claude.ai/code/session_01Bkw4TZLFDGeMdZR3vWZF13