tradingcodex 0.2.8__tar.gz → 0.2.9__tar.gz

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradingcodex
-Version: 0.2.8
+Version: 0.2.9
 Summary: Codex-native trading harness with a Django service plane and approved action boundary.
 Author: TradingCodex Authors
 License-Expression: Apache-2.0

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/docs/deployment.md

@@ -141,7 +141,7 @@ python3.11 -m venv /tmp/tcx-testpypi
 /tmp/tcx-testpypi/bin/pip install \
   --index-url https://test.pypi.org/simple/ \
   --extra-index-url https://pypi.org/simple/ \
-  tradingcodex==0.2.8
+  tradingcodex==0.2.9
 rm -rf /tmp/tcx-testpypi-smoke
 mkdir -p /tmp/tcx-testpypi-smoke
 cd /tmp/tcx-testpypi-smoke
@@ -170,7 +170,7 @@ After the PyPI workflow completes:
 
 ```bash
 python3.11 -m venv /tmp/tcx-pypi
-/tmp/tcx-pypi/bin/pip install tradingcodex==0.2.8
+/tmp/tcx-pypi/bin/pip install tradingcodex==0.2.9
 rm -rf /tmp/tcx-pypi-smoke
 mkdir -p /tmp/tcx-pypi-smoke
 cd /tmp/tcx-pypi-smoke
@@ -261,7 +261,9 @@ Use PEP 440 versions:
   spine improvements after `0.2.6`
 - `0.2.8` for artifact-supervisor loop concurrency and run-specific workflow
   loop state after `0.2.7`
-- later patch releases for compatible fixes after `0.2.8`
+- `0.2.9` for stale service replacement and web scroll-state fixes after
+  `0.2.8`
+- later patch releases for compatible fixes after `0.2.9`
 - pre-releases such as `0.3.0a1`, `0.3.0b1`, or `0.3.0rc1` when preparing
   the next minor contract
 

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/docs/generated-workspaces.md

@@ -305,6 +305,8 @@ service at `127.0.0.1:48267` while keeping MCP stdio stdout clean.
 If the port is already open, MCP startup verifies that the existing process is
 a TradingCodex service with the same package version and central DB path before
 using it.
+When the existing process is an older TradingCodex service backed by the same
+central DB, MCP autostart may stop it and launch the current package instead.
 
 The autostart path must be:
 
@@ -322,7 +324,7 @@ or performs package refresh on its own. The emitted context uses marker
 `allowed_next_actions`, and `routing_status`.
 
 `head-manager` uses `$tcx-server` for service/MCP doctor checks,
-`./tcx service status`, and `./tcx service ensure`. It tells the user that the
+`./tcx service status`, `./tcx service stop`, and `./tcx service ensure`. It tells the user that the
 local dashboard is available at `http://127.0.0.1:48267/` and opens it only
 when explicitly asked. If project MCP config was created or changed, the user
 must fully quit and restart Codex and start a new thread because Codex may not

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/docs/interfaces-and-surfaces.md

@@ -359,6 +359,7 @@ Top-level commands:
 - `tcx mcp stdio`
 - `tcx service runserver`
 - `tcx service ensure`
+- `tcx service stop`
 - `tcx service status [--json]`
 
 Generated workspace wrapper commands:

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "tradingcodex"
-version = "0.2.8"
+version = "0.2.9"
 description = "Codex-native trading harness with a Django service plane and approved action boundary."
 readme = "README.md"
 requires-python = ">=3.11,<3.15"

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tests/test_python_migration.py

@@ -137,7 +137,7 @@ def test_service_autostart_rejects_version_and_db_mismatch(monkeypatch) -> None:
     )
     with pytest.raises(RuntimeError, match="version mismatch") as version_mismatch:
         service_autostart._assert_compatible_service("127.0.0.1", 48267)
-    assert "Stop the older TradingCodex service" in str(version_mismatch.value)
+    assert "tcx service stop 127.0.0.1:48267" in str(version_mismatch.value)
 
     monkeypatch.setattr(service_autostart, "tradingcodex_db_path", lambda: Path("/tmp/current.sqlite3"))
     monkeypatch.setattr(
@@ -150,6 +150,38 @@ def test_service_autostart_rejects_version_and_db_mismatch(monkeypatch) -> None:
     assert "same central DB" in str(db_mismatch.value)
 
 
+def test_service_autostart_replaces_stale_same_db_service(monkeypatch, tmp_path: Path) -> None:
+    from tradingcodex_cli import service_autostart
+
+    current_db = tmp_path / "current.sqlite3"
+    state = {"open": True, "version": "0.0.1", "stopped": False, "started": False}
+
+    monkeypatch.setattr(service_autostart, "tradingcodex_db_path", lambda: current_db)
+    monkeypatch.setattr(service_autostart, "_tcp_open", lambda host, port: state["open"])
+    monkeypatch.setattr(
+        service_autostart,
+        "_service_health",
+        lambda host, port: {"service": "tradingcodex", "version": state["version"], "db_path": str(current_db)},
+    )
+
+    def stop(addr: str, timeout: float = 5.0) -> bool:
+        state["stopped"] = True
+        state["open"] = False
+        return True
+
+    def start(*args) -> None:
+        state["started"] = True
+        state["open"] = True
+        state["version"] = TRADINGCODEX_VERSION
+
+    monkeypatch.setattr(service_autostart, "stop_service", stop)
+    monkeypatch.setattr(service_autostart, "_start_service", start)
+
+    assert service_autostart.ensure_service_up(tmp_path, timeout=0.05) is True
+    assert state["stopped"] is True
+    assert state["started"] is True
+
+
 def test_service_status_reports_actionable_state(monkeypatch) -> None:
     from tradingcodex_cli import service_autostart
 
@@ -171,6 +203,7 @@ def test_service_status_reports_actionable_state(monkeypatch) -> None:
     assert mismatch["compatible"] is False
     assert mismatch["issue"] == "version_mismatch"
     assert mismatch["package_version"] == TRADINGCODEX_VERSION
+    assert "tcx service stop 127.0.0.1:48267" in mismatch["next_action"]
 
     monkeypatch.setattr(service_autostart, "tradingcodex_db_path", lambda: Path("/tmp/current.sqlite3"))
     monkeypatch.setattr(
@@ -213,9 +246,11 @@ def test_service_ensure_uses_autostart_helper(monkeypatch, tmp_path: Path, capsy
     from tradingcodex_cli import service_autostart
 
     calls: list[tuple[Path, str]] = []
+    stopped: list[str] = []
     monkeypatch.chdir(tmp_path)
     monkeypatch.delenv("TRADINGCODEX_WORKSPACE_ROOT", raising=False)
     monkeypatch.setattr(service_autostart, "ensure_service_up", lambda root, addr: calls.append((root, addr)) or True)
+    monkeypatch.setattr(service_autostart, "stop_service", lambda addr: stopped.append(addr) or True)
 
     cli_main.service(["ensure"])
 
@@ -224,6 +259,11 @@ def test_service_ensure_uses_autostart_helper(monkeypatch, tmp_path: Path, capsy
     assert "TradingCodex service started at http://127.0.0.1:48267/" in output
     assert "Health: http://127.0.0.1:48267/api/health" in output
 
+    cli_main.service(["stop"])
+
+    assert stopped == ["127.0.0.1:48267"]
+    assert "TradingCodex service stopped at http://127.0.0.1:48267/" in capsys.readouterr().out
+
 
 def test_service_status_cli_supports_plain_and_json(monkeypatch, tmp_path: Path, capsys) -> None:
     from tradingcodex_cli import __main__ as cli_main

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tradingcodex
-Version: 0.2.8
+Version: 0.2.9
 Summary: Codex-native trading harness with a Django service plane and approved action boundary.
 Author: TradingCodex Authors
 License-Expression: Apache-2.0

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_cli/__main__.py

@@ -119,6 +119,7 @@ Usage:
   tcx mcp stdio|external
   tcx service runserver [addrport] [django runserver args]
   tcx service ensure [addrport]
+  tcx service stop [addrport]
   tcx service status [addrport] [--json]
 """)
 

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_cli/commands/bootstrap.py

@@ -166,8 +166,15 @@ def service(argv: list[str]) -> None:
         print(f"TradingCodex service {'started' if started else 'ready'} at {dashboard_url}")
         print(f"Health: {dashboard_url.rstrip('/')}/api/health")
         return
+    if sub == "stop":
+        from tradingcodex_cli.service_autostart import service_http_url, stop_service
+
+        addr = argv[1] if len(argv) > 1 else DEFAULT_SERVICE_ADDR
+        stopped = stop_service(addr)
+        print(f"TradingCodex service {'stopped' if stopped else 'not running'} at {service_http_url(addr)}")
+        return
     if sub != "runserver":
-        raise ValueError(f"Usage: {PROGRAM_NAME} service runserver [addrport] [django runserver args]\n       {PROGRAM_NAME} service ensure [addrport]\n       {PROGRAM_NAME} service status [addrport] [--json]")
+        raise ValueError(f"Usage: {PROGRAM_NAME} service runserver [addrport] [django runserver args]\n       {PROGRAM_NAME} service ensure [addrport]\n       {PROGRAM_NAME} service stop [addrport]\n       {PROGRAM_NAME} service status [addrport] [--json]")
     from django.core.management import execute_from_command_line
     from tradingcodex_cli.service_autostart import compatible_service_running, service_http_url
 

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_cli/service_autostart.py

@@ -2,6 +2,7 @@ from __future__ import annotations
 
 import os
 import json
+import signal
 import socket
 import subprocess
 import sys
@@ -28,13 +29,13 @@ def maybe_autostart_service(workspace_root: Path, source_root: Path | None = Non
 
 def ensure_service_up(workspace_root: Path, addr: str = DEFAULT_SERVICE_ADDR, source_root: Path | None = None, timeout: float = 8.0) -> bool:
     host, port = _parse_addr(addr)
-    if _tcp_open(host, port):
-        _assert_compatible_service(host, port)
+    if _tcp_open(host, port) and _compatible_service(host, port):
         return False
     with tradingcodex_file_lock(f"service-{host}-{port}"):
-        if _tcp_open(host, port):
-            _assert_compatible_service(host, port)
+        if _tcp_open(host, port) and _compatible_service(host, port):
             return False
+        if _tcp_open(host, port):
+            _replace_stale_tradingcodex_service_or_raise(host, port, timeout=timeout)
         _start_service(workspace_root, addr, source_root)
         deadline = time.monotonic() + timeout
         while time.monotonic() < deadline:
@@ -53,6 +54,30 @@ def compatible_service_running(addr: str = DEFAULT_SERVICE_ADDR) -> bool:
     return True
 
 
+def stop_service(addr: str = DEFAULT_SERVICE_ADDR, timeout: float = 5.0) -> bool:
+    host, port = _parse_addr(addr)
+    normalized_addr = f"{host}:{port}"
+    if not _tcp_open(host, port):
+        return False
+    health = _service_health(host, port)
+    if not health or health.get("service") != "tradingcodex":
+        raise RuntimeError(f"{normalized_addr} is not a TradingCodex service; stop it outside tcx or choose a free service address.")
+    pids = _service_pids(host, port, health)
+    if not pids:
+        raise RuntimeError(f"Could not find the TradingCodex service process on {normalized_addr}; stop the process using port {port} and retry.")
+    for pid in pids:
+        try:
+            os.kill(pid, signal.SIGTERM)
+        except ProcessLookupError:
+            pass
+    deadline = time.monotonic() + timeout
+    while time.monotonic() < deadline:
+        if not _tcp_open(host, port):
+            return True
+        time.sleep(0.2)
+    raise RuntimeError(f"Timed out stopping TradingCodex service on {normalized_addr}.")
+
+
 def service_status(addr: str = DEFAULT_SERVICE_ADDR) -> dict:
     host, port = _parse_addr(addr)
     normalized_addr = f"{host}:{port}"
@@ -86,7 +111,7 @@ def service_status(addr: str = DEFAULT_SERVICE_ADDR) -> dict:
     })
     if status["version"] != TRADINGCODEX_VERSION:
         status["issue"] = "version_mismatch"
-        status["next_action"] = "Stop the older TradingCodex service or choose a free service address, then restart Codex if MCP uses the default address."
+        status["next_action"] = f"Run `tcx service stop {normalized_addr}` and then restart Codex if MCP uses the default address."
         return status
     if status["db_path"] and status["db_path"] != current_db:
         status["issue"] = "db_mismatch"
@@ -149,6 +174,19 @@ def _compatible_service(host: str, port: int) -> bool:
         return False
 
 
+def _replace_stale_tradingcodex_service_or_raise(host: str, port: int, *, timeout: float) -> None:
+    health = _service_health(host, port)
+    service_db = str(health.get("db_path") or "")
+    if (
+        health.get("service") == "tradingcodex"
+        and health.get("version") != TRADINGCODEX_VERSION
+        and (not service_db or service_db == str(tradingcodex_db_path()))
+    ):
+        stop_service(f"{host}:{port}", timeout=max(1.0, min(timeout, 5.0)))
+        return
+    _assert_compatible_service(host, port)
+
+
 def _assert_compatible_service(host: str, port: int) -> None:
     health = _service_health(host, port)
     addr = f"{host}:{port}"
@@ -160,8 +198,8 @@ def _assert_compatible_service(host: str, port: int) -> None:
     if health.get("version") != TRADINGCODEX_VERSION:
         raise RuntimeError(
             f"TradingCodex service version mismatch: service={health.get('version')} package={TRADINGCODEX_VERSION}. "
-            f"Stop the older TradingCodex service at {service_http_url(addr)} "
-            "or choose a free service address, then fully restart Codex if project MCP uses the default address."
+            f"Run `tcx service stop {addr}` or choose a free service address, "
+            "then fully restart Codex if project MCP uses the default address."
         )
     service_db = str(health.get("db_path") or "")
     current_db = str(tradingcodex_db_path())
@@ -182,3 +220,20 @@ def _service_health(host: str, port: int) -> dict:
         return data if isinstance(data, dict) else {}
     except Exception:
         return {}
+
+
+def _service_pids(host: str, port: int, health: dict) -> list[int]:
+    pid = health.get("pid")
+    pids = {int(pid)} if isinstance(pid, int) and pid > 0 else set()
+    try:
+        result = subprocess.run(
+            ["lsof", f"-tiTCP:{port}", "-sTCP:LISTEN"],
+            text=True,
+            capture_output=True,
+            timeout=1,
+            check=False,
+        )
+        pids.update(int(line) for line in result.stdout.splitlines() if line.strip().isdigit())
+    except Exception:
+        pass
+    return sorted(pids)

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_service/api.py

@@ -265,6 +265,7 @@ def health(request):
         "service": "tradingcodex",
         "version": __version__,
         "db_path": str(tradingcodex_db_path()),
+        "pid": os.getpid(),
         "central_local_service": True,
         "process_scope": os.environ.get("TRADINGCODEX_MCP_SCOPE", "local-service"),
     }

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_service/static/tradingcodex_web/app.css

@@ -631,6 +631,7 @@ pre {
 .tc-main-shell {
   min-width: 0;
   overflow: auto;
+  overflow-anchor: none;
 }
 
 .tc-content {

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_service/static/tradingcodex_web/app.js

@@ -179,8 +179,12 @@
   });
   window.addEventListener("pagehide", saveScroll);
   window.addEventListener("beforeunload", saveScroll);
+  document.addEventListener("htmx:beforeRequest", saveScroll);
+  document.addEventListener("htmx:afterSettle", () => {
+    if (!window.location.hash) restoreScroll();
+  });
   document.addEventListener("click", (event) => {
-    const link = event.target.closest(statefulLinks);
+    const link = event.target instanceof Element ? event.target.closest(statefulLinks) : null;
     if (link) saveScroll();
   });
 })();

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/tradingcodex_service/templates/web/base.html

@@ -6,9 +6,9 @@
   <meta name="viewport" content="width=device-width, initial-scale=1">
   <title>{% block title %}tcx{% endblock %}</title>
   <link rel="icon" type="image/svg+xml" href="{% static 'tradingcodex_admin/favicon.svg' %}">
-  <link rel="stylesheet" href="{% static 'tradingcodex_web/app.css' %}?v=tcx-base-9">
+  <link rel="stylesheet" href="{% static 'tradingcodex_web/app.css' %}?v=tcx-base-10">
   <script src="{% static 'vendor/htmx/htmx.min.js' %}" defer></script>
-  <script src="{% static 'tradingcodex_web/app.js' %}?v=tcx-web-3" defer></script>
+  <script src="{% static 'tradingcodex_web/app.js' %}?v=tcx-web-4" defer></script>
 </head>
 <body class="tc-web-body">
   <a class="tc-skip-link" href="#main-content">Skip to content</a>

tradingcodex-0.2.9/tradingcodex_service/version.py

@@ -0,0 +1 @@
+TRADINGCODEX_VERSION = "0.2.9"

{tradingcodex-0.2.8 → tradingcodex-0.2.9}/workspace_templates/modules/repo-skills/files/.agents/skills/tcx-server/SKILL.md

@@ -14,7 +14,7 @@ Use this skill for operate-plane TradingCodex status checks, service recovery, d
 3. Use `./tcx mode status` and `./tcx update status --json` for mode/update posture.
 4. Use read-only connector commands and MCP tools for broker profile, capability, instrument constraints, sync state, and order status.
 5. If update or build work is requested but build is not enabled, explain that Codex full access plus `tcx mode set build --reason <reason>` is required, or give the terminal command.
-6. If startup context reports `service_issue=version_mismatch`, `db_mismatch`, or `port_occupied`, report the issue with service/package versions or DB paths when present, and give the recorded next action before opening or recommending the dashboard.
+6. If startup context reports `service_issue=version_mismatch`, `db_mismatch`, or `port_occupied`, report the issue with service/package versions or DB paths when present, and give the recorded next action before opening or recommending the dashboard. For a stale same-DB TradingCodex service, prefer `./tcx service stop` and Codex restart so MCP autostart can launch the current package.
 
 ## Hard Stops
 

tradingcodex-0.2.8/tradingcodex_service/version.py

@@ -1 +0,0 @@
-TRADINGCODEX_VERSION = "0.2.8"