feat(#741): unify tool registry — single datamachine_tools filter with context arrays

[chubes4]

Summary

Phase 1 of #741: Replace the 3 separate tool registration channels with a single unified datamachine_tools filter. Every tool now declares a contexts array specifying where it's available.

• Single filter: datamachine_tools replaces datamachine_global_tools, datamachine_chat_tools, datamachine_system_tools
• Context arrays: Each tool declares contexts: ['chat', 'pipeline', 'standalone'] (or subset)
• No backward compatibility bridge — all callers updated directly

Changes

BaseTool.php

• registerTool(toolName, toolDefinition, contexts) replaces the old 3-param (agentType, toolName, toolDefinition) signature
• Deleted registerGlobalTool() and registerChatTool()
• Callable definitions wrapped with _callable key to keep contexts available pre-resolution

ToolManager.php

• get_all_tools() reads from datamachine_tools filter (replaces get_global_tools())
• get_global_tools() kept as deprecated alias
• get_raw_tools() and get_tool_contexts() added for pre-resolution context checks
• resolveToolDefinition() handles _callable wrapper format, merging contexts into resolved result

ToolPolicyResolver.php

• filterByContext() helper filters resolved tools by context string
• gatherPipelineTools(): filters to 'pipeline' context + handler tools from chubes_ai_tools
• gatherChatTools(): filters to 'chat' context with availability/enablement checks
• gatherStandaloneTools(): filters to 'standalone' context
• gatherSystemTools(): filters to 'system' context (currently empty — no tools register with system context)
• Surface constants kept for Phase 2 rename

ToolServiceProvider.php

• Merged registerGlobalTools() + registerChatTools() into single registerTools()

46+ tool registration sites updated

• Former global tools → contexts: ['chat', 'pipeline', 'standalone']
• Former chat-only tools → contexts: ['chat']
• Ability-registered tools (PostQuery, GetPostBlocks, EditPostBlocks, ReplacePostBlocks) → datamachine_tools filter with 'chat' context

Tests

• ToolPolicyResolverTest: system surface test updated (no tools have system context now)
• ImageGenerationTest: datamachine_global_tools → datamachine_tools
• ListFlowsTest: datamachine_chat_tools → datamachine_tools with _callable unwrapping

Test Results

764 tests, 0 failures, 3 skipped (pre-existing)

What's NOT in this PR (Phase 2)

• Renaming SURFACE_* constants to CONTEXT_* in ToolPolicyResolver
• Full ToolPolicyResolver rewrite with context-first architecture

[github-actions]

Homeboy Results — data-machine

Lint

Tooling versions

• Homeboy CLI: homeboy 0.72.0
• Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
• Extension revision: unknown
• Action: Extra-Chill/homeboy-action@v1

ℹ️ PR test scope resolved to full for compatibility with installed Homeboy CLI

✅ lint (changed files only)

• PHPCS: LINT SUMMARY: 0 errors, 5 warnings
• Fixable: 1 | Files with issues: 3 of 56
• PHPStan: PHPSTAN SUMMARY: 204 errors at level 5

Test

Tooling versions

• Homeboy CLI: homeboy 0.72.0
• Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
• Extension revision: unknown
• Action: Extra-Chill/homeboy-action@v1

⚡ PR test scope resolved to changed

✅ test

• PHPCS: LINT SUMMARY: 65 errors, 57 warnings
• Fixable: 103 | Files with issues: 19 of 368

Top violations

  WordPress.Arrays.ArrayDeclarationSpacing.ArrayItemNoNewLine    51
  Generic.Formatting.MultipleStatementAlignment.NotSameWarning    31
  WordPress.Arrays.MultipleStatementAlignment.DoubleArrowNotAligned    17
  Universal.Operators.DisallowShortTernary.Found              4
  WordPress.DB.PreparedSQL.NotPrepared                        4

- PHPStan: PHPSTAN SUMMARY: 204 errors at level 5 - OK (470 tests, 1535 assertions)

Audit

Tooling versions

• Homeboy CLI: homeboy 0.72.0
• Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
• Extension revision: unknown
• Action: Extra-Chill/homeboy-action@v1

ℹ️ PR test scope resolved to full for compatibility with installed Homeboy CLI

✅ audit (changed files only)

• Actionable audit summary:
• Alignment score: 0.701
• Severity counts: info: 13, unknown: 76, warning: 78
• Drift increased: no
• Outliers in current run: 76
• Parsed outlier entries: 76
• Top actionable findings:
  1. tests/Unit/Abilities/SystemAbilitiesTest.php — missing_registration — Missing registration: datamachine_cli_bypass_permissions
  2. tests/Unit/Abilities/PermissionHelperTest.php — missing_method — Missing method: set_up
  3. tests/Unit/Abilities/PermissionHelperTest.php — missing_registration — Missing registration: datamachine_cli_bypass_permissions
  4. tests/Unit/Abilities/ImageGenerationPromptRefinementTest.php — missing_registration — Missing registration: datamachine_cli_bypass_permissions
  5. tests/Unit/Abilities/ImageGenerationAbilitiesTest.php — missing_registration — Missing registration: datamachine_cli_bypass_permissions

---

Homeboy Action v1