Skip to content

feat(api): route /api/* platform paths and send experimental header#140

Open
steve-calvert-glean wants to merge 1 commit into
mainfrom
sc/platform-07-api-passthrough
Open

feat(api): route /api/* platform paths and send experimental header#140
steve-calvert-glean wants to merge 1 commit into
mainfrom
sc/platform-07-api-passthrough

Conversation

@steve-calvert-glean

Copy link
Copy Markdown
Collaborator

glean api now reaches the platform APIs: paths starting /rest/ or /api/ are used verbatim (bare paths keep the /rest/api/v1/ default, so existing usage is unchanged). Requests to /api/* automatically send X-Glean-Include-Experimental — without it every platform call returns a hidden 404 indistinguishable from a typo'd path. --experimental forces the header on classic endpoints; --preview displays it.

Independent of the migration stack: raw net/http only, no SDK dependency. Merge in any order.

🤖 Generated with Claude Code

- resolveAPIPath: /rest/* and /api/* pass verbatim; bare paths keep the
  /rest/api/v1/ default prefix
- auto-send X-Glean-Include-Experimental on /api/* requests; --experimental
  flag forces it on classic endpoints; --preview displays it
- update api schema registry entry

WIP: not yet verified (go module cache unavailable — see session notes)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Copy link
Copy Markdown
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@steve-calvert-glean
steve-calvert-glean marked this pull request as ready for review July 20, 2026 00:17
@steve-calvert-glean
steve-calvert-glean requested a review from a team as a code owner July 20, 2026 00:17
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