@red-hat-developer-hub/backstage-plugin-catalog-backend-module-ai-resource-agent 0.2.1 → 0.4.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # @red-hat-developer-hub/backstage-plugin-catalog-backend-module-ai-resource-agent
 
+## 0.4.0
+
+### Minor Changes
+
+- 0c396f8: switch ai-model-server and ai-resource-agent from catalog model layer's addModelSource to addProcessor
+
+## 0.3.0
+
+### Minor Changes
+
+- 6ddfd98: Add `AiResourceAgentProcessor` to validate agent-specific fields on
+  `kind: AiResource` entities with `spec.type: agent` at catalog ingestion.
+  Register the processor alongside the existing agent catalog model source.
+- 6ddfd98: Make `spec.instructions` optional on agent AiResource entities so agents
+  with a baked-in default prompt can be registered without a catalog-side
+  system prompt.
+
+### Patch Changes
+
+- Updated dependencies [6ddfd98]
+  - @red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent@0.3.0
+
 ## 0.2.1
 
 ### Patch Changes

package/README.md

@@ -0,0 +1,67 @@
+# @red-hat-developer-hub/backstage-plugin-catalog-backend-module-ai-resource-agent
+
+A Backstage catalog backend module that validates agent-shaped
+`AiResource` entities at ingestion time.
+
+Uses `catalogProcessingExtensionPoint.addProcessor()` so the catalog's
+built-in kind processor (`BuiltinKindsEntityProcessor`) remains active
+for standard kinds (User, Group, Component, etc.).
+
+This package is intentionally separate from
+`catalog-backend-module-ai-resource-extensions`, which owns shared RHDH
+extensions such as `spec.scope` and OCI `source-location` checks.
+
+## What it registers
+
+On init the module registers `AiResourceAgentProcessor`, which:
+
+1. Validates `kind: AiResource`, `spec.type: agent` entities via
+   `validateEntityKind`
+2. Validates agent-specific fields via `preProcessEntity`
+
+## Validation
+
+### `AiResourceAgentProcessor`
+
+Runs only for `kind: AiResource`. When `spec.type` is `'agent'`, validates
+agent-specific fields and collects all violations into a single error.
+Non-agent AiResource entities (`skill`, `rule`, `model`, …) are unaffected.
+
+| Field                     | Rule                                        |
+| ------------------------- | ------------------------------------------- |
+| `spec.instructions`       | Optional; must be a string if present       |
+| `spec.handoffs`           | Optional; must be an array if present       |
+| `spec.tools`              | Optional; must be an array if present       |
+| `spec.resetToolChoice`    | Optional; must be a boolean if present      |
+| `spec.modelSettings`      | Optional; must be a plain object if present |
+| `spec.toolUseBehavior`    | Optional; string or string array if present |
+| `spec.outputSchema`       | Optional; string or plain object if present |
+| `spec.handoffDescription` | Optional; must be a string if present       |
+| `spec.model`              | Optional; must be a string if present       |
+
+`handoffs` / `tools` accept opaque string arrays without entity-ref format
+enforcement. Core entity fields such as `spec.owner` and `spec.lifecycle`
+are not re-validated here.
+
+Deeper schema typing (including array element types) is handled by the
+companion `catalog-model-ai-resource-agent` package.
+
+## Related packages
+
+| Package                                                   | Responsibility                                     |
+| --------------------------------------------------------- | -------------------------------------------------- |
+| `catalog-model-ai-resource-agent`                         | Types, JSON schema, KindValidator                  |
+| `catalog-backend-module-ai-resource-agent` (this package) | Agent kind validation + field-level processor      |
+| `catalog-backend-module-ai-resource-extensions`           | RHDH `spec.scope` + OCI source-location validation |
+
+## Examples
+
+See [`examples/ai-resource-agents.yaml`](../../examples/ai-resource-agents.yaml)
+for router + specialist agent catalog entities.
+
+## Public API
+
+| Export                                   | Description                                            |
+| ---------------------------------------- | ------------------------------------------------------ |
+| `catalogModuleAiResourceAgent` (default) | Backend module that registers the agent processor      |
+| `AiResourceAgentProcessor`               | `CatalogProcessor` for agent-specific field validation |

package/dist/AiResourceAgentProcessor.cjs.js

