Skip to content

Control output handling for list commands#589

Merged
soumeh01 merged 3 commits into
mainfrom
list-command
Apr 8, 2026
Merged

Control output handling for list commands#589
soumeh01 merged 3 commits into
mainfrom
list-command

Conversation

@soumeh01

@soumeh01 soumeh01 commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

Fixes

Changes

  • Broaden .gitignore patterns for binaries, config folders, and files
  • Add IsListCmd field to utils.Runner to control output handling for list commands
  • Added env variable name FORCE_JAVASCRIPT_ACTIONS_TO_NODE24 matches GitHub's official documentation for the Node 24 opt-in mechanism.

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 20 package(s) with unknown licenses.
See the Details below.

Snapshot Warnings

⚠️: The number of snapshots compared for the base SHA (0) and the head SHA (1) do not match. You may see unexpected additions in the diff.
Re-running this action after a short time may resolve the issue. See the documentation for more information and troubleshooting advice.

License Issues

go.mod

PackageVersionLicenseIssue Type
github.com/creack/ptyv1.1.21NullUnknown License
github.com/davecgh/go-spewv1.1.1NullUnknown License
github.com/hashicorp/go-versionv1.9.0NullUnknown License
github.com/otiai10/copyv1.14.1NullUnknown License
github.com/pmezard/go-difflibv1.0.0NullUnknown License
github.com/sirupsen/logrusv1.9.4NullUnknown License
github.com/spf13/cobrav1.10.2NullUnknown License
github.com/spf13/pflagv1.0.10NullUnknown License
github.com/stretchr/testifyv1.11.1NullUnknown License
gopkg.in/yaml.v3v3.0.1NullUnknown License
github.com/creack/ptyv1.1.21NullUnknown License
github.com/davecgh/go-spewv1.1.1NullUnknown License
github.com/hashicorp/go-versionv1.9.0NullUnknown License
github.com/otiai10/copyv1.14.1NullUnknown License
github.com/pmezard/go-difflibv1.0.0NullUnknown License
github.com/sirupsen/logrusv1.9.4NullUnknown License
github.com/spf13/cobrav1.10.2NullUnknown License
github.com/spf13/pflagv1.0.10NullUnknown License
github.com/stretchr/testifyv1.11.1NullUnknown License
gopkg.in/yaml.v3v3.0.1NullUnknown License

OpenSSF Scorecard

