feat: import consumed REST client from OpenAPI 3.0 spec

[ako]

Rebased version of #268 (original PR by @khode-mx — original had merge conflicts against main).

Changes

• Resolves merge conflicts from feat: import consumed REST client from OpenAPI 3.0 spec #268 (generated parser files + CHANGELOG)
• Regenerated ANTLR4 parser via make grammar after rebase
• All tests pass (make build && make test)

Summary

CREATE REST CLIENT now accepts OpenAPI: 'path/or/url' to auto-generate a consumed REST service document from an OpenAPI 3.0 spec (JSON or YAML). Operations, path/query parameters, request bodies, response types, resource groups (tags), and Basic auth are derived automatically. DESCRIBE CONTRACT OPERATION FROM OPENAPI previews what would be generated without writing to the project.

Closes #207. Supersedes #268.

Co-authored-by: Dennis Kho dennis.kho@mendix.com
🤖 Generated with Claude Code

[github-actions]

AI Code Review

Moderate Issues

Missing Test Coverage

• Missing doctype test examples: The PR adds new MDL syntax (CREATE REST CLIENT with OpenAPI property and DESCRIBE CONTRACT OPERATION FROM OPENAPI) but doesn't include corresponding test examples in mdl-examples/doctype-tests/. Per the checklist: "New executor commands should have MDL examples in mdl-examples/doctype-tests/" and "MDL syntax changes — any PR that adds or modifies MDL syntax must include working examples in mdl-examples/doctype-tests/".

Incomplete Full-Stack Verification

• While the PR mentions regenerated parser files and passing tests, the visible diff doesn't show:
  • Grammar changes (MDLParser.g4/MDLLexer.g4)
  • AST node additions (mdl/ast/)
  • Visitor implementation (mdl/visitor/)
  • Executor handler (mdl/executor/)
  • Backend implementation (mdl/backend/mpr/)
  • DESCRIBE roundtrip verification (the DESCRIBE should output re-executable MDL)
  • VS Code extension updates (if new LSP capabilities were added beyond completions)

What Looks Good

Documentation Quality

• Excellent documentation updates in .claude/skills/mendix/rest-client.md with clear examples and explanations
• CHANGELOG.md properly documents the new feature
• Examples show usage from local files, URLs, and with BaseUrl overrides
• Includes preview capability via DESCRIBE CONTRACT OPERATION FROM OPENAPI

Syntax Compliance

• Follows standard CRUD verbs (CREATE, DESCRIBE)
• Reads as English: "create rest client OpenAPI: 'specs/capital.json'" is clear and understandable
• Property format correctly uses colon separators: OpenAPI: 'path/or/url'
• No keyword overloading - OPENAPI has one specific meaning
• Consistent with existing MDL patterns for property definitions
• Diff-friendly - adding the OpenAPI property produces a one-line diff in property lists

Code Consistency

• LSP completions correctly updated to include OPENAPI keyword
• Syntax feature definitions updated consistently across cmd/mxcli/syntax/ files
• PR is properly scoped to a single feature (OpenAPI import for REST clients)
• All tests pass per PR description (make build && make test)

Recommendation

Request changes to add the missing doctype test examples for the new MDL syntax before approval. Specifically:

1. Add test examples in mdl-examples/doctype-tests/ showing:

   • CREATE REST CLIENT with OpenAPI property (local file)
   • CREATE REST CLIENT with OpenAPI property (URL)
   • CREATE REST CLIENT with OpenAPI property and BaseUrl override
   • DESCRIBE CONTRACT OPERATION FROM OPENAPI for preview functionality

2. Verify and potentially add:

   • DESCRIBE roundtrip test showing the command outputs re-executable MDL
   • Any missing LSP capabilities beyond completions (if applicable)
   • VS Code extension updates if new LSP features were added

Once these test examples are added, the PR would be ready for approval as it otherwise follows all MDL syntax guidelines and maintains good documentation standards.

---

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