Skip to content

Add Dayline 0.2.8 changelog - #78

Merged
robin-liquidium merged 2 commits into
mainfrom
codex/changelog-0.2.8
Sep 6, 2026
Merged

robin-liquidium merged 2 commits into
mainfrom
codex/changelog-0.2.8

Conversation

@robin-liquidium

Copy link
Copy Markdown
Owner

Add the 0.2.8 release notes for #77, covering issue detail previews and configurable click actions, Markdown styling, Settings improvements, focused meeting-alert defaults, and native date/shortcut controls. Each item explains the user-facing behavior and links to its source PR.

Validation: website changelog tests pass. This entry is included in the tagged source so the release workflow can generate matching GitHub release notes.

@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 3 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Team

Run ID: cbf658e5-539a-4d98-9b6d-234307d4acb2

📥 Commits

Reviewing files that changed from the base of the PR and between 42500d2 and 01a12db.

📒 Files selected for processing (1)
  • changelog.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 6, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
dayline-website 01a12db Sep 06 2026, 08:00 PM

@opencode-agent

opencode-agent Bot commented Sep 6, 2026

Copy link
Copy Markdown

PR Review: #78 — Add Dayline 0.2.8 changelog

I reviewed the diff (a single new 0.2.8 entry in changelog.json) against the schema, the consumers (website changelog.ts/changelog.test.ts and release_notes_for_version in script/notarization_release.sh), and fact-checked each item against the actual PR #77 changes (commit 42500d2).

Verdict: Approve — one minor wording nit

✅ What checks out

  • Schema conformance: valid JSON, correct semver format, ISO date, only known keys (version/date/new/fixed, title/text/pr), newest-first ordering, unique item texts — all covered by website/src/changelog.test.ts.
  • Release-notes pipeline compatibility: the jq renderer handles the optional title and pr fields exactly as used here; "fixed" renders as "Improvements and bug fixes", so grouping polish items under fixed is consistent with how the GitHub release notes will read (and with precedent, e.g. 0.2.6).
  • Fact-checks against Polish issue details, settings, and native editors #77 — all accurate:
    • "Settings → Issues → Opening issues" — matches the new Label("Opening issues", …) section and IssueClickAction/IssueClickModifier models; "Space still shows details while hovering" matches the hint text verbatim.
    • Meeting alerts defaulting to link-only — confirmed: meetingAlertRequiresMeetingLink default flipped false → true while preserving stored values ("keeping existing saved choices" ✓).
    • Menu-bar title behavior — confirmed in CalendarEventItem.menuBarCandidate: ongoing events persist via the isHappening branch, upcoming events now use date < event.startDate.
    • Native date controls — GraphicalDatePicker.swift deleted, CalendarDatePickerField.swift added ✓.
    • Shortcut recorder Escape-to-cancel and Settings window/menu behavior — match the ShortcutRecorderView and DaylineApp.swift changes ✓.

🟡 Minor issue (non-blocking)

changelog.json line 26"The unused After event starts setting is removed": the actual setting label in Settings was "Show title after" (Settings → Calendar → Menu Bar Title, search item menuBarEventPostStartGrace). A user looking for "After event starts" won't recognize it. Suggest matching the real label:

"text": "Meeting alerts now default to events with a video-call link, while keeping existing saved choices. The unused \"Show title after\" setting is removed; ongoing events remain in the menu-bar title until they end."

💡 Optional suggestion

"More readable Markdown" and "Native date and shortcut controls" are arguably feature-ish enough for the new section, but since the renderer labels fixed as "Improvements and bug fixes," the current placement reads fine in both the website and GitHub release notes. No change needed.

No code-quality or correctness bugs — this is safe to merge, with the one wording fix above worth considering for user clarity.

New%20session%20-%202026-09-06T19%3A55%3A18.403Z
opencode session  |  github run

@opencode-agent

opencode-agent Bot commented Sep 6, 2026

Copy link
Copy Markdown

PR Review: #78 — Add Dayline 0.2.8 changelog

Verdict: ✅ Approve — no issues found.

This is a data-only change (one new 0.2.8 entry in changelog.json, +37/−0). I reviewed it independently against the schema, both consumers, and the actual PR #77 changes.

Code quality & correctness checks (all pass)

  • Valid JSON, and every constraint enforced by website/src/changelog.test.ts verified directly with jq: semver version, real ISO date, only known release/item keys, non-empty texts/titles, positive integer pr, unique item texts, and newest-first semver ordering (0.2.8 correctly precedes 0.2.7; the file contains 0.2.10-era entries handled correctly by numeric sort).
  • Release pipeline compatibility: ran the exact release_notes_for_version jq renderer from script/notarization_release.sh against 0.2.8 — output is well-formed, the optional title renders as **Title** — text (#77), the title-less item renders cleanly, and fixed maps to "Improvements and bug fixes", so the polish-item placement reads correctly on GitHub.

Fact-checks against PR #77 (commit 42500d2) — all accurate

  • "Settings → Issues → Opening issues" — matches the new Label("Opening issues", …) section; "Space still shows details while hovering" is verbatim from the hint text.
  • Meeting alerts default — confirmed meetingAlertRequiresMeetingLink default flipped false → true while preserving stored values, so "keeping existing saved choices" is precise.
  • "Show title after" — this is the actual label of the removed picker (search item menuBarEventPostStartGrace); the prior review's wording nit was already fixed in commit 01a12db.
  • Native date controlsGraphicalDatePicker.swift deleted, CalendarDatePickerField.swift added.
  • Escape-to-cancel — confirmed in ShortcutRecorderView.swift (kVK_Escape handling + accessibility hints).
  • Settings window behavior — matches the "Dismiss the menu for Settings" and "Recognize the current Settings window when bringing it forward" commits.

Notes

  • The earlier opencode-agent review's one nit (setting label wording) is already resolved in this branch.
  • Couldn't run bun test locally (bun not installed), so I replicated the test assertions via jq instead — all pass; the Cloudflare build is also in progress.

Nothing blocking — safe to merge.

New%20session%20-%202026-09-06T20%3A00%3A30.042Z
opencode session  |  github run

@robin-liquidium
robin-liquidium merged commit 23b156b into main Sep 6, 2026
12 checks passed
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.

1 participant