Skip to content

ci: TRAC-881 Drop the canary base_ref bootstrap for Linear sync#3090

Merged
chanceaclark merged 1 commit into
canaryfrom
chancellorclark/ltrac-913-remove-canary-base-ref
Jul 8, 2026
Merged

ci: TRAC-881 Drop the canary base_ref bootstrap for Linear sync#3090
chanceaclark merged 1 commit into
canaryfrom
chancellorclark/ltrac-913-remove-canary-base-ref

Conversation

@chanceaclark

@chanceaclark chanceaclark commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Jira: TRAC-881

What/Why?

Fast-follow to #3085/#3086. The bootstrap base_ref (@bigcommerce/catalyst-core@1.8.0) was meant to be a one-time seed so the first sync didn't scan this existing project's full history, but it was left hardcoded permanently. linear-release's own log makes the consequence explicit: passing --base-ref makes it "skip automatic baseline selection" every single run, so every push kept re-scanning the full 1.8.0..HEAD range instead of narrowing to just new commits.

That range included a45ab434 ("Release: Catalyst CLI 1.0.0... merge alpha → canary"), a squash-merge whose commit body is hundreds of lines of already-shipped alpha-branch history — which kept re-attaching ~23 unrelated old issues to the in-progress release on every run.

Verified locally with the standalone CLI (--dry-run) that with no base_ref and no stored baseline, it correctly finds 0 matching commits and skips release creation, rather than falling back to scanning full history. A real (non-dry-run) manual sync was run separately with an explicit --base-ref pointing just past the alpha-merge noise to seed a correctly-scoped release by hand before this PR lands, so the automatic baseline it establishes is clean.

Drops base_ref for both branches down to a single unconditional sync step (no per-branch conditional needed). integrations/makeswift's baseline will be seeded manually the same way, before merging in whatever brings this workflow into that branch — its copy doesn't have these Linear steps yet.

Testing

Watch the next push to canary: the log should show it using its own stored baseline (no "skipping automatic baseline selection" message) and only pick up genuinely new core/** commits.

@changeset-bot

changeset-bot Bot commented Jul 8, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ea18acb

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
catalyst Ready Ready Preview, Comment Jul 8, 2026 8:44pm

Request Review

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Bundle Size Report

Comparing against baseline from ce03afb (2026-07-08).

No bundle size changes detected.

The one-time bootstrap boundary (base_ref: @bigcommerce/catalyst-core@1.8.0)
was hardcoded permanently rather than removed after seeding a baseline, so
every push kept re-scanning the full 1.8.0..HEAD range instead of narrowing
to just new commits. That range included a squash-merged "alpha" branch
commit whose message body listed dozens of already-shipped issues, which
kept getting reattached to the in-progress release on every run.

Drop base_ref for both branches down to a single unconditional sync step.
A real baseline was seeded manually for canary before this lands, and
integrations/makeswift's baseline will be seeded manually the same way
before merging in whatever brings these workflow changes into that branch.

Refs TRAC-881

Co-Authored-By: Claude <noreply@anthropic.com>
@chanceaclark
chanceaclark force-pushed the chancellorclark/ltrac-913-remove-canary-base-ref branch from 29cba80 to ea18acb Compare July 8, 2026 20:44
@chanceaclark
chanceaclark marked this pull request as ready for review July 8, 2026 20:46
@chanceaclark
chanceaclark requested a review from a team as a code owner July 8, 2026 20:46
@chanceaclark
chanceaclark added this pull request to the merge queue Jul 8, 2026
Merged via the queue into canary with commit 4ef6b2c Jul 8, 2026
17 of 18 checks passed
@chanceaclark
chanceaclark deleted the chancellorclark/ltrac-913-remove-canary-base-ref branch July 8, 2026 21:22
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Unlighthouse Performance Comparison — Vercel

Comparing PR preview deployment Unlighthouse scores vs production Unlighthouse scores.

Summary Score

Aggregate score across all categories as reported by Unlighthouse.

Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Score 89 92 91 93

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 76 80 71 83
Accessibility 95 95 95 92
Best Practices 100 100 100 100
SEO 88 88 88 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 3.7 s 5.3 s 5.7 s 4.7 s
CLS 0.001 0 0.037 0
FCP 1.2 s 1.2 s 1.2 s 1.1 s
TBT 40 ms 40 ms 0 ms 0 ms
Max Potential FID 70 ms 90 ms 40 ms 50 ms
Time to Interactive 3.7 s 5.3 s 5.7 s 4.7 s

Full Unlighthouse report →

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.

2 participants