yt-briefing 1.3.1 → 1.3.2

package/dist/yt-sweep.js

@@ -29,7 +29,8 @@
  *                                      rating) + `seen`. Foreground is the SOLE writer.
  *                                      Tagged with built_at; auto-rebuilt on a new day or via --reset.
  *   - <DATA_DIR>/.cache/queue-rest.json  background-fill handoff written by the --fill child.
- *   - <DATA_DIR>/.cache/pending.json   current ratable video's metadata for yt-rating.ts.
+ *   - <DATA_DIR>/.cache/pending.json   current ratable video's metadata for yt-rating.ts, plus its
+ *                                      emitted summary so a repeat sweep returns it unchanged.
  *   - <DATA_DIR>/.cache/prefetch.json  background-computed summary for the NEXT video.
  *
  *   All .cache/ files are throwaway (rebuilt each run; safe to delete / gitignore).
@@ -143,22 +144,30 @@ function dropHead(queue) {
     if (v)
         queue.seen.push(v.videoId);
 }
-function loadPrefetch(videoId) {
-    if (!existsSync(PREFETCH_FILE))
+/**
+ * A summary cached today for `videoId` in `file`: prefetch.json (the warmed next video) or
+ * pending.json (the head already emitted, so a repeat sweep returns it at once and unchanged).
+ */
+function loadCachedSummary(file, videoId) {
+    if (!existsSync(file))
         return null;
     try {
-        const p = JSON.parse(readFileSync(PREFETCH_FILE, 'utf8'));
-        if (p.videoId === videoId && p.built_at === today)
+        const p = JSON.parse(readFileSync(file, 'utf8'));
+        if (p.videoId === videoId && p.built_at === today && p.summary)
             return p.summary;
     }
     catch { /* corrupt → ignore */ }
     return null;
 }
+const loadPrefetch = (videoId) => loadCachedSummary(PREFETCH_FILE, videoId);
 /** Atomic write (temp + rename) so a concurrent reader never sees a partial file. */
+function writeAtomic(file, value) {
+    const tmp = `${file}.${process.pid}.tmp`;
+    writeFileSync(tmp, JSON.stringify(value));
+    renameSync(tmp, file);
+}
 function writePrefetch(p) {
-    const tmp = `${PREFETCH_FILE}.${process.pid}.tmp`;
-    writeFileSync(tmp, JSON.stringify(p));
-    renameSync(tmp, PREFETCH_FILE);
+    writeAtomic(PREFETCH_FILE, p);
 }
 function clearPrefetch() {
     if (existsSync(PREFETCH_FILE))
@@ -458,10 +467,12 @@ async function advance(queue) {
             dropHead(queue);
             continue;
         }
-        // Warm prefetch from the background child? Use it and skip the live fetch + content filter.
-        const cached = loadPrefetch(item.videoId);
+        // Already emitted (the plugin re-runs the sweep after the briefing turn to open the rating
+        // dialog) or warmed by the background child? Use it and skip the live fetch + content filter.
+        const emitted = loadCachedSummary(PENDING_FILE, item.videoId);
+        const cached = emitted ?? loadPrefetch(item.videoId);
         if (cached)
-            log(`prefetch hit ${item.videoId}`);
+            log(`${emitted ? 'pending' : 'prefetch'} hit ${item.videoId}`);
         const result = cached
             ? { kind: 'ratable', summary: cached }
             : await processItem(item);
@@ -485,10 +496,12 @@ async function advance(queue) {
             channel: item.channel, videoId: item.videoId, title: item.title,
             type: item.type, publishedAt: item.publishedAt, is_baseline: item.is_baseline,
         };
-        writeFileSync(PENDING_FILE, JSON.stringify(pending));
+        writeAtomic(PENDING_FILE, { ...pending, summary: result.summary, built_at: today });
         writeFileSync(QUEUE_FILE, JSON.stringify(queue));
-        // Warm the NEXT video in the background while the user rates this one.
-        spawnPrefetch(queue.items[1]);
+        // Warm the NEXT video in the background while the user rates this one. A repeat emit of the
+        // same head already spawned that child; a second one would summarize the next video twice.
+        if (!emitted)
+            spawnPrefetch(queue.items[1]);
         emit({ status: 'rating_needed', summary: result.summary, pending, lang: LANG });
     }
     // All processed.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yt-briefing",
-  "version": "1.3.1",
+  "version": "1.3.2",
   "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 the chat, and adapts to your ratings — one video at a time.",
   "type": "module",
   "bin": {

package/plugin/hooks/register.tsx

@@ -79,26 +79,28 @@ const STRAY_TAP_MS = 1500
 
 const CHOICES = ['OK', 'Weak', 'Research', 'Stop'] as const
 
-/** What a briefing turn must do, read from the system prompt while the loop runs. */
-function loopSection(reset: boolean): string {
-  const sweep = [...engine('yt-sweep'), ...(reset ? ['--reset'] : [])].join(' ')
+/** What the chat shows of each briefing prompt; the instructions ride along as unseen context. */
+const NEXT_TEXT = 'yt: next video'
+/** The first prompt of a /yt run: the only one that rebuilds the queue. */
+const FIRST_TEXT = 'yt: first video'
+
+/**
+ * What a briefing turn must do, read from the system prompt while the loop runs. The text is the
+ * same for every turn: a session-scoped section may be kept as first composed, so a per-turn
+ * `--reset` here once stuck to every turn and threw away the warmed next video each time.
+ */
+function loopSection(): string {
+  const sweep = engine('yt-sweep').join(' ')
   return [
-    `The yt-briefing plugin is running the person's YouTube briefing. A user message reading exactly "${NEXT_TEXT}" is its request for the next video:`,
-    `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 plugin opens the rating dialog itself).`,
+    `The yt-briefing plugin is running the person's YouTube briefing. A user message reading exactly "${FIRST_TEXT}" or "${NEXT_TEXT}" is its request for the next video:`,
+    `for "${FIRST_TEXT}" run \`${sweep} --reset\`, for "${NEXT_TEXT}" 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 plugin opens the rating dialog itself).`,
     'Otherwise answer with one short line saying why there is nothing to rate. Write in the language of the summaries.',
   ].join('\n')
 }
 
-/** What the chat shows of each briefing prompt; the instructions ride along as unseen context. */
-const NEXT_TEXT = 'yt: next video'
-
-/** Whether the next briefing prompt starts a fresh sweep. */
-let resetNext = false
-
 /** Ask for a briefing once the hook that wants it has answered: one submitted from inside would wait on it. */
 function submitLater($: EngineInterface, reset: boolean) {
-  resetNext = reset
-  $.clock.after(0, () => void $.prompt.submit({ text: NEXT_TEXT, asUser: true }).catch(err => {
+  $.clock.after(0, () => void $.prompt.submit({ text: reset ? FIRST_TEXT : NEXT_TEXT, asUser: true }).catch(err => {
     looping = false
     $.ui.toast(`Could not continue the briefing: ${String(err)}`)
   }))
@@ -176,7 +178,7 @@ export const register: Register = on => {
   on('prompt.compose', async ($, e, next) => {
     const composed = await next(e)
     if (!looping) return composed
-    return { sections: [...composed.sections, { id: 'yt-briefing:loop', text: loopSection(resetNext), scope: 'session' as const }] }
+    return { sections: [...composed.sections, { id: 'yt-briefing:loop', text: loopSection(), scope: 'session' as const }] }
   })
 
   on('turn.complete', async ($, e, next) => {