@prorigo/protrak-forge 0.4.0 → 0.4.1

package/bin/protrak-forge.js

@@ -17948,7 +17948,7 @@ function getClient() {
 }
 
 // src/version.ts
-var PACKAGE_VERSION = "0.4.0";
+var PACKAGE_VERSION = "0.4.1";
 
 // src/tools/_schema-write-utils.ts
 var fs4 = __toESM(require("fs"));
@@ -22486,9 +22486,6 @@ function serializeStyle(style) {
       if (!FONT_STYLE_OPTIONS.includes(v)) {
         return { error: `font_style entry '${v}' must be one of: ${FONT_STYLE_OPTIONS.join(", ")}.` };
       }
-      if (seen.has(v)) {
-        return { error: `font_style contains duplicate entry '${v}'.` };
-      }
       seen.add(v);
     }
     if (seen.has("Bold")) tokens.push("fontWeight:bold");
@@ -22753,7 +22750,7 @@ var GROUP_INPUT_SCHEMA = {
 };
 var TOOL_DEF29 = {
   name: "generate_protrak_rule",
-  description: "Generate and write a Rule JSON file to <workspace>/Rules/<name>.json. Rules express attribute-filter logic used by display conditions, conditional formatting, dependent picklists, and Access Policy. Inputs are strictly structured: the agent should translate natural-language intent into attribute_filter_groups using the schema returned by get_protrak_rule_context. Validates: PascalCase name; Instance-context attribute_name exists in the workspace (or is a recognised basic attribute); the condition is valid for the attribute's type; value presence matches the condition (e.g. Between requires both, IsEmpty rejects values); dynamic values match @Instance.X, @User.X, or $StartOfToday()/$StartOfMonth(\xB1Nd) patterns; @Instance dynamic references resolve to known attributes. Pass error_message only when the rule will be used as an Access Policy. Refuses to overwrite an existing file unless overwrite:true.",
+  description: "Generate and write a Rule JSON file to <workspace>/Rules/<name>.json. Rules express attribute-filter logic used by display conditions, conditional formatting, dependent picklists, and Access Policy. Inputs are strictly structured: the agent should translate natural-language intent into attribute_filter_groups using the schema returned by get_protrak_rule_context. Validates: PascalCase name; Instance-context attribute_name exists in the workspace (or is a recognised basic attribute); the condition is valid for the attribute's type; value presence matches the condition (e.g. Between requires both, IsEmpty rejects values); dynamic values match @Instance.X, @User.X, or $StartOfToday()/$StartOfMonth(\xB1N[d|w|m|y]) patterns; @Instance dynamic references resolve to known attributes. Pass error_message only when the rule will be used as an Access Policy. Refuses to overwrite an existing file unless overwrite:true.",
   inputSchema: {
     type: "object",
     properties: {
@@ -22816,12 +22813,13 @@ function validateCondition(provider, cond, prefix) {
   if (typeof cond.condition !== "string" || !cond.condition) {
     return `${prefix}.condition is required.`;
   }
-  let validConditions;
   if (cond.attribute_context === "UserRoles") {
     if (cond.attribute_name) {
       return `${prefix}.attribute_name must be omitted when attribute_context is 'UserRoles'.`;
     }
-    validConditions = USER_ROLES_CONDITIONS;
+    if (!USER_ROLES_CONDITIONS.includes(cond.condition)) {
+      return `${prefix}.condition '${cond.condition}' is not valid for UserRoles context. Valid: ${USER_ROLES_CONDITIONS.join(", ")}.`;
+    }
   } else {
     if (!cond.attribute_name) {
       return `${prefix}.attribute_name is required for attribute_context='${cond.attribute_context}'.`;
@@ -22834,7 +22832,7 @@ function validateCondition(provider, cond, prefix) {
       if (!attrType) {
         return `${prefix} references Instance attribute '${cond.attribute_name}' which does not exist in the workspace. Add the attribute (or use a basic attribute name like Name/State/Created/Creator) before generating this rule.`;
       }
-      validConditions = CONDITIONS_BY_ATTR_TYPE[attrType] ?? CONDITIONS_BY_ATTR_TYPE["Text"];
+      const validConditions = CONDITIONS_BY_ATTR_TYPE[attrType] ?? CONDITIONS_BY_ATTR_TYPE["Text"];
       if (!validConditions.includes(cond.condition)) {
         return `${prefix}.condition '${cond.condition}' is not valid for Instance attribute '${cond.attribute_name}' (type=${attrType}). Valid conditions: ${validConditions.join(", ")}.`;
       }
@@ -22849,12 +22847,8 @@ function validateCondition(provider, cond, prefix) {
       if (!acceptable.has(cond.condition)) {
         return `${prefix}.condition '${cond.condition}' is not recognised for User-context conditions.`;
       }
-      validConditions = Array.from(acceptable);
     }
   }
-  if (cond.attribute_context === "UserRoles" && !USER_ROLES_CONDITIONS.includes(cond.condition)) {
-    return `${prefix}.condition '${cond.condition}' is not valid for UserRoles context. Valid: ${USER_ROLES_CONDITIONS.join(", ")}.`;
-  }
   const valueless = VALUE_LESS_CONDITIONS.has(cond.condition);
   const ranged = RANGE_CONDITIONS.has(cond.condition);
   const hasFirst = cond.first_value !== void 0;
@@ -22995,7 +22989,7 @@ var DISPLAY_TARGET_KINDS = ["layout_template", "form"];
 var FORMAT_TARGET_KINDS = ["type_widget", "form"];
 var TOOL_DEF30 = {
   name: "attach_protrak_rule",
-  description: "Attach an existing Rule to a Protrak schema artifact. One polymorphic tool covering five consumer shapes selected by `usage`: 'display_condition' (LayoutTemplate widget displayConditions[] or Form field displayCondition); 'format_condition' (ViewLayout field formatConditions); 'conditional_formatting' (TypeWidget column or Form field conditionalFormatting); 'access_policy' (Type.accessPolicyRule); 'dependent_picklist' (Picklist option.rule). Rule must already exist in Rules/. Target file/widget/attribute/option must exist; the tool refuses to silently create them. Style is structured ({ background_color?, fore_color?, font_style? }) and serialized to the comma-terminated CSS-string the SPA evaluator expects.",
+  description: "Attach an existing Rule to a Protrak schema artifact. One polymorphic tool covering five consumer shapes selected by `usage`: 'display_condition' (LayoutTemplate widget displayConditions[], Form container displayConditions[], or Form field displayCondition); 'format_condition' (ViewLayout field formatConditions); 'conditional_formatting' (TypeWidget column or Form field conditionalFormatting); 'access_policy' (Type.accessPolicyRule); 'dependent_picklist' (Picklist option.rule). Rule must already exist in Rules/. Target file/widget/attribute/option must exist; the tool refuses to silently create them. Style is structured ({ background_color?, fore_color?, font_style? }) and serialized to the comma-terminated CSS-string the SPA evaluator expects.",
   inputSchema: {
     type: "object",
     properties: {
@@ -23028,7 +23022,7 @@ var TOOL_DEF30 = {
       },
       container_id: {
         type: "string",
-        description: "Form target only: formContainerKey to disambiguate when multiple containers exist."
+        description: "Form target only: formContainerKey. For usage='display_condition': when provided WITHOUT attribute_name, attaches the rule to the container itself (plural displayConditions[]). When attribute_name is also provided, disambiguates which container to search for that field. For usage='conditional_formatting': disambiguates which container holds the field. When omitted on a field lookup and the same attribute exists in multiple containers, the first match wins \u2014 pass container_id to disambiguate."
       },
       section_name: {
         type: "string",
@@ -23084,7 +23078,6 @@ function findFormField(formData, attrName, containerId) {
       return {
         result: {
           field: fields[idx],
-          index: idx,
           parent: fields,
           containerKey: container["formContainerKey"] ?? ""
         }
@@ -23158,16 +23151,50 @@ function attachDisplayCondition(provider, args) {
     );
   }
   const attrName = args["attribute_name"];
-  if (!attrName) {
-    return errorResponse("attribute_name is required for usage='display_condition' on a form.");
+  const containerId = args["container_id"] ?? void 0;
+  if (!attrName && !containerId) {
+    return errorResponse(
+      "Either attribute_name (to attach to a field) or container_id (to attach to a whole container) is required for usage='display_condition' on a form."
+    );
   }
-  if (!isPascalCase(attrName)) return errorResponse(pascalCaseError("attribute_name", attrName));
   const filePath = path17.join(provider.ws.formsDir, `${targetName}.json`);
   if (!fs21.existsSync(filePath)) {
     return errorResponse(`Form '${targetName}' not found at ${filePath}.`);
   }
   const data = readJsonFile(filePath);
-  const containerId = args["container_id"] ?? void 0;
+  if (!attrName) {
+    const config2 = data["formConfiguration"] ?? {};
+    const containers = Array.isArray(config2["containers"]) ? config2["containers"] : [];
+    const container = containers.find((c) => c["formContainerKey"] === containerId);
+    if (!container) {
+      return errorResponse(`Container '${containerId}' not found in form '${targetName}'.`, {
+        available_containers: containers.map((c) => c["formContainerKey"] ?? "")
+      });
+    }
+    const existing2 = Array.isArray(container["displayConditions"]) ? container["displayConditions"] : [];
+    if (existing2.includes(ruleName)) {
+      if (!replace) {
+        return errorResponse(
+          `Rule '${ruleName}' is already in displayConditions for container '${containerId}'. Pass replace:true to no-op silently.`
+        );
+      }
+    } else {
+      existing2.push(ruleName);
+    }
+    container["displayConditions"] = existing2;
+    writeJsonFilePath(filePath, data);
+    return successResponse(
+      {
+        usage: "display_condition",
+        target_kind: "form",
+        target_path: filePath,
+        container_id: containerId,
+        display_conditions: existing2
+      },
+      { nextSteps: [`Verify by reading Forms/${targetName}.json.`] }
+    );
+  }
+  if (!isPascalCase(attrName)) return errorResponse(pascalCaseError("attribute_name", attrName));
   const lookup2 = findFormField(data, attrName, containerId);
   if (lookup2.error) return errorResponse(lookup2.error, lookup2.available_containers ? { available_containers: lookup2.available_containers } : void 0);
   const field = lookup2.result.field;
@@ -23279,7 +23306,8 @@ function attachFormatCondition(provider, args) {
       usage: "format_condition",
       target_path: filePath,
       attribute_name: attrName,
-      format_conditions: fc
+      entry,
+      total_format_conditions: fc.length
     },
     { nextSteps: [`Verify by reading ViewLayouts/${targetName}.json.`] }
   );
@@ -23371,7 +23399,8 @@ function attachConditionalFormatting(provider, args) {
       target_kind: targetKind,
       target_path: filePath,
       attribute_name: attrName,
-      conditional_formatting: cf
+      entry,
+      total_conditional_formatting: cf.length
     },
     { nextSteps: [`Verify by reading ${path17.basename(filePath)}.`] }
   );

package/data/CLAUDE.md.template

@@ -178,6 +178,9 @@ Call `attach_protrak_rule(usage, rule_name, ...)`. One polymorphic tool covers f
   — appends to the widget's `displayConditions: string[]`.
 - `usage='display_condition'`, `target_kind='form'`, `target_name`, `attribute_name`,
   `container_id?` — sets the **singular** `displayCondition` on the form field.
+- `usage='display_condition'`, `target_kind='form'`, `target_name`, `container_id`
+  (no `attribute_name`) — appends to the container's **plural** `displayConditions: string[]`
+  (same shape as a LayoutTemplate widget).
 - `usage='format_condition'`, `target_name=<ViewLayout>`, `attribute_name`, `section_name?`,
   `widget_name?`, `style`, `order_index?` — appends `{ rule:{name}, orderIndex, style }`
   to the ViewLayout field's `formatConditions`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prorigo/protrak-forge",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Protrak domain context for coding agents — MCP server for GitHub Copilot, Claude, and Cursor",
   "bin": {
     "protrak-forge": "./bin/protrak-forge.js"