@hublo/sentinel 1.4.0-alpha.40 → 1.4.0-alpha.41

package/dist/bin/sentinel.js

@@ -15,10 +15,10 @@ import {
   palette,
   registerAdapters,
   resolve
-} from "../chunk-USVNSL76.js";
+} from "../chunk-2UU57Y2T.js";
 import {
   resolveContext
-} from "../chunk-TDQGNNCV.js";
+} from "../chunk-I3FX32E7.js";
 import "../chunk-O7REVMOC.js";
 import "../chunk-QXFCZON7.js";
 import {
@@ -145,7 +145,7 @@ async function runValidate(ctx) {
     );
   }
   if (!ctx.targets.includes("test")) return 1;
-  const { validateModuleTests } = await import("../validate-JUMFX4ZF.js");
+  const { validateModuleTests } = await import("../validate-ANM7RQ3C.js");
   const outcomes = modules.flatMap(
     (module) => validateModuleTests(module.name, module.root)
   );

package/dist/{chunk-USVNSL76.js → chunk-2UU57Y2T.js}

@@ -53,7 +53,7 @@ import {
   selfCommand,
   splitToolArgs,
   walk
-} from "./chunk-TDQGNNCV.js";
+} from "./chunk-I3FX32E7.js";
 import {
   parseJsonc
 } from "./chunk-QXFCZON7.js";

package/dist/{chunk-TDQGNNCV.js → chunk-I3FX32E7.js}

@@ -1963,6 +1963,25 @@ function rewriteHookReturnValue(source, program) {
   return { source: result, rewritten: edits.length };
 }
 
+// src/roles/test/codemod/asyncify.ts
+function alreadyAPromise(text) {
+  return /^(?:Promise|PromiseLike)\s*</.test(text.trim());
+}
+function asyncifyEdits(source, fn) {
+  const edits = [{ start: fn.start, end: fn.start, text: "async " }];
+  const annotation = fn.returnType;
+  const declared = annotation?.typeAnnotation;
+  if (declared?.start === void 0 || declared.end === void 0) return edits;
+  const text = source.slice(declared.start, declared.end);
+  if (alreadyAPromise(text)) return edits;
+  edits.push({
+    start: declared.start,
+    end: declared.end,
+    text: `Promise<${text}>`
+  });
+  return edits;
+}
+
 // src/roles/test/codemod/named-rules.ts
 import { existsSync as existsSync11, readFileSync as readFileSync10 } from "node:fs";
 import { createRequire as createRequire2 } from "node:module";
@@ -2474,7 +2493,7 @@ function applyNamedRules(source, program, fileName = "") {
     }
   });
   for (const fn of asyncify) {
-    edits.push({ start: fn.start, end: fn.start, text: "async " });
+    edits.push(...asyncifyEdits(source, fn));
   }
   const namespaceLines = namespaces.planned().map(
     ({ specifier, binding }) => `import * as ${binding} from '${specifier}'${fileUsesSemicolons(source) ? ";" : ""}`
@@ -2545,6 +2564,12 @@ function rewriteIsolateModules(source, program) {
   const refusals = [];
   const asyncify = /* @__PURE__ */ new Set();
   let rewritten = 0;
+  const awaitedCall = /* @__PURE__ */ new Map();
+  walkWithEnclosingFunction(program, void 0, (node) => {
+    if (node.type !== "AwaitExpression") return;
+    const argument = node.argument;
+    if (argument !== void 0) awaitedCall.set(argument, node);
+  });
   walkWithEnclosingFunction(program, void 0, (node, enclosing) => {
     const unusable = unusableIsolatingCall(node);
     if (unusable !== void 0) {
@@ -2558,10 +2583,11 @@ function rewriteIsolateModules(source, program) {
     if (found === void 0) return;
     const callbackText = source.slice(found.callback.start, found.callback.end);
     const callback = found.callback.async === true ? callbackText : `async ${callbackText}`;
-    const indent = indentationAt(source, found.call.start);
+    const replaced = awaitedCall.get(found.call) ?? found.call;
+    const indent = indentationAt(source, replaced.start);
     edits.push({
-      start: found.call.start,
-      end: found.call.end,
+      start: replaced.start,
+      end: replaced.end,
       text: `vi.resetModules()
 ${indent}await (${callback})()`
     });
@@ -2569,7 +2595,7 @@ ${indent}await (${callback})()`
     if (enclosing !== void 0 && enclosing.async !== true) asyncify.add(enclosing);
   });
   for (const fn of asyncify) {
-    edits.push({ start: fn.start, end: fn.start, text: "async " });
+    edits.push(...asyncifyEdits(source, fn));
   }
   const result = applyEdits(source, edits);
   return { source: result, rewritten, refusals };

package/dist/index.js

@@ -6,10 +6,10 @@ import {
   registerAdapters,
   resolve,
   setDefaultRunner
-} from "./chunk-USVNSL76.js";
+} from "./chunk-2UU57Y2T.js";
 import {
   BaseAdapter
-} from "./chunk-TDQGNNCV.js";
+} from "./chunk-I3FX32E7.js";
 import "./chunk-O7REVMOC.js";
 import "./chunk-QXFCZON7.js";
 import "./chunk-NW6UHNYX.js";

package/dist/{validate-JUMFX4ZF.js → validate-ANM7RQ3C.js}

@@ -7,7 +7,7 @@ import {
   readTestAdoption,
   resolveVitest,
   suiteOf
-} from "./chunk-TDQGNNCV.js";
+} from "./chunk-I3FX32E7.js";
 import "./chunk-O7REVMOC.js";
 import "./chunk-QXFCZON7.js";
 import "./chunk-NW6UHNYX.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hublo/sentinel",
-  "version": "1.4.0-alpha.40",
+  "version": "1.4.0-alpha.41",
   "description": "One CLI that guards code health across Hublo repos: shared lint/typescript/build/test presets, static & dynamic analysis, and architecture checks.",
   "type": "module",
   "license": "MIT",