target-supabase-sdk 0.2.23 → 0.2.25

package/src/shared/idea/README.md

@@ -0,0 +1,82 @@
+# Idea Links (`shared/idea`)
+
+Ideas are a **convention on `Link`**, not a new Target category (`category` stays `link`).
+
+```text
+src/shared/idea/
+  index.ts              ← browser-safe barrel ONLY (never export ./node)
+  idea.interface.ts
+  idea.schema.ts
+  idea.build.ts
+  idea.normalize.ts
+  idea.parse.ts
+  idea.query.ts
+  README.md
+  VALIDATION-MATRIX.md
+  node/                 ← Node-only (hash + temp download URL helpers)
+    index.ts
+    idea.hash.ts
+    idea.locator.ts
+```
+
+## Variants
+
+| Kind | `value` | `name` | `original.content` | `original.url` |
+|------|---------|--------|--------------------|----------------|
+| Text note | `idea` | Title (caller) | Body text | Source page URL, etc. |
+| Clipboard image | `idea` | **`idea.image`** (fixed) | SHA-256 hex of bytes | Temp download URL |
+
+Filter rows with **`value=idea`** only — do not query `details.loaderKey`. When assembling drafts, set **`loaderKey === value === "idea"`**.
+
+There is **no content-hash dedup** for image ideas.
+
+## Package entry split
+
+| Consumer import | What you get | Source |
+|-----------------|--------------|--------|
+| `from "target-supabase-sdk"` | Draft builders, schemas, parse, list filters | `shared/idea/index.ts` → `browser.ts` |
+| `from "target-supabase-sdk/node"` | + `hashIdeaPasteImage`, temp download locators | `shared/idea/node/` → `node.ts` |
+
+**Do not** re-export `./node` from `index.ts` / `browser.ts`.
+
+```typescript
+import {
+  buildIdeaLinkDraft,
+  buildIdeaImageLinkDraft,
+  ideaLinkValueFilters,
+  IDEA_LINK_VALUE,
+  IDEA_IMAGE_LINK_NAME,
+  ideaCreateSchema,
+  parseIdeaOriginal,
+} from "target-supabase-sdk";
+
+import {
+  hashIdeaPasteImage,
+  buildIdeaTempDownloadLocator,
+  buildIdeaPasteRelativePath,
+} from "target-supabase-sdk/node";
+```
+
+Quick-note (chrome extension) should wrap `buildIdeaLinkDraft` **locally** with extension tags — not via a dedicated SDK export.
+
+---
+
+## Link field rules
+
+| Field | Rule |
+|-------|------|
+| `value` | Always `idea` |
+| `details.loaderKey` | Always `idea` |
+| Text `name` | Title; max 500; must not be `idea.image` |
+| Image `name` | Fixed `idea.image` via `buildIdeaImageLinkDraft` |
+| `description` / `preview` | Default `""` |
+| `tagList` | User/system tags; image default `[ ".raw" ]` |
+
+See [VALIDATION-MATRIX.md](./VALIDATION-MATRIX.md) for limits across Zod, builders, and HTTP.
+
+---
+
+## Related
+
+- upload-service: `.cursor/skills/idea-link/SKILL.md`
+- `.cursor/skills/link-conventions/SKILL.md`

package/src/shared/idea/VALIDATION-MATRIX.md

