Skip to content

docs: improve AGENTS.md and query_stats README - #1376

Merged
marc0olo merged 4 commits into
masterfrom
chore/add-ic-mops-prerequisite
Jun 11, 2026
Merged

docs: improve AGENTS.md and query_stats README#1376
marc0olo merged 4 commits into
masterfrom
chore/add-ic-mops-prerequisite

Conversation

@marc0olo

@marc0olo marc0olo commented Jun 11, 2026

Copy link
Copy Markdown
Member

Summary

Improves AGENTS.md and fixes the query_stats example README (PR #1364 was already merged so this is a follow-up).

Changes

AGENTS.md:

  • Add ic-mops: npm install -g ic-mops to the README prerequisites template
  • Add mops check / mops check --fix workflow note after the mops.toml section — both must pass before committing
  • Add mops check --fix to the migration checklist
  • Clarify .icp/ gitignore rules: only .icp/cache/ should be ignored (the root .gitignore already handles this with **/.icp/cache/); per-example gitignores must never add .icp/ as it hides .icp/data/ which holds canister ID mappings
  • Note to remove .dfx/ gitignore entries during migration — dfx is no longer used

motoko/query_stats/README.md:

  • Add ic-mops prerequisite

All other open migration PRs have ic-mops added directly on their branches.

🤖 Generated with Claude Code

ic-mops is required for building Motoko canisters locally when using
the @dfinity/motoko recipe. Add it to the README template in AGENTS.md
and to the query_stats example (other examples updated via their PRs).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@marc0olo
marc0olo requested review from a team as code owners June 11, 2026 19:42
marc0olo and others added 3 commits June 11, 2026 21:45
- Add mops check / mops check --fix workflow note after mops.toml section
- Add migration checklist item to run mops check --fix and commit fixes

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The root .gitignore has **/.icp/cache/ which is correct. Per-example
.gitignore files must not add .icp/ as that hides .icp/data/ containing
canister ID mappings.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@marc0olo marc0olo changed the title docs: add ic-mops as prerequisite for Motoko examples docs: improve AGENTS.md and query_stats README Jun 11, 2026
@marc0olo
marc0olo merged commit 20310d4 into master Jun 11, 2026
6 checks passed
@marc0olo
marc0olo deleted the chore/add-ic-mops-prerequisite branch June 11, 2026 21:09
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.

2 participants