refactor: Regroup src/tools by domain

src/tools/ was grouped by three inconsistent axes: common/ (mode-independence),
core/ (shared logic — the opposite meaning under a synonym), and default/+apps/
(a mode split that, post-#975, only exists for the 4 *-widget tools). Helper names
misled too: build.ts fetches Actor definitions; *_common.ts double-marked "shared".

Regroup into domain directories matching the toolCategories vocabulary —
actors/ runs/ storage/ docs/ widgets/ — dissolving common/, core/, default/, apps/.
Rename categories.ts -> registry.ts and build.ts -> actors/actor_definition.ts.
Pure git mv plus import-path fixes; no logic, export, or schema changes. utils.ts
and structured_output_schemas.ts stay top-level.

The *_common.ts shared-logic files keep their suffix inside the new domain dirs:
dropping it collides with the sibling tool-entry file (e.g. search_actors.ts), and
merging the two would be a logic change, not a move. Export/file-name normalization
and any _common merge are left to the follow-up naming PR (#977).

Anchor the .gitignore "storage" rule to "/storage" so it ignores only the repo-root
Apify local-storage dir, not the new src/tools/storage/ source directory.

Closes #978.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HbBNrt49Pp92M9ebUwoWu4

Author: claude

.gitignore

@@ -4,7 +4,7 @@
 .vscode/*
 !.vscode/extensions.json
 !.vscode/settings.json
-storage
+/storage
 apify_storage
 crawlee_storage
 node_modules

src/index_internals.ts

@@ -8,7 +8,7 @@ import { processParamsGetTools } from './mcp/utils.js';
 import { resolvePaymentProvider } from './payments/index.js';
 import type { PaymentProvider } from './payments/types.js';
 import { getServerCard } from './server_card.js';
-import { addTool } from './tools/common/add_actor.js';
+import { addTool } from './tools/actors/add_actor.js';
 import {
     getActorsAsTools,
     getCategoryTools,

src/mcp/server.ts

@@ -63,11 +63,11 @@ import type { AvailableWidget } from '../resources/widgets.js';
 import { resolveAvailableWidgets } from '../resources/widgets.js';
 import { getServerInfo } from '../server_card.js';
 import { getTelemetryEnv, trackToolCall } from '../telemetry.js';
-import { actorExecutor } from '../tools/actor_executor.js';
+import { actorExecutor } from '../tools/actors/actor_executor.js';
 import {
     buildPermissionApprovalResponse,
     checkPaymentProviderStandbyConflict,
-} from '../tools/core/call_actor_common.js';
+} from '../tools/actors/call_actor_common.js';
 import { getActorsAsTools } from '../tools/index.js';
 import type { ActorsAsToolsResult } from '../tools/index.js';
 import { decodeDotPropertyNames, legacyToolNameToNew } from '../tools/utils.js';

src/tools/AGENTS.md

@@ -1,11 +1,11 @@
-import type { ApifyClient } from '../apify_client.js';
-import { ACTOR_README_MAX_LENGTH } from '../const.js';
+import type { ApifyClient } from '../../apify_client.js';
+import { ACTOR_README_MAX_LENGTH } from '../../const.js';
 import type {
     ActorDefinitionPruned,
     ActorDefinitionWithDesc,
     ActorDefinitionWithInfo,
     SchemaProperties,
-} from '../types.js';
+} from '../../types.js';
 
 /**
  * Get Actor input schema by Actor name.

src/tools/build.ts src/tools/actors/actor_definition.tssrc/tools/build.ts renamed to src/tools/actors/actor_definition.ts

@@ -1,10 +1,10 @@
 import log from '@apify/log';
 
-import type { ActorExecutionParams, ActorExecutionResult, ActorExecutor } from '../types.js';
-import { getConsoleLinkContext } from '../utils/console_link.js';
-import { redactSkyfirePayId } from '../utils/logging.js';
-import { abortRunOnSignal, CALL_ACTOR_WAIT_SECS_DEFAULT, fetchActorRunData } from './core/actor_run_response.js';
-import { buildGetActorRunSuccessResponse } from './core/get_actor_run_common.js';
+import type { ActorExecutionParams, ActorExecutionResult, ActorExecutor } from '../../types.js';
+import { getConsoleLinkContext } from '../../utils/console_link.js';
+import { redactSkyfirePayId } from '../../utils/logging.js';
+import { buildGetActorRunSuccessResponse } from '../runs/get_actor_run_common.js';
+import { abortRunOnSignal, CALL_ACTOR_WAIT_SECS_DEFAULT, fetchActorRunData } from './actor_run_response.js';
 
 /**
  * Direct actor tool executor. Mode-agnostic — used in both default and apps modes.

src/tools/actor_executor.ts src/tools/actors/actor_executor.tssrc/tools/actor_executor.ts renamed to src/tools/actors/actor_executor.ts

@@ -1,14 +1,14 @@
 import { HelperTools } from '../../const.js';
 import type { InternalToolArgs, ToolEntry } from '../../types.js';
 import { TOOL_TYPE } from '../../types.js';
+import { actorRunOutputSchema } from '../structured_output_schemas.js';
 import {
     buildCallActorAppsDescription,
     buildCallActorDescription,
     callActorAjvValidate,
     callActorInputSchema,
     executeCallActor,
-} from '../core/call_actor_common.js';
-import { actorRunOutputSchema } from '../structured_output_schemas.js';
+} from './call_actor_common.js';
 
 /**
  * Single call-actor definition shared by both modes — only the description differs