chore(deps): clear npm security alerts in examples/demos#958
Open
alexluong wants to merge 2 commits into
Open
Conversation
dashboard-integration: next 15.5.2 -> 15.5.19 (+ eslint-config-next), clearing the Next.js RCE critical and 4 highs. Three moderates remain that only have next-16 fixes (one vendored inside next). nodejs: @google-cloud/pubsub v4 -> v5 and swap the abandoned esrun TS runner for tsx. Clears 3 criticals (protobufjs, form-data, fast-xml-parser) and all highs. Two residual transitive otel moderates have no real upstream fix. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Take both demo apps to the latest of every dependency. nodejs: @aws-sdk/client-sqs, amqplib 0.10->2, dotenv 16->17, typescript 5->6, @types/node 22->25, @hookdeck/outpost-sdk 1.0->1.4 (also fixes a stale typecheck error in publish-api.ts). dashboard-integration: next 15->16 (+ eslint-config-next, react 19.2), typescript 5->6, zod/pg/winston/lucide-react etc. Notable migration work: - next 16 defaults to Turbopack, which scoped module resolution to the demo dir and couldn't follow the symlinked @hookdeck/outpost-sdk (file: link to ../../../sdks/outpost-typescript). Set turbopack.root to the repo root so it traverses the link. - Migrate eslint flat config off the FlatCompat/@eslint/eslintrc shim to eslint-config-next 16's native flat exports (the shim throws a circular-config error under config-next 16). Drop @eslint/eslintrc. - eslint kept at ^9: eslint-config-next 16 breaks under eslint 10. - react-hooks/set-state-in-effect (new error in config-next 16) set to warn to match the demo's existing relaxed lint posture. - Fix pre-existing type error: events.list() returns EventPaginatedResult directly (.models), not wrapped in .result. Both apps: build + lint pass. Residual moderate advisories have no real upstream fix (otel core via pubsub; next/next-auth/postcss with no stable patch yet). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stacked on #940. Bumps every dependency in the two demo apps under
examples/demos/to latest, and clears all actionable npm Dependabot alerts. Merge after #940.nodejs@aws-sdk/client-sqs,amqplib0.10→2,dotenv16→17,typescript5→6,@types/node22→25,@hookdeck/outpost-sdk1.0→1.4,@google-cloud/pubsubv4→v5.esrunTS runner fortsx(drop-in; smoke-tested). esrun pinned a vulnerable esbuild and npm's only "fix" was a bogus 0.0.1 downgrade.publish-api.ts.typecheckclean, scripts run under tsx.dashboard-integrationnext15→16,react19.2,typescript5→6,zod/pg/winston/lucide-react/etc.@hookdeck/outpost-sdk(file:link to../../../sdks/outpost-typescript). Setturbopack.rootto the repo root.FlatCompat/@eslint/eslintrcshim (throws a circular-config error under config-next 16) to config-next 16's native flat exports; dropped@eslint/eslintrc.eslintkept at^9— config-next 16 breaks under eslint 10.react-hooks/set-state-in-effect(new error in config-next 16) set towarn, matching the demo's existing relaxed lint posture.events.list()returnsEventPaginatedResultdirectly (.models), not wrapped in.result.next buildandeslintboth pass.Residual (no real upstream fix)
nodejs: 2 transitive OpenTelemetry moderates via pubsub.dashboard-integration: 3 moderates (next,next-auth,postcss) — advisory ranges include the latest stable; only canary/major-downgrade "fixes" exist.sdks/*,website/,internal/portal/,docs/, andexamples/sdk-*alerts remain out of scope.