Enhance agentic features, fix bugs, and improve documentation by Snider · Pull Request #12 · dAppCore/agent

Snider · 2026-06-29T15:32:30Z

This pull request introduces several important configuration and documentation updates to the core-agent project, focusing on improving agent concurrency and rate limiting, updating plugin and MCP server integration, clarifying architecture and operational patterns, and adding new external dependencies. The changes also include a comprehensive implementation plan for aligning CLI and IPC handler behavior.

Configuration and Infrastructure Updates:

Updated the core agent version in .claude-plugin/marketplace.json from 0.14.0 to 0.18.0.
Changed the MCP server configuration in .mcp.json from a stdio-based server to an HTTP-based server, now requiring an authorization header.
Updated workspace root in .core/agents.yaml to resolve against ~/Lethean instead of $HOME/Code/.core, and clarified the comment.

Agent Concurrency and Rate Limiting Enhancements:

Added concurrency and rate limit configuration for the opencode agent in .core/agents.yaml, including per-model sub-limits and pacing settings. [1] [2]

Dependency Management:

Added new submodules to .gitmodules: external/api, external/cli, external/orm, and external/go-container, all tracking their respective dev branches.

Documentation and Architecture Clarifications:

Updated CLAUDE.md and AGENTS.md to reflect the current architecture, including new packages (pkg/runner, pkg/lemma, pkg/chathistory), expanded binary modes, and clarified plugin provider roles and personas/templates structure. [1] [2] [3] [4] [5] [6] [7]
Added a detailed implementation plan in PLAN-cli-square-up.md, outlining the correct approach for CLI and IPC handler integration, anti-patterns to avoid, and the status of event handler implementations.

The core plugin couldn't be installed/loaded as shipped: - provider/claude/core/.mcp.json was a bare { core: {...} } — Claude Code needs the { mcpServers: {...} } wrapper or the MCP server won't load. Added it (HTTP :9202 + Bearer ${MCP_AUTH_TOKEN}, the connect-not-spawn shape). - plugin.json name was 'agent' while the marketplace entry is 'core' — /plugin install core@core-agent silently fails on a name mismatch. Aligned to 'core'. - marketplace.json core version 0.14.0 → 0.18.0 to match the manifest. - removed the stray stdio dupes 000.mcp.json + 000mcp.json (superseded by the HTTP plane). Co-Authored-By: Virgil <virgil@lethean.io>

Advance external/go workspace submodule to v0.10.4 so dev (GOWORK on) and standalone (GOWORK=off) builds resolve the same core/go. Co-Authored-By: Virgil <virgil@lethean.io>

…leak loadAgentsConfig() sets a package-global workspace-root override from agents.yaml dispatch.workspace_root. A full-core test (newCoreAgent) leaves it set, so a later test relying on CORE_WORKSPACE resolved the leaked path instead — TestCommands_Check_Good_BranchWorkspaceCount failed in-suite but passed alone. Export SetWorkspaceRootOverride and reset it in newTestCore so each test starts from clean workspace resolution (mirroring the agentic package's own internal test guard). Co-Authored-By: Virgil <virgil@lethean.io>

Real per-symbol tests for the testable helpers: defaultHubTokenFile, defaultHubAuditPath, optStringOr, publicSuffix, auditMetaString, toBytes, hostIsLoopback (commands_hub.go); defaultUserChatsPath + the chat --user guard (commands_chat.go); detectBinaryName (main.go). Co-Authored-By: Virgil <virgil@lethean.io>

CI already uploads coverage to Codecov (.github/workflows/ci.yml) but had no codecov.yml, so there was no gate to "start using". Add one configured as a ratchet: project status target=auto (a PR may not drop total coverage below the base, 1% noise threshold) + patch target=70% (new/changed lines must be reasonably covered, pulling the number up as files are touched). Ignore tests, examples, and the vendored externals. `task cov` writes go/coverage.out (the path codecov + SonarCloud both read) and prints the total, for local runs. Co-Authored-By: Virgil <virgil@lethean.io>

Good/Bad/Ugly for the pure map-walk helper: nested path resolves to the terminal string; missing key at any step yields ""; non-map intermediate or non-string terminal yields "". Co-Authored-By: Virgil <virgil@lethean.io>

…> 39.2%) Add newTestService — builds a real opencode Service against a temp-$HOME DuckDB KV store, resetting the process-global kvOnce before+after (the cross-test hazard host_config_mode_test.go documented but sidestepped). auth.go and enable.go had no test files at all (hence 0%). With the harness: - auth.go: ServerPassword, InstallID, authHeader (+ SeedDefaultProfile) - enable.go: IsEnabled default-false, setEnabled round-trip - host_config.go: MergeHostConfig create + idempotent re-merge Co-Authored-By: Virgil <virgil@lethean.io>

….8%) Two table tests over the previously-0% command surface: the 5 pipeline routers (empty/help action → OK, unknown action → error) and the 16 leaf placeholders (not-yet-implemented). No side effects — these print help and return. Co-Authored-By: Virgil <virgil@lethean.io>

cmdPlanShow's slug guard (usage + error) and cmdPlanList against an empty workspace ("no plans" → OK). Read-only, no side effects. Co-Authored-By: Virgil <virgil@lethean.io>

Both card-listers, both lanes (human + --json), always-OK. Co-Authored-By: Virgil <virgil@lethean.io>

Pure option->input mappers (resume/scan/watch/mirror, incl. watch's single- workspace fallback) and brainValuePresent (nil/empty/whitespace absent, non-empty present). Co-Authored-By: Virgil <virgil@lethean.io>