Skip to content

feat(cli): add table/json output formats for repo commands - #57

Merged
cursor[bot] merged 6 commits into
mainfrom
table-output-v1
Jul 20, 2026
Merged

feat(cli): add table/json output formats for repo commands#57
cursor[bot] merged 6 commits into
mainfrom
table-output-v1

Conversation

@lbrealdev

@lbrealdev lbrealdev commented Aug 19, 2025 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

  • Add -f/--format support for get-repo and list-repo (table default, json option)
  • Introduce CliOutput / CliFormatOutput helpers with PrettyTable for list output
  • Ignore JetBrains .idea/ in .gitignore

Notes

  • list-repo default page size changed from 50 → 20
  • Adds prettytable dependency

Test plan

  • get-repo -n <repo> -f table renders table
  • get-repo -n <repo> -f json returns JSON
  • list-repo -f table / -f json both work
  • Invalid --format values are rejected
  • Failed API calls do not print None
  • Add/update unit tests for format helpers and command wiring
  • Document -f/--format in README

@lbrealdev lbrealdev changed the title Adds new format output options feat(cli): add table/json output formats for repo commands Jul 20, 2026
lbrealdev and others added 6 commits July 20, 2026 11:13
@lbrealdev @cursoragent
feat: adds the functionality to choose the format output to the listi…
5826eba 
…ng repositories
@lbrealdev @cursoragent
feat(api): update get_repository and list_repositories functions
a05cd15
@lbrealdev @cursoragent
style: run lint
f44b15c
@lbrealdev @cursoragent
feat: push last changes
55809fe
@lbrealdev @cursoragent
chore: ignore JetBrains .idea directory
87fe2ca 
Co-authored-by: Cursor <cursoragent@cursor.com>
@cursoragent @lbrealdev
fix(cli): honor --format for get-repo and harden output
c4f7746 
Wire get_repository table/json branching, reject invalid format
choices, skip printing None on API failure, and add format tests.

Co-authored-by: akae <lbrealdev@users.noreply.github.com>
@cursor
cursor Bot force-pushed the table-output-v1 branch from 0d0a20a to c4f7746 Compare July 20, 2026 11:13
@cursor
cursor Bot merged commit c323a2f into main Jul 20, 2026
4 checks passed
@cursor

cursor Bot commented Jul 20, 2026 •
edited by lbrealdev
Loading

Copy link
Copy Markdown
Contributor

Checklist status (updated after merge)

Test plan

  • get-repo -n <repo> -f table renders table
  • get-repo -n <repo> -f json returns JSON
  • list-repo -f table / -f json both work
  • Invalid --format values are rejected
  • Failed API calls do not print None
  • Add/update unit tests for format helpers and command wiring
  • Document -f/--format in README

Review follow-ups

Bugs

  • get_repository ignores --format
  • Failed API calls can still print(None)
  • --format has no choices=

Gaps / polish

  • Add tests for format branching
  • Document -f/--format in README; call out list-repo default page 50 → 20
  • Cleanup: remove tutorial comments in CliOutput; align method names; prefer one output path over bare print(...)

@lbrealdev
lbrealdev deleted the table-output-v1 branch July 20, 2026 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lbrealdev @cursoragent