[emaarco]

Summary

This PR makes two small cleanups that came out of an architecture review.

One internal platform value. The platform used to have three spellings inside the code: c7 | c8 | none in validation, camunda7 | camunda8 | undefined from Doc.platform() and EditContext.platform(), and zeebe in DecisionLinkWriter. Two hand-written ternaries translated between them. Now 'c7' | 'c8' | 'none' is the only internal value. The --target camunda7|camunda8|none spelling users type is translated in one place, platformOfTarget() in PlatformDetection.ts. That file now also holds TARGETS and the Target type. PlatformDetection.target() is removed, and the editing code compares against c7/c8 directly.

No in-memory document store. InMemoryDocumentStore was never read or written: the in-memory API always passes a loaded document and dryRun: true. The services now take their load and save ports as optional, after the project settings. Reading a file without a load port fails with E_FILE_NOT_FOUND, the error the store gave. Writing without a save port fails with E_NO_FILE. NoProjectSettings stays. docs/architecture.md and GLOSSARY.md are updated.

Breaking change for library callers: Doc.platform() now returns 'c7' | 'c8' | 'none' instead of 'camunda7' | 'camunda8' | undefined. TARGETS is still exported from the package entry. CLI options and output are unchanged.

Checklist

• npm run typecheck passes
• npm run gate passes
• src/report/guide.ts and the matching page in docs/ are updated (only if a command, option or output changed)
• PR title is a changelog-worthy Conventional-Commit summary

npm run gate passed on the branch head: 2398 tests, iso and bench within their budgets with no budget raised, fuzz with 0 errors. The first commit was also typechecked and tested on its own. The guide item does not apply because no command, option or output changed.

Linked issue

No linked issue.