feat: update command add long option #9610
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR introduces a new --long option for the update command to enable detailed output (including URLs) and to fetch full package metadata when listing outdated dependencies.
- Adds
longflag to CLI options and command handlers - Propagates
longthroughoutdatedDepsOfProjectsandgetUpdateChoices - Updates tests to remove URL column in short mode and documents the change in a changeset
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| reviewing/outdated/src/outdatedDepsOfProjects.ts | Map long to the fullMetadata flag for manifest retrieval |
| pkg-manager/plugin-commands-installation/src/update/index.ts | Register --long option and pass opts.long to handler |
| pkg-manager/plugin-commands-installation/src/update/getUpdateChoices.ts | Accept long flag and conditionally include the URL column |
| pkg-manager/plugin-commands-installation/test/update/interactive.ts | Remove URL column header when long is not set |
| pkg-manager/plugin-commands-installation/test/update/getUpdateChoices.test.ts | Remove URL in default test output |
| .changeset/cold-cows-roll.md | Document the addition of the long option |
Comments suppressed due to low confidence (2)
pkg-manager/plugin-commands-installation/test/update/getUpdateChoices.test.ts:79
- Add a test case where
long = trueto verify that the URL column is correctly included in the output when the--longflag is set.
{
pkg-manager/plugin-commands-installation/test/update/interactive.ts:61
- Introduce a scenario invoking the interactive update with
--longto assert that the URL column appears in the rendered prompt.
const storeDir = path.resolve('pnpm-store')
| @@ -37,7 +38,7 @@ export async function outdatedDepsOfProjects ( | |||
| const wantedLockfile = await readWantedLockfile(lockfileDir, { ignoreIncompatible: false }) ?? currentLockfile | |||
| const getLatestManifest = createManifestGetter({ | |||
| ...opts, | |||
| fullMetadata: opts.fullMetadata === true, | |||
| fullMetadata: opts.long === true, | |||
There was a problem hiding this comment.
Currently the code ignores any user-supplied opts.fullMetadata by only using opts.long. You may want to combine both flags, e.g. fullMetadata: opts.fullMetadata === true || opts.long === true, so existing consumers of fullMetadata continue working.
| fullMetadata: opts.long === true, | |
| fullMetadata: opts.fullMetadata === true || opts.long === true, |
Copilot uses AI. Check for mistakes.
close #9576