From 246dbd1b346ac0f9574e118eb66fc34339d2d95a Mon Sep 17 00:00:00 2001
From: Bill <bill@bballou.com>
Date: Fri, 31 Oct 2025 14:54:10 -0400
Subject: [PATCH] refactor: remove unused web UI port configuration

The web UI (docs/index.html, portfolio.html) exists but is not served
in API mode. Removing the port configuration to eliminate confusion.

Changes:
- Remove port 8888 mapping from docker-compose.yml
- Remove WEB_HTTP_PORT from .env.example
- Update Dockerfile EXPOSE to only port 8080
- Update CHANGELOG.md to document removal

Technical details:
- Web UI static files remain in docs/ folder (legacy from batch mode)
- These were designed for JSONL file format, not the new SQLite database
- No web server was ever started in entrypoint.sh for API mode
- Port 8888 was exposed but nothing listened on it

Result:
- Cleaner configuration (1 fewer port mapping)
- Only REST API (8080) is exposed
- Eliminates user confusion about non-functional web UI
---
 .env.example       | 4 ----
 CHANGELOG.md       | 8 ++++++++
 Dockerfile         | 4 ++--
 docker-compose.yml | 2 --
 4 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/.env.example b/.env.example
index c995ebf..9b1af33 100644
--- a/.env.example
+++ b/.env.example
@@ -21,10 +21,6 @@ JINA_API_KEY=your_jina_key_here # https://jina.ai/
 # Used for Windmill integration and external API access
 API_PORT=8080
 
-# Web Interface Host Port (exposed on host machine)
-# Container always uses 8888 internally
-WEB_HTTP_PORT=8888
-
 # Agent Configuration
 AGENT_MAX_STEP=30
 
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 265dfa2..b2b1bed 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Removed
+- **Web UI Port** - Removed unused web dashboard port configuration
+  - Removed port 8888 from docker-compose.yml (not implemented)
+  - Removed WEB_HTTP_PORT from .env.example
+  - Removed port 8888 from Dockerfile EXPOSE
+  - Web UI static files exist in docs/ but are not served in API mode
+  - Only port 8080 (REST API) is now exposed
+
 ## [0.3.0] - 2025-10-31
 
 ### Added - API Service Transformation
diff --git a/Dockerfile b/Dockerfile
index 13119e9..5ff6da8 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -33,8 +33,8 @@ RUN mkdir -p data logs data/agent_data
 # Make entrypoint executable
 RUN chmod +x entrypoint.sh
 
-# Expose MCP service ports, API server, and web dashboard
-EXPOSE 8000 8001 8002 8003 8080 8888
+# Expose API server port (MCP services are internal only)
+EXPOSE 8080
 
 # Set Python to run unbuffered for real-time logs
 ENV PYTHONUNBUFFERED=1
diff --git a/docker-compose.yml b/docker-compose.yml
index 8f1d385..e0757a3 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -23,8 +23,6 @@ services:
     ports:
       # API server port (primary interface for external access)
       - "${API_PORT:-8080}:8080"
-      # Web dashboard
-      - "${WEB_HTTP_PORT:-8888}:8888"
     restart: unless-stopped  # Keep API server running
     healthcheck:
       test: ["CMD", "curl", "-f", "http://localhost:8080/health"]