@mnci/az-durable 0.1.2 → 0.1.3

package/dist/eslint-plugin.cjs.js

@@ -8,7 +8,8 @@
  * `@typescript-eslint/utils` because this package ships **zero runtime
  * dependencies**, and a type-only dependency is still a dependency a consumer
  * must be able to resolve. These cover exactly what the three rules use.
- */ /**
+ */
+/**
  * Whether a call expression registers an orchestration.
  *
  * @remarks
@@ -25,21 +26,22 @@
  * @returns `true` when the call registers an orchestration.
  * @throws Never - pure inspection.
  * @typeParam None - this function has no generic type parameters.
- */ function isOrchestrationRegistration(node) {
-    var callee = node.callee;
-    if (callee === undefined) {
-        return false;
-    }
-    if (callee.type === 'Identifier' && callee.name === 'defineOrchestration') {
-        return true;
-    }
-    // df.app.orchestration(...) — match on the trailing property, so any local
-    // alias for the namespace still matches.
-    if (callee.type === 'MemberExpression') {
-        var property = callee.property;
-        return (property === null || property === void 0 ? void 0 : property.type) === 'Identifier' && property.name === 'orchestration';
-    }
+ */
+function isOrchestrationRegistration(node) {
+  const callee = node.callee;
+  if (callee === undefined) {
     return false;
+  }
+  if (callee.type === 'Identifier' && callee.name === 'defineOrchestration') {
+    return true;
+  }
+  // df.app.orchestration(...) — match on the trailing property, so any local
+  // alias for the namespace still matches.
+  if (callee.type === 'MemberExpression') {
+    const property = callee.property;
+    return property?.type === 'Identifier' && property.name === 'orchestration';
+  }
+  return false;
 }
 /**
  * The name a callee refers to, if it is a plain identifier or member access.
@@ -55,28 +57,30 @@
  * @returns The identifier or property name, or `undefined`.
  * @throws Never - pure inspection.
  * @typeParam None - this function has no generic type parameters.
- */ function calleeName(node) {
-    if (node === undefined) {
-        return undefined;
-    }
-    if (node.type === 'Identifier') {
-        return node.name;
-    }
-    if (node.type === 'MemberExpression') {
-        var property = node.property;
-        if ((property === null || property === void 0 ? void 0 : property.type) === 'Identifier') {
-            return property.name;
-        }
-    }
+ */
+function calleeName(node) {
+  if (node === undefined) {
     return undefined;
+  }
+  if (node.type === 'Identifier') {
+    return node.name;
+  }
+  if (node.type === 'MemberExpression') {
+    const property = node.property;
+    if (property?.type === 'Identifier') {
+      return property.name;
+    }
+  }
+  return undefined;
 }
 
-/** Global reads that differ on every replay, with the replacement to suggest. */ var FORBIDDEN_CALLS = {
-    'Date.now': 'now(context).getTime()',
-    'Math.random': 'context.df.newGuid(...) or an activity',
-    'crypto.randomUUID': 'context.df.newGuid(...)',
-    fetch: 'an activity — network calls must not run in an orchestrator',
-    axios: 'an activity — network calls must not run in an orchestrator'
+/** Global reads that differ on every replay, with the replacement to suggest. */
+const FORBIDDEN_CALLS = {
+  'Date.now': 'now(context).getTime()',
+  'Math.random': 'context.df.newGuid(...) or an activity',
+  'crypto.randomUUID': 'context.df.newGuid(...)',
+  fetch: 'an activity — network calls must not run in an orchestrator',
+  axios: 'an activity — network calls must not run in an orchestrator'
 };
 /**
  * Flags non-deterministic operations inside an orchestration body.
@@ -91,92 +95,88 @@
  * only half of what a reader needs.
  *
  * Heuristic by design — see {@link isOrchestrationRegistration}.
- */ var noNondeterministicOrchestrator = {
-    meta: {
-        type: 'problem',
-        docs: {
-            description: 'Disallow non-deterministic operations inside an orchestration.'
-        },
-        schema: [],
-        messages: {
-            forbidden: '{{what}} is non-deterministic on replay. Use {{fix}} instead.',
-            newDate: 'new Date() is non-deterministic on replay. Use now(context) instead.',
-            processEnv: 'process.env is read at replay time and may differ between deploys. ' + 'Read it in an activity, or pass it as orchestration input.'
-        }
+ */
+const noNondeterministicOrchestrator = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Disallow non-deterministic operations inside an orchestration.'
     },
-    create: function create(context) {
-        var depth = 0;
-        var enter = function enter(node) {
-            if (isOrchestrationRegistration(node)) {
-                depth += 1;
-            }
-        };
-        var exit = function exit(node) {
-            if (isOrchestrationRegistration(node)) {
-                depth -= 1;
-            }
-        };
-        return {
-            CallExpression: function CallExpression(node) {
-                var _calleeName;
-                enter(node);
-                if (depth === 0) {
-                    return;
-                }
-                var callee = node.callee;
-                var object = callee === null || callee === void 0 ? void 0 : callee.object;
-                var full = (object === null || object === void 0 ? void 0 : object.type) === 'Identifier' ? "".concat(String(object.name), ".").concat(String(calleeName(callee))) : (_calleeName = calleeName(callee)) !== null && _calleeName !== void 0 ? _calleeName : '';
-                var fix = FORBIDDEN_CALLS[full];
-                if (fix !== undefined) {
-                    context.report({
-                        node: node,
-                        messageId: 'forbidden',
-                        data: {
-                            what: full,
-                            fix: fix
-                        }
-                    });
-                }
-            },
-            'CallExpression:exit': exit,
-            NewExpression: function NewExpression(node) {
-                var _ref;
-                if (depth === 0) {
-                    return;
-                }
-                var callee = node.callee;
-                var args = node.arguments;
-                // `new Date(someInstant)` is fine and common — only the argument-less
-                // form reads the wall clock.
-                if ((callee === null || callee === void 0 ? void 0 : callee.type) === 'Identifier' && callee.name === 'Date' && ((_ref = args === null || args === void 0 ? void 0 : args.length) !== null && _ref !== void 0 ? _ref : 0) === 0) {
-                    context.report({
-                        node: node,
-                        messageId: 'newDate'
-                    });
-                }
-            },
-            MemberExpression: function MemberExpression(node) {
-                if (depth === 0) {
-                    return;
-                }
-                var object = node.object;
-                var property = node.property;
-                if ((object === null || object === void 0 ? void 0 : object.type) === 'Identifier' && object.name === 'process' && (property === null || property === void 0 ? void 0 : property.type) === 'Identifier' && property.name === 'env') {
-                    context.report({
-                        node: node,
-                        messageId: 'processEnv'
-                    });
-                }
-            }
-        };
+    schema: [],
+    messages: {
+      forbidden: '{{what}} is non-deterministic on replay. Use {{fix}} instead.',
+      newDate: 'new Date() is non-deterministic on replay. Use now(context) instead.',
+      processEnv: 'process.env is read at replay time and may differ between deploys. ' + 'Read it in an activity, or pass it as orchestration input.'
     }
+  },
+  create(context) {
+    let depth = 0;
+    const enter = node => {
+      if (isOrchestrationRegistration(node)) {
+        depth += 1;
+      }
+    };
+    const exit = node => {
+      if (isOrchestrationRegistration(node)) {
+        depth -= 1;
+      }
+    };
+    return {
+      CallExpression: node => {
+        enter(node);
+        if (depth === 0) {
+          return;
+        }
+        const callee = node.callee;
+        const object = callee?.object;
+        const full = object?.type === 'Identifier' ? `${String(object.name)}.${String(calleeName(callee))}` : calleeName(callee) ?? '';
+        const fix = FORBIDDEN_CALLS[full];
+        if (fix !== undefined) {
+          context.report({
+            node,
+            messageId: 'forbidden',
+            data: {
+              what: full,
+              fix
+            }
+          });
+        }
+      },
+      'CallExpression:exit': exit,
+      NewExpression: node => {
+        if (depth === 0) {
+          return;
+        }
+        const callee = node.callee;
+        const args = node.arguments;
+        // `new Date(someInstant)` is fine and common — only the argument-less
+        // form reads the wall clock.
+        if (callee?.type === 'Identifier' && callee.name === 'Date' && (args?.length ?? 0) === 0) {
+          context.report({
+            node,
+            messageId: 'newDate'
+          });
+        }
+      },
+      MemberExpression: node => {
+        if (depth === 0) {
+          return;
+        }
+        const object = node.object;
+        const property = node.property;
+        if (object?.type === 'Identifier' && object.name === 'process' && property?.type === 'Identifier' && property.name === 'env') {
+          context.report({
+            node,
+            messageId: 'processEnv'
+          });
+        }
+      }
+    };
+  }
 };
 
-/** SDK handler aliases that erase a handler's real signature. */ var ERASING_TYPES = new Set([
-    'ActivityHandler',
-    'OrchestrationHandler',
-    'FunctionHandler'
-]);
+/** SDK handler aliases that erase a handler's real signature. */
+const ERASING_TYPES = new Set(['ActivityHandler', 'OrchestrationHandler', 'FunctionHandler']);
 /**
  * The annotation's type name, if it is a bare type reference.
  *
@@ -184,13 +184,14 @@
  * @returns The referenced type's name, or `undefined` when it is not a bare reference.
  * @throws Never - pure inspection.
  * @typeParam None - this function has no generic type parameters.
- */ function referencedName(annotation) {
-    var typeAnnotation = annotation === null || annotation === void 0 ? void 0 : annotation.typeAnnotation;
-    if ((typeAnnotation === null || typeAnnotation === void 0 ? void 0 : typeAnnotation.type) !== 'TSTypeReference') {
-        return undefined;
-    }
-    var typeName = typeAnnotation.typeName;
-    return (typeName === null || typeName === void 0 ? void 0 : typeName.type) === 'Identifier' ? typeName.name : undefined;
+ */
+function referencedName(annotation) {
+  const typeAnnotation = annotation?.typeAnnotation;
+  if (typeAnnotation?.type !== 'TSTypeReference') {
+    return undefined;
+  }
+  const typeName = typeAnnotation.typeName;
+  return typeName?.type === 'Identifier' ? typeName.name : undefined;
 }
 /**
  * Flags annotations that collapse a typed handler back to `any`.
@@ -215,51 +216,44 @@
  * const withLogging = <I, O>(h: (i: I, c: InvocationContext) => O) =>
  *   (i: I, c: InvocationContext): O => { c.log('...'); return h(i, c) }
  * ```
- */ var noUntypedActivityHandler = {
-    meta: {
-        type: 'problem',
-        docs: {
-            description: 'Disallow handler annotations that erase inferred types.'
-        },
-        schema: [],
-        messages: {
-            erased: "Annotating with '{{name}}' erases the handler's real signature — it is an alias for " + '(triggerInput: any, context) => any, so the activity becomes `any` in and `any` out. ' + 'Drop the annotation and let defineActivity infer it.',
-            erasedWrapper: "A wrapper typed '{{name}}' collapses every handler it wraps to `any`. " + 'Make it generic: <I, O>(h: (i: I, c: InvocationContext) => O) => (i: I, c: InvocationContext): O.'
-        }
+ */
+const noUntypedActivityHandler = {
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Disallow handler annotations that erase inferred types.'
     },
-    create: function create(context) {
-        return {
-            // const handler: ActivityHandler = ...
-            Identifier: function Identifier(node) {
-                var _node_parent_type, _node_parent, _node_parent_params, _node_parent1;
-                var name = referencedName(node.typeAnnotation);
-                if (name === undefined || !ERASING_TYPES.has(name)) {
-                    return;
-                }
-                // A function PARAMETER annotated this way is the middleware form, which
-                // is worse: it erases every handler passed through it, not just one.
-                var isParameter = ((_node_parent = node.parent) === null || _node_parent === void 0 ? void 0 : (_node_parent_type = _node_parent.type) === null || _node_parent_type === void 0 ? void 0 : _node_parent_type.startsWith('TS')) === false && ((_node_parent1 = node.parent) === null || _node_parent1 === void 0 ? void 0 : (_node_parent_params = _node_parent1.params) === null || _node_parent_params === void 0 ? void 0 : _node_parent_params.includes(node)) === true;
-                context.report({
-                    node: node,
-                    messageId: isParameter ? 'erasedWrapper' : 'erased',
-                    data: {
-                        name: name
-                    }
-                });
-            }
-        };
+    schema: [],
+    messages: {
+      erased: "Annotating with '{{name}}' erases the handler's real signature — it is an alias for " + '(triggerInput: any, context) => any, so the activity becomes `any` in and `any` out. ' + 'Drop the annotation and let defineActivity infer it.',
+      erasedWrapper: "A wrapper typed '{{name}}' collapses every handler it wraps to `any`. " + 'Make it generic: <I, O>(h: (i: I, c: InvocationContext) => O) => (i: I, c: InvocationContext): O.'
     }
+  },
+  create(context) {
+    return {
+      // const handler: ActivityHandler = ...
+      Identifier: node => {
+        const name = referencedName(node.typeAnnotation);
+        if (name === undefined || !ERASING_TYPES.has(name)) {
+          return;
+        }
+        // A function PARAMETER annotated this way is the middleware form, which
+        // is worse: it erases every handler passed through it, not just one.
+        const isParameter = node.parent?.type?.startsWith('TS') === false && node.parent?.params?.includes(node) === true;
+        context.report({
+          node,
+          messageId: isParameter ? 'erasedWrapper' : 'erased',
+          data: {
+            name
+          }
+        });
+      }
+    };
+  }
 };
 
-/** The wrapper calls that must be delegated to, not yielded. */ var DELEGATED = new Set([
-    'callActivity',
-    'callSubOrchestration',
-    'all',
-    'any',
-    'waitForEvent',
-    'sleepFor',
-    'sleepUntil'
-]);
+/** The wrapper calls that must be delegated to, not yielded. */
+const DELEGATED = new Set(['callActivity', 'callSubOrchestration', 'all', 'any', 'waitForEvent', 'sleepFor', 'sleepUntil']);
 /**
  * Flags `yield callActivity(...)` where `yield *` is meant.
  *
@@ -278,48 +272,49 @@
  * So this rule earns its place only by reporting a clearer message than
  * `TS2345` does. It is in `recommended` for that reason, not because anything
  * depends on it.
- */ var requireYieldStar = {
-    meta: {
-        type: 'suggestion',
-        docs: {
-            description: 'Require `yield *` when calling a delegating helper.'
-        },
-        schema: [],
-        messages: {
-            useYieldStar: "Use 'yield *' rather than 'yield' with {{name}}(). Delegation is what carries the " + 'result type; a bare yield does not typecheck, but the compiler error is obscure.'
-        }
+ */
+const requireYieldStar = {
+  meta: {
+    type: 'suggestion',
+    docs: {
+      description: 'Require `yield *` when calling a delegating helper.'
     },
-    create: function create(context) {
-        return {
-            YieldExpression: function YieldExpression(node) {
-                if (node.delegate === true) {
-                    return;
-                }
-                var argument = node.argument;
-                if ((argument === null || argument === void 0 ? void 0 : argument.type) !== 'CallExpression') {
-                    return;
-                }
-                // A BARE IDENTIFIER only. `c.df.callActivity(...)` is the raw SDK call,
-                // which is correct code and must not be flagged — matching the trailing
-                // property of a member expression would flag it, which a negative
-                // fixture caught.
-                var callee = argument.callee;
-                if ((callee === null || callee === void 0 ? void 0 : callee.type) !== 'Identifier') {
-                    return;
-                }
-                var name = callee.name;
-                if (DELEGATED.has(name)) {
-                    context.report({
-                        node: node,
-                        messageId: 'useYieldStar',
-                        data: {
-                            name: name
-                        }
-                    });
-                }
-            }
-        };
+    schema: [],
+    messages: {
+      useYieldStar: "Use 'yield *' rather than 'yield' with {{name}}(). Delegation is what carries the " + 'result type; a bare yield does not typecheck, but the compiler error is obscure.'
     }
+  },
+  create(context) {
+    return {
+      YieldExpression: node => {
+        if (node.delegate === true) {
+          return;
+        }
+        const argument = node.argument;
+        if (argument?.type !== 'CallExpression') {
+          return;
+        }
+        // A BARE IDENTIFIER only. `c.df.callActivity(...)` is the raw SDK call,
+        // which is correct code and must not be flagged — matching the trailing
+        // property of a member expression would flag it, which a negative
+        // fixture caught.
+        const callee = argument.callee;
+        if (callee?.type !== 'Identifier') {
+          return;
+        }
+        const name = callee.name;
+        if (DELEGATED.has(name)) {
+          context.report({
+            node,
+            messageId: 'useYieldStar',
+            data: {
+              name
+            }
+          });
+        }
+      }
+    };
+  }
 };
 
 /**
@@ -328,10 +323,11 @@
  * @remarks
  * The keys are the names a config writes after the `az-durable/` prefix, so
  * renaming one is a breaking change for any consumer's config.
- */ var rules = {
-    'no-nondeterministic-orchestrator': noNondeterministicOrchestrator,
-    'no-untyped-activity-handler': noUntypedActivityHandler,
-    'require-yield-star': requireYieldStar
+ */
+const rules = {
+  'no-nondeterministic-orchestrator': noNondeterministicOrchestrator,
+  'no-untyped-activity-handler': noUntypedActivityHandler,
+  'require-yield-star': requireYieldStar
 };
 /**
  * The plugin object, for a flat config's `plugins` map.
@@ -339,8 +335,9 @@
  * @remarks
  * Shipped from a separate entry point so lint rules are never a runtime import
  * of the wrapper itself.
- */ var plugin = {
-    rules: rules
+ */
+const plugin = {
+  rules
 };
 /**
  * The recommended rule set.
@@ -351,15 +348,16 @@
  * failures nothing else does — silent replay corruption, and a type collapse
  * TypeScript accepts as legal. `require-yield-star` is a warning because the
  * compiler already rejects the code it flags; it only improves the message.
- */ var recommended = {
-    plugins: {
-        'az-durable': plugin
-    },
-    rules: {
-        'az-durable/no-nondeterministic-orchestrator': 'error',
-        'az-durable/no-untyped-activity-handler': 'error',
-        'az-durable/require-yield-star': 'warn'
-    }
+ */
+const recommended = {
+  plugins: {
+    'az-durable': plugin
+  },
+  rules: {
+    'az-durable/no-nondeterministic-orchestrator': 'error',
+    'az-durable/no-untyped-activity-handler': 'error',
+    'az-durable/require-yield-star': 'warn'
+  }
 };
 
 exports.noNondeterministicOrchestrator = noNondeterministicOrchestrator;
@@ -368,3 +366,4 @@ exports.plugin = plugin;
 exports.recommended = recommended;
 exports.requireYieldStar = requireYieldStar;
 exports.rules = rules;
+//# sourceMappingURL=eslint-plugin.cjs.js.map