agent-control-plane 0.1.14 → 0.1.16

package/tools/dashboard/app.js

@@ -1,9 +1,56 @@
 const refreshButton = document.querySelector("#refresh-button");
+const themeToggleButton = document.querySelector("#theme-toggle");
 const generatedAtNode = document.querySelector("#generated-at");
 const overviewNode = document.querySelector("#overview");
 const profilesNode = document.querySelector("#profiles");
 const seenAlertIds = new Set();
 let notificationPermissionRequested = false;
+const THEME_STORAGE_KEY = "acp-dashboard-theme";
+
+function systemPrefersDark() {
+  return typeof window.matchMedia === "function" && window.matchMedia("(prefers-color-scheme: dark)").matches;
+}
+
+function currentThemePreference() {
+  try {
+    const stored = window.localStorage.getItem(THEME_STORAGE_KEY);
+    if (stored === "light" || stored === "dark") return stored;
+  } catch (_error) {
+    // Ignore storage access issues and fall back to system preference.
+  }
+  return systemPrefersDark() ? "dark" : "light";
+}
+
+function updateThemeToggleLabel(theme) {
+  if (!themeToggleButton) return;
+  const nextTheme = theme === "dark" ? "light" : "dark";
+  const label = nextTheme === "dark" ? "Dark mode" : "Light mode";
+  themeToggleButton.textContent = label;
+  themeToggleButton.setAttribute("aria-label", `Switch to ${label.toLowerCase()}`);
+}
+
+function applyTheme(theme) {
+  document.documentElement.dataset.theme = theme;
+  updateThemeToggleLabel(theme);
+}
+
+function persistTheme(theme) {
+  try {
+    window.localStorage.setItem(THEME_STORAGE_KEY, theme);
+  } catch (_error) {
+    // Ignore storage access issues.
+  }
+}
+
+function initializeTheme() {
+  applyTheme(currentThemePreference());
+  if (!themeToggleButton) return;
+  themeToggleButton.addEventListener("click", () => {
+    const nextTheme = document.documentElement.dataset.theme === "dark" ? "light" : "dark";
+    applyTheme(nextTheme);
+    persistTheme(nextTheme);
+  });
+}
 
 function relativeTime(input) {
   if (!input) return "n/a";
@@ -314,6 +361,16 @@ function renderProfile(profile) {
     "No pending leased issues.",
   );
 
+  const claimsTable = renderTable(
+    [
+      { label: "Issue", key: "issue_id" },
+      { label: "Session", render: (row) => row.session ? `<div class="mono">${row.session}</div>` : "n/a" },
+      { label: "Updated", render: (row) => row.updated_at ? `${relativeTime(row.updated_at)}<div class="muted">${row.updated_at}</div>` : "n/a" },
+    ],
+    profile.issue_queue.claims || [],
+    "No claimed issues.",
+  );
+
   const codexRotationPanel =
     profile.coding_worker === "codex"
       ? `
@@ -384,6 +441,10 @@ function renderProfile(profile) {
           <h3>Pending Issue Queue</h3>
           ${queueTable}
         </section>
+        <section class="panel half">
+          <h3>Claimed Issues</h3>
+          ${claimsTable}
+        </section>
       </section>
     </article>
   `;
@@ -442,6 +503,7 @@ refreshButton.addEventListener("click", () => {
   void loadSnapshot();
 });
 
+initializeTheme();
 void loadSnapshot();
 window.setInterval(() => {
   void loadSnapshot();

package/tools/dashboard/dashboard_snapshot.py

@@ -200,6 +200,11 @@ GITHUB_RATE_LIMIT_PATTERNS = [
     ),
 ]
 
+WORKER_PREFLIGHT_NETWORK_BLOCKED_PATTERN = re.compile(
+    r"Blocked on external network access.*?What I ran:\s*-\s*`(?P<command>[^`]+)`.*?Exact failure:\s*`(?P<failure>[^`]+)`",
+    re.IGNORECASE | re.DOTALL,
+)
+
 
 def summarize_whitespace(text: str) -> str:
     return re.sub(r"\s+", " ", text).strip()
@@ -242,6 +247,52 @@ def extract_github_rate_limit_alert(run_dir: Path, run: dict[str, Any]) -> dict[
     return None
 
 
+def extract_worker_preflight_network_blocked_alert(run_dir: Path, run: dict[str, Any]) -> dict[str, Any] | None:
+    candidate_files = [
+        run_dir / "issue-comment.md",
+        run_dir / "pr-comment.md",
+    ]
+    for path in candidate_files:
+        text = read_tail_text(path)
+        if not text:
+            continue
+        match = WORKER_PREFLIGHT_NETWORK_BLOCKED_PATTERN.search(text)
+        if not match:
+            continue
+        command = summarize_whitespace(match.group("command"))
+        failure = summarize_whitespace(match.group("failure"))
+        message = f"Worker preflight `{command or 'unknown command'}` failed before implementation."
+        if failure:
+            message = f"{message} {failure}"
+        message = f"{message} Verify from the host if the same command succeeds; worker and host environment can diverge."
+        return {
+            "id": f"worker-preflight-network-blocked:{run.get('session', '')}:{command}:{failure}",
+            "kind": "worker-preflight-network-blocked",
+            "severity": "warn",
+            "title": "Worker preflight blocked by network",
+            "message": message,
+            "session": run.get("session", ""),
+            "task_kind": run.get("task_kind", ""),
+            "task_id": run.get("task_id", ""),
+            "reset_at": "",
+            "updated_at": run.get("updated_at", "") or file_mtime_iso(path),
+            "source_file": str(path),
+        }
+    return None
+
+
+def extract_run_alerts(run_dir: Path, run: dict[str, Any]) -> list[dict[str, Any]]:
+    alerts: list[dict[str, Any]] = []
+    for extractor in (
+        extract_github_rate_limit_alert,
+        extract_worker_preflight_network_blocked_alert,
+    ):
+        alert = extractor(run_dir, run)
+        if alert:
+            alerts.append(alert)
+    return alerts
+
+
 def collect_runs(runs_root: Path) -> list[dict[str, Any]]:
     if not runs_root.is_dir():
         return []
@@ -296,8 +347,7 @@ def collect_runs(runs_root: Path) -> list[dict[str, Any]]:
             "provider_pool_name": run_env.get("ACTIVE_PROVIDER_POOL_NAME", ""),
             "run_dir": str(run_dir),
         }
-        alert = extract_github_rate_limit_alert(run_dir, item)
-        item["alerts"] = [alert] if alert else []
+        item["alerts"] = extract_run_alerts(run_dir, item)
         runs.append(item)
     return runs
 
@@ -351,8 +401,7 @@ def collect_recent_history(history_root: Path, limit: int = 8) -> list[dict[str,
             "run_dir": str(run_dir),
             "archived": True,
         }
-        alert = extract_github_rate_limit_alert(run_dir, item)
-        item["alerts"] = [alert] if alert else []
+        item["alerts"] = extract_run_alerts(run_dir, item)
         items.append(item)
         seen_sessions.add(session)
         if len(items) >= limit:

package/tools/dashboard/index.html

@@ -3,6 +3,7 @@
   <head>
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1" />
+    <meta name="color-scheme" content="light dark" />
     <title>ACP Worker Dashboard</title>
     <link rel="stylesheet" href="./styles.css" />
   </head>
@@ -20,7 +21,10 @@
           </p>
         </div>
         <div class="hero-actions">
-          <button id="refresh-button" type="button">Refresh now</button>
+          <div class="hero-controls">
+            <button id="theme-toggle" type="button" aria-label="Toggle dark mode">Dark mode</button>
+            <button id="refresh-button" type="button">Refresh now</button>
+          </div>
           <div class="meta">
             <div>Auto refresh: <strong>5s</strong></div>
             <div id="generated-at">Loading snapshot...</div>

package/tools/dashboard/styles.css

@@ -12,6 +12,56 @@
   --danger: #b42318;
   --danger-soft: #fdd8d2;
   --shadow: 0 18px 50px rgba(25, 33, 38, 0.08);
+  --button-bg: var(--ink);
+  --button-ink: #ffffff;
+  --button-hover: #0f1720;
+  --hero-bg: rgba(255, 253, 247, 0.92);
+  --profile-bg: rgba(255, 253, 247, 0.94);
+  --body-gradient-top: rgba(15, 118, 110, 0.08);
+  --body-gradient-bottom: #faf7ef;
+  --theme-toggle-bg: var(--panel);
+  --theme-toggle-ink: var(--ink);
+  --theme-toggle-line: var(--line);
+  --theme-toggle-hover: var(--panel-strong);
+  --reported-soft: #dbeafe;
+  --reported-ink: #1d4ed8;
+  --implemented-soft: #dcfce7;
+  --implemented-ink: #166534;
+  --blocked-soft: #fef3c7;
+  --blocked-ink: #92400e;
+}
+
+:root[data-theme="dark"] {
+  --bg: #0d1418;
+  --panel: #142026;
+  --panel-strong: #1a2a31;
+  --ink: #ebf1f3;
+  --muted: #9ab0bb;
+  --line: #2a3c44;
+  --accent: #5ad4c7;
+  --accent-soft: #183d3a;
+  --warn: #f4c35f;
+  --warn-soft: #4d3a12;
+  --danger: #ff8a80;
+  --danger-soft: #4a2220;
+  --shadow: 0 24px 60px rgba(0, 0, 0, 0.32);
+  --button-bg: #ebf1f3;
+  --button-ink: #0d1418;
+  --button-hover: #d7e2e6;
+  --hero-bg: rgba(20, 32, 38, 0.92);
+  --profile-bg: rgba(20, 32, 38, 0.94);
+  --body-gradient-top: rgba(90, 212, 199, 0.12);
+  --body-gradient-bottom: #10181d;
+  --theme-toggle-bg: #1d2d35;
+  --theme-toggle-ink: #ebf1f3;
+  --theme-toggle-line: #35505b;
+  --theme-toggle-hover: #23363f;
+  --reported-soft: #1b3552;
+  --reported-ink: #9fc8ff;
+  --implemented-soft: #173a2b;
+  --implemented-ink: #85ddb1;
+  --blocked-soft: #4a3711;
+  --blocked-ink: #f4c35f;
 }
 
 * {
@@ -22,8 +72,8 @@ body {
   margin: 0;
   font-family: "Iowan Old Style", "Palatino Linotype", "Book Antiqua", serif;
   background:
-    radial-gradient(circle at top right, rgba(15, 118, 110, 0.08), transparent 34%),
-    linear-gradient(180deg, #faf7ef 0%, var(--bg) 100%);
+    radial-gradient(circle at top right, var(--body-gradient-top), transparent 34%),
+    linear-gradient(180deg, var(--body-gradient-bottom) 0%, var(--bg) 100%);
   color: var(--ink);
 }
 
@@ -41,7 +91,7 @@ body {
   padding: 24px;
   border: 1px solid var(--line);
   border-radius: 28px;
-  background: rgba(255, 253, 247, 0.92);
+  background: var(--hero-bg);
   box-shadow: var(--shadow);
 }
 
@@ -75,19 +125,42 @@ body {
   align-items: flex-end;
 }
 
+.hero-controls {
+  display: flex;
+  gap: 10px;
+  align-items: center;
+  flex-wrap: wrap;
+  justify-content: flex-end;
+}
+
 button {
   appearance: none;
   border: 0;
   border-radius: 999px;
   padding: 12px 18px;
   font: inherit;
-  background: var(--ink);
-  color: white;
+  background: var(--button-bg);
+  color: var(--button-ink);
   cursor: pointer;
+  transition: background 140ms ease, color 140ms ease, border-color 140ms ease, transform 140ms ease;
 }
 
 button:hover {
-  background: #0f1720;
+  background: var(--button-hover);
+}
+
+button:active {
+  transform: translateY(1px);
+}
+
+#theme-toggle {
+  background: var(--theme-toggle-bg);
+  color: var(--theme-toggle-ink);
+  border: 1px solid var(--theme-toggle-line);
+}
+
+#theme-toggle:hover {
+  background: var(--theme-toggle-hover);
 }
 
 .meta {
@@ -135,7 +208,7 @@ button:hover {
   padding: 22px;
   border-radius: 28px;
   border: 1px solid var(--line);
-  background: rgba(255, 253, 247, 0.94);
+  background: var(--profile-bg);
   box-shadow: var(--shadow);
 }
 
@@ -183,12 +256,12 @@ button:hover {
 
 .badge.good {
   background: var(--accent-soft);
-  color: #0d5a54;
+  color: var(--accent);
 }
 
 .badge.warn {
   background: var(--warn-soft);
-  color: #855500;
+  color: var(--warn);
 }
 
 .badge.danger {
@@ -243,8 +316,8 @@ button:hover {
 }
 
 .alert-card.warn {
-  border-color: #e7c76d;
-  background: #fff6dd;
+  border-color: color-mix(in srgb, var(--warn) 45%, var(--line));
+  background: color-mix(in srgb, var(--warn-soft) 82%, var(--panel) 18%);
 }
 
 .alert-card h4 {
@@ -323,7 +396,7 @@ th {
 .status-pill.launching,
 .status-pill.reconciling {
   background: var(--accent-soft);
-  color: #0d5a54;
+  color: var(--accent);
 }
 
 .status-pill.waiting-provider,
@@ -332,7 +405,7 @@ th {
 .status-pill.idle,
 .status-pill.sleeping {
   background: var(--warn-soft);
-  color: #855500;
+  color: var(--warn);
 }
 
 .status-pill.FAILED,
@@ -343,19 +416,19 @@ th {
 }
 
 .status-pill.implemented {
-  background: #dcfce7;
-  color: #166534;
+  background: var(--implemented-soft);
+  color: var(--implemented-ink);
 }
 
 .status-pill.reported,
 .status-pill.completed {
-  background: #dbeafe;
-  color: #1d4ed8;
+  background: var(--reported-soft);
+  color: var(--reported-ink);
 }
 
 .status-pill.blocked {
-  background: #fef3c7;
-  color: #92400e;
+  background: var(--blocked-soft);
+  color: var(--blocked-ink);
 }
 
 .status-pill.failed,
@@ -366,7 +439,7 @@ th {
 
 .status-pill.running {
   background: var(--accent-soft);
-  color: #0d5a54;
+  color: var(--accent);
 }
 
 .empty-state {
@@ -387,6 +460,10 @@ th {
     text-align: left;
   }
 
+  .hero-controls {
+    justify-content: flex-start;
+  }
+
   .panel.half,
   .panel.third {
     grid-column: span 12;

package/tools/templates/pr-fix-template.md

@@ -3,11 +3,11 @@ You are the PR repair worker for `{REPO_SLUG}`.
 Before making any change:
 
 1. Read `{REPO_ROOT}/AGENTS.md`.
-2. Read `{REPO_ROOT}/openspec/AGENT_RULES.md`.
-3. Read `{REPO_ROOT}/openspec/AGENTS.md`.
-4. Read `{REPO_ROOT}/openspec/project.md`.
-5. Read `{REPO_ROOT}/openspec/CONVENTIONS.md`.
-6. Read `{REPO_ROOT}/docs/TESTING_AND_SEED_POLICY.md`.
+2. If present, read `{REPO_ROOT}/openspec/AGENT_RULES.md`.
+3. If present, read `{REPO_ROOT}/openspec/AGENTS.md`.
+4. If present, read `{REPO_ROOT}/openspec/project.md`.
+5. If present, read `{REPO_ROOT}/openspec/CONVENTIONS.md`.
+6. If present, read `{REPO_ROOT}/docs/TESTING_AND_SEED_POLICY.md`.
 7. Stay on this PR branch worktree. Do not push or mutate GitHub from inside the worker.
 
 PR metadata:
@@ -58,7 +58,7 @@ PR body:
 Required flow:
 
 1. Inspect the current diff and the failing/pending CI signals first:
-   - `openspec list`
+   - `openspec list` if the repo uses OpenSpec
    - `git diff --stat origin/main...HEAD`
    - `git status --short`
    - if `Merge state` is not `CLEAN` or `Mergeable` is `FALSE`, treat branch drift/conflicts as the concrete blocker first

package/tools/templates/pr-merge-repair-template.md

@@ -3,11 +3,11 @@ You are the PR merge-repair worker for `{REPO_SLUG}`.
 Before making any change:
 
 1. Read `{REPO_ROOT}/AGENTS.md`.
-2. Read `{REPO_ROOT}/openspec/AGENT_RULES.md`.
-3. Read `{REPO_ROOT}/openspec/AGENTS.md`.
-4. Read `{REPO_ROOT}/openspec/project.md`.
-5. Read `{REPO_ROOT}/openspec/CONVENTIONS.md`.
-6. Read `{REPO_ROOT}/docs/TESTING_AND_SEED_POLICY.md`.
+2. If present, read `{REPO_ROOT}/openspec/AGENT_RULES.md`.
+3. If present, read `{REPO_ROOT}/openspec/AGENTS.md`.
+4. If present, read `{REPO_ROOT}/openspec/project.md`.
+5. If present, read `{REPO_ROOT}/openspec/CONVENTIONS.md`.
+6. If present, read `{REPO_ROOT}/docs/TESTING_AND_SEED_POLICY.md`.
 7. Stay on this PR branch worktree. Do not push or mutate GitHub from inside the worker.
 
 PR metadata:
@@ -53,7 +53,7 @@ Required flow:
    - do not run `git fetch`, `git pull`, `git merge`, `git rebase`, `git commit`, `git push`, or any command that writes Git metadata
    - do not abort or restart the prepared merge state
 3. Inspect only the concrete branch-repair state you were given:
-   - `openspec list`
+   - `openspec list` if the repo uses OpenSpec
    - `git status --short`
    - `git diff --check`
    - `git diff --name-only --diff-filter=U`