Repository navigation
inbox: field notes on a testable retry loop, three mxpatch lookups that miss, and the macOS + Mendix 10 mxbuild Docker shim - #533
Closed
saingge wants to merge 1 commit into
Conversation
…at miss, and the macOS + Mendix 10 mxbuild Docker shim Three Lane 1 items from a Mendix 10.24 app on mxcli v0.27.0. Each claim was re-run on a synthetic module or the toolkit's own fixture model.
Owner
|
Thanks @saingge! Independent review: MERGE. We moved your commit unchanged, with your authorship, into #558, so CI runs without fork approval and we can handle CHANGELOG conflicts. This PR closes when #558 merges; the merge commit carries a Co-authored-by line for you. Generated by Claude Code |
MendixMau
added a commit
that referenced
this pull request
Oct 22, 2026
…s, and the macOS mxbuild Docker shim (#558, from #533) Three Lane 1 inbox items from a Mendix 10.24 app on mxcli v0.27.0, each re-run on a synthetic module or the toolkit's fixture model. Unreviewed inbox content; nothing cites it until triage promotes it. Contributed by @saingge in #533. Review: MERGE — inbox lane; anonymised, no client names or real paths. Security review: safe, descriptive text only. Co-authored-by: saingge <262679532+saingge@users.noreply.github.com> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013jQvaonym9V1wKKfAEPmRr
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.
Branch: contrib/inbox-field-items, one commit (fc054c8) on origin/master 393d253.
PR body (Inbox lane:
contrib/inbox/files only)Three items, one idea each, from a Mendix 10.24 app on mxcli v0.27.0. Each claim was re-run on 2026-10-21 on a synthetic module or on the toolkit's own fixture model, so nothing depends on the field app.
contrib/inbox/2026-10-21-testable-retry-loop-shape.md(582 words, skill-draft). "Retry an external call up to N times" as a loop with noreturninside: lints clean (awhile truewithreturninside gives MDL-FLOW01 once perreturn),mx check0 errors, and its decision flow tests without the call. Master has nothing on retry loops (git grep -i retry -- skills contribchecked; the nearest section is "Per-row isolation in a loop" inlearned-microflow-patterns.md).contrib/inbox/2026-10-21-mxpatch-three-lookups-that-miss.md(519 words, learning). An entity's$TypeisDomainModels$EntityImpl; a domain-model unit has noName(sofind_unitand the CLI'sget/setcan't select it); a flow's ownDocumentationis on the unit root, whichfind_elementskips (project-bin/mxpatch.py:181). Master hasEntityImplonly in an archived bug log.contrib/inbox/2026-10-21-macos-mendix10-mxbuild-docker-shim.md(587 words, learning). On macOS with Mendix 10 there is no native mx/mxbuild, so the gate needs a Docker shim atMXBUILD_PATH. Gives the shim's contract in generic terms, and thatbin/doctor.sh'sprobe_runs(lines 145-166) reads a shim that exits 3 on--version/--helpas "cannot run". Master says "Docker is the only route" (skills/mpr-corruption-and-sp-load-errors.md:61) but has no contract and no probe note. Relates to Local lane: no single 'what runs where' answer (MxBuild/runtime/DB/Playwright, Docker or not); doctor-triage says the ~/.mxcli/mxbuild cache is Linux-only but it holds a working Mach-O arm64 build on macOS #525, which is about doctor-triage's "Linux only" row: item 3 adds that this is true for Mendix 10 on a Mac and (per Local lane: no single 'what runs where' answer (MxBuild/runtime/DB/Playwright, Docker or not); doctor-triage says the ~/.mxcli/mxbuild cache is Linux-only but it holds a working Mach-O arm64 build on macOS #525 and a localfilecheck) not for the 11 builds.Verification
bin/check-no-client-data.sh: "no client data detected in 1151 tracked files".LEAKGUARD_BASE=origin/master bin/check-no-client-data.sh: new capitalised wordsHandleOutcome MaxRetries RetryState Silicon, all generic (synthetic identifiers; "Apple Silicon").bin/check-pr-discipline.sh: clean.git diff --stat origin/master...HEAD | tail -1: 3 files changed, 218 insertions(+).Shopmodule (mxcli v0.27.0):mxcli checkpassed;execcreated the flows;mxcli linthad no hit on the Shop flows; thewhile true-with-returnvariant gave 2 MDL-FLOW01 hits;mx checkthrough the Docker shim: "The app contains: 0 errors." (52 s, exit 0, both versions present).mx-test-docker-style run of 5 tests: "5 passed, 0 failed, 0 skipped", exit 0. The same tests with one expectation flipped: "FAIL h2 ... expected $State/Attempt = 2, actual: 1", "4 passed, 1 failed", exit 1. The CE0104 statement is fromcheckpasses a statement that can never run (afterreturn, or after awhile truewith nobreak), andmx checkthen fails with CE0104 / CE0105 / CE0709 mendixlabs/mxcli#1491 (read withgh issue view), not re-run. The "68 of 68" in the front matter is the field app's own count.executescriptfromtests/wave2/fixtures/mxpatch/model/TestApp.mpr.sqlplusmprcontents,python3 -I.find_element(doc, "Car", "DomainModels$Entity")raised NotFound; withDomainModels$EntityImplit found Car, VCar and ApprovalContext. The domain-model unit'snameisNone;find_unit("DomainModels$DomainModel", "MyFirstModule")raised NotFound;mxpatch.py findprintedMyFirstModule.None;get --type DomainModels$DomainModel --key Documentationprinted "mxpatch: --type and --name are required". Selecting bymodule_ofthenset_value(car, "Documentation", ...)gavechangedand a dry-run line.find_element(u.doc, "<microflow name>")raised NotFound for each microflow;mxpatch.py set ... --key Documentationwith no--elementprinted "Documentation: '' -> 'Does X.'" (dry run).mxpatch.py verify: "0 mismatch(es), 0 missing file(s), 0 orphan file(s)." Not applied to a model; not opened in Studio Pro.fileon this Mac: the 10.24.18 cached mxbuild is "ELF 64-bit LSB pie executable, ARM aarch64"; the Studio Pro 10.24.18 bundle'sContents/modelerhas 502 files and none namedmxormxbuild; the Studio Pro 11.12.4 bundle hasmxandmxbuildas "Mach-O 64-bit executable arm64". The shim run is under item 1.probe_runswas extracted frombin/doctor.shand run on two stubs: a stub that exits 3 without an .mpr returned exit 3; one that answers--versionwith exit 0 returned exit 0 (how=--version). Themxcli test --localdarwin error text and the 110-of-110 field test run come from the project's bug log, not re-run today. Not run: the toolkit's gate on Mendix 11 on a Mac.