Merged
Conversation
12 CortexClient DAG methods, 10 MCP tools, 10 CLI subcommands, 2 DAG resources, and dag-audit prompt for full audit trail, time-travel, and verifiable history on the knowledge graph. Production hardening: request timeouts on all MCP fetches, parallel triple/stats operations, client lifecycle cleanup in all 7 CLI modules, prune confirmation prompt, input validation fixes, and Cortex offline resilience. Requires AIngle Cortex >= 0.6.0. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove unused imports (getWorkflowDef, ErrorCodes, CortexClient) and cast params to String in dag-tools template literals. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
v0.6.0 release binaries did not include the `dag` feature. v0.6.1 ships with DAG endpoints enabled by default. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Default port was 8080 in cortex-resolution.ts but Cortex runs on 19090 since v0.1.6. All 12 CLI help strings now show 19090. postinstall.mjs now renames platform-suffixed binaries (e.g. aingle-cortex-macos-aarch64 → aingle-cortex) matching the logic already in cortex-update-check.ts. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
DAG tools and resources register automatically unless explicitly disabled via cortex.dag.enabled = false in config. - DagConfig type + parseDagConfig() in cortex-config.ts - Conditional DAG tool registration in MCP server (10 tools) - Conditional DAG resource wiring (tips, stats) in service lifecycle - Pass authToken to all MCP tool factories (memory, cortex, DAG) — fixes silent auth failure for memory/cortex tools Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
MockCortexClient was missing destroy(), causing 7 test failures after the client lifecycle cleanup in the CLI refactor. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
## Summary - **12 CortexClient DAG methods**: tips, action, history, chain, stats, prune, at, diff, export, sync, syncPull, verify - **10 MCP tools** with 30s timeouts, auth support, and Cortex-offline resilience — enabled by default, opt-out via `cortex.dag.enabled = false` - **10 CLI subcommands** (`mayros dag tips|action|history|chain|stats|export|diff|at|verify|prune`) with interactive prune confirmation - **2 MCP resources** (`dag/tips`, `dag/stats`) wired in service lifecycle - **dag-audit prompt** for guided audit workflows - **Default port fix**: 8080 → 19090 across 12 CLI modules and `cortex-resolution.ts` - **postinstall binary rename**: handles platform-suffixed binaries from GitHub Releases - **authToken consistency**: all MCP tool factories (memory, cortex, DAG) now receive auth token - **Requires AIngle Cortex >= 0.6.1** (DAG compiled in by default) - **sync-cli test fix**: 7 broken tests from missing `destroy()` in mock ## Test plan - [x] 149/149 MCP server tests passing (dag-tools: 28 tests) - [x] 7/7 sync-cli tests passing (was 0/7) - [x] 11855 total tests, 0 failures - [x] TypeScript type check — no new errors in modified files - [x] Verified all 10 DAG CLI commands against live Cortex v0.6.1 - [x] Verified postinstall binary rename with real GitHub Release tarball - [x] Verified DAG tools register by default, skip when `dag.enabled = false` - [x] Verified prune confirmation flow (interactive + `--yes` flag)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
mayros dag) with interactive prune confirmationTest plan