Skip to content

fix(mcp): share app dependencies with stdio#29

Merged
hubab1 merged 1 commit into
mainfrom
17b-mcp-stdio-dependencies
Jul 26, 2026
Merged

fix(mcp): share app dependencies with stdio#29
hubab1 merged 1 commit into
mainfrom
17b-mcp-stdio-dependencies

Conversation

@hubab1

@hubab1 hubab1 commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • make the shipped app's --mcp-stdio path use the same AppServices-composed server as local HTTP
  • carry the workspace, keyword-research, Apple Ads, request-gate, ranking, review, and pricing dependencies into stdio
  • preserve crash-safe persistent-store startup and return a diagnostic stderr failure if the store cannot open
  • make transport shutdown cancellation-safe and correct the Settings guidance

Validation

  • 11 focused OpenASOMCPServerTests passed
  • 624 full tests passed across 54 suites on current main
  • real built-app stdio initialize handshake returned one JSON-RPC response
  • process-level handshake produced no extra stdout, zero stderr bytes, and exited successfully on EOF
  • Swift 6.2 complete strict-concurrency build passed
  • git diff --check passed

Source

Independently implemented from ideas in Felix Lisczyk fork commits b05eb34 and 3ad5b5f. These were fork commits rather than an upstream pull request, so this PR uses source links and no co-author trailer.

@hubab1
hubab1 force-pushed the 08b-provider-gate-wiring branch 4 times, most recently from 475386c to d860524 Compare July 20, 2026 07:25
@hubab1
hubab1 force-pushed the 17b-mcp-stdio-dependencies branch from 1a8b1c1 to 386280c Compare July 26, 2026 12:16
@hubab1
hubab1 changed the base branch from 08b-provider-gate-wiring to main July 26, 2026 12:16
@hubab1
hubab1 marked this pull request as ready for review July 26, 2026 12:16
@hubab1 hubab1 closed this Jul 26, 2026
@hubab1 hubab1 reopened this Jul 26, 2026
@hubab1
hubab1 merged commit b90e5e1 into main Jul 26, 2026
1 check passed
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