Skip to content

Conversation

@T-rav
Copy link
Collaborator

@T-rav T-rav commented Oct 10, 2025

Summary

Remove all demo-related functionality and update version to 2.3.2

Changes

Removed Files

  • docker/simple_demo.py - Demo server with interactive UI
  • docker/demo_utils.py - Demo utility functions
  • docker/endpoints/demo_client.py - Demo client endpoint

Makefile Updates

  • Removed dev-demo command (demo server with auto-reload)
  • Removed demo command (production demo server)
  • Removed demo-bg command (background demo server)
  • Simplified docker-run command (removed WEBCAT_MODE)
  • Updated stop-bg to only stop MCP server
  • Updated logs command to remove demo.log reference

README Updates

  • Removed demo UI mentions from Quick Start
  • Removed demo screenshot
  • Removed demo endpoint from Available Endpoints table
  • Removed demo commands from Local Development section
  • Updated Project Structure (replaced simple_demo.py with cli.py)
  • Updated version from 2.3.1 → 2.3.2

Version Update

  • Updated all version references to 2.3.2
  • Updated Docker build examples to use 2.3.2 tag

Rationale

Simplify the codebase by removing demo functionality that is not core to the MCP server functionality. Users can interact with the MCP server directly through MCP clients.

Test plan

  • All changes are code removal and documentation updates
  • Pre-commit hooks passed
  • No functional changes to core MCP server

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Updated version to 2.3.2 and refreshed Docker tags/instructions.
    • Rewrote getting-started to use MCP-based startup; removed references to the legacy demo UI/server.
  • Chores

    • Removed legacy demo components and related run targets.
    • Simplified run/stop workflows; docker run no longer exposes the extra demo port.
    • Updated logging: logs now written to /tmp/webcat*.log.

- Remove demo-related Python files (simple_demo.py, demo_utils.py, demo_client.py)
- Remove demo commands from Makefile (dev-demo, demo, demo-bg)
- Update README to remove all demo references
- Update version from 2.3.1 to 2.3.2
- Simplify docker-run command (remove WEBCAT_MODE)
- Update project structure documentation

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Oct 10, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Removes the legacy demo server and related utilities/endpoints, updates Makefile targets to drop demo workflows, adjusts logging paths and docker run settings, and revises README to version 2.3.2 with MCP/CLI-based startup instructions replacing the demo server references.

Changes

Cohort / File(s) Summary
Build and run targets
Makefile
Removed demo-related PHONY targets and commands (dev-demo, demo, demo-bg); updated docker-run to drop port 8001 and WEBCAT_MODE=demo; updated stop-bg to remove simple_demo.py pkill; logs now tails /tmp/webcat*.log.
Documentation and versioning
README.md
Bumped version 2.3.1 → 2.3.2; replaced demo UI/server instructions with MCP/CLI-focused flow; updated Docker tags and release notes references.
Demo utilities removal
docker/demo_utils.py
Deleted module providing SSE formatting, search/health async handlers, and server info helper.
Demo client endpoint removal
docker/endpoints/demo_client.py
Deleted helpers and endpoint logic for serving the demo HTML client.
Demo server removal
docker/simple_demo.py
Deleted FastAPI-based demo server (creation, health/status routes, SSE mounting, entrypoint).

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Dev as Developer
    participant Make as Makefile
    participant CLI as CLI / mcp_server.py
    Note over Dev,CLI: New flow (demo removed)
    Dev->>Make: make dev / run commands
    Make->>CLI: Start MCP/CLI server process
    CLI-->>Dev: Ready / logs in /tmp/webcat*.log
Loading
sequenceDiagram
    autonumber
    participant User as User (Browser)
    participant Demo as simple_demo.py (Removed)
    participant SSE as SSE Handlers (Removed)
    Note over User,SSE: Old demo flow (removed)
    User--X Demo: GET / (demo client)
    Demo--X SSE: Stream events (search/health)
    SSE--X User: SSE messages
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

I tidied my burrow, swept out the show,
No more demo carrots in a neat little row.
Now I hop to MCP, quick as a breeze,
Logs in /tmp whisper through trees.
Version bumps, whiskers up—2.3.2, let’s go! 🥕✨

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/remove-demo-v2.3.2

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a4756cc and d94fb9d.

📒 Files selected for processing (5)
  • Makefile (3 hunks)
  • README.md (6 hunks)
  • docker/demo_utils.py (0 hunks)
  • docker/endpoints/demo_client.py (0 hunks)
  • docker/simple_demo.py (0 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@T-rav T-rav merged commit e4c1603 into main Oct 10, 2025
11 checks passed
@T-rav T-rav deleted the chore/remove-demo-v2.3.2 branch October 10, 2025 00:34
@T-rav T-rav restored the chore/remove-demo-v2.3.2 branch October 10, 2025 00:40
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.

2 participants