Skip to content

Feat: Small Builder/Reader capability additions#99

Open
redaranj wants to merge 1 commit into
contentauth:mainfrom
redaranj:feat/builder-reader-capabilities
Open

Feat: Small Builder/Reader capability additions#99
redaranj wants to merge 1 commit into
contentauth:mainfrom
redaranj:feat/builder-reader-capabilities

Conversation

@redaranj

Copy link
Copy Markdown
Member

Changes in this pull request

Adds three small Builder/Reader capabilities:

  • Builder.setBasePath (resolve relative resource references against a directory)
  • Builder/Reader.supportedMimeTypes introspection
  • Ingredient archive import/export

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • All applicable changes have been documented
  • Any TO DO items (or similar) have been entered as GitHub issues and the link to that issue has been included in a comment

Additive Builder/Reader capabilities exposing previously-unwrapped FFI:

- Builder.setBasePath (GP-259): resolve relative resource references against a dir.
- Builder.supportedMimeTypes / Reader.supportedMimeTypes (GP-258): format introspection.
- Builder.writeIngredientArchive / addIngredientFromArchive (GP-257): compose
  ingredients across builders via archive round-tripping.

Wraps the corresponding c2pa_builder_*/c2pa_reader_* FFI through the JNI bridge,
with shared + Android instrumented tests.
@codecov

codecov Bot commented Jun 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 60.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.96%. Comparing base (5d4838e) to head (da485ee).

Files with missing lines Patch % Lines
...ry/src/main/kotlin/org/contentauth/c2pa/Builder.kt 61.53% 1 Missing and 4 partials ⚠️
...ary/src/main/kotlin/org/contentauth/c2pa/Reader.kt 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main      #99      +/-   ##
============================================
- Coverage     72.04%   71.96%   -0.09%     
- Complexity      267      272       +5     
============================================
  Files            59       59              
  Lines          2214     2229      +15     
  Branches        243      249       +6     
============================================
+ Hits           1595     1604       +9     
- Misses          482      483       +1     
- Partials        137      142       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@redaranj redaranj requested a review from scouten-adobe June 26, 2026 15:28
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.

2 participants