@@ -0,0 +1,31 @@
+'use strict';
+
+var backstagePluginCatalogModelAiResourceAgent = require('@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent');
+var collectAgentErrors = require('./collectAgentErrors.cjs.js');
+
+class AiResourceAgentProcessor {
+  getProcessorName() {
+    return "AiResourceAgentProcessor";
+  }
+  async validateEntityKind(entity) {
+    if (entity.kind !== "AiResource" || entity.spec?.type !== "agent") {
+      return false;
+    }
+    return backstagePluginCatalogModelAiResourceAgent.agentAiResourceEntityV1alpha1Validator.check(entity);
+  }
+  async preProcessEntity(entity, _location, _emit) {
+    if (entity.kind !== "AiResource") {
+      return entity;
+    }
+    const errors = collectAgentErrors.collectAgentErrors(entity);
+    if (errors.length > 0) {
+      throw new Error(
+        `Validation failed for AiResource agent entity: ${errors.join("; ")}`
+      );
+    }
+    return entity;
+  }
+}
+
+exports.AiResourceAgentProcessor = AiResourceAgentProcessor;
+//# sourceMappingURL=AiResourceAgentProcessor.cjs.js.map

package/dist/AiResourceAgentProcessor.cjs.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"AiResourceAgentProcessor.cjs.js","sources":["../src/AiResourceAgentProcessor.ts"],"sourcesContent":["/*\n * Copyright Red Hat, Inc.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport {\n  CatalogProcessor,\n  CatalogProcessorEmit,\n} from '@backstage/plugin-catalog-node';\nimport { Entity } from '@backstage/catalog-model';\nimport { LocationSpec } from '@backstage/plugin-catalog-common';\nimport { agentAiResourceEntityV1alpha1Validator } from '@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent';\nimport { collectAgentErrors } from './collectAgentErrors';\n\n/**\n * A CatalogProcessor that validates agent-specific fields on\n * AiResource entities with `spec.type: 'agent'`.\n *\n * Implements `validateEntityKind` so the catalog's built-in kind\n * processor remains active for standard kinds. The previous\n * `addModelSource` approach caused the built-in processor to be\n * replaced entirely, breaking validation of User, Group, Component,\n * and other standard entity kinds.\n *\n * Also validates via `preProcessEntity`:\n * - `spec.instructions`: optional; must be a string if present\n * - `spec.handoffs` / `spec.tools`: must be arrays if present\n * - `spec.resetToolChoice`: must be boolean if present\n * - `spec.modelSettings`: must be plain object if present\n * - `spec.toolUseBehavior`: must be string or string array if present\n * - `spec.outputSchema`: must be string or object if present\n * - `spec.handoffDescription` / `spec.model`: must be strings if present\n *\n * Non-agent AiResource entities (skill, rule, model) are unaffected.\n * All constraint violations are collected and reported in a single\n * error rather than stopping at the first failure.\n *\n * @public\n */\nexport class AiResourceAgentProcessor implements CatalogProcessor {\n  getProcessorName(): string {\n    return 'AiResourceAgentProcessor';\n  }\n\n  async validateEntityKind(entity: Entity): Promise<boolean> {\n    if (entity.kind !== 'AiResource' || entity.spec?.type !== 'agent') {\n      return false;\n    }\n    return agentAiResourceEntityV1alpha1Validator.check(entity);\n  }\n\n  async preProcessEntity(\n    entity: Entity,\n    _location: LocationSpec,\n    _emit: CatalogProcessorEmit,\n  ): Promise<Entity> {\n    if (entity.kind !== 'AiResource') {\n      return entity;\n    }\n\n    const errors = collectAgentErrors(entity);\n\n    if (errors.length > 0) {\n      throw new Error(\n        `Validation failed for AiResource agent entity: ${errors.join('; ')}`,\n      );\n    }\n\n    return entity;\n  }\n}\n"],"names":["agentAiResourceEntityV1alpha1Validator","collectAgentErrors"],"mappings":";;;;;AAkDO,MAAM,wBAAA,CAAqD;AAAA,EAChE,gBAAA,GAA2B;AACzB,IAAA,OAAO,0BAAA;AAAA,EACT;AAAA,EAEA,MAAM,mBAAmB,MAAA,EAAkC;AACzD,IAAA,IAAI,OAAO,IAAA,KAAS,YAAA,IAAgB,MAAA,CAAO,IAAA,EAAM,SAAS,OAAA,EAAS;AACjE,MAAA,OAAO,KAAA;AAAA,IACT;AACA,IAAA,OAAOA,iFAAA,CAAuC,MAAM,MAAM,CAAA;AAAA,EAC5D;AAAA,EAEA,MAAM,gBAAA,CACJ,MAAA,EACA,SAAA,EACA,KAAA,EACiB;AACjB,IAAA,IAAI,MAAA,CAAO,SAAS,YAAA,EAAc;AAChC,MAAA,OAAO,MAAA;AAAA,IACT;AAEA,IAAA,MAAM,MAAA,GAASC,sCAAmB,MAAM,CAAA;AAExC,IAAA,IAAI,MAAA,CAAO,SAAS,CAAA,EAAG;AACrB,MAAA,MAAM,IAAI,KAAA;AAAA,QACR,CAAA,+CAAA,EAAkD,MAAA,CAAO,IAAA,CAAK,IAAI,CAAC,CAAA;AAAA,OACrE;AAAA,IACF;AAEA,IAAA,OAAO,MAAA;AAAA,EACT;AACF;;;;"}

