Skip to content

Fix API routes for Mintlify docs compatibility#102

Closed
fengtality wants to merge 5 commits intomainfrom
feat/mintlify-docs-compat-v2
Closed

Fix API routes for Mintlify docs compatibility#102
fengtality wants to merge 5 commits intomainfrom
feat/mintlify-docs-compat-v2

Conversation

@fengtality
Copy link
Copy Markdown
Contributor

@fengtality fengtality commented Dec 20, 2025

Summary

  • Add redirect_slashes=False to FastAPI app
  • Remove trailing slashes from root routes in routers (accounts, connectors, controllers, scripts, archived_bots, docker)
  • Remove gateway_proxy router (not needed for dashboard)

Why

Mintlify docs playground strips trailing slashes from API paths, causing 307 redirects that the playground doesn't follow. This change makes routes work both with and without trailing slashes.

Test plan

  • Verify API still works with existing clients
  • Test Mintlify docs playground can make requests without redirect errors
  • Verify gateway endpoints still work through dedicated gateway routers

🤖 Generated with Claude Code

fengtality and others added 4 commits January 15, 2026 10:37
- Add bots/conf/ to .gitignore to prevent config files from being checked in
- Add gateway_proxy.py router

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
DELETE requests with a body (e.g., /wallet/remove) were failing because
the proxy only read the body for POST, PUT, and PATCH methods.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Removes trailing slashes from routes in controllers, docker, and scripts routers
to prevent 307 redirects that the Mintlify docs playground cannot follow.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@fengtality fengtality force-pushed the feat/mintlify-docs-compat-v2 branch from 420ee58 to abd97c1 Compare January 21, 2026 14:28
Gateway proxy functionality is not needed - dashboard will use direct
Gateway endpoints instead.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@fengtality fengtality closed this Apr 8, 2026
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