npm - @agent-native/core - Versions diffs - 0.90.3 → 0.90.5 - Mend

@agent-native/core 0.90.3 → 0.90.5

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (256) hide show

package/dist/history/store.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"store.js","sourceRoot":"","sources":["../../src/history/store.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AACjE,OAAO,EAAE,iBAAiB,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAQ1E,IAAI,uBAAkD,CAAC;AAyBvD,MAAM,CAAC,KAAK,UAAU,2BAA2B;IAC/C,IAAI,CAAC,uBAAuB,EAAE,CAAC;QAC7B,uBAAuB,GAAG,CAAC,KAAK,IAAI,EAAE;YACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;YAC3B,MAAM,SAAS,GAAG;;;;uBAID,OAAO,EAAE;;;;;;;;;;;MAW1B,CAAC;YACD,MAAM,OAAO,GAAG;gBACd;mFAC2E;gBAC3E;gEACwD;gBACxD;2DACmD;aACpD,CAAC;YAEF,IAAI,UAAU,EAAE,EAAE,CAAC;gBACjB,MAAM,iBAAiB,CAAC,yBAAyB,EAAE,SAAS,CAAC,CAAC;gBAC9D,MAAM,iBAAiB,CACrB,sCAAsC,EACtC,OAAO,CAAC,CAAC,CAAC,CACX,CAAC;gBACF,MAAM,iBAAiB,CACrB,mCAAmC,EACnC,OAAO,CAAC,CAAC,CAAC,CACX,CAAC;gBACF,MAAM,iBAAiB,CAAC,iCAAiC,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;YACzE,CAAC;iBAAM,CAAC;gBACN,MAAM,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;gBAChC,KAAK,MAAM,QAAQ,IAAI,OAAO,EAAE,CAAC;oBAC/B,MAAM,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;gBACjC,CAAC;YACH,CAAC;QACH,CAAC,CAAC,EAAE,CAAC;IACP,CAAC;IAED,MAAM,uBAAuB,CAAC;AAChC,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,KAAiC;IAEjC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,OAAO,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QACnC,GAAG,EAAE;;kDAEyC;QAC9C,IAAI,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,KAAK,CAAC,UAAU,CAAC;KAC7C,CAAC,CAAC;IACH,MAAM,aAAa,GACjB,MAAM,CACH,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,CAAyC,EAAE,WAAW;QACrE,CAAC,CACJ,GAAG,CAAC,CAAC;IACR,MAAM,EAAE,GAAG,eAAe,EAAE,CAAC;IAC7B,MAAM,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC3C,MAAM,OAAO,GAAoB;QAC/B,EAAE;QACF,YAAY,EAAE,KAAK,CAAC,YAAY;QAChC,UAAU,EAAE,KAAK,CAAC,UAAU;QAC5B,aAAa;QACb,SAAS;QACT,SAAS,EAAE,KAAK,CAAC,SAAS,IAAI,IAAI;QAClC,SAAS,EAAE,KAAK,CAAC,SAAS,IAAI,OAAO;QACrC,UAAU,EAAE,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,SAAS,IAAI,IAAI;QACvD,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,IAAI;QAC1B,UAAU,EACR,KAAK,CAAC,UAAU,KAAK,KAAK,IAAI,KAAK,CAAC,UAAU,KAAK,QAAQ;YACzD,CAAC,CAAC,KAAK,CAAC,UAAU;YAClB,CAAC,CAAC,SAAS;QACf,KAAK,EAAE,KAAK,CAAC,KAAK,IAAI,IAAI;QAC1B,OAAO,EAAE,KAAK,CAAC,OAAO,IAAI,IAAI;QAC9B,QAAQ,EAAE,KAAK,CAAC,QAAQ;QACxB,QAAQ,EAAE,KAAK,CAAC,QAAQ,IAAI,IAAI;KACjC,CAAC;IAEF,MAAM,MAAM,CAAC,OAAO,CAAC;QACnB,GAAG,EAAE;;;;;;;;;;;;;;;wDAe+C;QACpD,IAAI,EAAE;YACJ,OAAO,CAAC,EAAE;YACV,OAAO,CAAC,YAAY;YACpB,OAAO,CAAC,UAAU;YAClB,OAAO,CAAC,aAAa;YACrB,OAAO,CAAC,SAAS;YACjB,OAAO,CAAC,SAAS;YACjB,OAAO,CAAC,SAAS;YACjB,OAAO,CAAC,UAAU;YAClB,OAAO,CAAC,KAAK;YACb,OAAO,CAAC,UAAU;YAClB,OAAO,CAAC,KAAK;YACb,OAAO,CAAC,OAAO;YACf,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,QAAQ,CAAC;YAChC,qBAAqB,CAAC,OAAO,CAAC,QAAQ,CAAC;SACxC;KACF,CAAC,CAAC;IAEH,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,KAAiC;IAEjC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,KAAK,CAAC,WAAW;QAC1C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACtC,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACtC,MAAM,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1D,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,WAAW,EAAE;;wDAEwB,MAAM;;uBAEvC;QACnB,IAAI,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,KAAK,CAAC,UAAU,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC;KACvE,CAAC,CAAC;IACH,OAAO,CAAC,MAAM,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC;AACrE,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,EAAU,EACV,KAA2B,EAC3B,OAAO,GAA8B,EAAE;IAEvC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW;QAC5C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC;IAChC,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,UAAU,EAAE;;yBAEN,MAAM;cACjB;QACV,IAAI,EAAE,CAAC,EAAE,EAAE,GAAG,MAAM,CAAC;KACtB,CAAC,CAAC;IACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7B,OAAO,GAAG,CAAC,CAAC,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/C,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,0BAA0B,CAC9C,YAAoB,EACpB,UAAkB,EAClB,aAAqB,EACrB,KAA2B,EAC3B,OAAO,GAA8B,EAAE;IAEvC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW;QAC5C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC;IAChC,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,UAAU,EAAE;;;;;cAKjB,MAAM;cACN;QACV,IAAI,EAAE,CAAC,YAAY,EAAE,UAAU,EAAE,aAAa,EAAE,GAAG,MAAM,CAAC;KAC3D,CAAC,CAAC;IACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7B,OAAO,GAAG,CAAC,CAAC,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/C,CAAC;AAED,MAAM,UAAU,0BAA0B;IACxC,uBAAuB,GAAG,SAAS,CAAC;AACtC,CAAC;AAED,SAAS,WAAW;IAClB,OAAO;QACL,IAAI;QACJ,eAAe;QACf,aAAa;QACb,gBAAgB;QAChB,YAAY;QACZ,YAAY;QACZ,YAAY;QACZ,aAAa;QACb,QAAQ;QACR,YAAY;QACZ,OAAO;QACP,SAAS;QACT,eAAe;KAChB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC;AAED,SAAS,UAAU;IACjB,OAAO,GAAG,WAAW,EAAE,iBAAiB,CAAC;AAC3C,CAAC;AAED,SAAS,oBAAoB,CAAC,KAA2B;IAIvD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,MAAM,GAAc,EAAE,CAAC;IAE7B,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;QACpB,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC9B,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;IAC/B,CAAC;IACD,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;QAChB,KAAK,CAAC,IAAI,CAAC,qCAAqC,CAAC,CAAC;QAClD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IAC3B,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;IAEpC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvB,OAAO,EAAE,MAAM,EAAE,uBAAuB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;IACzD,CAAC;IAED,OAAO,EAAE,MAAM,EAAE,IAAI,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,MAAM,EAAE,CAAC;AACvD,CAAC;AAED,SAAS,aAAa,CACpB,GAA4B,EAC5B,eAAwB;IAExB,MAAM,OAAO,GAAoB;QAC/B,EAAE,EAAE,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;QAClB,YAAY,EAAE,MAAM,CAAC,GAAG,CAAC,aAAa,CAAC;QACvC,UAAU,EAAE,MAAM,CAAC,GAAG,CAAC,WAAW,CAAC;QACnC,aAAa,EAAE,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC;QACzC,SAAS,EAAE,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC;QACjC,SAAS,EAAE,cAAc,CAAC,GAAG,CAAC,UAAU,CAAC;QACzC,SAAS,EAAE,kBAAkB,CAAC,GAAG,CAAC,UAAU,CAAC;QAC7C,UAAU,EAAE,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC;QAC3C,KAAK,EAAE,cAAc,CAAC,GAAG,CAAC,MAAM,CAAC;QACjC,UAAU,EAAE,mBAAmB,CAAC,GAAG,CAAC,UAAU,CAAC;QAC/C,KAAK,EAAE,cAAc,CAAC,GAAG,CAAC,KAAK,CAAC;QAChC,OAAO,EAAE,cAAc,CAAC,GAAG,CAAC,OAAO,CAAC;QACpC,QAAQ,EAAE,iBAAiB,CAAC,GAAG,CAAC,aAAa,CAAC;KAC/C,CAAC;IACF,IAAI,eAAe,EAAE,CAAC;QACpB,OAAO,CAAC,QAAQ,GAAG,iBAAiB,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;IAC1D,CAAC;IACD,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,eAAe;IACtB,OAAO,OAAO,UAAU,CAAC,MAAM,CAAC,UAAU,EAAE,EAAE,CAAC;AACjD,CAAC;AAED,SAAS,UAAU,CAAC,KAAyB;IAC3C,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QAC5B,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,OAAO,IAAI,CAAC,GAAG,CAAC,GAAG,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;AAC7D,CAAC;AAED,SAAS,cAAc,CAAC,KAAc;IACpC,OAAO,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AACtE,CAAC;AAED,SAAS,mBAAmB,CAAC,KAAc;IACzC,OAAO,KAAK,KAAK,KAAK,IAAI,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC;AACnE,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAc;IACxC,OAAO,KAAK,KAAK,OAAO,IAAI,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC;AACnE,CAAC;AAED,SAAS,qBAAqB,CAAC,KAAc;IAC3C,OAAO,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;AAC9E,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAc;IACvC,IAAI,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,EAAE,EAAE,CAAC;QAC1D,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,MAAM,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;IACxC,OAAO,OAAO,MAAM,KAAK,QAAQ,IAAI,MAAM,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;QAC5E,CAAC,CAAE,MAAkC;QACrC,CAAC,CAAC,IAAI,CAAC;AACX,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAc;IACvC,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;AAC3B,CAAC","sourcesContent":["import { getDbExec, intType, isPostgres } from \"../db/client.js\";\nimport { ensureIndexExists, ensureTableExists } from \"../db/ddl-guard.js\";\nimport type { Visibility } from \"../sharing/schema.js\";\nimport type {\n HistoryActorKind,\n ResourceHistoryScope,\n ResourceVersion,\n} from \"./types.js\";\n\nlet historyTableInitPromise: Promise<void> \| undefined;\n\nexport interface InsertResourceVersionInput {\n resourceType: string;\n resourceId: string;\n createdBy?: string \| null;\n actorKind?: HistoryActorKind;\n ownerEmail?: string \| null;\n orgId?: string \| null;\n visibility?: Visibility \| null;\n title?: string \| null;\n summary?: string \| null;\n snapshot: unknown;\n metadata?: Record<string, unknown> \| null;\n}\n\nexport interface QueryResourceVersionsInput {\n resourceType: string;\n resourceId: string;\n scope: ResourceHistoryScope;\n bypassScope?: boolean;\n limit?: number;\n offset?: number;\n}\n\nexport async function ensureResourceVersionsTable(): Promise<void> {\n if (!historyTableInitPromise) {\n historyTableInitPromise = (async () => {\n const client = getDbExec();\n const createSql = `CREATE TABLE IF NOT EXISTS agent_resource_versions (\n id TEXT PRIMARY KEY,\n resource_type TEXT NOT NULL,\n resource_id TEXT NOT NULL,\n version_number ${intType()} NOT NULL,\n created_at TEXT NOT NULL,\n created_by TEXT,\n actor_kind TEXT NOT NULL DEFAULT 'human',\n owner_email TEXT,\n org_id TEXT,\n visibility TEXT NOT NULL DEFAULT 'private',\n title TEXT,\n summary TEXT,\n snapshot_json TEXT NOT NULL,\n metadata_json TEXT\n )`;\n const indexes = [\n `CREATE INDEX IF NOT EXISTS idx_agent_resource_versions_resource\n ON agent_resource_versions (resource_type, resource_id, version_number)`,\n `CREATE INDEX IF NOT EXISTS idx_agent_resource_versions_owner\n ON agent_resource_versions (owner_email, created_at)`,\n `CREATE INDEX IF NOT EXISTS idx_agent_resource_versions_org\n ON agent_resource_versions (org_id, created_at)`,\n ];\n\n if (isPostgres()) {\n await ensureTableExists(\"agent_resource_versions\", createSql);\n await ensureIndexExists(\n \"idx_agent_resource_versions_resource\",\n indexes[0],\n );\n await ensureIndexExists(\n \"idx_agent_resource_versions_owner\",\n indexes[1],\n );\n await ensureIndexExists(\"idx_agent_resource_versions_org\", indexes[2]);\n } else {\n await client.execute(createSql);\n for (const indexSql of indexes) {\n await client.execute(indexSql);\n }\n }\n })();\n }\n\n await historyTableInitPromise;\n}\n\nexport async function insertResourceVersion(\n input: InsertResourceVersionInput,\n): Promise<ResourceVersion> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const maxRows = await client.execute({\n sql: `SELECT MAX(version_number) as max_version\n FROM agent_resource_versions\n WHERE resource_type = ? AND resource_id = ?`,\n args: [input.resourceType, input.resourceId],\n });\n const versionNumber =\n Number(\n (maxRows.rows?.[0] as Record<string, unknown> \| undefined)?.max_version ??\n 0,\n ) + 1;\n const id = createVersionId();\n const createdAt = new Date().toISOString();\n const version: ResourceVersion = {\n id,\n resourceType: input.resourceType,\n resourceId: input.resourceId,\n versionNumber,\n createdAt,\n createdBy: input.createdBy ?? null,\n actorKind: input.actorKind ?? \"human\",\n ownerEmail: input.ownerEmail ?? input.createdBy ?? null,\n orgId: input.orgId ?? null,\n visibility:\n input.visibility === \"org\" \|\| input.visibility === \"public\"\n ? input.visibility\n : \"private\",\n title: input.title ?? null,\n summary: input.summary ?? null,\n snapshot: input.snapshot,\n metadata: input.metadata ?? null,\n };\n\n await client.execute({\n sql: `INSERT INTO agent_resource_versions (\n id,\n resource_type,\n resource_id,\n version_number,\n created_at,\n created_by,\n actor_kind,\n owner_email,\n org_id,\n visibility,\n title,\n summary,\n snapshot_json,\n metadata_json\n ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,\n args: [\n version.id,\n version.resourceType,\n version.resourceId,\n version.versionNumber,\n version.createdAt,\n version.createdBy,\n version.actorKind,\n version.ownerEmail,\n version.orgId,\n version.visibility,\n version.title,\n version.summary,\n JSON.stringify(version.snapshot),\n stringifyOptionalJson(version.metadata),\n ],\n });\n\n return version;\n}\n\nexport async function queryResourceVersions(\n input: QueryResourceVersionsInput,\n): Promise<ResourceVersion[]> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = input.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(input.scope);\n const limit = clampLimit(input.limit);\n const offset = Math.max(0, Math.floor(input.offset ?? 0));\n const result = await client.execute({\n sql: `SELECT ${listColumns()}\n FROM agent_resource_versions\n WHERE resource_type = ? AND resource_id = ? AND ${clause}\n ORDER BY version_number DESC\n LIMIT ? OFFSET ?`,\n args: [input.resourceType, input.resourceId, ...params, limit, offset],\n });\n return (result.rows ?? []).map((row) => mapVersionRow(row, false));\n}\n\nexport async function getResourceVersionById(\n id: string,\n scope: ResourceHistoryScope,\n options: { bypassScope?: boolean } = {},\n): Promise<ResourceVersion \| null> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = options.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(scope);\n const result = await client.execute({\n sql: `SELECT ${allColumns()}\n FROM agent_resource_versions\n WHERE id = ? AND ${clause}\n LIMIT 1`,\n args: [id, ...params],\n });\n const row = result.rows?.[0];\n return row ? mapVersionRow(row, true) : null;\n}\n\nexport async function getResourceVersionByNumber(\n resourceType: string,\n resourceId: string,\n versionNumber: number,\n scope: ResourceHistoryScope,\n options: { bypassScope?: boolean } = {},\n): Promise<ResourceVersion \| null> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = options.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(scope);\n const result = await client.execute({\n sql: `SELECT ${allColumns()}\n FROM agent_resource_versions\n WHERE resource_type = ?\n AND resource_id = ?\n AND version_number = ?\n AND ${clause}\n LIMIT 1`,\n args: [resourceType, resourceId, versionNumber, ...params],\n });\n const row = result.rows?.[0];\n return row ? mapVersionRow(row, true) : null;\n}\n\nexport function __resetHistoryInitForTests(): void {\n historyTableInitPromise = undefined;\n}\n\nfunction listColumns(): string {\n return [\n \"id\",\n \"resource_type\",\n \"resource_id\",\n \"version_number\",\n \"created_at\",\n \"created_by\",\n \"actor_kind\",\n \"owner_email\",\n \"org_id\",\n \"visibility\",\n \"title\",\n \"summary\",\n \"metadata_json\",\n ].join(\", \");\n}\n\nfunction allColumns(): string {\n return `${listColumns()}, snapshot_json`;\n}\n\nfunction scopedResourceClause(scope: ResourceHistoryScope): {\n clause: string;\n params: unknown[];\n} {\n const parts: string[] = [];\n const params: unknown[] = [];\n\n if (scope.userEmail) {\n parts.push(\"owner_email = ?\");\n params.push(scope.userEmail);\n }\n if (scope.orgId) {\n parts.push(\"(visibility = 'org' AND org_id = ?)\");\n params.push(scope.orgId);\n }\n parts.push(\"visibility = 'public'\");\n\n if (parts.length === 0) {\n return { clause: \"visibility = 'public'\", params: [] };\n }\n\n return { clause: `(${parts.join(\" OR \")})`, params };\n}\n\nfunction mapVersionRow(\n row: Record<string, unknown>,\n includeSnapshot: boolean,\n): ResourceVersion {\n const version: ResourceVersion = {\n id: String(row.id),\n resourceType: String(row.resource_type),\n resourceId: String(row.resource_id),\n versionNumber: Number(row.version_number),\n createdAt: String(row.created_at),\n createdBy: nullableString(row.created_by),\n actorKind: normalizeActorKind(row.actor_kind),\n ownerEmail: nullableString(row.owner_email),\n orgId: nullableString(row.org_id),\n visibility: normalizeVisibility(row.visibility),\n title: nullableString(row.title),\n summary: nullableString(row.summary),\n metadata: parseOptionalJson(row.metadata_json),\n };\n if (includeSnapshot) {\n version.snapshot = parseRequiredJson(row.snapshot_json);\n }\n return version;\n}\n\nfunction createVersionId(): string {\n return `ver_${globalThis.crypto.randomUUID()}`;\n}\n\nfunction clampLimit(value: number \| undefined): number {\n if (!Number.isFinite(value)) {\n return 50;\n }\n return Math.min(200, Math.max(1, Math.floor(value ?? 50)));\n}\n\nfunction nullableString(value: unknown): string \| null {\n return value === null \|\| value === undefined ? null : String(value);\n}\n\nfunction normalizeVisibility(value: unknown): Visibility {\n return value === \"org\" \|\| value === \"public\" ? value : \"private\";\n}\n\nfunction normalizeActorKind(value: unknown): HistoryActorKind {\n return value === \"agent\" \|\| value === \"system\" ? value : \"human\";\n}\n\nfunction stringifyOptionalJson(value: unknown): string \| null {\n return value === null \|\| value === undefined ? null : JSON.stringify(value);\n}\n\nfunction parseOptionalJson(value: unknown): Record<string, unknown> \| null {\n if (value === null \|\| value === undefined \|\| value === \"\") {\n return null;\n }\n const parsed = parseRequiredJson(value);\n return typeof parsed === \"object\" && parsed !== null && !Array.isArray(parsed)\n ? (parsed as Record<string, unknown>)\n : null;\n}\n\nfunction parseRequiredJson(value: unknown): unknown {\n if (typeof value !== \"string\") {\n return value;\n }\n return JSON.parse(value);\n}\n"]}
1	+ {"version":3,"file":"store.js","sourceRoot":"","sources":["../../src/history/store.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,SAAS,EACT,OAAO,EACP,UAAU,EACV,iBAAiB,GAClB,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,iBAAiB,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAC;AAQ1E,IAAI,uBAAkD,CAAC;AAEvD,MAAM,2BAA2B,GAAG,CAAC,CAAC;AAyBtC,MAAM,CAAC,KAAK,UAAU,2BAA2B;IAC/C,IAAI,CAAC,uBAAuB,EAAE,CAAC;QAC7B,uBAAuB,GAAG,CAAC,KAAK,IAAI,EAAE;YACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;YAC3B,MAAM,SAAS,GAAG;;;;uBAID,OAAO,EAAE;;;;;;;;;;;MAW1B,CAAC;YACD,MAAM,OAAO,GAAG;gBACd;mFAC2E;gBAC3E;gEACwD;gBACxD;2DACmD;aACpD,CAAC;YAEF,IAAI,UAAU,EAAE,EAAE,CAAC;gBACjB,MAAM,iBAAiB,CAAC,yBAAyB,EAAE,SAAS,CAAC,CAAC;gBAC9D,MAAM,iBAAiB,CACrB,6CAA6C,EAC7C,OAAO,CAAC,CAAC,CAAC,CACX,CAAC;gBACF,MAAM,iBAAiB,CACrB,mCAAmC,EACnC,OAAO,CAAC,CAAC,CAAC,CACX,CAAC;gBACF,MAAM,iBAAiB,CAAC,iCAAiC,EAAE,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;YACzE,CAAC;iBAAM,CAAC;gBACN,MAAM,MAAM,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;gBAChC,KAAK,MAAM,QAAQ,IAAI,OAAO,EAAE,CAAC;oBAC/B,MAAM,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;gBACjC,CAAC;YACH,CAAC;QACH,CAAC,CAAC,EAAE,CAAC;IACP,CAAC;IAED,MAAM,uBAAuB,CAAC;AAChC,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,KAAiC;IAEjC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,UAAU,GACd,KAAK,CAAC,UAAU,KAAK,KAAK,IAAI,KAAK,CAAC,UAAU,KAAK,QAAQ;QACzD,CAAC,CAAC,KAAK,CAAC,UAAU;QAClB,CAAC,CAAC,SAAS,CAAC;IAChB,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,OAAO,CAAC;IAC7C,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,IAAI,CAAC;IAC1C,MAAM,UAAU,GAAG,KAAK,CAAC,UAAU,IAAI,KAAK,CAAC,SAAS,IAAI,IAAI,CAAC;IAC/D,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,IAAI,IAAI,CAAC;IAClC,MAAM,KAAK,GAAG,KAAK,CAAC,KAAK,IAAI,IAAI,CAAC;IAClC,MAAM,OAAO,GAAG,KAAK,CAAC,OAAO,IAAI,IAAI,CAAC;IACtC,MAAM,QAAQ,GAAG,KAAK,CAAC,QAAQ,IAAI,IAAI,CAAC;IACxC,MAAM,YAAY,GAAG,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACpD,MAAM,YAAY,GAAG,qBAAqB,CAAC,QAAQ,CAAC,CAAC;IAErD,IAAI,SAAkB,CAAC;IACvB,KAAK,IAAI,OAAO,GAAG,CAAC,EAAE,OAAO,GAAG,2BAA2B,EAAE,OAAO,EAAE,EAAE,CAAC;QACvE,MAAM,OAAO,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;YACnC,GAAG,EAAE;;kDAEuC;YAC5C,IAAI,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,KAAK,CAAC,UAAU,CAAC;SAC7C,CAAC,CAAC;QACH,MAAM,aAAa,GACjB,MAAM,CACH,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,CAAyC;YACxD,EAAE,WAAW,IAAI,CAAC,CACrB,GAAG,CAAC,CAAC;QACR,MAAM,EAAE,GAAG,eAAe,EAAE,CAAC;QAC7B,MAAM,SAAS,GAAG,IAAI,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;QAC3C,MAAM,OAAO,GAAoB;YAC/B,EAAE;YACF,YAAY,EAAE,KAAK,CAAC,YAAY;YAChC,UAAU,EAAE,KAAK,CAAC,UAAU;YAC5B,aAAa;YACb,SAAS;YACT,SAAS;YACT,SAAS;YACT,UAAU;YACV,KAAK;YACL,UAAU;YACV,KAAK;YACL,OAAO;YACP,QAAQ,EAAE,KAAK,CAAC,QAAQ;YACxB,QAAQ;SACT,CAAC;QAEF,IAAI,CAAC;YACH,MAAM,MAAM,CAAC,OAAO,CAAC;gBACnB,GAAG,EAAE;;;;;;;;;;;;;;;wDAe2C;gBAChD,IAAI,EAAE;oBACJ,OAAO,CAAC,EAAE;oBACV,OAAO,CAAC,YAAY;oBACpB,OAAO,CAAC,UAAU;oBAClB,OAAO,CAAC,aAAa;oBACrB,OAAO,CAAC,SAAS;oBACjB,OAAO,CAAC,SAAS;oBACjB,OAAO,CAAC,SAAS;oBACjB,OAAO,CAAC,UAAU;oBAClB,OAAO,CAAC,KAAK;oBACb,OAAO,CAAC,UAAU;oBAClB,OAAO,CAAC,KAAK;oBACb,OAAO,CAAC,OAAO;oBACf,YAAY;oBACZ,YAAY;iBACb;aACF,CAAC,CAAC;YACH,OAAO,OAAO,CAAC;QACjB,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,SAAS,GAAG,KAAK,CAAC;YAClB,IAAI,CAAC,iBAAiB,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC9B,MAAM,KAAK,CAAC;YACd,CAAC;QACH,CAAC;IACH,CAAC;IAED,MAAM,SAAS,YAAY,KAAK;QAC9B,CAAC,CAAC,SAAS;QACX,CAAC,CAAC,IAAI,KAAK,CAAC,qDAAqD,CAAC,CAAC;AACvE,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,KAAiC;IAEjC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,KAAK,CAAC,WAAW;QAC1C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACtC,MAAM,KAAK,GAAG,UAAU,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IACtC,MAAM,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1D,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,WAAW,EAAE;;wDAEwB,MAAM;;uBAEvC;QACnB,IAAI,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,KAAK,CAAC,UAAU,EAAE,GAAG,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC;KACvE,CAAC,CAAC;IACH,OAAO,CAAC,MAAM,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC,CAAC;AACrE,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,EAAU,EACV,KAA2B,EAC3B,OAAO,GAA8B,EAAE;IAEvC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW;QAC5C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC;IAChC,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,UAAU,EAAE;;yBAEN,MAAM;cACjB;QACV,IAAI,EAAE,CAAC,EAAE,EAAE,GAAG,MAAM,CAAC;KACtB,CAAC,CAAC;IACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7B,OAAO,GAAG,CAAC,CAAC,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/C,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,0BAA0B,CAC9C,YAAoB,EACpB,UAAkB,EAClB,aAAqB,EACrB,KAA2B,EAC3B,OAAO,GAA8B,EAAE;IAEvC,MAAM,2BAA2B,EAAE,CAAC;IACpC,MAAM,MAAM,GAAG,SAAS,EAAE,CAAC;IAC3B,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,WAAW;QAC5C,CAAC,CAAC,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAe,EAAE;QAC9C,CAAC,CAAC,oBAAoB,CAAC,KAAK,CAAC,CAAC;IAChC,MAAM,MAAM,GAAG,MAAM,MAAM,CAAC,OAAO,CAAC;QAClC,GAAG,EAAE,UAAU,UAAU,EAAE;;;;;cAKjB,MAAM;cACN;QACV,IAAI,EAAE,CAAC,YAAY,EAAE,UAAU,EAAE,aAAa,EAAE,GAAG,MAAM,CAAC;KAC3D,CAAC,CAAC;IACH,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IAC7B,OAAO,GAAG,CAAC,CAAC,CAAC,aAAa,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/C,CAAC;AAED,MAAM,UAAU,0BAA0B;IACxC,uBAAuB,GAAG,SAAS,CAAC;AACtC,CAAC;AAED,SAAS,WAAW;IAClB,OAAO;QACL,IAAI;QACJ,eAAe;QACf,aAAa;QACb,gBAAgB;QAChB,YAAY;QACZ,YAAY;QACZ,YAAY;QACZ,aAAa;QACb,QAAQ;QACR,YAAY;QACZ,OAAO;QACP,SAAS;QACT,eAAe;KAChB,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACf,CAAC;AAED,SAAS,UAAU;IACjB,OAAO,GAAG,WAAW,EAAE,iBAAiB,CAAC;AAC3C,CAAC;AAED,SAAS,oBAAoB,CAAC,KAA2B;IAIvD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,MAAM,MAAM,GAAc,EAAE,CAAC;IAE7B,IAAI,KAAK,CAAC,SAAS,EAAE,CAAC;QACpB,KAAK,CAAC,IAAI,CAAC,iBAAiB,CAAC,CAAC;QAC9B,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;IAC/B,CAAC;IACD,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;QAChB,KAAK,CAAC,IAAI,CAAC,qCAAqC,CAAC,CAAC;QAClD,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;IAC3B,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;IAEpC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACvB,OAAO,EAAE,MAAM,EAAE,uBAAuB,EAAE,MAAM,EAAE,EAAE,EAAE,CAAC;IACzD,CAAC;IAED,OAAO,EAAE,MAAM,EAAE,IAAI,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,EAAE,MAAM,EAAE,CAAC;AACvD,CAAC;AAED,SAAS,aAAa,CACpB,GAA4B,EAC5B,eAAwB;IAExB,MAAM,OAAO,GAAoB;QAC/B,EAAE,EAAE,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;QAClB,YAAY,EAAE,MAAM,CAAC,GAAG,CAAC,aAAa,CAAC;QACvC,UAAU,EAAE,MAAM,CAAC,GAAG,CAAC,WAAW,CAAC;QACnC,aAAa,EAAE,MAAM,CAAC,GAAG,CAAC,cAAc,CAAC;QACzC,SAAS,EAAE,MAAM,CAAC,GAAG,CAAC,UAAU,CAAC;QACjC,SAAS,EAAE,cAAc,CAAC,GAAG,CAAC,UAAU,CAAC;QACzC,SAAS,EAAE,kBAAkB,CAAC,GAAG,CAAC,UAAU,CAAC;QAC7C,UAAU,EAAE,cAAc,CAAC,GAAG,CAAC,WAAW,CAAC;QAC3C,KAAK,EAAE,cAAc,CAAC,GAAG,CAAC,MAAM,CAAC;QACjC,UAAU,EAAE,mBAAmB,CAAC,GAAG,CAAC,UAAU,CAAC;QAC/C,KAAK,EAAE,cAAc,CAAC,GAAG,CAAC,KAAK,CAAC;QAChC,OAAO,EAAE,cAAc,CAAC,GAAG,CAAC,OAAO,CAAC;QACpC,QAAQ,EAAE,iBAAiB,CAAC,GAAG,CAAC,aAAa,CAAC;KAC/C,CAAC;IACF,IAAI,eAAe,EAAE,CAAC;QACpB,OAAO,CAAC,QAAQ,GAAG,iBAAiB,CAAC,GAAG,CAAC,aAAa,CAAC,CAAC;IAC1D,CAAC;IACD,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,SAAS,eAAe;IACtB,OAAO,OAAO,UAAU,CAAC,MAAM,CAAC,UAAU,EAAE,EAAE,CAAC;AACjD,CAAC;AAED,SAAS,UAAU,CAAC,KAAyB;IAC3C,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAC;QAC5B,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,OAAO,IAAI,CAAC,GAAG,CAAC,GAAG,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;AAC7D,CAAC;AAED,SAAS,cAAc,CAAC,KAAc;IACpC,OAAO,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;AACtE,CAAC;AAED,SAAS,mBAAmB,CAAC,KAAc;IACzC,OAAO,KAAK,KAAK,KAAK,IAAI,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC;AACnE,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAc;IACxC,OAAO,KAAK,KAAK,OAAO,IAAI,KAAK,KAAK,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC;AACnE,CAAC;AAED,SAAS,qBAAqB,CAAC,KAAc;IAC3C,OAAO,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC;AAC9E,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAc;IACvC,IAAI,KAAK,KAAK,IAAI,IAAI,KAAK,KAAK,SAAS,IAAI,KAAK,KAAK,EAAE,EAAE,CAAC;QAC1D,OAAO,IAAI,CAAC;IACd,CAAC;IACD,MAAM,MAAM,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;IACxC,OAAO,OAAO,MAAM,KAAK,QAAQ,IAAI,MAAM,KAAK,IAAI,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;QAC5E,CAAC,CAAE,MAAkC;QACrC,CAAC,CAAC,IAAI,CAAC;AACX,CAAC;AAED,SAAS,iBAAiB,CAAC,KAAc;IACvC,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QAC9B,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;AAC3B,CAAC","sourcesContent":["import {\n getDbExec,\n intType,\n isPostgres,\n isUniqueViolation,\n} from \"../db/client.js\";\nimport { ensureIndexExists, ensureTableExists } from \"../db/ddl-guard.js\";\nimport type { Visibility } from \"../sharing/schema.js\";\nimport type {\n HistoryActorKind,\n ResourceHistoryScope,\n ResourceVersion,\n} from \"./types.js\";\n\nlet historyTableInitPromise: Promise<void> \| undefined;\n\nconst VERSION_INSERT_MAX_ATTEMPTS = 8;\n\nexport interface InsertResourceVersionInput {\n resourceType: string;\n resourceId: string;\n createdBy?: string \| null;\n actorKind?: HistoryActorKind;\n ownerEmail?: string \| null;\n orgId?: string \| null;\n visibility?: Visibility \| null;\n title?: string \| null;\n summary?: string \| null;\n snapshot: unknown;\n metadata?: Record<string, unknown> \| null;\n}\n\nexport interface QueryResourceVersionsInput {\n resourceType: string;\n resourceId: string;\n scope: ResourceHistoryScope;\n bypassScope?: boolean;\n limit?: number;\n offset?: number;\n}\n\nexport async function ensureResourceVersionsTable(): Promise<void> {\n if (!historyTableInitPromise) {\n historyTableInitPromise = (async () => {\n const client = getDbExec();\n const createSql = `CREATE TABLE IF NOT EXISTS agent_resource_versions (\n id TEXT PRIMARY KEY,\n resource_type TEXT NOT NULL,\n resource_id TEXT NOT NULL,\n version_number ${intType()} NOT NULL,\n created_at TEXT NOT NULL,\n created_by TEXT,\n actor_kind TEXT NOT NULL DEFAULT 'human',\n owner_email TEXT,\n org_id TEXT,\n visibility TEXT NOT NULL DEFAULT 'private',\n title TEXT,\n summary TEXT,\n snapshot_json TEXT NOT NULL,\n metadata_json TEXT\n )`;\n const indexes = [\n `CREATE UNIQUE INDEX IF NOT EXISTS idx_agent_resource_versions_resource_number\n ON agent_resource_versions (resource_type, resource_id, version_number)`,\n `CREATE INDEX IF NOT EXISTS idx_agent_resource_versions_owner\n ON agent_resource_versions (owner_email, created_at)`,\n `CREATE INDEX IF NOT EXISTS idx_agent_resource_versions_org\n ON agent_resource_versions (org_id, created_at)`,\n ];\n\n if (isPostgres()) {\n await ensureTableExists(\"agent_resource_versions\", createSql);\n await ensureIndexExists(\n \"idx_agent_resource_versions_resource_number\",\n indexes[0],\n );\n await ensureIndexExists(\n \"idx_agent_resource_versions_owner\",\n indexes[1],\n );\n await ensureIndexExists(\"idx_agent_resource_versions_org\", indexes[2]);\n } else {\n await client.execute(createSql);\n for (const indexSql of indexes) {\n await client.execute(indexSql);\n }\n }\n })();\n }\n\n await historyTableInitPromise;\n}\n\nexport async function insertResourceVersion(\n input: InsertResourceVersionInput,\n): Promise<ResourceVersion> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const visibility =\n input.visibility === \"org\" \|\| input.visibility === \"public\"\n ? input.visibility\n : \"private\";\n const actorKind = input.actorKind ?? \"human\";\n const createdBy = input.createdBy ?? null;\n const ownerEmail = input.ownerEmail ?? input.createdBy ?? null;\n const orgId = input.orgId ?? null;\n const title = input.title ?? null;\n const summary = input.summary ?? null;\n const metadata = input.metadata ?? null;\n const snapshotJson = JSON.stringify(input.snapshot);\n const metadataJson = stringifyOptionalJson(metadata);\n\n let lastError: unknown;\n for (let attempt = 0; attempt < VERSION_INSERT_MAX_ATTEMPTS; attempt++) {\n const maxRows = await client.execute({\n sql: `SELECT MAX(version_number) as max_version\n FROM agent_resource_versions\n WHERE resource_type = ? AND resource_id = ?`,\n args: [input.resourceType, input.resourceId],\n });\n const versionNumber =\n Number(\n (maxRows.rows?.[0] as Record<string, unknown> \| undefined)\n ?.max_version ?? 0,\n ) + 1;\n const id = createVersionId();\n const createdAt = new Date().toISOString();\n const version: ResourceVersion = {\n id,\n resourceType: input.resourceType,\n resourceId: input.resourceId,\n versionNumber,\n createdAt,\n createdBy,\n actorKind,\n ownerEmail,\n orgId,\n visibility,\n title,\n summary,\n snapshot: input.snapshot,\n metadata,\n };\n\n try {\n await client.execute({\n sql: `INSERT INTO agent_resource_versions (\n id,\n resource_type,\n resource_id,\n version_number,\n created_at,\n created_by,\n actor_kind,\n owner_email,\n org_id,\n visibility,\n title,\n summary,\n snapshot_json,\n metadata_json\n ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,\n args: [\n version.id,\n version.resourceType,\n version.resourceId,\n version.versionNumber,\n version.createdAt,\n version.createdBy,\n version.actorKind,\n version.ownerEmail,\n version.orgId,\n version.visibility,\n version.title,\n version.summary,\n snapshotJson,\n metadataJson,\n ],\n });\n return version;\n } catch (error) {\n lastError = error;\n if (!isUniqueViolation(error)) {\n throw error;\n }\n }\n }\n\n throw lastError instanceof Error\n ? lastError\n : new Error(\"Failed to allocate a unique resource version number\");\n}\n\nexport async function queryResourceVersions(\n input: QueryResourceVersionsInput,\n): Promise<ResourceVersion[]> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = input.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(input.scope);\n const limit = clampLimit(input.limit);\n const offset = Math.max(0, Math.floor(input.offset ?? 0));\n const result = await client.execute({\n sql: `SELECT ${listColumns()}\n FROM agent_resource_versions\n WHERE resource_type = ? AND resource_id = ? AND ${clause}\n ORDER BY version_number DESC\n LIMIT ? OFFSET ?`,\n args: [input.resourceType, input.resourceId, ...params, limit, offset],\n });\n return (result.rows ?? []).map((row) => mapVersionRow(row, false));\n}\n\nexport async function getResourceVersionById(\n id: string,\n scope: ResourceHistoryScope,\n options: { bypassScope?: boolean } = {},\n): Promise<ResourceVersion \| null> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = options.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(scope);\n const result = await client.execute({\n sql: `SELECT ${allColumns()}\n FROM agent_resource_versions\n WHERE id = ? AND ${clause}\n LIMIT 1`,\n args: [id, ...params],\n });\n const row = result.rows?.[0];\n return row ? mapVersionRow(row, true) : null;\n}\n\nexport async function getResourceVersionByNumber(\n resourceType: string,\n resourceId: string,\n versionNumber: number,\n scope: ResourceHistoryScope,\n options: { bypassScope?: boolean } = {},\n): Promise<ResourceVersion \| null> {\n await ensureResourceVersionsTable();\n const client = getDbExec();\n const { clause, params } = options.bypassScope\n ? { clause: \"1 = 1\", params: [] as unknown[] }\n : scopedResourceClause(scope);\n const result = await client.execute({\n sql: `SELECT ${allColumns()}\n FROM agent_resource_versions\n WHERE resource_type = ?\n AND resource_id = ?\n AND version_number = ?\n AND ${clause}\n LIMIT 1`,\n args: [resourceType, resourceId, versionNumber, ...params],\n });\n const row = result.rows?.[0];\n return row ? mapVersionRow(row, true) : null;\n}\n\nexport function __resetHistoryInitForTests(): void {\n historyTableInitPromise = undefined;\n}\n\nfunction listColumns(): string {\n return [\n \"id\",\n \"resource_type\",\n \"resource_id\",\n \"version_number\",\n \"created_at\",\n \"created_by\",\n \"actor_kind\",\n \"owner_email\",\n \"org_id\",\n \"visibility\",\n \"title\",\n \"summary\",\n \"metadata_json\",\n ].join(\", \");\n}\n\nfunction allColumns(): string {\n return `${listColumns()}, snapshot_json`;\n}\n\nfunction scopedResourceClause(scope: ResourceHistoryScope): {\n clause: string;\n params: unknown[];\n} {\n const parts: string[] = [];\n const params: unknown[] = [];\n\n if (scope.userEmail) {\n parts.push(\"owner_email = ?\");\n params.push(scope.userEmail);\n }\n if (scope.orgId) {\n parts.push(\"(visibility = 'org' AND org_id = ?)\");\n params.push(scope.orgId);\n }\n parts.push(\"visibility = 'public'\");\n\n if (parts.length === 0) {\n return { clause: \"visibility = 'public'\", params: [] };\n }\n\n return { clause: `(${parts.join(\" OR \")})`, params };\n}\n\nfunction mapVersionRow(\n row: Record<string, unknown>,\n includeSnapshot: boolean,\n): ResourceVersion {\n const version: ResourceVersion = {\n id: String(row.id),\n resourceType: String(row.resource_type),\n resourceId: String(row.resource_id),\n versionNumber: Number(row.version_number),\n createdAt: String(row.created_at),\n createdBy: nullableString(row.created_by),\n actorKind: normalizeActorKind(row.actor_kind),\n ownerEmail: nullableString(row.owner_email),\n orgId: nullableString(row.org_id),\n visibility: normalizeVisibility(row.visibility),\n title: nullableString(row.title),\n summary: nullableString(row.summary),\n metadata: parseOptionalJson(row.metadata_json),\n };\n if (includeSnapshot) {\n version.snapshot = parseRequiredJson(row.snapshot_json);\n }\n return version;\n}\n\nfunction createVersionId(): string {\n return `ver_${globalThis.crypto.randomUUID()}`;\n}\n\nfunction clampLimit(value: number \| undefined): number {\n if (!Number.isFinite(value)) {\n return 50;\n }\n return Math.min(200, Math.max(1, Math.floor(value ?? 50)));\n}\n\nfunction nullableString(value: unknown): string \| null {\n return value === null \|\| value === undefined ? null : String(value);\n}\n\nfunction normalizeVisibility(value: unknown): Visibility {\n return value === \"org\" \|\| value === \"public\" ? value : \"private\";\n}\n\nfunction normalizeActorKind(value: unknown): HistoryActorKind {\n return value === \"agent\" \|\| value === \"system\" ? value : \"human\";\n}\n\nfunction stringifyOptionalJson(value: unknown): string \| null {\n return value === null \|\| value === undefined ? null : JSON.stringify(value);\n}\n\nfunction parseOptionalJson(value: unknown): Record<string, unknown> \| null {\n if (value === null \|\| value === undefined \|\| value === \"\") {\n return null;\n }\n const parsed = parseRequiredJson(value);\n return typeof parsed === \"object\" && parsed !== null && !Array.isArray(parsed)\n ? (parsed as Record<string, unknown>)\n : null;\n}\n\nfunction parseRequiredJson(value: unknown): unknown {\n if (typeof value !== \"string\") {\n return value;\n }\n return JSON.parse(value);\n}\n"]}

package/dist/notifications/routes.d.ts CHANGED Viewed

@@ -28,7 +28,7 @@ export declare function createNotificationsHandler(): import("h3").EventHandlerW
 } | {
     count?: undefined;
     updated?: undefined;
-    ok: boolean;
     error?: undefined;
+    ok: boolean;
 }>>;
 //# sourceMappingURL=routes.d.ts.map

package/dist/resources/handlers.d.ts CHANGED Viewed

@@ -49,11 +49,11 @@ export declare function handleUpdateResource(event: any): Promise<import("./stor
 }>;
 /** DELETE /_agent-native/resources/:id — delete a resource */
 export declare function handleDeleteResource(event: any): Promise<{
-    error: string;
     ok?: undefined;
+    error: string;
 } | {
-    ok: boolean;
     error?: undefined;
+    ok: boolean;
 }>;
 /** POST /_agent-native/resources/upload — upload a file as a resource */
 export declare function handleUploadResource(event: any): Promise<import("./store.js").Resource | {
@@ -73,9 +73,9 @@ export declare function handleUploadResource(event: any): Promise<import("./stor
     runId: string | null;
     expiresAt: number | null;
     metadata: string | null;
+    error?: undefined;
     url: string;
     provider: string;
-    error?: undefined;
 }>;
 export {};
 //# sourceMappingURL=handlers.d.ts.map

package/dist/review/actions/consume-review-feedback.d.ts CHANGED Viewed

@@ -4,6 +4,7 @@ declare const _default: import("../../action.js").ActionDefinition<{
     commentIds: string[];
 }, {
     consumedCommentIds: string[];
+    updatedCount: number;
 }>;
 export default _default;
 //# sourceMappingURL=consume-review-feedback.d.ts.map

package/dist/review/actions/consume-review-feedback.js CHANGED Viewed

@@ -13,11 +13,17 @@ export default defineAction({
     run: async (args, ctx) => {
         const actionCtx = ctx;
         await assertReviewableResourceAccess(args.resourceType, args.resourceId, actionCtx, "editor");
-        await consumeReviewFeedback(args.commentIds, undefined, {
+        const updatedCount = await consumeReviewFeedback(args.commentIds, undefined, {
             resourceType: args.resourceType,
             resourceId: args.resourceId,
         });
-        return { consumedCommentIds: args.commentIds };
+        if (updatedCount < 1) {
+            throw new Error("No matching review comments to consume");
+        }
+        return {
+            consumedCommentIds: args.commentIds,
+            updatedCount,
+        };
     },
     audit: {
         target: (args) => ({

package/dist/review/actions/consume-review-feedback.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"consume-review-feedback.js","sourceRoot":"","sources":["../../../src/review/actions/consume-review-feedback.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,8BAA8B,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AAGpD,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,UAAU,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;CAC9C,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EACT,qEAAqE;IACvE,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,8BAA8B,CAClC,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,EACT,QAAQ,CACT,CAAC;QACF,MAAM,qBAAqB,~~CAAC~~,IAAI,CAAC,UAAU,~~EAAE~~,SAAS,~~EAAE~~;~~YACtD~~,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,UAAU,EAAE,IAAI,CAAC,UAAU;SAC5B,CAAC,CAAC;~~QACH~~,OAAO,~~EAAE,~~kBAAkB,EAAE,IAAI,CAAC,UAAU,~~EAAE~~,CAAC;~~IACjD~~,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { assertReviewableResourceAccess } from \"../registry.js\";\nimport { consumeReviewFeedback } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n commentIds: z.array(z.string().min(1)).min(1),\n});\n\nexport default defineAction({\n description:\n \"Mark review feedback as consumed by the agent or reviewer workflow.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n await assertReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n \"editor\",\n );\n await consumeReviewFeedback(args.commentIds, undefined, {\n resourceType: args.resourceType,\n resourceId: args.resourceId,\n });\n return { consumedCommentIds: args.commentIds };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n"]}
1	+ {"version":3,"file":"consume-review-feedback.js","sourceRoot":"","sources":["../../../src/review/actions/consume-review-feedback.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,8BAA8B,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAC;AAGpD,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,UAAU,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;CAC9C,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EACT,qEAAqE;IACvE,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,8BAA8B,CAClC,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,EACT,QAAQ,CACT,CAAC;QACF,MAAM,YAAY,GAAG,MAAM,qBAAqB,CAC9C,IAAI,CAAC,UAAU,EACf,SAAS,EACT;YACE,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,UAAU,EAAE,IAAI,CAAC,UAAU;SAC5B,CACF,CAAC;QACF,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC;YACrB,MAAM,IAAI,KAAK,CAAC,wCAAwC,CAAC,CAAC;QAC5D,CAAC;QACD,OAAO;YACL,kBAAkB,EAAE,IAAI,CAAC,UAAU;YACnC,YAAY;SACb,CAAC;IACJ,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { assertReviewableResourceAccess } from \"../registry.js\";\nimport { consumeReviewFeedback } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n commentIds: z.array(z.string().min(1)).min(1),\n});\n\nexport default defineAction({\n description:\n \"Mark review feedback as consumed by the agent or reviewer workflow.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n await assertReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n \"editor\",\n );\n const updatedCount = await consumeReviewFeedback(\n args.commentIds,\n undefined,\n {\n resourceType: args.resourceType,\n resourceId: args.resourceId,\n },\n );\n if (updatedCount < 1) {\n throw new Error(\"No matching review comments to consume\");\n }\n return {\n consumedCommentIds: args.commentIds,\n updatedCount,\n };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n"]}

package/dist/review/actions/delete-review-comment.d.ts CHANGED Viewed

@@ -4,7 +4,8 @@ declare const _default: import("../../action.js").ActionDefinition<{
     commentId: string;
 }, {
     commentId: string;
-    deleted: boolean;
+    deleted: true;
+    updatedCount: number;
 }>;
 export default _default;
 //# sourceMappingURL=delete-review-comment.d.ts.map

package/dist/review/actions/delete-review-comment.js CHANGED Viewed

@@ -28,12 +28,16 @@ export default defineAction({
             throw new Error("Review comment not found");
         }
         const isAuthor = Boolean(actionCtx?.userEmail) &&
-            comment.authorEmail === actionCtx?.userEmail;
+            normalizeEmail(comment.authorEmail) ===
+                normalizeEmail(actionCtx?.userEmail);
         if (!isAuthor && (!access || !roleSatisfies(access.role, "editor"))) {
             throw new ForbiddenError("Not allowed to delete this review comment");
         }
-        await deleteReviewComment(comment.id, actionCtx?.userEmail ?? null);
-        return { commentId: comment.id, deleted: true };
+        const updatedCount = await deleteReviewComment(comment.id, actionCtx?.userEmail ?? null);
+        if (updatedCount < 1) {
+            throw new Error("Review comment not found");
+        }
+        return { commentId: comment.id, deleted: true, updatedCount };
     },
     audit: {
         target: (args) => ({
@@ -42,4 +46,8 @@ export default defineAction({
         }),
     },
 });
+function normalizeEmail(email) {
+    const normalized = email?.trim().toLowerCase();
+    return normalized || null;
+}
 //# sourceMappingURL=delete-review-comment.js.map

package/dist/review/actions/delete-review-comment.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"delete-review-comment.js","sourceRoot":"","sources":["../../../src/review/actions/delete-review-comment.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACzD,OAAO,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AACxD,OAAO,EAAE,+BAA+B,EAAE,MAAM,gBAAgB,CAAC;AACjE,OAAO,EAAE,mBAAmB,EAAE,oBAAoB,EAAE,MAAM,aAAa,CAAC;AAGxE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;CAC7B,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EAAE,+BAA+B;IAC5C,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,KAAK,GAAG;YACZ,SAAS,EAAE,SAAS,EAAE,SAAS,IAAI,IAAI;YACvC,KAAK,EAAE,SAAS,EAAE,KAAK,IAAI,IAAI;SAChC,CAAC;QACF,MAAM,MAAM,GAAG,MAAM,+BAA+B,CAClD,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,CACV,CAAC;QACF,MAAM,OAAO,GAAG,MAAM,oBAAoB,CAAC,IAAI,CAAC,SAAS,EAAE,KAAK,EAAE;YAChE,WAAW,EAAE,OAAO,CAAC,MAAM,CAAC;SAC7B,CAAC,CAAC;QACH,IACE,CAAC,OAAO;YACR,OAAO,CAAC,YAAY,KAAK,IAAI,CAAC,YAAY;YAC1C,OAAO,CAAC,UAAU,KAAK,IAAI,CAAC,UAAU,EACtC,CAAC;YACD,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC9C,CAAC;QACD,MAAM,QAAQ,GACZ,OAAO,CAAC,SAAS,EAAE,SAAS,CAAC;YAC7B,OAAO,CAAC,WAAW,~~KAAK~~,SAAS,EAAE,SAAS,CAAC;~~QAC/C~~,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,EAAE,CAAC;YACpE,MAAM,IAAI,cAAc,CAAC,2CAA2C,CAAC,CAAC;QACxE,CAAC;QACD,MAAM,mBAAmB,~~CAAC~~,OAAO,CAAC,EAAE,~~EAAE~~,SAAS,EAAE,SAAS,IAAI,IAAI,CAAC,CAAC;~~QACpE~~,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,EAAE,OAAO,EAAE,~~IAAI~~,EAAE,CAAC;~~IAClD~~,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { ForbiddenError } from \"../../sharing/access.js\";\nimport { roleSatisfies } from \"../../sharing/schema.js\";\nimport { resolveReviewableResourceAccess } from \"../registry.js\";\nimport { deleteReviewComment, getReviewCommentById } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n commentId: z.string().min(1),\n});\n\nexport default defineAction({\n description: \"Soft-delete a review comment.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n const scope = {\n userEmail: actionCtx?.userEmail ?? null,\n orgId: actionCtx?.orgId ?? null,\n };\n const access = await resolveReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n );\n const comment = await getReviewCommentById(args.commentId, scope, {\n bypassScope: Boolean(access),\n });\n if (\n !comment \|\|\n comment.resourceType !== args.resourceType \|\|\n comment.resourceId !== args.resourceId\n ) {\n throw new Error(\"Review comment not found\");\n }\n const isAuthor =\n Boolean(actionCtx?.userEmail) &&\n comment.authorEmail ~~===~~ actionCtx?.userEmail;\n if (!isAuthor && (!access \|\| !roleSatisfies(access.role, \"editor\"))) {\n throw new ForbiddenError(\"Not allowed to delete this review comment\");\n }\n await deleteReviewComment(comment.id, actionCtx?.userEmail ?? null);\n return { commentId: comment.id, deleted: true };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n"]}
1	+ {"version":3,"file":"delete-review-comment.js","sourceRoot":"","sources":["../../../src/review/actions/delete-review-comment.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACzD,OAAO,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AACxD,OAAO,EAAE,+BAA+B,EAAE,MAAM,gBAAgB,CAAC;AACjE,OAAO,EAAE,mBAAmB,EAAE,oBAAoB,EAAE,MAAM,aAAa,CAAC;AAGxE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;CAC7B,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EAAE,+BAA+B;IAC5C,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,KAAK,GAAG;YACZ,SAAS,EAAE,SAAS,EAAE,SAAS,IAAI,IAAI;YACvC,KAAK,EAAE,SAAS,EAAE,KAAK,IAAI,IAAI;SAChC,CAAC;QACF,MAAM,MAAM,GAAG,MAAM,+BAA+B,CAClD,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,CACV,CAAC;QACF,MAAM,OAAO,GAAG,MAAM,oBAAoB,CAAC,IAAI,CAAC,SAAS,EAAE,KAAK,EAAE;YAChE,WAAW,EAAE,OAAO,CAAC,MAAM,CAAC;SAC7B,CAAC,CAAC;QACH,IACE,CAAC,OAAO;YACR,OAAO,CAAC,YAAY,KAAK,IAAI,CAAC,YAAY;YAC1C,OAAO,CAAC,UAAU,KAAK,IAAI,CAAC,UAAU,EACtC,CAAC;YACD,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC9C,CAAC;QACD,MAAM,QAAQ,GACZ,OAAO,CAAC,SAAS,EAAE,SAAS,CAAC;YAC7B,cAAc,CAAC,OAAO,CAAC,WAAW,CAAC;gBACjC,cAAc,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;QACzC,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,EAAE,CAAC;YACpE,MAAM,IAAI,cAAc,CAAC,2CAA2C,CAAC,CAAC;QACxE,CAAC;QACD,MAAM,YAAY,GAAG,MAAM,mBAAmB,CAC5C,OAAO,CAAC,EAAE,EACV,SAAS,EAAE,SAAS,IAAI,IAAI,CAC7B,CAAC;QACF,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC;YACrB,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;QAC9C,CAAC;QACD,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,EAAE,OAAO,EAAE,IAAa,EAAE,YAAY,EAAE,CAAC;IACzE,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC;AAEH,SAAS,cAAc,CAAC,KAAgC;IACtD,MAAM,UAAU,GAAG,KAAK,EAAE,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC/C,OAAO,UAAU,IAAI,IAAI,CAAC;AAC5B,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { ForbiddenError } from \"../../sharing/access.js\";\nimport { roleSatisfies } from \"../../sharing/schema.js\";\nimport { resolveReviewableResourceAccess } from \"../registry.js\";\nimport { deleteReviewComment, getReviewCommentById } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n commentId: z.string().min(1),\n});\n\nexport default defineAction({\n description: \"Soft-delete a review comment.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n const scope = {\n userEmail: actionCtx?.userEmail ?? null,\n orgId: actionCtx?.orgId ?? null,\n };\n const access = await resolveReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n );\n const comment = await getReviewCommentById(args.commentId, scope, {\n bypassScope: Boolean(access),\n });\n if (\n !comment \|\|\n comment.resourceType !== args.resourceType \|\|\n comment.resourceId !== args.resourceId\n ) {\n throw new Error(\"Review comment not found\");\n }\n const isAuthor =\n Boolean(actionCtx?.userEmail) &&\n normalizeEmail(comment.authorEmail) ===\n normalizeEmail(actionCtx?.userEmail);\n if (!isAuthor && (!access \|\| !roleSatisfies(access.role, \"editor\"))) {\n throw new ForbiddenError(\"Not allowed to delete this review comment\");\n }\n const updatedCount = await deleteReviewComment(\n comment.id,\n actionCtx?.userEmail ?? null,\n );\n if (updatedCount < 1) {\n throw new Error(\"Review comment not found\");\n }\n return { commentId: comment.id, deleted: true as const, updatedCount };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n\nfunction normalizeEmail(email: string \| null \| undefined): string \| null {\n const normalized = email?.trim().toLowerCase();\n return normalized \|\| null;\n}\n"]}

package/dist/review/actions/resolve-review-thread.d.ts CHANGED Viewed

@@ -5,7 +5,8 @@ declare const _default: import("../../action.js").ActionDefinition<{
     commentId?: string;
 }, {
     threadId: string;
-    resolved: boolean;
+    resolved: true;
+    updatedCount: number;
 }>;
 export default _default;
 //# sourceMappingURL=resolve-review-thread.d.ts.map

package/dist/review/actions/resolve-review-thread.js CHANGED Viewed

@@ -30,11 +30,14 @@ export default defineAction({
         if (!threadId) {
             throw new Error("Provide threadId or commentId");
         }
-        await resolveReviewThread(threadId, actionCtx?.userEmail ?? null, {
+        const updatedCount = await resolveReviewThread(threadId, actionCtx?.userEmail ?? null, {
             resourceType: args.resourceType,
             resourceId: args.resourceId,
         });
-        return { threadId, resolved: true };
+        if (updatedCount < 1) {
+            throw new Error("Review thread not found");
+        }
+        return { threadId, resolved: true, updatedCount };
     },
     audit: {
         target: (args) => ({

package/dist/review/actions/resolve-review-thread.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"resolve-review-thread.js","sourceRoot":"","sources":["../../../src/review/actions/resolve-review-thread.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,8BAA8B,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,oBAAoB,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AAGxE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;CACjC,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EAAE,6CAA6C;IAC1D,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,8BAA8B,CAClC,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,EACT,QAAQ,CACT,CAAC;QACF,IAAI,QAAQ,GAAG,IAAI,CAAC,QAAQ,CAAC;QAC7B,IAAI,CAAC,QAAQ,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;YAChC,MAAM,OAAO,GAAG,MAAM,oBAAoB,CACxC,IAAI,CAAC,SAAS,EACd;gBACE,SAAS,EAAE,SAAS,EAAE,SAAS,IAAI,IAAI;gBACvC,KAAK,EAAE,SAAS,EAAE,KAAK,IAAI,IAAI;aAChC,EACD,EAAE,WAAW,EAAE,IAAI,EAAE,CACtB,CAAC;YACF,IACE,CAAC,OAAO;gBACR,OAAO,CAAC,YAAY,KAAK,IAAI,CAAC,YAAY;gBAC1C,OAAO,CAAC,UAAU,KAAK,IAAI,CAAC,UAAU,EACtC,CAAC;gBACD,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;YAC9C,CAAC;YACD,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;QAC9B,CAAC;QACD,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,+BAA+B,CAAC,CAAC;QACnD,CAAC;QACD,MAAM,mBAAmB,~~CAAC~~,QAAQ,~~EAAE~~,SAAS,EAAE,SAAS,IAAI,IAAI,~~EAAE~~;~~YAChE~~,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,UAAU,EAAE,IAAI,CAAC,UAAU;SAC5B,CAAC,CAAC;~~QACH~~,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,~~IAAI~~,EAAE,CAAC;~~IACtC~~,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { assertReviewableResourceAccess } from \"../registry.js\";\nimport { getReviewCommentById, resolveReviewThread } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n threadId: z.string().optional(),\n commentId: z.string().optional(),\n});\n\nexport default defineAction({\n description: \"Resolve an inline comment or review thread.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n await assertReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n \"editor\",\n );\n let threadId = args.threadId;\n if (!threadId && args.commentId) {\n const comment = await getReviewCommentById(\n args.commentId,\n {\n userEmail: actionCtx?.userEmail ?? null,\n orgId: actionCtx?.orgId ?? null,\n },\n { bypassScope: true },\n );\n if (\n !comment \|\|\n comment.resourceType !== args.resourceType \|\|\n comment.resourceId !== args.resourceId\n ) {\n throw new Error(\"Review comment not found\");\n }\n threadId = comment.threadId;\n }\n if (!threadId) {\n throw new Error(\"Provide threadId or commentId\");\n }\n await resolveReviewThread(threadId, actionCtx?.userEmail ?? null, {\n resourceType: args.resourceType,\n resourceId: args.resourceId,\n });\n return { threadId, resolved: true };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n"]}
1	+ {"version":3,"file":"resolve-review-thread.js","sourceRoot":"","sources":["../../../src/review/actions/resolve-review-thread.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAE,YAAY,EAAE,MAAM,iBAAiB,CAAC;AAC/C,OAAO,EAAE,8BAA8B,EAAE,MAAM,gBAAgB,CAAC;AAChE,OAAO,EAAE,oBAAoB,EAAE,mBAAmB,EAAE,MAAM,aAAa,CAAC;AAGxE,MAAM,MAAM,GAAG,CAAC,CAAC,MAAM,CAAC;IACtB,YAAY,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC/B,UAAU,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;IAC7B,QAAQ,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;IAC/B,SAAS,EAAE,CAAC,CAAC,MAAM,EAAE,CAAC,QAAQ,EAAE;CACjC,CAAC,CAAC;AAEH,eAAe,YAAY,CAAC;IAC1B,WAAW,EAAE,6CAA6C;IAC1D,MAAM;IACN,GAAG,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,EAAE,EAAE;QACvB,MAAM,SAAS,GAAG,GAAwC,CAAC;QAC3D,MAAM,8BAA8B,CAClC,IAAI,CAAC,YAAY,EACjB,IAAI,CAAC,UAAU,EACf,SAAS,EACT,QAAQ,CACT,CAAC;QACF,IAAI,QAAQ,GAAG,IAAI,CAAC,QAAQ,CAAC;QAC7B,IAAI,CAAC,QAAQ,IAAI,IAAI,CAAC,SAAS,EAAE,CAAC;YAChC,MAAM,OAAO,GAAG,MAAM,oBAAoB,CACxC,IAAI,CAAC,SAAS,EACd;gBACE,SAAS,EAAE,SAAS,EAAE,SAAS,IAAI,IAAI;gBACvC,KAAK,EAAE,SAAS,EAAE,KAAK,IAAI,IAAI;aAChC,EACD,EAAE,WAAW,EAAE,IAAI,EAAE,CACtB,CAAC;YACF,IACE,CAAC,OAAO;gBACR,OAAO,CAAC,YAAY,KAAK,IAAI,CAAC,YAAY;gBAC1C,OAAO,CAAC,UAAU,KAAK,IAAI,CAAC,UAAU,EACtC,CAAC;gBACD,MAAM,IAAI,KAAK,CAAC,0BAA0B,CAAC,CAAC;YAC9C,CAAC;YACD,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC;QAC9B,CAAC;QACD,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,MAAM,IAAI,KAAK,CAAC,+BAA+B,CAAC,CAAC;QACnD,CAAC;QACD,MAAM,YAAY,GAAG,MAAM,mBAAmB,CAC5C,QAAQ,EACR,SAAS,EAAE,SAAS,IAAI,IAAI,EAC5B;YACE,YAAY,EAAE,IAAI,CAAC,YAAY;YAC/B,UAAU,EAAE,IAAI,CAAC,UAAU;SAC5B,CACF,CAAC;QACF,IAAI,YAAY,GAAG,CAAC,EAAE,CAAC;YACrB,MAAM,IAAI,KAAK,CAAC,yBAAyB,CAAC,CAAC;QAC7C,CAAC;QACD,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,IAAa,EAAE,YAAY,EAAE,CAAC;IAC7D,CAAC;IACD,KAAK,EAAE;QACL,MAAM,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;YACjB,IAAI,EAAE,IAAI,CAAC,YAAY;YACvB,EAAE,EAAE,IAAI,CAAC,UAAU;SACpB,CAAC;KACH;CACF,CAAC,CAAC","sourcesContent":["import { z } from \"zod\";\n\nimport { defineAction } from \"../../action.js\";\nimport { assertReviewableResourceAccess } from \"../registry.js\";\nimport { getReviewCommentById, resolveReviewThread } from \"../store.js\";\nimport type { ReviewResourceContext } from \"../types.js\";\n\nconst schema = z.object({\n resourceType: z.string().min(1),\n resourceId: z.string().min(1),\n threadId: z.string().optional(),\n commentId: z.string().optional(),\n});\n\nexport default defineAction({\n description: \"Resolve an inline comment or review thread.\",\n schema,\n run: async (args, ctx) => {\n const actionCtx = ctx as ReviewResourceContext \| undefined;\n await assertReviewableResourceAccess(\n args.resourceType,\n args.resourceId,\n actionCtx,\n \"editor\",\n );\n let threadId = args.threadId;\n if (!threadId && args.commentId) {\n const comment = await getReviewCommentById(\n args.commentId,\n {\n userEmail: actionCtx?.userEmail ?? null,\n orgId: actionCtx?.orgId ?? null,\n },\n { bypassScope: true },\n );\n if (\n !comment \|\|\n comment.resourceType !== args.resourceType \|\|\n comment.resourceId !== args.resourceId\n ) {\n throw new Error(\"Review comment not found\");\n }\n threadId = comment.threadId;\n }\n if (!threadId) {\n throw new Error(\"Provide threadId or commentId\");\n }\n const updatedCount = await resolveReviewThread(\n threadId,\n actionCtx?.userEmail ?? null,\n {\n resourceType: args.resourceType,\n resourceId: args.resourceId,\n },\n );\n if (updatedCount < 1) {\n throw new Error(\"Review thread not found\");\n }\n return { threadId, resolved: true as const, updatedCount };\n },\n audit: {\n target: (args) => ({\n type: args.resourceType,\n id: args.resourceId,\n }),\n },\n});\n"]}

package/dist/review/index.d.ts CHANGED Viewed

@@ -1,5 +1,5 @@
 export type { ReviewActorKind, ReviewComment, ReviewCommentKind, ReviewCommentStatus, ReviewMention, ReviewResolutionTarget, ReviewResourceAccess, ReviewResourceContext, ReviewResourceRole, ReviewScope, ReviewStatus, ReviewStatusEntry, ReviewableResourceRegistration, } from "./types.js";
 export { extractReviewMentions, normalizeReviewMentions } from "./mentions.js";
 export { __resetReviewableResourcesForTests, assertReviewableResourceAccess, getReviewableResource, listReviewableResources, registerReviewableResource, resolveReviewableResourceAccess, } from "./registry.js";
-export { __resetReviewInitForTests, consumeReviewFeedback, deleteReviewComment, ensureReviewTables, getReviewCommentById, getReviewStatus, insertReviewComment, queryReviewComments, resolveReviewThread, upsertReviewStatus, } from "./store.js";
+export { __resetReviewInitForTests, ensureReviewTables, getReviewCommentById, getReviewStatus, queryReviewComments, } from "./store.js";
 //# sourceMappingURL=index.d.ts.map

package/dist/review/index.d.ts.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":"AAAA,YAAY,EACV,eAAe,EACf,aAAa,EACb,iBAAiB,EACjB,mBAAmB,EACnB,aAAa,EACb,sBAAsB,EACtB,oBAAoB,EACpB,qBAAqB,EACrB,kBAAkB,EAClB,WAAW,EACX,YAAY,EACZ,iBAAiB,EACjB,8BAA8B,GAC/B,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,qBAAqB,EAAE,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAC/E,OAAO,EACL,kCAAkC,EAClC,8BAA8B,EAC9B,qBAAqB,EACrB,uBAAuB,EACvB,0BAA0B,EAC1B,+BAA+B,GAChC,MAAM,eAAe,CAAC;AACvB,OAAO,EACL,yBAAyB,EACzB,~~qBAAqB,EACrB,mBAAmB,EACnB,~~kBAAkB,EAClB,oBAAoB,EACpB,eAAe,EACf,mBAAmB,~~EACnB~~,~~mBAAmB,EACnB,mBAAmB,EACnB,kBAAkB,GACnB,~~MAAM,YAAY,CAAC"}
1	+ {"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":"AAAA,YAAY,EACV,eAAe,EACf,aAAa,EACb,iBAAiB,EACjB,mBAAmB,EACnB,aAAa,EACb,sBAAsB,EACtB,oBAAoB,EACpB,qBAAqB,EACrB,kBAAkB,EAClB,WAAW,EACX,YAAY,EACZ,iBAAiB,EACjB,8BAA8B,GAC/B,MAAM,YAAY,CAAC;AACpB,OAAO,EAAE,qBAAqB,EAAE,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAC/E,OAAO,EACL,kCAAkC,EAClC,8BAA8B,EAC9B,qBAAqB,EACrB,uBAAuB,EACvB,0BAA0B,EAC1B,+BAA+B,GAChC,MAAM,eAAe,CAAC;AACvB,OAAO,EACL,yBAAyB,EACzB,kBAAkB,EAClB,oBAAoB,EACpB,eAAe,EACf,mBAAmB,GACpB,MAAM,YAAY,CAAC"}

package/dist/review/index.js CHANGED Viewed

@@ -1,4 +1,4 @@
 export { extractReviewMentions, normalizeReviewMentions } from "./mentions.js";
 export { __resetReviewableResourcesForTests, assertReviewableResourceAccess, getReviewableResource, listReviewableResources, registerReviewableResource, resolveReviewableResourceAccess, } from "./registry.js";
-export { __resetReviewInitForTests, consumeReviewFeedback, deleteReviewComment, ensureReviewTables, getReviewCommentById, getReviewStatus, insertReviewComment, queryReviewComments, resolveReviewThread, upsertReviewStatus, } from "./store.js";
+export { __resetReviewInitForTests, ensureReviewTables, getReviewCommentById, getReviewStatus, queryReviewComments, } from "./store.js";
 //# sourceMappingURL=index.js.map

package/dist/review/index.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":"AAeA,OAAO,EAAE,qBAAqB,EAAE,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAC/E,OAAO,EACL,kCAAkC,EAClC,8BAA8B,EAC9B,qBAAqB,EACrB,uBAAuB,EACvB,0BAA0B,EAC1B,+BAA+B,GAChC,MAAM,eAAe,CAAC;AACvB,OAAO,EACL,yBAAyB,EACzB,~~qBAAqB,EACrB,mBAAmB,EACnB,~~kBAAkB,EAClB,oBAAoB,EACpB,eAAe,EACf,mBAAmB,~~EACnB~~,~~mBAAmB,EACnB,mBAAmB,EACnB,kBAAkB,GACnB,~~MAAM,YAAY,CAAC","sourcesContent":["export type {\n ReviewActorKind,\n ReviewComment,\n ReviewCommentKind,\n ReviewCommentStatus,\n ReviewMention,\n ReviewResolutionTarget,\n ReviewResourceAccess,\n ReviewResourceContext,\n ReviewResourceRole,\n ReviewScope,\n ReviewStatus,\n ReviewStatusEntry,\n ReviewableResourceRegistration,\n} from \"./types.js\";\nexport { extractReviewMentions, normalizeReviewMentions } from \"./mentions.js\";\nexport {\n __resetReviewableResourcesForTests,\n assertReviewableResourceAccess,\n getReviewableResource,\n listReviewableResources,\n registerReviewableResource,\n resolveReviewableResourceAccess,\n} from \"./registry.js\";\nexport {\n __resetReviewInitForTests,\n ~~consumeReviewFeedback,\n deleteReviewComment,\n~~ ensureReviewTables,\n getReviewCommentById,\n getReviewStatus,\n ~~insertReviewComment,\n~~ queryReviewComments,\n ~~resolveReviewThread,\n upsertReviewStatus,\n~~} from \"./store.js\";\n"]}
1	+ {"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/review/index.ts"],"names":[],"mappings":"AAeA,OAAO,EAAE,qBAAqB,EAAE,uBAAuB,EAAE,MAAM,eAAe,CAAC;AAC/E,OAAO,EACL,kCAAkC,EAClC,8BAA8B,EAC9B,qBAAqB,EACrB,uBAAuB,EACvB,0BAA0B,EAC1B,+BAA+B,GAChC,MAAM,eAAe,CAAC;AACvB,OAAO,EACL,yBAAyB,EACzB,kBAAkB,EAClB,oBAAoB,EACpB,eAAe,EACf,mBAAmB,GACpB,MAAM,YAAY,CAAC","sourcesContent":["export type {\n ReviewActorKind,\n ReviewComment,\n ReviewCommentKind,\n ReviewCommentStatus,\n ReviewMention,\n ReviewResolutionTarget,\n ReviewResourceAccess,\n ReviewResourceContext,\n ReviewResourceRole,\n ReviewScope,\n ReviewStatus,\n ReviewStatusEntry,\n ReviewableResourceRegistration,\n} from \"./types.js\";\nexport { extractReviewMentions, normalizeReviewMentions } from \"./mentions.js\";\nexport {\n __resetReviewableResourcesForTests,\n assertReviewableResourceAccess,\n getReviewableResource,\n listReviewableResources,\n registerReviewableResource,\n resolveReviewableResourceAccess,\n} from \"./registry.js\";\nexport {\n __resetReviewInitForTests,\n ensureReviewTables,\n getReviewCommentById,\n getReviewStatus,\n queryReviewComments,\n} from \"./store.js\";\n"]}

package/dist/review/registry.d.ts.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/review/registry.ts"],"names":[],"mappings":"~~AAEA~~,OAAO,EAAiB,KAAK,UAAU,EAAE,MAAM,sBAAsB,CAAC;AACtE,OAAO,KAAK,EACV,oBAAoB,EACpB,qBAAqB,EACrB,kBAAkB,EAClB,8BAA8B,EAC/B,MAAM,YAAY,CAAC;AAIpB,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,8BAA8B,GAC3C,IAAI,CAON;AAED,wBAAgB,qBAAqB,CACnC,IAAI,EAAE,MAAM,GACX,8BAA8B,GAAG,SAAS,CAE5C;AAED,wBAAgB,uBAAuB,IAAI,8BAA8B,EAAE,CAE1E;~~AAED~~,wBAAsB,+BAA+B,CACnD,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,GAAG,CAAC,EAAE,qBAAqB,GAC1B,OAAO,CAAC,oBAAoB,GAAG,IAAI,CAAC,~~CA8BtC~~;AAED,wBAAsB,8BAA8B,CAClD,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,GAAG,EAAE,qBAAqB,GAAG,SAAS,EACtC,WAAW,EAAE,kBAAkB,GAC9B,OAAO,CAAC,oBAAoB,CAAC,CAY/B;AAED,wBAAgB,yBAAyB,CACvC,UAAU,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,GACxC,UAAU,CAIZ;AAED,wBAAgB,kCAAkC,IAAI,IAAI,CAEzD"}
1	+ {"version":3,"file":"registry.d.ts","sourceRoot":"","sources":["../../src/review/registry.ts"],"names":[],"mappings":"AAMA,OAAO,EAAiB,KAAK,UAAU,EAAE,MAAM,sBAAsB,CAAC;AACtE,OAAO,KAAK,EACV,oBAAoB,EACpB,qBAAqB,EACrB,kBAAkB,EAClB,8BAA8B,EAC/B,MAAM,YAAY,CAAC;AAIpB,wBAAgB,0BAA0B,CACxC,YAAY,EAAE,8BAA8B,GAC3C,IAAI,CAON;AAED,wBAAgB,qBAAqB,CACnC,IAAI,EAAE,MAAM,GACX,8BAA8B,GAAG,SAAS,CAE5C;AAED,wBAAgB,uBAAuB,IAAI,8BAA8B,EAAE,CAE1E;AAYD,wBAAsB,+BAA+B,CACnD,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,GAAG,CAAC,EAAE,qBAAqB,GAC1B,OAAO,CAAC,oBAAoB,GAAG,IAAI,CAAC,CA4BtC;AAED,wBAAsB,8BAA8B,CAClD,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,GAAG,EAAE,qBAAqB,GAAG,SAAS,EACtC,WAAW,EAAE,kBAAkB,GAC9B,OAAO,CAAC,oBAAoB,CAAC,CAY/B;AAED,wBAAgB,yBAAyB,CACvC,UAAU,EAAE,UAAU,GAAG,IAAI,GAAG,SAAS,GACxC,UAAU,CAIZ;AAED,wBAAgB,kCAAkC,IAAI,IAAI,CAEzD"}

package/dist/review/registry.js CHANGED Viewed

@@ -1,4 +1,4 @@
-import { ForbiddenError, resolveAccess } from "../sharing/access.js";
+import { ForbiddenError, resolveAccess, } from "../sharing/access.js";
 import { getShareableResource } from "../sharing/registry.js";
 import { roleSatisfies } from "../sharing/schema.js";
 const registrations = new Map();
@@ -14,13 +14,21 @@ export function getReviewableResource(type) {
 export function listReviewableResources() {
     return Array.from(registrations.values());
 }
+function accessContextFrom(ctx) {
+    if (!ctx)
+        return undefined;
+    return {
+        userEmail: ctx.userEmail ?? undefined,
+        orgId: ctx.orgId ?? undefined,
+    };
+}
 export async function resolveReviewableResourceAccess(resourceType, resourceId, ctx) {
     const registration = getReviewableResource(resourceType);
     if (registration?.resolveAccess) {
         return registration.resolveAccess(resourceId, ctx);
     }
     if (getShareableResource(resourceType)) {
-        const access = await resolveAccess(resourceType, resourceId);
+        const access = await resolveAccess(resourceType, resourceId, accessContextFrom(ctx));
         if (!access) {
             return null;
         }
@@ -32,16 +40,10 @@ export async function resolveReviewableResourceAccess(resourceType, resourceId,
             resource: access.resource,
         };
     }
-    const userEmail = ctx?.userEmail ?? null;
-    if (!userEmail) {
-        return null;
-    }
-    return {
-        role: "owner",
-        ownerEmail: userEmail,
-        orgId: ctx?.orgId ?? null,
-        visibility: "private",
-    };
+    // Fail closed: unregistered types, and registered types without an access
+    // resolver that aren't shareable, never invent ownership. bypassScope on
+    // review queries is only safe after a real resource ACL.
+    return null;
 }
 export async function assertReviewableResourceAccess(resourceType, resourceId, ctx, minimumRole) {
     const access = await resolveReviewableResourceAccess(resourceType, resourceId, ctx);

package/dist/review/registry.js.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"registry.js","sourceRoot":"","sources":["../../src/review/registry.ts"],"names":[],"mappings":"AAAA,OAAO,~~EAAE~~,cAAc,~~EAAE~~,aAAa,~~EAAE~~,MAAM,sBAAsB,CAAC;~~AACrE~~,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAC;AAC9D,OAAO,EAAE,aAAa,EAAmB,MAAM,sBAAsB,CAAC;AAQtE,MAAM,aAAa,GAAG,IAAI,GAAG,EAA0C,CAAC;AAExE,MAAM,UAAU,0BAA0B,CACxC,YAA4C;IAE5C,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CACb,+DAA+D,CAChE,CAAC;IACJ,CAAC;IACD,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC;AACrD,CAAC;AAED,MAAM,UAAU,qBAAqB,CACnC,IAAY;IAEZ,OAAO,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;AACjC,CAAC;AAED,MAAM,UAAU,uBAAuB;IACrC,OAAO,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,CAAC;AAC5C,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,+BAA+B,CACnD,YAAoB,EACpB,UAAkB,EAClB,GAA2B;IAE3B,MAAM,YAAY,GAAG,qBAAqB,CAAC,YAAY,CAAC,CAAC;IACzD,IAAI,YAAY,EAAE,aAAa,EAAE,CAAC;QAChC,OAAO,YAAY,CAAC,aAAa,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC;IACrD,CAAC;IAED,IAAI,oBAAoB,CAAC,YAAY,CAAC,EAAE,CAAC;QACvC,MAAM,MAAM,GAAG,MAAM,aAAa,~~CAAC~~,YAAY,~~EAAE~~,UAAU,CAAC,CAAC;~~QAC7D~~,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,OAAO,IAAI,CAAC;QACd,CAAC;QACD,OAAO;YACL,IAAI,EAAE,MAAM,CAAC,IAA0B;YACvC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,UAAU,IAAI,IAAI;YAC9C,KAAK,EAAE,MAAM,CAAC,QAAQ,CAAC,KAAK,IAAI,IAAI;YACpC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,UAAU;YACtC,QAAQ,EAAE,MAAM,CAAC,QAAQ;SAC1B,CAAC;IACJ,CAAC;IAED,~~MAAM,SAAS,GAAG,GAAG,EAAE,SAAS,IAAI,IAAI,CAAC~~;~~IACzC~~,~~IAAI,CAAC,SAAS,EAAE,CAAC~~;~~QACf~~,~~OAAO,IAAI,CAAC~~;~~IACd~~,~~CAAC;IACD,~~OAAO~~;QACL~~,IAAI,~~EAAE,OAAO;QACb,UAAU,EAAE,SAAS;QACrB,KAAK,EAAE,GAAG,EAAE,KAAK,IAAI,IAAI;QACzB,UAAU,EAAE,SAAS;KACtB,~~CAAC;~~AACJ~~,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,8BAA8B,CAClD,YAAoB,EACpB,UAAkB,EAClB,GAAsC,EACtC,WAA+B;IAE/B,MAAM,MAAM,GAAG,MAAM,+BAA+B,CAClD,YAAY,EACZ,UAAU,EACV,GAAG,CACJ,CAAC;IACF,IAAI,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,WAAW,CAAC,EAAE,CAAC;QACxD,MAAM,IAAI,cAAc,CACtB,yBAAyB,YAAY,IAAI,UAAU,EAAE,CACtD,CAAC;IACJ,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,yBAAyB,CACvC,UAAyC;IAEzC,OAAO,UAAU,KAAK,KAAK,IAAI,UAAU,KAAK,QAAQ;QACpD,CAAC,CAAC,UAAU;QACZ,CAAC,CAAC,SAAS,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,kCAAkC;IAChD,aAAa,CAAC,KAAK,EAAE,CAAC;AACxB,CAAC","sourcesContent":["import { ForbiddenError, resolveAccess } from \"../sharing/access.js\";\nimport { getShareableResource } from \"../sharing/registry.js\";\nimport { roleSatisfies, type Visibility } from \"../sharing/schema.js\";\nimport type {\n ReviewResourceAccess,\n ReviewResourceContext,\n ReviewResourceRole,\n ReviewableResourceRegistration,\n} from \"./types.js\";\n\nconst registrations = new Map<string, ReviewableResourceRegistration>();\n\nexport function registerReviewableResource(\n registration: ReviewableResourceRegistration,\n): void {\n if (!registration.type.trim()) {\n throw new Error(\n \"registerReviewableResource requires a non-empty resource type\",\n );\n }\n registrations.set(registration.type, registration);\n}\n\nexport function getReviewableResource(\n type: string,\n): ReviewableResourceRegistration \| undefined {\n return registrations.get(type);\n}\n\nexport function listReviewableResources(): ReviewableResourceRegistration[] {\n return Array.from(registrations.values());\n}\n\nexport async function resolveReviewableResourceAccess(\n resourceType: string,\n resourceId: string,\n ctx?: ReviewResourceContext,\n): Promise<ReviewResourceAccess \| null> {\n const registration = getReviewableResource(resourceType);\n if (registration?.resolveAccess) {\n return registration.resolveAccess(resourceId, ctx);\n }\n\n if (getShareableResource(resourceType)) {\n const access = await resolveAccess(resourceType, resourceId);\n if (!access) {\n return null;\n }\n return {\n role: access.role as ReviewResourceRole,\n ownerEmail: access.resource.ownerEmail ?? null,\n orgId: access.resource.orgId ?? null,\n visibility: access.resource.visibility,\n resource: access.resource,\n };\n }\n\n ~~const~~ ~~userEmail~~ = ~~ctx?.userEmail~~ ?? ~~null;\n if~~ ~~(!userEmail)~~ ~~{\n return~~ ~~null;\n }~~\n ~~return~~ ~~{\n role:~~ \~~"owner\",\~~n ~~ownerEmail:~~ ~~userEmail,\n orgId:~~ ~~ctx?.orgId~~ ?? ~~null,\n visibility:~~ ~~\"private\",\~~n };\n}\n\nexport async function assertReviewableResourceAccess(\n resourceType: string,\n resourceId: string,\n ctx: ReviewResourceContext \| undefined,\n minimumRole: ReviewResourceRole,\n): Promise<ReviewResourceAccess> {\n const access = await resolveReviewableResourceAccess(\n resourceType,\n resourceId,\n ctx,\n );\n if (!access \|\| !roleSatisfies(access.role, minimumRole)) {\n throw new ForbiddenError(\n `Not allowed to access ${resourceType}:${resourceId}`,\n );\n }\n return access;\n}\n\nexport function normalizeReviewVisibility(\n visibility: Visibility \| null \| undefined,\n): Visibility {\n return visibility === \"org\" \|\| visibility === \"public\"\n ? visibility\n : \"private\";\n}\n\nexport function __resetReviewableResourcesForTests(): void {\n registrations.clear();\n}\n"]}
1	+ {"version":3,"file":"registry.js","sourceRoot":"","sources":["../../src/review/registry.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,cAAc,EACd,aAAa,GAEd,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAAE,oBAAoB,EAAE,MAAM,wBAAwB,CAAC;AAC9D,OAAO,EAAE,aAAa,EAAmB,MAAM,sBAAsB,CAAC;AAQtE,MAAM,aAAa,GAAG,IAAI,GAAG,EAA0C,CAAC;AAExE,MAAM,UAAU,0BAA0B,CACxC,YAA4C;IAE5C,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CACb,+DAA+D,CAChE,CAAC;IACJ,CAAC;IACD,aAAa,CAAC,GAAG,CAAC,YAAY,CAAC,IAAI,EAAE,YAAY,CAAC,CAAC;AACrD,CAAC;AAED,MAAM,UAAU,qBAAqB,CACnC,IAAY;IAEZ,OAAO,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;AACjC,CAAC;AAED,MAAM,UAAU,uBAAuB;IACrC,OAAO,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,MAAM,EAAE,CAAC,CAAC;AAC5C,CAAC;AAED,SAAS,iBAAiB,CACxB,GAA2B;IAE3B,IAAI,CAAC,GAAG;QAAE,OAAO,SAAS,CAAC;IAC3B,OAAO;QACL,SAAS,EAAE,GAAG,CAAC,SAAS,IAAI,SAAS;QACrC,KAAK,EAAE,GAAG,CAAC,KAAK,IAAI,SAAS;KAC9B,CAAC;AACJ,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,+BAA+B,CACnD,YAAoB,EACpB,UAAkB,EAClB,GAA2B;IAE3B,MAAM,YAAY,GAAG,qBAAqB,CAAC,YAAY,CAAC,CAAC;IACzD,IAAI,YAAY,EAAE,aAAa,EAAE,CAAC;QAChC,OAAO,YAAY,CAAC,aAAa,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC;IACrD,CAAC;IAED,IAAI,oBAAoB,CAAC,YAAY,CAAC,EAAE,CAAC;QACvC,MAAM,MAAM,GAAG,MAAM,aAAa,CAChC,YAAY,EACZ,UAAU,EACV,iBAAiB,CAAC,GAAG,CAAC,CACvB,CAAC;QACF,IAAI,CAAC,MAAM,EAAE,CAAC;YACZ,OAAO,IAAI,CAAC;QACd,CAAC;QACD,OAAO;YACL,IAAI,EAAE,MAAM,CAAC,IAA0B;YACvC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,UAAU,IAAI,IAAI;YAC9C,KAAK,EAAE,MAAM,CAAC,QAAQ,CAAC,KAAK,IAAI,IAAI;YACpC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,UAAU;YACtC,QAAQ,EAAE,MAAM,CAAC,QAAQ;SAC1B,CAAC;IACJ,CAAC;IAED,0EAA0E;IAC1E,yEAAyE;IACzE,yDAAyD;IACzD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,CAAC,KAAK,UAAU,8BAA8B,CAClD,YAAoB,EACpB,UAAkB,EAClB,GAAsC,EACtC,WAA+B;IAE/B,MAAM,MAAM,GAAG,MAAM,+BAA+B,CAClD,YAAY,EACZ,UAAU,EACV,GAAG,CACJ,CAAC;IACF,IAAI,CAAC,MAAM,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,EAAE,WAAW,CAAC,EAAE,CAAC;QACxD,MAAM,IAAI,cAAc,CACtB,yBAAyB,YAAY,IAAI,UAAU,EAAE,CACtD,CAAC;IACJ,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,yBAAyB,CACvC,UAAyC;IAEzC,OAAO,UAAU,KAAK,KAAK,IAAI,UAAU,KAAK,QAAQ;QACpD,CAAC,CAAC,UAAU;QACZ,CAAC,CAAC,SAAS,CAAC;AAChB,CAAC;AAED,MAAM,UAAU,kCAAkC;IAChD,aAAa,CAAC,KAAK,EAAE,CAAC;AACxB,CAAC","sourcesContent":["import {\n ForbiddenError,\n resolveAccess,\n type AccessContext,\n} from \"../sharing/access.js\";\nimport { getShareableResource } from \"../sharing/registry.js\";\nimport { roleSatisfies, type Visibility } from \"../sharing/schema.js\";\nimport type {\n ReviewResourceAccess,\n ReviewResourceContext,\n ReviewResourceRole,\n ReviewableResourceRegistration,\n} from \"./types.js\";\n\nconst registrations = new Map<string, ReviewableResourceRegistration>();\n\nexport function registerReviewableResource(\n registration: ReviewableResourceRegistration,\n): void {\n if (!registration.type.trim()) {\n throw new Error(\n \"registerReviewableResource requires a non-empty resource type\",\n );\n }\n registrations.set(registration.type, registration);\n}\n\nexport function getReviewableResource(\n type: string,\n): ReviewableResourceRegistration \| undefined {\n return registrations.get(type);\n}\n\nexport function listReviewableResources(): ReviewableResourceRegistration[] {\n return Array.from(registrations.values());\n}\n\nfunction accessContextFrom(\n ctx?: ReviewResourceContext,\n): AccessContext \| undefined {\n if (!ctx) return undefined;\n return {\n userEmail: ctx.userEmail ?? undefined,\n orgId: ctx.orgId ?? undefined,\n };\n}\n\nexport async function resolveReviewableResourceAccess(\n resourceType: string,\n resourceId: string,\n ctx?: ReviewResourceContext,\n): Promise<ReviewResourceAccess \| null> {\n const registration = getReviewableResource(resourceType);\n if (registration?.resolveAccess) {\n return registration.resolveAccess(resourceId, ctx);\n }\n\n if (getShareableResource(resourceType)) {\n const access = await resolveAccess(\n resourceType,\n resourceId,\n accessContextFrom(ctx),\n );\n if (!access) {\n return null;\n }\n return {\n role: access.role as ReviewResourceRole,\n ownerEmail: access.resource.ownerEmail ?? null,\n orgId: access.resource.orgId ?? null,\n visibility: access.resource.visibility,\n resource: access.resource,\n };\n }\n\n // Fail closed: unregistered types, and registered types without an access\n // resolver that aren't shareable, never invent ownership. bypassScope on\n // review queries is only safe after a real resource ACL.\n return null;\n}\n\nexport async function assertReviewableResourceAccess(\n resourceType: string,\n resourceId: string,\n ctx: ReviewResourceContext \| undefined,\n minimumRole: ReviewResourceRole,\n): Promise<ReviewResourceAccess> {\n const access = await resolveReviewableResourceAccess(\n resourceType,\n resourceId,\n ctx,\n );\n if (!access \|\| !roleSatisfies(access.role, minimumRole)) {\n throw new ForbiddenError(\n `Not allowed to access ${resourceType}:${resourceId}`,\n );\n }\n return access;\n}\n\nexport function normalizeReviewVisibility(\n visibility: Visibility \| null \| undefined,\n): Visibility {\n return visibility === \"org\" \|\| visibility === \"public\"\n ? visibility\n : \"private\";\n}\n\nexport function __resetReviewableResourcesForTests(): void {\n registrations.clear();\n}\n"]}

package/dist/review/store.d.ts CHANGED Viewed

@@ -49,12 +49,12 @@ export declare function getReviewCommentById(id: string, scope: ReviewScope, opt
 export declare function resolveReviewThread(threadId: string, resolvedBy?: string | null, resource?: {
     resourceType: string;
     resourceId: string;
-}): Promise<void>;
-export declare function deleteReviewComment(id: string, deletedBy?: string | null): Promise<void>;
+}): Promise<number>;
+export declare function deleteReviewComment(id: string, deletedBy?: string | null): Promise<number>;
 export declare function consumeReviewFeedback(ids: string[], consumedAt?: string, resource?: {
     resourceType: string;
     resourceId: string;
-}): Promise<void>;
+}): Promise<number>;
 export declare function upsertReviewStatus(input: UpsertReviewStatusInput): Promise<ReviewStatusEntry>;
 export declare function getReviewStatus(resourceType: string, resourceId: string, scope: ReviewScope, options?: {
     bypassScope?: boolean;

package/dist/review/store.d.ts.map CHANGED Viewed

	@@ -1 +1 @@
1	- {"version":3,"file":"store.d.ts","sourceRoot":"","sources":["../../src/review/store.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AACvD,OAAO,KAAK,EACV,eAAe,EACf,aAAa,EACb,iBAAiB,EAEjB,aAAa,EACb,sBAAsB,EACtB,WAAW,EACX,YAAY,EACZ,iBAAiB,EAClB,MAAM,YAAY,CAAC;AAIpB,MAAM,WAAW,wBAAwB;IACvC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,IAAI,CAAC,EAAE,iBAAiB,CAAC;IACzB,MAAM,CAAC,EAAE,OAAO,GAAG,IAAI,CAAC;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,SAAS,CAAC,EAAE,eAAe,CAAC;IAC5B,gBAAgB,CAAC,EAAE,sBAAsB,GAAG,IAAI,CAAC;IACjD,QAAQ,CAAC,EAAE,aAAa,EAAE,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAAC;IAC/B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;CAC3C;AAED,MAAM,WAAW,wBAAwB;IACvC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,WAAW,CAAC;IACnB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,uBAAuB;IACtC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,YAAY,CAAC;IACrB,IAAI,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAAC;IAC/B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;CAC3C;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,IAAI,CAAC,CAmFxD;AAED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,wBAAwB,GAC9B,OAAO,CAAC,aAAa,CAAC,CAiGxB;AAED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,wBAAwB,GAC9B,OAAO,CAAC,aAAa,EAAE,CAAC,CAqC1B;AAED,wBAAsB,oBAAoB,CACxC,EAAE,EAAE,MAAM,EACV,KAAK,EAAE,WAAW,EAClB,OAAO,GAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAA;CAAO,GACtC,OAAO,CAAC,aAAa,GAAG,IAAI,CAAC,CAe/B;AAED,wBAAsB,mBAAmB,CACvC,QAAQ,EAAE,MAAM,EAChB,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,EAC1B,QAAQ,CAAC,EAAE;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,~~IAAI~~,CAAC,~~CAqBf~~;AAED,wBAAsB,mBAAmB,CACvC,EAAE,EAAE,MAAM,EACV,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,GACxB,OAAO,CAAC,~~IAAI~~,CAAC,~~CAYf~~;AAED,wBAAsB,qBAAqB,CACzC,GAAG,EAAE,MAAM,EAAE,EACb,UAAU,SAA2B,EACrC,QAAQ,CAAC,EAAE;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,~~IAAI~~,CAAC,~~CAqBf~~;AAED,wBAAsB,kBAAkB,CACtC,KAAK,EAAE,uBAAuB,GAC7B,OAAO,CAAC,iBAAiB,CAAC,CAkF5B;AAED,wBAAsB,eAAe,CACnC,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,KAAK,EAAE,WAAW,EAClB,OAAO,GAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAA;CAAO,GACtC,OAAO,CAAC,iBAAiB,GAAG,IAAI,CAAC,CAenC;AAED,wBAAgB,yBAAyB,IAAI,IAAI,CAEhD"}
1	+ {"version":3,"file":"store.d.ts","sourceRoot":"","sources":["../../src/review/store.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,sBAAsB,CAAC;AACvD,OAAO,KAAK,EACV,eAAe,EACf,aAAa,EACb,iBAAiB,EAEjB,aAAa,EACb,sBAAsB,EACtB,WAAW,EACX,YAAY,EACZ,iBAAiB,EAClB,MAAM,YAAY,CAAC;AAIpB,MAAM,WAAW,wBAAwB;IACvC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAChC,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,IAAI,CAAC,EAAE,iBAAiB,CAAC;IACzB,MAAM,CAAC,EAAE,OAAO,GAAG,IAAI,CAAC;IACxB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC5B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,SAAS,CAAC,EAAE,eAAe,CAAC;IAC5B,gBAAgB,CAAC,EAAE,sBAAsB,GAAG,IAAI,CAAC;IACjD,QAAQ,CAAC,EAAE,aAAa,EAAE,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAAC;IAC/B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;CAC3C;AAED,MAAM,WAAW,wBAAwB;IACvC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,WAAW,CAAC;IACnB,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,uBAAuB;IACtC,YAAY,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,YAAY,CAAC;IACrB,IAAI,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IAC3B,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACtB,UAAU,CAAC,EAAE,UAAU,GAAG,IAAI,CAAC;IAC/B,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,IAAI,CAAC;CAC3C;AAED,wBAAsB,kBAAkB,IAAI,OAAO,CAAC,IAAI,CAAC,CAmFxD;AAED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,wBAAwB,GAC9B,OAAO,CAAC,aAAa,CAAC,CAiGxB;AAED,wBAAsB,mBAAmB,CACvC,KAAK,EAAE,wBAAwB,GAC9B,OAAO,CAAC,aAAa,EAAE,CAAC,CAqC1B;AAED,wBAAsB,oBAAoB,CACxC,EAAE,EAAE,MAAM,EACV,KAAK,EAAE,WAAW,EAClB,OAAO,GAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAA;CAAO,GACtC,OAAO,CAAC,aAAa,GAAG,IAAI,CAAC,CAe/B;AAED,wBAAsB,mBAAmB,CACvC,QAAQ,EAAE,MAAM,EAChB,UAAU,CAAC,EAAE,MAAM,GAAG,IAAI,EAC1B,QAAQ,CAAC,EAAE;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,MAAM,CAAC,CAsBjB;AAED,wBAAsB,mBAAmB,CACvC,EAAE,EAAE,MAAM,EACV,SAAS,CAAC,EAAE,MAAM,GAAG,IAAI,GACxB,OAAO,CAAC,MAAM,CAAC,CAajB;AAED,wBAAsB,qBAAqB,CACzC,GAAG,EAAE,MAAM,EAAE,EACb,UAAU,SAA2B,EACrC,QAAQ,CAAC,EAAE;IAAE,YAAY,EAAE,MAAM,CAAC;IAAC,UAAU,EAAE,MAAM,CAAA;CAAE,GACtD,OAAO,CAAC,MAAM,CAAC,CAsBjB;AAED,wBAAsB,kBAAkB,CACtC,KAAK,EAAE,uBAAuB,GAC7B,OAAO,CAAC,iBAAiB,CAAC,CAkF5B;AAED,wBAAsB,eAAe,CACnC,YAAY,EAAE,MAAM,EACpB,UAAU,EAAE,MAAM,EAClB,KAAK,EAAE,WAAW,EAClB,OAAO,GAAE;IAAE,WAAW,CAAC,EAAE,OAAO,CAAA;CAAO,GACtC,OAAO,CAAC,iBAAiB,GAAG,IAAI,CAAC,CAenC;AAED,wBAAgB,yBAAyB,IAAI,IAAI,CAEhD"}

package/dist/review/store.js CHANGED Viewed

@@ -233,7 +233,7 @@ export async function resolveReviewThread(threadId, resolvedBy, resource) {
     const resourceClause = resource
         ? "AND resource_type = ? AND resource_id = ?"
         : "";
-    await getDbExec().execute({
+    const result = await getDbExec().execute({
         sql: `UPDATE agent_review_comments
         SET status = 'resolved',
             resolved_by = ?,
@@ -248,11 +248,12 @@ export async function resolveReviewThread(threadId, resolvedBy, resource) {
             ...(resource ? [resource.resourceType, resource.resourceId] : []),
         ],
     });
+    return result.rowsAffected ?? 0;
 }
 export async function deleteReviewComment(id, deletedBy) {
     await ensureReviewTables();
     const now = new Date().toISOString();
-    await getDbExec().execute({
+    const result = await getDbExec().execute({
         sql: `UPDATE agent_review_comments
         SET status = 'deleted',
             deleted_by = ?,
@@ -261,17 +262,18 @@ export async function deleteReviewComment(id, deletedBy) {
       WHERE id = ?`,
         args: [deletedBy ?? null, now, now, id],
     });
+    return result.rowsAffected ?? 0;
 }
 export async function consumeReviewFeedback(ids, consumedAt = new Date().toISOString(), resource) {
     if (!ids.length) {
-        return;
+        return 0;
     }
     await ensureReviewTables();
     const placeholders = ids.map(() => "?").join(", ");
     const resourceClause = resource
         ? "AND resource_type = ? AND resource_id = ?"
         : "";
-    await getDbExec().execute({
+    const result = await getDbExec().execute({
         sql: `UPDATE agent_review_comments
         SET consumed_at = ?,
             updated_at = ?
@@ -283,6 +285,7 @@ export async function consumeReviewFeedback(ids, consumedAt = new Date().toISOSt
             ...(resource ? [resource.resourceType, resource.resourceId] : []),
         ],
     });
+    return result.rowsAffected ?? 0;
 }
 export async function upsertReviewStatus(input) {
     await ensureReviewTables();