Skip to content

Reduce dependency footprint#28

Open
adsharma wants to merge 1 commit into
masterfrom
reduce-dependency-footprint
Open

Reduce dependency footprint#28
adsharma wants to merge 1 commit into
masterfrom
reduce-dependency-footprint

Conversation

@adsharma
Copy link
Copy Markdown
Contributor

@adsharma adsharma commented May 30, 2026

Related: #25

Summary

  • remove 14 direct npm dependencies from the app manifest and lockfile
  • replace small dependency usage with local fetch, UUID, date formatting, logging, CORS, upload, drag/drop, color, and session persistence helpers
  • keep existing API routes while changing import uploads to raw file bodies and session persistence to JSON

Direct dependencies remaining:

  • 13 runtime dependencies
  • 16 dev dependencies
  • 29 total direct dependencies

Installed tree size is 1127 packages including transitive dependencies.

Validation

  • npm run build
  • NODE_ENV=production LBUG_IN_MEMORY=true npm run lint
  • node --check src/server/Import.js
  • node --check src/server/utils/SessionDatabase.js
  • node --check src/server/utils/Logger.js

Notes

  • Build passes with existing Sass deprecation and asset-size warnings.
  • npm install still reports audit findings in the remaining dependency tree.

@adsharma adsharma marked this pull request as ready for review May 30, 2026 23:28
@adsharma adsharma force-pushed the reduce-dependency-footprint branch from 8a07315 to 31f9f1e Compare May 31, 2026 00:14
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