feat(tools-user): user-facing helpers for tools-as-plugins

* build, meta.import test package, lint rule to avoid incorrect use of memo keyHash
* options.defaults, tool timeouts, permissions base
* server.caching, typing updates to avoid incorrect use of args on keyHash
* server.helpers, function, object helper
* server.tools, inline vs external tool setup, proxy for running external tools
* server.toolsHost, copy update
* server.toolsHostCreator, apply strict static properties, toolName
* server.toolsUser, types, tool creation, normalization for both user and internal use
* server, adjustments to help with user facing types, copy updates
* tool, align and add static toolName property

Author: cdcabrera

eslint.config.js

@@ -81,7 +81,25 @@ export default [
       }],
       'n/no-process-exit': 0,
       // Disallow console.log/info in runtime to protect STDIO; allow warn/error
-      'no-console': ['error', { allow: ['warn', 'error'] }]
+      'no-console': ['error', { allow: ['warn', 'error'] }],
+      // Custom syntax rules
+      'no-restricted-syntax': [
+        'error',
+        {
+          // Disallow rest parameters in a property named `keyHash`
+          selector:
+            "Property[key.name='keyHash'] > :matches(FunctionExpression, ArrowFunctionExpression) > RestElement",
+          message:
+            'keyHash must accept a single array parameter (args). Do not use rest params (...args).'
+        },
+        {
+          // Also catch when `keyHash` lives in a CallExpression options object (e.g., memo(fn, { keyHash() {} }))
+          selector:
+            "CallExpression > ObjectExpression > Property[key.name='keyHash'] > :matches(FunctionExpression, ArrowFunctionExpression) > RestElement",
+          message:
+            'keyHash must accept a single array parameter (args). Do not use rest params (...args).'
+        }
+      ]
     }
   },
   {

jest.config.ts

@@ -25,7 +25,25 @@ export default {
       roots: ['src'],
       testMatch: ['<rootDir>/src/**/*.test.ts'],
       setupFilesAfterEnv: ['<rootDir>/jest.setupTests.ts'],
-      ...baseConfig
+      ...baseConfig,
+      transform: {
+        '^.+\\.(ts|tsx)$': [
+          'ts-jest',
+          {
+            ...tsConfig,
+            diagnostics: {
+              ignoreCodes: [1343]
+            },
+            astTransformers: {
+              before: [
+                {
+                  path: 'ts-jest-mock-import-meta'
+                }
+              ]
+            }
+          }
+        ]
+      }
     },
     {
       displayName: 'e2e',

package-lock.json

@@ -4,8 +4,14 @@
   "description": "PatternFly documentation MCP server built with Node.js and TypeScript",
   "main": "dist/index.js",
   "type": "module",
+  "imports": {
+    "#toolsHost": "./dist/server.toolsHost.js"
+  },
   "exports": {
-    ".": "./dist/index.js"
+    ".": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js"
+    }
   },
   "bin": {
     "patternfly-mcp": "dist/cli.js",
@@ -64,6 +70,7 @@
     "jest": "^30.2.0",
     "pkgroll": "^2.20.1",
     "ts-jest": "29.4.4",
+    "ts-jest-mock-import-meta": "^1.3.1",
     "ts-node": "^10.1.0",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",

package.json

@@ -37,6 +37,7 @@ exports[`options defaults should return specific properties: defaults 1`] = `
     "invokeTimeoutMs": 10000,
     "loadTimeoutMs": 5000,
   },
+  "pluginIsolation": "none",
   "repoName": "patternfly-mcp",
   "resourceMemoOptions": {
     "default": {
@@ -70,6 +71,7 @@ exports[`options defaults should return specific properties: defaults 1`] = `
       "expire": 60000,
     },
   },
+  "toolModules": [],
   "urlRegex": /\\^\\(https\\?:\\)\\\\/\\\\//i,
   "version": "0.0.0",
 }

src/__tests__/__snapshots__/options.defaults.test.ts.snap

@@ -168,7 +168,7 @@ exports[`runServer should attempt to run server, register a tool: diagnostics 1`
       "test-server-5 server running on stdio transport",
     ],
     [
-      "Tool "loremIpsum" has a non‑Zod inputSchema. This may cause unexpected issues.",
+      "Tool "loremIpsum" has a non Zod inputSchema. This may cause unexpected issues.",
     ],
   ],
   "hasDebugLogs": true,
@@ -213,10 +213,10 @@ exports[`runServer should attempt to run server, register multiple tools: diagno
       "test-server-6 server running on stdio transport",
     ],
     [
-      "Tool "loremIpsum" has a non‑Zod inputSchema. This may cause unexpected issues.",
+      "Tool "loremIpsum" has a non Zod inputSchema. This may cause unexpected issues.",
     ],
     [
-      "Tool "dolorSit" has a non‑Zod inputSchema. This may cause unexpected issues.",
+      "Tool "dolorSit" has a non Zod inputSchema. This may cause unexpected issues.",
     ],
   ],
   "hasDebugLogs": true,