Skip to content

Commit 28a2cfd

Browse files
committed
fix: differentiate empty --features error from missing --features
Agents couldn't tell whether they forgot --features or passed a garbage value. Now: missing flag says "requires --features", while a present but empty-after-parsing value says "value is empty".
1 parent 7d81baf commit 28a2cfd

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

source/__tests__/nonInteractive.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ describe('nonInteractive — validation', () => {
9494
runNonInteractive({ name: 'my_app', mode: 'custom', features: ',' }),
9595
).rejects.toThrow()
9696
const output = getLastJsonOutput()
97-
expect(output.error).toMatch(/--mode custom requires --features/)
97+
expect(output.error).toMatch(/--features value is empty/)
9898
})
9999

100100
it('rejects unknown feature names', async () => {

source/nonInteractive.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ function validate(flags: {
7777
const features = parseFeatures(flags.features)
7878

7979
if (features.length === 0) {
80-
fail('--mode custom requires --features. Use --info to see available features.')
80+
fail('--features value is empty. Use --info to see available features.')
8181
}
8282

8383
const invalidFeatures = features.filter((f) => !featureNames.includes(f))

0 commit comments

Comments
 (0)