Skip to content

Conversation

@ditadi
Copy link
Contributor

@ditadi ditadi commented Dec 10, 2025

Summary

  • Make ServerPlugin independent of Databricks - middleware now only applies to plugins that declare needsRequestContext = true
  • Add needsRequestContext property to Plugin base class (default: false)
  • Remove required env vars from ServerPlugin (DATABRICKS_APP_PORT, FLASK_RUN_HOST have defaults)
  • Auto-detect static path from common build directories (dist/, client/dist/, build/, public/, out/)

Test plan

  • createApp({ plugins: [server()] }) should start without any Databricks env vars
  • createApp({ plugins: [server(), analytics()] }) should apply Databricks middleware only to analytics routes
  • Static files should be auto-detected and served from first directory containing index.html
  • All existing tests pass

@ditadi ditadi requested review from a team and MarioCadenas December 10, 2025 21:17
@ditadi ditadi enabled auto-merge (squash) December 11, 2025 10:34
@ditadi ditadi merged commit a135f13 into main Dec 11, 2025
3 checks passed
@MarioCadenas MarioCadenas deleted the fix/app-kit-integration-without-mcp branch December 19, 2025 17:44
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.

3 participants