chore: sync mops-cli skill from upstream cli-v2.15.2#236
Merged
Conversation
Sync the mops-cli skill from caffeinelabs/mops cli-v2.14.0 → cli-v2.15.2 (commit 564a380). No icskills-owned sections — body is 1:1 with upstream. Upstream changes applied: - New `mops deployed` command (post-deploy `.most` baseline promotion) - New `mops generate candid` command - `check-stable` bootstrap now uses `mops deployed init` instead of a hand-written trivial `.most`; `[check-stable].path` example → `deployed/backend.most` - `--no-check-limit` flag for `mops check`/`check-stable`/`lint` - `mops check --fix` skips read-only files with a warning - Also fixes a pre-existing local drift: `mops update --major`/`--patch` ordering now matches upstream Other: - Bump `compatibility` to `mops >= 2.15.2` (new commands require it) - Update `.claude/upstream.md` tag/commit/last-synced - Add eval cases for the new commands and the changed check-stable workflow Closes #234
Skill Validation ReportValidating skill: /home/runner/work/icskills/icskills/skills/mops-cliStructure
Frontmatter
Markdown
Tokens
Content Analysis
Contamination Analysis
Result: passed Project Checks |
The "Regenerating candid interface" eval asked for "just the command" but expected an explanation — a prompt/behavior mismatch. Reword the prompt to ask what the command does, matching the checked behaviors.
lwshang
approved these changes
Jul 1, 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
Syncs the
mops-cliskill fromcaffeinelabs/mopscli-v2.14.0→cli-v2.15.2(commit564a380). Closes #234.Per
.claude/upstream.md,mops-clihas no icskills-owned sections — the body is 1:1 with upstream and only the frontmatter differs. Applied the full upstream delta and refreshed the frontmatter/tracking.Upstream body changes applied
mops deployedcommand — post-deploy hook that promotes the built.mostto thecheck-stablebaseline (mops deployed initfor bootstrap,mops deployedafter each deploy).mops generate candidcommand — (re)generates the curated.didfrom Motoko source.check-stablebootstrap rewritten — replaces the hand-written trivial.mostsnippet withmops deployed init; the[check-stable].pathexample is nowdeployed/backend.most, and the description explains it runs ICP's upgrade-time compatibility check locally.--no-check-limitflag documented formops check/mops check-stable/mops lint, plus the new pending-migration reporting behavior.mops check --fixnow notes read-only files (e.g. frozen migrations) are skipped with a warning.Other
compatibilitytomops >= 2.15.2(the new commands require it)..claude/upstream.md— tag, full commit SHA, and last-synced date.mops update --major/--patchordering now matches upstream.check-stablebootstrap workflow (including an adversarial case guarding against the now-removed hand-written.most).Validation
npm run validate— all 26 skills pass (warnings only, none new formops-cli).Eval results
Ran the new/changed cases with baseline. Every new command shows a clear with-skill vs baseline delta — without the skill, agents hand-write
.mostfiles, denymops generate candidexists, and fall back todfx generate.Output evals (new cases, with vs without skill)
--no-check-limitNotable baseline failures (without skill):
touch .baseline.mostinstead ofmops deployed init.dfx generate— the exact hallucination this skill prevents.* The
--no-check-limitbaseline run hit the 120s harness timeout and returned empty, so 0/3 reflects no output rather than a wrong answer; the with-skill run passed 3/3.The
check-stable baseline bootstrap3/4 miss is a soft "may note the defaultdeployed/<name>.mostpath" behavior; the three hard behaviors passed.Trigger evals
🤖 Generated with Claude Code