docs: add mxcli-dev contributor command namespace and /mxcli-dev:review#220
Merged
ako merged 3 commits intomendixlabs:mainfrom Apr 17, 2026
Merged
Conversation
Introduces .claude/skills/fix-issue.md — a shared skill for diagnosing and fixing bugs faster. Seeds the symptom table from issue mendixlabs#212. Each fix appends a new row, compounding over time. Wires the skill into the PR checklist in CLAUDE.md under a new Bug fixes section. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…JSON bodies in SEND REST REQUEST
Fixes CE7054 ("parameters updated") and CE7067 ("does not support body
entity") errors produced by mx check after mxcli generates a
Microflows$RestOperationCallAction.
Two root causes:
- All WITH-clause params were emitted as QueryParameterMappings regardless
of whether they are path or query params on the operation definition.
- BodyVariable was always serialised when BODY $var was present, but for
Rest$JsonBody / Rest$StringBody operations the body lives on the operation
template and BodyVariable must be nil.
Fix: look up the ConsumedRestService operation at build time via a new
restServices field on flowBuilder (populated from loadRestServices()), then
use lookupRestOperation / buildRestParameterMappings / shouldSetBodyVariable
helpers to emit the correct BSON. When the operation is not found we fall
back to the previous behaviour.
Closes mendixlabs#193
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Introduces a clear separation between commands meant for mxcli users (mendix/) and commands for contributors to this repo (mxcli-dev/). - Add .claude/commands/mxcli-dev/review.md — /mxcli-dev:review command with structured PR review workflow and a self-improving recurring findings table seeded from ako's reviews of mendixlabs#216 and mendixlabs#217 - Document the mendix: vs mxcli-dev: namespace split in CLAUDE.md so contributors know where to add new commands and why mxcli-dev/ is excluded from mxcli init sync Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
AI Code ReviewCritical Issues
Moderate Issues
Minor Issues
-ИLimit Automated review via OpenRouter (Nemotron Super 120B) — workflow source |
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
2 participants
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
.claude/commands/mxcli-dev/as a separate namespace for contributor commands — distinct frommendix/which is synced to user projects viamxcli init/mxcli-dev:review— a structured PR review command with a self-improving recurring findings table, seeded from @ako's reviews of docs: add fix-issue skill with TDD protocol and symptom→layer table #216 and fix: handle Microflows$Find, $Filter, $ListRange in list operation DESCRIBE #217mendix:vsmxcli-dev:split in CLAUDE.md so contributors know where to add new commands and whymxcli-dev/is excluded from syncReviewer
@ako — seeded directly from your feedback on #216 and #217. Would value your eye on whether the findings table captures the intent correctly and whether the namespace split makes sense to you.
🤖 Generated with Claude Code