ReviewOS

also looking at this

stacks/buddy

fix(cli): let packages.strategy reach its flags, and sync CLAUDE.md's embedded README

#1441
Merged glennmichael123 wants to merge fix/strategy-flag-default into main
4 files +182 -64
test/cli-surface.test.tsmodified+12-0
Changes to test/cli-surface.test.ts
@@ -93,6 +93,18 @@ describe('CLI surface', () => {
9393 }
9494 }, 30000)
9595
96 it('success case - the strategy flag no longer shadows the config', async () => {
97 // `--strategy` carried `{ default: 'all' }`, so the option was always
98 // "set" and the handler's `options.strategy ?? config.packages?.strategy`
99 // chain never reached the config — packages.strategy was dead on the
100 // scan/update/check/schedule paths unless a flag was typed.
101 const { code, output } = await run(['update', '--help'])
102
103 expect(code).toBe(0)
104 expect(output).toContain('packages.strategy from config')
105 expect(output).not.toContain('(default: all)')
106 }, 30000)
107
96108 it('failure case - an unknown flag is rejected rather than ignored', async () => {
97109 // clapp hard-errors on unknown options, which is why a documented-but-
98110 // unimplemented flag crashed rather than degrading.