claude-finops 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-finops",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "Local FinOps dashboard for Claude Code: what you used, what it cost, why it cost that much, and what to change. Reads your own transcripts, no API key, no data leaves the machine.",
   "bin": {
     "claude-finops": "bin/claude-finops.js"

package/run.py

@@ -17,6 +17,7 @@ import os
 import signal
 import subprocess
 import sys
+import time
 import zipfile
 
 ROOT = os.path.dirname(os.path.abspath(__file__))
@@ -44,7 +45,86 @@ def _alive(pid):
         return False
 
 
-def stop():
+def _free_port(start):
+    """The first free port at or above start+1, for the suggestion we print."""
+    import socket
+    for p in range(start + 1, start + 40):
+        with socket.socket() as sk:
+            try:
+                sk.bind(("127.0.0.1", p))
+                return p
+            except OSError:
+                continue
+    return start + 1
+
+
+def _port_owner(port):
+    """PID listening on 127.0.0.1:<port>, or None.
+
+    The pidfile is not enough on its own: a dashboard started from a different
+    copy (a global npm install alongside a checkout) writes its own, and one
+    that was killed hard leaves a stale file behind. Asking the OS who actually
+    holds the port is the only answer that is always true.
+    """
+    try:
+        if IS_WIN:
+            out = subprocess.run(["netstat", "-ano", "-p", "TCP"],
+                                 capture_output=True, text=True, timeout=10).stdout
+            for line in out.splitlines():
+                f = line.split()
+                if len(f) >= 5 and f[1].endswith(f":{port}") and f[3] == "LISTENING":
+                    return int(f[4])
+            return None
+        out = subprocess.run(["lsof", "-tnP", f"-iTCP:{port}", "-sTCP:LISTEN"],
+                             capture_output=True, text=True, timeout=10).stdout
+        return int(out.split()[0]) if out.split() else None
+    except (OSError, ValueError, subprocess.SubprocessError):
+        return None
+
+
+def _is_ours(pid):
+    """True only if that PID is a claude-finops server.
+
+    Whatever is on the port may be someone else's service. We stop our own
+    dashboard without asking; anything else we report and leave alone.
+    """
+    try:
+        if IS_WIN:
+            out = subprocess.run(
+                ["wmic", "process", "where", f"ProcessId={pid}", "get", "CommandLine"],
+                capture_output=True, text=True, timeout=10).stdout
+        else:
+            out = subprocess.run(["ps", "-o", "command=", "-p", str(pid)],
+                                 capture_output=True, text=True, timeout=10).stdout
+    except (OSError, subprocess.SubprocessError):
+        return False
+    return "finops.api" in out or "claude-finops" in out
+
+
+def _kill(pid):
+    if IS_WIN:
+        subprocess.run(["taskkill", "/PID", str(pid), "/F"], capture_output=True)
+    else:
+        os.kill(pid, signal.SIGTERM)
+
+
+def stop(port=None):
+    """Stop a running dashboard. Returns True if we stopped one."""
+    if stop_pidfile():
+        return True
+    # No usable pidfile: fall back to whoever owns the port, but only if it is
+    # ours to stop.
+    pid = _port_owner(port or os.environ.get("PORT", "8787"))
+    if pid and _is_ours(pid):
+        try:
+            _kill(pid)
+            return True
+        except OSError:
+            return False
+    return False
+
+
+def stop_pidfile():
     for pidfile in (PIDFILE, LEGACY_PIDFILE):   # LEGACY_: a server started before the move
         try:
             with open(pidfile) as fh:
@@ -55,10 +135,7 @@ def stop():
     else:
         return False
     if _alive(pid):
-        if IS_WIN:
-            subprocess.run(["taskkill", "/PID", str(pid), "/F"], capture_output=True)
-        else:
-            os.kill(pid, signal.SIGTERM)
+        _kill(pid)
         os.remove(pidfile)
         return True
     os.remove(pidfile)
@@ -105,6 +182,11 @@ Environment:
 """
 
 
+def _version():
+    from finops.update import _installed
+    return _installed()
+
+
 def version():
     """Which copy is this, and is it current?
 
@@ -222,13 +304,45 @@ def main():
         return
     if "--share" in args:
         return share()
-    stop()   # replace a previous detached server
+    # A flag we do not know used to fall straight through and start the
+    # dashboard, so a typo (or a flag from a newer release than the one you
+    # have installed) looked like the command silently doing nothing.
+    known = {"--rebuild", "--detach", "--foreground"}
+    unknown = [a for a in args if a.startswith("-") and a not in known]
+    if unknown:
+        print(f"Unknown option: {unknown[0]}")
+        print(f"This is claude-finops {_version() or 'unknown'}; "
+              f"run --help to see what it supports.")
+        sys.exit(2)
     port = os.environ.get("PORT", "8787")
+    # Decide from who actually holds *this* port, not from the pidfile alone:
+    # the pidfile may describe a dashboard serving some other port entirely.
+    owner = _port_owner(port)
+    if owner is None or _is_ours(owner):
+        if owner is not None:
+            # flush=True: execv below replaces this process without flushing,
+            # so an unflushed line would simply never reach your terminal.
+            print(f"Restarting the dashboard already running on {port} …", flush=True)
+        stop(port)                                # replace it, whoever started it
+        for _ in range(20):                       # give the socket a moment to clear
+            if _port_owner(port) is None:
+                break
+            time.sleep(0.25)
+    # Still occupied means it is not ours: say so instead of dying in a traceback.
+    if (busy := _port_owner(port)) is not None:
+        free = _free_port(int(port))
+        print(f"Port {port} is already in use by PID {busy}, and it is not a "
+              f"claude-finops dashboard.")
+        print(f"Start on another port:  PORT={free} claude-finops")
+        print(f"Or find out what is holding it:  "
+              + (f"netstat -ano -p TCP | findstr :{port}" if IS_WIN
+                 else f"lsof -iTCP:{port} -sTCP:LISTEN"))
+        sys.exit(1)
     source = os.environ.get("CLAUDE_PROJECTS", os.path.join(os.path.expanduser("~"), ".claude", "projects"))
     if "--rebuild" in args or not os.path.exists(DB_PATH):
         if not os.path.isdir(source):
             sys.exit(f"No Claude Code transcripts at {source}. Use Claude Code once, or set CLAUDE_PROJECTS.")
-        print(f"Building warehouse from {source} …")
+        print(f"Building warehouse from {source} …", flush=True)
         subprocess.run(_python() + ["-m", "finops.etl", source], check=True)
     rest = [a for a in args if a != "--rebuild"]
     if IS_WIN:

package/web/app.js

@@ -485,7 +485,10 @@ async function openSession(id) {
 /* ---------- charts added to table-first pages ---------- */
 // Inserts a chart card after the page's KPI row (or at the top) and draws into it.
 function addChart(page, title, draw, opts = {}) {
-  const c = h(card(title, '<div class="cchart"></div>', opts));
+  // .chart carries every chart style (text fill, gridlines, axes); .cchart is
+  // just the hook this helper looks up. Missing .chart left SVG labels at the
+  // browser default fill — black text, invisible on the dark theme.
+  const c = h(card(title, '<div class="chart cchart"></div>', opts));
   const anchor = opts.after ? page.querySelector(opts.after) : page.querySelector(':scope > .grid');
   if (anchor) anchor.after(c); else page.prepend(c);
   draw(c.querySelector('.cchart'));

package/web/styles.css

@@ -76,15 +76,19 @@ button, input, select { font: inherit; color: inherit; }
 
 /* ---------- shell ---------- */
 .app { display: grid; grid-template-columns: 216px 1fr; min-height: 100vh; }
+/* The sidebar itself no longer scrolls: the nav does. A sticky footer inside a
+   scrolling column still has the rest of the list running underneath it, which
+   showed as nav items bleeding through the account block. */
 .sidebar { background: var(--surface); border-right: 1px solid var(--border);
-           padding: 14px 0; position: sticky; top:0; height:100vh; overflow-y:auto;
+           padding: 14px 0; position: sticky; top:0; height:100vh; overflow:hidden;
            display:flex; flex-direction:column; }
 .brand { padding: 0 16px 14px; border-bottom: 1px solid var(--border); margin-bottom: 10px; }
 .brand .mark { display:flex; align-items:center; gap:8px; font-weight:660; font-size:14px;
                letter-spacing:-0.01em; }
 .brand .dot { width:9px; height:9px; border-radius:3px; background:var(--s2); flex:none; }
 .brand .sub { font-size:10.5px; color:var(--muted); margin-top:3px; letter-spacing:.02em; }
-.nav { display:flex; flex-direction:column; gap:1px; padding: 0 8px; }
+.nav { display:flex; flex-direction:column; gap:1px; padding: 0 8px;
+       flex:1 1 auto; min-height:0; overflow-y:auto; }
 .nav .group { font-size:9.5px; text-transform:uppercase; letter-spacing:.09em; color:var(--muted);
               padding: 13px 8px 5px; font-weight:600; }
 .nav a { display:flex; align-items:center; gap:8px; padding:6px 9px; border-radius:7px;
@@ -401,8 +405,7 @@ table.tbl .sub { font-size:10.5px; color:var(--muted); }
    so every screenshot carries the name of the account it came from. */
 /* Sticky, because the nav is taller than the viewport and an identity you have
    to scroll to find is one nobody checks. */
-.who { margin-top:auto; position:sticky; bottom:0; z-index:2;
-       display:flex; align-items:center; gap:9px;
+.who { flex:none; display:flex; align-items:center; gap:9px;
        padding:12px 16px; background:var(--surface);
        border-top:1px solid var(--border); }
 .who:empty { display:none; }