Scorecard details
PackageVersionScoreDetails
gomod/github.com/aymanbagabas/go-pty v0.2.2 🟢 3
Details
CheckScoreReason
Maintained⚠️ 23 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 2
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Code-Review⚠️ 0Found 0/3 approved changesets -- score normalized to 0
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
Security-Policy⚠️ 0security policy file not detected
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
gomod/github.com/creack/pty v1.1.21 UnknownUnknown
gomod/github.com/davecgh/go-spew v1.1.1 UnknownUnknown
gomod/github.com/hashicorp/go-version v1.9.0 UnknownUnknown
gomod/github.com/otiai10/copy v1.14.1 UnknownUnknown
gomod/github.com/pmezard/go-difflib v1.0.0 UnknownUnknown
gomod/github.com/sirupsen/logrus v1.9.4 UnknownUnknown
gomod/github.com/spf13/cobra v1.10.2 UnknownUnknown
gomod/github.com/spf13/pflag v1.0.10 UnknownUnknown
gomod/github.com/stretchr/testify v1.11.1 UnknownUnknown
gomod/github.com/u-root/u-root v0.11.0 🟢 7.4
Details
CheckScoreReason
Security-Policy🟢 4security policy file detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 2badge detected: InProgress
License🟢 10license file detected
Fuzzing🟢 10project is fuzzed
Binary-Artifacts🟢 9binaries present in source code
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Signed-Releases⚠️ -1no releases found
Packaging🟢 10packaging workflow detected
SAST🟢 10SAST tool is run on all commits
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
gomod/golang.org/x/crypto v0.45.0 UnknownUnknown
gomod/golang.org/x/sync v0.8.0 UnknownUnknown
gomod/golang.org/x/sys v0.42.0 UnknownUnknown
gomod/golang.org/x/term v0.41.0 UnknownUnknown
gomod/gopkg.in/yaml.v3 v3.0.1 UnknownUnknown
gomod/github.com/aymanbagabas/go-pty v0.2.2 🟢 3
Details
CheckScoreReason
Maintained⚠️ 23 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 2
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Binary-Artifacts🟢 10no binaries found in the repo
Code-Review⚠️ 0Found 0/3 approved changesets -- score normalized to 0
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
Packaging⚠️ -1packaging workflow not detected
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
Security-Policy⚠️ 0security policy file not detected
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
gomod/github.com/creack/pty v1.1.21 UnknownUnknown
gomod/github.com/davecgh/go-spew v1.1.1 UnknownUnknown
gomod/github.com/hashicorp/go-version v1.9.0 UnknownUnknown
gomod/github.com/otiai10/copy v1.14.1 UnknownUnknown
gomod/github.com/pmezard/go-difflib v1.0.0 UnknownUnknown
gomod/github.com/sirupsen/logrus v1.9.4 UnknownUnknown
gomod/github.com/spf13/cobra v1.10.2 UnknownUnknown
gomod/github.com/spf13/pflag v1.0.10 UnknownUnknown
gomod/github.com/stretchr/testify v1.11.1 UnknownUnknown
gomod/github.com/u-root/u-root v0.11.0 🟢 7.4
Details
CheckScoreReason
Security-Policy🟢 4security policy file detected
Code-Review🟢 10all changesets reviewed
Maintained🟢 1030 commit(s) and 3 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
CII-Best-Practices⚠️ 2badge detected: InProgress
License🟢 10license file detected
Fuzzing🟢 10project is fuzzed
Binary-Artifacts🟢 9binaries present in source code
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
Signed-Releases⚠️ -1no releases found
Packaging🟢 10packaging workflow detected
SAST🟢 10SAST tool is run on all commits
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
gomod/golang.org/x/crypto v0.45.0 UnknownUnknown
gomod/golang.org/x/sync v0.8.0 UnknownUnknown
gomod/golang.org/x/sys v0.42.0 UnknownUnknown
gomod/golang.org/x/term v0.41.0 UnknownUnknown
gomod/gopkg.in/yaml.v3 v3.0.1 UnknownUnknown

Scanned Files

  • go.mod

@qltysh

qltysh Bot commented Apr 8, 2026

Copy link
Copy Markdown

Qlty

Coverage Impact

⬇️ Merging this pull request will decrease total coverage on main by 0.19%.

Modified Files with Diff Coverage (5)

RatingFile% DiffUncovered Line #s
Coverage rating: B Coverage rating: B
pkg/utils/exec.go100.0%
Coverage rating: F Coverage rating: F
cmd/cbuild/commands/list/list_toolchains.go0.0%56-58
Coverage rating: D Coverage rating: D
cmd/cbuild/commands/list/list_contexts.go0.0%55-57
Coverage rating: D Coverage rating: F
cmd/cbuild/commands/list/list_environment.go0.0%29-31
Coverage rating: F Coverage rating: F
cmd/cbuild/commands/list/list_target_sets.go0.0%58-60
Total7.7%
🤖 Increase coverage with AI coding...

In the `list-command` branch, add test coverage for this new code:

- `cmd/cbuild/commands/list/list_contexts.go` -- Line 55-57
- `cmd/cbuild/commands/list/list_environment.go` -- Line 29-31
- `cmd/cbuild/commands/list/list_target_sets.go` -- Line 58-60
- `cmd/cbuild/commands/list/list_toolchains.go` -- Line 56-58

🚦 See full report on Qlty Cloud »

🛟 Help
  • Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

  • Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

  • File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

    • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Test Results

    6 files  ±0     60 suites  ±0   12s ⏱️ ±0s
  305 tests ±0    305 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 830 runs  ±0  1 830 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit d4ef9ec. ± Comparison against base commit 7c1ed5b.

♻️ This comment has been updated with latest results.

@soumeh01
soumeh01 requested a review from brondani April 8, 2026 13:40

@brondani brondani left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@soumeh01
soumeh01 marked this pull request as ready for review April 8, 2026 13:46
@soumeh01
soumeh01 merged commit 1068fdc into main Apr 8, 2026
45 of 46 checks passed
@soumeh01
soumeh01 deleted the list-command branch April 8, 2026 13:47
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