Skip to content

chore(deps): align @types/bun with the Bun 1.4.2 toolchain - #124

Merged
gjtorikian merged 1 commit into
mainfrom
fix/pr-123-review-followups
Sep 22, 2026
Merged

gjtorikian merged 1 commit into
mainfrom
fix/pr-123-review-followups

Conversation

@gjtorikian

Copy link
Copy Markdown
Collaborator

Summary

  • Bumps @types/bun from 1.3.14 to 1.4.2 so the type definitions match the Bun 1.4.2 runtime that packageManager, the Dockerfile, and the workflows (via bun-version-file) have pinned since chore(deps): update minor and patch updates #123.
  • Closes the remaining gap from the Greptile review on chore(deps): update minor and patch updates #123. r4075262536 flagged the Bun upgrade as incomplete; the workflow pins it mentioned were already fixed before merge, and the types package was the one piece still on 1.3.14.
  • The sibling thread r4075262525 about stale spec codegen needed no change: regenerating events, shapes, and SUPPORTED.md against @workos/openapi-spec@0.98.0 on main is a no-op.
  • The lockfile diff is limited to @types/bun and bun-types moving to 1.4.2. The nested bun-types/@types/node 22.19.21 override drops out because the top-level @types/node 22.20.4 now satisfies it.

Verified on Bun 1.4.2: typecheck, lint, format check, build, and the full suite (1344 tests) pass.

PR #123 moved packageManager, the Dockerfile, and the workflows
(via bun-version-file) to Bun 1.4.2 but left @types/bun on
1.3.14, so typechecking still described the old runtime. Review
thread r4075262536 flagged the gap; this closes it so the types
CI checks against match the Bun that builds and tests the code.
@greptile-apps

greptile-apps Bot commented Sep 22, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge because the dependency and lockfile changes consistently align the type definitions with the already-pinned Bun toolchain.

Summary

Aligns the Bun type definitions with the repository’s existing Bun 1.4.2 toolchain.

  • Updates @types/bun and its bun-types dependency from 1.3.14 to 1.4.2.
  • Deduplicates the transitive Node declarations onto the existing @types/node@22.20.4 resolution.
  • Keeps the dependency manifest and Bun lockfile synchronized.

Reviews (1) · Last reviewed commit: "chore(deps): align @types/bun with the B..."

@gjtorikian
gjtorikian merged commit 5b884dd into main Sep 22, 2026
10 checks passed
@gjtorikian
gjtorikian deleted the fix/pr-123-review-followups branch September 22, 2026 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant