Repository navigation
Fix stale command and flag names in agent docs and skills - #28
Open
Snodgrass-Wilkerschnoz wants to merge 1 commit into
Open
Snodgrass-Wilkerschnoz wants to merge 1 commit into
Snodgrass-Wilkerschnoz wants to merge 1 commit into
Conversation
`list-call-results` and `run-call` no longer exist on production or QC, and neither results command accepts `--since`/`--before`. Rename them to `list-results-by-call`, `run-monitor`, and `--from`/`--time`, document what the time-window flags mean, and note which API monitor commands are deprecated on platforms serving the v3 API. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
AGENTS.mdand the skill files name commands and flags that the current production and QC specs no longer serve. An agent following them hitsunknown commandorunknown flagerrors. This PR fixes the names to match both live specs.list-call-results <id>list-results-by-call <id>run-call <id>run-monitor <id>--since <ISO>--from <ISO>(window start)--from/--timeexist--from/--timeexist--before <ISO>--time <ISO>(window end, returns results before it)Other changes:
AGENTS.mdgets two new bullets:--fromis the window start and--timethe window end. Pass full timestamps; a bare date like2026-10-14returns no results.create-call,list-calls,get-call,update-callanddelete-callas deprecated in favour of the*-api-monitorcommands. Production does not have those yet, so the docs keep the older names and tell agents to check--help.skills/embed/setup-api-monitor.md: the verify step pointed at fields that aren't in the result rows (result.success,result.failure_reason). It now usesmeta.result_categoryandmeta.http_code.run-monitor's documented response now matches what it actually returns:{call_id, context, location_id, result_id, user}.create-call,list-callsand the other v1 names are left as they are because production still uses them. Switching the docs to the v3 names should wait until production serves them.How this was checked
--helpagainst both builds (-tags prodand the default QC build).--from/--timeagainst real results on QC to confirm which flag is the start and which the end.go test ./...passes.🤖 Generated with Claude Code