bpmnlint-plugin-camunda-compat 2.58.0 → 2.58.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bpmnlint-plugin-camunda-compat",
-  "version": "2.58.0",
+  "version": "2.58.2",
   "description": "A bpmnlint plug-in for Camunda compatibility",
   "main": "index.js",
   "scripts": {

package/rules/utils/element.js

@@ -524,13 +524,19 @@ module.exports.findAncestorAdHocSubProcess = findAncestorAdHocSubProcess;
 // detached tools ad-hoc sub-process as a tool container to lint. This is
 // read-only and has no execution effect; it parses on the current moddle with
 // no schema change. The marker is an independent boolean, not a single-valued
-// role enum, because a Camunda agent element can carry both the
-// `toolContainer` role (hosts tools) and the `agent` role at once; a
-// single-valued `io.camunda.agenticai.role` property could not represent
-// both roles on the same element.
+// role enum: the original design paired it with an `io.camunda.agenticai.agent`
+// value so a single element could in principle declare both, but no template
+// ever shipped that value, so this plugin only checks for `toolContainer`.
 const TOOL_CONTAINER_PROPERTY = 'io.camunda.agenticai.toolContainer';
 
-function hasToolContainerRoleProperty(node) {
+// The AI Agent job-worker template applied to an ad-hoc sub-process. Its id is
+// stable across template versions (versioning is tracked separately via
+// zeebe:modelerTemplateVersion), so matching it recognizes every version. Older
+// versions of this template predate the TOOL_CONTAINER_PROPERTY marker, so
+// their AHSPs are agentic but carry no marker; the template id identifies them.
+const LEGACY_AI_AGENT_TEMPLATE = 'io.camunda.connectors.agenticai.aiagent.jobworker.v1';
+
+function hasToolContainerProperty(node) {
   const properties = findExtensionElement(node, 'zeebe:Properties');
 
   if (!properties) {
@@ -543,7 +549,7 @@ function hasToolContainerRoleProperty(node) {
   );
 }
 
-module.exports.hasToolContainerRoleProperty = hasToolContainerRoleProperty;
+module.exports.hasToolContainerProperty = hasToolContainerProperty;
 
 // Whether an ad-hoc sub-process should have agent tool contracts linted.
 //
@@ -556,8 +562,16 @@ module.exports.hasToolContainerRoleProperty = hasToolContainerRoleProperty;
 // Deliberately does not fall back to a bare `zeebe:AdHoc` extension: that
 // extension is also carried by plain ad-hoc sub-processes using output
 // collection, so it can't reliably distinguish agentic from non-agentic ones.
+//
+// Older AI Agent job-worker templates predate the property marker; they are
+// still agentic and are recognized by their (version-agnostic) element template
+// id (see LEGACY_AI_AGENT_TEMPLATE).
 function isAgenticAdHocSubProcess(ahsp, version) {
-  if (hasToolContainerRoleProperty(ahsp)) {
+  if (hasToolContainerProperty(ahsp)) {
+    return true;
+  }
+
+  if (ahsp.get('modelerTemplate') === LEGACY_AI_AGENT_TEMPLATE) {
     return true;
   }