yt-briefing 1.2.2 → 1.2.4

package/README.md

@@ -137,9 +137,10 @@ The **Comment** field takes anything else. Type what you think in your own words
 shows, skip those") and press Enter: Claude turns it into a standing rule for that channel and
 infers the rating. `? your question` starts research with that question, `stop` closes the pane.
 Claude Code on a phone has no text fields, so there the pane shows the four keys only.
-Where no pane can be seen (`/yt` sent from claude.ai/code over Remote Control, or a surface that
-places no panes), the same loop runs in Claude Code's question dialog: the summary as plain text,
-OK / Weak / Research / Stop as choices, and a typed **Other** answer works as the Comment field.
+Where no pane can be seen (`/yt` sent from claude.ai/code or the phone app over Remote Control, or
+a surface that places no panes), `/yt` hands the same loop to Claude in the chat: each summary is
+Claude's message, a short dialog asks for the rating (OK / Weak / Research / Stop), and a typed
+**Other** answer works as the Comment field. This path uses the model, so it counts toward plan usage.
 
 Each step is the engine, not a chat turn: rating a video takes no turn and no context of your
 session, and the next summary is usually ready before you have finished reading the current one.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yt-briefing",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "description": "A self-learning YouTube briefing for Claude Code: it sweeps the channels you follow, filters noise in two stages (title, then transcript), gives you a short briefing of the rest in your language in a pane, and adapts to your ratings — one video at a time.",
   "type": "module",
   "bin": {

package/plugin/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "yt-briefing",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "description": "The /yt briefing loop as a Claude Code pane: summary, one-key rating, research hand-off.",
   "author": {
     "name": "Michal Ryzio"

package/plugin/hooks/register.tsx

@@ -153,64 +153,86 @@ async function comment($: EngineInterface, raw: string) {
   if (saved?.rule) $.ui.toast(`Rule saved: ${saved.rule}`)
 }
 
-/** The dialog's choices; anything else is text typed under "Other". */
-const CHOICES = ['OK', 'Weak', 'Research', 'Stop'] as const
-
-/** The dialog draws plain text: drop the briefing's Markdown marks so no `**` or `###` shows. */
-function plain(md: string): string {
-  return md
-    .replace(/^#{1,6}\s+/gm, '')
-    .replace(/\*\*(.+?)\*\*/g, '$1')
-    .replace(/^_(.+)_$/gm, '$1')
-    .replace(/^(\d+\.)\s+/gm, '$1 ')
-    .replace(/(\d{4}-\d{2}-\d{2})T[\d:.]+Z/g, '$1')
-}
-
 /** True when the person typed /yt through Remote Control and nothing attached can show a pane. */
 async function paneUnseen($: EngineInterface, origin: { kind: string } | undefined): Promise<boolean> {
   if (origin?.kind !== 'bridge') return false
   return !(await $.session.surfaces()).some(s => s !== 'terminal')
 }
 
-/** The briefing as transcript text: plain, with the filtered-out count at the end. */
-function briefing(out: SweepOut): string {
-  return [plain(out.summary ?? ''), out.skipped ? `(${out.skipped} filtered out)` : undefined]
-    .filter(line => line !== undefined).join('\n\n')
+// Where no pane can be seen the loop runs in the chat. A phone or web client draws a plugin's
+// command output and its dialogs, but not its log lines, and it folds text written mid-turn into a
+// one-line digest; only a turn's last message shows whole. So each briefing is the model's whole
+// answer to a prompt this plugin submits, and the rating dialog opens once that turn has ended.
+
+/** True while the chat loop runs; a prompt the person types ends it. */
+let looping = false
+
+/** An answer this soon after its dialog opened is a tap meant for the dialog before it. */
+const STRAY_TAP_MS = 1500
+
+const CHOICES = ['OK', 'Weak', 'Research', 'Stop'] as const
+
+/** The prompt for one briefing: the model sweeps and answers with the summary alone. */
+function fetchPrompt(reset: boolean): string {
+  const sweep = [...engine('yt-sweep'), ...(reset ? ['--reset'] : [])].join(' ')
+  return [
+    `Next video of my YouTube briefing: run \`${sweep}\`. It prints one JSON line.`,
+    'If `status` is `rating_needed`, answer with `summary` verbatim as your whole message, Markdown kept: nothing before or after it, no question, no tool call after it (the rating dialog follows on its own).',
+    'Otherwise answer with one short line saying why there is nothing to rate. Write in the language of the summaries.',
+  ].join('\n')
 }
 
-/** The rating loop in the engine's question dialog, from a sweep already run, for a session where no pane can be seen. */
-async function dialogLoop($: EngineInterface, first: SweepOut) {
-  let out = first
-  for (let shown = true; ; shown = false) {
-    if (out.status !== 'rating_needed' || !out.summary || !out.pending) return void $.ui.log(endText(out))
-
-    // The dialog draws its question in a large heading face, so the briefing goes to the
-    // transcript as text and the dialog asks only for the rating. The first is the command's output.
-    if (!shown) $.ui.log(briefing(out))
-    const question = `${out.pending.channel}: rating?`
-    let answer: string
+/** Submit a prompt once the hook that wants it has answered: one submitted from inside would wait on it. */
+function submitLater($: EngineInterface, text: string) {
+  $.clock.after(0, () => void $.prompt.submit({ text }).catch(err => {
+    looping = false
+    $.ui.toast(`Could not continue the briefing: ${String(err)}`)
+  }))
+}
+
+/** After a briefing turn: ask for the rating, record it and fetch the next one. */
+async function rateAfterTurn($: EngineInterface) {
+  let out: SweepOut
+  try {
+    out = await runSweep($, false)
+  } catch {
+    looping = false
+    return
+  }
+  if (out.status !== 'rating_needed' || !out.pending || !out.summary) {
+    looping = false
+    return
+  }
+  const title = out.pending.title.length > 80 ? `${out.pending.title.slice(0, 79)}…` : out.pending.title
+  let answer: string
+  for (;;) {
+    const opened = Date.now()
     try {
-      answer = (await $.ui.ask(question, { header: 'yt-briefing', options: CHOICES })).trim()
+      answer = (await $.ui.ask(`«${title}» — rating?`, { header: 'yt-briefing', options: CHOICES })).trim()
     } catch {
+      looping = false
       return
     }
+    if (!looping) return
+    if (Date.now() - opened >= STRAY_TAP_MS) break
+  }
 
-    if (answer === 'Stop' || answer.toLowerCase() === 'stop' || !answer) return
-    if (answer === 'Research' || answer.startsWith('?')) {
-      if (!(await record($, ['--rating', '1']))) return
-      const q = answer.startsWith('?') ? answer.slice(1).trim() || undefined : undefined
-      return handOff($, out.pending, out.summary, out.lang, q)
-    }
-    const args = answer === 'OK' ? ['--rating', '1'] : answer === 'Weak' ? ['--rating', '0'] : ['--raw-comment', answer]
-    const saved = await record($, args)
-    if (!saved) return
-    $.ui.log(saved.rule ? `Rule saved: ${saved.rule}. Looking for the next video…` : 'Saved. Looking for the next video…')
-    try {
-      out = await runSweep($, false)
-    } catch (err) {
-      return void $.ui.log(`The engine did not answer: ${String(err)}`)
-    }
+  if (answer === 'Stop' || answer.toLowerCase() === 'stop' || !answer) {
+    looping = false
+    return
+  }
+  if (answer === 'Research' || answer.startsWith('?')) {
+    looping = false
+    if (!(await record($, ['--rating', '1']))) return
+    const q = answer.startsWith('?') ? answer.slice(1).trim() || undefined : undefined
+    return handOff($, out.pending, out.summary, out.lang, q)
+  }
+  const args = answer === 'OK' ? ['--rating', '1'] : answer === 'Weak' ? ['--rating', '0'] : ['--raw-comment', answer]
+  if (!(await record($, args))) {
+    looping = false
+    return
   }
+  submitLater($, fetchPrompt(false))
 }
 
 export const register: Register = on => {
@@ -232,18 +254,20 @@ export const register: Register = on => {
       }
       await $.ui.close({ id: PANE })
     }
-    // No pane to show progress in: the command itself runs while the first sweep does, so the
-    // session shows it working, and only then hands over to the dialogs.
-    let first: SweepOut
-    try {
-      first = await runSweep($, true)
-    } catch (err) {
-      return { text: `The engine did not answer: ${String(err)}` }
-    }
-    if (first.status !== 'rating_needed') return { text: endText(first) }
-    void dialogLoop($, first)
+    looping = true
+    submitLater($, fetchPrompt(true))
+
+    return { text: 'No pane here: the briefing runs in the chat.' }
+  })
 
-    return { text: briefing(first) }
+  on('prompt.submit', async ($, e, next) => {
+    if (looping && e.origin?.kind !== 'plugin') looping = false
+    return next(e)
+  })
+
+  on('turn.complete', async ($, e, next) => {
+    if (looping && !e.agentId && e.reason === 'answer') void rateAfterTurn($)
+    return next(e)
   })
 
   on('ui.render', { component: 'Pane', requestId: PANE }, async ($, e) => {