Skip to content

chore!: modernize the repo - #17

Merged
pascal-botpress merged 1 commit into
masterfrom
pb/modernize
Aug 27, 2026
Merged

chore!: modernize the repo#17
pascal-botpress merged 1 commit into
masterfrom
pb/modernize

Conversation

@pascal-botpress

@pascal-botpress pascal-botpress commented Aug 26, 2026

Copy link
Copy Markdown
Member
  • Updates most deps
  • switches to vitest + tsdown + pnpm + mise
    • unfortunately, esbuild is still used under-the-hood by NodejsFunction
  • outputs both cjs and esm
  • publishes the package through OIDC (already wired on npmjs)
  • bumps the version by a major, but this should be drop-in for the infra repo (our only consumer); I checked
  • Removes a bunch of leftovers
  • Fixes issues encountered whilst doing the upgrade

Contributes to KKN-927

Copilot AI lite review requested due to automatic review settings August 26, 2026 18:49
@greptile-apps

greptile-apps Bot commented Aug 26, 2026

Copy link
Copy Markdown

Greptile Summary

This PR modernizes the repository’s package management, build, test, runtime, and publishing setup.

  • Migrates the workspace from Yarn/Jest/tsc builds to pnpm, Vitest, and tsdown.
  • Produces both CommonJS and ESM package exports and updates the provider Lambda runtime and bundle entry.
  • Adds OIDC-based package publishing and expands CI to build, check, test, and synthesize the workspace.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
.github/workflows/test.yml Builds the workspace before type-checking, so the linked package’s generated exports are available to the test stack.
.github/workflows/publish-package.yml Adds the trusted organization publishing workflow with OIDC permissions and pnpm-based build steps.
cdk-postgresql/package.json Defines the dual-module package exports and modernized build, checking, testing, and publishing scripts.
cdk-postgresql/tsdown.config.ts Configures ESM and CommonJS library output plus the CommonJS Lambda handler bundle.
cdk-postgresql/lib/provider.ts Updates the Node.js Lambda runtime and points NodejsFunction at the generated handler bundle.
pnpm-workspace.yaml Defines the pnpm workspace and permitted dependency build scripts.

Reviews (2): Last reviewed commit: "chore!: modernize the repo" | Re-trigger Greptile

Comment thread .github/workflows/test.yml Outdated
Comment thread .github/workflows/publish-package.yml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Modernizes the monorepo tooling and release pipeline for @botpress/cdk-postgresql, including the move to pnpm workspaces, Vitest for testing, and tsdown for producing both ESM and CJS artifacts, plus corresponding updates to the example/test CDK stack and GitHub Actions workflows.

Changes:

  • Migrate repo tooling to pnpm workspaces + mise, and update CI to use those.
  • Replace Jest with Vitest (unit + integration) and add tsdown-based builds with dual ESM/CJS package exports.
  • Update the test-stack to align with the new build/typecheck/synth workflow and new package entrypoints.

Reviewed changes

Copilot reviewed 29 out of 31 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
test-stack/tsconfig.json Updates TS target/module settings and switches to no-emit typechecking.
test-stack/README.md Updates local usage instructions/commands for pnpm-based workflow.
test-stack/package.json Updates scripts/deps to newer toolchain and workspace dependency.
test-stack/lib/app-stack.ts Updates imports to use package root exports.
test-stack/jest.config.js Removes Jest config from test stack.
test-stack/cdk.json Updates watch exclusions for pnpm workspace/lockfiles.
test-stack/.nvmrc Removes Node version pin (replaced by mise).
test-stack/.gitignore Removes Jest-config exception from ignore rules.
pnpm-workspace.yaml Defines pnpm workspace packages and allowed builds list.
pnpm-lock.yaml Adds pnpm lockfile for the workspace.
package.json Replaces npm/yarn workspaces config with repo-level pnpm scripts.
cdk-postgresql/vitest.config.ts Adds Vitest multi-project config (unit + integration).
cdk-postgresql/tsdown.config.ts Adds tsdown build config for library + handler output.
cdk-postgresql/tsconfig.json Updates TS compiler settings for modern TS + typecheck-only.
cdk-postgresql/test/util.test.ts Migrates test file to Vitest imports.
cdk-postgresql/test/lambda.integration.test.ts Updates integration tests for Vitest + newer testcontainers patterns.
cdk-postgresql/test/constructs.test.ts Migrates construct tests from Jest to Vitest and refactors formatting.
cdk-postgresql/README.md Updates installation guidance for pnpm.
cdk-postgresql/package.json Major bump + adds exports map for ESM/CJS, switches build/test tooling.
cdk-postgresql/LICENSE.txt Updates copyright year range.
cdk-postgresql/lib/provider.ts Switches provider Lambda entry to built dist artifact and sets runtime.
cdk-postgresql/lib/lambda.types.ts Adds ServiceToken to OldResourceProperties for update events.
cdk-postgresql/jest.config.js Removes Jest config from library package.
cdk-postgresql/.nvmrc Removes Node version pin (replaced by mise).
cdk-postgresql/.npmignore Removes npmignore (now using files: ["dist"]).
cdk-postgresql/.gitignore Adds dist to ignores and removes Jest-config exception.
.tool-versions Adds mise tool versions for pnpm + nodejs.
.nvmrc Removes root Node version pin (replaced by mise).
.github/workflows/test.yml Updates CI to mise + pnpm, runs typecheck/build/tests + synth.
.github/workflows/publish-package.yml Adds OIDC-enabled publish workflow using pnpm.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread cdk-postgresql/test/constructs.test.ts Outdated
Comment thread test-stack/package.json
Comment thread cdk-postgresql/README.md
@pascal-botpress
pascal-botpress merged commit bd096a6 into master Aug 27, 2026
2 checks passed
@linear-code

linear-code Bot commented Aug 27, 2026

Copy link
Copy Markdown

KKN-927

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.

3 participants