Repository navigation
fix(mcp-server): drop undefined cron, pass tier run notes, 1800s timeout, current examples - #1190
Merged
Merged
Conversation
…out, current examples - Tool descriptions read each skill's schedule from aeon.yml (the catalog no longer carries one), so they show "cron: <expr>", "on-demand" or "reactive" instead of "cron: undefined". Skill.schedule is now optional. - run-harness now gets --append-system-prompt with scripts/skill_mode.sh run-notes <mode>, as the scheduled workflow passes. - Per-run timeout is 1800s, matching the workflow's run-harness --timeout. - bin/add-mcp and docs use skills that exist (aeon-digest, aeon-article).
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
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.
Four small drifts between the local MCP server (apps/mcp-server) and the scheduled run path.
What changed
schedule:from its line in aeon.yml, using the same single-line lookup it already uses for the per-skillharness:(now one shared helper). Descriptions show "(cron: )", "(on-demand)" forworkflow_dispatch, "(reactive)" forreactive, and no suffix if a skill has no aeon.yml entry.Skill.scheduleis now optional.--append-system-prompt "$(scripts/skill_mode.sh run-notes <mode>)"to run-harness; the MCP path did not. It now calls the samerun-notescommand for the resolved mode and passes it the same way (only when non-empty, like the workflow).--timeout 1800to run-harness. Both the run-harness flag and the Node kill timer now use one 1800s constant. The README line that claimed the old value matched Actions is corrected.How it was verified
npm run buildandnpm run typecheckin apps/mcp-server pass.claudeon PATH that dumps its argv: called aeon-heartbeat (write) and aeon-competitor-monitor (read-only) through tools/call. Both returned the fake result, and the--append-system-promptvalue the fake received was byte-identical toscripts/skill_mode.sh run-notes <mode>for each tier.shellcheck bin/add-mcpis clean andscripts/lint-shell.shpasses; the add-mcp banner still lines up.