Skip to content

llm: add mz-dev, mz-test, mz-debug, mz-commit skills#35222

Merged
antiguru merged 2 commits intoMaterializeInc:mainfrom
antiguru:skill_run_test
Feb 26, 2026
Merged

llm: add mz-dev, mz-test, mz-debug, mz-commit skills#35222
antiguru merged 2 commits intoMaterializeInc:mainfrom
antiguru:skill_run_test

Conversation

@antiguru
Copy link
Member

@antiguru antiguru commented Feb 26, 2026

Add Claude Code skills for developing, testing, debugging, and committing in the Materialize repository.
Split into focused skills that trigger independently.

🤖 Generated with Claude Code

@github-actions
Copy link

Thanks for opening this PR! Here are a few tips to help make the review process smooth for everyone.

PR title guidelines

  • Use imperative mood: "Fix X" not "Fixed X" or "Fixes X"
  • Be specific: "Fix panic in catalog sync when controller restarts" not "Fix bug" or "Update catalog code"
  • Prefix with area if helpful: compute: , storage: , adapter: , sql:

Pre-merge checklist

  • The PR title is descriptive and will make sense in the git log.
  • This PR has adequate test coverage / QA involvement has been duly considered. (trigger-ci for additional test/nightly runs)
  • If this PR includes major user-facing behavior changes, I have pinged the relevant PM to schedule a changelog post.
  • This PR has an associated up-to-date design doc, is a design doc (template), or is sufficiently small to not require a design.
  • If this PR evolves an existing $T ⇔ Proto$T mapping (possibly in a backwards-incompatible way), then it is tagged with a T-proto label.
  • If this PR will require changes to cloud orchestration or tests, there is a companion cloud PR to account for those changes that is tagged with the release-blocker label (example).

@djahandarie
Copy link

(This is nice for external contributors too! 😄 )

@antiguru antiguru changed the title llm: add mz-run-test skill for running and testing Materialize llm: add mz-dev, mz-test, mz-debug, mz-commit skills Feb 26, 2026
@antiguru antiguru requested a review from mtabebe February 26, 2026 09:12
@antiguru antiguru force-pushed the skill_run_test branch 2 times, most recently from a62564f to 1826b3b Compare February 26, 2026 10:50
@antiguru antiguru requested a review from def- February 26, 2026 10:54
Copy link
Contributor

@mtabebe mtabebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am directionally aligned on this. I'll close my test skill PR.

Two minor comments.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@antiguru antiguru requested review from def- and mtabebe February 26, 2026 12:09
Copy link
Contributor

@mtabebe mtabebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢 LGTM

@antiguru
Copy link
Member Author

Thanks for the reviews!

@antiguru antiguru merged commit fcf046b into MaterializeInc:main Feb 26, 2026
5 checks passed
@antiguru antiguru deleted the skill_run_test branch February 26, 2026 13:58
mzcompose builds optimized binaries by default, which are nearly as fast to run as release and much faster to link than debug builds.

* `--release` builds full release binaries, which can be slow.
Instead of building release locally, let CI build the binary by creating a (draft) pull request and waiting for the build jobs to finish.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If mzcompose keeps rebuilding even if a CI image is available, make sure the src/ directory is clean. Untracked files are also considered as part of the image tag.

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.

4 participants