@unified-product-graph/core 0.10.0 → 0.10.2

package/dist/index.cjs

@@ -1353,6 +1353,10 @@ var UPG_EDGE_CATALOG = {
   // would be a separate catalog-mechanic change).
   competitive_analysis_dimensioned_by_classification_axis: { forward_verb: "dimensioned_by", reverse_verb: "dimensions", classification: "hierarchy", source_type: "competitive_analysis", target_type: "classification_axis" },
   classification_axis_includes_classification_value: { forward_verb: "includes", reverse_verb: "value_of", classification: "hierarchy", source_type: "classification_axis", target_type: "classification_value" },
+  // 0.10.2: also dual-registered as a cross-edge (UPG_CROSS_EDGE_TYPES) so a
+  // competitor can be classified directly against a `registry/{classification_value}`
+  // canonical, eliminating the per-graph taxonomy node. Within-graph (the catalogue
+  // case) here; cross-product against the registry canonical via create_cross_product_edge.
   competitor_classified_as_classification_value: { forward_verb: "classified_as", reverse_verb: "classification_of", classification: "semantic", source_type: "competitor", target_type: "classification_value" },
   persona_anti_fit_for_classification_value: { forward_verb: "is_anti_fit_for", reverse_verb: "should_not_be_picked_by", classification: "semantic", source_type: "persona", target_type: "classification_value" },
   persona_anti_fit_for_product: { forward_verb: "is_anti_fit_for", reverse_verb: "should_not_be_picked_by", classification: "semantic", source_type: "persona", target_type: "product" },
@@ -3248,7 +3252,8 @@ var UPG_CROSS_EDGE_TYPES = [
   "primitive_stored_as_data_type",
   "feature_rivals_competitor_feature",
   "competitor_signal_maps_to_feature",
-  "competitor_signal_surfaces_opportunity"
+  "competitor_signal_surfaces_opportunity",
+  "competitor_classified_as_classification_value"
 ];
 var REGISTRY_PRODUCT_ID = "registry";
 
@@ -7458,6 +7463,34 @@ function getDeprecatedTypes() {
   return deprecated;
 }
 var UPG_PROPERTY_MIGRATIONS = {
+  // ── v0.10.2: market_intelligence number -> assessment reshape ──────────────
+  //
+  // The v0.10.0 property-registry generator fix (#43) corrected several property
+  // types that interfaces already declared as `UPGAssessment` but the generator
+  // had emitted loosely. A pre-0.10.0 graph that stored a bare number for
+  // `market_trend.impact` / `relevance` is now invalid (the schema wants
+  // `{value,label}`). These rules wrap the bare number on its canonical scale so
+  // the migration is one command, not a hand-reshape. The enum tightenings the
+  // same #43 fix surfaced (data_flow.direction, integration_pattern.pattern_type,
+  // api_contract.protocol, service.service_type) already have rules under 0.9.12;
+  // 0.10.2 also makes ALL of them visible to validate_graph (they were applied by
+  // migrate_properties but never surfaced as a fixable drift).
+  "0.10.2": [
+    {
+      kind: "reshape_value_to_assessment",
+      type: "market_trend",
+      property: "impact",
+      scale_id: "impact_5",
+      reason: "market_trend.impact is a UPGAssessment (impact_5); the 0.10.0 generator fix (#43) tightened it from a bare number. A legacy numeric value wraps to {value,label} on impact_5."
+    },
+    {
+      kind: "reshape_value_to_assessment",
+      type: "market_trend",
+      property: "relevance",
+      scale_id: "importance_5",
+      reason: "market_trend.relevance is a UPGAssessment (importance_5); the 0.10.0 generator fix (#43) tightened it from a bare number. A legacy numeric value wraps to {value,label} on importance_5."
+    }
+  ],
   // ── v0.9.14: drop the key_result kr_status / status twin (A2) ──────────────
   //
   // key_result carried BOTH a lifecycle `status` and a `kr_status` property with
@@ -8030,6 +8063,26 @@ function migrateNodeProperties(node, fromVersion, toVersion) {
           mutated = true;
           break;
         }
+        case "reshape_value_to_assessment": {
+          const props = workingNode.properties;
+          if (!props || !(m.property in props)) break;
+          const oldValue = props[m.property];
+          let num = null;
+          if (typeof oldValue === "number" && Number.isFinite(oldValue)) num = oldValue;
+          else if (typeof oldValue === "string" && oldValue.trim() !== "" && Number.isFinite(Number(oldValue))) {
+            num = Number(oldValue);
+          }
+          if (num === null) break;
+          const label = getScale(m.scale_id)?.points.find((p) => p.value === num)?.label ?? String(num);
+          const nextProps = {
+            ...props,
+            [m.property]: { value: num, label, scale_id: m.scale_id }
+          };
+          workingNode.properties = nextProps;
+          mutated = true;
+          changes.push({ kind: "reshaped_to_assessment", property: m.property, scale_id: m.scale_id, value: num, label });
+          break;
+        }
       }
     }
   }
@@ -18250,6 +18303,7 @@ for (const fw of UPG_FRAMEWORKS) {
 
 // src/grammar/validate.ts
 var crossProductEdgeTypeSet = new Set(UPG_CROSS_EDGE_TYPES);
+var withinProductEdgeTypeSet = new Set(Object.keys(UPG_EDGE_CATALOG));
 var CONTENT_DEPTH_WARNING_RULES = /* @__PURE__ */ new Set([
   "property-type",
   "property-enum",
@@ -18352,7 +18406,7 @@ function validateUPGDocument(doc) {
         if (!e.id || typeof e.id !== "string") {
           errors.push({ path: `${path}.id`, message: "Edge id is required and must be a string" });
         }
-        if (typeof e.type === "string" && crossProductEdgeTypeSet.has(e.type)) {
+        if (typeof e.type === "string" && crossProductEdgeTypeSet.has(e.type) && !withinProductEdgeTypeSet.has(e.type)) {
           errors.push({
             path: `${path}.type`,
             message: `Cross-product edge type "${e.type}" must live in portfolio.cross_edges[], not product edges[].`
@@ -28583,7 +28637,7 @@ function isCanonical(text) {
 }
 
 // src/index.ts
-var UPG_VERSION = "0.10.0";
+var UPG_VERSION = "0.10.2";
 var UPG_FORMAT_VERSION = "0.4.0";
 var MARKDOWN_FORMAT_VERSION = "0.1";
 var UPG_TYPES = getTypes();