Skip to content

fix(fix): make --ecosystems case-insensitive#1308

Merged
Martin Torp (mtorp) merged 2 commits into
v1.xfrom
fix/ecosystems-case-insensitive
May 11, 2026
Merged

fix(fix): make --ecosystems case-insensitive#1308
Martin Torp (mtorp) merged 2 commits into
v1.xfrom
fix/ecosystems-case-insensitive

Conversation

@mtorp
Copy link
Copy Markdown
Contributor

@mtorp Martin Torp (mtorp) commented May 8, 2026

Summary

  • Lowercase --ecosystems input before validation so NPM, Npm, and npm are all accepted, matching the behavior already in place for --package-managers (feat(fix): add --package-managers flag #1292).
  • Update --ecosystems help text to mention case-insensitivity.
  • Upgrade @coana-tech/cli from 15.2.2 to 15.2.4.
  • Bump patch version to 1.1.93 and add a CHANGELOG entry.

Why

--package-managers was added in #1292 as case-insensitive (input gets uppercased before validation against an ALL-CAPS list). --ecosystems, however, validates input verbatim against an all-lowercase list, so passing NPM would fail with "Invalid ecosystem". Asymmetric UX between two related filters; this commit makes --ecosystems mirror --package-managers.

Coana Changelog

For details on what's included in the 15.2.4 Coana release, see the Coana Changelogs.

Test plan

  • New integration test: should accept --ecosystems case-insensitively (passes NPM,PyPI)
  • Existing --ecosystems integration tests still pass
  • Help-text snapshot updated
  • eslint + tsgo pass on the changed files
  • CI green

Note

Low Risk
Low risk: small input-normalization change for socket fix flag parsing plus a dependency bump; behavior change is limited to validation of --ecosystems values.

Overview
socket fix --ecosystems now normalizes provided ecosystem values to lowercase before validation, making the flag case-insensitive (aligned with --package-managers), and updates the help text accordingly.

Adds an integration test covering mixed-case --ecosystems input, bumps @coana-tech/cli to 15.2.4, and increments the CLI version to 1.1.93 with a matching CHANGELOG.md entry.

Reviewed by Cursor Bugbot for commit 444d139. Configure here.

Lowercase --ecosystems input before validation so values like NPM, Npm,
and npm are all accepted, mirroring --package-managers behavior. Update
help text accordingly and bump patch version.
@socket-security
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​coana-tech/​cli@​15.2.2 ⏵ 15.2.4100 +5100100 +21100 +3100

View full report

@socket-security-staging
Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​coana-tech/​cli@​15.2.2 ⏵ 15.2.497 +5010080 +198 +1100

View full report

@mtorp Martin Torp (mtorp) marked this pull request as ready for review May 11, 2026 07:53
@mtorp Martin Torp (mtorp) merged commit 8fb5e6e into v1.x May 11, 2026
13 checks passed
@mtorp Martin Torp (mtorp) deleted the fix/ecosystems-case-insensitive branch May 11, 2026 07:54
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