@leftoff/cli 0.9.4 → 0.9.6

package/ui/cp-text.js

@@ -0,0 +1,43 @@
+// 0.9.5: a checkpoint's text can be markdown (a model's write-up kept as it
+// came, see salvageCheckpointReply in mod/hooks/core.js). The session page
+// renders it; cards, the digest and one-line previews show it as plain text,
+// without the ### or ** marks. Shared by the board and the daemon.
+
+// Does this read as markdown (headings, lists, bold, links, fences)?
+export function looksMd(s) {
+  return /(^|\n)\s{0,3}(#{1,6}\s|[-*+]\s+\S|\d{1,3}[.)]\s+\S|>\s|```)|\*\*[^*\n]+\*\*|__[^_\n]+__|\[[^\]\n]+\]\([^)\s]+\)/.test(String(s == null ? '' : s))
+}
+
+// Markdown → one plain line: headings end with ":", list items start with "•",
+// emphasis and link syntax go, `code` stays (the board shows it as code).
+export function stripMd(s) {
+  const text = String(s == null ? '' : s).replace(/\r\n?/g, '\n')
+  const out = []
+  let fence = false
+  for (const raw of text.split('\n')) {
+    let line = raw.trim()
+    if (/^(```|~~~)/.test(line)) { fence = !fence; continue }
+    if (!line) continue
+    if (!fence) {
+      if (/^([-*_])(\s*\1){2,}$/.test(line)) continue // a rule
+      let m
+      if ((m = line.match(/^#{1,6}\s+(.*?)\s*#*$/))) { line = inline(m[1]); if (line && !/[.:!?…]$/.test(line)) line += ':'; out.push(line); continue }
+      line = line.replace(/^>\s?/, '')
+      if ((m = line.match(/^[-*+]\s+(.*)$/))) line = '• ' + m[1]
+    }
+    out.push(fence ? line : inline(line))
+  }
+  return out.join(' ').replace(/\s+/g, ' ').trim()
+}
+function inline(t) {
+  return t
+    .replace(/!\[([^\]]*)\]\([^)]*\)/g, '$1')
+    .replace(/\[([^\]]+)\]\([^)]*\)/g, '$1')
+    .replace(/\*\*([^*]+)\*\*/g, '$1').replace(/__([^_]+)__/g, '$1')
+    .replace(/(^|[^\w*])\*([^*\s][^*]*?)\*(?![\w*])/g, '$1$2').replace(/(^|[^\w])_([^_\s][^_]*?)_(?!\w)/g, '$1$2')
+    .replace(/~~([^~]+)~~/g, '$1')
+    .trim()
+}
+
+// For one-line places: plain when it reads as markdown, as it was otherwise.
+export const plainCp = (s) => (looksMd(s) ? stripMd(s) : String(s == null ? '' : s))

package/ui/handoffs.js

@@ -19,7 +19,7 @@ export function todoRow(t, { agoFn = null, mayEdit = true } = {}) {
 }
 
 export function todoAddForm(sessionId = '') {
-  return '<form class="todo-add addform" data-todo-add="' + esc(sessionId || '-') + '" autocomplete="off"><input type="text" name="todo" maxlength="200" placeholder="Add a to-do" aria-label="Add a to-do" enterkeyhint="done"><button type="submit" class="btn sm">Add</button></form>'
+  return '<form class="todo-add addform sess-add" data-todo-add="' + esc(sessionId || '-') + '" autocomplete="off"><input type="text" name="todo" maxlength="200" placeholder="Add a to-do" aria-label="Add a to-do" enterkeyhint="done"><button type="submit" class="btn sm">Add</button></form>'
 }
 
 // Open to-dos grouped by session (your own, with no session, last).

package/ui/icons.js

@@ -45,7 +45,8 @@ export const ICONS = {
   "pin-off": "<path d=\"M12 17v5\"/> <path d=\"M15 9.34V7a1 1 0 0 1 1-1 2 2 0 0 0 0-4H7.89\"/> <path d=\"m2 2 20 20\"/> <path d=\"M9 9v1.76a2 2 0 0 1-1.11 1.79l-1.78.9A2 2 0 0 0 5 15.24V16a1 1 0 0 0 1 1h11\"/>",
   "archive": "<rect width=\"20\" height=\"5\" x=\"2\" y=\"3\" rx=\"1\"/> <path d=\"M4 8v11a2 2 0 0 0 2 2h12a2 2 0 0 0 2-2V8\"/> <path d=\"M10 12h4\"/>",
   "bookmark-plus": "<path d=\"m19 21-7-4-7 4V5a2 2 0 0 1 2-2h10a2 2 0 0 1 2 2v16z\"/> <path d=\"M12 7v6\"/> <path d=\"M15 10H9\"/>",
-  "circle-check": "<circle cx=\"12\" cy=\"12\" r=\"10\"/> <path d=\"m9 12 2 2 4-4\"/>"
+  "circle-check": "<circle cx=\"12\" cy=\"12\" r=\"10\"/> <path d=\"m9 12 2 2 4-4\"/>",
+  "message-square": "<path d=\"M21 15a2 2 0 0 1-2 2H7l-4 4V5a2 2 0 0 1 2-2h14a2 2 0 0 1 2 2z\"/>"
 }
 
 export function icon(name, cls = 'ic') {

package/ui/prs.js

@@ -60,7 +60,7 @@ export function sessionPrsSection(s, { mayEdit = true } = {}) {
         (mayEdit ? '<button type="button" class="btn sm" data-action="pr-unlink" data-id="' + esc(s.id) + '" data-url="' + esc(p.url) + '">Unlink</button>' : '') + '</div>'
     }).join('') + '</div>'
   } else h += '<div class="nocp">No pull requests linked yet. They link on their own from gh pr create, PR links in the chat, or the branch.</div>'
-  if (mayEdit) h += '<form class="prlink" data-pr-link="' + esc(s.id) + '" autocomplete="off"><input name="ref" type="text" inputmode="url" placeholder="Link a PR: paste its link or #number" aria-label="Link a PR: paste its link or #number" autocapitalize="off" spellcheck="false"><button type="submit" class="btn">Link</button></form>'
+  if (mayEdit) h += '<form class="prlink addform sess-add" data-pr-link="' + esc(s.id) + '" autocomplete="off"><input name="ref" type="text" inputmode="url" placeholder="Link a PR: paste its link or #number" aria-label="Link a PR: paste its link or #number" autocapitalize="off" spellcheck="false" enterkeyhint="done"><button type="submit" class="btn sm">Link</button></form>'
   return h
 }
 // 0.8.5: why a PR is linked, in words (the daemon sends created / you / branch / manual).

package/ui/settings-dirty.js

@@ -0,0 +1,27 @@
+// Settings → Save (0.9.5): Save is on only while the form differs from what
+// was last saved. Pure, so the daemon-free unit tests can check the compare.
+
+// Deep equality for the plain values the Settings form reads: numbers, strings,
+// booleans, null, arrays and plain objects. A key that is missing equals one
+// that is undefined; NaN equals NaN (an emptied number field reads the same twice).
+export function sameValue(a, b) {
+  if (Object.is(a, b)) return true
+  if (a === undefined || b === undefined || a === null || b === null) return false
+  if (Array.isArray(a) || Array.isArray(b)) {
+    if (!Array.isArray(a) || !Array.isArray(b) || a.length !== b.length) return false
+    return a.every((x, i) => sameValue(x, b[i]))
+  }
+  if (typeof a !== 'object' || typeof b !== 'object') return false
+  const keys = new Set([...Object.keys(a), ...Object.keys(b)])
+  for (const k of keys) if (!sameValue(a[k], b[k])) return false
+  return true
+}
+
+// The top-level fields that differ, in the order of `now` (then `saved`).
+export function changedKeys(saved, now) {
+  if (!saved || !now) return []
+  const keys = [...new Set([...Object.keys(now), ...Object.keys(saved)])]
+  return keys.filter((k) => !sameValue(saved[k], now[k]))
+}
+
+export const settingsDirty = (saved, now) => changedKeys(saved, now).length > 0

package/ui/writing.js

@@ -0,0 +1,55 @@
+// "Checkpoint now" while it's being written (0.9.5): a state that stays on the
+// card and the session page until the checkpoint lands or fails, instead of a
+// toast you can miss. Pure, so the unit tests can drive it with a fake clock.
+//
+// One entry per session id:
+//   writing  the request is out (this Mac's summarizer or the Mac it lives on)
+//   syncing  the other Mac took longer than we wait; it shows up when it syncs
+//   late     ...and still nothing after WRITING_SYNC_MS: a quiet note instead
+//   failed   it didn't work; the error stays until dismissed or the next try
+
+export const WRITING_SYNC_MS = 3 * 60 * 1000
+export const FRESH_MS = 2500
+
+export function beginWriting(map, id, { length = null, lastCpId = null, machine = null, now = Date.now() } = {}) {
+  map[id] = { phase: 'writing', length, lastCpId, machine, since: now }
+  return map[id]
+}
+
+export const isWriting = (map, id) => !!(map[id] && (map[id].phase === 'writing' || map[id].phase === 'syncing'))
+
+// The request came back: { ok: true }, { slow: true } (the other Mac is still at
+// it) or { error: 'message' }.
+export function settleWriting(map, id, outcome, now = Date.now()) {
+  const w = map[id]
+  if (!w) return null
+  if (outcome && outcome.ok) { delete map[id]; return null }
+  if (outcome && outcome.slow) { w.phase = 'syncing'; w.syncSince = now; return w }
+  w.phase = 'failed'; w.error = String((outcome && outcome.error) || 'Couldn’t write the checkpoint'); w.at = now
+  return w
+}
+
+// Read against the session as it is now. A synced checkpoint newer than the one
+// we started from ends syncing/late (returns { landed: id }); syncing too long
+// turns into late. Writing and failed don't change here.
+export function reconcileWriting(map, s, now = Date.now()) {
+  const w = s && map[s.id]
+  if (!w) return null
+  const newer = s.lastCheckpointId && s.lastCheckpointId !== w.lastCpId
+  if ((w.phase === 'syncing' || w.phase === 'late' || w.phase === 'failed') && newer) { delete map[s.id]; return { landed: s.lastCheckpointId } }
+  if (w.phase === 'syncing' && now - (w.syncSince || w.since) >= WRITING_SYNC_MS) w.phase = 'late'
+  return w
+}
+
+export function dismissWriting(map, id) { if (map[id] && (map[id].phase === 'failed' || map[id].phase === 'late')) delete map[id] }
+
+// The words shown. lengthLabel maps 'short' → 'Short'.
+export function writingText(w, lengthLabel = (k) => k) {
+  if (!w) return ''
+  const len = w.length ? ' · ' + lengthLabel(w.length) : ''
+  if (w.phase === 'writing') return 'Writing checkpoint' + len + (w.machine ? ' on ' + w.machine : '') + '…'
+  if (w.phase === 'syncing') return (w.machine || 'The other Mac') + ' is still writing it · it shows up here when it syncs…'
+  if (w.phase === 'late') return 'Not here yet · it shows up when ' + (w.machine || 'the other Mac') + ' syncs'
+  if (w.phase === 'failed') return 'Couldn’t write the checkpoint: ' + w.error
+  return ''
+}