Skip to content

fix(examples): bump outdated npm dependencies - #3451

Merged
magurotuna merged 4 commits into
mainfrom
fix/example/deps
Aug 18, 2026
Merged

fix(examples): bump outdated npm dependencies#3451
magurotuna merged 4 commits into
mainfrom
fix/example/deps

Conversation

@magurotuna

Copy link
Copy Markdown
Member
  • npm.ts: express 4.18.2 -> 4.22.2 (keeps @types/express@4)
  • mongo.ts: mongodb 6.1.0 -> ^7
  • creating_and_verifying_jwt.ts: jose 5.9.6 -> ^6
  • redis.ts: redis ^4.5 -> ^6
  • openai_chat_completion.ts: migrate off frozen deno.land/x module to npm:openai@^7

deno.lock updated with the new resolution; no unrelated changes. All files pass deno check and deno fmt.

@magurotuna
magurotuna requested a review from avocet-bot August 17, 2026 10:06
@avocet-bot

avocet-bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Avocet review status

State: Completed
Target: c05536a89f86
Requested by: magurotuna
Model: Claude Opus
Attempt: 1 of 3
Updated: 2026-08-17T10:26:00.408Z
Formal review: #3451 (review)

The formal GitHub review is the authoritative result. This comment only reports operational status.

@avocet-bot avocet-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review model: Claude Opus

Model verdict: APPROVE (informational only)

Formal review policy did not authorize an authoritative event (required_checks_missing).

No blocking findings. This bumps five example dependencies to newer majors and refreshes @resource links.

Verified:

  • All target versions exist on npm and match the deno.lock resolutions exactly: jose 6.2.9, mongodb 7.5.0, redis 6.2.1, openai 7.4.0, and express 4.22.2 (the current latest-4 dist-tag). The added lockfile entries (transitive deps for each package) are consistent.
  • The import OpenAI from "npm:openai@^7" default-import change is correct for the v7 SDK; the remaining bumps touch only stable API surfaces (jose SignJWT/jwtVerify, mongodb basic CRUD, redis createClient/set/get/sendCommand/quit, express minor).
  • Updated doc links resolve: runtime/fundamentals/node.md exists in-repo and the URLs return 200; the retained manual/node/npm_specifiers link still redirects fine.
  • examples/_pages/examples_test.ts already runs deno check on every example in CI, catching any API-surface regression from these bumps before merge.

Non-blocking nit: mongo.ts prose still reads "Using the Deno MongoDB client," though it now imports the official npm mongodb driver — consider rewording to match the switch away from deno.land/x/mongo.

@magurotuna
magurotuna requested a review from avocet-bot August 18, 2026 03:03
@avocet-bot

avocet-bot commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Avocet review status

State: Completed
Target: 7fca2e39ef57
Requested by: magurotuna
Model: Claude Opus
Attempt: 1 of 3
Updated: 2026-08-18T03:11:23.172Z
Formal review: #3451 (review)

The formal GitHub review is the authoritative result. This comment only reports operational status.

@avocet-bot avocet-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review model: Claude Opus

Model verdict: APPROVE

No blocking findings. The only change to this PR's files since the previously reviewed head (c05536a) is the mongo.ts prose update from "Using the Deno MongoDB client" to "Using the official MongoDB driver, you can connect to a MongoDB database," which resolves the sole non-blocking nit from my prior review. The sveltekit.md edits appearing in the head-to-head diff originate from the merge of main, not this PR; the base/head manifest and git diff --stat confirm the net change remains the same six files (deno.lock plus five example scripts).

All prior verifications still hold: the bumped versions exist on npm and match the deno.lock resolutions exactly (jose 6.2.9, mongodb 7.5.0, redis 6.2.1, openai 7.4.0, express 4.22.2), the openai v7 default-import is correct, the retained API surfaces are stable, updated @resource links resolve, and examples/_pages/examples_test.ts deno checks every example in CI.

@magurotuna
magurotuna merged commit e6f6dff into main Aug 18, 2026
3 checks passed
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.

2 participants