Skip to content

sync integrations/makeswift with canary#3130

Merged
chanceaclark merged 6 commits into
integrations/makeswiftfrom
sync-integrations-makeswift
Jul 23, 2026
Merged

sync integrations/makeswift with canary#3130
chanceaclark merged 6 commits into
integrations/makeswiftfrom
sync-integrations-makeswift

Conversation

@chanceaclark

@chanceaclark chanceaclark commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Syncs integrations/makeswift with the latest canary (through the @bigcommerce/catalyst-core@1.10.1 release), and adds a changeset to release @bigcommerce/catalyst-makeswift (patch).

Notable changes pulled in from canary:

Conflict resolutions

  • core/package.json: kept name as @bigcommerce/catalyst-makeswift and version at 1.9.0 (latest published makeswift version); kept the makeswift-only swr dependency; took canary's storefront-kit bump to ^0.32.3.
  • core/package.json catalyst.version/catalyst.ref: these must mirror the package's own name/version (1.9.0 / @bigcommerce/catalyst-makeswift@1.9.0), not canary's. The merge auto-took canary's raw value here (@bigcommerce/catalyst-core@1.10.1) since this nested field wasn't a textual conflict; corrected by hand. This same bug is present on the current integrations/makeswift tip from the prior sync merge — this PR's merge commit supersedes it.
  • core/CHANGELOG.md: removed the raw ## 1.10.1 / ## 1.10.0 sections copied from canary's changelog — the new sync changeset will generate the correct combined entry when this release is versioned.
  • pnpm-lock.yaml: took canary's lockfile and regenerated with pnpm install --no-frozen-lockfile to account for the makeswift-specific package.json differences.

Do not squash or rebase-and-merge this PR. Use a true merge commit or rebase locally to preserve the merge base between canary and integrations/makeswift.

Test plan

  • CI passes

jairo-bc and others added 5 commits July 21, 2026 11:28
…etch (#3112)

* TRAC-1161: fix - Set request locale in generateMetadata before data fetch

generateMetadata runs before the page body and fetches page data via
cache()-memoized loaders without first calling setRequestLocale. During
ISR regeneration there is no request, so next-intl getServerLocale()
returns undefined, the storefront client omits Accept-Language, and
BigCommerce serves default-locale content that poisons the memoized
cache for the whole render.

Call setRequestLocale(locale) before the fetch in each generateMetadata.

Fixes TRAC-1161

* TRAC-1161: chore - Add changeset for locale metadata fix
…wift (#3126)

Add integrations/b2b-makeswift to the Changesets Release workflow so the branch
can cut its own @bigcommerce/catalyst-b2b-makeswift releases:

- Trigger the workflow on pushes to integrations/b2b-makeswift.
- Map that branch to @bigcommerce/catalyst-b2b-makeswift when resolving the
  headline package for the Linear release notes.

Both additions are inert on canary and integrations/makeswift: a push only runs
the workflow copy on the branch that was pushed, and the headline elif never
matches their ref_name. Keeping this on canary makes it the source of truth for
the shared workflow; integrations/makeswift picks it up via /sync-makeswift and
integrations/b2b-makeswift carries the same file.

Refs TRAC-1235

Co-authored-by: Claude <noreply@anthropic.com>
* build: TRAC-1250 Bump next from 16.2.6 to 16.2.11

Patches the July 2026 Next.js security release
(https://nextjs.org/blog/july-2026-security-release). Also updates
pnpm-lock.yaml and adds a changeset.

Fixes TRAC-1250
Co-Authored-By: Claude <noreply@anthropic.com>

* test: increase upgrade test timeout to 300 s for Windows CI runners

* Revert "test: increase upgrade test timeout to 300 s for Windows CI runners"

This reverts commit fd124cd.

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@chanceaclark
chanceaclark requested a review from a team as a code owner July 23, 2026 21:34
@changeset-bot

changeset-bot Bot commented Jul 23, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 475b35c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@bigcommerce/catalyst-makeswift Patch

Not sure what this means? Click here to learn what changesets are.

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

@vercel

vercel Bot commented Jul 23, 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 23, 2026 9:41pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

Bundle Size Report

Comparing against baseline from aa55606 (2026-07-23).

No bundle size changes detected.

…akeswift

# Conflicts:
#	core/CHANGELOG.md
#	core/package.json
#	pnpm-lock.yaml
@chanceaclark
chanceaclark merged commit 475b35c into integrations/makeswift Jul 23, 2026
16 of 18 checks passed
@chanceaclark
chanceaclark deleted the sync-integrations-makeswift branch July 23, 2026 22:24
@github-actions

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 88 90 89 91

Category Scores

Category Prod Desktop Prod Mobile Preview Desktop Preview Mobile
Performance 64 75 64 79
Accessibility 91 91 91 91
Best Practices 100 100 100 100
SEO 88 89 100 100

Core Web Vitals

Metric Prod Desktop Prod Mobile Preview Desktop Preview Mobile
LCP 5.8 s 7.1 s 6.1 s 5.7 s
CLS 0.001 0 0 0
FCP 1.4 s 1.5 s 1.5 s 1.2 s
TBT 20 ms 10 ms 30 ms 20 ms
Max Potential FID 80 ms 80 ms 80 ms 90 ms
Time to Interactive 7.0 s 7.2 s 7.0 s 6.3 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.

4 participants