Skip to content

Add MCP connection manager contract#2197

Open
chubes4 wants to merge 1 commit into
mainfrom
issue-1043-mcp-manager
Open

Add MCP connection manager contract#2197
chubes4 wants to merge 1 commit into
mainfrom
issue-1043-mcp-manager

Conversation

@chubes4
Copy link
Copy Markdown
Member

@chubes4 chubes4 commented May 22, 2026

Summary

  • Adds a generic MCP server registry and request-local connection manager contract in Data Machine core.
  • Provides public helpers for registering servers, reading redacted config/state, connecting through an adapter hook, restarting, and cleanup.
  • Keeps the slice transport-agnostic because this repo does not include a concrete MCP client/server package or transport runtime.

Refs #1043.

Testing

  • php tests/mcp-connection-manager-smoke.php
  • composer lint -- --standard=phpcs.xml.dist inc/Engine/MCP tests/mcp-connection-manager-smoke.php
  • php -l inc/Engine/MCP/MCPServerRegistry.php
  • php -l inc/Engine/MCP/MCPConnectionManager.php
  • php -l inc/Engine/MCP/functions.php
  • php -l tests/mcp-connection-manager-smoke.php

AI assistance

@homeboy-ci
Copy link
Copy Markdown
Contributor

homeboy-ci Bot commented May 22, 2026

Homeboy Results — data-machine

Lint

lint — failed

  • formatting — 1 finding(s)
  • Total: 1 finding(s)

ℹ️ Auto-fix: homeboy lint data-machine --path /home/runner/work/data-machine/data-machine --changed-since 9caa8a9 --fix (or homeboy refactor data-machine --path /home/runner/work/data-machine/data-machine --changed-since 9caa8a9 --from lint --write)
ℹ️ Some issues may require manual fixes
ℹ️ Full options: homeboy docs commands/lint
Deep dive: homeboy lint data-machine --changed-since 9caa8a9

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-lint-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-lint-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine/actions/runs/26316967373

Test

test — passed

ℹ️ Auto-fix lint issues: homeboy refactor data-machine --from lint --write
ℹ️ Collect coverage: homeboy test data-machine --coverage
ℹ️ Pass args to test runner: homeboy test -- [args]
ℹ️ Full options: homeboy docs commands/test
Deep dive: homeboy test data-machine --changed-since 9caa8a9

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-test-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-test-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine/actions/runs/26316967373

Audit

audit — passed

  • requested_detectors — 2 finding(s)
  • Total: 2 finding(s)

Deep dive: homeboy audit data-machine --changed-since 9caa8a9

Artifacts and drill-down
  • CI results artifact: homeboy-ci-results-data-machine-audit-quality-Linux-node24 contains immediate command JSON for this action invocation.
  • Observation artifact: homeboy-observations-data-machine-audit-quality-Linux-node24 contains exported Homeboy run history for deeper queries.
  • Drill-down: download the observation artifact, then run homeboy runs import <dir>, homeboy runs list, and homeboy runs findings <run-id>.
  • Artifacts are attached to the workflow run: https://github.com/Extra-Chill/data-machine/actions/runs/26316967373
Tooling versions
  • Homeboy CLI: homeboy 0.197.2+4884f169
  • Extension: wordpress from https://github.com/Extra-Chill/homeboy-extensions
  • Extension revision: dd23ee80
  • Action: unknown@unknown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant