Address PR review comments

- SKILL.md: Instruct agents to check validation results before importing
  and prompt the user if validation fails (promeris feedback)
- Fix "500 on import" to "Errors on import (400 / 500)" in skill and
  guide (promeris feedback)
- Add module-level tests for asset-registry handlers to improve coverage

Includes-AI-Code: true
Made-with: Cursor

Author: ZgjimHaziri

.cursor/skills/asset-registry-endpoints/SKILL.md

@@ -232,6 +232,12 @@ importing:
 $CLI config import -d <export_dir> --validate --overwrite -p <profile>
 ```
 
+**Important**: If validation returns errors, do **not** proceed with the import.
+Instead, fix the schema violations in the node JSON and re-run the command. If
+you cannot resolve the errors automatically, present the validation results to
+the user and ask whether they want to continue importing with invalid
+configuration or stop to fix it manually.
+
 ## Troubleshooting
 
 **404 on examples / methodology** — Not all asset services have deployed these
@@ -242,8 +248,8 @@ are optional.
 service. A 500 typically means the downstream service is unavailable or returned
 an unexpected response.
 
-**500 on import** — Ensure `spaceId` is set on every node and `schemaVersion`
-matches the descriptor's `assetSchema.version`.
+**Errors on import (400 / 500)** — Ensure `spaceId` is set on every node and
+`schemaVersion` matches the descriptor's `assetSchema.version`.
 
 ## Full worked example
 

docs/user-guide/agentic-development-guide.md

@@ -95,7 +95,7 @@ content-cli config export --keysByVersion <packageKey>_<version> --unzip
 
 **500 on proxy endpoints** — The platform proxies requests to the owning asset service. A 500 typically means the downstream service is unavailable or returned an unexpected response.
 
-**500 on import** — Ensure `spaceId` is set on every node and `schemaVersion` matches the descriptor's `assetSchema.version`.
+**Errors on import (400 / 500)** — Ensure `spaceId` is set on every node and `schemaVersion` matches the descriptor's `assetSchema.version`.
 
 ## Further reading
 

tests/commands/asset-registry/asset-registry-module.spec.ts

@@ -0,0 +1,59 @@
+import Module = require("../../../src/commands/asset-registry/module");
+import { Command, OptionValues } from "commander";
+import { AssetRegistryService } from "../../../src/commands/asset-registry/asset-registry.service";
+import { testContext } from "../../utls/test-context";
+
+jest.mock("../../../src/commands/asset-registry/asset-registry.service");
+
+describe("Asset Registry Module", () => {
+    let module: Module;
+    let mockCommand: Command;
+    let mockService: jest.Mocked<AssetRegistryService>;
+
+    beforeEach(() => {
+        jest.clearAllMocks();
+        module = new Module();
+        mockCommand = {} as Command;
+
+        mockService = {
+            listTypes: jest.fn().mockResolvedValue(undefined),
+            getType: jest.fn().mockResolvedValue(undefined),
+            getSchema: jest.fn().mockResolvedValue(undefined),
+            getExamples: jest.fn().mockResolvedValue(undefined),
+            getMethodology: jest.fn().mockResolvedValue(undefined),
+        } as any;
+
+        (AssetRegistryService as jest.MockedClass<typeof AssetRegistryService>)
+            .mockImplementation(() => mockService);
+    });
+
+    it("should call getSchema with correct parameters", async () => {
+        const options: OptionValues = { assetType: "BOARD_V2", json: true };
+        await (module as any).getSchema(testContext, mockCommand, options);
+        expect(mockService.getSchema).toHaveBeenCalledWith("BOARD_V2", true);
+    });
+
+    it("should call getExamples with correct parameters", async () => {
+        const options: OptionValues = { assetType: "BOARD_V2", json: "" };
+        await (module as any).getExamples(testContext, mockCommand, options);
+        expect(mockService.getExamples).toHaveBeenCalledWith("BOARD_V2", false);
+    });
+
+    it("should call getMethodology with correct parameters", async () => {
+        const options: OptionValues = { assetType: "SEMANTIC_MODEL" };
+        await (module as any).getMethodology(testContext, mockCommand, options);
+        expect(mockService.getMethodology).toHaveBeenCalledWith("SEMANTIC_MODEL", false);
+    });
+
+    it("should call listTypes", async () => {
+        const options: OptionValues = { json: true };
+        await (module as any).listTypes(testContext, mockCommand, options);
+        expect(mockService.listTypes).toHaveBeenCalledWith(true);
+    });
+
+    it("should call getType", async () => {
+        const options: OptionValues = { assetType: "BOARD_V2", json: "" };
+        await (module as any).getType(testContext, mockCommand, options);
+        expect(mockService.getType).toHaveBeenCalledWith("BOARD_V2", false);
+    });
+});