package/dist/collectAgentErrors.cjs.js

@@ -0,0 +1,81 @@
+'use strict';
+
+function collectAgentErrors(entity) {
+  const spec = entity.spec;
+  if (spec?.type !== "agent") {
+    return [];
+  }
+  const errors = [];
+  if (spec.instructions !== void 0 && typeof spec.instructions !== "string") {
+    errors.push(
+      `spec.instructions must be a string; got ${typeLabel(spec.instructions)}`
+    );
+  }
+  if (spec.handoffs !== void 0 && !Array.isArray(spec.handoffs)) {
+    errors.push(
+      `spec.handoffs must be an array; got ${typeLabel(spec.handoffs)}`
+    );
+  }
+  if (spec.tools !== void 0 && !Array.isArray(spec.tools)) {
+    errors.push(`spec.tools must be an array; got ${typeLabel(spec.tools)}`);
+  }
+  if (spec.resetToolChoice !== void 0 && typeof spec.resetToolChoice !== "boolean") {
+    errors.push(
+      `spec.resetToolChoice must be a boolean; got ${typeLabel(
+        spec.resetToolChoice
+      )}`
+    );
+  }
+  if (spec.modelSettings !== void 0) {
+    if (typeof spec.modelSettings !== "object" || spec.modelSettings === null || Array.isArray(spec.modelSettings)) {
+      errors.push(
+        `spec.modelSettings must be an object; got ${typeLabel(
+          spec.modelSettings
+        )}`
+      );
+    }
+  }
+  if (spec.toolUseBehavior !== void 0) {
+    if (typeof spec.toolUseBehavior !== "string" && !Array.isArray(spec.toolUseBehavior)) {
+      errors.push(
+        `spec.toolUseBehavior must be a string or an array; got ${typeLabel(
+          spec.toolUseBehavior
+        )}`
+      );
+    }
+  }
+  if (spec.outputSchema !== void 0) {
+    const isString = typeof spec.outputSchema === "string";
+    const isObject = typeof spec.outputSchema === "object" && spec.outputSchema !== null && !Array.isArray(spec.outputSchema);
+    if (!isString && !isObject) {
+      errors.push(
+        `spec.outputSchema must be a string or an object; got ${typeLabel(
+          spec.outputSchema
+        )}`
+      );
+    }
+  }
+  if (spec.handoffDescription !== void 0 && typeof spec.handoffDescription !== "string") {
+    errors.push(
+      `spec.handoffDescription must be a string; got ${typeLabel(
+        spec.handoffDescription
+      )}`
+    );
+  }
+  if (spec.model !== void 0 && typeof spec.model !== "string") {
+    errors.push(`spec.model must be a string; got ${typeLabel(spec.model)}`);
+  }
+  return errors;
+}
+function typeLabel(value) {
+  if (value === null) {
+    return "null";
+  }
+  if (Array.isArray(value)) {
+    return "array";
+  }
+  return typeof value;
+}
+
+exports.collectAgentErrors = collectAgentErrors;
+//# sourceMappingURL=collectAgentErrors.cjs.js.map

