chore(api): hide duplicate /admin/accounts route from OpenAPI

[mmabrouk]

Summary

The legacy admin_router.create_accounts endpoint
(api/oss/src/routers/admin_router.py) and the new
fastapi/accounts/router.create_accounts both surface in the OpenAPI
schema. Fern's TypeScript codegen collapses both to the same method
name (createAccounts), forcing downstream Fern post-processors to
disambiguate via a manual rename pass.

This excludes the legacy admin route from the schema with
include_in_schema=False, removing the collision at the source.

Why

This is a prerequisite for cleaner Fern client generation (see the
feat/sdk/typescript-fern-client PR, which currently carries a Python
post-processor in clients/scripts/generate.sh to rename the duplicate
method). With this fix in place, the post-processor can eventually be
removed.

Diff

• 1 file changed: api/oss/src/routers/admin_router.py (1 insertion)

Risks

• Backwards compatibility: include_in_schema=False only hides the
  route from the OpenAPI schema and any auto-generated docs/clients. The
  HTTP endpoint itself (POST /admin/accounts) continues to work
  unchanged. Anyone calling it directly via HTTP is unaffected.
• The route is admin-only and unauthenticated callers are already
  rejected — exposure surface does not change.

QA

• CI green
• Hit POST /admin/accounts directly (curl or Postman) — should
  return the same response as before
• Regenerate the Fern Python client locally (clients/scripts/generate.sh)
  and verify only one createAccounts method appears in the
  generated TS admin client
• Confirm the admin_create_accounts operation in the new
  fastapi/accounts/router is still in the OpenAPI schema (it is
  the canonical one going forward)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
agenta-documentation	Ready	Preview, Comment	May 4, 2026 10:54am

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 277acfe9-4d77-4faa-b65a-4b8af2886f13

📥 Commits

Reviewing files that changed from the base of the PR and between c63d6e9 and ecd5853.

📒 Files selected for processing (1)

• api/oss/src/routers/admin_router.py

---

📝 Walkthrough

Summary by CodeRabbit

Release Notes

No user-visible changes in this release. This update makes internal adjustments to API documentation generation.

Walkthrough

The /accounts POST endpoint in the admin router is updated to exclude itself from OpenAPI schema generation by adding include_in_schema=False to the route decorator. No function logic or behavior is modified.

Changes

OpenAPI Schema Exclusion

Layer / File(s)	Summary
Route Configuration api/oss/src/routers/admin_router.py	The @router.post(...) decorator for create_accounts now includes include_in_schema=False, preventing the endpoint from appearing in generated OpenAPI documentation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 60.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: hiding a duplicate admin route from OpenAPI by using the include_in_schema=False decorator.
Description check	✅ Passed	The description is comprehensive and directly related to the changeset, explaining the duplicate route problem, the solution implemented, and the rationale behind the change.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/api/admin-accounts-openapi

---

_{Review rate limit: 9/10 reviews remaining, refill in 6 minutes.}

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Railway Preview Environment

Status	Destroyed (PR closed)

Updated at 2026-05-05T08:36:31.807Z