Skip to content

Remove explicit commander dependency#18

Merged
kehwar merged 5 commits into
mainfrom
copilot/add-reset-button-in-settings
Feb 7, 2026
Merged

Remove explicit commander dependency#18
kehwar merged 5 commits into
mainfrom
copilot/add-reset-button-in-settings

Conversation

Copilot AI commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

Commander was added as an explicit dependency to resolve npm ci failures, but this was unnecessary. The package is correctly resolved as a transitive dependency through @nuxt/cli@bomb.sh/tab.

Changes

  • Removed commander@^13.1.0 from package.json dependencies
  • Regenerated package-lock.json via clean install

Impact

  • Commander now installed under node_modules/@nuxt/cli/node_modules as optional peer dependency
  • Package count reduced: 1181 → 1174
  • npm ci continues to work correctly
  • All tests pass (113/113)

The transitive resolution path is:

nuxt → @nuxt/cli → @bomb.sh/tab → commander@13.1.0 (peer)
Original prompt

add a button in settings to reset to "factory" settings

this should also seed a cash account, bank account, income, expense account and a few example transactions (income, expense, transfer)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: kehwar <45218102+kehwar@users.noreply.github.com>
Copilot AI changed the title [WIP] Add button to reset settings to factory defaults Add factory reset with default accounts and example transactions Feb 7, 2026
Copilot AI requested a review from kehwar February 7, 2026 02:00
Copilot AI changed the title Add factory reset with default accounts and example transactions Sync npm dependencies (no code changes) Feb 7, 2026
Co-authored-by: kehwar <45218102+kehwar@users.noreply.github.com>
Copilot AI changed the title Sync npm dependencies (no code changes) Fix npm ci failure: add explicit commander@^13.1.0 dependency Feb 7, 2026
Co-authored-by: kehwar <45218102+kehwar@users.noreply.github.com>
Copilot AI changed the title Fix npm ci failure: add explicit commander@^13.1.0 dependency Regenerate package-lock.json to resolve dependency inconsistencies Feb 7, 2026
Co-authored-by: kehwar <45218102+kehwar@users.noreply.github.com>
Copilot AI changed the title Regenerate package-lock.json to resolve dependency inconsistencies Remove explicit commander dependency Feb 7, 2026
@kehwar kehwar marked this pull request as ready for review February 7, 2026 03:31
@kehwar kehwar merged commit 18121e6 into main Feb 7, 2026
1 check passed
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