@overaihq/cli 0.5.0 → 0.6.0

package/README.md

@@ -24,7 +24,7 @@ overai clone <workspaceId>      # materialize resources and WORKSPACE.md
 # ...edit files...
 overai status                   # local edits + remote drift
 overai diff                     # what changed
-overai review                   # validation + best-practice review packet
+overai review                   # local checks + authenticated TypeScript lint + guides
 overai report bug "<details>"    # report broken or unexplained behavior
 overai report feature "<details>" # request a missing platform capability
 overai push -m "why"            # create new versions (not live yet)

package/dist/index.js

@@ -34764,7 +34764,8 @@ var workflow = `# CLI editing workflow
    each nested workspace you need separately.
 2. Edit files. The id in each filename is authoritative; slugs are cosmetic.
 3. \`overai status\` / \`overai diff\` \u2014 review what changed.
-4. \`overai review\` \u2014 validation + these guides; fix findings first.
+4. \`overai review\` \u2014 local checks, authenticated TypeScript lint against the
+   live workspace, and these guides; fix findings first.
 5. \`overai push -m "why"\` \u2014 creates NEW versions (not live). Fast-forward
    only: if the resource moved remotely, pull, re-apply, retry. Write commit
    messages like PR titles \u2014 they're the version history.
@@ -34802,11 +34803,11 @@ var runBestPractices = (args) => {
   );
 };
 var runReview = async () => {
-  const root = findWorkspaceRoot();
-  const manifest = readManifest(root);
+  const { root, manifest } = openClone();
   const findings2 = [];
   const touchedTypes = /* @__PURE__ */ new Set();
   const changed = [];
+  const lintTargets = [];
   for (const [id, resource] of Object.entries(manifest.resources)) {
     const { state, changedFiles } = localState(root, resource);
     if (state !== "modified") continue;
@@ -34853,6 +34854,37 @@ var runReview = async () => {
         }
       }
     }
+    if (resource.type === "webPage") {
+      changedFiles.filter((relPath) => relPath.endsWith(".tsx")).forEach((relPath) => {
+        const code = readWorkingFile(root, relPath);
+        if (code != null) {
+          lintTargets.push({ label: relPath, code, surface: "webPage" });
+        }
+      });
+    }
+    if (resource.type === "function") {
+      const codeFile = resource.files.find((path) => path.endsWith(".ts"));
+      const metaFile = resource.files.find(
+        (path) => path.endsWith(".meta.json")
+      );
+      const code = codeFile == null ? null : readWorkingFile(root, codeFile);
+      const meta = metaFile == null ? null : readWorkingFile(root, metaFile);
+      if (codeFile != null && code != null) {
+        const parsedMeta = (() => {
+          try {
+            return JSON.parse(meta ?? "null");
+          } catch {
+            return null;
+          }
+        })();
+        lintTargets.push({
+          label: codeFile,
+          code,
+          surface: "function",
+          functionOutputSchema: parsedMeta != null && typeof parsedMeta === "object" && "outputSchema" in parsedMeta && typeof parsedMeta.outputSchema === "string" ? parsedMeta.outputSchema : void 0
+        });
+      }
+    }
     if (resource.type === "automation") {
       const docFile = resource.files.find((f) => f.endsWith(".json"));
       const raw2 = docFile == null ? null : readWorkingFile(root, docFile);
@@ -34864,6 +34896,25 @@ var runReview = async () => {
               (f) => `${docFile} elements.${f}`
             )
           );
+          const nodes = Array.isArray(doc.elements) ? [...doc.elements] : [];
+          for (const node of nodes) {
+            if (node == null || typeof node !== "object") continue;
+            const nodeRecord = node;
+            if (Array.isArray(nodeRecord.elements)) {
+              nodes.push(...nodeRecord.elements);
+            }
+            const inputs = nodeRecord.inputs;
+            if (nodeRecord.nodeTypeId !== "TYPESCRIPT_CODE" || inputs == null || typeof inputs !== "object") {
+              continue;
+            }
+            const code = inputs.code;
+            if (typeof code !== "string") continue;
+            lintTargets.push({
+              label: `${docFile} ${typeof nodeRecord.variableId === "string" ? nodeRecord.variableId : "unknown node"}`,
+              code,
+              surface: "automationCode"
+            });
+          }
         } catch {
         }
       }
@@ -34873,6 +34924,47 @@ var runReview = async () => {
     process.stdout.write("Nothing modified locally \u2014 nothing to review.\n");
     return;
   }
+  for (const target of lintTargets) {
+    try {
+      const response = await v2Fetch("/v2/architect/lint", {
+        method: "POST",
+        headers: { "Content-Type": "application/json" },
+        body: JSON.stringify({
+          code: target.code,
+          surface: target.surface,
+          workspaceId: manifest.workspaceId,
+          functionOutputSchema: target.functionOutputSchema
+        })
+      });
+      const responseText = await response.text();
+      if (!response.ok) {
+        throw new Error(
+          `${target.label}: HTTP ${String(response.status)}${responseText ? ` \u2014 ${responseText}` : ""}`
+        );
+      }
+      const parsed = lintResponseSchema.safeParse(
+        (() => {
+          try {
+            return JSON.parse(responseText);
+          } catch {
+            return null;
+          }
+        })()
+      );
+      if (!parsed.success) {
+        throw new Error(`${target.label}: invalid response from lint service`);
+      }
+      findings2.push(
+        ...parsed.data.diagnostics.map(
+          (diagnostic) => `${target.label}:${String(diagnostic.line)}:${String(diagnostic.column)}: ${diagnostic.severity}: ${diagnostic.message}${diagnostic.ruleId == null ? "" : ` (${diagnostic.ruleId})`}`
+        )
+      );
+    } catch (error) {
+      findings2.push(
+        `${target.label}: remote lint could not run \u2014 ${error instanceof Error ? error.message : String(error)}`
+      );
+    }
+  }
   const guides = ["workflow", ...touchedTypes].map((t) => t === "webPage" || t === "printablePage" ? "pages" : t).map((t) => t === "automation" ? "automations" : t).map((t) => t === "function" ? "functions" : t);
   const guideDocs = [...new Set(guides)].map((t) => BEST_PRACTICES[t]).filter((d) => d != null);
   process.stdout.write(
@@ -35540,7 +35632,7 @@ Workspace sync (edit resources as local files):
   overai open [filter | id]            Open a resource (or the workspace) in the webapp
   overai dev <filter> [--port N] [--open]
                                        Live-preview a web page from local files
-  overai review                        Validation + best-practice review packet
+  overai review                        Local checks + authenticated TypeScript lint + guides
   overai best-practices [topic]        Print bundled guides
   overai report bug|feature "<details>"
                                        Send product feedback (use - for stdin)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@overaihq/cli",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "Create, edit, deploy, and run OverAI workspace resources from the command line",
   "type": "module",
   "homepage": "https://overai.com",
@@ -30,7 +30,7 @@
     "lint": "echo 'Run lint from the root of the repo'",
     "ts-check": "tsc --noEmit",
     "build": "tsup",
-    "build:watch": "tsc -w",
+    "build:watch": "tsup --watch",
     "start": "node dist/index.js",
     "test": "vitest",
     "test:localhost": "vitest --run --config vitest.localhost.config.js"