package/dist/collectAgentErrors.cjs.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"collectAgentErrors.cjs.js","sources":["../src/collectAgentErrors.ts"],"sourcesContent":["/*\n * Copyright Red Hat, Inc.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport { Entity } from '@backstage/catalog-model';\n\n/**\n * Collects agent-specific validation errors for an AiResource entity\n * with `spec.type: 'agent'` without throwing. Returns an array of\n * error messages (empty if valid or if `spec.type` is not `'agent'`).\n *\n * Validates agent-specific fields only; does NOT re-validate core\n * entity fields such as `spec.owner` or `spec.lifecycle`.\n *\n * @internal\n */\nexport function collectAgentErrors(entity: Entity): string[] {\n  const spec = entity.spec;\n  if (spec?.type !== 'agent') {\n    return [];\n  }\n\n  const errors: string[] = [];\n\n  // Optional: spec.instructions — must be a string if present.\n  // Omitted when the agent image/runtime already bakes in a default prompt.\n  if (\n    spec.instructions !== undefined &&\n    typeof spec.instructions !== 'string'\n  ) {\n    errors.push(\n      `spec.instructions must be a string; got ${typeLabel(spec.instructions)}`,\n    );\n  }\n\n  // Optional: spec.handoffs — must be an array if present\n  if (spec.handoffs !== undefined && !Array.isArray(spec.handoffs)) {\n    errors.push(\n      `spec.handoffs must be an array; got ${typeLabel(spec.handoffs)}`,\n    );\n  }\n\n  // Optional: spec.tools — must be an array if present\n  if (spec.tools !== undefined && !Array.isArray(spec.tools)) {\n    errors.push(`spec.tools must be an array; got ${typeLabel(spec.tools)}`);\n  }\n\n  // Optional: spec.resetToolChoice — must be a boolean if present\n  if (\n    spec.resetToolChoice !== undefined &&\n    typeof spec.resetToolChoice !== 'boolean'\n  ) {\n    errors.push(\n      `spec.resetToolChoice must be a boolean; got ${typeLabel(\n        spec.resetToolChoice,\n      )}`,\n    );\n  }\n\n  // Optional: spec.modelSettings — must be a plain object if present\n  if (spec.modelSettings !== undefined) {\n    if (\n      typeof spec.modelSettings !== 'object' ||\n      spec.modelSettings === null ||\n      Array.isArray(spec.modelSettings)\n    ) {\n      errors.push(\n        `spec.modelSettings must be an object; got ${typeLabel(\n          spec.modelSettings,\n        )}`,\n      );\n    }\n  }\n\n  // Optional: spec.toolUseBehavior — must be a string or string array\n  if (spec.toolUseBehavior !== undefined) {\n    if (\n      typeof spec.toolUseBehavior !== 'string' &&\n      !Array.isArray(spec.toolUseBehavior)\n    ) {\n      errors.push(\n        `spec.toolUseBehavior must be a string or an array; got ${typeLabel(\n          spec.toolUseBehavior,\n        )}`,\n      );\n    }\n  }\n\n  // Optional: spec.outputSchema — must be a string or plain object\n  if (spec.outputSchema !== undefined) {\n    const isString = typeof spec.outputSchema === 'string';\n    const isObject =\n      typeof spec.outputSchema === 'object' &&\n      spec.outputSchema !== null &&\n      !Array.isArray(spec.outputSchema);\n    if (!isString && !isObject) {\n      errors.push(\n        `spec.outputSchema must be a string or an object; got ${typeLabel(\n          spec.outputSchema,\n        )}`,\n      );\n    }\n  }\n\n  // Optional: spec.handoffDescription — must be a string if present\n  if (\n    spec.handoffDescription !== undefined &&\n    typeof spec.handoffDescription !== 'string'\n  ) {\n    errors.push(\n      `spec.handoffDescription must be a string; got ${typeLabel(\n        spec.handoffDescription,\n      )}`,\n    );\n  }\n\n  // Optional: spec.model — must be a string if present\n  if (spec.model !== undefined && typeof spec.model !== 'string') {\n    errors.push(`spec.model must be a string; got ${typeLabel(spec.model)}`);\n  }\n\n  return errors;\n}\n\n/**\n * Returns a human-friendly type label for error messages.\n * Distinguishes arrays and null from plain \"object\".\n */\nfunction typeLabel(value: unknown): string {\n  if (value === null) {\n    return 'null';\n  }\n  if (Array.isArray(value)) {\n    return 'array';\n  }\n  return typeof value;\n}\n"],"names":[],"mappings":";;AA4BO,SAAS,mBAAmB,MAAA,EAA0B;AAC3D,EAAA,MAAM,OAAO,MAAA,CAAO,IAAA;AACpB,EAAA,IAAI,IAAA,EAAM,SAAS,OAAA,EAAS;AAC1B,IAAA,OAAO,EAAC;AAAA,EACV;AAEA,EAAA,MAAM,SAAmB,EAAC;AAI1B,EAAA,IACE,KAAK,YAAA,KAAiB,MAAA,IACtB,OAAO,IAAA,CAAK,iBAAiB,QAAA,EAC7B;AACA,IAAA,MAAA,CAAO,IAAA;AAAA,MACL,CAAA,wCAAA,EAA2C,SAAA,CAAU,IAAA,CAAK,YAAY,CAAC,CAAA;AAAA,KACzE;AAAA,EACF;AAGA,EAAA,IAAI,IAAA,CAAK,aAAa,MAAA,IAAa,CAAC,MAAM,OAAA,CAAQ,IAAA,CAAK,QAAQ,CAAA,EAAG;AAChE,IAAA,MAAA,CAAO,IAAA;AAAA,MACL,CAAA,oCAAA,EAAuC,SAAA,CAAU,IAAA,CAAK,QAAQ,CAAC,CAAA;AAAA,KACjE;AAAA,EACF;AAGA,EAAA,IAAI,IAAA,CAAK,UAAU,MAAA,IAAa,CAAC,MAAM,OAAA,CAAQ,IAAA,CAAK,KAAK,CAAA,EAAG;AAC1D,IAAA,MAAA,CAAO,KAAK,CAAA,iCAAA,EAAoC,SAAA,CAAU,IAAA,CAAK,KAAK,CAAC,CAAA,CAAE,CAAA;AAAA,EACzE;AAGA,EAAA,IACE,KAAK,eAAA,KAAoB,MAAA,IACzB,OAAO,IAAA,CAAK,oBAAoB,SAAA,EAChC;AACA,IAAA,MAAA,CAAO,IAAA;AAAA,MACL,CAAA,4CAAA,EAA+C,SAAA;AAAA,QAC7C,IAAA,CAAK;AAAA,OACN,CAAA;AAAA,KACH;AAAA,EACF;AAGA,EAAA,IAAI,IAAA,CAAK,kBAAkB,MAAA,EAAW;AACpC,IAAA,IACE,OAAO,IAAA,CAAK,aAAA,KAAkB,QAAA,IAC9B,IAAA,CAAK,aAAA,KAAkB,IAAA,IACvB,KAAA,CAAM,OAAA,CAAQ,IAAA,CAAK,aAAa,CAAA,EAChC;AACA,MAAA,MAAA,CAAO,IAAA;AAAA,QACL,CAAA,0CAAA,EAA6C,SAAA;AAAA,UAC3C,IAAA,CAAK;AAAA,SACN,CAAA;AAAA,OACH;AAAA,IACF;AAAA,EACF;AAGA,EAAA,IAAI,IAAA,CAAK,oBAAoB,MAAA,EAAW;AACtC,IAAA,IACE,OAAO,KAAK,eAAA,KAAoB,QAAA,IAChC,CAAC,KAAA,CAAM,OAAA,CAAQ,IAAA,CAAK,eAAe,CAAA,EACnC;AACA,MAAA,MAAA,CAAO,IAAA;AAAA,QACL,CAAA,uDAAA,EAA0D,SAAA;AAAA,UACxD,IAAA,CAAK;AAAA,SACN,CAAA;AAAA,OACH;AAAA,IACF;AAAA,EACF;AAGA,EAAA,IAAI,IAAA,CAAK,iBAAiB,MAAA,EAAW;AACnC,IAAA,MAAM,QAAA,GAAW,OAAO,IAAA,CAAK,YAAA,KAAiB,QAAA;AAC9C,IAAA,MAAM,QAAA,GACJ,OAAO,IAAA,CAAK,YAAA,KAAiB,QAAA,IAC7B,IAAA,CAAK,YAAA,KAAiB,IAAA,IACtB,CAAC,KAAA,CAAM,OAAA,CAAQ,IAAA,CAAK,YAAY,CAAA;AAClC,IAAA,IAAI,CAAC,QAAA,IAAY,CAAC,QAAA,EAAU;AAC1B,MAAA,MAAA,CAAO,IAAA;AAAA,QACL,CAAA,qDAAA,EAAwD,SAAA;AAAA,UACtD,IAAA,CAAK;AAAA,SACN,CAAA;AAAA,OACH;AAAA,IACF;AAAA,EACF;AAGA,EAAA,IACE,KAAK,kBAAA,KAAuB,MAAA,IAC5B,OAAO,IAAA,CAAK,uBAAuB,QAAA,EACnC;AACA,IAAA,MAAA,CAAO,IAAA;AAAA,MACL,CAAA,8CAAA,EAAiD,SAAA;AAAA,QAC/C,IAAA,CAAK;AAAA,OACN,CAAA;AAAA,KACH;AAAA,EACF;AAGA,EAAA,IAAI,KAAK,KAAA,KAAU,MAAA,IAAa,OAAO,IAAA,CAAK,UAAU,QAAA,EAAU;AAC9D,IAAA,MAAA,CAAO,KAAK,CAAA,iCAAA,EAAoC,SAAA,CAAU,IAAA,CAAK,KAAK,CAAC,CAAA,CAAE,CAAA;AAAA,EACzE;AAEA,EAAA,OAAO,MAAA;AACT;AAMA,SAAS,UAAU,KAAA,EAAwB;AACzC,EAAA,IAAI,UAAU,IAAA,EAAM;AAClB,IAAA,OAAO,MAAA;AAAA,EACT;AACA,EAAA,IAAI,KAAA,CAAM,OAAA,CAAQ,KAAK,CAAA,EAAG;AACxB,IAAA,OAAO,OAAA;AAAA,EACT;AACA,EAAA,OAAO,OAAO,KAAA;AAChB;;;;"}

