Skip to content

fix: legalize phantom dependencies - #5549

Open
slagiewka wants to merge 1 commit into
advplyr:masterfrom
slagiewka:phantom-deps
Open

slagiewka wants to merge 1 commit into
advplyr:masterfrom
slagiewka:phantom-deps

Conversation

@slagiewka

@slagiewka slagiewka commented Sep 10, 2026

Copy link
Copy Markdown

Brief summary

These are in use server code only.

  1. server/model/ApiKey.js imports jsonwebtoken
  2. server/ imports uuid in many places
  3. server/libs vendored code imports all the rest

These currently work by coincidence because something else defines them to be available in node_modules/.

Minimal package-lock.json ensures that no new version is introduced and server code still uses same versions as before.

Which issue is fixed?

No reported issue (server works without it).

In-depth Description

How have you tested this?

Started server, generated clients, walked through the UI, downloaded and played a podcast.

Screenshots

These are in use server code only.

1. server/model/ApiKey.js imports jsonwebtoken
2. server/ imports uuid in many places
3. server/libs vendored code imports all the rest

These currently work by coincidence because something else defines them
to be available in node_modules/.

Minimal package-lock.json ensures that no new version is introduced and
server code still uses same versions as before.
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.

1 participant