@aryaminus/controlkeel-opencode 0.3.2 → 0.3.5

package/.opencode/plugins/controlkeel-governance.ts

@@ -180,25 +180,45 @@ export const ControlKeelGovernance: Plugin = async ({ project, client, $, direct
     }
   }
 
+  const normalizeReviewId = (value: string | number | null | undefined, label: string) => {
+    if (value == null || value === "") return null
+
+    if (typeof value === "number") {
+      if (!Number.isFinite(value) || !Number.isInteger(value) || value <= 0) {
+        throw new Error(`${label} must be a positive finite integer. Omit it to let ControlKeel infer scope from the bound project.`)
+      }
+
+      return String(value)
+    }
+
+    const trimmed = String(value).trim()
+    if (/^[1-9]\d*$/.test(trimmed)) return trimmed
+
+    throw new Error(`${label} must be a positive integer string. Omit it to let ControlKeel infer scope from the bound project.`)
+  }
+
   const resolveReviewScope = async (
     explicitTaskId?: string | number | null,
     explicitSessionId?: string | number | null
   ) => {
-    if (explicitTaskId || explicitSessionId) {
+    const normalizedExplicitTaskId = normalizeReviewId(explicitTaskId, "task_id")
+    const normalizedExplicitSessionId = normalizeReviewId(explicitSessionId, "session_id")
+
+    if (normalizedExplicitTaskId || normalizedExplicitSessionId) {
       return {
-        taskId: explicitTaskId != null ? String(explicitTaskId) : null,
-        sessionId: explicitSessionId != null ? String(explicitSessionId) : null,
+        taskId: normalizedExplicitTaskId,
+        sessionId: normalizedExplicitSessionId,
         source: "explicit",
       }
     }
 
-    const envTaskId = process.env.CONTROLKEEL_TASK_ID
-    const envSessionId = process.env.CONTROLKEEL_SESSION_ID
+    const envTaskId = normalizeReviewId(process.env.CONTROLKEEL_TASK_ID, "CONTROLKEEL_TASK_ID")
+    const envSessionId = normalizeReviewId(process.env.CONTROLKEEL_SESSION_ID, "CONTROLKEEL_SESSION_ID")
 
     if (envTaskId || envSessionId) {
       return {
-        taskId: envTaskId ?? null,
-        sessionId: envSessionId ?? null,
+        taskId: envTaskId,
+        sessionId: envSessionId,
         source: "env",
       }
     }

package/index.js

@@ -173,22 +173,42 @@ export const ControlKeelGovernance = async ({ $, directory }) => {
     }
   }
 
+  const normalizeReviewId = (value, label) => {
+    if (value == null || value === "") return null
+
+    if (typeof value === "number") {
+      if (!Number.isFinite(value) || !Number.isInteger(value) || value <= 0) {
+        throw new Error(`${label} must be a positive finite integer. Omit it to let ControlKeel infer scope from the bound project.`)
+      }
+
+      return String(value)
+    }
+
+    const trimmed = String(value).trim()
+    if (/^[1-9]\d*$/.test(trimmed)) return trimmed
+
+    throw new Error(`${label} must be a positive integer string. Omit it to let ControlKeel infer scope from the bound project.`)
+  }
+
   const resolveReviewScope = async (explicitTaskId, explicitSessionId) => {
-    if (explicitTaskId || explicitSessionId) {
+    const normalizedExplicitTaskId = normalizeReviewId(explicitTaskId, "task_id")
+    const normalizedExplicitSessionId = normalizeReviewId(explicitSessionId, "session_id")
+
+    if (normalizedExplicitTaskId || normalizedExplicitSessionId) {
       return {
-        taskId: explicitTaskId != null ? String(explicitTaskId) : null,
-        sessionId: explicitSessionId != null ? String(explicitSessionId) : null,
+        taskId: normalizedExplicitTaskId,
+        sessionId: normalizedExplicitSessionId,
         source: "explicit",
       }
     }
 
-    const envTaskId = process.env.CONTROLKEEL_TASK_ID
-    const envSessionId = process.env.CONTROLKEEL_SESSION_ID
+    const envTaskId = normalizeReviewId(process.env.CONTROLKEEL_TASK_ID, "CONTROLKEEL_TASK_ID")
+    const envSessionId = normalizeReviewId(process.env.CONTROLKEEL_SESSION_ID, "CONTROLKEEL_SESSION_ID")
 
     if (envTaskId || envSessionId) {
       return {
-        taskId: envTaskId ?? null,
-        sessionId: envSessionId ?? null,
+        taskId: envTaskId,
+        sessionId: envSessionId,
         source: "env",
       }
     }

package/package.json

@@ -35,5 +35,5 @@
     "url": "git+https://github.com/aryaminus/controlkeel.git"
   },
   "type": "module",
-  "version": "0.3.2"
+  "version": "0.3.5"
 }