@saptools/service-flow 0.1.53 → 0.1.55

package/dist/cli.js

@@ -29,14 +29,13 @@ import {
   parsePackageJson,
   parseServiceBindings,
   parseVars,
-  projectBounded,
+  projectBoundedInOrder,
   reposByName,
   selectorRepoAmbiguousDiagnostic,
-  stableProjectionValue,
   trace,
   upsertRepository,
   upsertWorkspace
-} from "./chunk-LFH7C46B.js";
+} from "./chunk-GXYVIHJ5.js";
 
 // src/cli.ts
 import { Command } from "commander";
@@ -238,7 +237,7 @@ function migrate(db) {
 // package.json
 var package_default = {
   name: "@saptools/service-flow",
-  version: "0.1.53",
+  version: "0.1.55",
   description: "Trace SAP CAP service-to-service flows across multi-repository workspaces with runtime-aware graph resolution",
   type: "module",
   publishConfig: {
@@ -1178,7 +1177,7 @@ function boundDoctorValue(value) {
       output[key] = boundDoctorValue(child);
       continue;
     }
-    const projection = projectBounded(child.map(boundDoctorValue), compareDiagnostic);
+    const projection = projectBoundedInOrder(child.map(boundDoctorValue));
     output[key] = projection.items;
     addProjectionMetadata(output, input, key, projection);
   }
@@ -1240,9 +1239,6 @@ function projectionStem(key) {
   };
   return stems[key] ?? "repository";
 }
-function compareDiagnostic(left, right) {
-  return stableProjectionValue(left).localeCompare(stableProjectionValue(right));
-}
 function numericValue(value) {
   return typeof value === "number" && Number.isFinite(value) ? value : 0;
 }
@@ -1851,13 +1847,20 @@ function asRecords(value) {
 
 // src/output/table-output.ts
 function location(evidence) {
+  const selected = isRecord(evidence.selectedHandler) ? evidence.selectedHandler : void 0;
+  const selectedFile = selected?.sourceFile;
+  const selectedLine = selected?.sourceLine;
+  if (selectedFile || selectedLine)
+    return `${String(selectedFile ?? "")}:${String(selectedLine ?? "")}`;
   const file = evidence.file ?? evidence.sourceFile ?? evidence.handlerSourceFile ?? evidence.operationSourceFile ?? evidence.registrationSourceFile;
   const line = evidence.line ?? evidence.sourceLine ?? evidence.handlerSourceLine ?? evidence.operationSourceLine ?? evidence.registrationSourceLine;
   if (file || line) return `${String(file ?? "")}:${String(line ?? "")}`;
   const candidates = evidence.candidates;
+  if (Array.isArray(candidates) && candidates.some((candidate) => isRecord(candidate) && candidate.methodId !== void 0)) return ":";
   if (Array.isArray(candidates) && candidates.length > 0) {
-    const first = candidates[0];
-    return `${String(first.sourceFile ?? "")}:${String(first.sourceLine ?? "")}`;
+    const first = candidates.find(isRecord);
+    if (first)
+      return `${String(first.sourceFile ?? "")}:${String(first.sourceLine ?? "")}`;
   }
   return ":";
 }
@@ -2056,6 +2059,44 @@ function renderMermaid(trace2) {
 `;
 }
 
+// src/output/000-stdout-policy.ts
+var writers = /* @__PURE__ */ new WeakMap();
+function isRecord3(value) {
+  return Boolean(value && typeof value === "object" && !Array.isArray(value));
+}
+function isBrokenPipe(error) {
+  return isRecord3(error) && error.code === "EPIPE";
+}
+function asError(value) {
+  return value instanceof Error ? value : new Error(String(value));
+}
+function handleStreamError(state, error, onUnexpectedError) {
+  state.blocked = true;
+  if (isBrokenPipe(error) || state.unexpectedReported) return;
+  state.unexpectedReported = true;
+  onUnexpectedError(error);
+}
+function createStdoutWriter(stream, onUnexpectedError) {
+  const existing = writers.get(stream);
+  if (existing) return existing.writer;
+  const state = { blocked: false, unexpectedReported: false };
+  const writer = {
+    write(value) {
+      if (state.blocked) return false;
+      try {
+        stream.write(value);
+        return true;
+      } catch (error) {
+        handleStreamError(state, asError(error), onUnexpectedError);
+        return false;
+      }
+    }
+  };
+  stream.on("error", (error) => handleStreamError(state, error, onUnexpectedError));
+  writers.set(stream, { state, writer });
+  return writer;
+}
+
 // src/cli/000-clean.ts
 import fs6 from "fs/promises";
 import path6 from "path";
@@ -2116,6 +2157,10 @@ async function markCleanClaimFailed(dbPath, runId, error) {
 }
 
 // src/cli.ts
+var stdout = createStdoutWriter(process.stdout, fail);
+function writeStdout(value) {
+  stdout.write(value);
+}
 async function init(workspace, options) {
   const config = createWorkspaceConfig(
     workspace,
@@ -2138,7 +2183,7 @@ async function init(workspace, options) {
     });
   }
   db.close();
-  process.stdout.write(
+  writeStdout(
     `Workspace: ${config.rootPath}
 Database: ${config.dbPath}
 Repositories: ${repos.length}
@@ -2204,7 +2249,7 @@ function createProgram() {
         repo: opts.repo,
         force: Boolean(opts.force)
       });
-      process.stdout.write(
+      writeStdout(
         `Indexed ${r.indexedCount} repositories, skipped ${r.skippedCount}, ${r.fileCount} files, ${r.diagnosticCount} diagnostics
 `
       );
@@ -2214,7 +2259,7 @@ function createProgram() {
     (opts) => void withWorkspace(opts.workspace, (db, workspaceId) => {
       const r = linkWorkspace(db, workspaceId);
       const upgradeWarnings = linkUpgradeWarnings(db);
-      process.stdout.write(
+      writeStdout(
         `${upgradeWarnings.length ? `Warnings: ${upgradeWarnings.map((item) => String(item.code)).join(", ")}. Run service-flow doctor --strict for remediation.
 ` : ""}Linked ${r.edgeCount} edges: ${r.remoteResolvedCount} remote operation calls resolved, ${r.localResolvedCount} local operation calls resolved, ${r.unresolvedCount} unresolved operation calls, ${r.ambiguousCount} ambiguous operation calls, ${r.dynamicCount} dynamic operation calls, ${r.terminalCount} terminal call edges, ${r.dependencyResolvedCount} dependency resolved, ${r.dependencyAmbiguousCount} dependency ambiguous, ${r.implementationResolvedCount} implementation resolved, ${r.implementationAmbiguousCount} implementation ambiguous, ${r.implementationUnresolvedCount} implementation unresolved
 `
@@ -2245,7 +2290,7 @@ function createProgram() {
           maxDynamicCandidates: parsePositiveInteger(opts.maxDynamicCandidates, 5)
         }
       );
-      process.stdout.write(
+      writeStdout(
         opts.format === "json" ? renderTraceJson(result) : opts.format === "mermaid" ? renderMermaid(result) : renderTraceTable(result)
       );
     }).catch(fail)
@@ -2254,7 +2299,7 @@ function createProgram() {
   list.command("repos").option("--workspace <path>").action(
     (opts) => void withReadOnlyWorkspace(
       opts.workspace,
-      (db, workspaceId) => process.stdout.write(
+      (db, workspaceId) => writeStdout(
         renderJson(
           listRepositories(db, workspaceId).map((r) => ({
             name: r.name,
@@ -2269,35 +2314,35 @@ function createProgram() {
     (opts) => void withReadOnlyWorkspace(opts.workspace, (db, workspaceId) => {
       const selection = opts.repo ? selectRepository(db, opts.repo, workspaceId) : {};
       if (selection.diagnostic) {
-        process.stdout.write(renderJson([selection.diagnostic]));
+        writeStdout(renderJson([selection.diagnostic]));
         return;
       }
       const repo = selection.repo;
       const rows = db.prepare(
         "SELECT r.name repo,s.service_path servicePath,s.qualified_name qualifiedName FROM cds_services s JOIN repositories r ON r.id=s.repo_id WHERE r.workspace_id=? AND (? IS NULL OR s.repo_id=?) ORDER BY r.name,s.service_path"
       ).all(workspaceId, repo?.id, repo?.id);
-      process.stdout.write(renderJson(rows));
+      writeStdout(renderJson(rows));
     }).catch(fail)
   );
   list.command("operations").option("--workspace <path>").option("--repo <name>").option("--service <path>").action(
     (opts) => void withReadOnlyWorkspace(opts.workspace, (db, workspaceId) => {
       const selection = opts.repo ? selectRepository(db, opts.repo, workspaceId) : {};
       if (selection.diagnostic) {
-        process.stdout.write(renderJson([selection.diagnostic]));
+        writeStdout(renderJson([selection.diagnostic]));
         return;
       }
       const repo = selection.repo;
       const rows = db.prepare(
         "SELECT r.name repo,s.service_path servicePath,o.operation_name operation,o.operation_path path FROM cds_operations o JOIN cds_services s ON s.id=o.service_id JOIN repositories r ON r.id=s.repo_id WHERE r.workspace_id=? AND (? IS NULL OR s.repo_id=?) AND (? IS NULL OR s.service_path=?)"
       ).all(workspaceId, repo?.id, repo?.id, opts.service, opts.service);
-      process.stdout.write(renderJson(rows));
+      writeStdout(renderJson(rows));
     }).catch(fail)
   );
   list.command("calls").option("--workspace <path>").option("--repo <name>").option("--operation <name>").action(
     (opts) => void withReadOnlyWorkspace(opts.workspace, (db, workspaceId) => {
       const selection = opts.repo ? selectRepository(db, opts.repo, workspaceId) : {};
       if (selection.diagnostic) {
-        process.stdout.write(renderJson([selection.diagnostic]));
+        writeStdout(renderJson([selection.diagnostic]));
         return;
       }
       const repo = selection.repo;
@@ -2312,7 +2357,7 @@ function createProgram() {
         opts.operation ? `/${opts.operation}` : void 0,
         opts.operation ? `%${opts.operation}%` : void 0
       );
-      process.stdout.write(renderJson(rows));
+      writeStdout(renderJson(rows));
     }).catch(fail)
   );
   program.command("graph").option("--workspace <path>").option("--repo <name>").option("--operation <name>").option("--service <path>").option("--path <operationPath>").option("--format <format>", "mermaid|json", "mermaid").option("--implementation-repo <name>").option("--implementation-hint <scope>", "scoped implementation hint", collect, []).option("--var <key=value>", "dynamic variable", collect, []).option("--dynamic-mode <mode>", "strict|candidates|infer", "strict").option("--max-dynamic-candidates <n>", "maximum dynamic candidates to show", "5").action(
@@ -2338,7 +2383,7 @@ function createProgram() {
           maxDynamicCandidates: parsePositiveInteger(opts.maxDynamicCandidates, 5)
         }
       );
-      process.stdout.write(
+      writeStdout(
         opts.format === "json" ? renderTraceJson(result) : renderMermaid(result)
       );
     }).catch(fail)
@@ -2347,7 +2392,7 @@ function createProgram() {
   inspect.command("repo").argument("<name>").option("--workspace <path>").action(
     (name, opts) => void withReadOnlyWorkspace(opts.workspace, (db, workspaceId) => {
       const selection = selectRepository(db, name, workspaceId);
-      process.stdout.write(renderJson(
+      writeStdout(renderJson(
         selection.repo ?? selection.diagnostic ?? { error: "repo not found" }
       ));
     }).catch(fail)
@@ -2357,20 +2402,20 @@ function createProgram() {
       const rows = db.prepare(
         "SELECT o.* FROM cds_operations o JOIN cds_services s ON s.id=o.service_id JOIN repositories r ON r.id=s.repo_id WHERE r.workspace_id=? AND (o.operation_name=? OR o.operation_path=?)"
       ).all(workspaceId, selector, selector);
-      process.stdout.write(renderJson(rows));
+      writeStdout(renderJson(rows));
     }).catch(fail)
   );
   program.command("doctor").option("--workspace <path>").option("--strict").option("--detail").option("--format <format>", "json|table").action(
     (opts) => void withReadOnlyWorkspace(opts.workspace, (db) => {
       const allDiagnostics = doctorDiagnostics(db, Boolean(opts.strict), { detail: Boolean(opts.detail) });
-      process.stdout.write(renderDoctorDiagnostics(allDiagnostics, opts.format));
+      writeStdout(renderDoctorDiagnostics(allDiagnostics, opts.format));
     }).catch(fail)
   );
   program.command("clean").option("--workspace <path>").option("--db-only").action(
     (opts) => void (async () => {
       const config = await loadWorkspaceConfig(opts.workspace);
       await cleanWorkspaceState(config, Boolean(opts.dbOnly));
-      process.stdout.write("Cleaned service-flow state\n");
+      writeStdout("Cleaned service-flow state\n");
     })().catch(fail)
   );
   return program;