package/dist/index.cjs.js

@@ -3,8 +3,10 @@
 Object.defineProperty(exports, '__esModule', { value: true });
 
 var module$1 = require('./module.cjs.js');
+var AiResourceAgentProcessor = require('./AiResourceAgentProcessor.cjs.js');
 
 
 
 exports.default = module$1.catalogModuleAiResourceAgent;
+exports.AiResourceAgentProcessor = AiResourceAgentProcessor.AiResourceAgentProcessor;
 //# sourceMappingURL=index.cjs.js.map

package/dist/index.cjs.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.cjs.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;;;;;"}
+{"version":3,"file":"index.cjs.js","sources":[],"sourcesContent":[],"names":[],"mappings":";;;;;;;;;;"}

package/dist/index.d.ts

@@ -1,14 +1,50 @@
 import * as _backstage_backend_plugin_api from '@backstage/backend-plugin-api';
+import { CatalogProcessor, CatalogProcessorEmit } from '@backstage/plugin-catalog-node';
+import { Entity } from '@backstage/catalog-model';
+import { LocationSpec } from '@backstage/plugin-catalog-common';
 
 /**
  * Registers the agent specType for the AiResource kind in the catalog.
  *
- * Follows the upstream pattern established by
- * `@backstage/plugin-catalog-backend-module-ai-model` which registers
- * skill/rule via `catalogModelExtensionPoint.addModelSource`.
+ * Uses `addProcessor` so the catalog's built-in kind processor
+ * (`BuiltinKindsEntityProcessor`) remains active for standard kinds.
+ * The previous `addModelSource` approach caused the built-in processor
+ * to be replaced entirely, breaking validation of User, Group,
+ * Component, and other standard entity kinds.
  *
  * @public
  */
 declare const catalogModuleAiResourceAgent: _backstage_backend_plugin_api.BackendFeature;
 
-export { catalogModuleAiResourceAgent as default };
+/**
+ * A CatalogProcessor that validates agent-specific fields on
+ * AiResource entities with `spec.type: 'agent'`.
+ *
+ * Implements `validateEntityKind` so the catalog's built-in kind
+ * processor remains active for standard kinds. The previous
+ * `addModelSource` approach caused the built-in processor to be
+ * replaced entirely, breaking validation of User, Group, Component,
+ * and other standard entity kinds.
+ *
+ * Also validates via `preProcessEntity`:
+ * - `spec.instructions`: optional; must be a string if present
+ * - `spec.handoffs` / `spec.tools`: must be arrays if present
+ * - `spec.resetToolChoice`: must be boolean if present
+ * - `spec.modelSettings`: must be plain object if present
+ * - `spec.toolUseBehavior`: must be string or string array if present
+ * - `spec.outputSchema`: must be string or object if present
+ * - `spec.handoffDescription` / `spec.model`: must be strings if present
+ *
+ * Non-agent AiResource entities (skill, rule, model) are unaffected.
+ * All constraint violations are collected and reported in a single
+ * error rather than stopping at the first failure.
+ *
+ * @public
+ */
+declare class AiResourceAgentProcessor implements CatalogProcessor {
+    getProcessorName(): string;
+    validateEntityKind(entity: Entity): Promise<boolean>;
+    preProcessEntity(entity: Entity, _location: LocationSpec, _emit: CatalogProcessorEmit): Promise<Entity>;
+}
+
+export { AiResourceAgentProcessor, catalogModuleAiResourceAgent as default };

