vcluster-yaml-mcp-server 1.0.7 → 1.0.9

package/README.md

@@ -1,5 +1,13 @@
 # vCluster YAML MCP Server
 
+[![npm version](https://img.shields.io/npm/v/vcluster-yaml-mcp-server.svg)](https://www.npmjs.com/package/vcluster-yaml-mcp-server)
+[![codecov](https://codecov.io/gh/Piotr1215/vcluster-yaml-mcp/branch/main/graph/badge.svg)](https://codecov.io/gh/Piotr1215/vcluster-yaml-mcp)
+[![Node.js](https://img.shields.io/badge/node-18%20%7C%2020%20%7C%2022-brightgreen)](https://nodejs.org/)
+[![Response Time](https://img.shields.io/badge/response-<100ms-brightgreen)](https://github.com/Piotr1215/vcluster-yaml-mcp-server#token-optimization)
+[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
+[![Docker Pulls](https://img.shields.io/docker/pulls/piotrzan/vcluster-yaml-mcp-server.svg)](https://hub.docker.com/r/piotrzan/vcluster-yaml-mcp-server)
+[![MCP](https://img.shields.io/badge/MCP-Server-blue.svg)](https://modelcontextprotocol.io)
+
 A Model Context Protocol (MCP) server that lets AI assistants query and validate [vCluster](https://github.com/loft-sh/vcluster) YAML configurations directly from GitHub.
 
 ## What Does It Do?
@@ -71,11 +79,11 @@ The package also provides a standalone CLI for quick queries and validation with
 
 ```bash
 # Quick start with npx (no installation)
-npx -p vcluster-yaml-mcp-server vcluster-yaml query sync --format table
+npx vcluster-yaml-mcp-server query sync
 
 # Or install globally
 npm install -g vcluster-yaml-mcp-server
-vcluster-yaml query sync --format table
+vcluster-yaml query sync
 
 # Validate configurations with ease
 vcluster-yaml validate my-config.yaml

package/package.json

@@ -1,10 +1,11 @@
 {
   "name": "vcluster-yaml-mcp-server",
-  "version": "1.0.7",
+  "version": "1.0.9",
   "description": "MCP server for querying vcluster YAML configurations using jq",
   "main": "src/index.js",
   "type": "module",
   "bin": {
+    "vcluster-yaml-mcp-server": "./src/cli.js",
     "vcluster-yaml-mcp": "./src/index.js",
     "vcluster-yaml": "./src/cli.js"
   },
@@ -17,11 +18,17 @@
     "start:http": "node src/http-server.js",
     "docker:build": "docker build -t piotrzan/vcluster-yaml-mcp-server:latest .",
     "docker:push": "docker push piotrzan/vcluster-yaml-mcp-server:latest",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "test:coverage": "vitest run --coverage",
+    "test": "vitest run --exclude tests/integration/**/*.test.js --exclude tests/ci/**/*.test.js --exclude tests/performance.test.js",
+    "test:unit": "vitest run --exclude tests/integration/**/*.test.js --exclude tests/ci/**/*.test.js --exclude tests/performance.test.js",
+    "test:integration": "vitest run tests/integration/",
     "test:perf": "vitest run tests/performance.test.js",
     "test:ci": "vitest run tests/ci/",
+    "test:all": "vitest run",
+    "test:watch": "vitest --exclude tests/integration/**/*.test.js",
+    "test:coverage": "vitest run --coverage --exclude tests/integration/**/*.test.js",
+    "test:mutation": "stryker run",
+    "complexity": "eslint --config eslint.config.complexity.js src/**/*.js",
+    "complexity:json": "eslint --config eslint.config.complexity.js --format json --output-file reports/complexity.json src/**/*.js || true",
     "lint:workflows": "actionlint .github/workflows/*.yml"
   },
   "dependencies": {
@@ -42,8 +49,12 @@
     "node": ">=18"
   },
   "devDependencies": {
+    "@stryker-mutator/core": "^9.2.0",
+    "@stryker-mutator/vitest-runner": "^9.2.0",
     "@vitest/coverage-v8": "^3.2.4",
+    "eslint": "^9.38.0",
     "execa": "^9.5.2",
+    "fast-check": "^4.3.0",
     "json-schema-library": "^10.3.0",
     "jsonschema": "^1.5.0",
     "strip-ansi": "^7.1.0",

package/src/cli.js

@@ -35,12 +35,12 @@ program
   .description('Search for vCluster configuration fields')
   .option('--file <file>', 'Configuration file to search', 'chart/values.yaml')
   .option('-s, --schema-version <version>', 'vCluster version or branch', 'main')
-  .option('-f, --format <format>', 'Output format (json, yaml, table)', 'json')
+  .option('-f, --format <format>', 'Output format: json, yaml, table (default: table)', 'table')
   .addHelpText('after', `
 Examples:
   $ vcluster-yaml query sync
   $ vcluster-yaml query sync --schema-version v0.24.0
-  $ vcluster-yaml query "controlPlane.replicas" --format table
+  $ vcluster-yaml query "controlPlane.replicas"
   `)
   .action(async (query, options) => {
     try {
@@ -84,7 +84,7 @@ Examples:
 program
   .command('list-versions')
   .description('List available vCluster versions')
-  .option('-f, --format <format>', 'Output format (json, yaml, table)', 'json')
+  .option('-f, --format <format>', 'Output format: json, yaml, table (default: table)', 'table')
   .action(async (options) => {
     try {
       // Validate format option
@@ -118,7 +118,7 @@ program
   .command('validate [file]')
   .description('Validate vCluster configuration')
   .option('-s, --schema-version <version>', 'vCluster version for schema', 'main')
-  .option('-f, --format <format>', 'Output format (json, yaml, table)', 'json')
+  .option('-f, --format <format>', 'Output format: json, yaml, table (default: table)', 'table')
   .addHelpText('after', `
 Arguments:
   file                   YAML file to validate (use '-' for stdin, omit to read from stdin)
@@ -128,7 +128,6 @@ Examples:
   $ vcluster-yaml validate vcluster.yaml --schema-version v0.24.0
   $ cat vcluster.yaml | vcluster-yaml validate -
   $ vcluster-yaml validate - < vcluster.yaml
-  $ vcluster-yaml validate vcluster.yaml --format table
   `)
   .action(async (file, options) => {
     try {

package/src/schema-validator.js

@@ -153,50 +153,79 @@ function validatePath(path, value, schema) {
 }
 
 /**
- * Find similar paths in schema for suggestions
+ * Find similar paths using priority-based matching (no magic numbers)
+ * Priority 1: Exact match on last segment (e.g., "version" matches "apiVersion")
+ * Priority 2: Substring match (e.g., "vers" matches "apiVersion")
+ * Priority 3: Shared prefix (e.g., "api.vers" matches "api.version")
  */
 function findSimilarPaths(targetPath, schema, maxSuggestions = 3) {
   const allPaths = extractSchemaPaths(schema);
+  if (allPaths.length === 0) return [];
+
   const targetParts = targetPath.toLowerCase().split('.');
   const targetLast = targetParts[targetParts.length - 1];
 
-  const scored = allPaths.map(schemaPath => {
+  // Categorize matches by clear priorities (no arbitrary scoring)
+  const exactMatches = [];
+  const substringMatches = [];
+  const prefixMatches = [];
+
+  for (const schemaPath of allPaths) {
     const parts = schemaPath.toLowerCase().split('.');
     const last = parts[parts.length - 1];
 
-    let score = 0;
-    if (last === targetLast) score += 10;
-    if (last.includes(targetLast) || targetLast.includes(last)) score += 5;
+    // Priority 1: Exact last segment match
+    if (last === targetLast) {
+      exactMatches.push(schemaPath);
+      continue;
+    }
+
+    // Priority 2: Substring match in last segment
+    if (last.includes(targetLast) || targetLast.includes(last)) {
+      substringMatches.push(schemaPath);
+      continue;
+    }
+
+    // Priority 3: Shared path prefix
+    const sharedPrefixLength = getSharedPrefix(targetParts, parts);
+    if (sharedPrefixLength > 0) {
+      prefixMatches.push({ path: schemaPath, prefixLength: sharedPrefixLength });
+    }
+  }
 
-    // Shared prefix
-    const sharedPrefix = getSharedPrefix(targetParts, parts);
-    score += sharedPrefix * 2;
+  // Sort prefix matches by shared prefix length (longer is better)
+  prefixMatches.sort((a, b) => b.prefixLength - a.prefixLength);
 
-    return { path: schemaPath, score };
-  });
+  // Combine in strict priority order
+  const suggestions = [
+    ...exactMatches,
+    ...substringMatches,
+    ...prefixMatches.map(m => m.path)
+  ];
 
-  return scored
-    .filter(s => s.score > 0)
-    .sort((a, b) => b.score - a.score)
-    .slice(0, maxSuggestions)
-    .map(s => s.path);
+  return suggestions.slice(0, maxSuggestions);
 }
 
 /**
  * Extract all paths from schema
+ * Contract: schema must be an object with optional .properties field
  */
 function extractSchemaPaths(schema, prefix = '') {
+  if (!schema || typeof schema !== 'object') {
+    return [];
+  }
+
   const paths = [];
+  // Explicit contract: use .properties if present, otherwise treat schema as properties object
   const props = schema.properties || schema;
 
-  if (props && typeof props === 'object') {
-    for (const [key, value] of Object.entries(props)) {
-      const path = prefix ? `${prefix}.${key}` : key;
-      paths.push(path);
+  for (const [key, value] of Object.entries(props)) {
+    const path = prefix ? `${prefix}.${key}` : key;
+    paths.push(path);
 
-      if (value && value.properties) {
-        paths.push(...extractSchemaPaths(value, path));
-      }
+    // Recurse if nested properties exist
+    if (value && typeof value === 'object' && value.properties) {
+      paths.push(...extractSchemaPaths(value, path));
     }
   }