refactor(routes): streamline provider-specific messages route setup

This commit is contained in:
Vaayne 2025-09-25 23:04:20 +08:00
parent 5dbd38721f
commit a7d6065b08

View File

@ -118,12 +118,11 @@ app.get('/', (_req, res) => {
}) })
}) })
// Provider-specific API routes with auth (must be before /v1 to avoid conflicts) // Setup OpenAPI documentation before protected routes so docs remain public
const providerRouter = express.Router({ mergeParams: true }) setupOpenAPIDocumentation(app)
providerRouter.use(authMiddleware)
// Mount provider-specific messages route // Provider-specific messages route requires authentication
providerRouter.use('/v1/messages', messagesProviderRoutes) app.use('/:provider/v1/messages', authMiddleware, messagesProviderRoutes)
app.use('/:provider', providerRouter)
// API v1 routes with auth // API v1 routes with auth
const apiRouter = express.Router() const apiRouter = express.Router()
@ -136,9 +135,6 @@ apiRouter.use('/models', modelsRoutes)
apiRouter.use('/agents', agentsRoutes) apiRouter.use('/agents', agentsRoutes)
app.use('/v1', apiRouter) app.use('/v1', apiRouter)
// Setup OpenAPI documentation
setupOpenAPIDocumentation(app)
// Error handling (must be last) // Error handling (must be last)
app.use(errorHandler) app.use(errorHandler)