reflectt-node 0.1.6 → 0.1.8

package/public/dashboard.js

@@ -33,6 +33,8 @@ function activatePage(page) {
   document.querySelectorAll('.sidebar-link[data-page]').forEach(link => {
     link.classList.toggle('active', link.dataset.page === page);
   });
+  // Load doctor page data when shown
+  if (page === 'doctor' && typeof loadDoctorPage === 'function') loadDoctorPage();
 }
 
 function toggleSidebar() {
@@ -724,9 +726,10 @@ async function loadPresence() {
 // ---- Tasks ----
 async function loadTasks(forceFull = false) {
   try {
-    const useDelta = !forceFull && lastTaskSync > 0;
+    // Force full load if we have no tasks yet (e.g. first load failed)
+    const useDelta = !forceFull && lastTaskSync > 0 && allTasks.length > 0;
     const qs = new URLSearchParams();
-    qs.set('limit', '80');
+    qs.set('limit', '200');
     if (useDelta) qs.set('updatedSince', String(lastTaskSync));
 
     const r = await fetch(BASE + '/tasks?' + qs.toString());
@@ -1382,16 +1385,8 @@ async function loadSharedArtifacts() {
     const pinnedRows = pinned.map(p => {
       const found = files.find(f => (String(f.name || '')).toLowerCase() === p.name.toLowerCase());
       if (!found) {
-        return `<div style="padding:10px 12px;border-bottom:1px solid var(--border-subtle)">
-          <div style="display:flex;align-items:center;justify-content:space-between;gap:10px">
-            <div style="font-size:13px;color:var(--text-bright);font-weight:600">${esc(p.label)}</div>
-            <span class="assignee-tag" style="color:var(--yellow)">missing</span>
-          </div>
-          <div style="font-size:11px;color:var(--text-muted);margin-top:4px">
-            To make this available here, create a symlink in the shared workspace:<br/>
-            <code>ln -s ../RYANS-THOUGHTS.md ~/.openclaw/workspace-shared/process/RYANS-THOUGHTS.md</code>
-          </div>
-        </div>`;
+        // Operator notes are optional. If not configured, do not show an operator-only instruction to end users.
+        return '';
       }
       const href = '/shared/view?path=' + encodeURIComponent(found.path);
       return `<div style="padding:10px 12px;border-bottom:1px solid var(--border-subtle)">
@@ -1420,9 +1415,13 @@ async function loadSharedArtifacts() {
       </div>`;
     }).join('');
 
+    const pinnedBox = pinnedRows
+      ? `<div style="border:1px solid var(--border-subtle);border-radius:8px;overflow:hidden">${pinnedRows}</div>
+         <div style="height:10px"></div>`
+      : '';
+
     body.innerHTML = `
-      <div style="border:1px solid var(--border-subtle);border-radius:8px;overflow:hidden">${pinnedRows}</div>
-      <div style="height:10px"></div>
+      ${pinnedBox}
       <div style="font-size:11px;color:var(--text-muted);margin:0 0 8px">Shared workspace directory: <code>process/</code></div>
       <div style="border:1px solid var(--border-subtle);border-radius:8px;overflow:hidden">${listRows}</div>
     `;
@@ -1658,12 +1657,7 @@ async function loadDoctorPage() {
   }
 }
 
-// Hook into page activation: load doctor data when the page is shown
-const _origActivatePage = activatePage;
-function activatePage(page) {
-  _origActivatePage(page);
-  if (page === 'doctor') loadDoctorPage();
-}
+// Doctor page activation handled in activatePage() above
 
 async function loadReleaseStatus(force = false) {
   const badge = document.getElementById('release-badge');
@@ -1809,6 +1803,18 @@ function normalizeEpochMs(v) {
   return v;
 }
 
+function hasReviewerDecision(task) {
+  const meta = task && task.metadata && typeof task.metadata === 'object' ? task.metadata : null;
+  const decision = meta && meta.reviewer_decision && typeof meta.reviewer_decision === 'object'
+    ? meta.reviewer_decision
+    : null;
+  return !!decision;
+}
+
+function shouldEscalateReviewerSla(task) {
+  return task && task.slaState === 'breach' && !hasReviewerDecision(task);
+}
+
 function renderReviewQueue() {
   const panel = document.getElementById('review-queue-panel');
   const body = document.getElementById('review-queue-body');
@@ -1820,11 +1826,23 @@ function renderReviewQueue() {
   const validating = allTasks
     .filter(t => t.status === 'validating')
     .map(t => {
-      const rawEntered = t.metadata?.entered_validating_at || t.updatedAt || t.createdAt;
+      const meta = t.metadata || {};
+      const reviewState = typeof meta.review_state === 'string' ? meta.review_state : '';
+      const reviewerDecision = meta.reviewer_decision;
+
+      // If reviewer has acted (needs_author or reviewer_decision recorded), the ball is with the assignee.
+      // We still track an SLA timer, but it should page the assignee (author), not the reviewer.
+      const waitOn = (reviewState === 'needs_author' || reviewerDecision != null) ? 'author' : 'reviewer';
+
+      const rawEntered = waitOn === 'author'
+        ? (reviewerDecision && reviewerDecision.decidedAt) || meta.review_last_activity_at || meta.entered_validating_at || t.updatedAt || t.createdAt
+        : meta.entered_validating_at || t.updatedAt || t.createdAt;
+
       const enteredAt = normalizeEpochMs(rawEntered) || now;
       const timeInReview = Math.min(Math.max(0, now - enteredAt), MAX_REVIEW_MS);
       const slaState = getReviewSlaState(timeInReview);
-      return { ...t, timeInReview, slaState, enteredAt };
+
+      return { ...t, timeInReview, slaState, enteredAt, waitOn, reviewState, hasReviewerDecision: reviewerDecision != null };
     })
     .sort((a, b) => {
       // Breaches first, then by time descending
@@ -1834,6 +1852,9 @@ function renderReviewQueue() {
       return b.timeInReview - a.timeInReview;
     });
 
+  const reviewerQueue = validating.filter(t => t.waitOn === 'reviewer');
+  const authorQueue = validating.filter(t => t.waitOn === 'author');
+
   if (validating.length === 0) {
     panel.style.display = '';
     count.textContent = '';
@@ -1849,16 +1870,29 @@ function renderReviewQueue() {
   }
 
   panel.style.display = '';
-  count.textContent = validating.length + ' awaiting review';
-  // Update sidebar badge
+  const reviewerBreachCount = reviewerQueue.filter(t => t.slaState === 'breach').length;
+  const authorBreachCount = authorQueue.filter(t => t.slaState === 'breach').length;
+
+  // Primary count is "awaiting reviewer" — author-wait tasks are tracked separately.
+  count.textContent = reviewerQueue.length + ' awaiting review';
+
+  // Update sidebar badge (reviewer queue only)
   const navReviewBadge = document.getElementById('nav-review-count');
-  if (navReviewBadge) navReviewBadge.textContent = validating.length;
+  if (navReviewBadge) navReviewBadge.textContent = String(reviewerQueue.length);
+
+  const headerExtra = (reviewerBreachCount > 0 || authorBreachCount > 0)
+    ? ' <span style="color:var(--red);font-size:11px;font-weight:600">'
+        + (reviewerBreachCount > 0 ? (reviewerBreachCount + ' reviewer breach' + (reviewerBreachCount > 1 ? 'es' : '')) : '')
+        + (reviewerBreachCount > 0 && authorBreachCount > 0 ? ' · ' : '')
+        + (authorBreachCount > 0 ? (authorBreachCount + ' author breach' + (authorBreachCount > 1 ? 'es' : '')) : '')
+      + '</span>'
+    : '';
 
-  const breachCount = validating.filter(t => t.slaState === 'breach').length;
-  const headerExtra = breachCount > 0
-    ? ' <span style="color:var(--red);font-size:11px;font-weight:600">' + breachCount + ' breach' + (breachCount > 1 ? 'es' : '') + '</span>'
+  const authorExtra = authorQueue.length > 0
+    ? ' <span style="color:var(--text-muted);font-size:11px;font-weight:500">· ' + authorQueue.length + ' awaiting author</span>'
     : '';
-  count.innerHTML = validating.length + ' awaiting review' + headerExtra;
+
+  count.innerHTML = reviewerQueue.length + ' awaiting review' + authorExtra + headerExtra;
 
   body.innerHTML = validating.map(t => {
     const reviewer = t.reviewer || '<span style="color:var(--yellow)">unassigned</span>';
@@ -1887,33 +1921,40 @@ function renderReviewQueue() {
 
   bindTaskLinkHandlers(body);
 
-  // SLA breach escalation: post to watchdog if any breach found
-  if (breachCount > 0) {
-    escalateReviewBreaches(validating.filter(t => t.slaState === 'breach'));
+  // SLA breach escalation: split reviewer-wait vs author-wait so we page the right person.
+  // shouldEscalateReviewerSla() suppresses escalation when reviewer_decision already exists.
+  if (reviewerBreachCount > 0) {
+    escalateReviewerBreaches(reviewerQueue.filter(shouldEscalateReviewerSla));
+  }
+  if (authorBreachCount > 0) {
+    escalateAuthorBreaches(authorQueue.filter(t => t.slaState === 'breach'));
   }
 }
 
-let lastReviewEscalationAt = 0;
+let lastReviewerEscalationAt = 0;
+let lastAuthorEscalationAt = 0;
 const REVIEW_ESCALATION_COOLDOWN = 20 * 60 * 1000; // 20m
 
-async function escalateReviewBreaches(breachedTasks) {
+async function escalateReviewerBreaches(breachedTasks) {
   const now = Date.now();
-  if (now - lastReviewEscalationAt < REVIEW_ESCALATION_COOLDOWN) return;
-  lastReviewEscalationAt = now;
+  if (now - lastReviewerEscalationAt < REVIEW_ESCALATION_COOLDOWN) return;
+  lastReviewerEscalationAt = now;
 
   const lines = breachedTasks.slice(0, 5).map(t => {
     const reviewer = t.reviewer || 'unassigned';
-    return '- ' + t.id + ' (' + (t.title || '').slice(0, 50) + ') — reviewer: @' + reviewer + ', waiting ' + formatDuration(t.timeInReview);
+    const prUrl = t.metadata?.review_handoff?.pr_url || t.metadata?.review_handoff?.prUrl || '';
+    const prPart = prUrl ? ' — ' + prUrl : '';
+    return '- ' + t.id + ' (' + (t.title || '').slice(0, 50) + ') — reviewer: @' + reviewer + ', waiting ' + formatDuration(t.timeInReview) + prPart;
   });
 
-  const content = '@owner Review SLA breach detected:\n' + lines.join('\n');
+  const content = 'Review overdue (reviewer SLA):\n' + lines.join('\n');
 
   try {
     await fetch(BASE + '/chat/messages', {
       method: 'POST',
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify({
-        from: 'system',
+        from: 'dashboard',
         content,
         channel: 'general',
         timestamp: now
@@ -1924,6 +1965,36 @@ async function escalateReviewBreaches(breachedTasks) {
   }
 }
 
+async function escalateAuthorBreaches(breachedTasks) {
+  const now = Date.now();
+  if (now - lastAuthorEscalationAt < REVIEW_ESCALATION_COOLDOWN) return;
+  lastAuthorEscalationAt = now;
+
+  const lines = breachedTasks.slice(0, 5).map(t => {
+    const assignee = t.assignee || 'unassigned';
+    const prUrl = t.metadata?.review_handoff?.pr_url || t.metadata?.review_handoff?.prUrl || '';
+    const prPart = prUrl ? ' — ' + prUrl : '';
+    return '- ' + t.id + ' (' + (t.title || '').slice(0, 50) + ') — assignee: @' + assignee + ', waiting ' + formatDuration(t.timeInReview) + prPart;
+  });
+
+  const content = 'Author action needed (post-review):\n' + lines.join('\n');
+
+  try {
+    await fetch(BASE + '/chat/messages', {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        from: 'dashboard',
+        content,
+        channel: 'general',
+        timestamp: now
+      })
+    });
+  } catch (err) {
+    console.error('Failed to escalate author breach:', err);
+  }
+}
+
 // ---- Feedback ----
 let feedbackData = null;
 
@@ -2670,7 +2741,7 @@ function openTaskModal(taskId) {
   loadPrReviewPanel(currentTask);
 }
 
-function formatDuration(sec) {
+function formatDurationSec(sec) {
   if (sec == null) return '';
   if (sec < 60) return sec + 's';
   return Math.floor(sec / 60) + 'm ' + (sec % 60) + 's';
@@ -2763,7 +2834,7 @@ function renderPrReviewPanel(data) {
       html += '<div class="ci-check-row">';
       html += '<span class="check-icon">' + icon + '</span>';
       html += '<span class="check-name">' + esc(c.name) + '</span>';
-      if (c.durationSec != null) html += '<span class="check-duration">' + formatDuration(c.durationSec) + '</span>';
+      if (c.durationSec != null) html += '<span class="check-duration">' + formatDurationSec(c.durationSec) + '</span>';
       if (c.detailsUrl) html += '<a href="' + esc(c.detailsUrl) + '" target="_blank">logs</a>';
       html += '</div>';
     });
@@ -2979,7 +3050,7 @@ async function checkGettingStarted() {
     // Step 1 done: check if system health loops are ticking (not just uptime > 0)
     const hasHeartbeat = !!(health.system?.loops?.lastTickAt || (health.tasks?.total > 0));
     const hasTasks = (health.tasks?.total || 0) > 0;
-    const hasMessages = (health.chat?.total || 0) > 0;
+    const hasMessages = (health.chat?.totalMessages || health.chat?.total || 0) > 0;
 
     // Step 1: server running — always done if dashboard loads
     const step1 = document.getElementById('gs-preflight');
@@ -3005,23 +3076,39 @@ async function checkGettingStarted() {
       step3.querySelector('.gs-icon').textContent = '✓';
     }
 
-    // Auto-hide if all steps done
+    // Auto-hide if all steps done, OR if the system clearly isn't a fresh install
     const allDone = panel.querySelectorAll('.gs-step.done').length === 3;
-    if (allDone) {
+    const clearlyNotFresh = (health.tasks?.total || 0) > 5 || (health.chat?.totalMessages || health.chat?.total || 0) > 100;
+    if (allDone || clearlyNotFresh) {
       panel.classList.add('hidden');
     }
+
+    // Pre-populate task badge from health before full task fetch completes
+    const taskTotal = health.tasks?.total || 0;
+    const navTaskBadge = document.getElementById('nav-task-count');
+    if (navTaskBadge && taskTotal > 0 && allTasks.length === 0) {
+      navTaskBadge.textContent = taskTotal;
+    }
   } catch {}
 }
 
 function dismissGettingStarted() {
   const panel = document.getElementById('getting-started');
-  if (panel) panel.classList.add('hidden');
+  if (panel) {
+    panel.classList.add('hidden');
+    panel.style.display = 'none';  // belt + suspenders
+  }
   try { localStorage.setItem('reflectt-gs-dismissed', '1'); } catch {}
 }
+// Expose globally for onclick handlers (safety net)
+window.dismissGettingStarted = dismissGettingStarted;
+window.dismissFirstBootBanner = dismissFirstBootBanner;
 
 updateClock();
 setInterval(updateClock, 30000);
 checkGettingStarted();
+// Retry getting-started check after 5s in case health wasn't ready at boot
+setTimeout(checkGettingStarted, 5000);
 refresh();
 connectEventStream();
 startAdaptiveRefresh();
@@ -3401,20 +3488,13 @@ function fileIcon(mimeType, name) {
 }
 
 function formatFileSize(bytes) {
-  if (!bytes) return '0 B';
+  if (typeof bytes !== 'number' || bytes <= 0) return '0 B';
   if (bytes < 1024) return bytes + ' B';
   if (bytes < 1048576) return (bytes / 1024).toFixed(1) + ' KB';
   return (bytes / 1048576).toFixed(1) + ' MB';
 }
 
-function timeAgo(ts) {
-  const d = typeof ts === 'string' ? new Date(ts) : new Date(ts);
-  const s = Math.floor((Date.now() - d.getTime()) / 1000);
-  if (s < 60) return 'just now';
-  if (s < 3600) return Math.floor(s / 60) + 'm ago';
-  if (s < 86400) return Math.floor(s / 3600) + 'h ago';
-  return Math.floor(s / 86400) + 'd ago';
-}
+// timeAgo defined above (line ~3286)
 
 async function loadFiles() {
   try {
@@ -3457,7 +3537,7 @@ function renderFiles(files) {
     return '<div class="file-card" tabindex="0" role="button" aria-label="' + (f.originalName || f.filename) + '">' +
       '<div class="thumb" style="display:flex;align-items:center;justify-content:center;height:80px;background:var(--surface-raised);border-radius:var(--radius-sm);overflow:hidden">' + thumb + '</div>' +
       '<div class="card-name">' + (f.originalName || f.filename) + '</div>' +
-      '<div class="card-meta">' + formatFileSize(f.size) + ' · ' + timeAgo(f.uploadedAt || f.createdAt) + '</div>' +
+      '<div class="card-meta">' + formatFileSize(f.sizeBytes) + ' · ' + timeAgo(f.uploadedAt || f.createdAt) + '</div>' +
       '<div class="card-actions">' +
         '<a href="/files/' + f.id + '" download="' + (f.originalName || f.filename) + '" class="action-btn" aria-label="Download" onclick="event.stopPropagation()">⬇</a>' +
         '<button class="action-btn delete" aria-label="Delete" onclick="event.stopPropagation();deleteFile(\'' + f.id + '\')">🗑</button>' +
@@ -3470,7 +3550,7 @@ function renderFiles(files) {
     return '<div class="file-list-item" tabindex="0" role="button">' +
       '<span class="list-icon">' + icon + '</span>' +
       '<span class="list-name">' + (f.originalName || f.filename) + '</span>' +
-      '<span class="list-meta">' + formatFileSize(f.size) + '</span>' +
+      '<span class="list-meta">' + formatFileSize(f.sizeBytes) + '</span>' +
       '<span class="list-meta">' + timeAgo(f.uploadedAt || f.createdAt) + '</span>' +
       '<div class="list-actions">' +
         '<a href="/files/' + f.id + '" download="' + (f.originalName || f.filename) + '" class="action-btn" aria-label="Download" onclick="event.stopPropagation()">⬇</a>' +

package/public/docs.md

@@ -216,7 +216,10 @@ If your deployment needs quiet-hours behavior today, enforce it in scheduler/gat
 | POST | `/tasks/batch-create` | Batch create up to 20 tasks. Body: `{ "tasks": [...], "createdBy": "agent", "deduplicate": true, "dryRun": false }`. Each task follows the same schema as `POST /tasks`. Returns per-task results (created/duplicate/error) with summary counts. Deduplication checks exact title match + fuzzy word overlap (Jaccard >0.6) against active tasks. |
 | GET | `/tasks/heartbeat-status` | All doing tasks with stale comment activity (>30m). Returns `{ threshold, doingTaskCount, staleCount, staleTasks[] }`. Use for monitoring status heartbeat discipline compliance. |
 | GET | `/tasks/board-health` | Board-level health metrics for backlog replenishment. Returns per-agent breakdown (doing, validating, todo, active counts), `needsWork`/`lowWatermark` flags, and `replenishNeeded` trigger (fires when 2+ agents idle or <3 backlog tasks). Query: `include_test=1` to include test-harness tasks. |
+| GET | `/agents` | Agent list — alias for /agents/roles. Returns all agents with roles, WIP status, affinity tags. |
 | GET | `/agents/roles` | Agent role registry with live WIP status. Returns all agents with `name`, `displayName`, `role`, `affinityTags`, `protectedDomains`, `wipCap`, `wipCount`, `overCap`. |
+| POST | `/agents` | Add agent to team. Body: `{ name, role, description?, affinityTags?, wipCap? }`. Hot-reloads TEAM-ROLES.yaml. |
+| DELETE | `/agents/:name` | Remove agent from team. Hot-reloads TEAM-ROLES.yaml. |
 | POST | `/config/identity` | Set an agent's display name. Body: `{ "agent": "agent-1", "displayName": "Juniper" }`. Persists to TEAM-ROLES.yaml, hot-reloads. Dashboard and mentions use display name. |
 | PUT | `/config/team-roles` | Write TEAM-ROLES.yaml. Body: `{ "yaml": "agents:\n  - name: link\n    role: engineer..." }`. Hot-reloads on save. Used by bootstrap agent to configure team from user intent. |
 | GET | `/resolve/mention/:mention` | Resolve a mention string (name, displayName, or alias) to canonical agent ID. Returns `{ agent, displayName, role }`. |
@@ -863,6 +866,7 @@ Set via `reflectionNudge` in policy config:
 | POST | `/usage/caps` | Create spend cap. Body: `{ scope: "global"\|"agent"\|"team", scope_id?, period: "daily"\|"weekly"\|"monthly", limit_usd, action: "warn"\|"throttle"\|"block" }`. |
 | DELETE | `/usage/caps/:id` | Delete a spend cap. |
 | GET | `/usage/routing-suggestions` | Smart routing savings suggestions (which low-stakes categories could use cheaper models). Query: `since`. |
+| GET | `/costs` | Cost dashboard — aggregated spend for COO/PM monitoring. Query: `days` (1–90, default 7). Returns: `daily_by_model` (spend per model per day), `daily_totals` (per-day rolled-up for threshold alerting), `avg_cost_by_lane` (avg cost per closed task by `qa_bundle.lane`, 30-day floor), `avg_cost_by_agent` (avg cost per closed task per agent + `top_model`, 30-day floor), `top_tasks_by_cost` (top 20 most expensive tasks in window), `summary` (total tokens + cost), `lane_agent_window_days` (actual window used for lane/agent averages). |
 
 ### Model Pricing (built-in estimates, per 1M tokens)
 | Model | Input | Output |
@@ -1034,6 +1038,20 @@ Export → import preserves: summary, description, organizer, attendees, locatio
 | GET | `/calendar/events/:id/export.ics` | Export single event as .ics file. |
 | POST | `/calendar/import` | Import events from .ics content. Body: `{ ics: string, organizer?: string }` or raw .ics string. Returns created/updated events. UID-based dedup on re-import. |
 
+## Schedule Feed
+
+Shared time-awareness for the team. Canonical records for deploy windows, focus blocks, and scheduled task work — so agents can coordinate timing without chat.
+
+**MVP scope (v1):** One-off windows only. No iCal/RRULE, no reminders, no recurring rules. For per-agent availability and recurring blocks use `/calendar/blocks`. For notifications use the Calendar Reminder Engine.
+
+| Method | Path | Description |
+|--------|------|-------------|
+| GET | `/schedule/feed` | Upcoming entries in chronological order. Query: `kinds` (comma-separated: `deploy_window,focus_block,scheduled_task`), `owner`, `after` (epoch ms, default: now), `before` (epoch ms), `limit` (default: 50, max: 200). |
+| POST | `/schedule/entries` | Create a schedule entry. Body: `{ kind, title, start, end, owner, task_id?, status?, meta? }`. `kind` must be `deploy_window`, `focus_block`, or `scheduled_task`. `start`/`end` are epoch ms. Default status: `open` / `active` / `pending`. |
+| GET | `/schedule/entries/:id` | Get a single entry by ID. |
+| PATCH | `/schedule/entries/:id` | Update an entry. Body: `{ title?, start?, end?, status?, meta? }`. |
+| DELETE | `/schedule/entries/:id` | Delete an entry. Returns 204. |
+
 ## Remote Node Management
 
 Auth-gated endpoints for managing a reflectt-node instance remotely. Provide `REFLECTT_MANAGE_TOKEN` env var; authenticate via `x-manage-token` header or `Authorization: Bearer <token>`. Loopback (localhost) access is always allowed.