Skip to content

fix(cli): Generate accurate command references - #1429

Merged
BYK merged 2 commits into
mainfrom
fix/agent-conversation-reference
Oct 6, 2026
Merged

BYK merged 2 commits into
mainfrom
fix/agent-conversation-reference

Conversation

@BYK

@BYK BYK commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Keep agent-conversation usage and runnable examples in command metadata so help, generated docs, and CLI skills agree.
  • Render optional, variadic, and compound argument syntax accurately across command references.
  • Reject malformed organization and conversation IDs before target resolution or API access, and require the view target at parse time.

Testing

  • On the merged tree, pnpm run tsc and pnpm run lint passed; CLI lint, policy, generated-file checks, and 126 focused tests passed.
  • Full CLI suite with --maxWorkers=2 --testTimeout=30000 --printConsoleTrace --coverage: 10,526 passed, 16 skipped; all 504 files passed. Other workspace suites: MCP core 1,752 passed / 6 skipped; MCP server 83 passed; MCP test client 80 passed; MCP Cloudflare 433 passed. Combined: 12,874 passed, 22 skipped.
  • Standard pnpm run test failed twice on the merged tree with SQLite disk-I/O errors across unrelated CLI tests and a timeout in the 1 MB ZIP test. The affected files passed in isolation; the complete CLI suite passed with reduced concurrency. The standard aggregate command has not passed on the merged tree.

Keep positional syntax and examples in command metadata, use them for help, docs, and skills, and validate agent-conversation targets before API access.

Co-Authored-By: OpenAI GPT-6 Sol <noreply@openai.com>
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Oct 6, 2026
Co-Authored-By: OpenAI GPT-6 Sol <noreply@openai.com>
@BYK
BYK merged commit 03fb39c into main Oct 6, 2026
24 checks passed
@BYK
BYK deleted the fix/agent-conversation-reference branch October 6, 2026 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant