@contentful/experience-design-system-cli 2.11.4-dev-build-355c69b.0 → 2.11.4-dev-build-86a5f6e.0

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.11.4-dev-build-355c69b.0",
+  "version": "2.11.4-dev-build-86a5f6e.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",

package/dist/src/analyze/command.js

@@ -159,15 +159,25 @@ export function registerAnalyzeCommand(program) {
                     filterWarnings.push(`${component.name}: ${reviewNotes}`);
                 }
             }
+            // Preserve any extractor-level review reasons (e.g. `props-type-unresolved`
+            // from the Svelte parser) by merging them into the post-processing recompute.
+            // Without this, recomputing here clobbers the per-extractor signal.
+            const extractorReasons = component.reviewReasons ?? [];
             const { confidence, reasons } = computeExtractionScore(component, {
-                additionalIssueCount: wrapperConfidenceToIssueCount(inspection.wrapperConfidence),
-                additionalReasons: inspection.reviewReasons,
+                additionalIssueCount: wrapperConfidenceToIssueCount(inspection.wrapperConfidence) + extractorReasons.length,
+                additionalReasons: [...extractorReasons, ...inspection.reviewReasons],
             });
             filteredComponents.push({
                 ...component,
                 extractionConfidence: confidence,
                 reviewReasons: reasons,
-                needsReview: deriveNeedsReview(confidence) || inspection.wrapperConfidence >= 4 || inspection.keepDespiteZeroSurface,
+                needsReview: deriveNeedsReview(confidence) ||
+                    inspection.wrapperConfidence >= 4 ||
+                    inspection.keepDespiteZeroSurface ||
+                    // An extractor-level type-resolution failure is a strong signal regardless
+                    // of the otherwise-derived confidence threshold; force review.
+                    extractorReasons.includes('props-type-unresolved') ||
+                    (component.needsReview ?? false),
             });
         }
         const validatedComponents = validateExtractedComponents(filteredComponents);

package/dist/src/analyze/extract/svelte.js

@@ -30,7 +30,7 @@ export async function extractSvelteComponents(filePaths, onProgress) {
                 }
             }
             catch (e) {
-                warnings.push(`Failed to extract from ${filePath}: ${e instanceof Error ? e.message : String(e)}`);
+                warnings.push(`${getSvelteComponentName(filePath)}: failed to extract from ${filePath} — ${e instanceof Error ? e.message : String(e)}`);
             }
             filesProcessed++;
             onProgress?.({ filesProcessed, componentsFound });
