refactor: rename enhanced scripts and update tool metadata for consistency

Author: Coderrob

README.md

@@ -200,8 +200,8 @@ src/
 └── index.ts       # Main server entry point
 
 scripts/
-├── validate-build-enhanced.sh    # Build validation with operational transparency
-├── dependency-manager-enhanced.sh # Dependency analysis with cross-platform support
+├── validate-build.sh    # Build validation with operational transparency
+├── dependency-manager.sh # Dependency analysis with cross-platform support
 ├── deps-crossplatform.sh         # Cross-platform dependency operations
 ├── monitor.sh                    # System monitoring and health checks
 └── deps.sh                       # Legacy dependency scripts

package.json

@@ -18,6 +18,7 @@
     "@rollup/plugin-json": "^6.1.0",
     "@rollup/plugin-node-resolve": "^16.0.1",
     "@rollup/plugin-replace": "^6.0.2",
+    "@rollup/plugin-terser": "^0.4.4",
     "@rollup/plugin-typescript": "^12.1.4",
     "@types/express": "^5.0.3",
     "@types/jest": "^30.0.0",
@@ -41,7 +42,6 @@
     "rollup": "^4.50.2",
     "rollup-plugin-dts": "^6.2.3",
     "rollup-plugin-preserve-shebang": "^1.0.1",
-    "rollup-plugin-terser": "^7.0.2",
     "ts-jest": "^29.4.2",
     "ts-morph": "^27.0.0",
     "ts-node": "^10.9.2",
