@sarj/eslint-plugin 19.4.0 → 19.7.0

package/dist/catalog.js

@@ -2,7 +2,7 @@ import {
   publicDocumentation,
   ruleSpec,
   src_default
-} from "./chunk-6RX5RU75.js";
+} from "./chunk-OX5DMK42.js";
 
 // src/docs/directive-disable-enable-pair.ts
 var DIRECTIVE_DISABLE_ENABLE_PAIR_DOCUMENTATION = {
@@ -4120,22 +4120,175 @@ var PREFER_AWAIT_IN_ASYNC_RETURN_DOCUMENTATION = {
 var PREFER_CODED_ERROR_RESULT_DOCUMENTATION = {
   summary: "Return a stable error code and typed context instead of only a rendered failure message.",
   defaultLevel: "warning",
-  rationale: "A message-only failure hides the identity needed for repair, conflict handling, localization and exhaustive dispatch. Preserve a finite error code and the minimum handling context; let the consumer format its message.",
+  rationale: "A message-only or opaque failure hides the identity needed for repair, conflict handling, localization and exhaustive dispatch. Preserve a finite error code and the minimum handling context; let the consumer format its message. Reuse the native no-explicit-any rule, compiler narrowing and type-aware exhaustiveness checks.",
   remediation: "Use the existing Result or outcome union with a finite error code and variant-specific context. Preserve public behavior through a presentation adapter, and handle or propagate each relevant variant without parsing message text.",
   category: "correctness",
   autofix: "none",
   limitations: [
-    "Checks source-resolved local unions with required literal ok/success true and false branches, and named neverthrow Result/ResultAsync imports. Broad string error/message/reason payloads and open string code fields are review candidates. String literal error-code unions remain valid.",
-    "Local aliases, unmerged interfaces without inheritance and generic substitutions are bounded to 16 resolution levels, 64 union members and 256 traversal steps. Imported local declarations, namespace imports, conditional/intersection/mapped types and unresolved contracts abstain. Optional discriminants, independent success/error payloads and partial-data failures are excluded.",
+    "Accepting an inferred producer does not prove a finite inferred contract: object payloads or errAsync can widen a code to string. Annotate the existing Result/ResultAsync return when necessary. Native assignability, narrowing and exhaustive-handler checks remain authoritative.",
+    "Checks source-resolved local unions with required literal ok/success booleans, or status/state/kind/type/tag values ok/success versus err/error/failure/failed (case insensitive). Conflicting outcome tags abstain. A shared outcome tag does not also count as a domain error code. Named and namespace neverthrow Result/ResultAsync imports establish their generic error position.",
+    "Presence-selected unions require an error property on every failure branch and the same single required non-control value property on every success branch. Optional error/value fields, multiple value keys, loading states and error branches carrying the success value abstain. Required errorMessage and Error-suffixed failure channels are also reviewed; multiple custom channels abstain and a primary error payload retains precedence.",
+    "Broad string and unknown error channels, unshadowed native Error-family types and open string/unknown code fields receive review. Explicit Any stays with the existing no-explicit-any rule. Finite literal codes need no invented payload. Imported domain errors, validation-library errors, generic error parameters, ordinary formatters, legitimate absence and redacted failure states remain valid.",
+    "Local aliases, unmerged interfaces without inheritance and generic substitutions are bounded to 16 resolution levels, 64 union members and 256 traversal steps. Imported local declarations, conditional/intersection/mapped types and unresolved contracts abstain. Optional discriminants, independent success/error payloads and partial-data failures are excluded.",
     "Required code/kind/type/tag fields of unresolved or imported type defer to the type checker; a finite code does not prove that its cases or payloads are useful. Generic error parameters are retained rather than guessed. Ordinary string formatters, absence, validation-library errors and redacted failure states without a string payload are not flagged.",
-    "A directly returned plain object with literal ok/success false and a literal string code/kind/type/tag also receives review when its message/reason is an interpolated template or proven string concatenation. Nested plain error objects and expression-bodied arrows are supported. Spreads, accessors, computed keys, helper formatting calls and nonliteral codes abstain. This proves formatting ownership, not which context the domain must expose; presentation adapters remain a reviewed exception.",
-    "This warning requests domain review: a final presentation boundary may intentionally accept every failure and render one message. No fix invents error codes, fields, handlers or changes serialization, exception unwinding, transactions, retries or logging."
+    "A directly returned failure object, neverthrow err/errAsync call or new Err with an interpolated template or proven string concatenation receives review. Without a code, the diagnostic asks for a domain contract; with a literal code, it asks for consumer formatting. Literal-only compositions, including earlier stable local constants, remain valid codes. Nested plain error objects, expression-bodied arrows and one direct await are supported. Import identity and value-space shadowing are checked; arbitrary aliases, spreads, accessors, computed keys, helper formatting calls and nonliteral codes abstain. This proves formatting ownership, not which context the domain must expose; presentation adapters remain a reviewed exception.",
+    "This warning requests domain review: an infrastructure adapter may intentionally retain unknown/Error, and a final presentation boundary may accept every failure and render one message. Neither proves a recoverable domain rejection. No fix invents error codes, fields, handlers or changes serialization, exception unwinding, transactions, retries or logging."
   ],
   references: [
     "https://www.typescriptlang.org/docs/handbook/2/narrowing.html#discriminated-unions",
     "https://github.com/supermacro/neverthrow"
   ],
   examples: [
+    {
+      id: "presence-selected-message",
+      scenarioId: "presence-selected-result",
+      title: "An error-presence check still needs a useful error contract",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { value: T } | { error: string; field?: 'name' };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "presence-selected-domain-error",
+      scenarioId: "presence-selected-result",
+      title: "Retain the existing presence check with a finite error union",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "type SaveError = { code: 'duplicate'; name: string } | { code: 'denied' }; export type Outcome<T> = { value: T } | { error: SaveError };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "string-outcome-tag",
+      scenarioId: "string-outcome",
+      title: "An outcome tag distinguishes success from failure, not the failure's cause",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { kind: 'ok'; value: T } | { kind: 'error'; message: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "string-outcome-domain-code",
+      scenarioId: "string-outcome",
+      title: "Preserve the established string tag and add the actionable error",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { kind: 'ok'; value: T } | { kind: 'error'; error: { code: 'duplicate'; name: string } };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "opaque-error-channel",
+      scenarioId: "opaque-error",
+      title: "Unknown preserves the need to narrow but hides the domain's cases",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "type Result<T, E> = { ok: true; value: T } | { ok: false; error: E }; export type Saved = Result<number, unknown>;"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "specific-error-channel",
+      scenarioId: "opaque-error",
+      title: "Expose the domain's rejection case without inventing context",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "type Result<T, E> = { ok: true; value: T } | { ok: false; error: E }; export type Saved = Result<number, { code: 'unavailable' }>;"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "named-error-carrier",
+      scenarioId: "named-error-channel",
+      title: "A failure-only Error-suffixed field can still erase its cases",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { ok: true; value: T } | { ok: false; fallbackError: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "typed-named-error-carrier",
+      scenarioId: "named-error-channel",
+      title: "Preserve the existing field name with the required context",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { ok: true; value: T } | { ok: false; fallbackError: { code: 'unsupported'; requestedKind: string } };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "library-error-constructor",
+      scenarioId: "library-error-presentation",
+      title: "A proven Result constructor does not make formatted text actionable",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "import { err } from 'neverthrow'; export function reject(name: string) { return err(`Missing ${name}`); }"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "library-domain-constructor",
+      scenarioId: "library-error-presentation",
+      title: "Reuse the established Result library with a typed domain error",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "import { err, type Result } from 'neverthrow'; type Missing = { code: 'missing'; name: string }; export function reject(name: string): Result<never, Missing> { return err({ code: 'missing', name }); }"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
     {
       id: "message-only-result",
       title: "A failure erases identity into a message",
@@ -4194,6 +4347,66 @@ var PREFER_CODED_ERROR_RESULT_DOCUMENTATION = {
       expectedCount: 0,
       public: true
     },
+    {
+      id: "exclusive-message-outcome",
+      scenarioId: "partial-data",
+      title: "An exclusive failure still needs an actionable identity",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { ok: true; data: T } | { ok: false; error: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "independent-partial-outcomes",
+      scenarioId: "partial-data",
+      title: "Partial data and its independent error remain separate outcomes",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Partial<T> = { ok: true; data: T; error?: string } | { ok: false; data: T; error: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "exclusive-error-state",
+      scenarioId: "absence-state",
+      title: "Conventional success and failure tags establish one Result",
+      outcome: "match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Outcome<T> = { status: 'success'; value: T } | { status: 'failure'; reason: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "legitimate-unconfigured-state",
+      scenarioId: "absence-state",
+      title: "A loading report can preserve an unconfigured absence state",
+      outcome: "no-match",
+      files: [
+        {
+          path: "app/result.ts",
+          source: "export type Source<T> = { status: 'loaded'; value: T } | { status: 'failed' | 'unconfigured'; reason: string };"
+        }
+      ],
+      focusPath: "app/result.ts",
+      expectedCount: 0,
+      public: true
+    },
     {
       id: "producer-renders-context",
       scenarioId: "consumer-formatting",

package/dist/{chunk-6RX5RU75.js → chunk-OX5DMK42.js}

@@ -21504,8 +21504,30 @@ var NAME91 = "prefer-coded-error-result";
 var MAX_DEPTH = 16;
 var MAX_VARIANTS = 64;
 var MAX_TYPE_STEPS = 256;
-var STATUS_NAMES = ["ok", "success"];
+var BOOLEAN_STATUS_NAMES = ["ok", "success"];
+var STRING_STATUS_NAMES = ["status", "state", "kind", "type", "tag"];
+var STATUS_NAMES = [
+  ...BOOLEAN_STATUS_NAMES,
+  ...STRING_STATUS_NAMES
+];
 var CODE_NAMES = ["code", "kind", "type", "tag"];
+var SUCCESS_TAGS = /* @__PURE__ */ new Set(["ok", "success"]);
+var FAILURE_TAGS = /* @__PURE__ */ new Set([
+  "err",
+  "error",
+  "failure",
+  "failed"
+]);
+var NATIVE_ERRORS = /* @__PURE__ */ new Set([
+  "Error",
+  "EvalError",
+  "RangeError",
+  "ReferenceError",
+  "SyntaxError",
+  "TypeError",
+  "URIError",
+  "AggregateError"
+]);
 function view(node) {
   return { node, origin: node, bindings: /* @__PURE__ */ new Map() };
 }
@@ -21515,7 +21537,7 @@ function resolve2(value, source, depth = 0) {
   if (node.type !== "TSTypeReference") return value;
   if (node.typeName.type !== "Identifier") return null;
   const variable = typeBinding(source, node.typeName);
-  if (variable === null) return null;
+  if (variable === null || variable.defs.length === 0) return value;
   const bound = value.bindings.get(variable);
   if (bound !== void 0) return resolve2(bound, source, depth + 1);
   const declaration = variable.defs.length === 1 ? variable.defs[0]?.node : void 0;
@@ -21577,30 +21599,40 @@ function propertyView(property, owner) {
   const node = property.typeAnnotation?.typeAnnotation;
   return node === void 0 ? null : { node, origin: node, bindings: owner.bindings };
 }
-function booleanTag(property, owner, source) {
+function outcomeTag(property, owner, source, name) {
   if (property === void 0 || property.optional) return null;
   const type = propertyView(property, owner);
   const resolved = type === null ? null : resolve2(type, source);
   const node = resolved?.node;
-  return node?.type === "TSLiteralType" && node.literal.type === "Literal" && typeof node.literal.value === "boolean" ? node.literal.value : null;
+  return node?.type === "TSLiteralType" && node.literal.type === "Literal" ? outcomeValue(name, node.literal.value) : null;
 }
-function hasCode(fields2, owner, source) {
+function outcomeValue(name, value) {
+  if (BOOLEAN_STATUS_NAMES.some((candidate2) => candidate2 === name))
+    return typeof value === "boolean" ? value : null;
+  if (!STRING_STATUS_NAMES.some((candidate2) => candidate2 === name) || typeof value !== "string")
+    return null;
+  const tag = value.toLowerCase();
+  return SUCCESS_TAGS.has(tag) ? true : FAILURE_TAGS.has(tag) ? false : null;
+}
+function hasCode(fields2, owner, source, ignoredCodes = []) {
   return CODE_NAMES.some((name) => {
+    if (ignoredCodes.includes(name)) return false;
     const property = fields2.get(name);
     if (property === void 0 || property.optional) return false;
     const type = propertyView(property, owner);
-    return type !== null && openString(type, source) === null;
+    return type !== null && openScalar(type, source) === null;
   });
 }
-function openString(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEPS }) {
+function openScalar(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEPS }) {
   if (depth >= MAX_DEPTH || --budget.remaining < 0) return null;
   const resolved = resolve2(value, source);
   if (resolved === null) return null;
-  if (resolved.node.type === "TSStringKeyword") return resolved.origin;
+  if (resolved.node.type === "TSStringKeyword" || resolved.node.type === "TSUnknownKeyword")
+    return resolved.origin;
   if (resolved.node.type !== "TSUnionType" || resolved.node.types.length > MAX_VARIANTS)
     return null;
   for (const member3 of resolved.node.types) {
-    const broad = openString(
+    const broad = openScalar(
       { ...resolved, node: member3 },
       source,
       depth + 1,
@@ -21614,7 +21646,9 @@ function weakError(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEP
   if (depth >= MAX_DEPTH || --budget.remaining < 0) return null;
   const resolved = resolve2(value, source);
   if (resolved === null) return null;
-  if (resolved.node.type === "TSStringKeyword") return resolved.origin;
+  if (resolved.node.type === "TSStringKeyword" || resolved.node.type === "TSUnknownKeyword")
+    return resolved.origin;
+  if (nativeErrorType(resolved.node, source)) return resolved.origin;
   if (resolved.node.type === "TSUnionType") {
     if (resolved.node.types.length > MAX_VARIANTS) return null;
     for (const member3 of resolved.node.types) {
@@ -21632,22 +21666,44 @@ function weakError(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEP
   if (fields2 === null || hasCode(fields2, resolved, source)) return null;
   return weakObject(fields2, resolved, source, depth, budget);
 }
-function weakObject(fields2, owner, source, depth, budget) {
-  const code = openStringField(fields2, owner, source, CODE_NAMES, budget);
+function nativeErrorType(node, source) {
+  if (node.type !== "TSTypeReference" || node.typeName.type !== "Identifier" || !NATIVE_ERRORS.has(node.typeName.name) || (node.typeArguments?.params.length ?? 0) !== 0)
+    return false;
+  const binding3 = typeBinding(source, node.typeName);
+  return binding3 === null || binding3.defs.length === 0;
+}
+function weakObject(fields2, owner, source, depth, budget, ignoredCodes = []) {
+  const code = openStringField(
+    fields2,
+    owner,
+    source,
+    CODE_NAMES.filter((name) => !ignoredCodes.includes(name)),
+    budget
+  );
   if (code !== null) return code;
   const error = fields2.get("error");
   if (error !== void 0) {
     const inner = propertyView(error, owner);
     return inner === null || error.optional ? null : weakError(inner, source, depth + 1, budget);
   }
+  const carriers = [...fields2].filter(([name]) => errorCarrier(name));
+  if (carriers.length > 1) return null;
+  const carrier = carriers[0]?.[1];
+  if (carrier !== void 0) {
+    const inner = propertyView(carrier, owner);
+    return inner === null || carrier.optional ? null : weakError(inner, source, depth + 1, budget);
+  }
   return openStringField(fields2, owner, source, ["message", "reason"], budget);
 }
+function errorCarrier(name) {
+  return name === "error" || name === "errorMessage" || name.endsWith("Error");
+}
 function openStringField(fields2, owner, source, names, budget) {
   for (const name of names) {
     const property = fields2.get(name);
     if (property === void 0 || property.optional) continue;
     const inner = propertyView(property, owner);
-    const broad = inner === null ? null : openString(inner, source, 0, budget);
+    const broad = inner === null ? null : openScalar(inner, source, 0, budget);
     if (broad !== null) return broad;
   }
   return null;
@@ -21660,26 +21716,70 @@ function failureMessages(value, source) {
     fields: properties(member3)
   }));
   if (objects.some((object) => object.fields === null)) return [];
+  let selected = null;
+  const discriminators = [];
   for (const status of STATUS_NAMES) {
-    const tags = objects.map(
-      (object) => booleanTag(object.fields?.get(status), object.member, source)
+    const tags2 = objects.map(
+      (object) => outcomeTag(object.fields?.get(status), object.member, source, status)
     );
-    if (tags.some((tag) => tag === null) || !tags.includes(true) || !tags.includes(false))
-      continue;
-    if (objects.some(
-      (object, index) => tags[index] === true && ["error", "errors"].some((name) => object.fields?.has(name))
-    ))
+    if (tags2.some((tag) => tag === null) || !tags2.includes(true) || !tags2.includes(false))
       continue;
-    return objects.flatMap((object, index) => {
-      if (tags[index] !== false || object.fields === null || hasCode(object.fields, object.member, source))
-        return [];
-      if (["data", "value", "result"].some((name) => object.fields?.has(name)))
-        return [];
-      const error = weakError(object.member, source);
-      return error === null ? [] : [error];
-    });
+    if (selected !== null && tags2.some((tag, index) => tag !== selected?.[index]))
+      return [];
+    discriminators.push(status);
+    if (selected === null) {
+      selected = tags2;
+    }
   }
-  return [];
+  if (selected === null) return presenceFailures(objects, source);
+  const tags = selected;
+  if (objects.some(
+    (object, index) => tags[index] === true && [...object.fields?.keys() ?? []].some(
+      (name) => name === "errors" || errorCarrier(name)
+    )
+  ))
+    return [];
+  return objects.flatMap((object, index) => {
+    if (tags[index] !== false || object.fields === null || hasCode(object.fields, object.member, source, discriminators) || ["data", "value", "result"].some((name) => object.fields?.has(name)))
+      return [];
+    const error = weakObject(
+      object.fields,
+      object.member,
+      source,
+      0,
+      { remaining: MAX_TYPE_STEPS },
+      discriminators
+    );
+    return error === null ? [] : [error];
+  });
+}
+function presenceFailures(objects, source) {
+  const failed2 = objects.filter((object) => object.fields?.has("error"));
+  const successful = objects.filter((object) => !object.fields?.has("error"));
+  if (failed2.length === 0 || successful.length === 0) return [];
+  let successKey = null;
+  for (const object of successful) {
+    if (object.fields === null || object.fields.size !== 1) return [];
+    const property = [...object.fields][0];
+    if (property === void 0 || property[1].optional || errorCarrier(property[0]) || property[0] === "errors" || STATUS_NAMES.includes(property[0]) || CODE_NAMES.some((name) => name === property[0]))
+      return [];
+    if (successKey !== null && successKey !== property[0]) return [];
+    successKey = property[0];
+  }
+  if (successKey === null) return [];
+  const key = successKey;
+  if (failed2.some(
+    (object) => object.fields?.get("error")?.optional || object.fields?.has(key)
+  ))
+    return [];
+  return failed2.flatMap((object) => {
+    if (object.fields === null || hasCode(object.fields, object.member, source))
+      return [];
+    const error = object.fields.get("error");
+    const inner = error === void 0 ? null : propertyView(error, object.member);
+    const weak = inner === null ? null : weakError(inner, source);
+    return weak === null ? [] : [weak];
+  });
 }
 function variants(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEPS }) {
   if (--budget.remaining < 0) return null;
@@ -21701,18 +21801,37 @@ function variants(value, source, depth = 0, budget = { remaining: MAX_TYPE_STEPS
   return result;
 }
 function libraryError(reference, source) {
-  if (reference.typeName.type !== "Identifier" || reference.typeArguments?.params.length !== 2)
+  if (reference.typeArguments?.params.length !== 2) return null;
+  if (!["Result", "ResultAsync"].includes(
+    libraryName(reference.typeName, source, true) ?? ""
+  ))
     return null;
-  const variable = typeBinding(source, reference.typeName);
+  const error = reference.typeArguments.params[1];
+  return error === void 0 ? null : weakError(view(error), source);
+}
+function libraryName(node, source, typeSpace) {
+  const reference = libraryReference(node);
+  if (reference === null) return null;
+  const { root, member: member3 } = reference;
+  const variable = typeSpace ? typeBinding(source, root) : findVariable(source.getScope(root), root.name);
   const definition = variable?.defs.length === 1 ? variable.defs[0]?.node : void 0;
-  if (definition?.type !== "ImportSpecifier") return null;
+  if (definition?.type !== "ImportSpecifier" && definition?.type !== "ImportNamespaceSpecifier")
+    return null;
+  if (variable === null || !stableValueBinding(variable)) return null;
   const declaration = parentOf(definition);
   if (declaration?.type !== "ImportDeclaration" || declaration.source.value !== "neverthrow")
     return null;
-  if (!["Result", "ResultAsync"].includes(importSpecifierName(definition) ?? ""))
+  if (!typeSpace && (declaration.importKind === "type" || definition.type === "ImportSpecifier" && definition.importKind === "type"))
     return null;
-  const error = reference.typeArguments.params[1];
-  return error === void 0 ? null : weakError(view(error), source);
+  return member3 === null && definition.type === "ImportSpecifier" ? importSpecifierName(definition) : member3 !== null && definition.type === "ImportNamespaceSpecifier" ? member3 : null;
+}
+function libraryReference(node) {
+  if (node.type === "Identifier") return { root: node, member: null };
+  if (node.type === "TSQualifiedName" && node.left.type === "Identifier" && node.right.type === "Identifier")
+    return { root: node.left, member: node.right.name };
+  if (node.type === "MemberExpression" && !node.computed && !node.optional && node.object.type === "Identifier" && node.property.type === "Identifier")
+    return { root: node.object, member: node.property.name };
+  return null;
 }
 function weakAnnotation(node, source, depth = 0, budget = { remaining: MAX_TYPE_STEPS }) {
   if (depth >= MAX_DEPTH || --budget.remaining < 0) return false;
@@ -21724,23 +21843,65 @@ function weakAnnotation(node, source, depth = 0, budget = { remaining: MAX_TYPE_
   const argument = node.typeArguments.params[0];
   return argument !== void 0 && weakAnnotation(argument, source, depth + 1, budget);
 }
-function renderedFailure(node) {
-  const outer = objectFields(unwrapExpression(node));
-  if (outer === null || !STATUS_NAMES.some((name) => {
+function renderedFailure(node, source) {
+  let value = unwrapExpression(node);
+  if (value.type === "AwaitExpression")
+    value = unwrapExpression(value.argument);
+  if (value.type === "CallExpression" || value.type === "NewExpression") {
+    if (value.type === "CallExpression" && value.optional || value.arguments.length !== 1)
+      return null;
+    const name = libraryName(value.callee, source, false);
+    if (value.type === "NewExpression" ? name !== "Err" : name !== "err" && name !== "errAsync")
+      return null;
+    const argument = value.arguments[0];
+    if (argument === void 0 || argument.type === "SpreadElement")
+      return null;
+    return renderedPayload(argument, source);
+  }
+  const outer = objectFields(value);
+  if (outer === null) return null;
+  const discriminators = STATUS_NAMES.filter((name) => {
     const status = outer.get(name);
-    return status?.type === "Literal" && status.value === false;
-  }))
+    return status?.type === "Literal" && outcomeValue(name, status.value) === false;
+  });
+  if (discriminators.length === 0) return null;
+  if (STATUS_NAMES.some((name) => {
+    const status = outer.get(name);
+    return status?.type === "Literal" && outcomeValue(name, status.value) === true;
+  }) || ["data", "value", "result"].some((name) => outer.has(name)))
     return null;
   const error = outer.get("error");
-  const fields2 = error === void 0 ? outer : objectFields(unwrapExpression(error));
-  if (fields2 === null || !CODE_NAMES.some((name) => {
+  return renderedPayload(
+    error ?? value,
+    source,
+    error === void 0 ? discriminators : []
+  );
+}
+function renderedPayload(node, source, ignoredCodes = []) {
+  if (renderedString(node, source))
+    return { node, messageId: "preferCodedErrorResult" };
+  const fields2 = objectFields(unwrapExpression(node));
+  if (fields2 === null) return null;
+  const codes = CODE_NAMES.filter(
+    (name) => !ignoredCodes.includes(name) && fields2.has(name)
+  );
+  const coded = codes.some((name) => {
     const code = fields2.get(name);
     return code?.type === "Literal" && typeof code.value === "string";
-  }))
-    return null;
-  for (const name of ["message", "reason"]) {
-    const value = fields2.get(name);
-    if (value !== void 0 && renderedString(value)) return value;
+  });
+  if (codes.length !== 0 && !coded) return null;
+  const primary = fields2.get("error");
+  const carriers = [...fields2].filter(([name]) => errorCarrier(name));
+  if (primary === void 0 && carriers.length > 1) return null;
+  const carrier = primary ?? carriers[0]?.[1];
+  if (!coded && carrier !== void 0)
+    return renderedString(carrier, source) ? { node: carrier, messageId: "preferCodedErrorResult" } : null;
+  for (const [name, value] of fields2) {
+    if ((name === "message" || name === "reason" || errorCarrier(name)) && renderedString(value, source))
+      return {
+        node: value,
+        messageId: coded ? "formatAtConsumer" : "preferCodedErrorResult"
+      };
   }
   return null;
 }
@@ -21756,14 +21917,36 @@ function objectFields(node) {
   }
   return fields2;
 }
-function renderedString(node, depth = 0) {
-  const value = unwrapExpression(node);
-  if (depth >= MAX_DEPTH) return false;
-  if (value.type === "TemplateLiteral") return value.expressions.length > 0;
-  if (value.type !== "BinaryExpression" || value.operator !== "+") return false;
-  return [value.left, value.right].some(
-    (part) => part.type === "Literal" && typeof part.value === "string" || renderedString(part, depth + 1)
-  );
+function renderedString(node, source) {
+  let remaining = MAX_TYPE_STEPS;
+  function literalComposition(input, depth) {
+    if (depth >= MAX_DEPTH || --remaining < 0) return null;
+    const value = unwrapExpression(input);
+    if (value.type === "Identifier") {
+      const initializer = stableConstInitializer(source, value);
+      return initializer === null ? false : literalComposition(initializer.value, depth + 1);
+    }
+    if (value.type === "Literal")
+      return value.value === null || ["string", "number", "boolean", "bigint"].includes(typeof value.value);
+    const children = value.type === "TemplateLiteral" ? value.expressions : value.type === "BinaryExpression" && value.operator === "+" ? [value.left, value.right] : null;
+    if (children === null) return false;
+    for (const child of children) {
+      const constant = literalComposition(child, depth + 1);
+      if (constant !== true) return constant;
+    }
+    return true;
+  }
+  function rendered(input, depth) {
+    if (depth >= MAX_DEPTH || --remaining < 0) return false;
+    const value = unwrapExpression(input);
+    if (value.type === "TemplateLiteral") return value.expressions.length > 0;
+    if (value.type !== "BinaryExpression" || value.operator !== "+")
+      return false;
+    return [value.left, value.right].some(
+      (part) => part.type === "Literal" && typeof part.value === "string" || rendered(part, depth + 1)
+    );
+  }
+  return literalComposition(node, 0) === false && rendered(node, 0);
 }
 var prefer_coded_error_result_default = defineRule({
   meta: {
@@ -21774,7 +21957,7 @@ var prefer_coded_error_result_default = defineRule({
     },
     schema: [],
     messages: {
-      preferCodedErrorResult: "This Result failure carries only a rendered string or an open string code. Return a finite error code with the typed context needed by the caller, and format the message at the consumer. Keep an intentional presentation boundary explicit.",
+      preferCodedErrorResult: "This Result failure hides its identity in a broad string, unknown, or generic Error channel. Expose expected failures as finite domain error codes with the typed context needed by the caller, and format the message at the consumer. Review intentional infrastructure and presentation boundaries explicitly.",
       formatAtConsumer: "This coded failure formats context into a message at the producer. Return the code and typed context, and let the consumer format the message. Preserve an intentional presentation boundary explicitly."
     }
   },
@@ -21794,13 +21977,13 @@ var prefer_coded_error_result_default = defineRule({
     }
     function checkProducer(owner, value) {
       if (owner === null) return;
-      const rendered = renderedFailure(value);
-      if (rendered === null || reported.has(rendered)) return;
+      const rendered = renderedFailure(value, context.sourceCode);
+      if (rendered === null || reported.has(rendered.node)) return;
       const contract = owner.returnType?.typeAnnotation;
       if (contract !== void 0 && weakAnnotation(contract, context.sourceCode))
         return;
-      reported.add(rendered);
-      context.report({ node: rendered, messageId: "formatAtConsumer" });
+      reported.add(rendered.node);
+      context.report(rendered);
     }
     return {
       before() {

package/dist/index.js

@@ -1,6 +1,6 @@
 import {
   src_default
-} from "./chunk-6RX5RU75.js";
+} from "./chunk-OX5DMK42.js";
 export {
   src_default as default
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "19.4.0",
+  "version": "19.7.0",
   "packageManager": "npm@12.2.1",
   "description": "Custom Oxlint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",