@@ -0,0 +1,69 @@
+# Idea Link validation matrix
+
+Single reference for **where** each constraint is enforced. Limits match upload-service `POST/PATCH /v0/upload/idea` and paste-image behavior.
+
+Legend: **Z** = `ideaOriginalSchema` / `ideaCreateSchema` / `ideaUpdateSchema` (Zod); **B** = `buildIdeaLinkDraft` / `buildIdeaImageLinkDraft`; **H** = upload-service HTTP handler (Zod on request body); **P** = paste-image route (MIME, size, empty buffer).
+
+## Row identity
+
+| Rule | Z | B | H | Notes |
+|------|---|---|---|-------|
+| `category=link` | — | — | — | Set by `createTarget` / SDK category, not in draft builder |
+| `value=idea` | — | ✓ | — | Fixed in builders |
+| `loaderKey=idea` | — | ✓ | — | Same as `value` |
+| Text `name` non-empty | ✓ create | ✓ | ✓ | `ideaCreateSchema` / builder |
+| Text `name` max 500 | ✓ create/update | ✓ | ✓ | |
+| Text `name` ≠ `idea.image` | — | ✓ | — | Use `buildIdeaImageLinkDraft` for images |
+| Image `name=idea.image` | — | ✓ | — | Fixed in `buildIdeaImageLinkDraft` |
+| No title uniqueness / dedup | — | — | — | By design |
+
+## `details.original`
+
+| Field | Z | B | H | Limit / rule |
+|-------|---|---|---|----------------|
+| `content` | ✓ min 1, max 100_000 | ✓ same | ✓ create/update | Text body or image SHA-256 hex |
+| `reviewList` | ✓ max 32 items, each 1–8000 | ✓ | ✓ | Normalized (trim, dedupe) |
+| `url` | ✓ max 2000, default `""` | ✓ | ✓ | Image: temp download URL required in builder |
+| `heat` | ✓ int ≥ 0, default 0 | ✓ floor 0 | — | PATCH +1 is upload-service only |
+| Image `reviewList` | — | ✓ `[]` | — | Builder fixed |
+| Image `heat` | — | ✓ `0` | — | Builder fixed |
+
+## `tagList`
+
+| Rule | Z create | B text | B image | Notes |
+|------|----------|--------|---------|-------|
+| Max 32 tags | ✓ | ✓ | ✓ | |
+| Each tag 1–128 chars | ✓ | ✓ | — | Builder checks length on text tags |
+| Default image tags | — | — | ✓ `[ ".raw" ]` if omitted | |
+| Quick-note `.raw` | — | — | — | Extension local wrapper only |
+
+## Paste-image (bytes → Link)
+
+| Rule | P | B image | Node hash/locator |
+|------|---|---------|-------------------|
+| MIME whitelist | ✓ | — | — |
+| Non-empty buffer | ✓ | — | — |
+| Max file size | ✓ `TEMP_FS_MAX_FILE_SIZE` | — | — |
+| Optional paste `content` field | — | — | upload-service: caption → `original.content`; empty → hash |
+| Content hash (file path) | — | — | `hashIdeaPasteImage` + `buildIdeaPasteRelativePath` |
+| Stored `original.content` | ✓ min 1 | ✓ | user text or hash fallback |
+| Temp path `idea/paste/{hash}.{ext}` | — | — | `buildIdeaPasteRelativePath` |
+| `original.url` temp download | — | ✓ non-empty URL | `buildIdeaTempDownloadLocator` |
+| Content-hash dedup before create | — | — | **Not implemented** |
+
+## Query / read
+
+| Rule | SDK | upload-service |
+|------|-----|----------------|
+| Filter by `value=idea` | `ideaLinkValueFilters` | list/get/delete |
+| Image-only list | `ideaImageLinkFilters` | `linkName=idea.image` |
+| Parse row `original` | `parseIdeaOriginal` | `toIdeaView` throws if invalid |
+| Search `name` / `content` ilike | — | list API only |
+
+## Error style
+
+| Layer | Behavior |
+|-------|----------|
+| Zod (H / optional client) | Field errors in HTTP 400 |
+| `buildIdea*Draft` | `throw new Error("buildIdeaLinkDraft: …")` |
+| `parseIdeaOriginal` | Returns `null` (no throw) |

package/src/shared/idea/idea.build.ts

