Skip to content

ENG-1943 Update Roam to version 0.22.0#1152

Merged
mdroidian merged 2 commits into
mainfrom
eng-1943-roam-release-0210-changelogpackage
Jun 22, 2026
Merged

ENG-1943 Update Roam to version 0.22.0#1152
mdroidian merged 2 commits into
mainfrom
eng-1943-roam-release-0210-changelogpackage

Conversation

@mdroidian

@mdroidian mdroidian commented Jun 22, 2026

Copy link
Copy Markdown
Member

@linear-code

linear-code Bot commented Jun 22, 2026

Copy link
Copy Markdown

ENG-1943

@supabase

supabase Bot commented Jun 22, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@vercel

vercel Bot commented Jun 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
discourse-graph Skipped Skipped Jun 22, 2026 10:18pm

Request Review

@mdroidian mdroidian merged commit ee5c5d2 into main Jun 22, 2026
8 checks passed
@mdroidian mdroidian deleted the eng-1943-roam-release-0210-changelogpackage branch June 22, 2026 22:19

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

Open in Devin Review

Comment thread apps/roam/package.json
{
"name": "roam",
"version": "0.21.0",
"version": "0.22.0",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 Post-submission version bump bundled with changelog

The apps/roam/RELEASE.md:19-26 release process states that the post-submission package bump to the next version "must happen after roam-release.yaml runs, because that workflow reads apps/roam/package.json for the version being sent to Roam" and should be done "in a follow-up PR." This PR bundles the 0.21.0 changelog with the bump to 0.22.0 in a single PR. If the roam-release.yaml workflow has not yet been triggered for 0.21.0, merging this PR first would cause the workflow to read 0.22.0 from apps/roam/package.json:3 instead of 0.21.0, resulting in incorrect version metadata in Linear, PostHog source maps, and Roam Depot. The previous release cycle kept these separate (commit c51cd7f4 for changelog, f845cb5a for bump). If the workflow has already been run, this is fine.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Comment thread apps/roam/CHANGELOG.md
Comment on lines +16 to +17
- **Build metadata includes the commit SHA** - Release builds now stamp version metadata with the build commit, and branch builds keep the date/branch/commit format.
- **Per-install Roam extension version telemetry** - Extension load analytics now include per-install version metadata to help debug Roam app dev URL caching.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 Changelog includes build-only and telemetry-only items

The apps/roam/RELEASE.md:43-45 guidance says "Internal chores, release-process tasks, telemetry-only work, and build-only work should usually stay out of the public changelog." Two of the five entries in the new 0.21.0 changelog section appear to fall into these categories:

  • "Build metadata includes the commit SHA" (build-only)
  • "Per-install Roam extension version telemetry" (telemetry-only)

The word "usually" provides flexibility, and the team may have decided these are user-relevant, but it's worth confirming intentionality.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

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