Skip to content

Replace HttpPlugin with HttpServer + HttpServerAdapter#310

Open
heyitsaamir wants to merge 1 commit intoaamirj/simplifyHttpfrom
aamirj/http-server-adapter
Open

Replace HttpPlugin with HttpServer + HttpServerAdapter#310
heyitsaamir wants to merge 1 commit intoaamirj/simplifyHttpfrom
aamirj/http-server-adapter

Conversation

@heyitsaamir
Copy link
Collaborator

Summary

  • Replace HttpPlugin with HttpServer + HttpServerAdapter pattern — follows the TS PR #433 approach
  • HttpServer is no longer a plugin — it's owned directly by App
  • Introduces HttpServerAdapter protocol with FastAPIAdapter as the default implementation
  • Plugins inject HttpServer via DI and access the framework via server.adapter
  • Framework-agnostic handler pattern: async (HttpRequest) -> HttpResponse
  • Rewrites remote_function_jwt_middleware to be framework-agnostic
  • Adds examples/http-adapters/ with Starlette adapter and non-managed FastAPI examples

Stacked on #249

Test plan

  • Run poe test — all tests pass
  • Run echo example end-to-end
  • Verify DevTools plugin starts correctly alongside main server
  • Verify BotBuilder plugin route registration

🤖 Generated with Claude Code

Move to HttpServer instead of HttPlugin following the TS PR.
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