@@ -0,0 +1,160 @@
+import type { TargetDraft } from "../../core.interface";
+import { buildLinkTargetDraft } from "../../link/link.build";
+import type { Link } from "../../link/link.interface";
+import {
+    IDEA_IMAGE_LINK_NAME,
+    IDEA_IMAGE_TAG_RAW,
+    IDEA_LINK_VALUE,
+    type IdeaOriginal,
+    isIdeaImageLinkName,
+} from "./idea.interface";
+import { normalizeStringList } from "./idea.normalize";
+
+export interface BuildIdeaLinkDraftInput {
+    /** Article title — Link.name (not used for image ideas). */
+    name: string;
+    /** Body or image content hash — `details.original.content`. */
+    content: string;
+    reviewList?: readonly string[];
+    url?: string;
+    heat?: number;
+    tagList?: readonly string[];
+    description?: string;
+    preview?: string;
+}
+
+export interface BuildIdeaImageLinkDraftInput {
+    /** SHA-256 hex of image bytes — stored in `details.original.content`. */
+    contentHash: string;
+    /** Temp download or other absolute URL — `details.original.url`. */
+    url: string;
+    heat?: number;
+    tagList?: readonly string[];
+    description?: string;
+    preview?: string;
+}
+
+function resolveIdeaOriginal(input: {
+    content: string;
+    reviewList?: readonly string[];
+    url?: string;
+    heat?: number;
+}): IdeaOriginal {
+    const content = input.content.trim();
+    if (content === "") {
+        throw new Error("buildIdeaLinkDraft: content is empty");
+    }
+    if (content.length > 100_000) {
+        throw new Error("buildIdeaLinkDraft: content exceeds 100000 characters");
+    }
+
+    const url = input.url?.trim() ?? "";
+    if (url.length > 2000) {
+        throw new Error("buildIdeaLinkDraft: url exceeds 2000 characters");
+    }
+
+    const reviewList = normalizeStringList(input.reviewList);
+    if (reviewList.length > 32) {
+        throw new Error("buildIdeaLinkDraft: reviewList exceeds 32 entries");
+    }
+    for (const entry of reviewList) {
+        if (entry.length > 8000) {
+            throw new Error("buildIdeaLinkDraft: reviewList entry exceeds 8000 characters");
+        }
+    }
+
+    return {
+        content,
+        reviewList,
+        url,
+        heat: Math.max(0, Math.floor(input.heat ?? 0)),
+    };
+}
+
+/**
+ * Assemble a text idea {@link Link} draft (`category=link`, `loaderKey === value === "idea"`).
+ * Does not write to Supabase or enforce title uniqueness.
+ */
+export function buildIdeaLinkDraft(input: BuildIdeaLinkDraftInput): TargetDraft<Link> {
+    const name = input.name.trim();
+    if (name === "") {
+        throw new Error("buildIdeaLinkDraft: name is empty");
+    }
+    if (name.length > 500) {
+        throw new Error("buildIdeaLinkDraft: name exceeds 500 characters");
+    }
+    if (isIdeaImageLinkName(name)) {
+        throw new Error("buildIdeaLinkDraft: name is reserved for image ideas; use buildIdeaImageLinkDraft");
+    }
+
+    const tagList = normalizeStringList(input.tagList);
+    if (tagList.length > 32) {
+        throw new Error("buildIdeaLinkDraft: tagList exceeds 32 entries");
+    }
+    for (const tag of tagList) {
+        if (tag.length > 128) {
+            throw new Error("buildIdeaLinkDraft: tag exceeds 128 characters");
+        }
+    }
+
+    const original = resolveIdeaOriginal({
+        content: input.content,
+        reviewList: input.reviewList,
+        url: input.url,
+        heat: input.heat,
+    });
+
+    return buildLinkTargetDraft({
+        name,
+        value: IDEA_LINK_VALUE,
+        description: input.description ?? "",
+        preview: input.preview ?? "",
+        loaderKey: IDEA_LINK_VALUE,
+        tagList,
+        original,
+    });
+}
+
+/**
+ * Assemble a clipboard-image idea Link (`name=idea.image`, `value=idea`).
+ * Caller stores bytes and supplies hash + temp URL; no dedup by content hash.
+ */
+export function buildIdeaImageLinkDraft(input: BuildIdeaImageLinkDraftInput): TargetDraft<Link> {
+    const contentHash = input.contentHash.trim();
+    if (contentHash === "") {
+        throw new Error("buildIdeaImageLinkDraft: contentHash is empty");
+    }
+    if (contentHash.length > 100_000) {
+        throw new Error("buildIdeaImageLinkDraft: contentHash exceeds 100000 characters");
+    }
+
+    const url = input.url.trim();
+    if (url === "") {
+        throw new Error("buildIdeaImageLinkDraft: url is empty");
+    }
+    if (url.length > 2000) {
+        throw new Error("buildIdeaImageLinkDraft: url exceeds 2000 characters");
+    }
+
+    const tagList = normalizeStringList(input.tagList ?? [IDEA_IMAGE_TAG_RAW]);
+    if (tagList.length === 0) {
+        throw new Error("buildIdeaImageLinkDraft: tagList is empty");
+    }
+
+    const original: IdeaOriginal = {
+        content: contentHash,
+        reviewList: [],
+        url,
+        heat: Math.max(0, Math.floor(input.heat ?? 0)),
+    };
+
+    return buildLinkTargetDraft({
+        name: IDEA_IMAGE_LINK_NAME,
+        value: IDEA_LINK_VALUE,
+        description: input.description ?? "",
+        preview: input.preview ?? "",
+        loaderKey: IDEA_LINK_VALUE,
+        tagList,
+        original,
+    });
+}

package/src/shared/idea/idea.interface.ts

@@ -0,0 +1,34 @@
+/**
+ * Idea Link protocol — Link rows with `value` / `loaderKey` === {@link IDEA_LINK_VALUE}.
+ * Clipboard images use fixed `name` {@link IDEA_IMAGE_LINK_NAME} (not a separate value).
+ */
+
+export const IDEA_LINK_VALUE = "idea" as const;
+
+/** Discriminator stored in both `Link.value` and `details.loaderKey`. */
+export type IdeaLinkValue = typeof IDEA_LINK_VALUE;
+
+/** Link.name for clipboard-image ideas (distinct from title-based text ideas). */
+export const IDEA_IMAGE_LINK_NAME = "idea.image" as const;
+
+/** Default tag for raw captures (quick-note, paste-image). Leading `.` = system tag. */
+export const IDEA_IMAGE_TAG_RAW = ".raw" as const;
+
+/** Temp sandbox prefix for pasted images (`TEMP_FS_ROOT` on upload-service). */
+export const IDEA_IMAGE_TEMP_DIR = "idea/paste" as const;
+
+/** `Link.details.original` for idea Links (text and image). */
+export interface IdeaOriginal {
+    content: string;
+    reviewList: string[];
+    url: string;
+    heat: number;
+}
+
+export function isIdeaLinkValue(value: string): value is IdeaLinkValue {
+    return value === IDEA_LINK_VALUE;
+}
+
+export function isIdeaImageLinkName(name: string): boolean {
+    return name.trim() === IDEA_IMAGE_LINK_NAME;
+}

package/src/shared/idea/idea.normalize.ts

@@ -0,0 +1,17 @@
+/** Trim, dedupe, and drop empty strings from tag/review lists. */
+export function normalizeStringList(raw: readonly string[] | undefined | null): string[] {
+    if (raw == null || raw.length === 0) {
+        return [];
+    }
+    const seen = new Set<string>();
+    const out: string[] = [];
+    for (const item of raw) {
+        const text = item.trim();
+        if (text === "" || seen.has(text)) {
+            continue;
+        }
+        seen.add(text);
+        out.push(text);
+    }
+    return out;
+}

package/src/shared/idea/idea.parse.ts

@@ -0,0 +1,16 @@
+import type { IdeaOriginal } from "./idea.interface";
+import { normalizeStringList } from "./idea.normalize";
+import { ideaOriginalSchema } from "./idea.schema";
+
+export function parseIdeaOriginal(original: unknown): IdeaOriginal | null {
+    const parsed = ideaOriginalSchema.safeParse(original);
+    if (!parsed.success) {
+        return null;
+    }
+    return {
+        content: parsed.data.content,
+        reviewList: normalizeStringList(parsed.data.reviewList),
+        url: parsed.data.url,
+        heat: parsed.data.heat,
+    };
+}

package/src/shared/idea/idea.query.ts

@@ -0,0 +1,16 @@
+import type { QueryFilter } from "../../core.interface";
+import { CategoryLink } from "../../link/link.interface";
+import { IDEA_IMAGE_LINK_NAME, IDEA_LINK_VALUE } from "./idea.interface";
+
+/** List/get filters: `category=link` and `value=idea` (do not filter on `details.loaderKey`). */
+export function ideaLinkValueFilters(): QueryFilter[] {
+    return [
+        { field: "category", operator: "eq", value: CategoryLink.LINK },
+        { field: "value", operator: "eq", value: IDEA_LINK_VALUE },
+    ];
+}
+
+/** Clipboard-image ideas only (`name=idea.image`). */
+export function ideaImageLinkFilters(): QueryFilter[] {
+    return [...ideaLinkValueFilters(), { field: "name", operator: "eq", value: IDEA_IMAGE_LINK_NAME }];
+}

package/src/shared/idea/idea.schema.ts

@@ -0,0 +1,32 @@
+import { z } from "zod";
+
+/** Runtime shape of `Link.details.original` for value=idea. */
+export const ideaOriginalSchema = z.object({
+    content: z.string().trim().min(1).max(100_000),
+    reviewList: z.array(z.string().trim().min(1).max(8000)).max(32).default([]),
+    url: z.string().trim().max(2000).default(""),
+    heat: z.number().int().min(0).default(0),
+});
+
+const ideaTagSchema = z.string().trim().min(1).max(128);
+
+/** HTTP / API create body limits (upload-service `POST /v0/upload/idea`). */
+export const ideaCreateSchema = z.object({
+    name: z.string().trim().min(1, "name is required").max(500),
+    content: z.string().trim().min(1, "content is required").max(100_000),
+    reviewList: z.array(z.string().trim().min(1).max(8000)).max(32).default([]),
+    url: z.string().trim().max(2000).default(""),
+    tagList: z.array(ideaTagSchema).max(32).default([]),
+});
+
+export type IdeaCreateSchemaInput = z.infer<typeof ideaCreateSchema>;
+
+export const ideaUpdateSchema = z.object({
+    name: z.string().trim().min(1).max(500).optional(),
+    content: z.string().trim().min(1).max(100_000).optional(),
+    reviewList: z.array(z.string().trim().min(1).max(8000)).max(32).optional(),
+    url: z.string().trim().max(2000).optional(),
+    tagList: z.array(ideaTagSchema).max(32).optional(),
+});
+
+export type IdeaUpdateSchemaInput = z.infer<typeof ideaUpdateSchema>;

package/src/shared/idea/index.ts

@@ -0,0 +1,21 @@
+/**
+ * Idea Link — browser-safe public barrel only.
+ *
+ * Node-only code lives in `./node/` and is exported from `src/node.ts`.
+ */
+
+export type { BuildIdeaImageLinkDraftInput, BuildIdeaLinkDraftInput } from "./idea.build";
+export { buildIdeaImageLinkDraft, buildIdeaLinkDraft } from "./idea.build";
+export type { IdeaOriginal } from "./idea.interface";
+export {
+    IDEA_IMAGE_LINK_NAME,
+    IDEA_IMAGE_TAG_RAW,
+    IDEA_IMAGE_TEMP_DIR,
+    IDEA_LINK_VALUE,
+    isIdeaImageLinkName,
+} from "./idea.interface";
+export { normalizeStringList } from "./idea.normalize";
+export { parseIdeaOriginal } from "./idea.parse";
+export { ideaLinkValueFilters } from "./idea.query";
+export type { IdeaCreateSchemaInput, IdeaUpdateSchemaInput } from "./idea.schema";
+export { ideaCreateSchema, ideaUpdateSchema } from "./idea.schema";

package/src/shared/idea/node/idea.hash.ts

@@ -0,0 +1,6 @@
+import { createHash } from "node:crypto";
+
+/** SHA-256 hex of image bytes (filename + `details.original.content` for paste-image ideas). */
+export function hashIdeaPasteImage(buffer: Buffer): string {
+    return createHash("sha256").update(buffer).digest("hex");
+}

package/src/shared/idea/node/idea.locator.ts

@@ -0,0 +1,61 @@
+import { IDEA_IMAGE_TEMP_DIR } from "../idea.interface";
+
+/** Path suffix on upload-service public URL for temp file download. */
+export const IDEA_TEMP_DOWNLOAD_PATH_SUFFIX = "/v0/upload/temp/download";
+
+/** Flat path under temp sandbox: `idea/paste/{sha256}.{ext}`. */
+export function buildIdeaPasteRelativePath(contentHash: string, ext: string): string {
+    const hash = contentHash.trim();
+    const extension = ext.trim().replace(/^\./, "");
+    if (hash === "" || extension === "") {
+        throw new Error("buildIdeaPasteRelativePath: contentHash and ext are required");
+    }
+    return `${IDEA_IMAGE_TEMP_DIR}/${hash}.${extension}`;
+}
+
+function normalizeRelativeSandboxPath(path: string): string | null {
+    const normalized = path.replace(/\\/g, "/").trim();
+    if (normalized === "" || normalized.startsWith("/") || normalized.includes("..")) {
+        return null;
+    }
+    return normalized;
+}
+
+/** Build absolute locator stored in `details.original.url`. */
+export function buildIdeaTempDownloadLocator(relativePath: string, publicBaseUrl: string): string {
+    const sanitized = normalizeRelativeSandboxPath(relativePath);
+    if (sanitized == null) {
+        throw new Error("buildIdeaTempDownloadLocator: invalid relativePath");
+    }
+    const base = publicBaseUrl.trim().replace(/\/+$/, "");
+    const url = new URL(`${base}${IDEA_TEMP_DOWNLOAD_PATH_SUFFIX}`);
+    url.searchParams.set("path", sanitized);
+    return url.href;
+}
+
+/** Parse temp download URL → sandbox-relative path, or null when not a temp download URL. */
+export function parseIdeaTempDownloadLocator(locator: string): string | null {
+    const trimmed = locator.trim();
+    if (trimmed === "") {
+        return null;
+    }
+
+    let parsed: URL;
+    try {
+        parsed = new URL(trimmed, "http://localhost");
+    } catch {
+        return null;
+    }
+
+    const pathname = parsed.pathname.replace(/\\/g, "/");
+    if (!pathname.endsWith(IDEA_TEMP_DOWNLOAD_PATH_SUFFIX)) {
+        return null;
+    }
+
+    const pathParam = parsed.searchParams.get("path");
+    if (pathParam == null || pathParam.trim() === "") {
+        return null;
+    }
+
+    return normalizeRelativeSandboxPath(pathParam.trim());
+}

package/src/shared/idea/node/index.ts

@@ -0,0 +1,7 @@
+export { hashIdeaPasteImage } from "./idea.hash";
+export {
+    buildIdeaPasteRelativePath,
+    buildIdeaTempDownloadLocator,
+    IDEA_TEMP_DOWNLOAD_PATH_SUFFIX,
+    parseIdeaTempDownloadLocator,
+} from "./idea.locator";

package/src/shared/log/README.md

@@ -18,8 +18,7 @@ src/shared/log/
 
   upload/                  # shared Supabase upload pipeline (Node)
     env.ts                 # LOG_PERSIST_ENABLED, SERVICE, PROCESS, runtime dir
-    interface.ts           # LogBatch meta, loader keys, lane type
-    batch-id.ts            # computeLogBatchIdempotencyKey
+    interface.ts           # LogBatch meta, LOG_BATCH_LIST_VALUE, lane type
     flush.ts               # buildLogListDraft, postLogBatch
     hook.ts                # LogManager → spool writer offer bridge
     logger.ts              # internal pipeline loggers (no feedback loop)
@@ -58,7 +57,7 @@ App code
   → {spoolRoot}/{serviceId}/{role}/*.tmp
 
 Guard collect-log runner (1 min tick)
-  → read *.tmp → postLogBatch → rename *.json
+  → read *.tmp → postLogBatch (List value=log-batch, name=service) → rename *.json / .synced marker
   → GC old tmp/json (once per day; 7 d retention)
 
 log-service (downstream)

package/src/shared/log/core/log-batch.ts

@@ -5,8 +5,8 @@
  */
 
 import type { List } from "../../../list/list.interface";
-import type { LogEntry } from "../core/log-manager";
-import { LOG_PERSIST_LOADER_KEY, type LogBatchMeta, type LogPersistLane } from "../upload/interface";
+import { LOG_BATCH_LIST_VALUE, type LogBatchMeta, type LogPersistLane } from "../upload/interface";
+import type { LogEntry } from "./log-manager";
 
 const LANES: ReadonlySet<string> = new Set<LogPersistLane>(["fast", "medium", "slow"]);
 
@@ -26,8 +26,8 @@ export interface FlatLogEntry extends LogEntry {
     process: string;
     lane: LogPersistLane;
     pid: number;
-    /** Idempotency key of the source batch. */
-    batchId: string;
+    /** Source List row id. */
+    listId: string;
 }
 
 function asRecord(value: unknown): Record<string, unknown> | null {
@@ -61,8 +61,7 @@ export function parseLogBatchMeta(rawMeta: unknown): LogBatchMeta | null {
     const service = asString(source.service);
     const processName = asString(source.process);
     const laneRaw = asString(source.lane);
-    const idempotencyKey = asString(source.idempotencyKey);
-    if (service == null || processName == null || laneRaw == null || !LANES.has(laneRaw) || idempotencyKey == null) {
+    if (service == null || processName == null || laneRaw == null || !LANES.has(laneRaw)) {
         return null;
     }
 
@@ -75,7 +74,6 @@ export function parseLogBatchMeta(rawMeta: unknown): LogBatchMeta | null {
         from: asFiniteNumber(source.from),
         to: asFiniteNumber(source.to),
         flushedAt: asFiniteNumber(source.flushedAt) ?? 0,
-        idempotencyKey,
     };
 }
 
@@ -93,9 +91,9 @@ export function isLogEntry(value: unknown): value is LogEntry {
     );
 }
 
-/** Is this List a persisted log batch (by loaderKey)? */
+/** Is this List a persisted log batch (by `value`)? */
 export function isLogBatchList(list: List): boolean {
-    return list.details?.loaderKey === LOG_PERSIST_LOADER_KEY;
+    return list.value === LOG_BATCH_LIST_VALUE;
 }
 
 /**
@@ -106,6 +104,10 @@ export function decodeLogBatch(list: List): DecodedLogBatch | null {
     if (!isLogBatchList(list)) {
         return null;
     }
+    const loaderKey = list.details?.loaderKey;
+    if (typeof loaderKey === "string" && loaderKey.endsWith(".bad")) {
+        return null;
+    }
     const meta = parseLogBatchMeta(list.details?.meta);
     if (meta == null) {
         return null;
@@ -142,7 +144,7 @@ export function flattenLogBatch(batch: DecodedLogBatch): FlatLogEntry[] {
         process: batch.meta.process,
         lane: batch.meta.lane,
         pid: batch.meta.pid,
-        batchId: batch.meta.idempotencyKey,
+        listId: batch.listId,
     }));
 }
 

package/src/shared/log/index.ts

@@ -42,5 +42,6 @@ export {
     resolveLogData,
     withModule,
 } from "./core/log-scope";
+export { formatLogSpoolBatchTimestamp } from "./spool/file";
 export type { LogBatchMeta, LogPersistLane } from "./upload/interface";
-export { LOG_BATCH_LOADER_KEY, LOG_PERSIST_LOADER_KEY } from "./upload/interface";
+export { LOG_BATCH_LIST_VALUE } from "./upload/interface";