Skip to content

ci(DAK-5438): remove MCP jobs from install-test — dk only#85

Merged
ferhimedamine merged 1 commit into
mainfrom
dak-5439-move-mcp-install-tests
May 21, 2026
Merged

ci(DAK-5438): remove MCP jobs from install-test — dk only#85
ferhimedamine merged 1 commit into
mainfrom
dak-5439-move-mcp-install-tests

Conversation

@ferhimedamine
Copy link
Copy Markdown
Contributor

Summary

  • Removes test-brew-dakera-mcp, test-npx, and all dakera-mcp steps from cargo-install/cargo-binstall jobs
  • This workflow now tests only dk distribution channels: brew, apt, yum, cargo install, cargo binstall
  • MCP tests have been moved to dakera-mcp in a parallel PR

Why

Founder directive DAK-5438: "MCP stuff on cli repo, a bit of spaghetti, keep it bound to corresponding repo."

Each product's CI workflows must live in that product's own repo. Cross-product CI jobs are banned per the REPO BOUNDARY RULE added to SHARED_PROTOCOLS.md, CTO AGENTS.md, Platform AGENTS.md, and Core Engine AGENTS.md.

Changes

Job Before After
test-brew-dk ✅ stays ✅ unchanged
test-apt ✅ stays ✅ unchanged
test-yum ✅ stays ✅ unchanged
test-brew-dakera-mcp ❌ was here moved to dakera-mcp
test-npx ❌ was here moved to dakera-mcp
test-cargo-install dk + mcp together dk only
test-cargo-binstall dk + mcp together dk only

Test plan

  • Workflow file validates (YAML lint)
  • On next weekly schedule (Monday 06:00 UTC): brew/apt/yum jobs should pass; cargo jobs continue-on-error until crates.io publish (DAK-5436)
  • No MCP-related jobs appear in dakera-cli runs

🤖 Generated with Claude Code

Move test-brew-dakera-mcp, test-npx, and dakera-mcp steps from
cargo-install/cargo-binstall jobs to dakera-mcp repo per the repo
boundary rule (DAK-5438). This workflow now tests dk distribution
channels only: brew, apt, yum, cargo install, cargo binstall.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@ferhimedamine ferhimedamine force-pushed the dak-5439-move-mcp-install-tests branch from 7997358 to 4b51257 Compare May 21, 2026 14:06
@ferhimedamine ferhimedamine merged commit 3feeed1 into main May 21, 2026
6 checks 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