Skip to content

Restore /content/search schemas to Preview spec#546

Merged
kilian-tennyson merged 1 commit into
mainfrom
work/ddf22236/worker
Jun 9, 2026
Merged

Restore /content/search schemas to Preview spec#546
kilian-tennyson merged 1 commit into
mainfrom
work/ddf22236/worker

Conversation

@kilian-tennyson

Copy link
Copy Markdown
Contributor

Why?

The GET /content/search endpoint is live in production but missing from the Preview spec. It was originally documented in #513 (merged 2026-05-22), then silently removed four days later by sync #518 — a 2,097-line docs sync that dropped schemas not yet present in the source repo's main branch. The endpoint is invisible to SDK generation as a result.

How?

Re-adds the path block and five content_search_* component schemas (with oneOf + type discriminator), plus a new Content tag between Contacts and Content Snippets. Schema matches the runtime contract:

  • query is optional (returns most recent when omitted), maxLength 500
  • per_page defaults 10, max 50
  • page defaults 1, minimum 1

Validated locally — YAML parses cleanly, npx fern-api check passes, diff is purely additive (1 file changed, 246 insertions, 0 deletions). Unblocks planned filter-parameter extensions to /content/search.

Coordination

A competing inlined-schema approach exists in the docs source repo. This PR supersedes that approach with the named-schema design from #513 (plus a `query: required: false` correction matching runtime). A sibling docs-side PR mirroring this shape will follow.

Generated with Claude Code

Re-adds the GET /content/search path and the five content_search_*
component schemas that were silently removed by sync PR #518. Matches
runtime behavior of Api::V3::Content::SearchController:

- query is optional (returns most recent when omitted), maxLength 500
- per_page defaults 10, max 50
- page defaults 1, minimum 1

Supersedes the inlined-schema design of developer-docs PR #942 with
the named-schema design from OpenAPI PR #513, so the path block can be
cleanly extended by the planned Tier-1 filter wave.

@bhavishnadar bhavishnadar 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.

Thanks a mil @kilian-tennyson

@kilian-tennyson kilian-tennyson merged commit 8b3ca83 into main Jun 9, 2026
3 checks passed
@kilian-tennyson kilian-tennyson deleted the work/ddf22236/worker branch June 9, 2026 13:34
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