describe: end every statement with ';', never print '/' (#744)

[ako]

Closes #744

describe now ends every statement with ; and never prints a SQL*Plus / line, for every document type, so its output means the same under mdl 0 and mdl 1 (ADR-0010 R11/R12, ADR-0011).

Base note: this branch is based on feature/734-retrieve-first-vs-limit (origin/main + #739, #737, #740). Until those merge the PR also shows their commits; only the last two commits are this change.

What changed

• / lines removed from describe of agents, knowledge bases, consumed MCP services, models, constants, contract entities, associations, database connections, entities, enumerations, image collections, modules (describe module with and without with all), OData clients / services / external entities, published REST services, module roles, user roles, demo users, microflows, nanoflows and rules.
• Missing ; added: pages, snippets and layouts end with };; workflows with end workflow;; a published OData service ends with ; after whichever of its property list, authentication clause or entity block comes last (}; for the block); an external entity without attributes ends );.
• Published REST service with no resources now prints an empty { }; block. The block is mandatory in the grammar, so the bare ; it printed before did not parse even under mdl 0 (finding appended to mdl-executor.jsonl).
• diff / diff-local render statements without the / line too (separate commit), so they stay comparable with describe.
• Not changed: building-block describe (read-only, not a statement, already on the round-trip allowlist under Round-trip harness: untracked describe → exec losses on the Studio Pro fixture (microflows, entities, pages, snippets, menus, roles, JS actions) #721 G); testrunner generators and doc examples that still use / in authored scripts (valid under mdl 0; mdl 1: strict parsing (property keys, ';', no '/', '' escapes); trailing commas everywhere (#732) #741 owns the docs).

Tests

• New mdl/roundtrip/TestPedAppDescribeIsValidMdl1 (integration): runs describe for every PedApp document the round trip enumerates, plus every module (with and without with all), navigation and settings. For each it checks on the parse tree that every statement ends with ; and none is followed by /, then re-parses the output under an mdl 1; header and expects no errors. TestDescribeTerminatorCheck is the control for the checker. 177 targets are checked; 64 are skipped because their output does not parse under mdl 0 either (allowlisted under Round-trip harness: untracked describe → exec losses on the Studio Pro fixture (microflows, entities, pages, snippets, menus, roles, JS actions) #721).
• New mdl/executor/describe_terminators_test.go covers what PedApp does not contain: published OData service (properties only, authentication, block, grant) and external entity (with and without attributes), plus a control for the checker. The same assertTerminated check now also runs in every Describe*_Mock test for agent-editor documents, image collections, published REST, workflows, module/user roles and demo users, in the describe output that does not re-parse or loses data; configuration describe leaks DatabasePassword #707 reparse helper and in the OData client re-parse helper.
• Checked that the fix is the cause: with the executor changes reverted, 21 executor tests fail and TestPedAppDescribeIsValidMdl1 fails on 124 documents.

Test plan

• make build
• make lint
• go test ./mdl/executor/ ./cmd/mxcli/ -timeout 45m
• go test -tags integration ./mdl/roundtrip/ -timeout 45m: passes. The allowlist is unchanged, since none of its entries were about /.
• Throwaway local merge with origin/feature/732-strict-parsing (mdl 1: strict parsing (property keys, ';', no '/', '' escapes); trailing commas everywhere (#732) #741), not pushed:
  • go test -tags integration ./mdl/roundtrip/ -run 'TestPedAppListActivitiesUnderMdl1|TestPedAppRoundTrip_RetrieveRange|TestPedAppDescribeIsValidMdl1' passes.
  • The full roundtrip suite and ./mdl/executor/ pass.
  • With this PR's describe changes reverted on that merge, all three tests fail with `/` is not a statement terminator under mdl 1 (124 documents in the new test).

🤖 Generated with Claude Code