@contentful/experience-design-system-cli 2.26.5-dev-build-e549301.0 → 2.26.5-dev-build-00663c7.0

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.26.5-dev-build-e549301.0",
+  "version": "2.26.5-dev-build-00663c7.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",

package/dist/src/index.js

@@ -118,16 +118,6 @@ function parseToolCallLines(stdout) {
       if (typeof rec.reason === "string")
         call.reason = rec.reason;
       calls.push(call);
-    } else if (tool === "exclude_prop") {
-      if (typeof rec.prop !== "string" || !rec.prop) {
-        warnings.push("exclude_prop missing prop name \u2014 skipped");
-        continue;
-      }
-      calls.push({
-        tool: "exclude_prop",
-        prop: rec.prop,
-        reason: typeof rec.reason === "string" ? rec.reason : ""
-      });
     } else if (tool === "classify_component") {
       const call = { tool: "classify_component" };
       if (typeof rec.description === "string")
@@ -383,7 +373,7 @@ var init_agent_runner = __esm({
     "use strict";
     init_agent_names();
     VALID_SELECT_TOOL_NAMES = /* @__PURE__ */ new Set(["select_component", "reject_component"]);
-    VALID_TOOL_NAMES = /* @__PURE__ */ new Set(["classify_prop", "exclude_prop", "classify_component", "classify_slot"]);
+    VALID_TOOL_NAMES = /* @__PURE__ */ new Set(["classify_prop", "classify_component", "classify_slot"]);
     VALID_TOKEN_TOOL_NAMES = /* @__PURE__ */ new Set(["set_token", "set_group"]);
     VALID_CDF_TYPES = /* @__PURE__ */ new Set(["string", "richtext", "media", "enum", "token", "boolean"]);
     VALID_CATEGORIES = /* @__PURE__ */ new Set(["content", "design", "state"]);
@@ -564,7 +554,7 @@ The three tool calls you may emit are:
 
 Rules:
 - Emit exactly one JSON object per line. No multi-line JSON. No markdown fences around the lines.
-- Every prop in the input must have exactly one classify_prop call. There is no exclude call \u2014 a prop that isn't marketer-configurable still gets classify_prop with cdf_category "state" instead of being dropped.
+- Every prop in the input must have exactly one classify_prop call.
 - Every slot in the input must have exactly one classify_slot call.
 - Valid cdf_type values: string, richtext, media, enum, token, boolean
 - Valid cdf_category values: content, design, state
@@ -8730,19 +8720,11 @@ function applyToolCalls(db, sessionId2, componentId, componentName, calls, incom
   const now = (/* @__PURE__ */ new Date()).toISOString();
   const warnings = [...incomingWarnings];
   let classified = 0;
-  let excluded = 0;
   let slots = 0;
   const updateProp = db.prepare(
     `UPDATE raw_props SET cdf_type = ?, cdf_category = ?, cdf_token_kind = ?, required = ?, description = ?, rationale = ?
      WHERE session_id = ? AND component_id = ? AND name = ?`
   );
-  const getRawPropType = db.prepare(
-    `SELECT type FROM raw_props WHERE session_id = ? AND component_id = ? AND name = ?`
-  );
-  const clearProp = db.prepare(
-    `UPDATE raw_props SET cdf_type = ?, cdf_category = 'state', cdf_token_kind = NULL, required = 0, description = ?, rationale = ?
-     WHERE session_id = ? AND component_id = ? AND name = ?`
-  );
   const deleteAllowedValues = db.prepare(
     `DELETE FROM raw_prop_allowed_values WHERE session_id = ? AND component_id = ? AND prop_name = ?`
   );
@@ -8819,18 +8801,6 @@ function applyToolCalls(db, sessionId2, componentId, componentName, calls, incom
           ).run(storedDefault, sessionId2, componentId, call.prop);
         }
         classified++;
-      } else if (call.tool === "exclude_prop") {
-        const rawProp = getRawPropType.get(sessionId2, componentId, call.prop);
-        const fallbackType = rawProp?.type === "boolean" ? "boolean" : "string";
-        clearProp.run(
-          fallbackType,
-          "Not exposed for content or design editing.",
-          call.reason || null,
-          sessionId2,
-          componentId,
-          call.prop
-        );
-        excluded++;
       } else if (call.tool === "classify_slot") {
         const slotRequired = call.required !== void 0 ? call.required ? 1 : 0 : 1;
         const slotChanges = updateSlot.run(
@@ -8870,7 +8840,7 @@ function applyToolCalls(db, sessionId2, componentId, componentName, calls, incom
     db.exec("ROLLBACK");
     throw e;
   }
-  return { classified, excluded, slots, warnings };
+  return { classified, slots, warnings };
 }
 function getOrCreateSession(db, sessionFlag, sessionName, _hints) {
   const now = (/* @__PURE__ */ new Date()).toISOString();
@@ -9345,9 +9315,8 @@ function loadCDFComponents(db, sessionId2) {
   ).all(sessionId2);
   if (components.length === 0) return [];
   const props = db.prepare(
-    // 'excluded' is no longer written by applyToolCalls (exclude_prop now lands
-    // in the 'state' bucket instead) — this guard stays for session DBs
-    // persisted before that change.
+    // Guards against stale rows in session DBs persisted before props
+    // were guaranteed to always be classified into content/design/state.
     `SELECT component_id, name, required, default_value, description,
               cdf_type, cdf_category, cdf_token_kind, position
        FROM raw_props
@@ -28499,11 +28468,6 @@ function formatToolCall(obj) {
       const cat = String(obj["cdf_category"] ?? "");
       return `    ${c.green("+")}  ${prop}  ${c.dim(`${type}  ${cat}`)}`;
     }
-    case "exclude_prop": {
-      const prop = String(obj["prop"] ?? "");
-      const reason = typeof obj["reason"] === "string" ? obj["reason"] : "";
-      return `    ${c.dim("\u2013")}  ${prop}  ${c.dim(reason)}`;
-    }
     case "classify_slot": {
       const slot = String(obj["slot"] ?? "");
       const desc = typeof obj["description"] === "string" ? obj["description"] : "";
@@ -31009,7 +30973,6 @@ async function runOneComponent(agent, model, db, sessionId2, component, tokensIn
       return {
         componentName: component.name,
         classified: 0,
-        excluded: 0,
         slots: 0,
         warnings: [],
         failed: false,
@@ -31025,7 +30988,6 @@ async function runOneComponent(agent, model, db, sessionId2, component, tokensIn
       return {
         componentName: component.name,
         classified: 0,
-        excluded: 0,
         slots: 0,
         warnings: [`${component.name}: source changed but human edits preserved`],
         failed: false,
@@ -31094,7 +31056,6 @@ ${outputBuf}`);
       return {
         componentName: component.name,
         classified: 0,
-        excluded: 0,
         slots: 0,
         warnings: [],
         failed: true,
@@ -31119,7 +31080,6 @@ ${outputBuf}`);
     return {
       componentName: component.name,
       classified: applied.classified,
-      excluded: applied.excluded,
       slots: applied.slots,
       warnings: applied.warnings,
       failed: false,
@@ -31129,7 +31089,6 @@ ${outputBuf}`);
   return {
     componentName: component.name,
     classified: 0,
-    excluded: 0,
     slots: 0,
     warnings: [],
     failed: true,
@@ -31355,12 +31314,11 @@ async function runGenerateSkill(skill, opts, verbose = false) {
       }
     }
     const totalClassified = generated.reduce((s, r) => s + r.classified, 0);
-    const totalExcluded = generated.reduce((s, r) => s + r.excluded, 0);
     const totalRenamedSlots = componentResults.reduce((s, r) => s + r.renamedSlotsCount, 0);
     const allOk = failed.length === 0;
     const cachedNote = cachedResults.length > 0 ? c.dim(`  (${cachedResults.length} cached)`) : "";
     process.stderr.write(
-      (allOk ? c.green("\u2713") : c.yellow("\u26A0")) + `  ${generated.length + cachedResults.length}/${componentResults.length} components` + cachedNote + c.dim(`  ${totalClassified} classified, ${totalExcluded} unattached`) + "\n"
+      (allOk ? c.green("\u2713") : c.yellow("\u26A0")) + `  ${generated.length + cachedResults.length}/${componentResults.length} components` + cachedNote + c.dim(`  ${totalClassified} classified`) + "\n"
     );
     process.stdout.write(`renamed-slots: ${totalRenamedSlots}
 `);

package/dist/src/session/db.d.ts

@@ -28,7 +28,6 @@ export declare function getPipelineDbPath(): string;
 export declare function openPipelineDb(dbPath?: string): DatabaseSync;
 export interface ApplyToolCallsResult {
     classified: number;
-    excluded: number;
     slots: number;
     warnings: string[];
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.26.5-dev-build-e549301.0",
+  "version": "2.26.5-dev-build-00663c7.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",
@@ -39,7 +39,7 @@
     "svelte": "^5.56.4",
     "ts-morph": "^27.0.2",
     "typescript": "^5.9.3",
-    "@contentful/experience-design-system-types": "2.26.5-dev-build-e549301.0"
+    "@contentful/experience-design-system-types": "2.26.5-dev-build-00663c7.0"
   },
   "devDependencies": {
     "@tsconfig/node24": "^24.0.4",
@@ -51,9 +51,9 @@
     "ink-testing-library": "^4.0.0",
     "typescript-eslint": "^8.67.0",
     "vitest": "^4.0.16",
-    "@contentful/experience-design-system-extraction": "2.26.5-dev-build-e549301.0",
-    "@contentful/experience-design-system-client": "2.26.5-dev-build-e549301.0",
-    "@contentful/experience-design-system-generation": "2.26.5-dev-build-e549301.0"
+    "@contentful/experience-design-system-client": "2.26.5-dev-build-00663c7.0",
+    "@contentful/experience-design-system-extraction": "2.26.5-dev-build-00663c7.0",
+    "@contentful/experience-design-system-generation": "2.26.5-dev-build-00663c7.0"
   },
   "repository": {
     "type": "git",

package/skills/generate-components.md

@@ -66,7 +66,7 @@ It is correct approximately 80% of the time for simple props. You should:
 
 ## Target schema
 
-The CLI assembles your output into CDF (Component Definition Format), a JSON schema with `$schema: "https://contentful.com/schemas/cdf/v1"`. Each component you classify produces a CDF component entry (`$type: "component"`) in the pipeline database. Properties carry `$category` (`content`, `design`, or `state`) and a `$type`. Every prop you're given ends up in the CDF — there is no "excluded" output. You do not produce this JSON directly — emit tool calls and the CLI writes the DB columns.
+The CLI assembles your output into CDF (Component Definition Format), a JSON schema with `$schema: "https://contentful.com/schemas/cdf/v1"`. Each component you classify produces a CDF component entry (`$type: "component"`) in the pipeline database. Properties carry `$category` (`content`, `design`, or `state`) and a `$type`. You do not produce this JSON directly — emit tool calls and the CLI writes the DB columns.
 
 ## Output protocol
 
@@ -82,7 +82,7 @@ Emit one JSON object per line. The CLI parses lines starting with `{`. Lines not
 {"tool":"classify_slot","slot":"<slotName>","required":<bool>,"allowed_components":["ComponentName"],"description":"<short customer-facing description>","rationale":"<why this slot was kept / its role>"}
 ```
 
-**`exclude_prop` no longer exists.** Every prop — including framework internals, DOM/accessibility pass-through, and props with no clean flat representation — must produce a `classify_prop` call. Props that are not marketer-configurable are classified with `cdf_category: "state"` instead of being dropped: they still ship in the CDF (visible in review, editable if someone wants to override the category), but stay unattached from the content/design editing surfaces by default. See "Valid cdf_category values" below for exactly what falls into `state`.
+Every prop — including framework internals, DOM/accessibility pass-through, and props with no clean flat representation — must produce a `classify_prop` call. Props that are not marketer-configurable are classified with `cdf_category: "state"`: they still ship in the CDF (visible in review, editable if someone wants to override the category), but stay unattached from the content/design editing surfaces by default. See "Valid cdf_category values" below for exactly what falls into `state`.
 
 **Rules:**
 - Emit exactly one JSON object per line. No multi-line JSON.
@@ -135,7 +135,7 @@ Exactly **6** valid types:
 |---|---|
 | `content` | Data the component *displays* — what a copywriter or editor fills in: text, labels, headings, body copy, rich text, images, media, URLs, link targets, counts, locale |
 | `design` | Values that control *how the component looks* — what a designer sets: color, size (sm/md/lg), variant (primary/secondary/ghost), layout orientation, alignment, background, visual toggles (imageOnLeft, enableEffect), design tokens |
-| `state` | Two groups, both unattached from the content/design editing surfaces by default: (1) runtime behavioral or interactive flags — disabled, loading, expanded, isOpen, isSearchVisible, preview, identifiers used for analytics/tracking (componentId, sectionKey, componentName); (2) props that aren't marketer-configurable at all — framework internals, DOM/accessibility pass-through, callbacks, refs, and any type with no clean flat representation. Group (2) is where `exclude_prop` calls used to go — classify them here instead of dropping them. |
+| `state` | Two groups, both unattached from the content/design editing surfaces by default: (1) runtime behavioral or interactive flags — disabled, loading, expanded, isOpen, isSearchVisible, preview, identifiers used for analytics/tracking (componentId, sectionKey, componentName); (2) props that aren't marketer-configurable at all — framework internals, DOM/accessibility pass-through, callbacks, refs, and any type with no clean flat representation. |
 
 The pre-classified `category` in the raw input is a starting point — correct it when it is wrong. Contentful uses this category to decide where the property appears in the editor UI, so accuracy matters.