Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 17, 2025

The has-installed-dependencies action was printing the full output of package manager list commands (npm list, pnpm list, yarn list) to the action logs, cluttering the output with unnecessary dependency tree information.

Changes

  • Added silent: true option to exec.getExecOutput() in the parseDependencies function
const { stdout } = await exec.getExecOutput(command, undefined, {
  cwd: workingDirectory,
+ silent: true,
});

The stdout is still captured and parsed as JSON—only the console echo is suppressed.

Original prompt

On action.yml, has-dependencies step is printed the output of the parseDependencies command, we dont want it (except if there is an error).

We want silent exec.getExecOutput.

Example: https://github.com/hoverkraft-tech/ci-github-nodejs/actions/runs/19423570202/job/55565941072#step:5:276


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Update has-dependencies to use silent exec.getExecOutput Silence parseDependencies command output in has-installed-dependencies action Nov 17, 2025
Copilot AI requested a review from neilime November 17, 2025 17:11
@neilime neilime marked this pull request as ready for review November 17, 2025 17:15
@neilime neilime changed the title Silence parseDependencies command output in has-installed-dependencies action fix(has-installed-dependencies): Silence parseDependencies command output in has-installed-dependencies action Nov 17, 2025
@neilime neilime changed the title fix(has-installed-dependencies): Silence parseDependencies command output in has-installed-dependencies action fix(has-installed-dependencies): silence command output to reduce noise Nov 17, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 17, 2025

Code Coverage Report

Summary

Summary
Generated on: 11/17/2025 - 17:18:51
Parser: LCov
Assemblies: 1
Classes: 1
Files: 1
Line coverage: 100% (2 of 2)
Covered lines: 2
Uncovered lines: 0
Coverable lines: 2
Total lines: 5
Method coverage: Feature is only available for sponsors
Tag: 448_19438029010

Coverage

Default - 100%
Name Line
Default 100%
src/index.js 100%

@neilime neilime merged commit 2d60ad9 into main Nov 17, 2025
59 of 68 checks passed
@neilime neilime deleted the copilot/silent-exec-output branch November 17, 2025 17:28
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