v2.0.3 fix: decode XML-plist recurrence rules, hide templates from someday

[kaufmanhenry]

Summary

Two recurring-task bugs, both from the same wrong-format assumption as 2.0.2 — the code expected an older recurrence encoding that current Things no longer writes. Found while auditing a real database where someday returned 26 items (app showed ~10) and every repeat decoded as UNKNOWN.

src/lib/rrule.js — recurrence decoder rewrite. Current Things stores rt1_recurrenceRule as an XML property list (fu = NSCalendarUnit unit: 4=year, 8=month, 16=day, 256=weekday; fa = interval; of = weekday/day-of-month), not the binary-plist frequency/interval format the old heuristic sniffed for. Parse the XML so cadences decode (every week, every 2 weeks, every month, every 2 days).

src/lib/queries.js + src/commands/stats.js — stop templates leaking into Someday. Repeating-task templates are stored with start = 2 (same value as Someday); the app hides them and shows only generated instances. somedayTasks and the stats someday count now exclude rows with a recurrence rule.

src/commands/repeating.js — next-instance decoding. rt1_nextInstanceStartDate is a bit-packed calendar date, not Unix seconds. The old >= 1000000000 guard was never true for packed dates, so nextInstance was always null. Now decoded with thingsDateToIso / formatThingsShortDate.

Test Coverage

• test/fixtures/build.js: fixture now emits real recurrence-rule XML and adds a Someday-template row (start=2, todayIndex=0, recurrence rule) that reproduces the real-DB leak.
• test/integration/lists.test.js: someday excludes the template; repeating lists it and decodes freq/interval; nextInstance resolves to a real calendar date.
• test/unit/rrule.test.js: decodes daily/weekly/monthly/yearly from fu, reads interval from fa, extracts weekday/day-of-month, accepts string or Buffer.

Tests: 79 → 79 passing (added recurrence + someday-exclusion regression cases).

Verification (real database)

• someday: 26 → 10 items (templates gone, matches the app)
• repeating: all 33 repeaters decode with cadence + next-instance date (genuinely-null next-instances stay null)

Pre-Landing Review

Self-reviewed; no SQL/trust-boundary/side-effect issues (read-only queries, no user input interpolated). No prompt files (evals skipped), no frontend (design review skipped).

Test plan

• `npm test` — 13 suites, 79 tests pass (fixture rebuilt each run)
• Verified `someday` + `repeating` against the live Things DB

🤖 Generated with Claude Code