Skip to content

Conversation

@tgmendes
Copy link
Contributor

@tgmendes tgmendes commented Dec 10, 2025

Creates the API to be able to query for tool logs.

Examples

curl -X GET 'https://localhost:8080/rpc/logs.listToolExecutionLogs?per_page=2&sort=DESC' 
-H 'Gram-Key: <redacted>' 
-H 'Gram-Project: default' 
--insecure -s | jq
image

Fetching next page:

curl -X GET 'https://localhost:8080/rpc/logs.listToolExecutionLogs?per_page=2&sort=DESC&cursor=019b0900-0004-7000-8000-000000000004' 
-H 'Gram-Key: redacted' 
-H 'Gram-Project: default' 
--insecure -s | jq
image

@linear
Copy link

linear bot commented Dec 10, 2025

@changeset-bot
Copy link

changeset-bot bot commented Dec 10, 2025

🦋 Changeset detected

Latest commit: 349336f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
server Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link

vercel bot commented Dec 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
gram Ready Ready Preview Comment Dec 11, 2025 2:38pm
gram-docs-redirect Ready Ready Preview Comment Dec 11, 2025 2:38pm
gram-landing-redirect Ready Ready Preview Comment Dec 11, 2025 2:38pm

Base automatically changed from tiago/age-972-refactor-telemetry-structure to main December 10, 2025 14:24
}
}

func (s *Service) ListToolExecutionLogs(ctx context.Context, payload *gen.ListToolExecutionLogsPayload) (res *gen.ListToolExecutionLogsResult, err error) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: I had to rename these methods to ListToolExecutionLogs rather than just ListToolLogs because of conflicts with names of other endpoints for the service.

For consistency I'll update the name used in the repo queries as well in a separate PR (which are now called ListToolLogs).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The tests here were redundant now that we have the broader service tests


Meta("openapi:operationId", "listToolExecutionLogs")
Meta("openapi:extension:x-speakeasy-name-override", "listToolExecutionLogs")
Meta("openapi:extension:x-speakeasy-react-hook", `{"name": "ListToolExecutionLogs"}`)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Meta("openapi:extension:x-speakeasy-react-hook", `{"name": "ListToolExecutionLogs"}`)
Meta("openapi:extension:x-speakeasy-react-hook", `{"name": "ToolExecutionLogs"}`)

will map to useToolExecutionLogs instead of useListToolExecutionLogs

Comment on lines 260 to 261
Enum("ASC", "DESC")
Default("DESC")
Copy link
Contributor

Choose a reason for hiding this comment

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

are these case insensitive? (prefer they're lowercase regardless)

- explicit error handling of incorrect params
- lowercase DESC and ASC in pagination
- change react name to follow convention
@tgmendes tgmendes merged commit e0b26ea into main Dec 11, 2025
24 of 29 checks passed
@tgmendes tgmendes deleted the tiago/age-972-api-for-tool-logs branch December 11, 2025 14:48
@github-actions github-actions bot locked and limited conversation to collaborators Dec 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants