feat: preserve change object refresh modifier

[hjotha]

Summary

Adds explicit MDL syntax for preserving ChangeObjectAction.RefreshInClient.

Changes

• Accepts change $Object refresh; and change $Object (...) refresh;.
• Carries the refresh flag through the AST and builder.
• Emits refresh when describing change-object actions with RefreshInClient=true.
• Adds proposal docs, a doctype fixture, quick-reference entry, and skill guidance.

Validation

• go test ./mdl/visitor -run TestChangeObjectParsesRefresh
• go test ./mdl/executor -run 'Test(FormatAction_ChangeObject.*Refresh|ChangeObjectBuilderWritesRefreshInClient)'
• ./bin/mxcli check mdl-examples/doctype-tests/change_refresh_modifier.test.mdl
• make build
• make lint-go
• make test

Closes #345
Part of #332

[github-actions]

AI Code Review

What Looks Good

The PR implements the change object refresh modifier feature comprehensively and correctly:

1. Full-stack consistency: All required layers are properly updated:

   • Grammar: MDLParser.g4 updated with REFRESH? option
   • AST: ChangeObjectStmt now includes RefreshInClient bool field
   • Executor: Builder logic updated to handle the flag, formatting functions updated to emit refresh keyword
   • Tests: Added doctest fixture (change_refresh_modifier.test.mdl) and unit tests for builder and formatter
   • Documentation: Updated quick reference, skills documentation, and added proposal document

2. MDL syntax compliance:

   • Follows standard patterns using existing change verb
   • Reads as clear English: change $Customer refresh;
   • Uses proper qualified names for variables ($Customer)
   • Consistent with existing property format patterns
   • refresh is a clear, non-overloading modifier keyword
   • Diff-friendly implementation (one-line addition for the feature)

3. Test coverage:

   • Added doctrtest fixture demonstrating both forms
   • Added unit tests for builder serialization and formatter output
   • PR mentions running relevant validation tests

4. Documentation:

   • Comprehensive proposal document explaining motivation and semantics
   • Updated quick reference with new syntax
   • Added examples to microflow skills documentation

Recommendation

The PR is ready for approval. It correctly implements the feature following all architectural guidelines, includes proper tests and documentation, and maintains consistency with existing MDL patterns. No changes are needed.

Approve

---

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