build: split graph-compose-render-pptx out of render-docx#302
Merged
Conversation
render-docx carried both the DOCX backend (Apache POI) and the semantic PPTX backend. Split PPTX into its own graph-compose-render-pptx module so a consumer that wants only slide output does not pull POI, and each backend keeps a minimal dependency surface. The PPTX backend is a POI-free skeleton today, so the new module depends only on graph-compose-core. - New graph-compose-render-pptx module (core-only, no POI). PptxSemanticBackend moves here from render-docx; its package/FQN is unchanged. - Split the shared SessionSemanticExportTest: the docx half stays in render-docx, the pptx half becomes PptxSemanticExportTest in render-pptx. - Wire the module into the aggregator, publish (deploy after render-docx), cut-release (version bump + commit list), VersionConsistencyGuardTest (version lockstep), and the CHANGELOG. - The CI build gate now also builds and tests render-docx and render-pptx; the render-docx tests were not running in CI before (it is a reactor leaf, not an also-make upstream of the engine/testing/qa slice).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
graph-compose-render-docxcarried both the DOCX backend (Apache POI) and thesemantic PPTX backend. Splitting PPTX into its own module lets a consumer who wants
only slide output avoid pulling POI, and keeps each backend's dependency surface
minimal. The PPTX backend is a POI-free skeleton today (it validates slide-safe
nodes and returns an export manifest), so the new module depends only on
graph-compose-core.What
graph-compose-render-pptxmodule —graph-compose-coreonly, no POI.PptxSemanticBackendmoves here from render-docx with its package/FQN unchanged.SessionSemanticExportTest: the docx half stays in render-docx,the pptx half becomes
PptxSemanticExportTestin render-pptx (both assertionspreserved).
cut-release.ps1(version bump + commit list),VersionConsistencyGuardTest(version lockstep), and the CHANGELOG.
render-docx tests weren't running in CI before (it's a reactor leaf, not an
also-make upstream of the engine/testing/qa slice).
Tests
./mvnw -f aggregator/pom.xml clean verify→ BUILD SUCCESS, allmodules; the new
PptxSemanticExportTestpasses and render-docx stays green afterthe trim.