-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
enhancementNew feature or requestNew feature or request
Description
Context
From QA review of issue-37-cli-add-bundle-update-awareness branch.
Problem
In packages/cli/src/commands/packages/update.ts, outdated bundles are updated sequentially in a for...of loop. Since each download is independent, Promise.allSettled would be faster for users with multiple outdated bundles.
The current sequential approach is correct and handles partial failure gracefully. This is a performance improvement, not a bug.
Suggested Fix
const results = await Promise.allSettled(
outdated.map(async (entry) => {
const downloadInfo = await resolveBundle(entry.name, client);
const { version } = await downloadAndExtract(entry.name, downloadInfo);
return { name: entry.name, from: entry.current, to: version };
})
);Severity
Suggestion — non-blocking.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request