chore(deps): bump kysely from 0.27.4 to 0.28.11#1899
chore(deps): bump kysely from 0.27.4 to 0.28.11#1899dependabot[bot] wants to merge 4 commits intomainfrom
Conversation
Bumps [kysely](https://github.com/kysely-org/kysely) from 0.27.4 to 0.28.11. - [Release notes](https://github.com/kysely-org/kysely/releases) - [Commits](kysely-org/kysely@0.27.4...v0.28.11) --- updated-dependencies: - dependency-name: kysely dependency-version: 0.28.11 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Stale comment
Security Review
No high-confidence vulnerabilities found in this PR.
What I reviewed
- Dependency/supply-chain change:
kyselyupgraded from0.27.xto0.28.11inapps/studio/package.jsonandpackage-lock.json.- Code change:
PostgresQueryCompiler.compileQuery(args.node, queryId)update inapps/studio/src/server/modules/database/database.ts.Findings
- No confirmed injection, authz/authn bypass, secret leakage, SSRF, XSS, CSRF, or deserialization vulnerabilities introduced by this diff.
- The tracing path still logs SQL text and parameter count (not parameter values); this behavior appears pre-existing and unchanged in risk profile by this PR.
Result
- Security assessment: pass (no high-confidence vulnerabilities identified).
Sent by Cursor Automation: Find vulnerabilities (PR)
There was a problem hiding this comment.
Stale comment
Risk assessment: Medium.
Evidence from diff:
- Upgrades shared query-builder dependency
kyselyfrom0.27.xto0.28.11inapps/studio/package.json(and lockfile).- Updates runtime DB tracing path in
apps/studio/src/server/modules/database/database.ts(compileQuery(args.node, queryId)), which is part of shared database execution instrumentation.- Lockfile also reflects dependency metadata/engine changes for
kysely.Decision:
- Review is required (medium risk; shared backend/database surface).
- I did not approve this PR.
- Assigned reviewers:
@gautammmanek,@seaerchin.CODEOWNERSapplies (* @opengovsg/isomer-engineers), so this should land with owner review.Sent by Cursor Automation: Assign PR reviewers (PR)
There was a problem hiding this comment.
Stale comment
Risk assessment: Medium.
Evidence from diff:
apps/studio/package.json: bumpskyselyfrom^0.27.3to^0.28.11.apps/studio/src/server/modules/database/database.ts: updates shared DB tracing path fromcompileQuery(args.node)tocompileQuery(args.node, queryId).package-lock.json: lockfile update includingkyselypackage metadata (notably engine constraint nownode >=20).Why not Low/Very Low:
- This is not a docs/test-only or formatting-only change.
- It modifies a shared database plugin codepath and upgrades a core query-builder dependency used in production DB access.
- Regression blast radius is limited to one app, but the touched path is central to query execution instrumentation.
Decision:
- Code review required.
- Requested 2 reviewers with history on this codepath:
@gautammmanek,@adriangohjw.- No self-approval (medium-risk change and CODEOWNERS-governed path).
Sent by Cursor Automation: Assign PR reviewers (PR)
There was a problem hiding this comment.
Stale comment
Security review result: no high-confidence vulnerabilities found in this PR.
Evidence checked in diff:
- Dependency change is limited to
kyselyversion bump (0.27.x->0.28.11) with lockfileresolvedURL andintegrityhash present.- Code change in
database.tsupdatescompileQuery(args.node, queryId)to match new API usage; it does not introduce dynamic SQL concatenation, auth boundary changes, or new external input handling.- Existing tracing behavior still logs SQL text and parameter count only (not parameter values), so this PR does not introduce new secret-leak logging paths.
Uncertain concerns requiring additional validation: none from this diff.
Sent by Cursor Automation: Find vulnerabilities (PR)
…urce.service.ts for consistency
There was a problem hiding this comment.
Security review for dependency bump kysely 0.27.4 -> 0.28.11:
No high-confidence vulnerabilities found in this PR diff.
Confirmed findings:
- None.
Uncertain / validation-needed concern (low confidence):
package-lock.jsonalso rewrites some workspace metadata ranges (for exampleeslint-plugin-react-hooks: beta -> *andtypescript: latest -> *). This does not appear to change the currently locked resolved artifacts in this PR, but it can increase future supply-chain drift risk if lockfiles are regenerated inconsistently.
Recommended hardening:
- Keep installs in CI/release paths on
npm ci(notnpm install) to enforce lockfile integrity. - Regenerate lockfiles using a pinned npm version to reduce unintended lockfile rewrites.
Sent by Cursor Automation: Find vulnerabilities (PR)
There was a problem hiding this comment.
Risk assessment: Medium.
Evidence from current diff:
apps/studio/package.jsonandpackage-lock.jsonupgradekyselyfrom0.27.xto0.28.11(shared query builder dependency).apps/studio/src/server/modules/database/database.tschanges DB tracing compilation call tocompileQuery(args.node, queryId)in shared database plugin code.apps/studio/src/server/modules/resource/resource.service.tsupdatesorderBycall signature in runtime service code.apps/studio/src/server/modules/collection/__tests__/collection.router.test.tsupdates test syntax accordingly.
Decision:
- Code review required (medium risk; shared backend/database surface changed).
- No self-approval (medium risk and CODEOWNERS-governed codepaths).
- Reviewer assignment unchanged: PR already has 3 requested reviewers, so no additional reviewers were added (limit is max 2 to add).
Sent by Cursor Automation: Assign PR reviewers (PR)
|
A newer version of kysely exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |


Bumps kysely from 0.27.4 to 0.28.11.
Release notes
Sourced from kysely's releases.
... (truncated)
Commits
102d4770.28.11d365e74docs: fix conditional selects examples.c37c3c6Prevent re-use of finished trx outside callback (#1694)0a265cbAdd MariaDB Dialect (#1646)6862970bump hono.af85e9dbump wrangler.68e1b4dbump react.c98cc47bump pnpm.117e50cbump jsr.468491fbump playwright.Maintainer changes
This version was pushed to npm by [GitHub Actions](https://www.npmjs.com/~GitHub Actions), a new releaser for kysely since your current version.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.