@@ -68,11 +68,11 @@
   "scripts": {
     "build": "yarn clean && rollup -c",
     "build:dev": "yarn clean && rollup -c --environment NODE_ENV:development",
-    "build:validate": "sh -c 'bash scripts/validate-build-enhanced.sh'",
+    "build:validate": "sh -c 'bash scripts/validate-build.sh'",
     "build:watch": "yarn clean && rollup -c --watch",
     "clean": "rimraf dist",
     "deps": "sh -c 'bash scripts/deps.sh'",
-    "deps:analyze": "sh -c 'bash scripts/dependency-manager-enhanced.sh'",
+    "deps:analyze": "sh -c 'bash scripts/dependency-manager.sh'",
     "deps:audit": "sh -c 'bash scripts/deps.sh audit'",
     "deps:backup": "sh -c 'bash scripts/deps-crossplatform.sh backup'",
     "deps:check": "sh -c 'bash scripts/dependency-manager.sh --dry-run'",

rollup.config.js

@@ -20,7 +20,7 @@ import typescript from '@rollup/plugin-typescript';
 import nodeResolve from '@rollup/plugin-node-resolve';
 import commonjs from '@rollup/plugin-commonjs';
 import json from '@rollup/plugin-json';
-import { terser } from 'rollup-plugin-terser';
+import terser from '@rollup/plugin-terser';
 import replace from '@rollup/plugin-replace';
 import preserveShebang from 'rollup-plugin-preserve-shebang';
 import dts from 'rollup-plugin-dts';

scripts/dependency-manager.sh

@@ -806,4 +806,4 @@ parse_args_enhanced() {
 
 # Enhanced entry point - simplified
 parse_args_enhanced "$@"
-main_enhanced
+main_enhanced

src/tools/get_entities.tool.test.ts

@@ -115,8 +115,21 @@ describe('GetEntitiesTool', () => {
 
       // FormattedTextResponse returns formatted text, not JSON
       const responseText = result.content[0].text;
-      expect(responseText).toContain('Found 1 entities');
-      expect(responseText).toContain('Component: 1');
+      const expectedResponseText = {
+        status: 'success',
+        data: {
+          items: [
+            {
+              apiVersion: 'backstage.io/v1alpha1',
+              kind: 'Component',
+              metadata: { name: 'comp1', namespace: 'default' },
+              spec: { type: 'service' },
+            },
+          ],
+        },
+      };
+
+      expect(responseText).toEqual(JSON.stringify(expectedResponseText, null, 2));
     });
 
     it('should call the catalog client getEntitiesJsonApi method with jsonapi format', async () => {

src/tools/get_entities.tool.ts

@@ -24,7 +24,7 @@ import { ApiStatus } from '../types/apis.js';
 import { ToolName } from '../types/constants.js';
 import { IToolRegistrationContext } from '../types/tools.js';
 import { logger } from '../utils/core/logger.js';
-import { formatEntityList, FormattedTextResponse, JsonToTextResponse } from '../utils/formatting/responses.js';
+import { JsonToTextResponse } from '../utils/formatting/responses.js';
 import { ToolErrorHandler } from '../utils/tools/tool-error-handler.js';
 
 const entityFilterSchema = z.object({
@@ -77,17 +77,12 @@ export class GetEntitiesTool {
             status: ApiStatus.SUCCESS,
             data: jsonApiResult,
           });
-        } else if (req.format === 'standard') {
-          // Use the old formatted text response for 'standard' format
-          const result = await ctx.catalogClient.getEntities(sanitizedRequest);
-          logger.debug('Returning standard formatted entities', { count: result.items?.length || 0 });
-          return FormattedTextResponse({ status: ApiStatus.SUCCESS, data: result.items }, formatEntityList);
-        } else {
-          // Default to JSON format for better LLM access
-          const result = await ctx.catalogClient.getEntities(sanitizedRequest);
-          logger.debug('Returning JSON formatted entities', { count: result.items?.length || 0 });
-          return JsonToTextResponse({ status: ApiStatus.SUCCESS, data: result });
         }
+
+        // Default to JSON format for better LLM access
+        const result = await ctx.catalogClient.getEntities(sanitizedRequest);
+        logger.debug('Returning JSON formatted entities', { count: result.items?.length || 0 });
+        return JsonToTextResponse({ status: ApiStatus.SUCCESS, data: result });
       },
       request,
       context,

tools-manifest.json

@@ -2,66 +2,103 @@
   {
     "name": "add_location",
     "description": "Create a new location in the catalog.",
-    "params": ["type", "target"]
+    "params": [
+      "type",
+      "target"
+    ]
   },
   {
     "name": "get_entities_by_query",
     "description": "Get entities by query filters.",
-    "params": ["filter", "fields", "limit", "offset", "order"]
+    "params": [
+      "filter",
+      "fields",
+      "limit",
+      "offset",
+      "order"
+    ]
   },
   {
     "name": "get_entities_by_refs",
     "description": "Get multiple entities by their refs.",
-    "params": ["entityRefs"]
+    "params": [
+      "entityRefs"
+    ]
   },
   {
     "name": "get_entities",
     "description": "Get all entities in the catalog. Supports pagination and JSON:API formatting for enhanced LLM context.",
-    "params": ["filter", "fields", "limit", "offset", "format"]
+    "params": [
+      "filter",
+      "fields",
+      "limit",
+      "offset",
+      "format"
+    ]
   },
   {
     "name": "get_entity_ancestors",
     "description": "Get the ancestry tree for an entity.",
-    "params": ["entityRef"]
+    "params": [
+      "entityRef"
+    ]
   },
   {
     "name": "get_entity_by_ref",
     "description": "Get a single entity by its reference (namespace/name or compound ref).",
-    "params": ["entityRef"]
+    "params": [
+      "entityRef"
+    ]
   },
   {
     "name": "get_entity_facets",
     "description": "Get entity facets for a specified field.",
-    "params": ["filter", "facets"]
+    "params": [
+      "filter",
+      "facets"
+    ]
   },
   {
     "name": "get_location_by_entity",
     "description": "Get the location associated with an entity.",
-    "params": ["entityRef"]
+    "params": [
+      "entityRef"
+    ]
   },
   {
     "name": "get_location_by_ref",
     "description": "Get location by ref.",
-    "params": ["locationRef"]
+    "params": [
+      "locationRef"
+    ]
   },
   {
     "name": "refresh_entity",
     "description": "Trigger a refresh of an entity.",
-    "params": ["entityRef"]
+    "params": [
+      "entityRef"
+    ]
   },
   {
     "name": "remove_entity_by_uid",
     "description": "Remove an entity by UID.",
-    "params": ["uid"]
+    "params": [
+      "uid"
+    ]
   },
   {
     "name": "remove_location_by_id",
     "description": "Remove a location from the catalog by id.",
-    "params": ["locationId"]
+    "params": [
+      "locationId"
+    ]
   },
   {
     "name": "validate_entity",
     "description": "Validate an entity structure.",
-    "params": ["entity", "locationRef"]
+    "params": [
+      "entity",
+      "locationRef"
+    ]
   }
-]
+]

yarn.lock

@@ -5,7 +5,7 @@ __metadata:
   version: 8
   cacheKey: 10c0
 
-"@babel/code-frame@npm:^7.0.0, @babel/code-frame@npm:^7.10.4, @babel/code-frame@npm:^7.27.1":
+"@babel/code-frame@npm:^7.0.0, @babel/code-frame@npm:^7.27.1":
   version: 7.27.1
   resolution: "@babel/code-frame@npm:7.27.1"
   dependencies:
@@ -434,6 +434,7 @@ __metadata:
     "@rollup/plugin-json": "npm:^6.1.0"
     "@rollup/plugin-node-resolve": "npm:^16.0.1"
     "@rollup/plugin-replace": "npm:^6.0.2"
+    "@rollup/plugin-terser": "npm:^0.4.4"
     "@rollup/plugin-typescript": "npm:^12.1.4"
     "@types/express": "npm:^5.0.3"
     "@types/jest": "npm:^30.0.0"
@@ -461,7 +462,6 @@ __metadata:
     rollup: "npm:^4.50.2"
     rollup-plugin-dts: "npm:^6.2.3"
     rollup-plugin-preserve-shebang: "npm:^1.0.1"
-    rollup-plugin-terser: "npm:^7.0.2"
     ts-jest: "npm:^29.4.2"
     ts-morph: "npm:^27.0.0"
     ts-node: "npm:^10.9.2"
@@ -1426,6 +1426,22 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@rollup/plugin-terser@npm:^0.4.4":
+  version: 0.4.4
+  resolution: "@rollup/plugin-terser@npm:0.4.4"
+  dependencies:
+    serialize-javascript: "npm:^6.0.1"
+    smob: "npm:^1.0.0"
+    terser: "npm:^5.17.4"
+  peerDependencies:
+    rollup: ^2.0.0||^3.0.0||^4.0.0
+  peerDependenciesMeta:
+    rollup:
+      optional: true
+  checksum: 10c0/b9cb6c8f02ac1c1344019e9fb854321b74f880efebc41b6bdd84f18331fce0f4a2aadcdb481042245cd3f409b429ac363af71f9efec4a2024731d67d32af36ee
+  languageName: node
+  linkType: hard
+
 "@rollup/plugin-typescript@npm:^12.1.4":
   version: 12.1.4
   resolution: "@rollup/plugin-typescript@npm:12.1.4"
@@ -6031,17 +6047,6 @@ __metadata:
   languageName: node
   linkType: hard
 
-"jest-worker@npm:^26.2.1":
-  version: 26.6.2
-  resolution: "jest-worker@npm:26.6.2"
-  dependencies:
-    "@types/node": "npm:*"
-    merge-stream: "npm:^2.0.0"
-    supports-color: "npm:^7.0.0"
-  checksum: 10c0/07e4dba650381604cda253ab6d5837fe0279c8d68c25884995b45bfe149a7a1e1b5a97f304b4518f257dac2a9ddc1808d57d650649c3ab855e9e60cf824d2970
-  languageName: node
-  linkType: hard
-
 "jest@npm:^30.1.3":
   version: 30.1.3
   resolution: "jest@npm:30.1.3"
@@ -7862,20 +7867,6 @@ __metadata:
   languageName: node
   linkType: hard
 
-"rollup-plugin-terser@npm:^7.0.2":
-  version: 7.0.2
-  resolution: "rollup-plugin-terser@npm:7.0.2"
-  dependencies:
-    "@babel/code-frame": "npm:^7.10.4"
-    jest-worker: "npm:^26.2.1"
-    serialize-javascript: "npm:^4.0.0"
-    terser: "npm:^5.0.0"
-  peerDependencies:
-    rollup: ^2.0.0
-  checksum: 10c0/f79b851c6f7b06555d3a8ce7a4e32abd2b7cb8318e89fb8db73e662fa6e3af1a59920e881d111efc65a7437fd9582b61b1f4859b6fd839ba948616829d92432d
-  languageName: node
-  linkType: hard
-
 "rollup@npm:^4.50.2":
   version: 4.50.2
   resolution: "rollup@npm:4.50.2"
@@ -8105,12 +8096,12 @@ __metadata:
   languageName: node
   linkType: hard
 
-"serialize-javascript@npm:^4.0.0":
-  version: 4.0.0
-  resolution: "serialize-javascript@npm:4.0.0"
+"serialize-javascript@npm:^6.0.1":
+  version: 6.0.2
+  resolution: "serialize-javascript@npm:6.0.2"
   dependencies:
     randombytes: "npm:^2.1.0"
-  checksum: 10c0/510dfe7f0311c0b2f7ab06311afa1668ba2969ab2f1faaac0a4924ede76b7f22ba85cfdeaa0052ec5a047bca42c8cd8ac8df8f0efe52f9bd290b3a39ae69fe9d
+  checksum: 10c0/2dd09ef4b65a1289ba24a788b1423a035581bef60817bea1f01eda8e3bda623f86357665fe7ac1b50f6d4f583f97db9615b3f07b2a2e8cbcb75033965f771dd2
   languageName: node
   linkType: hard
 
@@ -8269,6 +8260,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"smob@npm:^1.0.0":
+  version: 1.5.0
+  resolution: "smob@npm:1.5.0"
+  checksum: 10c0/a1067f23265812de8357ed27312101af49b89129eb973e3f26ab5856ea774f88cace13342e66e32470f933ccfa916e0e9d0f7ca8bbd4f92dfab2af45c15956c2
+  languageName: node
+  linkType: hard
+
 "socks-proxy-agent@npm:^8.0.3":
   version: 8.0.5
   resolution: "socks-proxy-agent@npm:8.0.5"
@@ -8580,7 +8578,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"supports-color@npm:^7.0.0, supports-color@npm:^7.1.0":
+"supports-color@npm:^7.1.0":
   version: 7.2.0
   resolution: "supports-color@npm:7.2.0"
   dependencies:
@@ -8635,7 +8633,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"terser@npm:^5.0.0":
+"terser@npm:^5.17.4":
   version: 5.44.0
   resolution: "terser@npm:5.44.0"
   dependencies: