fix: align BSON properties with Mendix schema for mx diff compatibility

[engalar]

Summary

• Add missing CustomSettings and Tracing fields to ServerConfiguration BSON serialization
• Remove invalid IsReusableComponent from module BSON (not in Mendix schema)
• Prevent orphaned mxunit files when document insert fails in writer_units
• Add mx diff diagnostics workflow to debug-bson skill

Split from #107 per code review — these are independent bug fixes unrelated to ALTER WORKFLOW.

Test plan

• go build ./... passes
• go test ./... passes
• mx diff shows no unexpected changes after roundtrip write

🤖 Generated with Claude Code

[github-actions]

AI Code Review

Critical Issues
None

Moderate Issues
None

Minor Issues

• The PR modifies multiple files across different areas (documentation, module writer, settings writer, units writer). While related to BSON serialization, these could be considered separate concerns. However, the PR author states they were split from feat: ALTER WORKFLOW command with full activity manipulation #107 per code review and are independent bug fixes, so this is acceptable.
• No test files were added or modified. The changes should be covered by existing tests, but it would be good to see explicit test coverage for the BSON field additions/removals.

What Looks Good

• The debug-bson.md update adds valuable diagnostic information about different tolerance levels of Mendix tools (mx check vs mx diff vs Studio Pro) and provides clear steps to reproduce and diagnose BSON issues.
• Removing IsReusableComponent from module BSON aligns with the Mendix schema (as documented in the debug-bson skill).
• Adding CustomSettings and Tracing to ServerConfiguration BSON with appropriate default values (empty array and nil) follows the patterns described in the debug-bson skill.
• The fix in writer_units.go properly handles orphaned file cleanup on insert failure, improving robustness.

Recommendation
Approve. The changes are correct, well-documented, and address specific bugs that would cause mx diff crashes or orphaned files. The documentation update significantly improves the debugging workflow for BSON issues. Minor concerns about test coverage are outweighed by the correctness of the fixes and the existence of existing test suites.

---

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

[ako]

Clean split from #107. All four changes are well-motivated independent bug fixes:

• writer_settings.go: Missing CustomSettings/Tracing fields fix mx diff crash
• writer_modules.go: IsReusableComponent removal fixes crash on Mendix 11.6.4
• writer_units.go: Orphan mxunit cleanup on insert failure prevents project corruption
• debug-bson.md: mx check vs mx diff tolerance documentation is valuable operational knowledge

LGTM.