yt-briefing 0.12.2 → 0.12.3

package/dist/lib/yt-lib.js

@@ -87,6 +87,16 @@ export function bumpStatePointer(content, handle, type, videoId, date) {
         throw new Error(`bumpStatePointer: row for ${handle} not found`);
     return bumpStateFrontmatterDate(lines.join('\n'), date);
 }
+/**
+ * Is this video already resolved for the current run? True when the per-(channel,type)
+ * state pointer has landed on it, OR it is in the run's `seen` set. The `seen` arm is the
+ * race-proof guard: `bumpStatePointer` is not monotonic, so a detached `--fill` re-bumping
+ * a stale pointer can transiently regress it and make a pointer-only check re-emit an
+ * already-rated video. `seen` is authoritative and pointer-independent.
+ */
+export function isResolved(pointer, videoId, seen) {
+    return pointer === videoId || seen.includes(videoId);
+}
 // ---------- channel profile: durable signal writes ----------
 /**
  * Append `line` under `## <section>`, creating the section if missing (before `## Notes`

package/dist/yt-rating.js

@@ -19,7 +19,7 @@
 import { readFileSync, writeFileSync, existsSync } from 'fs';
 import { loadEnv } from "./lib/env.js";
 import { parseChannels, appendSkipTitle, appendNote, bumpStatePointer } from "./lib/yt-lib.js";
-import { CHANNELS_MD, STATE_MD, PENDING_FILE, profilePath } from "./lib/paths.js";
+import { CHANNELS_MD, STATE_MD, PENDING_FILE, QUEUE_FILE, profilePath } from "./lib/paths.js";
 loadEnv();
 function getArg(args, name) {
     const idx = args.indexOf(name);
@@ -103,6 +103,20 @@ if (!args.noState) {
         stateBumped = true;
     }
 }
+// 3. Mark the rated video resolved in the run queue (`seen`) — independent of the state
+//    pointer, so a cursor-regression race in yt-sweep (a detached `--fill` re-bumping a
+//    stale pointer) can't re-emit an already-rated video. Best-effort: the queue is a
+//    same-day throwaway and no sweep process writes it while the user rates.
+if (!args.noState && existsSync(QUEUE_FILE)) {
+    try {
+        const q = JSON.parse(readFileSync(QUEUE_FILE, 'utf8'));
+        if (q?.built_at === date && Array.isArray(q.seen) && !q.seen.includes(args.id)) {
+            q.seen.push(args.id);
+            writeFileSync(QUEUE_FILE, JSON.stringify(q), 'utf8');
+        }
+    }
+    catch { /* corrupt / foreign queue → ignore; the next sweep rebuilds it */ }
+}
 console.log(JSON.stringify({
     ok: true,
     profile: `channels/${ch.slug}.md`,

package/dist/yt-sweep.js

@@ -50,7 +50,7 @@
 import { readFileSync, writeFileSync, existsSync, rmSync, mkdirSync, renameSync, appendFileSync } from 'node:fs';
 import { spawn } from 'node:child_process';
 import { loadEnv, missingEnv, missingEnvMessage, REQUIRED_LLM, REQUIRED_YOUTUBE } from "./lib/env.js";
-import { parseChannels, parseState, bumpStatePointer } from "./lib/yt-lib.js";
+import { parseChannels, parseState, bumpStatePointer, isResolved } from "./lib/yt-lib.js";
 import { chat, getModel } from "./lib/llm.js";
 import { outputLang } from "./lib/config.js";
 import { PKG_ROOT, CHANNELS_MD, STATE_MD, CACHE_DIR, QUEUE_FILE, REST_FILE, PENDING_FILE, PREFETCH_FILE, LOG_FILE, profilePath, script, } from "./lib/paths.js";
@@ -437,8 +437,9 @@ async function advance(queue) {
         if (queue.items.length === 0)
             break;
         const item = queue.items[0];
-        // Head already resolved last round (rated/skipped → pointer landed on it) → drop.
-        if (statePointer(item) === item.videoId) {
+        // Head already resolved last round (rated/skipped → pointer landed on it, or it's in
+        // `seen`) → drop. The `seen` arm survives a --fill pointer-regression race (isResolved).
+        if (isResolved(statePointer(item), item.videoId, queue.seen)) {
             dropHead(queue);
             continue;
         }
@@ -489,7 +490,7 @@ async function runPrefetch(videoId) {
     const item = queue.items.find(i => i.videoId === videoId);
     if (!item)
         process.exit(0);
-    if (statePointer(item) === item.videoId)
+    if (isResolved(statePointer(item), item.videoId, queue.seen))
         process.exit(0); // already resolved
     if (loadPrefetch(item.videoId))
         process.exit(0); // already warm

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yt-briefing",
-  "version": "0.12.2",
+  "version": "0.12.3",
   "description": "A self-learning YouTube briefing engine: it sweeps the channels you follow, filters noise in two stages (title, then transcript), summarizes the rest in your language, and adapts to your ratings — one video at a time.",
   "type": "module",
   "bin": {