praxis-kit 6.1.0 → 6.1.1

package/dist/vue/index.js

@@ -4,7 +4,20 @@ import { computed, defineComponent as defineComponent2 } from "vue";
 // ../../lib/primitive/src/guards/children/component-id.ts
 var COMPONENT_DEFAULT_TAG = /* @__PURE__ */ Symbol.for("praxis.component-default-tag");
 
-// ../../lib/primitive/src/utils/is-object.ts
+// ../../lib/primitive/src/guards/foundational/is-defined.ts
+function isUndefined(value) {
+  return value === void 0;
+}
+
+// ../../lib/primitive/src/guards/foundational/is-null.ts
+function isNull(value) {
+  return value === null;
+}
+function isNonNull(value) {
+  return value != null;
+}
+
+// ../../lib/primitive/src/utils/type-guards.ts
 function isObject(value, excludeArrays = false) {
   if (value === null || typeof value !== "object") return false;
   return excludeArrays ? !Array.isArray(value) : true;
@@ -16,16 +29,6 @@ function isNumber(value) {
   return typeof value === "number";
 }
 
-// ../../lib/primitive/src/guards/foundational/is-defined.ts
-function isUndefined(value) {
-  return value === void 0;
-}
-
-// ../../lib/primitive/src/guards/foundational/is-null.ts
-function isNull(value) {
-  return value === null;
-}
-
 // ../../lib/primitive/src/guards/children/is-tag.ts
 function getAsProp(child) {
   if (!isObject(child) || !("props" in child)) return void 0;
@@ -1076,7 +1079,7 @@ var InvariantBase = class {
 };
 
 // ../../lib/contract/src/aria/polymorphic-validator.ts
-var VALID = [{ valid: true }];
+var NO_VIOLATIONS = [{ valid: true }];
 function isIntrinsicTag(tag) {
   return isString(tag);
 }
@@ -1118,17 +1121,20 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
   static #deriveContext(tag, props) {
     if (!isIntrinsicTag(tag)) return { proceed: false, result: { props, violations: [] } };
     const implicitRole = getImplicitRole(tag, props);
-    const hasRole2 = implicitRole != null || isString(props.role) && props.role.length > 0;
+    const hasRole2 = isNonNull(implicitRole) || isString(props.role) && props.role.length > 0;
     if (!hasRole2) return { proceed: false, result: { props, violations: [] } };
     const normalized = _AriaPolicyEngine.#normalizeEmptyRole(tag, props);
-    if (normalized.normalized) return { proceed: false, result: normalized.result };
-    const effectiveRole = props.role ?? implicitRole;
+    const workingProps = normalized.normalized ? normalized.result.props : props;
+    const preExistingViolations = normalized.normalized ? normalized.result.violations : [];
+    const effectiveRole = workingProps.role ?? implicitRole;
     return {
       proceed: true,
       tag,
       implicitRole,
       effectiveRole,
-      context: { tag, props, implicitRole, effectiveRole }
+      props: workingProps,
+      preExistingViolations,
+      context: { tag, props: workingProps, implicitRole, effectiveRole }
     };
   }
   static #runRules(rules, context) {
@@ -1143,7 +1149,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
         } = context;
         const { message, attribute, severity } = result;
         const resolvedMessage = message ?? result.diagnostic?.message;
-        const fallbackDiag = resolvedMessage == null ? AriaDiagnostics.invalidRole(role, tag) : void 0;
+        const fallbackDiag = isNonNull(resolvedMessage) ? void 0 : AriaDiagnostics.invalidRole(role, tag);
         violations.push({
           message: resolvedMessage ?? fallbackDiag.message,
           tag,
@@ -1151,8 +1157,8 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
           attribute,
           severity,
           phase: "evaluate",
-          ...result.diagnostic != null && { diagnostic: result.diagnostic },
-          ...fallbackDiag != null && { diagnostic: fallbackDiag }
+          ...isNonNull(result.diagnostic) && { diagnostic: result.diagnostic },
+          ...isNonNull(fallbackDiag) && { diagnostic: fallbackDiag }
         });
         if (result.fixable) fixes.push(result.fix);
       });
@@ -1160,7 +1166,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     return { violations, fixes };
   }
   static #getRules(context) {
