Skip to content

Conversation

@anthonykim1
Copy link
Contributor

@anthonykim1 anthonykim1 commented Jan 6, 2026

Task from package.json fail to re-run when ran by command palette.
/cc @meganrogge
Resolves: #270296

@anthonykim1 anthonykim1 added this to the December 2025 milestone Jan 6, 2026
@anthonykim1 anthonykim1 self-assigned this Jan 6, 2026
Copilot AI review requested due to automatic review settings January 6, 2026 22:36
@anthonykim1 anthonykim1 marked this pull request as ready for review January 6, 2026 22:36
rzhao271
rzhao271 previously approved these changes Jan 6, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a regression where tasks defined in package.json fail to re-run when invoked via the command palette. The fix modifies the restart logic to attempt running the original task when an updated task cannot be found, rather than immediately showing an error.

Key Changes:

  • Modified the _restart method to attempt running the original task even when _findUpdatedTask returns undefined
  • Added conditional warning display based on task execution exit code

rzhao271
rzhao271 previously approved these changes Jan 6, 2026
@anthonykim1 anthonykim1 requested a review from rzhao271 January 6, 2026 23:00
rzhao271
rzhao271 previously approved these changes Jan 6, 2026
@anthonykim1 anthonykim1 marked this pull request as draft January 6, 2026 23:08
@anthonykim1 anthonykim1 marked this pull request as ready for review January 6, 2026 23:16
@anthonykim1 anthonykim1 enabled auto-merge (squash) January 6, 2026 23:58
@anthonykim1 anthonykim1 merged commit ca80d6a into main Jan 7, 2026
32 of 33 checks passed
@anthonykim1 anthonykim1 deleted the anthonykim1/fixTaskPackageFix branch January 7, 2026 00:04
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.

The npm task failed to restart

4 participants