Skip to content

feat(setup): validate CommandCode integration candidate for PR #1123 - #1368

Closed
dnlrsls wants to merge 2 commits into
Gentleman-Programming:mainfrom
dnlrsls:feat/pr-1123-ci-candidate
Closed

dnlrsls wants to merge 2 commits into
Gentleman-Programming:mainfrom
dnlrsls:feat/pr-1123-ci-candidate

Conversation

@dnlrsls

@dnlrsls dnlrsls commented Sep 23, 2026

Copy link
Copy Markdown
Member

Linked Issue

Closes #1119

Temporary CI-only draft. Do not merge this PR. This branch validates the integration of contributor PR #1123 with current main; it does not replace or update #1123. After CI, the maintainer will decide how to hand the verified resolution back to the contributor PR.

PR Type

  • type:feature — New feature
  • type:bug — Bug fix
  • type:docs — Documentation only
  • type:refactor — Code refactoring
  • type:chore — Maintenance
  • type:breaking-change — Breaking change

Summary

  • Reconcile the eight conflicts between CommandCode PR feat(setup): add CommandCode CLI as supported agent #1123 (b994389) and upstream main (f2efd5f), retaining both CommandCode and Kimi setup.
  • Run Linux CI against the local merge candidate d872b87 before touching the contributor's PR branch.

Changes

Files Change
internal/setup/agents.go, internal/setup/registry.go, internal/setup/registry_test.go Register and test the CommandCode declarative MCP and memory adapter alongside existing agents.
cmd/engram/main.go, cmd/engram/main_test.go Include CommandCode in setup help and post-install tests.
README.md, docs/AGENT-SETUP.md, docs/INSTALLATION.md, docs/codebase/integrations.md Document the CommandCode integration alongside current main content.

Test Plan

  • Focused setup and CLI tests passed locally: go test ./internal/setup ./cmd/engram.
  • Server E2E tests passed locally: go test -tags e2e ./internal/server/....
  • Full go test ./... passes locally: Windows/MSYS plugin tests fail with the same named failures on a clean main baseline; this draft exists to check Linux CI on the integrated candidate.
  • Reviewed merge diff against exact main parent: only the original nine CommandCode paths (77 additions, 4 deletions).

Native review of the local candidate was approved and acknowledged. Review is not merge or delivery authorization. No changes to contributor PR #1123 are authorized by this draft.

eSagraAI and others added 2 commits September 20, 2026 12:51
Register CommandCode as a declarative agent so `engram setup commandcode` wires the MCP server and the Memory Protocol without a manual config edit. Writes mcpServers.engram to the user-scope ~/.commandcode/mcp.json using the {transport:stdio, enabled, command, args} shape verified against real `cmd mcp add --scope user` output, and the protocol as a marker block in the user-tier ~/.commandcode/AGENTS.md. Closes Gentleman-Programming#1119
@coderabbitai

coderabbitai Bot commented Sep 23, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@dnlrsls dnlrsls added the type:feature New feature label Sep 23, 2026
@dnlrsls dnlrsls closed this Sep 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(setup): add CommandCode CLI as supported agent

2 participants