Skip to content

🐛 add warning if webserver deps isn't installed when running dev or serve#28

Open
MFA-X-AI wants to merge 2 commits intomainfrom
fahreza/webserver-import-warning
Open

🐛 add warning if webserver deps isn't installed when running dev or serve#28
MFA-X-AI wants to merge 2 commits intomainfrom
fahreza/webserver-import-warning

Conversation

@MFA-X-AI
Copy link
Member

When running xaibo dev or xaibo serve without the webserver optional dependencies installed, users got a cryptic TypeError: 'NoneType' object is not callable because XaiboWebServer import failed silently. This PR adds some handling for that.

Changes:

  • Added explicit None checks in dev() and serve() functions before using XaiboWebServer
  • Print helpful error message with installation instructions: uv add xaibo[webserver]

Before:

TypeError: 'NoneType' object is not callable

After:

Error: XaiboWebServer is not available.
The webserver dependencies are required for 'xaibo dev'.
Install them with: uv add xaibo[webserver]

Reference:
#26

@MFA-X-AI MFA-X-AI requested a review from treo February 19, 2026 18:57
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