social-autoposter 1.3.5 → 1.3.6

package/bin/server.js

@@ -1077,6 +1077,20 @@ async function enrichPostCommentsLinkedInRuns(runs) {
 //   posted    = COUNT(*) twitter_candidates status='posted' in window
 //   pending   = global COUNT(*) status='pending' (small for Twitter; each cycle
 //                self-expires its own batch)
+// Parse a twitter cycle batch_id (`twcycle-YYYYMMDD-HHMMSS`) into epoch ms.
+// Used to map each candidate/search row back to the cycle that owns it, so a
+// long-running cycle's posts aren't also attributed to the next cycle's run
+// window (the prior overlap-based attribution triple-counted posts when three
+// cycles were live simultaneously, e.g. 2026-05-14 16:13/16:15/16:30 all
+// reported posted=8/8/4 for the same 8 unique posts).
+function parseTwitterBatchIdMs(batchId) {
+  if (!batchId) return NaN;
+  const m = batchId.match(/^twcycle-(\d{4})(\d{2})(\d{2})-(\d{2})(\d{2})(\d{2})$/);
+  if (!m) return NaN;
+  const [, y, mo, d, hh, mm, ss] = m;
+  return new Date(`${y}-${mo}-${d}T${hh}:${mm}:${ss}`).getTime();
+}
+
 async function enrichPostCommentsTwitterRuns(runs) {
   const txRuns = runs.filter(r =>
     r.job_type === 'post-comments' && r.platform_key === 'twitter'
@@ -1088,6 +1102,19 @@ async function enrichPostCommentsTwitterRuns(runs) {
     if (ms < oldestMs) oldestMs = ms;
   }
   const since = new Date(oldestMs - 2 * 60 * 1000).toISOString();
+  // Cycle log files (`twitter-cycle-YYYY-MM-DD_HHMMSS.log`) carry the Phase 0
+  // salvage marker we need for the salvaged pill. Read once per enricher call.
+  let logFiles = [];
+  try {
+    logFiles = fs.readdirSync(LOG_DIR).filter(f => f.startsWith('twitter-cycle-') && f.endsWith('.log'));
+  } catch { /* empty */ }
+  const cycleFileTs = (name) => {
+    const m = name.match(/^twitter-cycle-(\d{4}-\d{2}-\d{2})_(\d{2})(\d{2})(\d{2})\.log$/);
+    if (!m) return NaN;
+    const [, day, hh, mm, ss] = m;
+    return new Date(`${day}T${hh}:${mm}:${ss}`).getTime();
+  };
+  const phase0SalvageRe = /Phase 0: salvaged (\d+) orphaned pending rows/;
   const searchRows = await pq(
     "SELECT ran_at, tweets_found, batch_id FROM twitter_search_attempts " +
     "WHERE ran_at >= $1::timestamp",
@@ -1173,22 +1200,78 @@ async function enrichPostCommentsTwitterRuns(runs) {
   for (const run of txRuns) {
     const startMs = new Date(run.started_at).getTime();
     const endMs = new Date(run.finished_at).getTime() + 60 * 1000;
+    // Attribute funnel counters to the ONE batch this run owns — derived by
+    // matching `twcycle-YYYYMMDD-HHMMSS` to run.started_at within ±10s. The
+    // prior code attributed every batch whose search_attempt fell in this
+    // window, which triple-counted posts under overlapping cycles.
+    let ownBatchId = null;
+    let ownBatchDelta = Infinity;
+    for (const s of searchNorm) {
+      if (!s.batch_id) continue;
+      const bms = parseTwitterBatchIdMs(s.batch_id);
+      if (!Number.isFinite(bms)) continue;
+      const delta = Math.abs(bms - startMs);
+      if (delta > 10 * 1000) continue;
+      if (delta < ownBatchDelta) { ownBatchDelta = delta; ownBatchId = s.batch_id; }
+    }
+    // Fall back to scanning candidate rows if no search_attempt matched (e.g.
+    // Phase 1 logged nothing because the cycle aborted before scraping).
+    if (!ownBatchId) {
+      for (const c of candNorm) {
+        if (!c.batch_id) continue;
+        const bms = parseTwitterBatchIdMs(c.batch_id);
+        if (!Number.isFinite(bms)) continue;
+        const delta = Math.abs(bms - startMs);
+        if (delta > 10 * 1000) continue;
+        if (delta < ownBatchDelta) { ownBatchDelta = delta; ownBatchId = c.batch_id; }
+      }
+    }
     let searches = 0, candidatesRaw = 0, posted = 0, expired = 0;
-    const batchIds = new Set();
     for (const s of searchNorm) {
       if (s.ms == null || s.ms < startMs || s.ms > endMs) continue;
+      if (!ownBatchId || s.batch_id !== ownBatchId) continue;
       searches++;
       candidatesRaw += s.found;
-      if (s.batch_id) batchIds.add(s.batch_id);
     }
     let candidatesPassed = 0;
+    let salvagePosted = 0;
     for (const c of candNorm) {
-      if (!c.batch_id || !batchIds.has(c.batch_id)) continue;
+      if (!ownBatchId || c.batch_id !== ownBatchId) continue;
       candidatesPassed++;
-      if (c.status === 'posted') posted++;
-      else if (c.status === 'expired') expired++;
+      if (c.status === 'posted') {
+        posted++;
+        // Salvage signature: candidate's discovered_at predates this cycle's
+        // start by enough that it must have been pulled in by Phase 0 (which
+        // rewrites batch_id to the current BATCH_ID). Tolerance covers Phase 1
+        // re-discovery via SERP repeat (very fast — <run_duration window).
+        if (c.discoveredMs != null && c.discoveredMs < startMs - 30 * 60 * 1000) {
+          salvagePosted++;
+        }
+      } else if (c.status === 'expired') expired++;
     }
     const candidatesDropped = Math.max(0, candidatesRaw - candidatesPassed);
+    // Phase 0 salvage attempt count from the cycle log — what was actually
+    // pulled in (vs `salvageable_now`, which is the future pool). Surfaces in
+    // the salvaged pill so it shows what THIS cycle salvaged, not what NEXT
+    // cycle could salvage.
+    let salvageAttempted = 0;
+    let chosenLog = null;
+    let chosenLogDelta = Infinity;
+    for (const f of logFiles) {
+      const ts = cycleFileTs(f);
+      if (!Number.isFinite(ts)) continue;
+      if (ts > startMs + 90 * 1000) continue;
+      if (ts < startMs - 90 * 1000) continue;
+      const delta = Math.abs(ts - startMs);
+      if (delta < chosenLogDelta) { chosenLogDelta = delta; chosenLog = f; }
+    }
+    if (chosenLog) {
+      try {
+        const body = fs.readFileSync(path.join(LOG_DIR, chosenLog), 'utf8');
+        const m = body.match(phase0SalvageRe);
+        if (m) salvageAttempted = parseInt(m[1], 10);
+      } catch { /* empty */ }
+    }
     // Per-run queue delta. ADD = candidates whose discovered_at fell in this
     // run's window (Phase 1 SERP discovery wrote them into twitter_candidates
     // as 'pending'). DRAIN = candidates that left 'pending' inside the same
@@ -1282,6 +1365,14 @@ async function enrichPostCommentsTwitterRuns(runs) {
       salvageable_now: salvageableNow,
       salvageable_added: salvAdded,
       salvageable_drained: salvDrained,
+      // Actual salvage performed by THIS cycle's Phase 0. Read from the
+      // matching cycle log (`twitter-cycle-*.log`); falls back to 0 when the
+      // log was rotated. salvage_posted counts posted candidates whose
+      // discovered_at predates the cycle by >30min (those rows can only be
+      // here via Phase 0 salvage rewriting their batch_id).
+      salvage_attempted: salvageAttempted,
+      salvage_posted: salvagePosted,
+      own_batch_id: ownBatchId,
       cost_usd: prior.cost_usd || 0,
       failed: prior.failed || 0,
       failure_reasons: Array.isArray(prior.failure_reasons) ? prior.failure_reasons : [],
@@ -5945,6 +6036,15 @@ const HTML = `<!DOCTYPE html>
   }
   .sa-del-btn { position: relative; }
   @keyframes saDelSpin { to { transform: rotate(360deg); } }
+  /* Inline editable weight cell: input + small spinner shown while saving. */
+  .pw-cell { display: inline-flex; align-items: center; gap: 4px; justify-content: flex-end; }
+  .pw-spinner {
+    display: none; width: 10px; height: 10px; border: 1.5px solid transparent;
+    border-top-color: var(--text-muted); border-right-color: var(--text-muted);
+    border-radius: 50%; animation: saDelSpin 0.7s linear infinite;
+  }
+  .pw-cell.is-saving .pw-spinner { display: inline-block; }
+  .pw-cell.is-saving input { opacity: 0.6; }
   .sa-del-btn.is-pending {
     opacity: 1; color: #f59e0b; border-color: rgba(245, 158, 11, 0.35);
     background: rgba(245, 158, 11, 0.08);
@@ -7595,6 +7695,12 @@ function renderResult(run) {
     const salvageableLive = r.salvageable_now || 0;
     const salvAdded = r.salvageable_added || 0;
     const salvDrained = r.salvageable_drained || 0;
+    // Actual Phase 0 salvage this cycle did (read from cycle log) and the
+    // count of those salvaged rows that ended up posted. Distinct from
+    // salvageable_now, which is the pool size for the NEXT cycle. Mirrors
+    // Reddit's salvage_attempted / salvage_posted split.
+    const salvAttempted = r.salvage_attempted || 0;
+    const salvPosted = r.salvage_posted || 0;
     // Legacy queue fields kept for the tooltip (operator can still see queue
     // depth + drain breakdown if they hover the pill).
     const queue = (r.queue_end != null) ? r.queue_end : (r.pending_queue || 0);
@@ -7621,22 +7727,24 @@ function renderResult(run) {
         'style="display:inline-block;margin-right:10px;font-size:12px;color:var(--muted);">' +
         label + (count ? ' <span style="color:var(--text);font-weight:600;">' + count + '</span>' : '') + '</span>';
     };
-    // Salvaged pill with per-run delta. Format: salvaged N (+A/-D)
-    // N = live count of pending rows with a persisted draft (Phase 0 salvage
-    //     pool — orphaned drafts that the next cycle's salvage lane will retry).
-    // +A = drafts created during this run's window (became salvageable).
-    // -D = drafts that exited pending during this run's window.
-    // Delta omitted when both sides are zero so old (pre-patch) rows stay
-    // clean instead of showing noisy (+0/-0).
-    const salvDeltaSuffix = (salvAdded || salvDrained)
-      ? ' <span style="color:var(--muted);font-weight:400;">(' +
-          '+' + salvAdded + '/-' + salvDrained +
-          ')</span>'
-      : '';
+    // Salvaged pill. Primary number is what THIS cycle's Phase 0 actually
+    // salvaged (from the cycle log). Falls back to the future-pool size when
+    // no cycle log was found, so old rows still surface something. Bracket
+    // shows posted-from-salvage when an attempt happened, otherwise +A/-D
+    // pool delta.
+    const salvPrimary = salvAttempted || salvageableLive;
+    let salvBracket = '';
+    if (salvAttempted > 0) {
+      salvBracket = ' <span style="color:var(--muted);font-weight:400;">(' +
+                    salvPosted + ' posted)</span>';
+    } else if (salvAdded || salvDrained) {
+      salvBracket = ' <span style="color:var(--muted);font-weight:400;">(' +
+                    '+' + salvAdded + '/-' + salvDrained + ' pool)</span>';
+    }
     const queuePill =
       '<span style="display:inline-block;margin-right:10px;font-size:12px;color:var(--muted);">' +
-      'salvaged <span style="color:var(--text);font-weight:600;">' + salvageableLive + '</span>' +
-      salvDeltaSuffix +
+      'salvaged <span style="color:var(--text);font-weight:600;">' + salvPrimary + '</span>' +
+      salvBracket +
       '</span>';
     const tooltip = 'searches: ' + searches +
       ' / raw tweets: ' + raw +
@@ -7645,7 +7753,9 @@ function renderResult(run) {
       ' / expired (delta<1 floor): ' + expired +
       ' / above review cap (delta>=10, gates POST_LIMIT=3): ' + aboveFloor +
       ' / posted: ' + posted +
-      ' / salvageable now (pending+drafted): ' + salvageableLive +
+      ' / Phase 0 salvaged into this cycle: ' + salvAttempted +
+      ' (of which posted: ' + salvPosted + ')' +
+      ' / salvageable now (pool size for next cycle): ' + salvageableLive +
       ' (+' + salvAdded + ' became salvageable / -' + salvDrained + ' drained this run)' +
       ' / pending end-of-run: ' + queue +
       ' (start: ' + queueStart + ', +' + qAdded + ' added, -' + qDrained + ' drained = ' +
@@ -9825,7 +9935,7 @@ function renderDailyMetrics() {
 // the same Trends-tab filters because it reuses _dailyMetricsSeries directly,
 // no new fetch needed. Values are percentages (0-100), formatted to one
 // decimal place; days with views=0 are dropped (ratios are undefined).
-const RATIO_METRICS = [
+let RATIO_METRICS = [
   { id: 'upvotes_per_view',         label: 'Upvotes / Views',          color: '#f97316', numerator: 'upvotes',         denominator: 'views',     format: 'pct',     scaleFactor: 100 },
   { id: 'comments_per_view',        label: 'Comments / Views',         color: '#14b8a6', numerator: 'comments',        denominator: 'views',     format: 'pct',     scaleFactor: 100 },
   { id: 'clicks_per_view',          label: 'Clicks / Views',           color: '#0ea5e9', numerator: 'clicks',          denominator: 'views',     format: 'pct',     scaleFactor: 100 },
@@ -9841,8 +9951,8 @@ const RATIO_METRICS = [
   // format='usd' switches the legend pill, axis labels, bar labels, and
   // tooltip to dollar rendering. Days with denominator=0 still drop out
   // (NaN) so the chart shows a gap rather than a misleading $0 bar.
-  { id: 'cost_per_kviews',               label: 'Cost / 1k Views',            color: '#dc2626', numerator: 'cost',            denominator: 'views',     format: 'usd',     scaleFactor: 1000 },
-  { id: 'cost_per_kvisitors',            label: 'Cost / 1k Visitors',         color: '#7c3aed', numerator: 'cost',            denominator: 'pageviews', format: 'usd',     scaleFactor: 1000 },
+  { id: 'cost_per_kviews',               label: 'Cost / 1k Views',            color: '#dc2626', numerator: 'cost',            denominator: 'views',     format: 'usd',     scaleFactor: 1000, adminOnly: true },
+  { id: 'cost_per_kvisitors',            label: 'Cost / 1k Visitors',         color: '#7c3aed', numerator: 'cost',            denominator: 'pageviews', format: 'usd',     scaleFactor: 1000, adminOnly: true },
 ];
 const RATIO_METRICS_DEFAULTS = ['upvotes_per_view', 'comments_per_view', 'clicks_per_view', 'email_signups_per_session', 'schedule_clicks_per_session', 'get_started_per_session', 'cost_per_kviews', 'cost_per_kvisitors'];
 // .v2: ratio set expanded to include cost_per_kviews + cost_per_kvisitors.
@@ -13568,8 +13678,67 @@ const PROJECT_STATUS_PLATFORM_LABELS = {
   moltbook: 'MoltBook', github: 'GitHub',
 };
 let _projectStatusLoading = false;
+let _projectStatusData = null;
+let _projectStatusOrder = null;
+const PROJECT_STATUS_SORT_STORAGE = 'sa.projectStatus.sort.v1';
+let _projectStatusSort = { field: 'weight', dir: 'desc' };
+try {
+  const saved = JSON.parse(localStorage.getItem(PROJECT_STATUS_SORT_STORAGE) || 'null');
+  if (saved && typeof saved.field === 'string' && (saved.dir === 'asc' || saved.dir === 'desc')) {
+    _projectStatusSort = { field: saved.field, dir: saved.dir };
+  }
+} catch (e) {}
+function _persistProjectStatusSort() {
+  try { localStorage.setItem(PROJECT_STATUS_SORT_STORAGE, JSON.stringify(_projectStatusSort)); } catch (e) {}
+}
+const PROJECT_STATUS_SORT_FIELDS = {
+  name:         { type: 'string',  value: r => r.name || '' },
+  weight:       { type: 'numeric', value: r => Number(r.weight) || 0 },
+  target_share: { type: 'numeric', value: r => Number(r.target_share) || 0 },
+  total:        { type: 'numeric', value: r => Number(r.total) || 0 },
+  cost:         { type: 'numeric', value: r => Number(r.cost_usd) || 0 },
+  reddit:       { type: 'numeric', value: r => Number(r.by_platform && r.by_platform.reddit) || 0 },
+  twitter:      { type: 'numeric', value: r => Number(r.by_platform && r.by_platform.twitter) || 0 },
+  linkedin:     { type: 'numeric', value: r => Number(r.by_platform && r.by_platform.linkedin) || 0 },
+  moltbook:     { type: 'numeric', value: r => Number(r.by_platform && r.by_platform.moltbook) || 0 },
+  github:       { type: 'numeric', value: r => Number(r.by_platform && r.by_platform.github) || 0 },
+};
+function _sortProjectRows(rows) {
+  const { field, dir } = _projectStatusSort;
+  const cfg = PROJECT_STATUS_SORT_FIELDS[field] || PROJECT_STATUS_SORT_FIELDS.weight;
+  const mul = dir === 'asc' ? 1 : -1;
+  return rows.slice().sort((a, b) => {
+    // Unassigned rows always live at the bottom regardless of sort.
+    if (!!a.unassigned !== !!b.unassigned) return a.unassigned ? 1 : -1;
+    const va = cfg.value(a); const vb = cfg.value(b);
+    if (cfg.type === 'numeric') {
+      const diff = (Number(va) - Number(vb)) * mul;
+      if (diff !== 0) return diff;
+      return String(a.name || '').localeCompare(String(b.name || ''));
+    }
+    return String(va).localeCompare(String(vb)) * mul;
+  });
+}
+function _applyProjectStatusOrder(rows) {
+  if (!_projectStatusOrder) return _sortProjectRows(rows);
+  // Render previously-captured order, appending any new rows at the bottom
+  // (still respecting unassigned-at-bottom). This is what keeps a freshly-
+  // edited row from jumping after a save: order is frozen until the user
+  // explicitly resorts (header click) or refreshes (↻).
+  const byName = new Map(rows.map(r => [r.name, r]));
+  const used = new Set();
+  const out = [];
+  for (const name of _projectStatusOrder) {
+    const r = byName.get(name);
+    if (r) { out.push(r); used.add(name); }
+  }
+  const leftovers = rows.filter(r => !used.has(r.name));
+  // Keep unassigned at the bottom even within the leftovers slice.
+  leftovers.sort((a, b) => (a.unassigned ? 1 : 0) - (b.unassigned ? 1 : 0));
+  return out.concat(leftovers);
+}
 function formatPct(v) { return (Number(v || 0) * 100).toFixed(1) + '%'; }
-function renderProjectStatus(data) {
+function renderProjectStatus(data, opts) {
   const body = document.getElementById('project-status-body');
   const totalEl = document.getElementById('project-status-total');
   const heading = document.getElementById('project-status-heading');
@@ -13579,6 +13748,8 @@ function renderProjectStatus(data) {
     body.innerHTML = '<div class="style-stats-empty">' + escapeHtml(data.error) + '</div>';
     return;
   }
+  _projectStatusData = data;
+  const preserveOrder = !!(opts && opts.preserveOrder);
   const hours = Number(data && data.hours) || 24;
   if (heading) heading.textContent = 'Project Status (last ' + hours + 'h)';
   const projects = (data && data.projects) || [];
@@ -13638,16 +13809,27 @@ function renderProjectStatus(data) {
     body.innerHTML = '<div class="style-stats-empty">No projects configured with weight &gt; 0.</div>';
     return;
   }
+  const sortHeader = (key, label, align) => {
+    const alignStyle = align === 'left' ? 'text-align:left;' : 'text-align:right;';
+    const active = _projectStatusSort.field === key;
+    const arrow = active ? (_projectStatusSort.dir === 'asc' ? '▲' : '▼') : '';
+    const arrowCls = 'activity-sort-arrow' + (active ? ' active' : '');
+    return '<th class="activity-sortable" data-project-sort-key="' + key + '" style="' + alignStyle + '">' +
+      '<span class="activity-header-label">' + label +
+        ' <span class="' + arrowCls + '" data-project-sort-arrow="' + key + '">' + arrow + '</span>' +
+      '</span>' +
+    '</th>';
+  };
   const header =
     '<thead><tr>' +
-      '<th style="text-align:left;">Project</th>' +
-      '<th style="text-align:right;">Weight</th>' +
-      '<th style="text-align:right;">Target&nbsp;%</th>' +
+      sortHeader('name', 'Project', 'left') +
+      sortHeader('weight', 'Weight') +
+      sortHeader('target_share', 'Target&nbsp;%') +
       PROJECT_STATUS_PLATFORMS.map(p =>
-        '<th style="text-align:right;">' + PROJECT_STATUS_PLATFORM_LABELS[p] + '</th>'
+        sortHeader(p, PROJECT_STATUS_PLATFORM_LABELS[p])
       ).join('') +
-      '<th style="text-align:right;">Total</th>' +
-      (costAvailable ? '<th style="text-align:right;" title="Claude session cost in this window">Cost</th>' : '') +
+      sortHeader('total', 'Total') +
+      (costAvailable ? sortHeader('cost', 'Cost') : '') +
     '</tr></thead>';
   const cellWithShare = (n, platformTotal, targetShare, opts) => {
     const num = Number(n) || 0;
@@ -13697,12 +13879,15 @@ function renderProjectStatus(data) {
     const editable = canEditWeight && (!r.unassigned || r.configured);
     const weightCellHtml = editable
       ? '<td style="text-align:right;font-variant-numeric:tabular-nums;">' +
-          '<input type="number" min="0" step="1" value="' + weightVal + '" ' +
-            'data-project-weight-input="' + escapeHtml(r.name) + '" ' +
-            'data-original-weight="' + weightVal + '" ' +
-            'class="project-weight-input" ' +
-            'style="width:60px;text-align:right;font-variant-numeric:tabular-nums;padding:2px 6px;border:1px solid var(--border);background:var(--bg-subtle,transparent);color:var(--text);border-radius:4px;font-size:13px;font-weight:600;" ' +
-            'title="Edit and press Enter or blur to save" />' +
+          '<span class="pw-cell" data-project-weight-cell="' + escapeHtml(r.name) + '">' +
+            '<input type="number" min="0" step="1" value="' + weightVal + '" ' +
+              'data-project-weight-input="' + escapeHtml(r.name) + '" ' +
+              'data-original-weight="' + weightVal + '" ' +
+              'class="project-weight-input" ' +
+              'style="width:56px;text-align:right;font-variant-numeric:tabular-nums;padding:2px 6px;border:1px solid var(--border);background:var(--bg-subtle,transparent);color:var(--text);border-radius:4px;font-size:13px;font-weight:600;" ' +
+              'title="Edit and press Enter or blur to save" />' +
+            '<span class="pw-spinner" aria-hidden="true"></span>' +
+          '</span>' +
         '</td>'
       : '<td style="text-align:right;font-variant-numeric:tabular-nums;">' + weightVal + '</td>';
     return '<tr>' +
@@ -13714,7 +13899,12 @@ function renderProjectStatus(data) {
       costCellHtml +
     '</tr>';
   };
-  const bodyRows = projects.map(rowHtml).join('') + unassigned.map(rowHtml).join('');
+  const allRows = projects.concat(unassigned);
+  const ordered = preserveOrder ? _applyProjectStatusOrder(allRows) : _sortProjectRows(allRows);
+  // Capture the order we just rendered so future in-place saves (or
+  // background reloads) don't reshuffle rows under the operator.
+  _projectStatusOrder = ordered.map(r => r.name);
+  const bodyRows = ordered.map(rowHtml).join('');
   const footerCells = PROJECT_STATUS_PLATFORMS.map(p =>
     '<td style="text-align:right;font-variant-numeric:tabular-nums;">' + (Number(totals[p]) || 0) + '</td>'
   ).join('');
@@ -13738,6 +13928,25 @@ function renderProjectStatus(data) {
         '<tbody>' + bodyRows + footerHtml + '</tbody>' +
       '</table>' +
     '</div>' + legend;
+  body.querySelectorAll('[data-project-sort-key]').forEach(th => {
+    th.addEventListener('click', () => {
+      const key = th.getAttribute('data-project-sort-key');
+      if (!PROJECT_STATUS_SORT_FIELDS[key]) return;
+      const cfg = PROJECT_STATUS_SORT_FIELDS[key];
+      const defaultDir = cfg.type === 'numeric' ? 'desc' : 'asc';
+      if (_projectStatusSort.field === key) {
+        _projectStatusSort.dir = _projectStatusSort.dir === 'asc' ? 'desc' : 'asc';
+      } else {
+        _projectStatusSort.field = key;
+        _projectStatusSort.dir = defaultDir;
+      }
+      _persistProjectStatusSort();
+      // Header click is an explicit user-initiated re-sort, so drop the
+      // sticky order and let _sortProjectRows recompute.
+      _projectStatusOrder = null;
+      if (_projectStatusData) renderProjectStatus(_projectStatusData);
+    });
+  });
   if (canEditWeight) {
     body.querySelectorAll('input.project-weight-input').forEach(inp => {
       inp.addEventListener('keydown', e => {
@@ -13761,7 +13970,9 @@ async function saveProjectWeight(inp) {
     inp.value = String(original);
     return;
   }
-  if (Math.trunc(next) === Math.trunc(original) && next === original) return;
+  if (next === original) return;
+  const cell = inp.closest('.pw-cell');
+  if (cell) cell.classList.add('is-saving');
   inp.disabled = true;
   const prevBorder = inp.style.borderColor;
   inp.style.borderColor = 'var(--text-muted)';
@@ -13783,8 +13994,11 @@ async function saveProjectWeight(inp) {
     inp.style.borderColor = '#15803d';
     setTimeout(() => { inp.style.borderColor = prevBorder; }, 800);
     try { window.posthog && window.posthog.capture('project_weight_edit', { project: name, weight: next, previous: original }); } catch (er) {}
+    // Pull fresh totals + target % from the server, but preserve the row
+    // order so the just-edited row stays where the operator saw it. The
+    // order will be refreshed on header click or the ↻ refresh button.
     _projectStatusLoading = false;
-    loadProjectStatus(true);
+    loadProjectStatus(true, { preserveOrder: true });
   } catch (e) {
     inp.value = String(original);
     inp.style.borderColor = '#b91c1c';
@@ -13792,6 +14006,7 @@ async function saveProjectWeight(inp) {
     console.error('[project-weight] save error', e);
   } finally {
     inp.disabled = false;
+    if (cell) cell.classList.remove('is-saving');
   }
 }
 async function refreshAllData() {
@@ -13819,7 +14034,7 @@ async function refreshAllData() {
   loadTopDms(true);
   loadActivity();
 }
-async function loadProjectStatus(force) {
+async function loadProjectStatus(force, opts) {
   if (_projectStatusLoading) return;
   if (saAuthNotReady()) return;
   _projectStatusLoading = true;
@@ -13827,7 +14042,7 @@ async function loadProjectStatus(force) {
     const hours = currentStatusWindow().hours;
     const res = await fetch('/api/project/status?hours=' + hours);
     const data = await res.json();
-    renderProjectStatus(data);
+    renderProjectStatus(data, opts);
   } catch (e) {
     renderProjectStatus({ error: String(e && e.message || e) });
   } finally {
@@ -14333,7 +14548,10 @@ function saStartApp() {
   document.body.classList.remove('sa-authed-pending');
   const isCloud = document.body.classList.contains('sa-cloud');
   const isAdmin = window.SA_IS_ADMIN !== false;
-  if (!isAdmin) DAILY_METRICS = DAILY_METRICS.filter(m => !m.adminOnly);
+  if (!isAdmin) {
+    DAILY_METRICS = DAILY_METRICS.filter(m => !m.adminOnly);
+    RATIO_METRICS = RATIO_METRICS.filter(m => !m.adminOnly);
+  }
   try { window.posthog && window.posthog.capture('dashboard_opened', { is_admin: isAdmin, is_cloud: isCloud }); } catch (e) {}
   // Status + pending are local-only (UI hidden by body.sa-cloud). Endpoints
   // are admin-only too, so skipping them on cloud also stops 403 spam for
@@ -14353,7 +14571,10 @@ function saStartApp() {
     loadDeployHealth();
     setInterval(loadDeployHealth, 60000);
     loadProjectStatus();
-    setInterval(loadProjectStatus, 60000);
+    // Silent background polls preserve the current row order so editing one
+    // weight + waiting 60s doesn't shuffle rows under the operator. Explicit
+    // ↻ refresh and column-header clicks re-sort.
+    setInterval(() => loadProjectStatus(false, { preserveOrder: true }), 60000);
   }
   // Funnel + DM stats sections are \`<details open>\` by default; load them
   // here (post-auth) rather than in their wire IIFEs, which fire before

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "social-autoposter",
-  "version": "1.3.5",
+  "version": "1.3.6",
   "description": "Automated social posting pipeline for Reddit, X/Twitter, LinkedIn, and Moltbook. Install as a Claude Code agent skill.",
   "bin": {
     "social-autoposter": "bin/cli.js"