Skip to content

refactor: curated key/value detail table for repo get#97

Merged
cursor[bot] merged 1 commit into
mainfrom
cursor/repo-get-detail-output-503e
Jul 21, 2026
Merged

refactor: curated key/value detail table for repo get#97
cursor[bot] merged 1 commit into
mainfrom
cursor/repo-get-detail-output-503e

Conversation

@lbrealdev

@lbrealdev lbrealdev commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

repo get table mode no longer reuses the list summary columns. It now renders a curated Field | Value detail view, while --format json still returns the full raw GitHub API payload.

Changes

  • Replaced CliOutput / CliFormatOutput with:
    • generic renderers (to_json, to_table, to_key_value_table)
    • projections (project_repo_summary, project_repo_detail)
    • entry points (format_repo_get, format_repo_list)
  • repo list summary fields (name, owner, url, visibility) unchanged
  • Updated CLI docs and changelog
  • Added projection/format unit tests

Related

Test plan

  • uv run pytest -v (46 passed)
  • Manually: github-rest-cli repo get -n <repo> -o <owner> shows key/value detail
  • Manually: github-rest-cli repo get -n <repo> -f json still dumps full API object
  • Manually: github-rest-cli repo list table/JSON unchanged
Open in Web Open in Cursor 

Split output formatting into projection helpers and generic renderers.
repo get table now shows curated Field/Value details; JSON stays raw.

Co-authored-by: akae <lbrealdev@users.noreply.github.com>
@lbrealdev
lbrealdev marked this pull request as ready for review July 21, 2026 06:59
@cursor

cursor Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@cursor

cursor Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Housekeeping note: These issues were completed in #96 / v2.0.0 but left open (no auto-close keywords). Please close them manually:

Leave open: #93 (pagination) — still outstanding.

This PR (#97) does not close those; it only relates to the “richer fields” idea from #80.

@cursor
cursor Bot merged commit b7979d4 into main Jul 21, 2026
4 checks passed
@cursor
cursor Bot deleted the cursor/repo-get-detail-output-503e branch July 21, 2026 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add docs/cli.md CLI usage guide

2 participants