Mastra server.build config for swaggerUI, openAPIDocs, apiReqLogs

[TheIsrael1]

Description

Related Issue(s)

Type of Change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update
• Code refactoring
• Performance improvement
• Test update

Checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation (if applicable)
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have generated a changeset for this PR

[codesandbox]

Review or Edit in CodeSandbox

Open the branch in Web Editor • VS Code • Insiders

Open Preview

[changeset-bot]

🦋 Changeset detected

Latest commit: c54f9c2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 54 packages

Name	Type
@mastra/deployer-cloudflare	Patch
@mastra/deployer-netlify	Patch
@mastra/deployer	Patch
@mastra/deployer-vercel	Patch
@mastra/core	Patch
mastra	Patch
create-mastra	Patch
@mastra/evals	Patch
@mastra/loggers	Patch
@mastra/mcp-docs-server	Patch
@mastra/mcp-registry-registry	Patch
@mastra/mcp	Patch
@mastra/memory	Patch
@mastra/playground-ui	Patch
@mastra/rag	Patch
@mastra/server	Patch
@mastra/astra	Patch
@mastra/chroma	Patch
@mastra/clickhouse	Patch
@mastra/cloudflare-d1	Patch
@mastra/cloudflare	Patch
@mastra/pg	Patch
@mastra/pinecone	Patch
@mastra/qdrant	Patch
@mastra/turbopuffer	Patch
@mastra/upstash	Patch
@mastra/vectorize	Patch
@mastra/speech-azure	Patch
@mastra/speech-deepgram	Patch
@mastra/speech-elevenlabs	Patch
@mastra/speech-google	Patch
@mastra/speech-ibm	Patch
@mastra/speech-murf	Patch
@mastra/speech-openai	Patch
@mastra/speech-playai	Patch
@mastra/speech-replicate	Patch
@mastra/speech-speechify	Patch
@mastra/voice-azure	Patch
@mastra/voice-cloudflare	Patch
@mastra/voice-deepgram	Patch
@mastra/voice-elevenlabs	Patch
@mastra/voice-google	Patch
@mastra/voice-murf	Patch
@mastra/voice-openai-realtime	Patch
@mastra/voice-openai	Patch
@mastra/voice-playai	Patch
@mastra/voice-sarvam	Patch
@mastra/voice-speechify	Patch
@mastra/client-js	Patch
@mastra/firecrawl	Patch
@mastra/github	Patch
@mastra/mem0	Patch
@mastra/ragie	Patch
@mastra/dane	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
assistant-ui	❌ Failed (Inspect)			Apr 22, 2025 3:09pm
bird-checker	❌ Failed (Inspect)			Apr 22, 2025 3:09pm
crypto-chatbot	❌ Failed (Inspect)			Apr 22, 2025 3:09pm
mastra-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 22, 2025 3:09pm
openapi-spec-writer	❌ Failed (Inspect)			Apr 22, 2025 3:09pm
travel-ai	🛑 Canceled (Inspect)			Apr 22, 2025 3:09pm

[promptless]

📝 Documentation updates detected! You can review documentation updates here

[pechitook]

Hey @TheIsrael1 - I tried to do something similar in #3641. Also not a contributor [yet 😄] so I probably don't have enough context but still gonna throw some comments - hopefully some of them are useful

1. How about defaulting the same way for OpenAPI? Opting out from exposing SwaggerUI by default but keeping the OpenAPI detailed spec including all API endpoints seems inconsistent in terms of hiding sensitive endpoint data.
2. Can we delete dev.entry.js now in favor of the new bundle file? Seems unused outside BuildBundler.ts at least by following code around.
3. I know it's a draft, but bundleOptions is probably worth typing as custom deployers may need to use it directly when implementing bundle

[promptless]

📝 Documentation updates detected! You can review documentation updates here

[wardpeet]

We should probably always enable all of these on dev and only read the config on production builds

[TheIsrael1]

oh ok, we can do that, we can have a build property on the server config

[TheIsrael1]

ok we have a server.build config now that applies if !isDev in the hono server

[moret]

Hi @TheIsrael1 . Could you help me understand if I'm using this wrong? It tried these options on a production build, but they are not being respected. Following the built code I noticed it's calling await createNodeServer(mastra); without options, and inside it there's a call for const app = await createHonoServer(mastra, options);, which makes it look like these options under Mastra.server.build are not being read. Do I need to set something else? Also, would it make sense to also keep the playground option?

[github-actions]

Braintrust eval report

No experiments to report