Skip to content

refactor(mcp): port service business logic in Go - #96

Merged
couragehong merged 11 commits into
CryptoLabInc:feat/go-migrationfrom
esifea:refactor/esifea/go-service-orchestration
May 6, 2026
Merged

couragehong merged 11 commits into
CryptoLabInc:feat/go-migrationfrom
esifea:refactor/esifea/go-service-orchestration

Conversation

@esifea

@esifea esifea commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • What changed: Service orchestration
  • Why:
  • Scope: internal/service, internal/domain

Validation

  • Tests run (or explain why not):
  • Docs updated (if behavior/setup changed)

Cross-Agent Invariants

  • scripts/bootstrap-mcp.sh remains the single source of truth for runtime prep (venv/deps/self-heal)
  • No agent-specific script duplicates bootstrap/setup logic
  • Agent-specific scripts remain thin adapters (registration/wiring only)
  • Codex-only commands (codex mcp ...) are clearly separated from cross-agent/common instructions
  • Claude/Gemini/OpenAI instructions do not include Codex-only commands
  • SKILL.md, commands/rune/*.toml, and AGENT_INTEGRATION.md stay consistent on boundaries

Notes for Reviewers

  • Risk areas: go.mod and go.sum might be duplicated with other PR
  • Backward compatibility impact:
  • Follow-up work (if any):

Comment thread internal/service/capture.go
Comment thread internal/service/capture.go
Comment thread internal/service/capture.go
Comment thread internal/service/diagnostics_classify.go
Comment thread internal/service/lifecycle.go
Comment thread internal/service/lifecycle.go Outdated
Comment thread internal/service/lifecycle.go Outdated
@esifea
esifea force-pushed the refactor/esifea/go-service-orchestration branch from d76d40b to 829fec9 Compare May 6, 2026 10:23
Comment thread internal/service/lifecycle.go Outdated
Comment thread internal/service/lifecycle.go Outdated
Comment thread internal/service/recall.go
@couragehong
couragehong merged commit 6d2a85a into CryptoLabInc:feat/go-migration May 6, 2026
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