@optimizely-opal/opal-tool-ocp-sdk 0.0.0-OCP-1487.9 → 0.0.0-OCP-1487.10

package/README.md

@@ -7,7 +7,7 @@ A TypeScript SDK for building Opal tools in Optimizely Connect Platform. This SD
 ## Features
 
 - 🎯 **Decorator-based Tool Registration** - Use `@tool` and `@interaction` decorators to easily register functions
-- 🌐 **Global and Regular Tools** - Support for both organization-scoped and global tools
+- 🌐 **Global and Regular Function Modes** - SDK can be used in either global or organization-scoped mode
 - 🔧 **Type-safe Development** - Full TypeScript support with comprehensive type definitions  
 - 🏗️ **Abstract Base Classes** - Extend `ToolFunction` or `GlobalToolFunction` for standardized request processing
 - 🔐 **Authentication Support** - OptiID authentication
@@ -29,14 +29,14 @@ yarn add @optimizely-opal/opal-tool-ocp-sdk
 
 ## Quick Start
 
-The SDK supports two types of functions:
+The SDK supports two function modes:
 
-- **Regular Functions** (`ToolFunction`) - Organization-scoped tools that validate customer organization IDs
-- **Global Functions** (`GlobalToolFunction`) - Platform-wide tools that work across all organizations
+- **Regular Functions** (`ToolFunction`) - Organization-scoped functions that validate customer organization IDs
+- **Global Functions** (`GlobalToolFunction`) - Platform-wide functions that work across all organizations
 
 ### Regular Tool Function
 
-Create a tool function class by extending `ToolFunction` for organization-scoped tools:
+Create a tool function class by extending `ToolFunction` for organization-scoped functionality:
 
 ```typescript
 import { ToolFunction, tool, interaction, ParameterType, InteractionResult, OptiIdAuthData } from '@optimizely-opal/opal-tool-ocp-sdk';
@@ -122,7 +122,7 @@ export class MyToolFunction extends ToolFunction {
 
 ### Global Tool Function
 
-Create a global tool function by extending `GlobalToolFunction` for platform-wide tools:
+Create a global tool function by extending `GlobalToolFunction` for platform-wide functionality:
 
 ```typescript
 import { GlobalToolFunction, tool, interaction, ParameterType, InteractionResult, OptiIdAuthData } from '@optimizely-opal/opal-tool-ocp-sdk';
@@ -151,14 +151,14 @@ export class MyGlobalToolFunction extends GlobalToolFunction {
 }
 ```
 
-### Function Types and Tool Discovery
+### Function Modes
 
-The SDK provides intelligent tool discovery based on function type:
+The SDK operates in one of two modes based on the base class you extend:
 
-- **Regular Functions** (`ToolFunction`): Can only discover and access non-global tools
-- **Global Functions** (`GlobalToolFunction`): Can only discover and access global tools
+- **Regular Function Mode** (`ToolFunction`): All tools are organization-scoped and validate organization IDs
+- **Global Function Mode** (`GlobalToolFunction`): All tools are platform-wide.
 
-This ensures proper isolation between global platform tools and organization-specific tools.
+The discovery endpoint returns all tools registered within that function mode.
 
 Your function class inherits a `perform()` method from `ToolFunction` or `GlobalToolFunction` that serves as the main entry point for handling all incoming requests. When called, the SDK automatically:
 
@@ -177,27 +177,25 @@ Tools are functions that can be discovered and executed through the OCP platform
 - Define parameters with types and validation
 - Can require authentication
 - Return structured responses
-- Can be **regular tools** (organization-scoped) or **global tools** (platform-wide)
+- Are automatically registered based on the function mode you choose
 
-#### Regular Tools (Default)
+### Function Modes
 
-Regular tools are scoped to specific organizations and validate that requests come from the same organization:
+#### Regular Functions (`ToolFunction`)
 
-- Validate OptiID organization ID matches the tool's organization
-- Only discoverable by `ToolFunction` instances
-- **Automatic Registration**: Tools defined in regular classes or imported without context
-- **Explicit Registration**: Tools imported using `importAsRegular()`
+Regular functions are scoped to specific organizations and validate that requests come from the same organization:
+
+- Validate OptiID organization ID matches the function's organization context
+- All tools within the function are organization-scoped
 - **Per-Organization Configuration**: Can implement organization-specific configuration, authentication credentials, and API keys since they're tied to a single organization
 - **Per-Organization Authentication**: Can store and use organization-specific authentication tokens, connection strings, and other sensitive data securely
 
-#### Global Tools
+#### Global Functions (`GlobalToolFunction`)
 
-Global tools work across all organizations without organization validation:
+Global functions work across all organizations without organization validation:
 
-- Accept OptiID authentication but don't validate organization ID
-- Only discoverable by `GlobalToolFunction` instances  
-- **Automatic Registration**: Tools defined in classes extending `GlobalToolFunction`
-- **Explicit Registration**: Tools imported using `importAsGlobal()`
+- Accept OptiID authentication
+- All tools within the function are platform-wide
 - **No Per-Organization Configuration**: Cannot implement per-organization configuration since they work across all organizations
 - **No Per-Organization Authentication**: Cannot store organization-specific credentials or authentication data
 - **Global Discovery**: Have a global discovery URL that can be used by any organization without requiring them to install the app first
@@ -293,41 +291,6 @@ async handlerMethod(
 - **params**: The input parameters for tools, or interaction data for webhooks
 - **authData**: Available when OptiID user authentication is configured and successful
 
-### Import Utilities
-
-The SDK provides utilities for importing tools with specific registration contexts:
-
-#### `importAsGlobal(modulePath: string, baseUrl?: string)`
-
-Imports a module and registers all decorated tools as global tools.
-
-```typescript
-import { importAsGlobal } from '@optimizely-opal/opal-tool-ocp-sdk';
-
-// Import tools as global
-await importAsGlobal('./global-tools');
-await importAsGlobal('../global-tools', import.meta.url);
-```
-
-**Parameters:**
-- `modulePath`: Path to the module to import (relative or absolute)
-- `baseUrl`: Base URL for resolving relative paths (optional, use `import.meta.url` for relative to current file)
-
-#### `importAsRegular(modulePath: string, baseUrl?: string)`
-
-Imports a module and registers all decorated tools as regular tools.
-
-```typescript
-import { importAsRegular } from '@optimizely-opal/opal-tool-ocp-sdk';
-
-// Import tools as regular
-await importAsRegular('./tools');
-await importAsRegular('../tools', import.meta.url);
-```
-
-**Parameters:**
-- `modulePath`: Path to the module to import (relative or absolute)
-- `baseUrl`: Base URL for resolving relative paths (optional, use `import.meta.url` for relative to current file)
 
 ### Decorators
 
@@ -376,13 +339,13 @@ Extend this class for regular tools that validate organization IDs. The `perform
 Abstract base class for global OCP functions:
 
 ```typescript
-export abstract class GlobalToolFunction extends Function {
+export abstract class GlobalToolFunction extends GlobalFunction {
   protected ready(): Promise<boolean>;
   public async perform(): Promise<Response>;
 }
 ```
 
-Extend this class for global tools that work across organizations. The `perform` method routes requests to registered tools but skips organization validation for global tools.
+Extend this class for tools that work across organizations. The `perform` method routes requests to registered tools but skips organization validation for tools.
 
 ### Models
 
@@ -401,14 +364,14 @@ The SDK automatically provides two important endpoints:
 
 ### Discovery Endpoint (`/discovery`)
 
-Returns registered tools in the proper OCP format for platform integration. The discovery endpoint intelligently filters tools based on the function type and automatic registration context:
+Returns all registered tools in the proper OCP format for platform integration. The discovery endpoint returns all tools registered within the function, regardless of their individual configuration:
 
-- **Regular Functions** (`ToolFunction`): Only returns regular tools (organization-scoped, including those imported with `importAsRegular()`)
-- **Global Functions** (`GlobalToolFunction`): Only returns global tools (platform-wide, including those from classes extending `GlobalToolFunction` or imported with `importAsGlobal()`)
+- **Regular Functions** (`ToolFunction`): Returns all tools with organization-scoped behavior
+- **Global Functions** (`GlobalToolFunction`): Returns all tools with platform-wide behavior
 
-This ensures proper tool isolation and security boundaries between global and organization-specific tools.
+All tools within a function operate in the same mode - there is no mixing of global and organization-scoped tools within a single function.
 
-Example response for a regular and global function:
+Example response for a regular function and global function:
 
 ```json
 {
@@ -562,10 +525,6 @@ src/
 │   ├── index.ts
 │   ├── TaskTool.ts
 │   └── NotificationTool.ts
-├── global-tools/
-│   ├── index.ts
-│   ├── UtilityTool.ts
-│   └── AnalyticsTool.ts
 ├── MyToolFunction.ts
 └── MyGlobalToolFunction.ts
 ```

package/dist/decorator/Decorator.d.ts

@@ -1,10 +1,4 @@
 import { ParameterType } from '../types/Models';
-/**
- * Global context flag to track import context
- */
-declare global {
-    var __IMPORT_CONTEXT: 'global' | 'regular' | undefined;
-}
 /**
  * Configuration for @tool decorator
  */

package/dist/decorator/Decorator.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"Decorator.d.ts","sourceRoot":"","sources":["../../src/decorator/Decorator.ts"],"names":[],"mappings":"AAAA,OAAO,EAA8B,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAG5E;;GAEG;AACH,OAAO,CAAC,MAAM,CAAC;IACb,IAAI,gBAAgB,EAAE,QAAQ,GAAG,SAAS,GAAG,SAAS,CAAC;CACxD;AAiCD;;GAEG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,eAAe,EAAE,CAAC;IAC9B,gBAAgB,CAAC,EAAE,qBAAqB,EAAE,CAAC;IAC3C,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,OAAO,CAAC;CACnB;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;;;GAIG;AACH,wBAAgB,IAAI,CAAC,MAAM,EAAE,UAAU,IACrB,QAAQ,GAAG,EAAE,cAAc,MAAM,EAAE,YAAY,kBAAkB,UAsClF;AAED;;;;GAIG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,iBAAiB,IACnC,QAAQ,GAAG,EAAE,cAAc,MAAM,EAAE,YAAY,kBAAkB,UAqBlF"}
+{"version":3,"file":"Decorator.d.ts","sourceRoot":"","sources":["../../src/decorator/Decorator.ts"],"names":[],"mappings":"AAAA,OAAO,EAA8B,aAAa,EAAE,MAAM,iBAAiB,CAAC;AAG5E;;GAEG;AACH,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,EAAE,eAAe,EAAE,CAAC;IAC9B,gBAAgB,CAAC,EAAE,qBAAqB,EAAE,CAAC;IAC3C,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;GAEG;AACH,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,OAAO,CAAC;CACnB;AAED;;GAEG;AACH,MAAM,WAAW,qBAAqB;IACpC,QAAQ,EAAE,MAAM,CAAC;IACjB,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;;;GAIG;AACH,wBAAgB,IAAI,CAAC,MAAM,EAAE,UAAU,IACrB,QAAQ,GAAG,EAAE,cAAc,MAAM,EAAE,YAAY,kBAAkB,UAkClF;AAED;;;;GAIG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,iBAAiB,IACnC,QAAQ,GAAG,EAAE,cAAc,MAAM,EAAE,YAAY,kBAAkB,UAqBlF"}

package/dist/decorator/Decorator.js

@@ -4,34 +4,6 @@ exports.tool = tool;
 exports.interaction = interaction;
 const Models_1 = require("../types/Models");
 const Service_1 = require("../service/Service");
-/**
- * Helper function to check if a class extends GlobalToolFunction
- */
-function isGlobalToolFunction(constructor) {
-    // Walk up the prototype chain to check for GlobalToolFunction
-    let currentConstructor = constructor;
-    while (currentConstructor) {
-        if (currentConstructor.name === 'GlobalToolFunction') {
-            return true;
-        }
-        currentConstructor = Object.getPrototypeOf(currentConstructor);
-    }
-    return false;
-}
-/**
- * Determine if tool should be registered as global based on definition context or import context
- */
-function shouldRegisterAsGlobal(targetConstructor) {
-    // First check if there's an active import context
-    if (globalThis.__IMPORT_CONTEXT === 'global') {
-        return true;
-    }
-    if (globalThis.__IMPORT_CONTEXT === 'regular') {
-        return false;
-    }
-    // Fallback to checking where the tool is defined
-    return isGlobalToolFunction(targetConstructor);
-}
 /**
  * Decorator for registering tool functions
  * Immediately registers the tool with the global ToolsService
@@ -53,10 +25,8 @@ function tool(config) {
             // eslint-disable-next-line @typescript-eslint/no-unsafe-call
             return originalMethod.call(instance, params, authData);
         };
-        // Determine if this tool should be registered as global based on definition or import context
-        const isGlobal = shouldRegisterAsGlobal(target.constructor);
-        // Register tool with global flag based on where it's defined or imported
-        Service_1.toolsService.registerTool(config.name, config.description, boundHandler, parameters, config.endpoint, authRequirements, isGlobal);
+        // Immediately register with global ToolsService
+        Service_1.toolsService.registerTool(config.name, config.description, boundHandler, parameters, config.endpoint, authRequirements);
     };
 }
 /**

package/dist/decorator/Decorator.js.map

@@ -1 +1 @@
-{"version":3,"file":"Decorator.js","sourceRoot":"","sources":["../../src/decorator/Decorator.ts"],"names":[],"mappings":";;AAoFA,oBAuCC;AAOD,kCAsBC;AAxJD,4CAA4E;AAC5E,gDAAkD;AASlD;;GAEG;AACH,SAAS,oBAAoB,CAAC,WAAgB;IAC5C,8DAA8D;IAC9D,IAAI,kBAAkB,GAAG,WAAW,CAAC;IACrC,OAAO,kBAAkB,EAAE,CAAC;QAC1B,IAAI,kBAAkB,CAAC,IAAI,KAAK,oBAAoB,EAAE,CAAC;YACrD,OAAO,IAAI,CAAC;QACd,CAAC;QACD,kBAAkB,GAAG,MAAM,CAAC,cAAc,CAAC,kBAAkB,CAAC,CAAC;IACjE,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;GAEG;AACH,SAAS,sBAAsB,CAAC,iBAAsB;IACpD,kDAAkD;IAClD,IAAI,UAAU,CAAC,gBAAgB,KAAK,QAAQ,EAAE,CAAC;QAC7C,OAAO,IAAI,CAAC;IACd,CAAC;IACD,IAAI,UAAU,CAAC,gBAAgB,KAAK,SAAS,EAAE,CAAC;QAC9C,OAAO,KAAK,CAAC;IACf,CAAC;IAED,iDAAiD;IACjD,OAAO,oBAAoB,CAAC,iBAAiB,CAAC,CAAC;AACjD,CAAC;AAwCD;;;;GAIG;AACH,SAAgB,IAAI,CAAC,MAAkB;IACrC,OAAO,UAAS,MAAW,EAAE,YAAoB,EAAE,UAA8B;QAC/E,mDAAmD;QACnD,MAAM,UAAU,GAAG,CAAC,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACrD,IAAI,kBAAS,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,QAAQ,CAAC,CACzD,CAAC;QAEF,gEAAgE;QAChE,MAAM,gBAAgB,GAAG,CAAC,MAAM,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACjE,IAAI,wBAAe,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,QAAQ,CAAC,CAC3D,CAAC;QAEF,MAAM,cAAc,GAAG,UAAU,CAAC,KAAK,CAAC;QAExC,MAAM,YAAY,GAAG,UAAS,eAAoB,EAAE,MAAW,EAAE,QAAa;YAC5E,0EAA0E;YAC1E,uDAAuD;YACvD,MAAM,QAAQ,GAAG,CAAC,eAAe,IAAI,eAAe,YAAY,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC;gBACnF,6DAA6D;gBAC7D,eAAe,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAE7C,6DAA6D;YAC7D,OAAO,cAAc,CAAC,IAAI,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;QACzD,CAAC,CAAC;QAEF,8FAA8F;QAC9F,MAAM,QAAQ,GAAG,sBAAsB,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC;QAE5D,yEAAyE;QACzE,sBAAY,CAAC,YAAY,CACvB,MAAM,CAAC,IAAI,EACX,MAAM,CAAC,WAAW,EAClB,YAAY,EACZ,UAAU,EACV,MAAM,CAAC,QAAQ,EACf,gBAAgB,EAChB,QAAQ,CACT,CAAC;IACJ,CAAC,CAAC;AACJ,CAAC;AAED;;;;GAIG;AACH,SAAgB,WAAW,CAAC,MAAyB;IACnD,OAAO,UAAS,MAAW,EAAE,YAAoB,EAAE,UAA8B;QAC/E,MAAM,cAAc,GAAG,UAAU,CAAC,KAAK,CAAC;QAExC,MAAM,YAAY,GAAG,UAAS,eAAoB,EAAE,IAAS,EAAE,QAAc;YAC3E,0EAA0E;YAC1E,uDAAuD;YACvD,MAAM,QAAQ,GAAG,CAAC,eAAe,IAAI,eAAe,YAAY,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC;gBACnF,6DAA6D;gBAC7D,eAAe,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAE7C,6DAA6D;YAC7D,OAAO,cAAc,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,EAAE,QAAQ,CAAC,CAAC;QACvD,CAAC,CAAC;QAEF,gDAAgD;QAChD,sBAAY,CAAC,mBAAmB,CAC9B,MAAM,CAAC,IAAI,EACX,YAAY,EACZ,MAAM,CAAC,QAAQ,CAChB,CAAC;IACJ,CAAC,CAAC;AACJ,CAAC"}
+{"version":3,"file":"Decorator.js","sourceRoot":"","sources":["../../src/decorator/Decorator.ts"],"names":[],"mappings":";;AA8CA,oBAmCC;AAOD,kCAsBC;AA9GD,4CAA4E;AAC5E,gDAAkD;AAwClD;;;;GAIG;AACH,SAAgB,IAAI,CAAC,MAAkB;IACrC,OAAO,UAAS,MAAW,EAAE,YAAoB,EAAE,UAA8B;QAC/E,mDAAmD;QACnD,MAAM,UAAU,GAAG,CAAC,MAAM,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACrD,IAAI,kBAAS,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,QAAQ,CAAC,CACzD,CAAC;QAEF,gEAAgE;QAChE,MAAM,gBAAgB,GAAG,CAAC,MAAM,CAAC,gBAAgB,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACjE,IAAI,wBAAe,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,QAAQ,CAAC,CAC3D,CAAC;QAEF,MAAM,cAAc,GAAG,UAAU,CAAC,KAAK,CAAC;QAExC,MAAM,YAAY,GAAG,UAAS,eAAoB,EAAE,MAAW,EAAE,QAAa;YAC5E,0EAA0E;YAC1E,uDAAuD;YACvD,MAAM,QAAQ,GAAG,CAAC,eAAe,IAAI,eAAe,YAAY,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC;gBACnF,6DAA6D;gBAC7D,eAAe,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAE7C,6DAA6D;YAC7D,OAAO,cAAc,CAAC,IAAI,CAAC,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;QACzD,CAAC,CAAC;QAEF,gDAAgD;QAChD,sBAAY,CAAC,YAAY,CACvB,MAAM,CAAC,IAAI,EACX,MAAM,CAAC,WAAW,EAClB,YAAY,EACZ,UAAU,EACV,MAAM,CAAC,QAAQ,EACf,gBAAgB,CACjB,CAAC;IACJ,CAAC,CAAC;AACJ,CAAC;AAED;;;;GAIG;AACH,SAAgB,WAAW,CAAC,MAAyB;IACnD,OAAO,UAAS,MAAW,EAAE,YAAoB,EAAE,UAA8B;QAC/E,MAAM,cAAc,GAAG,UAAU,CAAC,KAAK,CAAC;QAExC,MAAM,YAAY,GAAG,UAAS,eAAoB,EAAE,IAAS,EAAE,QAAc;YAC3E,0EAA0E;YAC1E,uDAAuD;YACvD,MAAM,QAAQ,GAAG,CAAC,eAAe,IAAI,eAAe,YAAY,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC;gBACnF,6DAA6D;gBAC7D,eAAe,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,WAAW,EAAE,CAAC;YAE7C,6DAA6D;YAC7D,OAAO,cAAc,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,EAAE,QAAQ,CAAC,CAAC;QACvD,CAAC,CAAC;QAEF,gDAAgD;QAChD,sBAAY,CAAC,mBAAmB,CAC9B,MAAM,CAAC,IAAI,EACX,YAAY,EACZ,MAAM,CAAC,QAAQ,CAChB,CAAC;IACJ,CAAC,CAAC;AACJ,CAAC"}

package/dist/decorator/Decorator.test.js

@@ -16,9 +16,6 @@ jest.mock('../service/Service', () => ({
         registerInteraction: jest.fn()
     }
 }));
-// Mock base class for testing auto-detection
-class GlobalToolFunction {
-}
 describe('Decorators', () => {
     beforeEach(() => {
         jest.clearAllMocks();
@@ -39,7 +36,7 @@ describe('Decorators', () => {
             __decorate([
                 (0, Decorator_1.tool)(config)
             ], TestClass.prototype, "testMethod", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('testTool', 'A test tool', expect.any(Function), [], '/test-tool', [], false);
+            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('testTool', 'A test tool', expect.any(Function), [], '/test-tool', []);
             // Ensure TestClass is considered "used" by TypeScript
             expect(TestClass).toBeDefined();
         });
@@ -71,7 +68,7 @@ describe('Decorators', () => {
                     description: 'An input parameter',
                     required: true
                 })
-            ]), '/tool-with-params', [], false);
+            ]), '/tool-with-params', []);
             // Ensure TestClass is considered "used" by TypeScript
             expect(TestClass).toBeDefined();
         });
@@ -102,7 +99,7 @@ describe('Decorators', () => {
                     scopeBundle: 'calendar',
                     required: true
                 })
-            ]), false);
+            ]));
             // Ensure TestClass is considered "used" by TypeScript
             expect(TestClass).toBeDefined();
         });
@@ -172,7 +169,7 @@ describe('Decorators', () => {
                     scopeBundle: 'drive',
                     required: false
                 })
-            ]), false);
+            ]));
             // Ensure TestClass is considered "used" by TypeScript
             expect(TestClass).toBeDefined();
         });
@@ -191,7 +188,7 @@ describe('Decorators', () => {
             __decorate([
                 (0, Decorator_1.tool)(config)
             ], TestClass.prototype, "emptyParamsTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('emptyParamsTool', 'Tool with empty parameters', expect.any(Function), [], '/empty-params', [], false);
+            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('emptyParamsTool', 'Tool with empty parameters', expect.any(Function), [], '/empty-params', []);
             expect(TestClass).toBeDefined();
         });
         it('should handle empty auth requirements array', () => {
@@ -210,7 +207,7 @@ describe('Decorators', () => {
             __decorate([
                 (0, Decorator_1.tool)(config)
             ], TestClass.prototype, "noAuthTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('noAuthTool', 'Tool with no auth requirements', expect.any(Function), [], '/no-auth', [], false);
+            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('noAuthTool', 'Tool with no auth requirements', expect.any(Function), [], '/no-auth', []);
             expect(TestClass).toBeDefined();
         });
         it('should register the actual method as the handler', () => {
@@ -265,7 +262,7 @@ describe('Decorators', () => {
                 expect.objectContaining({ name: 'boolParam', type: Models_1.ParameterType.Boolean }),
                 expect.objectContaining({ name: 'listParam', type: Models_1.ParameterType.List }),
                 expect.objectContaining({ name: 'dictParam', type: Models_1.ParameterType.Dictionary })
-            ]), '/multi-type', [], false);
+            ]), '/multi-type', []);
         });
     });
     describe('@interaction decorator', () => {
@@ -361,7 +358,7 @@ describe('Decorators', () => {
                     description: 'String param',
                     required: true
                 })
-            ]), '/mixed-tool', [], false);
+            ]), '/mixed-tool', []);
             expect(Service_1.toolsService.registerInteraction).toHaveBeenCalledWith('mixedInteraction', expect.any(Function), '/mixed-interaction');
         });
     });
@@ -394,7 +391,7 @@ describe('Decorators', () => {
                     scopeBundle: 'calendar',
                     required: true // Should default to true
                 })
-            ]), false);
+            ]));
         });
         it('should handle undefined parameters and authRequirements arrays', () => {
             const config = {
@@ -413,8 +410,8 @@ describe('Decorators', () => {
             ], TestClass.prototype, "undefinedArraysTool", null);
             expect(TestClass).toBeDefined();
             expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('undefinedArraysTool', 'Tool with undefined arrays', expect.any(Function), [], // Should be empty array when parameters is undefined
-            '/undefined-arrays', [], // Should be empty array when authRequirements is undefined
-            false);
+            '/undefined-arrays', [] // Should be empty array when authRequirements is undefined
+            );
         });
     });
     describe('handler instance context', () => {
@@ -527,196 +524,5 @@ describe('Decorators', () => {
             expect(TestClass).toBeDefined();
         });
     });
-    describe('global tool registration', () => {
-        it('should auto-detect global tools when class extends GlobalToolFunction', () => {
-            const config = {
-                name: 'autoGlobalTool',
-                description: 'An auto-detected global tool',
-                parameters: [],
-                endpoint: '/auto-global-tool'
-            };
-            class TestGlobalClass extends GlobalToolFunction {
-                async testTool() {
-                    return { result: 'global-tool-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(config)
-            ], TestGlobalClass.prototype, "testTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('autoGlobalTool', 'An auto-detected global tool', expect.any(Function), [], '/auto-global-tool', [], true);
-            expect(TestGlobalClass).toBeDefined();
-        });
-        it('should register regular tools when class does not extend GlobalToolFunction', () => {
-            const config = {
-                name: 'regularTool',
-                description: 'A regular tool',
-                parameters: [],
-                endpoint: '/regular-tool'
-            };
-            class TestRegularClass {
-                async testTool() {
-                    return { result: 'regular-tool-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(config)
-            ], TestRegularClass.prototype, "testTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('regularTool', 'A regular tool', expect.any(Function), [], '/regular-tool', [], false);
-            expect(TestRegularClass).toBeDefined();
-        });
-        it('should register tools as global when import context is set', () => {
-            // Set global import context
-            globalThis.__IMPORT_CONTEXT = 'global';
-            const config = {
-                name: 'importedGlobalTool',
-                description: 'A tool imported as global',
-                parameters: [],
-                endpoint: '/imported-global-tool'
-            };
-            class TestImportedClass {
-                async testTool() {
-                    return { result: 'imported-global-tool-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(config)
-            ], TestImportedClass.prototype, "testTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('importedGlobalTool', 'A tool imported as global', expect.any(Function), [], '/imported-global-tool', [], true);
-            // Clean up
-            globalThis.__IMPORT_CONTEXT = undefined;
-            expect(TestImportedClass).toBeDefined();
-        });
-        it('should register tools as regular when import context is set to regular', () => {
-            // Set regular import context
-            globalThis.__IMPORT_CONTEXT = 'regular';
-            const config = {
-                name: 'importedRegularTool',
-                description: 'A tool imported as regular',
-                parameters: [],
-                endpoint: '/imported-regular-tool'
-            };
-            class TestImportedClass {
-                async testTool() {
-                    return { result: 'imported-regular-tool-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(config)
-            ], TestImportedClass.prototype, "testTool", null);
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('importedRegularTool', 'A tool imported as regular', expect.any(Function), [], '/imported-regular-tool', [], false);
-            // Clean up
-            globalThis.__IMPORT_CONTEXT = undefined;
-            expect(TestImportedClass).toBeDefined();
-        });
-        it('should handle mixed imports - both regular and global tools together', () => {
-            // Test importing regular tools first
-            globalThis.__IMPORT_CONTEXT = 'regular';
-            const regularConfig = {
-                name: 'mixedRegularTool',
-                description: 'A regular tool in mixed scenario',
-                parameters: [],
-                endpoint: '/mixed-regular-tool'
-            };
-            class RegularToolClass {
-                async regularTool() {
-                    return { result: 'mixed-regular-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(regularConfig)
-            ], RegularToolClass.prototype, "regularTool", null);
-            // Switch to global context
-            globalThis.__IMPORT_CONTEXT = 'global';
-            const globalConfig = {
-                name: 'mixedGlobalTool',
-                description: 'A global tool in mixed scenario',
-                parameters: [],
-                endpoint: '/mixed-global-tool'
-            };
-            class GlobalToolClass {
-                async globalTool() {
-                    return { result: 'mixed-global-result' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)(globalConfig)
-            ], GlobalToolClass.prototype, "globalTool", null);
-            // Clear context
-            globalThis.__IMPORT_CONTEXT = undefined;
-            // Verify regular tool was registered as regular
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('mixedRegularTool', 'A regular tool in mixed scenario', expect.any(Function), [], '/mixed-regular-tool', [], false);
-            // Verify global tool was registered as global
-            expect(Service_1.toolsService.registerTool).toHaveBeenCalledWith('mixedGlobalTool', 'A global tool in mixed scenario', expect.any(Function), [], '/mixed-global-tool', [], true);
-            expect(RegularToolClass).toBeDefined();
-            expect(GlobalToolClass).toBeDefined();
-        });
-        it('should handle context switching correctly within same test', () => {
-            let callCount = 0;
-            const originalRegisterTool = jest.mocked(Service_1.toolsService.registerTool);
-            // Track call order and contexts
-            originalRegisterTool.mockImplementation(() => {
-                callCount++;
-                // Store the isGlobal flag from each call for verification
-            });
-            // First: Import as regular
-            globalThis.__IMPORT_CONTEXT = 'regular';
-            class FirstClass {
-                async tool1() {
-                    return { result: 'test1' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)({
-                    name: 'contextTest1',
-                    description: 'First context test',
-                    parameters: [],
-                    endpoint: '/context-test-1'
-                })
-            ], FirstClass.prototype, "tool1", null);
-            // Second: Switch to global
-            globalThis.__IMPORT_CONTEXT = 'global';
-            class SecondClass {
-                async tool2() {
-                    return { result: 'test2' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)({
-                    name: 'contextTest2',
-                    description: 'Second context test',
-                    parameters: [],
-                    endpoint: '/context-test-2'
-                })
-            ], SecondClass.prototype, "tool2", null);
-            // Third: Clear context (should fallback to class inheritance)
-            globalThis.__IMPORT_CONTEXT = undefined;
-            class ThirdClass {
-                async tool3() {
-                    return { result: 'test3' };
-                }
-            }
-            __decorate([
-                (0, Decorator_1.tool)({
-                    name: 'contextTest3',
-                    description: 'Third context test',
-                    parameters: [],
-                    endpoint: '/context-test-3'
-                })
-            ], ThirdClass.prototype, "tool3", null);
-            // Verify the sequence of calls
-            expect(Service_1.toolsService.registerTool).toHaveBeenNthCalledWith(callCount - 2, // First call
-            'contextTest1', 'First context test', expect.any(Function), [], '/context-test-1', [], false // Should be regular
-            );
-            expect(Service_1.toolsService.registerTool).toHaveBeenNthCalledWith(callCount - 1, // Second call
-            'contextTest2', 'Second context test', expect.any(Function), [], '/context-test-2', [], true // Should be global
-            );
-            expect(Service_1.toolsService.registerTool).toHaveBeenNthCalledWith(callCount, // Third call
-            'contextTest3', 'Third context test', expect.any(Function), [], '/context-test-3', [], false // Should be regular (fallback to class check, ThirdClass doesn't extend GlobalToolFunction)
-            );
-            expect(FirstClass).toBeDefined();
-            expect(SecondClass).toBeDefined();
-            expect(ThirdClass).toBeDefined();
-        });
-    });
 });
 //# sourceMappingURL=Decorator.test.js.map