feat(cli): add --detailed flag to ix subsystems --list#209
Conversation
Pass detailed=true to GET /v1/subsystems for member_files and enriched call/import edges. Bump CLI version to 0.6.1. Co-authored-by: Cursor <[email protected]>
|
Re-reviewed against the updated server side ( However, the CLI side hasn't been updated to match the new server contract, which causes a real UX regression: ReproFresh ingest of guava (178 scored regions) against the updated memory-layer: The CLI calls Cause
Suggested fixes, pick whichever fits your taste
My vote is (1) + (2). Auto-paginate by default for the common case, plus expose flags for power users / scripts that want to scope or stream. Other small notes from the re-review
Memory-layer side LGTM, CLI side blocked on this. Once the CLI is wired up, both can ship together as 0.6.1. Adjust the "my vote" line if you have a strong preference different from mine. |
Summary
--detailedtoix subsystems --list, callingGET /v1/subsystems?detailed=truemember_filesand enrichedcalls_*/imports_*edgesType
Changes
ix-cli/src/cli/commands/subsystems.ts— new flag, validation, JSON output pathsix-cli/src/client/api.ts—listSubsystems({ detailed })query paramValidation
curl "http://localhost:8090/v1/subsystems?detailed=true"returnsmember_files,calls_*,imports_*node ix-cli/dist/cli/main.js subsystems --list --detailed --format jsonmatches API shapeix subsystems --detailedwithout--listerrors as expectedChecklist
ix-cli/package.jsonversion bumped to 0.6.1v0.6.1on upstream to trigger release workflowMade with Cursor