-    if (_AriaPolicyEngine.#hasRole(context.props) || context.effectiveRole != null && _AriaPolicyEngine.#LIVE_REGION_ROLES.has(context.effectiveRole)) {
+    if (_AriaPolicyEngine.#hasRole(context.props) || isNonNull(context.effectiveRole) && _AriaPolicyEngine.#LIVE_REGION_ROLES.has(context.effectiveRole)) {
       return _AriaPolicyEngine.#pipeline;
     }
     return _AriaPolicyEngine.#implicitOnlyRules;
@@ -1168,24 +1174,36 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
   static evaluate(tag, props) {
     const derived = _AriaPolicyEngine.#deriveContext(tag, props);
     if (!derived.proceed) return derived.result;
-    const { tag: narrowedTag, implicitRole, context } = derived;
+    const {
+      tag: narrowedTag,
+      implicitRole,
+      context,
+      props: workingProps,
+      preExistingViolations
+    } = derived;
     const { violations, fixes } = _AriaPolicyEngine.#runRules(
       _AriaPolicyEngine.#getRules(context),
       context
     );
-    const next = _AriaPolicyEngine.#applyFixes(narrowedTag, implicitRole, props, fixes);
-    return { props: next, violations };
+    const next = _AriaPolicyEngine.#applyFixes(narrowedTag, implicitRole, workingProps, fixes);
+    return { props: next, violations: [...preExistingViolations, ...violations] };
   }
   static #evaluateWithRules(tag, props, extraRules) {
     const derived = _AriaPolicyEngine.#deriveContext(tag, props);
     if (!derived.proceed) return derived.result;
-    const { tag: narrowedTag, implicitRole, context } = derived;
+    const {
+      tag: narrowedTag,
+      implicitRole,
+      context,
+      props: workingProps,
+      preExistingViolations
+    } = derived;
     const { violations, fixes } = _AriaPolicyEngine.#runRules(
       [..._AriaPolicyEngine.#getRules(context), ...extraRules],
       context
     );
-    const next = _AriaPolicyEngine.#applyFixes(narrowedTag, implicitRole, props, fixes);
-    return { props: next, violations };
+    const next = _AriaPolicyEngine.#applyFixes(narrowedTag, implicitRole, workingProps, fixes);
+    return { props: next, violations: [...preExistingViolations, ...violations] };
   }
   report(violations) {
     iterate.forEach(violations, (v) => {
@@ -1194,12 +1212,12 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
       else this.warn(d);
     });
   }
-  // Cache key covers only the aria-relevant subset of props (tag + role + aria-* attrs).
-  // Non-aria props (className, onClick, etc.) do not affect ARIA decisions and are excluded
-  // so cache hits survive re-renders that only change non-aria props.
-  // Note: #extraRules are NOT included in the key — each engine instance has its own Map,
-  // so two engines with different rules never share cache entries. If caching ever becomes
-  // static/shared, rule identity would need to be folded into the key.
+  // Cache key covers only the aria-relevant subset of props (tag + role + aria-* attrs) —
+  // exactly what the built-in pipeline reads. Non-aria props (className, onClick, etc.) do
+  // not affect built-in ARIA decisions and are excluded so cache hits survive re-renders
+  // that only change non-aria props. This key alone is unsound for #extraRules, which may
+  // read arbitrary props outside this set — validate() extends it with #extraRulesKeySuffix,
+  // or bypasses the cache, to account for that.
   static #createPlanKey(tag, props) {
     if (!isIntrinsicTag(tag)) return null;
     const parts = [tag];
@@ -1215,6 +1233,23 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     if (ariaEntries.length > 0) parts.push(...ariaEntries.sort());
     return parts.join("|");
   }
+  // Extends the base cache key with the props each extra rule declares it reads (`readsProps`).
+  // Returns null — meaning "don't cache" — if any extra rule omits `readsProps` (it may read
+  // arbitrary props the key can't account for) or if a declared prop's value isn't a primitive
+  // (object/array identity isn't stably representable in a string key).
+  static #extraRulesKeySuffix(extraRules, props) {
+    const parts = [];
+    for (const rule of extraRules) {
+      const readsProps = rule.readsProps;
+      if (!isNonNull(readsProps)) return null;
+      for (const propKey of readsProps) {
+        const v = props[propKey];
+        if (v !== void 0 && !isString(v) && !isNumber(v) && typeof v !== "boolean") return null;
+        parts.push(`x:${propKey}:${String(v)}`);
+      }
+    }
+    return parts.sort().join("|");
+  }
   static #computePlan(inputProps, resultProps) {
     const removals = /* @__PURE__ */ new Set();
     const updates = {};
