Skip to content

docs: Fix typo in config_option_update - should be singular - #707

Merged
benbrandt merged 1 commit into
agentclientprotocol:mainfrom
carlos-algms:patch-1
Mar 9, 2026
Merged

benbrandt merged 1 commit into
agentclientprotocol:mainfrom
carlos-algms:patch-1

Conversation

@carlos-algms

Copy link
Copy Markdown
Contributor

Summary

The session-config-options.mdx page uses "config_options_update" (plural) as the sessionUpdate discriminator value, but the schema (schema.md) defines it as "config_option_update" (singular)

References

  • Schema definition uses ConfigOptionUpdate"config_option_update"
  • Both Claude and Codex ACP send "config_option_update" (singular):

## Summary

The `session-config-options.mdx` page uses `"config_options_update"`
(plural) as the `sessionUpdate` discriminator value, but the schema
(`schema.md`) defines it as `"config_option_update"` (singular)

## References

- Schema definition uses `ConfigOptionUpdate` → `"config_option_update"`
- Both Claude and Codex ACP send `"config_option_update"` (singular):
  - [claude-agent-acp](https://github.com/zed-industries/claude-agent-acp) (`src/acp-agent.ts`)
  - [codex-acp](https://github.com/zed-industries/codex-acp) (`src/thread.rs` via `SessionUpdate::ConfigOptionUpdate`)
@carlos-algms
carlos-algms requested a review from a team as a code owner March 9, 2026 21:31

@benbrandt benbrandt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@benbrandt benbrandt changed the title Fix typo in config_option_update - should be singular docs: Fix typo in config_option_update - should be singular Mar 9, 2026
@benbrandt
benbrandt enabled auto-merge (squash) March 9, 2026 21:55
@benbrandt
benbrandt disabled auto-merge March 9, 2026 21:55
@benbrandt
benbrandt enabled auto-merge (squash) March 9, 2026 21:55
@benbrandt
benbrandt merged commit 01b96b9 into agentclientprotocol:main Mar 9, 2026
1 check passed
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.

2 participants