Repository navigation
flutter-marionette 0.2.0: take the Marionette skills from the package - #35
Merged
mateusz-pietras merged 1 commit intoOct 31, 2026
Merged
Conversation
The Marionette skills now live in one place, the marionette_flutter package. A new external entry, flutter-marionette-skills, reads packages/marionette_flutter from marionette_mcp main (git-subdir, strict: false) and loads marionette-flutter-setup, marionette-flutter-drive-app and marionette-flutter-to-patrol. flutter-marionette stays local: it keeps its usage skill and the marionette MCP launcher, and lists flutter-marionette-skills under dependencies, so installing it installs the skills too. The stale references (Marionette 0.6.0 API) are removed; the setup and drive skills cover them. The README is shorter and AGENTS.md describes the local-plus-external-dependency pattern. Merge after leancodepl/marionette_mcp ships the setup and to-patrol skills on main; until then the external entry points at missing paths. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Hi @mateusz-pietras! 👋 A quick check before this ships: is this PR meant to be public? Everything merged here is publicly visible. If any of it should stay inside LeanCode, it doesn't belong in this repository. If public is what you intended, just ignore this comment. 🙂 |
wzslr321
approved these changes
Oct 29, 2026
mateusz-pietras
deleted the
feat/flutter-marionette-skills-from-package
branch
October 31, 2026 19:11
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.
Merge after leancodepl/marionette_mcp#145. That PR adds two of the skill folders this one points at.
Summary
The Marionette skills now have one source, the
marionette_flutterpackage.flutter-marionettekeeps what only a Claude Code plugin needs and installs the skills through a dependency:references/marionette.mdandreferences/marionette-widget-config.md. They describe the Marionette 0.6.0 API, and the setup and drive skills cover the same ground for the current one.Evidence
/plugin install flutter-marionette@leancode-ai-pluginsinstalls one plugin with one skill,flutter-marionette-usage, and the 0.6.0 references.After: the same command, against a copy of this marketplace with the external entry pointed at the #145 branch, prints
(+ 1 dependency: flutter-marionette-skills).flutter-marionettehas 1 skill and themarionetteMCP server;flutter-marionette-skillshas the 3 skills.go test ./...passes,validate-pluginsreportsOK … 14 plugin(s), andclaude plugin validate . --strictexits 0.Merge Danger
Door: two-way
A revert restores 0.1.1.
Blast Radius: medium
flutter-marionette-skillspoints at two folders that do not exist yet on marionettemain.main, so a skill change there reaches plugin users without a release here.After merging
leancodepl/ai-plugins-internal loads
flutter-marionettefromplugins/flutter-marionettehere, and that plugin now depends onflutter-marionette-skills. The internal marketplace needs the sameflutter-marionette-skillsentry, or the dependency does not resolve there.🤖 Generated with Claude Code