fallow-type-aware 3.28.0 → 3.30.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fallow-type-aware",
-  "version": "3.28.0",
+  "version": "3.30.0",
   "description": "Optional TypeScript-Go semantic refinement sidecar for Fallow",
   "license": "MIT",
   "repository": {

package/src/cli.mjs

@@ -2,7 +2,6 @@ import path from "node:path";
 import { createInterface } from "node:readline";
 
 import {
-  ANALYSIS_OPERATION,
   SESSION_ENVELOPE_TYPES,
   STATUS_OPERATION,
   WIRE_PROTOCOL_VERSION,
@@ -35,18 +34,6 @@ const writeJson = (output, value) => {
   output.write(`${JSON.stringify(value)}\n`);
 };
 
-const handleArguments = (args, output) => {
-  if (args.length === 1 && args[0] === "--status") {
-    writeJson(output, createStatusResponse());
-    return true;
-  }
-  // --session is dispatched by run() before this function is reached.
-  if (args.length > 0) {
-    throw new Error(`unknown argument: ${args[0]}`);
-  }
-  return false;
-};
-
 const sessionFileChanges = (value, root) => {
   if (value === undefined) return undefined;
   if (value?.invalidate_all === true && Object.keys(value).length === 1) {
@@ -117,7 +104,7 @@ const runSession = async (input, output) => {
       writeJson(output, {
         request_id: envelope.request_id,
         revision: envelope.revision,
-        response: responseFor(request, result, performance.now() - startedAt),
+        response: responseFor(result, performance.now() - startedAt),
       });
     }
   } finally {
@@ -137,19 +124,20 @@ const isStatusRequest = (request) =>
   Object.keys(request ?? {}).length === STATUS_FIELDS.length &&
   STATUS_FIELDS.every(([name, value]) => request[name] === value);
 
-const responseFor = (request, result, elapsedMs) => {
-  if (request.protocolVersion !== WIRE_PROTOCOL_VERSION) {
-    throw new Error(`unsupported protocol_version ${String(request.protocolVersion)}`);
-  }
-  return createSemanticResponse({ ...result, elapsedMs });
-};
+const responseFor = (result, elapsedMs) => createSemanticResponse({ ...result, elapsedMs });
 
 export const run = async ({ input, output, args = [] }) => {
-  if (args.length === 1 && args[0] === "--session") {
-    await runSession(input, output);
-    return;
+  if (args.length > 0) {
+    if (args.length === 1 && args[0] === "--session") {
+      await runSession(input, output);
+      return;
+    }
+    if (args.length === 1 && args[0] === "--status") {
+      writeJson(output, createStatusResponse());
+      return;
+    }
+    throw new Error(`unknown argument: ${args[0]}`);
   }
-  if (handleArguments(args, output)) return;
   const startedAt = performance.now();
   const rawRequest = parseJsonRequest(await readAll(input));
   if (isStatusRequest(rawRequest)) {
@@ -157,9 +145,6 @@ export const run = async ({ input, output, args = [] }) => {
     return;
   }
   const request = parseRequest(rawRequest);
-  if (rawRequest.operation !== ANALYSIS_OPERATION) {
-    throw new Error(`unsupported operation ${String(rawRequest.operation)}`);
-  }
   const result = analyzeSemanticQueries(request);
-  writeJson(output, responseFor(request, result, performance.now() - startedAt));
+  writeJson(output, responseFor(result, performance.now() - startedAt));
 };

package/src/protocol.mjs

@@ -372,7 +372,7 @@ const parseBatchRequest = (value) => {
   validateGraphQueryCount(queries);
   requireUniqueCandidateIds(queries);
   const evidenceLimit = parseEvidenceLimit(value.evidence_limit);
-  return { protocolVersion: WIRE_PROTOCOL_VERSION, root, projects, queries, evidenceLimit };
+  return { root, projects, queries, evidenceLimit };
 };
 
 export const parseRequest = (value) => {

package/src/semantic-identity.mjs

@@ -63,7 +63,7 @@ export const relativePath = (root, fileName) =>
 
 export const sourceFileIdentity = (sourceFile) => canonicalFileIdentity(sourceFile.fileName);
 
-export const positions = (node) => {
+const positions = (node) => {
   const sourceFile = node.getSourceFile();
   const nodes = node.name ? [node, node.name] : [node];
   return nodes.map((candidate) => {
@@ -79,7 +79,7 @@ export const positions = (node) => {
 
 export const nodeText = (node) => node?.text;
 
-export const declarationKind = (node) =>
+const declarationKind = (node) =>
   DECLARATION_KIND_RULES.find(([checks]) => matchesAny(node, checks))?.[1];
 
 export const declarationNamespaces = (node) =>
@@ -95,7 +95,7 @@ const ownerName = (node) => {
   return ownerName(node.parent);
 };
 
-export const declarationOwner = (node) => ownerName(node.parent);
+const declarationOwner = (node) => ownerName(node.parent);
 
 export const ownerDeclaration = (node) => {
   let current = node?.parent;