Skip to content

Align Knowledge endpoints in 2.17 and Preview with API responses - #762

Open
cathalging wants to merge 1 commit into
mainfrom
cathalging/b3-knowledge
Open

cathalging wants to merge 1 commit into
mainfrom
cathalging/b3-knowledge

Conversation

@cathalging

Copy link
Copy Markdown
Contributor

Why?

The Knowledge operations (internal articles, folders, content snippets, content search and bulk actions) document errors the API never returns, leave out ones it does, and don't say which OAuth scope each one needs. Several examples also show fields or values the API doesn't send, so integrators build against the wrong contract.

How?

Each documented behaviour was checked against what the API returns, and the scopes, error responses, parameters and examples were corrected in both the 2.17 and Preview specs. No version after 2.17 changes these operations, so both get the same edits.

Generated with Claude Code

Correct the internal article, folder, content snippet, content search and
bulk action operations so the spec matches what the API returns.

- Add the OAuth scope line to the operations that lacked one
- Document the real error responses (401 token scope, 400 and 422
  validation, 404 unknown teammate or folder, live-experiment conflict)
  and fix examples whose codes or messages could not be produced
- Document page and per_page on list and search internal articles, the
  optional admin_id on tag removal, and the locale and json_blocks
  request fields
- Note that omitted keys (folder list fields, content search next and
  prev) are absent rather than null
- Regenerate the internal article update and search examples with every
  returned field, and drop the heading the markdown output never emits
- Add tags to the internal article list item (2.17) and widen the tag
  schema description to cover internal articles and content snippets

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

1 participant