Refactor imports to use .js extensions and update tool registration logic

- Updated import statements across various files to include .js extensions for consistency.
- Refactored the ToolLoader class to register tools directly from static imports instead of loading from file paths.
- Removed obsolete test files related to tool loading and metadata validation.
- Enhanced the tools manifest with new entries for additional tools.
- Updated TypeScript configuration to use NodeNext module resolution.
- Added new dependencies for esbuild and rimraf in yarn.lock.

Author: Coderrob

.eslintrc.json

@@ -27,7 +27,7 @@
   "rules": {
     "@typescript-eslint/no-explicit-any": "error",
     "@typescript-eslint/explicit-function-return-type": "error",
-    "@typescript-eslint/strict-boolean-expressions": "warn",
+    "@typescript-eslint/strict-boolean-expressions": "off",
     "@typescript-eslint/no-unused-vars": [
       "error",
       {

README.md

@@ -69,7 +69,15 @@ The server requires environment variables for Backstage API access:
 ### Required Environment Variables
 
 - `BACKSTAGE_BASE_URL` - Base URL of your Backstage instance (e.g., `https://backstage.example.com`)
-- `BACKSTAGE_TOKEN` - Authentication token for API access (optional, depending on your Backstage setup)
+
+### Authentication Configuration
+
+Choose one of the following authentication methods:
+
+- `BACKSTAGE_TOKEN` - Bearer token for API access
+- `BACKSTAGE_CLIENT_ID`, `BACKSTAGE_CLIENT_SECRET`, `BACKSTAGE_TOKEN_URL` - OAuth credentials
+- `BACKSTAGE_API_KEY` - API key authentication
+- `BACKSTAGE_SERVICE_ACCOUNT_KEY` - Service account key
 
 ### Example Configuration
 

mcp.json

@@ -0,0 +1,12 @@
+{
+  "mcpServers": {
+    "backstage": {
+      "command": "node",
+      "args": ["dist/bundle.cjs"],
+      "env": {
+        "BACKSTAGE_BASE_URL": "http://localhost:7007",
+        "BACKSTAGE_TOKEN": "your-backstage-token"
+      }
+    }
+  }
+}

package.json

@@ -18,6 +18,7 @@
     "@types/node": "^22.0.0",
     "@typescript-eslint/eslint-plugin": "^8.43.0",
     "@typescript-eslint/parser": "^8.43.0",
+    "esbuild": "^0.18.18",
     "eslint": "^8.57.0",
     "eslint-config-prettier": "^9.0.0",
     "eslint-import-resolver-typescript": "^4.4.4",
@@ -26,17 +27,22 @@
     "jest": "^29.7.0",
     "jest-util": "^29.7.0",
     "prettier": "^3.5.3",
+    "rimraf": "^5.0.0",
     "ts-jest": "^29.3.2",
     "typescript": "^5.8.3"
   },
-  "main": "dist/index.js",
+  "main": "dist/bundle.cjs",
   "name": "@coderrob/mcp-backstage-server",
   "packageManager": "[email protected]",
   "scripts": {
-    "build": "tsc",
+    "clean": "rimraf dist",
+    "build:types": "tsc --emitDeclarationOnly",
+    "build:bundle": "esbuild src/index.ts --bundle --platform=node --target=node22 --format=cjs --outfile=dist/bundle.cjs --minify --sourcemap --metafile=dist/esbuild-meta.json",
+    "build": "yarn clean && yarn build:bundle",
+    "build:full": "yarn clean && yarn build:types && tsc && yarn build:bundle",
     "lint": "eslint 'src/**/*.ts' --ext .ts",
     "lint:fix": "prettier . --write && eslint 'src/**/*.ts' --ext .ts --fix",
-    "start": "node dist/index.js",
+    "start": "node dist/bundle.cjs",
     "test": "jest --coverage"
   },
   "types": "dist/index.d.ts",

src/api/backstage-catalog-api.ts

@@ -18,10 +18,18 @@ import {
 import { CompoundEntityRef, Entity, stringifyEntityRef } from '@backstage/catalog-model';
 import axios, { AxiosInstance, isAxiosError } from 'axios';
 
-import { AuthConfig, AuthManager, securityAuditor } from '../auth';
-import { CacheManager } from '../cache';
-import { IBackstageCatalogApi, JsonApiDocument, PaginationParams, SecurityEventType } from '../types';
-import { isNonEmptyString, isNumber, isString, JsonApiFormatter, logger, PaginationHelper } from '../utils';
+import { AuthConfig, AuthManager, securityAuditor } from '../auth/index.js';
+import { CacheManager } from '../cache/index.js';
+import { IBackstageCatalogApi, JsonApiDocument, PaginationParams, SecurityEventType } from '../types/index.js';
+import {
+  EntityRef,
+  isNonEmptyString,
+  isNumber,
+  isString,
+  JsonApiFormatter,
+  logger,
+  PaginationHelper,
+} from '../utils/index.js';
 
 interface BackstageCatalogApiOptions {
   baseUrl: string;
@@ -32,18 +40,14 @@ export class BackstageCatalogApi implements IBackstageCatalogApi {
   private readonly client: AxiosInstance;
   private readonly authManager: AuthManager;
   private readonly cacheManager: CacheManager;
-  private readonly paginationHelper: PaginationHelper;
-  private readonly jsonApiFormatter: JsonApiFormatter;
 
   constructor({ baseUrl, auth }: BackstageCatalogApiOptions) {
     logger.debug('Initializing BackstageCatalogApi', { baseUrl, authType: auth.type });
     this.authManager = new AuthManager(auth);
     this.cacheManager = new CacheManager();
-    this.paginationHelper = new PaginationHelper();
-    this.jsonApiFormatter = new JsonApiFormatter();
 
     this.client = axios.create({
-      baseURL: `${baseUrl.replace(/\/$/, '')}/v1`,
+      baseURL: `${baseUrl.replace(/\/$/, '')}/api/catalog`,
       timeout: 30000, // 30 second timeout
     });
     logger.debug('Axios client created with base URL', { baseUrl: this.client.defaults.baseURL });
@@ -204,7 +208,7 @@ export class BackstageCatalogApi implements IBackstageCatalogApi {
   ): Promise<GetEntityAncestorsResponse> {
     const { entityRef } = request;
     const { data } = await this.client.get<GetEntityAncestorsResponse>(
-      `/entities/by-ref/${encodeURIComponent(entityRef)}/ancestry`
+      `/entities/by-name/${encodeURIComponent(entityRef)}/ancestry`
     );
     return data;
   }
@@ -227,7 +231,13 @@ export class BackstageCatalogApi implements IBackstageCatalogApi {
 
     try {
       logger.debug('Fetching entity from API', { entityRef: refString });
-      const { data } = await this.client.get<Entity>(`/entities/by-ref/${encodeURIComponent(refString)}`);
+
+      // Parse the entity reference using the EntityRef class
+      const entityRef = EntityRef.parse(refString);
+
+      const { data } = await this.client.get<Entity>(
+        `/entities/by-name/${encodeURIComponent(entityRef.kind)}/${encodeURIComponent(entityRef.namespace)}/${encodeURIComponent(entityRef.name)}`
+      );
 
       // Cache the result for 5 minutes
       this.cacheManager.set(cacheKey, data, 5 * 60 * 1000);

src/api/index.ts

@@ -1 +1 @@
-export { BackstageCatalogApi } from './backstage-catalog-api';
+export { BackstageCatalogApi } from './backstage-catalog-api.js';

src/auth/auth-manager.ts

@@ -1,6 +1,6 @@
 import axios, { AxiosResponse } from 'axios';
 
-import { isNonEmptyString, isNumber, logger } from '../utils';
+import { isNonEmptyString, isNumber, logger } from '../utils/index.js';
 
 export interface AuthConfig {
   type: 'bearer' | 'oauth' | 'api-key' | 'service-account';

src/auth/index.ts

@@ -1,4 +1,4 @@
 /* eslint-disable import/no-unused-modules */
-export { type AuthConfig, AuthManager, type TokenInfo } from './auth-manager';
-export { InputSanitizer, inputSanitizer } from './input-sanitizer';
-export { SecurityAuditor, securityAuditor } from './security-auditor';
+export { type AuthConfig, AuthManager, type TokenInfo } from './auth-manager.js';
+export { InputSanitizer, inputSanitizer } from './input-sanitizer.js';
+export { SecurityAuditor, securityAuditor } from './security-auditor.js';

src/auth/input-sanitizer.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { isObject, isString } from '../utils';
+import { isObject, isString } from '../utils/index.js';
 
 export class InputSanitizer {
   private readonly maxStringLength = 10000;

src/auth/security-auditor.ts

@@ -1,6 +1,6 @@
 import { z } from 'zod';
 
-import { ISecurityEvent, SecurityEventType } from '../types';
+import { ISecurityEvent, SecurityEventType } from '../types/index.js';
 
 const SecurityEventSchema = z.object({
   id: z.string(),