@@ -44,6 +44,9 @@ export async function extractSvelteComponents(filePaths, onProgress) {
 }
 async function extractFromSvelteFile(filePath, source) {
     const warnings = [];
+    // Derive the component name up front so warning messages can prefix it for
+    // TUI grouping, even when parsing fails.
+    const name = getSvelteComponentName(filePath);
     let ast;
     try {
         ast = parseSvelte(source, { modern: true });
@@ -51,16 +54,15 @@ async function extractFromSvelteFile(filePath, source) {
     catch (e) {
         return {
             component: null,
-            warnings: [`Parse error in ${filePath}: ${e instanceof Error ? e.message : String(e)}`],
+            warnings: [`${name}: parse error in ${filePath}: ${e instanceof Error ? e.message : String(e)}`],
         };
     }
-    const name = getSvelteComponentName(filePath);
     const instance = ast['instance'];
     const moduleScript = ast['module'];
     const fragment = ast['fragment'];
     // Detect Svelte 4 export-let syntax — currently unsupported.
     if (instance && hasV4ExportLetProps(instance)) {
-        warnings.push(`Svelte 4 export let syntax not yet supported (file: ${filePath}); see INTEG-4267 for v5-only scope and follow-up`);
+        warnings.push(`${name}: Svelte 4 export let syntax not yet supported (${filePath}); see INTEG-4267 for v5-only scope and follow-up`);
         return { component: null, warnings };
     }
     // Find the $props() call (Svelte 5 runes).
@@ -78,6 +80,7 @@ async function extractFromSvelteFile(filePath, source) {
             instance: instance,
             moduleScript,
             filePath,
+            componentName: name,
             source,
             snippetLocals,
         });
@@ -95,13 +98,13 @@ async function extractFromSvelteFile(filePath, source) {
     }
     else {
         // No script block at all — nothing to extract on the props side.
-        warnings.push(`${filePath} has no instance script block; no props extracted`);
+        warnings.push(`${name}: no instance script block (${filePath}); no props extracted`);
     }
     // --- Slot extraction ---
     const templateSlots = fragment ? extractTemplateSlots(fragment) : [];
     const { slots, mixedWarning } = mergeSlots(snippetSlotsFromProps, templateSlots);
     if (mixedWarning) {
-        warnings.push(`${filePath}: mixed Snippet and <slot> usage detected; preferring Snippet entries`);
+        warnings.push(`${name}: mixed Snippet and <slot> usage detected (${filePath}); preferring Snippet entries`);
     }
     const component = {
         name,
@@ -248,7 +251,7 @@ async function extractPropsFromCall(ctx) {
     if (unresolved) {
         const refLabel = describeAnnotationForUser(annotation);
         const heritageNote = unresolved === 'partial-heritage' ? ' (heritage clauses extending unreachable types)' : '';
-        warnings.push(`${ctx.filePath}: declared Props type ${refLabel} resolved to ${unresolved === 'empty' ? '0' : 'only Snippet-typed'} properties${heritageNote} — possible cross-package extends or unreachable type. ` +
+        warnings.push(`${ctx.componentName}: declared Props type ${refLabel} resolved to ${unresolved === 'empty' ? '0' : 'only Snippet-typed'} properties${heritageNote} (${ctx.filePath}) — possible cross-package extends or unreachable type. ` +
             `See https://github.com/contentful/experience-design-system-sdk-public/pull/44 for context and partner workarounds.`);
         additionalReasons.push('props-type-unresolved');
     }
@@ -261,11 +264,11 @@ async function extractPropsFromCall(ctx) {
             return { ...extractFromTypeMembersOnly(typeMembers), warnings, additionalReasons };
         }
         if (!unresolved) {
-            warnings.push(`${ctx.filePath}: $props() called without destructuring; cannot extract individual props`);
+            warnings.push(`${ctx.componentName}: $props() called without destructuring (${ctx.filePath}); cannot extract individual props`);
         }
         return { props: [], snippetNames: new Set(), snippetSlots: [], warnings, additionalReasons };
     }
-    warnings.push(`${ctx.filePath}: unrecognized $props() binding pattern (${idType})`);
+    warnings.push(`${ctx.componentName}: unrecognized $props() binding pattern '${idType}' (${ctx.filePath})`);
     return { props: [], snippetNames: new Set(), snippetSlots: [], warnings, additionalReasons };
 }
 /**
@@ -710,7 +713,7 @@ function extractFromDestructure(propsCall, ctx, typeMembers, warnings, additiona
     // contract = the destructure list. (Type-members-only path runs separately for the
     // `const props: Props = $props()` no-destructure case.)
     if (dropsRest) {
-        warnings.push(`${ctx.filePath}: rest element in $props() destructure dropped (cannot enumerate)`);
+        warnings.push(`${ctx.componentName}: rest element in $props() destructure dropped (${ctx.filePath}); cannot enumerate`);
     }
     return {
         props: props.sort((a, b) => sortStable(a.name, b.name, propertyOrder(properties))),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.11.4-dev-build-355c69b.0",
+  "version": "2.11.4-dev-build-86a5f6e.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",
@@ -37,7 +37,7 @@
     "svelte": "^5.56.4",
     "ts-morph": "^27.0.2",
     "typescript": "^5.9.3",
-    "@contentful/experience-design-system-types": "2.11.4-dev-build-355c69b.0"
+    "@contentful/experience-design-system-types": "2.11.4-dev-build-86a5f6e.0"
   },
   "devDependencies": {
     "@tsconfig/node24": "^24.0.3",