@contentful/experience-design-system-extraction 2.35.1-dev-build-1f9cff15.0 → 2.35.1-dev-build-08991bee.0

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-extraction",
-  "version": "2.35.1-dev-build-1f9cff15.0",
+  "version": "2.35.1-dev-build-08991bee.0",
   "description": "Component extraction engine for the Contentful Experience Design System SDK",
   "license": "MIT",
   "type": "module",

package/dist/src/extract/services/classification/classify-component-props.js

@@ -1,4 +1,5 @@
 import { isStringLiteralUnion, isBooleanType, isStringType, isNumberType, isComplexType, } from './helpers/check-prop-types.js';
+import { isDomPassThroughProp } from './helpers/detect-dom-props.js';
 /**
  * Deterministic pre-classification rule engine.
  * Applies rules in priority order and returns on the first match.
@@ -25,6 +26,16 @@ export function preClassifyProp(prop) {
     if (type.includes('Dispatch<') || type.includes('SetStateAction')) {
         return { category: 'exclude' };
     }
+    // Rule 6: DOM / a11y / framework pass-through props
+    if (isDomPassThroughProp(name)) {
+        return { category: 'exclude' };
+    }
+    // `name` is overloaded: form controls forward it to the DOM, while icons,
+    // flags, and animations often use it as an authorable semantic selector.
+    // Exclude only when the extractor retained concrete DOM provenance.
+    if (name === 'name' && prop.domAttribute) {
+        return { category: 'exclude' };
+    }
     // Rule 7: String literal union
     if (isStringLiteralUnion(type)) {
         return { category: 'design', cdfTypeHint: 'enum' };
@@ -102,13 +113,21 @@ export function preClassifyComponent(component) {
         if (result?.category === 'exclude') {
             return [];
         }
+        // Provenance is extraction-only evidence. Do not add it to the payload
+        // presented to downstream agents.
+        const { domAttribute: _domAttribute, ...authorableProp } = prop;
         if (prop.category) {
-            return [prop];
+            return [authorableProp];
         }
         if (!result) {
-            return [prop];
+            return [authorableProp];
         }
-        return [{ ...prop, category: result.category }];
+        return [
+            {
+                ...authorableProp,
+                category: result.category,
+            },
+        ];
     });
     return { ...component, props };
 }

package/dist/src/extract/services/classification/helpers/detect-dom-props.d.ts

@@ -0,0 +1,2 @@
+/** Returns true when the prop name is a DOM attribute, a11y attribute, or framework pass-through escape hatch. */
+export declare function isDomPassThroughProp(name: string): boolean;

package/dist/src/extract/services/classification/helpers/detect-dom-props.js

@@ -0,0 +1,63 @@
+const DOM_PASS_THROUGH_PROPS = new Set([
+    // Bare HTML / framework styling pass-through
+    'className',
+    'class',
+    'classes',
+    'classNames',
+    'rootClassName',
+    'prefixCls',
+    'style',
+    'styles',
+    // Bare HTML attributes
+    'id',
+    'role',
+    'tabIndex',
+    'tabindex',
+    'htmlFor',
+    'for',
+    'slot',
+    'is',
+    'lang',
+    'dir',
+    'hidden',
+    'draggable',
+    'spellCheck',
+    'spellcheck',
+    'contentEditable',
+    'contenteditable',
+    'inputMode',
+    'inputmode',
+    'autoComplete',
+    'autocomplete',
+    'autoFocus',
+    'autofocus',
+    'translate',
+    'part',
+    'exportparts',
+    'aria',
+    // Framework theming / pass-through escape hatches — dev-facing, never marketer-configurable
+    'dt',
+    'pt',
+    'ptOptions',
+    'unstyled',
+    'sx',
+    // Polymorphic component props — change rendered HTML/component, not marketer-visible behavior
+    'as',
+    'element',
+    'component',
+    // QA / vendor test attributes
+    'dataQa',
+    'data-qa',
+    // Vue v-model internals — framework wiring, never marketer-configurable
+    'modelValue',
+    'modelModifiers',
+]);
+/** Returns true when the prop name is a DOM attribute, a11y attribute, or framework pass-through escape hatch. */
+export function isDomPassThroughProp(name) {
+    if (DOM_PASS_THROUGH_PROPS.has(name))
+        return true;
+    // aria-label, aria-hidden, ariaLabel, ariaHidden — both kebab and camel forms
+    if (/^aria[-A-Z]/.test(name))
+        return true;
+    return name.startsWith('data-');
+}

