Skip to content

chore: sync mops-cli skill from upstream cli-v2.16.1#243

Merged
marc0olo merged 1 commit into
mainfrom
chore/sync-upstream-mops-cli-v2.16.1
Jul 13, 2026
Merged

chore: sync mops-cli skill from upstream cli-v2.16.1#243
marc0olo merged 1 commit into
mainfrom
chore/sync-upstream-mops-cli-v2.16.1

Conversation

@marc0olo

Copy link
Copy Markdown
Member

Syncs mops-cli from caffeinelabs/mops cli-v2.15.2cli-v2.16.1.

Upstream delta

One added sentence in Enhanced migrations:

After mops check --fix (or mops check <canister>) confirms the chain
compiles, run mops build to produce the wasm artifact.

Body is 1:1 with upstream (no icskills-owned sections).

Changes

  • Add build-after-check sentence to Enhanced migrations
  • Bump compatibilitymops >= 2.16.1
  • Update .claude/upstream.md (tag, full commit SHA 4bccdc13b512048f41b3296c3590b776c1836fe1, last synced 2026-07-10)
  • Reword Migration-workflow eval to assert on written guidance rather than command execution (the "how do I" prompt elicits instructions, not runs)

Eval

Migration workflow (#6) — with baseline
Migration workflow: WITH 5/5 | WITHOUT 5/5

  WITH skill: 5/5 passed
    ✅ Recommends `mops check --fix` (or `mops check backend`) to verify the migration resolves the error
    ✅ Places `mops build` after the check step — never before
    ✅ Does NOT suggest `mops migrate new` (command no longer exists)
    ✅ Does NOT suggest `mops migrate freeze` (command no longer exists)
    ✅ Does NOT suggest adding `--enhanced-migration` to mops.toml manually (mops auto-injects it)

  WITHOUT skill: 5/5 passed
    (same 5 behaviors)

Reworded behaviors now pass with the skill. No with/without delta on this case — its regression value is guarding the removed-command (mops migrate new/freeze) and manual --enhanced-migration pitfalls.

Closes #239

Sync mops-cli from caffeinelabs/mops cli-v2.15.2 → cli-v2.16.1.

Upstream diff: one added sentence in Enhanced migrations clarifying that
`mops build` produces the wasm artifact after `mops check` confirms the
chain compiles. Body is 1:1 with upstream (no icskills-owned sections).

- Add build-after-check sentence to Enhanced migrations
- Bump compatibility to mops >= 2.16.1
- Update .claude/upstream.md (tag, commit SHA, last synced)

Closes #239
@marc0olo marc0olo requested review from a team and JoshDFN as code owners July 10, 2026 14:30
@github-actions

Copy link
Copy Markdown

Skill Validation Report

Validating skill: /home/runner/work/icskills/icskills/skills/mops-cli

Structure

  • Pass: SKILL.md found

Frontmatter

  • Pass: name: "mops-cli" (valid)
  • Pass: description: (341 chars)
  • Pass: license: "Apache-2.0"
  • Pass: compatibility: (14 chars)
  • Pass: metadata: (2 entries)

Markdown

  • Pass: no unclosed code fences found

Tokens

File Tokens
SKILL.md body 2,745
Total 2,745

Content Analysis

Metric Value
Word count 1,506
Code block ratio 0.30
Imperative ratio 0.09
Information density 0.20
Instruction specificity 0.60
Sections 23
List items 9
Code blocks 15

Contamination Analysis

Metric Value
Contamination level low
Contamination score 0.01
Primary language category shell
Scope breadth 2
  • Warning: Language mismatch: config (1 category differ from primary)

Result: passed

Project Checks


✓ Project checks passed for 1 skills (0 warnings)

@marc0olo marc0olo enabled auto-merge (squash) July 10, 2026 15:23
@marc0olo marc0olo merged commit c3cde18 into main Jul 13, 2026
6 checks passed
@marc0olo marc0olo deleted the chore/sync-upstream-mops-cli-v2.16.1 branch July 13, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upstream sync available — caffeinelabs/mops cli-v2.15.2 → cli-v2.16.1

2 participants