fix(cli): recognize --no-upload option, decouple --json from upload
Summary
- Fix
--no-uploadoption not being recognized incheckupcommand - Remove automatic
--no-uploadimplication when--jsonis used (now independent) - Add comment documenting the decoupled behavior
- Add comprehensive tests for upload flag combinations
Tests added
-
--uploadand--no-uploadoptions are recognized -
--jsondoes not imply--no-upload(decoupled behavior) -
--json --uploadrequires API key -
--json --no-uploaddisables upload -
--uploadrequires API key (without config) -
--no-uploaddoes not require API key
Test plan
-
bun test checkup.test.tspasses (70 tests) -
bun ./bin/postgres-ai.ts checkup --helpshows both options -
bun ./bin/postgres-ai.ts checkup postgresql://... --no-uploadno longer errors with "unknown option"
Closes #100 (closed)
Edited by Bogdan Tsechoev