package/dist/src/extract/types/classification.d.ts

@@ -2,11 +2,3 @@ export interface PreClassification {
     category: 'content' | 'design' | 'state' | 'exclude';
     cdfTypeHint?: 'string' | 'enum' | 'richtext' | 'media' | 'boolean';
 }
-export type PropBucket = 'custom' | 'dom-passthrough' | 'other';
-export interface PropBucketAssignment {
-    component: string;
-    source: string;
-    customPropNames: string[];
-    domPassthroughPropNames: string[];
-    otherPropNames: string[];
-}

package/dist/src/index.d.ts

@@ -24,5 +24,4 @@ export { validateExtractedComponents, shouldExcludeDueToValidation, formatExclus
 export { parseImportedNames } from './extract/helpers/evidence/parse-source-imports.js';
 export { CONTENT_NAME_EXCEPTIONS, isReactNodeType, isArrayReactNodeType, shouldBeSlot, } from './extract/helpers/evidence/detect-slot-from-prop.js';
 export { preClassifyProp, preClassifyComponent } from './extract/services/classification/classify-component-props.js';
-export { bucketComponentProps, bucketComponentsProps, } from './extract/services/classification/bucket-component-props.js';
-export type { PreClassification, PropBucket, PropBucketAssignment } from './extract/types/classification.js';
+export type { PreClassification } from './extract/types/classification.js';

package/dist/src/index.js

@@ -25,4 +25,3 @@ export { parseImportedNames } from './extract/helpers/evidence/parse-source-impo
 export { CONTENT_NAME_EXCEPTIONS, isReactNodeType, isArrayReactNodeType, shouldBeSlot, } from './extract/helpers/evidence/detect-slot-from-prop.js';
 // Pre-classification
 export { preClassifyProp, preClassifyComponent } from './extract/services/classification/classify-component-props.js';
-export { bucketComponentProps, bucketComponentsProps, } from './extract/services/classification/bucket-component-props.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-extraction",
-  "version": "2.35.1-dev-build-1f9cff15.0",
+  "version": "2.35.1-dev-build-08991bee.0",
   "description": "Component extraction engine for the Contentful Experience Design System SDK",
   "license": "MIT",
   "type": "module",

package/dist/src/extract/services/classification/bucket-component-props.d.ts

@@ -1,4 +0,0 @@
-import type { RawComponentDefinition } from '../../types/component.js';
-import type { PropBucketAssignment } from '../../types/classification.js';
-export declare function bucketComponentProps(component: RawComponentDefinition): PropBucketAssignment;
-export declare function bucketComponentsProps(components: readonly RawComponentDefinition[]): PropBucketAssignment[];

package/dist/src/extract/services/classification/bucket-component-props.js

@@ -1,29 +0,0 @@
-export function bucketComponentProps(component) {
-    const customPropNames = [];
-    const domPassthroughPropNames = [];
-    const otherPropNames = [];
-    for (const prop of component.props) {
-        if (!prop.type.trim()) {
-            otherPropNames.push(prop.name);
-        }
-        else if (prop.domAttribute === true) {
-            domPassthroughPropNames.push(prop.name);
-        }
-        else {
-            customPropNames.push(prop.name);
-        }
-    }
-    const sortNames = (names) => names.toSorted((left, right) => left.localeCompare(right));
-    return {
-        component: component.name,
-        source: component.source,
-        customPropNames: sortNames(customPropNames),
-        domPassthroughPropNames: sortNames(domPassthroughPropNames),
-        otherPropNames: sortNames(otherPropNames),
-    };
-}
-export function bucketComponentsProps(components) {
-    return components
-        .map(bucketComponentProps)
-        .toSorted((left, right) => `${left.component}\u0000${left.source}`.localeCompare(`${right.component}\u0000${right.source}`));
-}