fix: @W-19772910@ Fixed issue with relative paths (#291)

* @W-19772910@ Fixed issue with relative paths

* @W-19772910@ Fixed failing tests

Author: jfeingold35

packages/mcp-provider-code-analyzer/package.json

@@ -14,12 +14,12 @@
   "types": "dist/index.d.ts",
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.18.0",
-    "@salesforce/code-analyzer-core": "^0.35.0",
-    "@salesforce/code-analyzer-engine-api": "^0.29.0",
-    "@salesforce/code-analyzer-eslint-engine": "^0.31.0",
-    "@salesforce/code-analyzer-pmd-engine": "^0.30.0",
-    "@salesforce/code-analyzer-regex-engine": "^0.27.0",
-    "@salesforce/code-analyzer-retirejs-engine": "^0.26.0",
+    "@salesforce/code-analyzer-core": "^0.38.0",
+    "@salesforce/code-analyzer-engine-api": "^0.30.0",
+    "@salesforce/code-analyzer-eslint-engine": "^0.34.0",
+    "@salesforce/code-analyzer-pmd-engine": "^0.31.0",
+    "@salesforce/code-analyzer-regex-engine": "^0.28.0",
+    "@salesforce/code-analyzer-retirejs-engine": "^0.27.0",
     "@salesforce/mcp-test-client": "0.0.1",
     "@salesforce/mcp-provider-api": "^0.4.0",
     "zod": "^3.25.76"

packages/mcp-provider-code-analyzer/salesforce-mcp-provider-code-analyzer-v0.2.0.tgz

@@ -20,7 +20,7 @@ const DESCRIPTION: string = `A tool for performing static analysis against code.
     `- When the user asks you to check code for problems, use this tool to do that.\n`;
 
 export const inputSchema = z.object({
-    target: z.array(z.string()).describe(`A JSON-formatted array of between 1 and ${MAX_ALLOWABLE_TARGET_COUNT} files on the users machine that should be scanned.`)
+    target: z.array(z.string()).describe(`A JSON-formatted array of between 1 and ${MAX_ALLOWABLE_TARGET_COUNT} files on the users machine that should be scanned. These paths MUST be ABSOLUTE paths, and not relative paths.`)
 });
 type InputArgsShape = typeof inputSchema.shape;
 

packages/mcp-provider-code-analyzer/src/tools/run_code_analyzer.ts

@@ -218,9 +218,16 @@ describe('RunAnalyzerActionImpl', () => {
             const pathSepVar: string = path.sep.replaceAll('\\', '\\\\');
             const runDir: string = process.cwd().replaceAll('\\' , '\\\\');
 
+            const codeAnalyzerVersion: string = (JSON.parse((await fs.promises.readFile(path.join(require.resolve('@salesforce/code-analyzer-core'), '..', '..', 'package.json'), 'utf-8'))) as any).version;
+            const pmdEngineVersion: string = (JSON.parse((await fs.promises.readFile(path.join(require.resolve('@salesforce/code-analyzer-pmd-engine'), '..', '..', 'package.json'), 'utf-8'))) as any).version;
+            const regexEngineVersion: string = (JSON.parse((await fs.promises.readFile(path.join(require.resolve('@salesforce/code-analyzer-regex-engine'), '..', '..', 'package.json'), 'utf-8'))) as any).version;
+
             const expectedOutfile: string =  (await fs.promises.readFile(comparisonFile, 'utf-8'))
                 .replaceAll('{{RUNDIR}}', runDir)
                 .replaceAll(`{{SEP}}`, pathSepVar)
+                .replaceAll('{{CODE_ANALYZER_VERSION}}', codeAnalyzerVersion)
+                .replaceAll('{{PMD_ENGINE_VERSION}}', pmdEngineVersion)
+                .replaceAll('{{REGEX_ENGINE_VERSION}}', regexEngineVersion)
                 .replaceAll('{{PMD_VERSION}}', PMD_VERSION);
 
             expect(outputFileContents).toContain(expectedOutfile);

packages/mcp-provider-code-analyzer/test/actions/run-analyzer.test.ts

@@ -9,7 +9,7 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
     "UnrunnableEngine": "1.0.0"
   },
   "violations": [

packages/mcp-provider-code-analyzer/test/fixtures/comparison-files/UnexpectedEngineError-violation.goldfile.txt

@@ -9,7 +9,7 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
     "EngineThatCannotReturnRules": "unknown"
   },
   "violations": [

packages/mcp-provider-code-analyzer/test/fixtures/comparison-files/UninstantiableEngineError-for-EngineThatCannotReturnRules.goldfile.txt

@@ -9,9 +9,9 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
-    "cpd": "0.30.0",
-    "regex": "0.27.0",
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
+    "cpd": "{{PMD_ENGINE_VERSION}}",
+    "regex": "{{REGEX_ENGINE_VERSION}}",
     "pmd": "unknown"
   },
   "violations": [

packages/mcp-provider-code-analyzer/test/fixtures/comparison-files/invalid-pmd-config-violation.goldfile.txt

@@ -9,7 +9,7 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
     "EngineThatLogsError": "1.0.0"
   },
   "violations": []

packages/mcp-provider-code-analyzer/test/fixtures/comparison-files/no-custom-engine-violations.goldfile.json

@@ -9,9 +9,9 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
-    "pmd": "0.30.0",
-    "regex": "0.27.0"
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
+    "pmd": "{{PMD_ENGINE_VERSION}}",
+    "regex": "{{REGEX_ENGINE_VERSION}}"
   },
   "violations": []
 }

packages/mcp-provider-code-analyzer/test/fixtures/comparison-files/no-pmd-violations.goldfile.json

@@ -9,9 +9,9 @@
     "sev5": 0
   },
   "versions": {
-    "code-analyzer": "0.35.0",
-    "pmd": "0.30.0",
-    "regex": "0.27.0"
+    "code-analyzer": "{{CODE_ANALYZER_VERSION}}",
+    "pmd": "{{PMD_ENGINE_VERSION}}",
+    "regex": "{{REGEX_ENGINE_VERSION}}"
   },
   "violations": [
     {