Skip to content

chore: switch kysely to workspace catalog#1144

Merged
ascorbic merged 1 commit into
mainfrom
fix/workerd-kysely-catalog
May 22, 2026
Merged

chore: switch kysely to workspace catalog#1144
ascorbic merged 1 commit into
mainfrom
fix/workerd-kysely-catalog

Conversation

@ascorbic
Copy link
Copy Markdown
Collaborator

What does this PR do?

The new packages/workerd (@emdash-cms/sandbox-workerd, added in #426) declared kysely at ^0.27.0, while the rest of the monorepo has been on ^0.29.0 for a while. That meant pnpm install was happy to resolve two different kysely versions side by side.

This PR:

  • Adds kysely: ^0.29.0 to the pnpm-workspace.yaml catalog.
  • Switches the direct kysely references in emdash, @emdash-cms/auth-atproto, and @emdash-cms/sandbox-workerd to catalog:.
  • Bumps sandbox-workerd's kysely peer range from >=0.27.0 to >=0.29.0 so it reflects the version actually exercised in CI.
  • Leaves the peer-only references in @emdash-cms/auth and @emdash-cms/cloudflare alone — peers stay as explicit ranges.
  • Regenerates pnpm-lock.yaml.

Closes #

Type of change

  • Bug fix
  • Feature (requires maintainer-approved Discussion)
  • Refactor (no behavior change)
  • Translation
  • Documentation
  • Performance improvement
  • Tests
  • Chore (dependencies, CI, tooling)

Checklist

  • I have read CONTRIBUTING.md
  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm test passes (or targeted tests for my change)
  • pnpm format has been run
  • I have added/updated tests for my changes (if applicable)
  • User-visible strings in the admin UI are wrapped for translation (if applicable). Do not include messages.po changes except in translation PRs — a workflow extracts catalogs on merge to main.
  • I have added a changeset (if this PR changes a published package)
  • New features link to an approved Discussion: https://github.com/emdash-cms/emdash/discussions/...

AI-generated code disclosure

  • This PR includes AI-generated code — model/tool: Claude Opus 4.7

Screenshots / test output

None — package metadata + lockfile only.

Adds kysely to the pnpm-workspace.yaml catalog and points the direct
references in core, auth-atproto, and sandbox-workerd at it. Also bumps
sandbox-workerd's kysely peer range from >=0.27.0 to >=0.29.0 to match
the version the rest of the monorepo has been on since the kysely 0.29
upgrade.
Copilot AI review requested due to automatic review settings May 22, 2026 13:06
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: 9f4e73c

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

This PR includes changesets to release 1 package
Name Type
@emdash-cms/sandbox-workerd 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

@cloudflare-workers-and-pages
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
emdash-i18n 9f4e73c May 22 2026, 01:06 PM

@cloudflare-workers-and-pages
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
emdash-perf-coordinator 9f4e73c May 22 2026, 01:06 PM

@cloudflare-workers-and-pages
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
docs 9f4e73c May 22 2026, 01:07 PM

@cloudflare-workers-and-pages
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
emdash-demo-cache 9f4e73c May 22 2026, 01:07 PM

@cloudflare-workers-and-pages
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
emdash-playground 9f4e73c May 22 2026, 01:08 PM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 22, 2026

Open in StackBlitz

@emdash-cms/admin

npm i https://pkg.pr.new/@emdash-cms/admin@1144

@emdash-cms/auth

npm i https://pkg.pr.new/@emdash-cms/auth@1144

@emdash-cms/blocks

npm i https://pkg.pr.new/@emdash-cms/blocks@1144

@emdash-cms/cloudflare

npm i https://pkg.pr.new/@emdash-cms/cloudflare@1144

emdash

npm i https://pkg.pr.new/emdash@1144

create-emdash

npm i https://pkg.pr.new/create-emdash@1144

@emdash-cms/gutenberg-to-portable-text

npm i https://pkg.pr.new/@emdash-cms/gutenberg-to-portable-text@1144

@emdash-cms/x402

npm i https://pkg.pr.new/@emdash-cms/x402@1144

@emdash-cms/plugin-ai-moderation

npm i https://pkg.pr.new/@emdash-cms/plugin-ai-moderation@1144

@emdash-cms/plugin-atproto

npm i https://pkg.pr.new/@emdash-cms/plugin-atproto@1144

@emdash-cms/plugin-audit-log

npm i https://pkg.pr.new/@emdash-cms/plugin-audit-log@1144

@emdash-cms/plugin-color

npm i https://pkg.pr.new/@emdash-cms/plugin-color@1144

@emdash-cms/plugin-embeds

npm i https://pkg.pr.new/@emdash-cms/plugin-embeds@1144

@emdash-cms/plugin-forms

npm i https://pkg.pr.new/@emdash-cms/plugin-forms@1144

@emdash-cms/plugin-webhook-notifier

npm i https://pkg.pr.new/@emdash-cms/plugin-webhook-notifier@1144

commit: 9f4e73c

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR standardizes the monorepo’s kysely versioning by adding it to the pnpm-workspace.yaml catalog and switching selected packages to use catalog: so pnpm won’t install multiple Kysely versions side-by-side.

Changes:

  • Added kysely: ^0.29.0 to the workspace catalog.
  • Switched emdash, @emdash-cms/auth-atproto, and @emdash-cms/sandbox-workerd to use kysely: "catalog:" where appropriate.
  • Updated @emdash-cms/sandbox-workerd’s kysely peer dependency minimum to >=0.29.0 and regenerated pnpm-lock.yaml.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
pnpm-workspace.yaml Adds kysely to the shared catalog for consistent versioning.
pnpm-lock.yaml Reflects catalog entry + updated specifiers/versions for affected importers.
packages/workerd/package.json Aligns kysely peer range and uses catalog for dev dependency.
packages/core/package.json Switches emdash’s kysely dependency to catalog:.
packages/auth-atproto/package.json Switches kysely dependency to catalog:.
.changeset/workerd-kysely-catalog.md Records the dependency alignment change for release notes.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"@atcute/oauth-node-client": "^1.1.0",
"@emdash-cms/auth": "workspace:*",
"kysely": "^0.29.0"
"kysely": "catalog:"
"@emdash-cms/sandbox-workerd": patch
---

Aligns the `kysely` peer dependency with the rest of the monorepo (`>=0.29.0`) and switches the dev/peer references to the workspace catalog so all packages bump in lockstep going forward.
@ascorbic ascorbic enabled auto-merge (squash) May 22, 2026 13:13
@ascorbic ascorbic merged commit c50c3b2 into main May 22, 2026
40 of 41 checks passed
@ascorbic ascorbic deleted the fix/workerd-kysely-catalog branch May 22, 2026 13:15
@emdashbot emdashbot Bot mentioned this pull request May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants