Skip to content

feat: prototype actor mailbox collaboration via MCP and skills - #79

Merged
reiase merged 11 commits into
mainfrom
feat/actor-mailbox-collaboration
Nov 7, 2026
Merged

reiase merged 11 commits into
mainfrom
feat/actor-mailbox-collaboration

Conversation

@reiase

@reiase reiase commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Prototype agent coordination on Pulsing actors, inspired by DeepSeek's public dsh-superagent-prototype domain/architecture (reference commit 8c32a9dc5d72c033816c6a8110b37f40cf94fba4). This is original Python implementation, not a port or a reintroduction of Forge/LLM serving.

  • One durable mailbox actor per router/worker/subagent/reviewer/observer, plus an actor role directory.
  • unread → read → processing → completed/failed, explicit read semantics, sender receipts, thread/reply correlation.
  • SQLite persistence, idempotent delivery/finish, bounded inboxes, expiring claims and stale-attempt fencing.
  • Optional stdio MCP adapter with identity-bound tools, resources, prompt, and packaged/repository skills.
  • Runnable router → worker → subagent/reviewer demo; real MCP SDK subprocess/network/restart tests.
  • Dedicated optional-MCP CI job; allow checks on stacked PRs.

Scope and trust

Trusted local/private cluster only. MCP identity binding is not authentication. Mail never grants execution/merge authority. External harnesses start model sessions; creating a mailbox does not launch an agent. No SMTP, public MCP, model integration, automatic execution, HA, or exactly-once side effects.

Validation

  • Full Python suite including chaos: 612 passed, 15 skipped (RustPython CLI parity now also runs).
  • Mailbox suite: 20 passed, including real stdio MCP tools/resources/prompts, networked actor communication, host restart, idempotency and concurrent claims.
  • Actor mailbox collaboration demo completed with all four states and reviewed result.
  • Wheel built and checked to include the installable SKILL.md.
  • Python lint/format checks passed.

Base and CI follow-up

#78 was merged on the repository side while this work was in progress. This PR now targets main and includes the subsequent CI corrections: propagate installer/pytest exit codes, use an interpreter-specific virtual environment, and replace the CLI parameter dependency (which required a Rust source build on ARM64) with standard-library argparse. No automatic merge was performed by the agent.

CI builds and tests on native GitHub ARM64 runners as well as x86-64 and macOS; superseded emulated runs were cancelled, not counted as passing checks.

Additional CI validation: the exact just ci-test 3.12 wheel runner passed 610 tests with 16 skips in an isolated environment without the optional MCP SDK. The 20 mailbox tests (including actual MCP subprocesses) pass with the SDK installed. Regression tests verify that environment creation, installer and pytest failures propagate; the chosen virtualenv is also exported to subprocesses.

@codecov-commenter

codecov-commenter commented Nov 7, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 68.22917% with 122 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
python/pulsing/mailbox/mcp.py 0.00% 60 Missing ⚠️
python/pulsing/mailbox/__main__.py 0.00% 49 Missing ⚠️
python/pulsing/mailbox/actors.py 93.50% 10 Missing ⚠️
python/pulsing/cli/__main__.py 95.94% 3 Missing ⚠️
Files with missing lines Coverage Δ
python/pulsing/mailbox/__init__.py 100.00% <100.00%> (ø)
python/pulsing/mailbox/client.py 100.00% <100.00%> (ø)
python/pulsing/cli/__main__.py 71.23% <95.94%> (+30.80%) ⬆️
python/pulsing/mailbox/actors.py 93.50% <93.50%> (ø)
python/pulsing/mailbox/__main__.py 0.00% <0.00%> (ø)
python/pulsing/mailbox/mcp.py 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from refactor/actor-only-distribution to main November 7, 2026 03:23
@reiase

reiase commented Nov 7, 2026

Copy link
Copy Markdown
Contributor Author

CI guarding complete for commit 0bc4483: all 27 checks are successful. Latest CI run: https://github.com/DeepLink-org/Pulsing/actions/runs/42336394991 . This includes the dedicated real MCP/actor suite, native ARM64 builds/tests, Fedora, macOS, Ubuntu, Python matrices, lint/Clippy, coverage and CodeQL. The legacy false-green wheel runner is corrected; installation/pytest failures are now fatal and subprocesses inherit the selected virtualenv. No automatic merge or package publication was performed.

@reiase
reiase merged commit 244ec29 into main Nov 7, 2026
27 checks passed
@reiase
reiase deleted the feat/actor-mailbox-collaboration branch November 7, 2026 13:18
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.

2 participants