Skip to content

Style improvements to ls tool - #14

Merged
rakyll merged 1 commit into
mainfrom
ls-tool
Apr 18, 2025
Merged

rakyll merged 1 commit into
mainfrom
ls-tool

Conversation

@rakyll

@rakyll rakyll commented Apr 18, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@rakyll
rakyll requested a review from NTaylorMullen April 18, 2025 00:35
@rakyll

rakyll commented Apr 18, 2025

Copy link
Copy Markdown
Contributor Author

Every tool needs similar improvement. This is only the beginning of a series of PRs.


private errorResult(params: LSToolParams, llmContent: string, returnDisplay: string): LSToolResult {
return {
entries: [],

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want, you can delete the LSToolResult type and just return ToolResult. I don't believe we actully use this output today.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll revisit this as a follow up.

@rakyll
rakyll merged commit 93fd6a9 into main Apr 18, 2025
@rakyll
rakyll deleted the ls-tool branch April 18, 2025 17:57
keithballinger pushed a commit that referenced this pull request Jun 13, 2025
yaleh pushed a commit to yaleh/gemini-cli that referenced this pull request Jun 29, 2025
viridivn pushed a commit to viridivn/gemini-cli that referenced this pull request Jun 29, 2025
yewenchen pushed a commit to yewenchen/gemini-cli that referenced this pull request Jul 8, 2025
skeshive pushed a commit that referenced this pull request Nov 18, 2025
skeshive pushed a commit that referenced this pull request Nov 18, 2025
@sripasg sripasg added the size/m A medium sized PR label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m A medium sized PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants