target-supabase-sdk 0.2.20 → 0.2.21

package/dist/node.js

@@ -6,8 +6,8 @@ export { C as CLI_PROD_FLAG, S as SERVICE_ENV_PROFILE_ENV_KEY, w as SERVICE_ENV_
 import { randomBytes, createHash } from 'node:crypto';
 import { readFile, unlink, mkdir, writeFile, rename, access, stat, readdir, constants } from 'node:fs/promises';
 import { dirname, resolve, join, isAbsolute } from 'node:path';
-import { bx as sha256Hex, bD as getErrorMessage, bE as LOG_TOPIC_NODE, a_ as postRegisterNode, aE as patchChangeNodeStatus, N as NodeStatus, aP as patchStopNode, aM as patchNodeHeartBeat, E as EMPTY_REGISTRY_SLOT_RUNTIME_STATE, bn as runRegistrySlotGuardCheck, bc as registrySlotRuntimePatchFromGuardResult, c as CategoryNode, a1 as evaluateBusyNodeLiveness, aO as patchServiceRuntime, at as isLogEntry, J as claimServiceRegistrySlot, m as ServiceRegistryError, a4 as fetchWithRetry, aw as isRetryableHttpStatus, a3 as fetchBinaryWithRetry, av as isOpaqueChunkUrl, e as CategoryRepo, aq as isHttpUrl, bF as isFeedIngestSource, bG as resolveFeedOriginalStorageProvider, b as CategoryLink, bH as buildFeedTagList, ap as isFeedValue, an as guessMediaMime, M as MEDIA_CONTENT_HASH_ALGORITHM, au as isMediaValue, ay as mediaLinkDedupFilters, B as buildMediaLinkDraft, R as ResultCode, bI as LOG_TOPIC_TASK, o as TASK_REPO_USAGE, ah as getScanRemoteRepoValues, aI as patchClaimTask, bJ as CRITICAL_UNKNOWN_TASK_TRACE_ID, aG as patchChangeTaskStatus, q as TaskStatusAction, bK as toError, b6 as postTaskSchema, bL as createTaskRow, p as TaskStatus } from './chunks/task-queue-CqmIVCBi.js';
-export { A as ApiMethod, C as CategoryConfig, a as CategoryExtraction, d as CategoryParcel, f as CategoryService, g as CategoryTask, D as DEFAULT_SYSTEM_REGISTRY_SEED_SLOTS, bM as FEED_INGEST_SOURCES, F as FEED_VALUES, L as LINK_MANIFEST_VERSION, h as LIST_MANIFEST_VERSION, i as MEDIA_CONTENT_HASH_PREFIX_LENGTH, j as MEDIA_VALUES, P as POST_TASK_DEFAULT_MAX_ATTEMPTS, k as ParcelManager, l as PostLinkValidator, S as ServiceLifecycleStatus, n as ServiceSlotStatus, T as TARGET_SYSTEM_REGISTRY_KEY, r as apiDetailsSchema, s as appendSystemRegistryEmptySlots, t as assertRegistrySlotAvailable, u as assertRegistrySlotOwner, v as buildEmptyServiceSlots, w as buildFeedLinkDraft, x as buildFeedLinkName, y as buildLinkTargetDraft, z as buildListTargetDraft, G as buildMediaNameFromLocator, H as buildReviewLinkDraft, I as buildSystemRegistryConfigDetails, K as classifyNetworkError, O as countTasksByType, Q as createActiveServiceLifecycle, U as createClaimedRegistrySlotRuntimeState, V as createStorageProviderRegistry, W as createUploadTracker, X as decodeLogBatch, Y as decodeLogBatchList, Z as defaultL3ServiceDetails, _ as deleteParcel, $ as deleteParcelSchema, a0 as escapeIlikePattern, a2 as feedLinkDedupFilters, a5 as fieldDefinitionSchema, a6 as flattenLogBatch, a7 as flattenLogBatches, a8 as formatNetworkError, a9 as getApi, aa as getApiSchema, ab as getConfig, ac as getConfigSchema, ad as getExtraction, ae as getExtractionSchema, af as getParcel, ag as getParcelSchema, ai as getScanRemoteRepoValuesSchema, aj as getService, ak as getServiceSchema, al as getTargetSystemRegistry, am as getValueAtPath, ao as installChunkFetchRegistry, ar as isLocalFilesystemPath, as as isLogBatchList, ax as linkTargetDraftBuildInputSchema, az as parcelDetailsSchema, aA as parseLogBatchMeta, aB as parseProviderPrefixedUrl, aC as parseServiceSlot, aD as parseServiceSlots, aF as patchChangeNodeStatusSchema, aH as patchChangeTaskStatusSchema, aJ as patchClaimTaskSchema, aK as patchExtractionObjects, aL as patchExtractionObjectsSchema, aN as patchNodeHeartBeatSchema, aQ as patchStopNodeSchema, aR as patchTaskProgress, aS as patchTaskProgressSchema, aT as postApi, aU as postApiSchema, aV as postExtraction, aW as postExtractionSchema, aX as postLinkCreate, aY as postParcel, aZ as postParcelSchema, a$ as postRegisterNodeSchema, b0 as postService, b1 as postServiceInstance, b2 as postServiceSchema, b3 as postSystemRegistryConfig, b4 as postSystemRegistryConfigSchema, b5 as postTask, b7 as postTaskWithRetry, b8 as publishParcel, b9 as registerProviderChunkResolver, ba as registerService, bb as registerServiceAtStartup, bd as releaseSystemRegistrySlots, be as releaseSystemRegistrySlotsByServiceId, bf as requestHasBody, bg as resetSystemRegistryConfig, bh as resetSystemRegistryConfigSchema, bi as resolveActiveRegistryServiceId, bj as resolveFetchUrl, bk as restoreParcel, bl as restoreParcelById, bm as reviewLinksForTargetFilters, bo as sanitizeFileName, bp as sanitizeParcelFileName, bq as scanExtractionList, br as scanExtractionListSchema, bs as schemaDefinitionSchema, bt as serviceDetailsSchema, bu as serviceLifecycleSchema, bv as serviceNodeSnapshotSchema, bw as serviceRuntimeSchema, by as summarizeTaskQueue, bz as systemRegistrySeedSlotSchema, bA as toPostgrestTextArrayLiteral, bB as unregisterService, bC as unregisterServiceAtShutdown } from './chunks/task-queue-CqmIVCBi.js';
+import { bx as sha256Hex, bD as getErrorMessage, bE as LOG_TOPIC_NODE, a_ as postRegisterNode, aE as patchChangeNodeStatus, N as NodeStatus, aP as patchStopNode, aM as patchNodeHeartBeat, E as EMPTY_REGISTRY_SLOT_RUNTIME_STATE, bn as runRegistrySlotGuardCheck, bc as registrySlotRuntimePatchFromGuardResult, c as CategoryNode, a1 as evaluateBusyNodeLiveness, aO as patchServiceRuntime, at as isLogEntry, J as claimServiceRegistrySlot, m as ServiceRegistryError, a4 as fetchWithRetry, aw as isRetryableHttpStatus, a3 as fetchBinaryWithRetry, av as isOpaqueChunkUrl, e as CategoryRepo, aq as isHttpUrl, bF as isFeedIngestSource, bG as resolveFeedOriginalStorageProvider, b as CategoryLink, bH as buildFeedTagList, ap as isFeedValue, an as guessMediaMime, M as MEDIA_CONTENT_HASH_ALGORITHM, au as isMediaValue, ay as mediaLinkDedupFilters, B as buildMediaLinkDraft, R as ResultCode, bI as LOG_TOPIC_TASK, o as TASK_REPO_USAGE, ah as getScanRemoteRepoValues, aI as patchClaimTask, bJ as CRITICAL_UNKNOWN_TASK_TRACE_ID, aG as patchChangeTaskStatus, q as TaskStatusAction, bK as toError, b6 as postTaskSchema, bL as createTaskRow, p as TaskStatus } from './chunks/task-queue-FygwujNS.js';
+export { A as ApiMethod, C as CategoryConfig, a as CategoryExtraction, d as CategoryParcel, f as CategoryService, g as CategoryTask, D as DEFAULT_SYSTEM_REGISTRY_SEED_SLOTS, bM as FEED_INGEST_SOURCES, F as FEED_VALUES, L as LINK_MANIFEST_VERSION, h as LIST_MANIFEST_VERSION, i as MEDIA_CONTENT_HASH_PREFIX_LENGTH, j as MEDIA_VALUES, P as POST_TASK_DEFAULT_MAX_ATTEMPTS, k as ParcelManager, l as PostLinkValidator, S as ServiceLifecycleStatus, n as ServiceSlotStatus, T as TARGET_SYSTEM_REGISTRY_KEY, r as apiDetailsSchema, s as appendSystemRegistryEmptySlots, t as assertRegistrySlotAvailable, u as assertRegistrySlotOwner, v as buildEmptyServiceSlots, w as buildFeedLinkDraft, x as buildFeedLinkName, y as buildLinkTargetDraft, z as buildListTargetDraft, G as buildMediaNameFromLocator, H as buildReviewLinkDraft, I as buildSystemRegistryConfigDetails, K as classifyNetworkError, O as countTasksByType, Q as createActiveServiceLifecycle, U as createClaimedRegistrySlotRuntimeState, V as createStorageProviderRegistry, W as createUploadTracker, X as decodeLogBatch, Y as decodeLogBatchList, Z as defaultL3ServiceDetails, _ as deleteParcel, $ as deleteParcelSchema, a0 as escapeIlikePattern, a2 as feedLinkDedupFilters, a5 as fieldDefinitionSchema, a6 as flattenLogBatch, a7 as flattenLogBatches, a8 as formatNetworkError, a9 as getApi, aa as getApiSchema, ab as getConfig, ac as getConfigSchema, ad as getExtraction, ae as getExtractionSchema, af as getParcel, ag as getParcelSchema, ai as getScanRemoteRepoValuesSchema, aj as getService, ak as getServiceSchema, al as getTargetSystemRegistry, am as getValueAtPath, ao as installChunkFetchRegistry, ar as isLocalFilesystemPath, as as isLogBatchList, ax as linkTargetDraftBuildInputSchema, az as parcelDetailsSchema, aA as parseLogBatchMeta, aB as parseProviderPrefixedUrl, aC as parseServiceSlot, aD as parseServiceSlots, aF as patchChangeNodeStatusSchema, aH as patchChangeTaskStatusSchema, aJ as patchClaimTaskSchema, aK as patchExtractionObjects, aL as patchExtractionObjectsSchema, aN as patchNodeHeartBeatSchema, aQ as patchStopNodeSchema, aR as patchTaskProgress, aS as patchTaskProgressSchema, aT as postApi, aU as postApiSchema, aV as postExtraction, aW as postExtractionSchema, aX as postLinkCreate, aY as postParcel, aZ as postParcelSchema, a$ as postRegisterNodeSchema, b0 as postService, b1 as postServiceInstance, b2 as postServiceSchema, b3 as postSystemRegistryConfig, b4 as postSystemRegistryConfigSchema, b5 as postTask, b7 as postTaskWithRetry, b8 as publishParcel, b9 as registerProviderChunkResolver, ba as registerService, bb as registerServiceAtStartup, bd as releaseSystemRegistrySlots, be as releaseSystemRegistrySlotsByServiceId, bf as requestHasBody, bg as resetSystemRegistryConfig, bh as resetSystemRegistryConfigSchema, bi as resolveActiveRegistryServiceId, bj as resolveFetchUrl, bk as restoreParcel, bl as restoreParcelById, bm as reviewLinksForNameFilters, bo as sanitizeFileName, bp as sanitizeParcelFileName, bq as scanExtractionList, br as scanExtractionListSchema, bs as schemaDefinitionSchema, bt as serviceDetailsSchema, bu as serviceLifecycleSchema, bv as serviceNodeSnapshotSchema, bw as serviceRuntimeSchema, by as summarizeTaskQueue, bz as systemRegistrySeedSlotSchema, bA as toPostgrestTextArrayLiteral, bB as unregisterService, bC as unregisterServiceAtShutdown } from './chunks/task-queue-FygwujNS.js';
 import { spawn } from 'node:child_process';
 import { ProxyAgent } from 'undici';
 import { tmpdir } from 'node:os';

package/dist/shared/review/index.d.ts

@@ -3,5 +3,5 @@
  */
 export type { BuildReviewLinkDraftInput } from "./review.build";
 export { buildReviewLinkDraft } from "./review.build";
-export { reviewLinksForTargetFilters } from "./review.query";
+export { reviewLinksForNameFilters } from "./review.query";
 //# sourceMappingURL=index.d.ts.map

package/dist/shared/review/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/shared/review/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,YAAY,EAAE,yBAAyB,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAC;AACtD,OAAO,EAAE,2BAA2B,EAAE,MAAM,gBAAgB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/shared/review/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,YAAY,EAAE,yBAAyB,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAC;AACtD,OAAO,EAAE,yBAAyB,EAAE,MAAM,gBAAgB,CAAC"}

package/dist/shared/review/review.build.d.ts

@@ -1,11 +1,8 @@
 import type { TargetDraft } from "../../core.interface";
 import type { Link } from "../../link/link.interface";
-import { type ReviewValue } from "./review.interface";
 export interface BuildReviewLinkDraftInput {
-    /** Must be {@link REVIEW_VALUE}. */
-    value: ReviewValue;
-    /** Associated Target id — written to `Link.name`. */
-    refId: string;
+    /** Written to `Link.name` (caller-defined; often a subject Target id). */
+    name: string;
     /** Interaction payload — written to `Link.details.original`. */
     original: unknown;
     tagList?: string[];
@@ -13,7 +10,7 @@ export interface BuildReviewLinkDraftInput {
 }
 /**
  * Assemble a review {@link Link} draft (`category=link`, `loaderKey === value === "review"`).
- * Does not validate `refId` existence or write to Supabase.
+ * Does not validate `name` or write to Supabase.
  */
 export declare function buildReviewLinkDraft(input: BuildReviewLinkDraftInput): TargetDraft<Link>;
 //# sourceMappingURL=review.build.d.ts.map

package/dist/shared/review/review.build.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review.build.d.ts","sourceRoot":"","sources":["../../../src/shared/review/review.build.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AAExD,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,2BAA2B,CAAC;AACtD,OAAO,EAA+B,KAAK,WAAW,EAAE,MAAM,oBAAoB,CAAC;AAEnF,MAAM,WAAW,yBAAyB;IACtC,oCAAoC;IACpC,KAAK,EAAE,WAAW,CAAC;IACnB,qDAAqD;IACrD,KAAK,EAAE,MAAM,CAAC;IACd,gEAAgE;IAChE,QAAQ,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;CACxB;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,yBAAyB,GAAG,WAAW,CAAC,IAAI,CAAC,CAkBxF"}
+{"version":3,"file":"review.build.d.ts","sourceRoot":"","sources":["../../../src/shared/review/review.build.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AAExD,OAAO,KAAK,EAAE,IAAI,EAAE,MAAM,2BAA2B,CAAC;AAGtD,MAAM,WAAW,yBAAyB;IACtC,0EAA0E;IAC1E,IAAI,EAAE,MAAM,CAAC;IACb,gEAAgE;IAChE,QAAQ,EAAE,OAAO,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,EAAE,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;CACxB;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,yBAAyB,GAAG,WAAW,CAAC,IAAI,CAAC,CAexF"}

package/dist/shared/review/review.interface.d.ts

@@ -2,8 +2,8 @@
  * Review Link protocol — Link rows with `value` / `loaderKey` === {@link REVIEW_VALUE}.
  * No separate Target category; category remains `link`.
  *
- * A Review records one user interaction with a subject Target (`Link.name` = that Target's id).
- * Many review Links may share the same subject id.
+ * A Review records one user interaction; `Link.name` is caller-defined (often a subject Target id).
+ * Many review Links may share the same `name`.
  */
 export declare const REVIEW_VALUE: "review";
 /** Discriminator stored in both `Link.value` and `details.loaderKey`. */

package/dist/shared/review/review.query.d.ts

@@ -1,7 +1,7 @@
 import type { QueryFilter } from "../../core.interface";
 /**
- * Filters for all review Links whose subject Target is `refId` (`Link.name === refId`).
+ * Filters for review Links with `Link.name === name`.
  * Many rows may match — the SDK does not define dedup for reviews.
  */
-export declare function reviewLinksForTargetFilters(refId: string): QueryFilter[];
+export declare function reviewLinksForNameFilters(name: string): QueryFilter[];
 //# sourceMappingURL=review.query.d.ts.map

package/dist/shared/review/review.query.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review.query.d.ts","sourceRoot":"","sources":["../../../src/shared/review/review.query.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AAIxD;;;GAGG;AACH,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,MAAM,GAAG,WAAW,EAAE,CAOxE"}
+{"version":3,"file":"review.query.d.ts","sourceRoot":"","sources":["../../../src/shared/review/review.query.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,sBAAsB,CAAC;AAIxD;;;GAGG;AACH,wBAAgB,yBAAyB,CAAC,IAAI,EAAE,MAAM,GAAG,WAAW,EAAE,CAOrE"}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "target-supabase-sdk",
-    "version": "0.2.20",
+    "version": "0.2.21",
     "description": "Supabase client initializer and data access APIs",
     "license": "MIT",
     "author": "KyLoc20",

package/src/browser.ts

@@ -242,7 +242,7 @@ export {
     mediaLinkDedupFilters,
 } from "./shared/media";
 export type { BuildReviewLinkDraftInput } from "./shared/review";
-export { buildReviewLinkDraft, reviewLinksForTargetFilters } from "./shared/review";
+export { buildReviewLinkDraft, reviewLinksForNameFilters } from "./shared/review";
 export { isHttpUrl, resolveFetchUrl } from "./shared/utils/fetch-url";
 export { getValueAtPath } from "./shared/utils/get-value-at-path";
 export type {

package/src/shared/review/README.md

@@ -2,7 +2,7 @@
 
 Review is a **convention on `Link`**, not a new Target category (`category` stays `link`).
 
-A **Review** records **one user interaction** with a **subject Target** (for example: open, play, complete, rate, dismiss). **Each interaction creates a new Review row.** The subject is linked by storing the Target id on `Link.name` (`refId` at create time). **Many review Links may share the same `refId`** — the SDK does not define dedup or idempotent create for reviews.
+A **Review** records **one user interaction** (for example: open, play, complete, rate, dismiss). **Each interaction creates a new Review row.** **Many review Links may share the same `Link.name`** — the SDK does not define dedup or idempotent create for reviews.
 
 All review code lives under **`src/shared/review/`**. There is no Node-only ingest layer (unlike feed/media register flows) — create drafts on any host and persist with `postTarget` / `createTarget`.
 
@@ -19,12 +19,12 @@ src/shared/review/
 
 | Consumer import | What you get |
 |-----------------|--------------|
-| `from "target-supabase-sdk"` | Draft builder, constants, list filters |
+| `from "target-supabase-sdk"` | Draft builder, list filters |
 
 ```typescript
 import {
   buildReviewLinkDraft,
-  reviewLinksForTargetFilters,
+  reviewLinksForNameFilters,
 } from "target-supabase-sdk";
 ```
 
@@ -34,12 +34,12 @@ import {
 
 | Concept | Where it lives |
 |---------|----------------|
-| Subject Target | `Link.name` — the associated Target id (`refId` when building the draft) |
+| Grouping key | `Link.name` — caller-defined (often a subject Target id, not required) |
 | Interaction body | `Link.details.original` — caller-defined JSON-serializable payload |
 | Review kind | Fixed `Link.value` / `details.loaderKey` = `"review"` |
 | Labels | Optional `tagList` (e.g. interaction type, client, experiment) |
 
-Each persisted Review is its **own** Target row (its own `id`). Repeating the same `refId` on `name` is expected when listing interaction history for one subject.
+Each persisted Review is its **own** Target row (its own `id`). Repeating the same `name` is expected when listing interaction history under one grouping key.
 
 `details.original` is intentionally **untyped** (`unknown`) in the SDK so products can store scores, dwell time, UI state, model output, etc., without a shared schema in this package.
 
@@ -51,8 +51,7 @@ Built via `buildReviewLinkDraft` → `buildLinkTargetDraft`.
 
 | Create field | Required | Link field |
 |--------------|----------|------------|
-| `value` | yes | `value` — must be `"review"` |
-| `refId` | yes | `name` — subject Target id |
+| `name` | yes | `name` |
 | `original` | yes | `details.original` |
 | `tagList` | no | `tagList` (default `[]`) |
 | `description` | no | `details.description` (default `""`) |
@@ -61,8 +60,9 @@ Fixed at build time (not on create input):
 
 | Link field | Value |
 |------------|--------|
+| `value` | `"review"` |
 | `category` | `link` |
-| `details.loaderKey` | `"review"` (same as `value`) |
+| `details.loaderKey` | `"review"` |
 | `details.manifestVersion` | `0` |
 | `details.preview` | `""` |
 
@@ -75,8 +75,7 @@ import { buildReviewLinkDraft } from "target-supabase-sdk";
 import { postTarget } from "target-supabase-sdk"; // or your create API
 
 const draft = buildReviewLinkDraft({
-  value: "review",
-  refId: subjectTargetId,
+  name: subjectTargetId,
   original: {
     action: "complete",
     progress: 1,
@@ -88,23 +87,23 @@ const draft = buildReviewLinkDraft({
 await postTarget(draft);
 ```
 
-Every interaction: build a new draft (same `refId` is fine) and insert again.
+Every interaction: build a new draft (same `name` is fine) and insert again.
 
 ---
 
 ## Query
 
-List review Links for one subject Target:
+List review Links sharing a `Link.name`:
 
 ```typescript
-import { reviewLinksForTargetFilters, scanTargetList } from "target-supabase-sdk";
+import { reviewLinksForNameFilters, scanTargetList } from "target-supabase-sdk";
 
 const reviews = await scanTargetList({
-  filters: reviewLinksForTargetFilters(subjectTargetId),
+  filters: reviewLinksForNameFilters(subjectTargetId),
 });
 ```
 
-Filters: `category=link`, `value=review`, `name=refId`. Use pagination / `scanTargetList` when history is long.
+Filters: `category=link`, `value=review`, `name=<trimmed name>`. Use pagination / `scanTargetList` when history is long.
 
 There is **no** SDK helper for dedup or “find existing review” — callers always create new rows unless they add their own policy.
 
@@ -114,7 +113,7 @@ There is **no** SDK helper for dedup or “find existing review” — callers a
 
 ```typescript
 if (link.category === "link" && link.value === "review") {
-  const subjectTargetId = link.name;
+  const groupingKey = link.name;
   const payload = link.details.original;
 }
 ```

package/src/shared/review/index.ts

@@ -4,4 +4,4 @@
 
 export type { BuildReviewLinkDraftInput } from "./review.build";
 export { buildReviewLinkDraft } from "./review.build";
-export { reviewLinksForTargetFilters } from "./review.query";
+export { reviewLinksForNameFilters } from "./review.query";

package/src/shared/review/review.build.ts

@@ -1,13 +1,11 @@
 import type { TargetDraft } from "../../core.interface";
 import { buildLinkTargetDraft } from "../../link/link.build";
 import type { Link } from "../../link/link.interface";
-import { isReviewValue, REVIEW_VALUE, type ReviewValue } from "./review.interface";
+import { REVIEW_VALUE } from "./review.interface";
 
 export interface BuildReviewLinkDraftInput {
-    /** Must be {@link REVIEW_VALUE}. */
-    value: ReviewValue;
-    /** Associated Target id — written to `Link.name`. */
-    refId: string;
+    /** Written to `Link.name` (caller-defined; often a subject Target id). */
+    name: string;
     /** Interaction payload — written to `Link.details.original`. */
     original: unknown;
     tagList?: string[];
@@ -16,19 +14,16 @@ export interface BuildReviewLinkDraftInput {
 
 /**
  * Assemble a review {@link Link} draft (`category=link`, `loaderKey === value === "review"`).
- * Does not validate `refId` existence or write to Supabase.
+ * Does not validate `name` or write to Supabase.
  */
 export function buildReviewLinkDraft(input: BuildReviewLinkDraftInput): TargetDraft<Link> {
-    if (!isReviewValue(input.value)) {
-        throw new Error(`buildReviewLinkDraft: unsupported review value: ${String(input.value)}`);
-    }
-    const refId = input.refId.trim();
-    if (refId === "") {
-        throw new Error("buildReviewLinkDraft: refId is empty");
+    const name = input.name.trim();
+    if (name === "") {
+        throw new Error("buildReviewLinkDraft: name is empty");
     }
 
     return buildLinkTargetDraft({
-        name: refId,
+        name,
         value: REVIEW_VALUE,
         description: input.description ?? "",
         preview: "",

package/src/shared/review/review.interface.ts

@@ -2,8 +2,8 @@
  * Review Link protocol — Link rows with `value` / `loaderKey` === {@link REVIEW_VALUE}.
  * No separate Target category; category remains `link`.
  *
- * A Review records one user interaction with a subject Target (`Link.name` = that Target's id).
- * Many review Links may share the same subject id.
+ * A Review records one user interaction; `Link.name` is caller-defined (often a subject Target id).
+ * Many review Links may share the same `name`.
  */
 
 export const REVIEW_VALUE = "review" as const;

package/src/shared/review/review.query.ts

@@ -3,14 +3,14 @@ import { CategoryLink } from "../../link/link.interface";
 import { REVIEW_VALUE } from "./review.interface";
 
 /**
- * Filters for all review Links whose subject Target is `refId` (`Link.name === refId`).
+ * Filters for review Links with `Link.name === name`.
  * Many rows may match — the SDK does not define dedup for reviews.
  */
-export function reviewLinksForTargetFilters(refId: string): QueryFilter[] {
-    const name = refId.trim();
+export function reviewLinksForNameFilters(name: string): QueryFilter[] {
+    const trimmed = name.trim();
     return [
         { field: "category", operator: "eq", value: CategoryLink.LINK },
         { field: "value", operator: "eq", value: REVIEW_VALUE },
-        { field: "name", operator: "eq", value: name },
+        { field: "name", operator: "eq", value: trimmed },
     ];
 }