@@ -1238,7 +1273,12 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     return next;
   }
   validate(tag, props) {
-    const key = _AriaPolicyEngine.#createPlanKey(tag, props);
+    const baseKey = _AriaPolicyEngine.#createPlanKey(tag, props);
+    let key = baseKey;
+    if (this.#extraRules.length > 0) {
+      const suffix = _AriaPolicyEngine.#extraRulesKeySuffix(this.#extraRules, props);
+      key = isNonNull(baseKey) && isNonNull(suffix) ? `${baseKey}|${suffix}` : null;
+    }
     if (!isNull(key)) {
       const cached = this.#planCache.get(key);
       if (cached !== void 0) {
@@ -1328,7 +1368,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     implicitRole
   }) {
     const role = props.role;
-    if (!implicitRole || !role || role === implicitRole) return VALID;
+    if (!implicitRole || !role || role === implicitRole) return NO_VIOLATIONS;
     if (isStrongImplicitRole(tag) && role === "region") {
       return [
         {
@@ -1340,11 +1380,11 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
         }
       ];
     }
-    return VALID;
+    return NO_VIOLATIONS;
   }
   static #checkRedundantRole({ tag, props, implicitRole }) {
     const role = props.role;
-    if (!implicitRole || !role || role !== implicitRole) return VALID;
+    if (!implicitRole || !role || role !== implicitRole) return NO_VIOLATIONS;
     return [
       {
         valid: false,
@@ -1357,8 +1397,8 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
   }
   static #checkStandaloneRegion({ tag, props, implicitRole }) {
     const role = props.role;
-    if (role !== "region") return VALID;
-    if (!isStandaloneTag(tag)) return VALID;
+    if (role !== "region") return NO_VIOLATIONS;
+    if (!isStandaloneTag(tag)) return NO_VIOLATIONS;
     return [
       {
         valid: false,
@@ -1374,7 +1414,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     props,
     effectiveRole
   }) {
-    if (effectiveRole === "none" || effectiveRole === "presentation") return VALID;
+    if (effectiveRole === "none" || effectiveRole === "presentation") return NO_VIOLATIONS;
     const results = [];
     iterate.forEachEntry(props, (key) => {
       if (!key.startsWith("aria-")) return;
@@ -1492,12 +1532,12 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     }
   }
   static #checkAriaAttributeValues({ props, effectiveRole }) {
-    if (effectiveRole === "none" || effectiveRole === "presentation") return VALID;
+    if (effectiveRole === "none" || effectiveRole === "presentation") return NO_VIOLATIONS;
     const results = [];
     iterate.forEachEntry(props, (key, value) => {
       if (!key.startsWith("aria-")) return;
       const type = _AriaPolicyEngine.#ARIA_VALUE_TYPES.get(key);
-      if (type == null) return;
+      if (!isNonNull(type)) return;
       if (_AriaPolicyEngine.#isValidAriaValue(value, type)) return;
       results.push({
         valid: false,
@@ -1528,13 +1568,13 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     props,
     effectiveRole
   }) {
-    if (effectiveRole === "none" || effectiveRole === "presentation") return VALID;
+    if (effectiveRole === "none" || effectiveRole === "presentation") return NO_VIOLATIONS;
     const implicitLevel = _AriaPolicyEngine.#HEADING_IMPLICIT_LEVELS.get(tag);
-    if (implicitLevel == null) return VALID;
+    if (!isNonNull(implicitLevel)) return NO_VIOLATIONS;
     const raw = props["aria-level"];
-    if (raw == null) return VALID;
+    if (!isNonNull(raw)) return NO_VIOLATIONS;
     const n = typeof raw === "number" ? raw : typeof raw === "string" ? parseInt(raw, 10) : NaN;
-    if (!Number.isFinite(n) || n !== implicitLevel) return VALID;
+    if (!Number.isFinite(n) || n !== implicitLevel) return NO_VIOLATIONS;
     return [
       {
         valid: false,
@@ -1557,9 +1597,10 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     props,
     effectiveRole
   }) {
-    if (!effectiveRole || !_AriaPolicyEngine.#NAME_REQUIRED_ROLES.has(effectiveRole)) return VALID;
-    if ("aria-label" in props || "aria-labelledby" in props) return VALID;
-    if (tag === "img" && typeof props.alt === "string" && props.alt.length > 0) return VALID;
+    if (!effectiveRole || !_AriaPolicyEngine.#NAME_REQUIRED_ROLES.has(effectiveRole))
+      return NO_VIOLATIONS;
+    if ("aria-label" in props || "aria-labelledby" in props) return NO_VIOLATIONS;
+    if (tag === "img" && typeof props.alt === "string" && props.alt.length > 0) return NO_VIOLATIONS;
     return [
       {
         valid: false,
@@ -1582,9 +1623,9 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     props,
     effectiveRole
   }) {
-    if (!effectiveRole) return VALID;
+    if (!effectiveRole) return NO_VIOLATIONS;
     const required = _AriaPolicyEngine.#REQUIRED_PROPERTIES.get(effectiveRole);
-    if (required == null) return VALID;
+    if (!isNonNull(required)) return NO_VIOLATIONS;
     const results = [];
     iterate.forEach(required, (attr) => {
       if (attr in props) return;
@@ -1608,12 +1649,12 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
   ]);
   // WAI-ARIA 1.2 §6.6: aria-hidden="true" must not be placed on focusable elements.
   static #checkAriaHiddenOnFocusable({ tag, props }) {
-    if (props["aria-hidden"] !== "true" && props["aria-hidden"] !== true) return VALID;
+    if (props["aria-hidden"] !== "true" && props["aria-hidden"] !== true) return NO_VIOLATIONS;
     const isInteractive = _AriaPolicyEngine.#INTERACTIVE_TAGS.has(tag);
     if (!isInteractive) {
       const tabindex = props.tabindex;
       const n = typeof tabindex === "number" ? tabindex : typeof tabindex === "string" ? parseInt(tabindex, 10) : NaN;
-      if (!Number.isFinite(n) || n < 0) return VALID;
+      if (!Number.isFinite(n) || n < 0) return NO_VIOLATIONS;
     }
     return [
       {
@@ -1632,7 +1673,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     props,
     effectiveRole
   }) {
-    if (effectiveRole !== "none" && effectiveRole !== "presentation") return VALID;
+    if (effectiveRole !== "none" && effectiveRole !== "presentation") return NO_VIOLATIONS;
     const results = [];
     iterate.forEachEntry(props, (key) => {
       if (!key.startsWith("aria-")) return;
@@ -1656,10 +1697,10 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     ["timer", "off"]
   ]);
   static #checkMissingLiveRegion({ effectiveRole, props }) {
-    if (!effectiveRole) return VALID;
+    if (!effectiveRole) return NO_VIOLATIONS;
     const impliedLive = _AriaPolicyEngine.#LIVE_REGION_ROLES.get(effectiveRole);
-    if (!impliedLive) return VALID;
-    if ("aria-live" in props) return VALID;
+    if (!impliedLive) return NO_VIOLATIONS;
+    if ("aria-live" in props) return NO_VIOLATIONS;
     const injectLive = {
       kind: `injectLive:${effectiveRole}`,
       apply: (ctx) => ({
@@ -1679,8 +1720,9 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
     ];
   }
   static #checkMissingAtomic({ effectiveRole, props }) {
-    if (!effectiveRole || !_AriaPolicyEngine.#LIVE_REGION_ROLES.has(effectiveRole)) return VALID;
-    if ("aria-atomic" in props) return VALID;
+    if (!effectiveRole || !_AriaPolicyEngine.#LIVE_REGION_ROLES.has(effectiveRole))
+      return NO_VIOLATIONS;
+    if ("aria-atomic" in props) return NO_VIOLATIONS;
     return [
       {
         valid: false,
@@ -1704,8 +1746,8 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
   };
   static #checkInvalidAriaRelevant({ props }) {
     const relevant = props["aria-relevant"];
-    if (relevant === void 0) return VALID;
-    if (typeof relevant !== "string") return VALID;
+    if (relevant === void 0) return NO_VIOLATIONS;
+    if (typeof relevant !== "string") return NO_VIOLATIONS;
     const tokens = relevant.trim().split(/\s+/);
     const invalid = tokens.filter((t) => !_AriaPolicyEngine.#VALID_RELEVANT_TOKENS.has(t));
     if (invalid.length > 0) {
@@ -1732,7 +1774,7 @@ var AriaPolicyEngine = class _AriaPolicyEngine extends InvariantBase {
         }
       ];
     }
-    return VALID;
+    return NO_VIOLATIONS;
   }
 };
 
@@ -2433,7 +2475,7 @@ function createClassPipeline(resolved) {
     const staticClasses = staticResolver.resolve(tag, recipe !== void 0);
     const variantClasses = variantResolver.resolve({ props, recipe });
     if (!className)
-      return staticClasses && variantClasses ? `${staticClasses} ${variantClasses}` : staticClasses || variantClasses;
+      return (staticClasses && variantClasses ? `${staticClasses} ${variantClasses}` : staticClasses || variantClasses) || void 0;
     return cn(staticClasses, variantClasses, className);
   };
 }
@@ -2644,7 +2686,7 @@ function createPolymorphic2(options = {}) {
       if (process.env.NODE_ENV !== "production") {
         validateRenderProps(resolved.diagnostics, resolved, props, recipe);
       }
-      return classPipeline(tag, props, className, recipe);
+      return classPipeline(tag, props, className, recipe) || void 0;
     },
     resolveAria(tag, props) {
       return resolveAriaFn(tag, props);
@@ -2861,7 +2903,10 @@ function buildElementProps(props, className) {
   const { role, ...rest } = props;
   return {
     ...normalizeListenerKeys(rest),
-    class: className,
+    // Vue's SSR renderer and its client-side patcher disagree on `class: undefined` — SSR's
+    // normalizeClass turns it into '' and still emits class="", while the client patcher
+    // removes the attribute outright. Omitting the key entirely keeps both paths consistent.
+    ...className !== void 0 && { class: className },
     ...isKnownAriaRole(role) && { role }
   };
 }
@@ -2882,7 +2927,10 @@ function validateSlotDirectives(directives, validator) {
 function tryRenderAsChild(state, children, discarded, validator) {
   if (!validateSlotDirectives(state.directives, validator)) return null;
   if (discarded > 0) validator.warnDiscardedChildren(discarded);
-  const attrs = { ...pickAttributes(state.props), class: state.className };
+  const attrs = {
+    ...pickAttributes(state.props),
+    ...state.className !== void 0 && { class: state.className }
+  };
   const slottable = extractSlottable(children);
   if (slottable) return slottable.rebuild(cloneVNode(slottable.child, attrs));
   if (children.length === 1 && children[0] !== void 0) {

package/dist/web/index.d.ts

@@ -138,7 +138,7 @@ interface BaseClassOptions {
     baseClassName?: ClassName;
 }
 
-type ClassPipelineFn = (tag: unknown, props: AnyRecord, className?: ClassName, recipe?: string) => string;
+type ClassPipelineFn = (tag: unknown, props: AnyRecord, className?: ClassName, recipe?: string) => string | undefined;
 
 interface RecipeOptions<TVariants extends VariantMap = VariantMap> {
     recipeMap?: Record<string, RecipeTarget<TVariants>>;
@@ -216,7 +216,9 @@ type AriaInvalidWithoutFix<M extends string = string> = AriaInvalidBase<M> & {
 type AriaInvalidResult<M extends string = string> = AriaInvalidWithFix<M> | AriaInvalidWithoutFix<M>;
 type AriaResult = ValidResult | AriaInvalidResult;
 
-type AriaRule<C extends AriaContext = AriaContext> = (context: C) => readonly AriaResult[];
+type AriaRule<C extends AriaContext = AriaContext> = ((context: C) => readonly AriaResult[]) & {
+    readonly readsProps?: readonly string[];
+};
 
 type PropNormalizer = (props: Readonly<AnyRecord & IntrinsicProps>) => Partial<AnyRecord & IntrinsicProps>;