Skip to content

Reduce dependency footprint#28

Merged
adsharma merged 1 commit into
masterfrom
reduce-dependency-footprint
May 31, 2026
Merged

Reduce dependency footprint#28
adsharma merged 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
@adsharma adsharma merged commit b767b4b into master May 31, 2026
1 check passed
@adsharma adsharma deleted the reduce-dependency-footprint branch May 31, 2026 01:39
@aheev
Copy link
Copy Markdown
Contributor

aheev commented May 31, 2026

@adsharma can we get rid of ladybug submodule dependency too somehow? maybe maitaining a copy of antlr4 files here. It is confusing to have both submodule dependency and npm dependency

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