Skip to content

docs(API): improve get /projects/{project_id}/documents documentation#1183

Open
Stephen Lumenta (sbl) wants to merge 2 commits into
mainfrom
devex-119-documents-list-documents-v2
Open

docs(API): improve get /projects/{project_id}/documents documentation#1183
Stephen Lumenta (sbl) wants to merge 2 commits into
mainfrom
devex-119-documents-list-documents-v2

Conversation

@sbl

Copy link
Copy Markdown
Contributor

Improves the documentation for get /projects/{project_id}/documents: sharper descriptions, parameter docs, error responses, and usage examples.

Drafted with AI assistance and grounded in the API implementation. Please review for technical accuracy before merging; nothing is merged automatically.

🤖 Generated with Claude Code

@sbl Stephen Lumenta (sbl) added the developer-hub-api-quality API doc quality fix from the API Grader label Jun 15, 2026
@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

API changelog (oasdiff)

Doc-only edits (descriptions, examples) do not appear here.

10 changes: 0 error, 0 warning, 10 info
info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/api_name` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/default_encoding` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/default_file` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/description` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/exportable` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/extension` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/importable` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/includes_locale_information` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/name` became required for the status `200`

info	[response-property-became-required] at doc/compiled.json
	in API GET /formats
		the response property `items/renders_default_locale` became required for the status `200`

@sbl

Copy link
Copy Markdown
Contributor Author

This PR replaces #1178, which was closed after reviews from jablan and theSoenke.

Feedback addressed from #1178:

Reviewer Comment Status
theSoenke "The policy should not be part of the API docs" ✅ Fixed
theSoenke "per_page and rate limit details do not need to be repeated per endpoint" ✅ Fixed
jablan "I don't think it's helpful to say the list is paginated here" ✅ Fixed
jablan "Pagination information is superfluous here" (RFC 5988 / Link header sentence) ✅ Fixed
jablan "Please do not expose identifier names from internal applications" ✅ Fixed
jablan "Rate limiting applies universally and is explained on a dedicated place" ✅ Fixed

What changed: The description is fully rewritten. All six flagged items are removed:

  • No DocumentPolicy#list_documents? or any internal class/policy name
  • No pagination prose ("returns a paginated list", RFC 5988, Link header)
  • No per_page cap or default
  • No rate-limit pitfall

What remains is endpoint-specific: what a document is, when you'd use this endpoint, how the q prefix match works, and the auth requirement.

🤖 Generated with Claude Code

@sbl Stephen Lumenta (sbl) changed the title feat(API): improve get /projects/{project_id}/documents documentation fix(API): improve get /projects/{project_id}/documents documentation Jun 17, 2026
Comment thread paths/documents/index.yaml Outdated
Comment thread paths/documents/index.yaml Outdated
Comment thread paths/documents/index.yaml Outdated
- Correct the document-type description: only HTML and DOCX files are
  listed (was 'HTML or YAML'); fix the example value invoice.yaml ->
  invoice.docx accordingly. (theSoenke)
- Drop the 'Requires the read OAuth scope ...' sentence from the
  description; no other endpoint documents scopes in prose and the 403
  response already explains the read requirement. (theSoenke)
- Remove the 'Response' code sample; the 200 example already shows the
  response shape and no other endpoint inlines a response sample. (theSoenke)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@sbl Stephen Lumenta (sbl) changed the title fix(API): improve get /projects/{project_id}/documents documentation docs(API): improve get /projects/{project_id}/documents documentation Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

developer-hub-api-quality API doc quality fix from the API Grader

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants