@bryan-thompson/inspector-assessment-cli 1.29.0 → 1.29.1

package/build/__tests__/assessment-runner/config-builder.test.js

@@ -12,6 +12,7 @@ jest.unstable_mockModule("../../profiles.js", () => ({
 }));
 jest.unstable_mockModule("../../../../client/lib/lib/assessmentTypes.js", () => ({
     DEFAULT_ASSESSMENT_CONFIG: {
+        configVersion: 2,
         enableExtendedAssessment: false,
         parallelTesting: false,
         testTimeout: 10000,
@@ -286,4 +287,52 @@ describe("buildConfig", () => {
             expect(result.logging?.level).toBe("error");
         });
     });
+    describe("config version validation (Issue #107)", () => {
+        let consoleWarnSpy;
+        beforeEach(() => {
+            consoleWarnSpy = jest
+                .spyOn(console, "warn")
+                .mockImplementation(() => { });
+        });
+        afterEach(() => {
+            consoleWarnSpy.mockRestore();
+        });
+        it("should not warn when configVersion is present in defaults", () => {
+            // DEFAULT_ASSESSMENT_CONFIG mock includes configVersion: 2
+            buildConfig({ serverName: "test" });
+            // Should NOT warn because configVersion is set
+            expect(consoleWarnSpy).not.toHaveBeenCalledWith(expect.stringContaining("Config missing configVersion"));
+        });
+        it("should still build config successfully with configVersion", () => {
+            const result = buildConfig({ serverName: "test" });
+            expect(result).toBeDefined();
+            expect(result.configVersion).toBe(2);
+            expect(result.testTimeout).toBeDefined();
+            expect(result.logging).toBeDefined();
+        });
+        it("should include configVersion in final config", () => {
+            const result = buildConfig({ serverName: "test" });
+            expect(result.configVersion).toBe(2);
+        });
+        it("should preserve configVersion through profile-based config", () => {
+            modulesToLegacyConfig.mockReturnValue({
+                functionality: true,
+                security: true,
+            });
+            const result = buildConfig({ serverName: "test", profile: "quick" });
+            expect(result.configVersion).toBe(2);
+        });
+        it("should preserve configVersion through module filtering", () => {
+            getAllModulesConfig.mockReturnValue({
+                functionality: true,
+                security: true,
+            });
+            resolveModuleNames.mockReturnValue(["functionality"]);
+            const result = buildConfig({
+                serverName: "test",
+                onlyModules: ["functionality"],
+            });
+            expect(result.configVersion).toBe(2);
+        });
+    });
 });

package/build/__tests__/jsonl-events.test.js

@@ -33,6 +33,10 @@ describe("JSONL Event Emission", () => {
             emitJSONL({ event: "test" });
             expect(emittedEvents[0]).toHaveProperty("version");
         });
+        it("should include schemaVersion field", () => {
+            emitJSONL({ event: "test" });
+            expect(emittedEvents[0]).toHaveProperty("schemaVersion", 1);
+        });
         it("should handle complex nested objects", () => {
             emitJSONL({
                 event: "complex",

package/build/lib/assessment-runner/config-builder.js

@@ -123,5 +123,12 @@ export function buildConfig(options) {
     const envLogLevel = process.env.LOG_LEVEL;
     const logLevel = options.logLevel ?? envLogLevel ?? "info";
     config.logging = { level: logLevel };
+    // Config version validation (Issue #107)
+    // Warn if config is missing version field - will be required in v2.0.0
+    if (!config.configVersion) {
+        console.warn("⚠️  Config missing configVersion field. " +
+            "This will be required in v2.0.0. " +
+            "See docs/DEPRECATION_GUIDE.md for migration info.");
+    }
     return config;
 }

package/build/lib/jsonl-events.js

@@ -7,16 +7,22 @@
  * This is a CLI-local version that imports from the built client lib
  * to avoid rootDir conflicts in TypeScript compilation.
  */
-import { INSPECTOR_VERSION } from "../../../client/lib/lib/moduleScoring.js";
+import { INSPECTOR_VERSION, SCHEMA_VERSION, } from "../../../client/lib/lib/moduleScoring.js";
+// Re-export for consumers of this module
+export { SCHEMA_VERSION };
 // ============================================================================
 // Core Functions
 // ============================================================================
 /**
  * Emit a JSONL event to stderr for real-time machine parsing.
- * Automatically includes version field for compatibility checking.
+ * Automatically includes version and schemaVersion fields for compatibility checking.
  */
 export function emitJSONL(event) {
-    console.error(JSON.stringify({ ...event, version: INSPECTOR_VERSION }));
+    console.error(JSON.stringify({
+        ...event,
+        version: INSPECTOR_VERSION,
+        schemaVersion: SCHEMA_VERSION,
+    }));
 }
 /**
  * Emit server_connected event after successful connection.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bryan-thompson/inspector-assessment-cli",
-  "version": "1.29.0",
+  "version": "1.29.1",
   "description": "CLI for the Enhanced MCP Inspector with assessment capabilities",
   "license": "MIT",
   "author": "Bryan Thompson <bryan@triepod.ai>",