Skip to content

fix(docs): update custom-agent docs to use required 'name' field - #4969

Open
Battleplus wants to merge 3 commits into
bytedance:mainfrom
Battleplus:fix/4942-agent-docs
Open

fix(docs): update custom-agent docs to use required 'name' field#4969
Battleplus wants to merge 3 commits into
bytedance:mainfrom
Battleplus:fix/4942-agent-docs

Conversation

@Battleplus

@Battleplus Battleplus commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

Summary

The custom-agent docs describe a display_name field and an auto-derived slug, but POST /api/agents requires callers to provide name. The documented Chinese payload is rejected with a 422 validation error.

Changes

Updated 4 affected pages (en/zh) to match the actual AgentCreateRequest contract:

  • frontend/src/content/en/application/agents-and-threads.mdx: Replaced display_name slug description with name field requirement; removed display_name from YAML example.
  • frontend/src/content/zh/application/agents-and-threads.mdx: Fixed curl example to use name; replaced slug/auto-derive explanation with name field requirement.
  • frontend/src/content/en/harness/lead-agent.mdx: Removed stale display_name references from Lead Agent docs.
  • frontend/src/content/zh/harness/lead-agent.mdx: Removed stale display_name references from Lead Agent docs (Chinese).

The custom-agent docs described a 'display_name' field and an
auto-derived slug, but POST /api/agents requires callers to provide
'name'. Updated all 4 affected pages (en/zh) to match the actual
AgentCreateRequest contract.

Closes bytedance#4942
@github-actions github-actions Bot added area:docs Documentation and Markdown only area:frontend Next.js frontend under frontend/ risk:low Low risk: docs / i18n / assets only size/XS PR changes < 20 lines labels Aug 23, 2026

@willem-bd willem-bd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One remaining documentation inconsistency leaves the reported display_name contract issue partially unresolved.

Comment thread frontend/src/content/en/application/agents-and-threads.mdx
@Battleplus

Copy link
Copy Markdown
Contributor Author

Fixed. Updated the capabilities bullet at line 22 to remove the auto-derived slug claim:

-- a **display name** and an auto-derived ASCII slug (the `name` used internally)
+- a **name** (caller-provided unique slug identifier) and an optional **display name** for the UI

This accurately reflects that name is caller-provided with no automatic derivation, consistent with the corrected paragraph below.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:docs Documentation and Markdown only area:frontend Next.js frontend under frontend/ risk:low Low risk: docs / i18n / assets only size/XS PR changes < 20 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants