no-mistakes 0.46.1 → 0.48.0

package/README.md

@@ -43,8 +43,6 @@ const {
     root: process.cwd(),
     files: ["src/main.mts"],
     relationships: ["import"],
-    timeout: 30,
-    lockTimeout: 30,
   });
   const tests = await dependents({
     root: process.cwd(),
@@ -121,10 +119,11 @@ const {
 
 CLI and Node analyses share a per-user machine-wide lock. CLI flags
 `--timeout`, `--lock-timeout`, and `--fail-on-lock` have Node equivalents
-`timeout`, `lockTimeout`, and `failOnLock`. Both timeouts default to 30 seconds;
-`0` disables either CLI timeout, while `0` or `null` disables it in Node.
-Waiting does not alter successful output, and Node lock/timeout failures reject
-the returned Promise.
+`timeout`, `lockTimeout`, and `failOnLock`. CLI timeouts default to 30 seconds;
+Node/N-API omits both deadlines unless you set them. `0` disables either CLI
+timeout, while `0` or `null` disables it in Node. While waiting, stderr reports
+the holder pid and elapsed seconds. Waiting does not alter successful output,
+and Node lock/timeout failures reject the returned Promise.
 
 Dependency graph, query, and test-planning resolution is per workspace by
 default: when `tsconfig` is omitted, each import uses the config that owns its

package/flow-types.d.ts

@@ -15,7 +15,14 @@ export interface FlowOptions {
 
 export interface FlowNode {
   id: string;
-  kind: "file" | "symbol" | "module" | "queue-job" | "workflow-job" | "workflow-step";
+  kind:
+    | "file"
+    | "symbol"
+    | "module"
+    | "queue-job"
+    | "workflow-job"
+    | "workflow-step"
+    | "trpc-procedure";
   depth: number;
   file?: string;
   symbol?: string;
@@ -27,6 +34,10 @@ export interface FlowNode {
   job?: string;
   /** Zero-based step index for a virtual GitHub Actions workflow step node. */
   step?: number;
+  /** Router file for a virtual tRPC procedure node. */
+  routerFile?: string;
+  /** Dotted procedure path for a virtual tRPC procedure node (`user.get`). */
+  procedure?: string;
 }
 
 export interface FlowEdge {

package/index.d.ts

@@ -24,6 +24,7 @@ import type {
   ResolveCheckFilesOptions,
   ResolveCheckResult,
   ResolveCheckBatchResult,
+  ResolvedConfig,
   GraphEdge,
   PlaywrightOptions,
   PlaywrightRelatedOptions,
@@ -98,6 +99,9 @@ export function resolveCheck(
 export function fetches(options?: WithInvocationOptions<FetchesOptions>): Promise<FetchReport>;
 export function flow(options: WithInvocationOptions<FlowOptions>): Promise<FlowReport>;
 export function check(options?: WithInvocationOptions<CheckOptions>): Promise<CheckReport>;
+export function resolveConfig(
+  options?: WithInvocationOptions<ProjectOptions>,
+): Promise<ResolvedConfig>;
 export function validateMermaidMarkdown(
   options: WithInvocationOptions<MermaidValidationOptions>,
 ): Promise<MermaidValidationResult>;

package/index.js

@@ -7,7 +7,8 @@ const planning = require("./planning");
 const { createWorkflowTopologyIndex } = require("./workflow-topology-index");
 
 async function callJson(fn, options) {
-  return JSON.parse(await fn(JSON.stringify(options || {})));
+  const input = Buffer.from(JSON.stringify(options || {}));
+  return JSON.parse(await fn(input));
 }
 
 function createJsonApis(descriptors) {
@@ -23,6 +24,7 @@ const jsonApis = createJsonApis({
   analyzeProject: "analyzeProjectJson",
   callSites: "callSitesJson",
   check: "checkJson",
+  resolveConfig: "resolveConfigJson",
   ciEnv: "ciEnvJson",
   ciImpact: "ciImpactJson",
   ciTopology: "ciTopologyJson",
@@ -66,6 +68,7 @@ module.exports.version = version;
 module.exports.analyzeProject = jsonApis.analyzeProject;
 module.exports.callSites = jsonApis.callSites;
 module.exports.check = jsonApis.check;
+module.exports.resolveConfig = jsonApis.resolveConfig;
 module.exports.ciEnv = jsonApis.ciEnv;
 module.exports.ciImpact = jsonApis.ciImpact;
 module.exports.ciTopology = jsonApis.ciTopology;

package/index.mjs

@@ -0,0 +1,64 @@
+import { createRequire } from "node:module";
+
+const require = createRequire(import.meta.url);
+const cjs = require("./index.js");
+
+export const {
+  analyzeProject,
+  callSites,
+  check,
+  ciEnv,
+  ciImpact,
+  ciTopology,
+  createWorkflowTopologyIndex,
+  dataPw,
+  deadExports,
+  dependencies,
+  dependents,
+  effects,
+  exportsOf,
+  fetches,
+  flow,
+  impactedChecks,
+  importUsages,
+  importers,
+  infraOutputs,
+  infraResourceRefs,
+  infraTestFor,
+  lockfileDiff,
+  playwrightCheck,
+  playwrightEdges,
+  playwrightRelated,
+  playwrightTests,
+  queueCheck,
+  queueEdges,
+  queueRelated,
+  queues,
+  reactAnalyze,
+  reactCheck,
+  reactUsages,
+  registryExtension,
+  related,
+  resolveCheck,
+  resolveConfig,
+  rscCallers,
+  serverContracts,
+  serverRouteEdges,
+  serverRouteList,
+  serverRouteRelated,
+  serverRoutes,
+  swiftImporters,
+  swiftTestTargets,
+  symbols,
+  testsComment,
+  testsGraph,
+  testsGraphMermaid,
+  testsImpact,
+  testsPlan,
+  testsTargets,
+  testsWhy,
+  validateMermaidMarkdown,
+  version,
+} = cjs;
+
+export default cjs;

package/invocation-types.d.ts

@@ -1,8 +1,8 @@
 /** Controls shared by every analysis invocation. Durations are in seconds. */
 export interface InvocationOptions {
-  /** Command execution timeout. Defaults to 30; 0 or null disables it. */
+  /** Command execution timeout in whole non-negative seconds. Omit, `0`, or `null` disables it. CLI default remains 30. */
   timeout?: number | null;
-  /** Maximum time to wait for the machine-wide lock. Defaults to 30; 0 or null waits indefinitely. */
+  /** Maximum time to wait for the machine-wide lock, in whole non-negative seconds. Omit, `0`, or `null` waits indefinitely. CLI default remains 30. */
   lockTimeout?: number | null;
   /** Fail immediately instead of waiting when another invocation holds the lock. */
   failOnLock?: boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "no-mistakes",
-  "version": "0.46.1",
+  "version": "0.48.0",
   "description": "Static codebase analysis tools for TS/JS dependencies, dependents, and symbols",
   "license": "MIT",
   "repository": {
@@ -15,6 +15,7 @@
     "bin/",
     "*.d.ts",
     "index.js",
+    "index.mjs",
     "planning.js",
     "workflow-topology-index.js",
     "scripts/install.js",
@@ -24,6 +25,15 @@
   ],
   "main": "index.js",
   "types": "index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./index.d.ts",
+      "import": "./index.mjs",
+      "require": "./index.js"
+    },
+    "./planning.js": "./planning.js",
+    "./workflow-topology-index.js": "./workflow-topology-index.js"
+  },
   "publishConfig": {
     "access": "public"
   },

package/planning.js

@@ -3,7 +3,8 @@
 const native = require(process.env.NO_MISTAKES_TEST_NAPI_ADDON_PATH || "./bin/no-mistakes.node");
 
 async function callJson(fn, options) {
-  return JSON.parse(await fn(JSON.stringify(options || {})));
+  const input = Buffer.from(JSON.stringify(options || {}));
+  return JSON.parse(await fn(input));
 }
 
 function createJsonApis(descriptors) {
@@ -16,11 +17,13 @@ function createJsonApis(descriptors) {
 }
 
 async function testsComment(options) {
-  return native.testsCommentMarkdown(JSON.stringify(options || {}));
+  const input = Buffer.from(JSON.stringify(options || {}));
+  return String(await native.testsCommentMarkdown(input));
 }
 
 async function testsGraphMermaid(options) {
-  return native.testsGraphMermaid(JSON.stringify(options || {}));
+  const input = Buffer.from(JSON.stringify(options || {}));
+  return String(await native.testsGraphMermaid(input));
 }
 
 const jsonApis = createJsonApis({

package/resolve-config-types.d.ts

@@ -0,0 +1,50 @@
+import type { TestPlanFramework } from "./test-types";
+
+export interface ResolvedConfig {
+  configPath?: string | null;
+  frontendApps: ResolvedFrontendApp[];
+  playwright: ResolvedPlaywright;
+  vitestFullSuiteTriggers: ResolvedTrigger[];
+  fullSuiteTriggers: ResolvedFrameworkTriggers[];
+}
+
+export interface ResolvedFrontendApp {
+  project?: string | null;
+  root: string;
+  routeRoot: string;
+  selectorRoots: string[];
+}
+
+export interface ResolvedPlaywright {
+  coverageRoutes: boolean;
+  coverageSelectors: boolean;
+  frontendRoot?: string | null;
+  selectorRoots: string[];
+  apps: ResolvedPlaywrightApp[];
+}
+
+export interface ResolvedPlaywrightApp {
+  playwrightProject: string;
+  project?: string | null;
+  frontendRoot?: string | null;
+  selectorRoots: string[];
+  rewrites: ResolvedRewrite[];
+  ignoreRoutes: string[];
+}
+
+export interface ResolvedRewrite {
+  source: string;
+  destination: string;
+}
+
+export interface ResolvedTrigger {
+  name: string;
+  paths: string[];
+  targets: string[];
+  source: "triggers" | "projects";
+}
+
+export interface ResolvedFrameworkTriggers {
+  framework: TestPlanFramework;
+  triggers: ResolvedTrigger[];
+}

package/test-types.d.ts

@@ -10,7 +10,10 @@ export type TestPlanFramework =
   | "go"
   | "cargo"
   | "rails"
-  | "php";
+  | "php"
+  | "java"
+  | "kotlin"
+  | "jest";
 
 interface TestsPlanOptionsBase {
   framework?: TestPlanFramework;
@@ -36,6 +39,8 @@ interface TestsPlanOptionsBase {
   limitPercent?: number;
   limitFiles?: number;
   globalConfigFallback?: boolean;
+  /** Include the markdown PR comment as `comment` on the returned plan. */
+  includeComment?: boolean;
 }
 
 /**
@@ -86,11 +91,31 @@ export interface TestsTargetsOptions {
 export interface TestPlan {
   /** Complete deterministic changed-file inventory, relative to the request root. */
   changed_files: string[];
+  /** @deprecated Prefer `changedFiles` once both keys are present. */
+  changedFiles?: string[];
   selected_tests: SelectedTest[];
+  selectedTests?: SelectedTest[];
   groups?: TestPlanGroup[];
   warnings: TestPlanWarning[];
   fallback_triggered: boolean;
+  fallbackTriggered?: boolean;
   fallback_reason?: string | null;
+  fallbackReason?: string | null;
+  execution_targets?: GroupedExecutionTarget[];
+  executionTargets?: GroupedExecutionTarget[];
+  comment?: string | null;
+}
+
+export interface GroupedExecutionTarget {
+  runner: TestPlanFramework;
+  config?: string | null;
+  project?: string | null;
+  baseCommand?: string[];
+  base_command: string[];
+  runnerArgs?: string[];
+  runner_args: string[];
+  testFiles?: string[];
+  test_files: string[];
 }
 
 export interface SelectedTest {

package/traversal-types.d.ts

@@ -31,7 +31,10 @@ export type Relationship =
   | "rust"
   | "ruby"
   | "php"
+  | "java"
+  | "kotlin"
   | "resource"
+  | "trpc"
   | "all";
 
 export interface TraverseOptions {
@@ -59,6 +62,10 @@ export interface DependencyFile {
   job?: string;
   /** Zero-based step index for a virtual GitHub Actions workflow step node. */
   step?: number;
+  /** Router file for a virtual tRPC procedure node. */
+  routerFile?: string;
+  /** Dotted procedure path for a virtual tRPC procedure node (`user.get`). */
+  procedure?: string;
   module?: string;
   depth: number;
   via?: string[];

package/types.d.ts

@@ -11,3 +11,4 @@ export * from "./workflow-topology-types";
 export * from "./query-types";
 export * from "./named-query-types";
 export * from "./mermaid-types";
+export * from "./resolve-config-types";