Bill/AI-Trader
1
0
Fork 0
mirror of https://github.com/Xe138/AI-Trader.git synced 2026-04-12 13:37:24 -04:00
Code Issues Packages Projects Releases Wiki Activity

refactor: remove unused web UI port configuration

Browse Source 
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
This commit is contained in:
Bill Ballou
2025-10-31 14:54:10 -04:00
parent 9539d63103
commit 246dbd1b34
4 changed files with 10 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
.env.example
View File

@@ -21,10 +21,6 @@ JINA_API_KEY=your_jina_key_here # https://jina.ai/
# Used for Windmill integration and external API access # Used for Windmill integration and external API access
API_PORT=8080 API_PORT=8080
# Web Interface Host Port (exposed on host machine)
# Container always uses 8888 internally
WEB_HTTP_PORT=8888
# Agent Configuration # Agent Configuration
AGENT_MAX_STEP=30 AGENT_MAX_STEP=30

8
CHANGELOG.md
View File

@@ -7,6 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
## [Unreleased] ## [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 ## [0.3.0] - 2025-10-31
### Added - API Service Transformation ### Added - API Service Transformation

4
Dockerfile
View File

@@ -33,8 +33,8 @@ RUN mkdir -p data logs data/agent_data
# Make entrypoint executable # Make entrypoint executable
RUN chmod +x entrypoint.sh RUN chmod +x entrypoint.sh
# Expose MCP service ports, API server, and web dashboard # Expose API server port (MCP services are internal only)
EXPOSE 8000 8001 8002 8003 8080 8888 EXPOSE 8080
# Set Python to run unbuffered for real-time logs # Set Python to run unbuffered for real-time logs
ENV PYTHONUNBUFFERED=1 ENV PYTHONUNBUFFERED=1

2
docker-compose.yml
View File

@@ -23,8 +23,6 @@ services:
ports: ports:
# API server port (primary interface for external access) # API server port (primary interface for external access)
- "${API_PORT:-8080}:8080" - "${API_PORT:-8080}:8080"
# Web dashboard
- "${WEB_HTTP_PORT:-8888}:8888"
restart: unless-stopped # Keep API server running restart: unless-stopped # Keep API server running
healthcheck: healthcheck:
test: ["CMD", "curl", "-f", "http://localhost:8080/health"] test: ["CMD", "curl", "-f", "http://localhost:8080/health"]