Skip to content

Drop --limit from get and get-external - #254

Merged
rochala merged 1 commit into
mainfrom
remove-get-limit
Oct 21, 2026
Merged

rochala merged 1 commit into
mainfrom
remove-get-limit

Conversation

@rochala

@rochala rochala commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

-l/--limit on get and get-external cut the member list at N with no way to page past the cut. An agent had to rerun with a bigger N, or got an arbitrary prefix of the API. --hide-inherited is the meaningful way to shrink output: it drops whole parent types.

  • The option is removed from both commands. list/search keep --limit (default 50).
  • The limit parameter is removed from GetHandler, ProjectGetHandler and GetFormatter, and with it the // … N more members truncation.
  • Its two tests are removed. README and the skill are updated.

Breaking: cellar get-external -l 3 <coord> cats.Monad now fails with Unexpected option: -l.

Checked end to end: the command above is rejected, get-external --group-inherited on cats.Monad prints the full grouped list, and list-external -l 2 still truncates.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP

It truncated members with no way to page past the cut; --hide-inherited is
the way to shrink get output.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP
rochala added a commit that referenced this pull request Oct 21, 2026
--limit on get goes away in #254.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP
@rochala
rochala merged commit 5eebd36 into main Oct 21, 2026
14 checks passed
rochala added a commit that referenced this pull request Oct 21, 2026
* Tell agents when to use each get member mode

Refresh the example output to the current format, including the Supertypes
line from #247.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP

* Show the full member list in the skill example

--limit on get goes away in #254.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP

* Keep the skill example verbatim and word Supertypes order for Java too

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP

* Regenerate the skill example now that objects are listed once

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NKsT2pk4Cgn8JG8KsYEWoP

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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.

1 participant