Skip to content

chore: remove unused moment.js dependency#92

Open
dukunline-cyber wants to merge 1 commit into
BAWES-Universe:developfrom
dukunline-cyber:chore/migrate-moment-to-date-fns
Open

chore: remove unused moment.js dependency#92
dukunline-cyber wants to merge 1 commit into
BAWES-Universe:developfrom
dukunline-cyber:chore/migrate-moment-to-date-fns

Conversation

@dukunline-cyber

Copy link
Copy Markdown

Summary

Removes moment from package.json dependencies as it is not actually used anywhere in the codebase.

Investigation

A full repo scan (grep -rniE moment --include=*.ts --include=*.html --include=*.scss) found only two textual matches:

  1. package.json:62 — the dependency declaration itself.
  2. src/app/pages/logged-in/transfer/transfer-form/transfer-form.page.ts:62 — inside a code comment describing a type literal (// 'string' | 'js-date' | 'moment' | 'time' | 'object'), not an actual import or call.

There are zero import or require statements referencing moment.

Changes

  • Removed moment ^2.30.1 from dependencies in package.json.
  • Regenerated package-lock.json accordingly (npm install --package-lock-only --legacy-peer-deps).

The remaining "moment" reference in package-lock.json is a peerDependencies declaration from another package and is not an installed copy.

Benefits

  • Smaller install footprint and node_modules size.
  • Removes a known-large legacy dependency that is in maintenance-only mode (per the official moment.js project status).
  • No runtime behavior change since the package was never imported.

Testing

  • Lock file regenerates cleanly with --legacy-peer-deps (the same flag the project already requires due to pre-existing capacitor peer-dep conflicts).
  • No source files needed editing.

moment is declared in package.json but not imported anywhere in src/. The only textual occurrence is in a comment (transfer-form.page.ts:62). Removing reduces bundle size and dependency footprint.
@coderabbitai

coderabbitai Bot commented May 16, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: bd1bee5b-756b-4fd2-af4d-4454cb44209e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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