docs: add Commonalities consumption and bundling design#436
Conversation
- add a ReleaseManagement supporting document describing the controlled local copy model for CAMARA_common.yaml - document the bundling strategy for main, PR review, and snapshot/release branches - capture the validation, sync, and rollout model discussed in Commonalities issue #577
Split section 8 into placeholder removal (8.1) and API-specific error code extension (8.2) as independent concerns. Recommend commenting out placeholders rather than deleting, which unblocks $ref consumption without requiring structural refactoring.
|
Updated section 8 (Commonalities Content Expectations) to refine the placeholder handling approach. Thanks and credits to @rartych for the idea and testing it in a PoC Key change: Section 8 is now split into two independent concerns: 8.1 Placeholder Removal — Instead of deleting placeholder patterns ( 8.2 API-Specific Error Code Extension — How APIs add their own error codes (allOf, property override, own response composition) is explicitly separated as an independent topic. It does not need to be resolved before Why this matters: The previous version coupled placeholder removal with the allOf extension pattern, making both a prerequisite for bundling. Separating them means the Commonalities side can make a minimal change (commenting ~30 lines) to unblock |
Source and bundled files occupy the same path — on main the file contains $ref, on snapshot/release branches the same file is replaced with the standalone bundled version. No separate source or output directory. Stated in executive summary, design goals, and reinforced in sections 5 and 9.1.
|
Added in-place replacement as an explicit design principle (executive summary point 5, design goals, sections 5 and 9.1): The API source file and the bundled file occupy the same path ( |
Use consistent phrasing: source file may contain external $ref, snapshot/release branches have the standalone bundled version.
documentation/SupportingDocuments/Commonalities-Consumption-and-Bundling-Design.md
Show resolved
Hide resolved
|
Comments addressed, approved and no new comments since a week. |
What type of PR is this?
What this PR does / why we need it:
This PR adds a supporting design document for the Commonalities consumption and bundling model discussed in Commonalities issue #577.
The document captures the agreed direction to:
CAMARA_common.yamlthrough a controlled local copy onmainrelease-plan.yaml.dependencies.commonalities_releaseas the authoritative dependency declarationmain, while still providing bundled review artifacts in PRsThe document is intended to support cross-project review in Release Management, Commonalities, tooling, and pilot API repositories before implementation issues are split out.
Which issue(s) this PR references:
Related to #435
Special notes for reviewers:
This is a design/supporting document PR.
Implementation is expected to follow through linked issues in ReleaseManagement, tooling, Commonalities, and pilot API repositories after agreement on the design direction.
Changelog input
Additional documentation
This PR adds the following documentation: