Skip to content

docs: update docs for --sidebar, --app-name, --disable-commands flags, OAuth meta keys rename, and hooks model_id gap#2922

Closed
aheritier wants to merge 1 commit into
mainfrom
docs/update-36h-changes
Closed

docs: update docs for --sidebar, --app-name, --disable-commands flags, OAuth meta keys rename, and hooks model_id gap#2922
aheritier wants to merge 1 commit into
mainfrom
docs/update-36h-changes

Conversation

@aheritier
Copy link
Copy Markdown
Contributor

Summary

Catches up /docs with user-facing changes merged in the last 36 hours.

Changes

docs/features/cli/index.md — 3 new flags documented

Added to the docker agent run flags table (alongside --lean which they complement):

Flag Description
--sidebar Show the sidebar in the TUI (default true; --sidebar=false hides it permanently, disabling Ctrl+B and mouse toggle)
--app-name <name> Override the application name shown in the TUI status bar, terminal title, and /exit notifications
--disable-commands <list> Comma-separated slash commands to hide from TUI, command palette, and editor completion. Leading slashes optional, case-insensitive.

Refs: #2917, #2914, #2913

docs/features/remote-mcp/index.md — OAuth elicitation meta key prefix corrected

The unmanaged OAuth section referenced the old cagent/ key prefix. Updated all four keys to the renamed docker-agent/ prefix per #2915:

  • cagent/typedocker-agent/type
  • cagent/server_urldocker-agent/server_url
  • cagent/authorize_urldocker-agent/authorize_url
  • cagent/statedocker-agent/state

Refs: #2915

docs/configuration/hooks/index.mdmodel_id added to hook payload tables

The per-event extra fields table was missing model_id for both before_llm_call and after_llm_call. Added the field and a note explaining its provider/model canonical form. This closes the docs gap explicitly flagged in #2911.

Refs: #2911

No changes needed

  • docs/features/tui/index.md — does not enumerate CLI flags; no update needed.
  • docs/features/mcp-mode/index.md — no stale OAuth meta key references found.
  • docs/features/remote-mcp/index.md OAuth drive-flow section — already written by Extend unmanaged OAuth flow to drive code exchange in-process #2896 itself; only the key-name rename was stale.

…x hooks model_id gap

- docs/features/cli/index.md: add --sidebar, --app-name, --disable-commands
  flags to the docker agent run flags table (PR #2917, #2914, #2913)
- docs/features/remote-mcp/index.md: rename OAuth elicitation Meta keys from
  cagent/ to docker-agent/ prefix (PR #2915)
- docs/configuration/hooks/index.md: add model_id field to before_llm_call
  and after_llm_call per-event payload table and notes (PR #2911)
@aheritier aheritier requested a review from a team as a code owner May 29, 2026 07:52
Copy link
Copy Markdown

@docker-agent docker-agent left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The documentation updates look accurate and complete:

  • , , flags: All three entries are correctly described, with accurate default values and behavior descriptions. The note that --sidebar=false disables both Ctrl+B and mouse toggle is consistent with the permanent-hide semantics.
  • OAuth meta key rename: All 4 keys (cagent/docker-agent/) were updated consistently across both the table and all prose references in the unmanaged OAuth section.
  • model_id in hook payload tables: Added to both before_llm_call and after_llm_call with a correct clarifying note about the provider/model canonical form.

No bugs or inaccuracies found in the changed lines.

@aheritier aheritier added area/cli CLI commands, flags, output formatting area/mcp MCP protocol, MCP tool servers, integration kind/docs Documentation-only changes status/needs-rebase PR has merge conflicts or is out of date with main labels May 29, 2026
@aheritier aheritier marked this pull request as draft May 29, 2026 09:32
@aheritier aheritier closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cli CLI commands, flags, output formatting area/mcp MCP protocol, MCP tool servers, integration kind/docs Documentation-only changes status/needs-rebase PR has merge conflicts or is out of date with main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants