fix: docker cache detection on Windows and reload schema warning

[engalar]

Summary

• Fix mxcli docker run re-downloads MxBuild on Windows even when Linux binary already exists #126: mxcli docker run on Windows re-downloads MxBuild (~394MB) even when the Linux binary is already cached. Root cause: cache check looked for mxbuild.exe but Docker needs mxbuild. Now checks both names on Windows.
• Fix docker reload silently fails for new entities (no schema sync) #115: docker reload silently succeeds when new entities are added, but the app crashes at runtime ("Entity does not exist") because the database schema wasn't synced. Now calls get_ddl_commands after reload and prints a warning with pending DDL text if schema changes are detected.

Test plan

• All existing docker package tests pass
• New TestReload_ModelOnly_PendingDDL validates DDL warning output
• TestReload_ModelOnly verifies both reload_model and get_ddl_commands actions are called
• Verify get_ddl_commands response format against a live Mendix runtime (feedback field name: ddl_commands string)

Closes #126
Closes #115

🤖 Generated with Claude Code

[github-actions]

AI Code Review

What Looks Good

• Windows caching fix: Correctly addresses the root cause by checking both mxbuild.exe and mxbuild on Windows Docker environments where the Linux binary is cached
• DDL warning after reload: Proactively prevents runtime crashes by checking for pending schema changes and providing clear remediation guidance (mxcli docker up --fresh)
• Test improvements: New test validates DDL warning output; existing tests updated to verify correct action sequence
• Helper function: mxbuildBinaryNames() extraction reduces duplication and clarifies intent
• Backwards compatibility: Changes are additive and don't break existing functionality
• Clear documentation: Comments explain why checks are necessary for both fixes

Recommendation

Approve the PR. The changes are well-scoped, properly tested, address real user issues (unnecessary downloads and silent failures), and follow existing code patterns. No blocking issues found. The minor incomplete test item (verifying against live Mendix runtime) is appropriately acknowledged as needing feedback but unit tests adequately mock the HTTP interaction.

---

Automated review via OpenRouter (Nemotron Super 120B) — workflow source