@principal-ai/principal-view-cli 0.3.5 → 0.3.7

package/dist/index.cjs

@@ -229771,9 +229771,9 @@ ${e.message}`;
   }
 });
 
-// node_modules/ignore/index.js
+// node_modules/@principal-ai/codebase-composition/node_modules/ignore/index.js
 var require_ignore2 = __commonJS({
-  "node_modules/ignore/index.js"(exports2, module2) {
+  "node_modules/@principal-ai/codebase-composition/node_modules/ignore/index.js"(exports2, module2) {
     function makeArray(subject) {
       return Array.isArray(subject) ? subject : [subject];
     }
@@ -241497,11 +241497,12 @@ var NarrativeValidator = class {
     const violations = [];
     violations.push(...this.checkSchema(context));
     violations.push(...this.checkCanvasExists(context));
+    violations.push(...this.checkScenarios(context));
+    violations.push(...this.checkEventNameSyntax(context));
     if (context.canvas) {
       violations.push(...this.checkEventReferences(context));
       violations.push(...this.checkAttributeReferences(context));
     }
-    violations.push(...this.checkScenarios(context));
     violations.push(...this.checkTemplateSyntax(context));
     violations.push(...this.checkFormattingOptions(context));
     return this.aggregateResults(violations);
@@ -241938,6 +241939,64 @@ var NarrativeValidator = class {
     }
     return violations;
   }
+  /**
+   * Check that event names don't use attribute filter syntax
+   */
+  checkEventNameSyntax(context) {
+    const violations = [];
+    const { narrative, narrativePath } = context;
+    narrative.scenarios.forEach((scenario, scenarioIdx) => {
+      if (scenario.condition?.requires) {
+        scenario.condition.requires.forEach((eventPattern, idx) => {
+          if (eventPattern.includes("[") && eventPattern.includes("]")) {
+            violations.push({
+              ruleId: "narrative-event-name-syntax",
+              severity: "error",
+              file: narrativePath,
+              path: `scenarios[${scenarioIdx}].condition.requires[${idx}]`,
+              message: `Event name uses unsupported [attribute=value] syntax: "${eventPattern}"`,
+              impact: "Attribute filter syntax is not supported - event will not match",
+              suggestion: `Use a distinct event name instead (e.g., "${this.extractBaseEventName(eventPattern)}.${this.extractAttributeValue(eventPattern)}")`,
+              fixable: false
+            });
+          }
+        });
+      }
+      if (scenario.condition?.excludes) {
+        scenario.condition.excludes.forEach((eventPattern, idx) => {
+          if (eventPattern.includes("[") && eventPattern.includes("]")) {
+            violations.push({
+              ruleId: "narrative-event-name-syntax",
+              severity: "error",
+              file: narrativePath,
+              path: `scenarios[${scenarioIdx}].condition.excludes[${idx}]`,
+              message: `Event name uses unsupported [attribute=value] syntax: "${eventPattern}"`,
+              impact: "Attribute filter syntax is not supported - event will not match",
+              suggestion: `Use a distinct event name instead (e.g., "${this.extractBaseEventName(eventPattern)}.${this.extractAttributeValue(eventPattern)}")`,
+              fixable: false
+            });
+          }
+        });
+      }
+      if (scenario.template?.events) {
+        Object.keys(scenario.template.events).forEach((eventName) => {
+          if (eventName.includes("[") && eventName.includes("]")) {
+            violations.push({
+              ruleId: "narrative-event-name-syntax",
+              severity: "error",
+              file: narrativePath,
+              path: `scenarios[${scenarioIdx}].template.events["${eventName}"]`,
+              message: `Event name uses unsupported [attribute=value] syntax: "${eventName}"`,
+              impact: "Attribute filter syntax is not supported - template will never render",
+              suggestion: `Use a distinct event name instead (e.g., "${this.extractBaseEventName(eventName)}.${this.extractAttributeValue(eventName)}")`,
+              fixable: false
+            });
+          }
+        });
+      }
+    });
+    return violations;
+  }
   /**
    * Check template syntax (balanced braces, valid expressions)
    */
@@ -242130,6 +242189,36 @@ var NarrativeValidator = class {
     const regex = new RegExp(`^${pattern}$`);
     return availableEvents.some((e) => regex.test(e));
   }
+  /**
+   * Extract base event name from event pattern
+   *
+   * Examples:
+   * - "installation.started" -> "installation.started"
+   * - "installation.progress[stage=skills_discovered]" -> "installation.progress"
+   * - "error.*[severity=high]" -> "error.*"
+   */
+  extractBaseEventName(eventPattern) {
+    const bracketIndex = eventPattern.indexOf("[");
+    if (bracketIndex === -1) {
+      return eventPattern;
+    }
+    return eventPattern.substring(0, bracketIndex);
+  }
+  /**
+   * Extract attribute value from event pattern for suggestion
+   *
+   * Examples:
+   * - "installation.progress[stage=skills_discovered]" -> "skills_discovered"
+   * - "error.*[severity=high]" -> "high"
+   * - "installation.started" -> ""
+   */
+  extractAttributeValue(eventPattern) {
+    const match = eventPattern.match(/\[.*?=(.*?)\]/);
+    if (!match) {
+      return "";
+    }
+    return match[1];
+  }
 };
 function createNarrativeValidator() {
   return new NarrativeValidator();