Repository navigation
feat: sync LoT/broker skills from coreflux-skills - #7
Merged
HugoAVaz merged 7 commits intoOct 20, 2026
Merged
Conversation
…rning lot_lint (Cursor plugin MCP) treated DEFINE AGENT as unknown. Mirror the broker parser: USES LLM ROUTE|CONNECTOR first, required DEFAULT_USER, ADD TOOL "name" with TOPICS|TARGET|LEVEL, deploy as -addAgent after routes. Also skip the missing-AS-TRIGGER warning when a model inherits via FROM.
…tten copies The plugin's skills are now a pinned copy of CorefluxCommunity/coreflux-skills (tag 2.21.16-rc.186), which is generated from the broker. scripts/sync-skills.mjs updates the copy (npm run skills:sync -- --ref <tag>) and --check fails CI on drift. The hand-written lot-authoring, broker-commands, broker-observability, broker-security and lot-projects skills are removed; a plugin-owned coreflux-cursor-tools skill maps the broker tool names in the shared skills to this plugin's MCP tools. Rules, agents, commands, the session hook and docs point at the new skill names. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The synced skills teach agents and things, but lot_deploy refused both. Agents deploy with -addAgent after routes (they need their LLM connector); every DEFINE THING block, templates included, goes as one -addThing after rules, the way the broker's own deploy and folder loader group them. lot_remove takes AGENT and THING. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…to feat/sync-coreflux-skills # Conflicts: # plugins/coreflux/scripts/mcp/coreflux-mqtt-mcp.mjs # plugins/coreflux/scripts/mcp/lib/lot.mjs # plugins/coreflux/test/lot.test.mjs
github-actions
Bot
added
area: mcp-server
…ools Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
HugoAVaz
approved these changes
Oct 20, 2026
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.
Summary
The plugin stops carrying its own drifted copies of the LoT/broker skills. It now ships a pinned copy of CorefluxCommunity/coreflux-skills, which is generated from the broker's skill sources and tagged per broker release.
2.21.16-rc.186):broker-tools,lot-actions,lot-models,lot-routes,lot-rules,lot-agents,lot-things,lot-panels,lot-themes,python-scripts.scripts/sync-skills.mjs:npm run skills:sync -- --ref <tag>moves the pin, which is recorded asbrokerVersioninskills/skills.json.npm run skills:checkruns in CI and the release flow, and fails if the bundle differs from the pinned tag. Fixes go to the broker, not here.lot-authoring,broker-commands,broker-observability,broker-securityandlot-projectsskills. Several of their claims were wrong against the broker, including Python import blocklists,USER HAS "AllowedSubscribe", userSubscribeSysrules,METHOD/SWITCHkeywords and a number of route settings.coreflux-cursor-toolsskill: the shared skills name the broker's own MCP tools (deploy_lot,validate_lot,diagnose, …), and this skill maps them to this plugin's tools./coreflux-secureno longer recommends userSubscribeSysrules, which are never consulted.check-sources: the "only mention tools this server exposes" check skips the synced skills, which describe the broker's tool surfaces on purpose.lot_deploy/lotnb_deploynow deployDEFINE THING: every Thing block, templates included, goes as one-addThingafter rules, matching the broker's deploy and folder loader.lot_removetakesTHING.Stacked on #6
This branch merges
feat/ENG-426/lint-define-agent(#6), which addsDEFINE AGENTdeploy and lint. Merge #6 first; until then this PR's diff includes its changes.Content still missing from the broker skills
The guidance the old skills had that the broker lacked (the raw MQTT command surface and reply format, the
$SYStopic map, the troubleshooting checklist, users and secrets, projects and.lotnb, and LoT trigger and route examples) is being added to the broker as a newbroker-operationsskill plus edits to the LoT skills. It reaches this plugin with the next coreflux-skills tag, throughnpm run skills:sync -- --ref <tag>.Test plan
npm run ci: validate, check-sources, skills:check, 74 unit testsskills:checkfails on a hand-edited synced file🤖 Generated with Claude Code