refactor: remove adapter files, use direct tool variants in categories

Remove the runtime-dispatching adapter layer now that buildCategories(uiMode)
resolves the correct tool variants at build time:

- Delete src/tools/actor.ts (call-actor adapter)
- Delete src/tools/store_collection.ts (search-actors adapter)
- Delete src/tools/fetch-actor-details.ts (fetch-actor-details adapter)
- Remove get-actor-run adapter from common/run.ts (keep mode-independent tools)
- Update categories.ts to use default variants directly in static toolCategories
- Update tools/index.ts to import from core modules instead of adapter

Author: MQ37

src/tools/actor.ts

@@ -9,7 +9,6 @@
  * which also auto-injects get-actor-run and get-actor-output right after call-actor.
  */
 import type { ToolEntry, UiMode } from '../types.js';
-import { callActor } from './actor.js';
 import { getDataset, getDatasetItems, getDatasetSchema } from './common/dataset.js';
 import { getUserDatasetsList } from './common/dataset_collection.js';
 import { fetchApifyDocsTool } from './common/fetch-apify-docs.js';
@@ -18,21 +17,19 @@ import { getHtmlSkeleton } from './common/get-html-skeleton.js';
 import { addTool } from './common/helpers.js';
 import { getKeyValueStore, getKeyValueStoreKeys, getKeyValueStoreRecord } from './common/key_value_store.js';
 import { getUserKeyValueStoresList } from './common/key_value_store_collection.js';
-import { abortActorRun, getActorRun, getActorRunLog } from './common/run.js';
+import { abortActorRun, getActorRunLog } from './common/run.js';
 import { getUserRunsList } from './common/run_collection.js';
 import { searchApifyDocsTool } from './common/search-apify-docs.js';
 import { defaultCallActor } from './default/call-actor.js';
 import { defaultFetchActorDetails } from './default/fetch-actor-details.js';
 import { defaultGetActorRun } from './default/get-actor-run.js';
 import { defaultSearchActors } from './default/search-actors.js';
-import { fetchActorDetailsTool } from './fetch-actor-details.js';
 import { openaiCallActor } from './openai/call-actor.js';
 import { openaiFetchActorDetails } from './openai/fetch-actor-details.js';
 import { fetchActorDetailsInternalTool } from './openai/fetch-actor-details-internal.js';
 import { openaiGetActorRun } from './openai/get-actor-run.js';
 import { openaiSearchActors } from './openai/search-actors.js';
 import { searchActorsInternalTool } from './openai/search-actors-internal.js';
-import { searchActors } from './store_collection.js';
 
 /**
  * Static tool categories using adapter tools that dispatch at runtime based on uiMode.
@@ -46,9 +43,9 @@ export const toolCategories = {
         addTool,
     ],
     actors: [
-        searchActors,
-        fetchActorDetailsTool,
-        callActor,
+        defaultSearchActors,
+        defaultFetchActorDetails,
+        defaultCallActor,
     ],
     ui: [
         searchActorsInternalTool,
@@ -59,7 +56,7 @@ export const toolCategories = {
         fetchApifyDocsTool,
     ],
     runs: [
-        getActorRun,
+        defaultGetActorRun,
         getUserRunsList,
         getActorRunLog,
         abortActorRun,

src/tools/categories.ts

@@ -1,39 +1,17 @@
 /**
- * Actor run tools — get-actor-run (adapter), get-actor-run-log, and abort-actor-run.
+ * Mode-independent Actor run tools: get-actor-run-log and abort-actor-run.
  *
- * The get-actor-run tool has been split into mode-specific variants:
+ * The get-actor-run tool has mode-specific variants in:
  * - `default/get-actor-run.ts` — full JSON dump without widget metadata
  * - `openai/get-actor-run.ts` — abbreviated text with widget metadata
- * - `core/get-actor-run-common.ts` — shared schema, metadata, and data-fetching logic
- *
- * The getActorRunLog and abortActorRun tools are mode-independent and remain here.
- * PR #4 will wire variants directly into the category registry, making the adapter unnecessary.
+ * These are resolved via buildCategories(uiMode) in tools-loader.ts.
  */
 import { z } from 'zod';
 
 import { createApifyClientWithSkyfireSupport } from '../../apify-client.js';
 import { HelperTools } from '../../const.js';
-import type { HelperTool, InternalToolArgs, ToolEntry, ToolInputSchema } from '../../types.js';
+import type { InternalToolArgs, ToolEntry, ToolInputSchema } from '../../types.js';
 import { compileSchema } from '../../utils/ajv.js';
-import { defaultGetActorRun } from '../default/get-actor-run.js';
-import { openaiGetActorRun } from '../openai/get-actor-run.js';
-
-const defaultVariant = defaultGetActorRun as HelperTool;
-
-/**
- * Adapter get-actor-run tool that dispatches to the correct mode-specific variant at runtime.
- */
-export const getActorRun: ToolEntry = Object.freeze({
-    ...defaultVariant,
-    call: async (toolArgs: InternalToolArgs) => {
-        const variant = (toolArgs.apifyMcpServer.options.uiMode === 'openai'
-            ? openaiGetActorRun
-            : defaultGetActorRun) as HelperTool;
-        return variant.call(toolArgs);
-    },
-});
-
-// --- Mode-independent tools below ---
 
 const GetRunLogArgs = z.object({
     runId: z.string().describe('The ID of the Actor run.'),

src/tools/common/run.ts

@@ -1,8 +1,9 @@
 import { HelperTools } from '../const.js';
 import type { ToolCategory } from '../types.js';
 import { getExpectedToolsByCategories } from '../utils/tool-categories-helpers.js';
-import { callActorGetDataset, getActorsAsTools } from './actor.js';
 import { buildCategories, CATEGORY_NAMES, toolCategories, toolCategoriesEnabledByDefault } from './categories.js';
+import { callActorGetDataset } from './core/actor-execution.js';
+import { getActorsAsTools } from './core/actor-tools-factory.js';
 
 // Use string constants instead of importing tool objects to avoid circular dependency
 export const unauthEnabledTools: string[] = [