package/dist/module.cjs.js

@@ -1,8 +1,8 @@
 'use strict';
 
 var backendPluginApi = require('@backstage/backend-plugin-api');
-var alpha = require('@backstage/plugin-catalog-node/alpha');
-var backstagePluginCatalogModelAiResourceAgent = require('@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent');
+var pluginCatalogNode = require('@backstage/plugin-catalog-node');
+var AiResourceAgentProcessor = require('./AiResourceAgentProcessor.cjs.js');
 
 const catalogModuleAiResourceAgent = backendPluginApi.createBackendModule({
   pluginId: "catalog",
@@ -10,15 +10,10 @@ const catalogModuleAiResourceAgent = backendPluginApi.createBackendModule({
   register(reg) {
     reg.registerInit({
       deps: {
-        model: alpha.catalogModelExtensionPoint
+        catalog: pluginCatalogNode.catalogProcessingExtensionPoint
       },
-      async init({ model }) {
-        const source = {
-          async *read() {
-            yield { data: [{ layer: backstagePluginCatalogModelAiResourceAgent.agentAiResourceEntityModel }] };
-          }
-        };
-        model.addModelSource(source);
+      async init({ catalog }) {
+        catalog.addProcessor(new AiResourceAgentProcessor.AiResourceAgentProcessor());
       }
     });
   }

package/dist/module.cjs.js.map

@@ -1 +1 @@
-{"version":3,"file":"module.cjs.js","sources":["../src/module.ts"],"sourcesContent":["/*\n * Copyright Red Hat, Inc.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport { createBackendModule } from '@backstage/backend-plugin-api';\nimport type { CatalogModelSource } from '@backstage/catalog-model/alpha';\nimport { catalogModelExtensionPoint } from '@backstage/plugin-catalog-node/alpha';\nimport { agentAiResourceEntityModel } from '@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent';\n\n/**\n * Registers the agent specType for the AiResource kind in the catalog.\n *\n * Follows the upstream pattern established by\n * `@backstage/plugin-catalog-backend-module-ai-model` which registers\n * skill/rule via `catalogModelExtensionPoint.addModelSource`.\n *\n * @public\n */\nexport const catalogModuleAiResourceAgent = createBackendModule({\n  pluginId: 'catalog',\n  moduleId: 'ai-resource-agent',\n  register(reg) {\n    reg.registerInit({\n      deps: {\n        model: catalogModelExtensionPoint,\n      },\n      async init({ model }) {\n        const source: CatalogModelSource = {\n          async *read() {\n            yield { data: [{ layer: agentAiResourceEntityModel }] };\n          },\n        };\n        model.addModelSource(source);\n      },\n    });\n  },\n});\n"],"names":["createBackendModule","catalogModelExtensionPoint","agentAiResourceEntityModel"],"mappings":";;;;;;AA8BO,MAAM,+BAA+BA,oCAAA,CAAoB;AAAA,EAC9D,QAAA,EAAU,SAAA;AAAA,EACV,QAAA,EAAU,mBAAA;AAAA,EACV,SAAS,GAAA,EAAK;AACZ,IAAA,GAAA,CAAI,YAAA,CAAa;AAAA,MACf,IAAA,EAAM;AAAA,QACJ,KAAA,EAAOC;AAAA,OACT;AAAA,MACA,MAAM,IAAA,CAAK,EAAE,KAAA,EAAM,EAAG;AACpB,QAAA,MAAM,MAAA,GAA6B;AAAA,UACjC,OAAO,IAAA,GAAO;AACZ,YAAA,MAAM,EAAE,IAAA,EAAM,CAAC,EAAE,KAAA,EAAOC,qEAAA,EAA4B,CAAA,EAAE;AAAA,UACxD;AAAA,SACF;AACA,QAAA,KAAA,CAAM,eAAe,MAAM,CAAA;AAAA,MAC7B;AAAA,KACD,CAAA;AAAA,EACH;AACF,CAAC;;;;"}
+{"version":3,"file":"module.cjs.js","sources":["../src/module.ts"],"sourcesContent":["/*\n * Copyright Red Hat, Inc.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport { createBackendModule } from '@backstage/backend-plugin-api';\nimport { catalogProcessingExtensionPoint } from '@backstage/plugin-catalog-node';\n\nimport { AiResourceAgentProcessor } from './AiResourceAgentProcessor';\n\n/**\n * Registers the agent specType for the AiResource kind in the catalog.\n *\n * Uses `addProcessor` so the catalog's built-in kind processor\n * (`BuiltinKindsEntityProcessor`) remains active for standard kinds.\n * The previous `addModelSource` approach caused the built-in processor\n * to be replaced entirely, breaking validation of User, Group,\n * Component, and other standard entity kinds.\n *\n * @public\n */\nexport const catalogModuleAiResourceAgent = createBackendModule({\n  pluginId: 'catalog',\n  moduleId: 'ai-resource-agent',\n  register(reg) {\n    reg.registerInit({\n      deps: {\n        catalog: catalogProcessingExtensionPoint,\n      },\n      async init({ catalog }) {\n        catalog.addProcessor(new AiResourceAgentProcessor());\n      },\n    });\n  },\n});\n"],"names":["createBackendModule","catalogProcessingExtensionPoint","AiResourceAgentProcessor"],"mappings":";;;;;;AAgCO,MAAM,+BAA+BA,oCAAA,CAAoB;AAAA,EAC9D,QAAA,EAAU,SAAA;AAAA,EACV,QAAA,EAAU,mBAAA;AAAA,EACV,SAAS,GAAA,EAAK;AACZ,IAAA,GAAA,CAAI,YAAA,CAAa;AAAA,MACf,IAAA,EAAM;AAAA,QACJ,OAAA,EAASC;AAAA,OACX;AAAA,MACA,MAAM,IAAA,CAAK,EAAE,OAAA,EAAQ,EAAG;AACtB,QAAA,OAAA,CAAQ,YAAA,CAAa,IAAIC,iDAAA,EAA0B,CAAA;AAAA,MACrD;AAAA,KACD,CAAA;AAAA,EACH;AACF,CAAC;;;;"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@red-hat-developer-hub/backstage-plugin-catalog-backend-module-ai-resource-agent",
-  "version": "0.2.1",
+  "version": "0.4.0",
   "license": "Apache-2.0",
   "description": "The ai-resource-agent backend module for the catalog plugin. Registers the agent specType for AiResource entities.",
   "main": "dist/index.cjs.js",
@@ -35,10 +35,12 @@
   "dependencies": {
     "@backstage/backend-plugin-api": "^1.9.2",
     "@backstage/catalog-model": "^1.9.0",
+    "@backstage/plugin-catalog-common": "^1.1.10",
     "@backstage/plugin-catalog-node": "^2.2.2",
-    "@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent": "^0.2.0"
+    "@red-hat-developer-hub/backstage-plugin-catalog-model-ai-resource-agent": "^0.3.0"
   },
   "devDependencies": {
+    "@backstage/backend-test-utils": "^1.11.4",
     "@backstage/cli": "^0.36.3"
   },
   "files": [