limn-review 0.1.185 → 0.1.186

package/dist/db/migrations/0169_approved_merged_since.sql

@@ -0,0 +1,6 @@
+-- MY TURN "Merged after your Limn approved" counts only merges from the moment the card type
+-- reached this install — an upgrade must not drop every recent approved-then-merged PR on My Turn.
+-- NULL (an account created later) reads as that account's own `created_at`.
+ALTER TABLE `accounts` ADD `approved_merged_since` integer;
+--> statement-breakpoint
+UPDATE `accounts` SET `approved_merged_since` = unixepoch();

package/dist/db/migrations/meta/_journal.json

@@ -1184,6 +1184,13 @@
       "when": 1792224079016,
       "tag": "0168_ai_review_seen_per_run",
       "breakpoints": true
+    },
+    {
+      "idx": 169,
+      "version": "6",
+      "when": 1792310479016,
+      "tag": "0169_approved_merged_since",
+      "breakpoints": true
     }
   ]
 }

package/dist/db/queries.js

@@ -8354,7 +8354,8 @@ repoIds) {
 /**
  * MERGED AFTER YOUR LIMN APPROVED IT — the My Turn `ai_approved_merged` section's seeds (docs/BACKEND.md
  * § My Turn). One item per merged PR in scope, merged within `APPROVED_MERGED_DAYS` of `now` and at or
- * after the repo's onboarding floor (`passesFloor`), where:
+ * after the repo's onboarding floor (`passesFloor`) AND the account's `approvedMergedSince` (going forward
+ * only — never retrospective), where:
  *   • an OWN run's auto-post LANDED a review SENT as APPROVE (`auto_post.githubReviewId` set,
  *     `verdict.submitted = 'APPROVE'`) before the merge — a pair's one review is recorded on both
  *     halves, so the earliest record wins;
@@ -8364,7 +8365,15 @@ repoIds) {
  * DB only, account-scoped. The dismissal filter is applied by `getMyTurn`.
  */
 export async function getApprovedThenMerged(accountId, repoIds, now, passesFloor) {
-    const from = new Date(now - APPROVED_MERGED_DAYS * 86_400_000);
+    // ⚠ GOING FORWARD ONLY: no merge before the card type reached this account counts
+    // (`accounts.approvedMergedSince`, NULL = the account's own `createdAt`).
+    const [acct] = await db
+        .select({ since: schema.accounts.approvedMergedSince, createdAt: schema.accounts.createdAt })
+        .from(schema.accounts)
+        .where(eq(schema.accounts.id, accountId))
+        .execute();
+    const sinceMs = (acct?.since ?? acct?.createdAt)?.getTime() ?? now;
+    const from = new Date(Math.max(now - APPROVED_MERGED_DAYS * 86_400_000, sinceMs));
     const rows = await db
         .select({
         reviewId: claudeReviews.id,

package/dist/db/schema.sqlite.js

@@ -63,6 +63,10 @@ export const accounts = sqliteTable('accounts', {
     // a positive integer the user typed (the route rejects anything else), so a reader that sees
     // null knows the user never expressed an opinion, rather than guessing at a sentinel.
     largePrCodeLocThreshold: integer('large_pr_code_loc_threshold'),
+    // MY TURN "Merged after your Limn approved" starts HERE, never retrospectively: only merges at or
+    // after this moment make a card. Migration 0169 stamps existing accounts with the upgrade time;
+    // NULL (an account created later) reads as the account's own `createdAt`.
+    approvedMergedSince: integer('approved_merged_since', { mode: 'timestamp' }),
     // BLAST RADIUS reading settings: `{sensitivity, surfacesOff[], overrides?}` — the dial, the
     // per-account surface opt-out, and (stored, honoured, no UI in v1) numeric overrides. Same
     // ACCOUNT grain and same two-state rule as the threshold above: NULL means the user has never

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "limn-review",
-  "version": "0.1.185",
+  "version": "0.1.186",
   "description": "Dashboard for tracking your team's GitHub PR activity across repos. Runs on your machine with your gh login.",
   "type": "module",
   "author": "Alex Wakeman",