fix: kebab-case CLI flags broke in peowly migration#125
Merged
Conversation
…ion tests Co-authored-by: voxpelli <34457+voxpelli@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Investigate and patch regressions in codebase
Fix kebab-case CLI flags broken by meow→peowly migration
Feb 23, 2026
voxpelli
approved these changes
Feb 23, 2026
voxpelli
reviewed
Feb 23, 2026
…own flags Co-authored-by: voxpelli <34457+voxpelli@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR fixes a critical regression introduced during the migration from meow to peowly. Unlike meow, peowly does not automatically convert camelCase flag object keys to kebab-case CLI flag names, causing all multi-word CLI flags to break for users.
Changes:
- Renamed 7 multi-word flag keys from camelCase to kebab-case in
cli.jsflag definitions - Updated flag destructuring to use kebab-case rename syntax (provides TypeScript compile-time safety)
- Added comprehensive test suite with 7 regression tests and 1 negative test to prevent future regressions
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| cli.js | Renamed 7 flag keys to kebab-case ('engine-check', 'peer-check', 'version-check', 'ignore-dev', 'workspace-ignore', 'engine-ignore', 'engine-no-dev') and updated destructuring syntax |
| test/integration.js | Added assertFlagRecognised helper and 8 tests (7 positive + 1 negative) to verify all multi-word flags work correctly |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
engineCheck→--engineCheckonly, not--engine-checkcli.jscli.flagsdestructuring to kebab-case rename syntax (also serves as a TypeScript compile-time check)CLI flag namestest suite intest/integration.jswith 7 regression tests covering every previously broken flagrejects an unknown flag) to proveassertFlagRecognisedwould catch a regression — verifies that--engineCheck(camelCase) is correctly rejected as unknownOriginal prompt
💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.