Skip to content

feat: migrate cap4k addon to pipeline extension - #3

Merged
LDmoxeii merged 2 commits into
masterfrom
feature/cap4k-pipeline-extension-migration
Aug 1, 2026
Merged

feat: migrate cap4k addon to pipeline extension#3
LDmoxeii merged 2 commits into
masterfrom
feature/cap4k-pipeline-extension-migration

Conversation

@LDmoxeii

@LDmoxeii LDmoxeii commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • register a single OnlyEnginePipelineExtensionProvider through the new cap4k Pipeline Extension ServiceLoader SPI
  • expose the existing enum translation and validator artifact addons as declarative extension contributions
  • migrate README configuration from cap4kAddon(...) to cap4kPipelineExtension(...) and contribution-scoped options
  • replace the legacy per-addon service descriptor and consolidate ServiceLoader coverage at the extension root
  • update Composite Build CI to follow cap4k master instead of a deleted historical branch

Dependency

Verification

  • GitHub CI Composite Build passed against cap4k master: ./gradlew check -PonlyEngine.cap4kCompositePath="$GITHUB_WORKSPACE/cap4k"
  • Pipeline Extension ServiceLoader smoke passed
  • direct JUnit Platform execution passed: 23/23 tests
  • git diff --check passed
  • CI run: https://github.com/LDmoxeii/only-engine/actions/runs/30679003683

@LDmoxeii

LDmoxeii commented Aug 1, 2026

Copy link
Copy Markdown
Owner Author

CI dependency note: the workflow now correctly checks cap4k master instead of the deleted historical branch. The current failure is expected while LDmoxeii/cap4k#150 is unmerged: PipelineExtensionProvider, PipelineExtensionDescriptor, and related API symbols are not yet present on cap4k master. After #150 merges, rerun this check before marking the PR ready.

@LDmoxeii
LDmoxeii marked this pull request as ready for review August 1, 2026 02:17
@LDmoxeii
LDmoxeii merged commit 7f840ed into master Aug 1, 2026
1 of 2 checks passed
@LDmoxeii
LDmoxeii deleted the feature/cap4k-pipeline-extension-migration branch August 1, 2026 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant