Skip to content

feat(search): extract more data from tika - #3198

Draft
dschmidt wants to merge 4 commits into
feat/search-schema-change-handlingfrom
feat/search-tika-facets
Draft

feat(search): extract more data from tika#3198
dschmidt wants to merge 4 commits into
feat/search-schema-change-handlingfrom
feat/search-tika-facets

Conversation

@dschmidt

@dschmidt dschmidt commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Stacked on #3197 (search schema-change handling).

Draft: needs the unreleased Apache Tika 4.0 (pinned by snapshot digest) for the new metadata keys.

Bumps libre-graph-api-go, splits the tika extractor and its tests into per-facet files, pins the acceptance tika to a 4.0.0 snapshot, and fills the facet properties Tika 4.0 now exposes:

  • Audio: bitrate, isVariableBitrate, hasDrm, trackCount, discCount, copyright, composers
  • Location: altitude (EXIF GPS, metres → feet)
  • GPS now also resolves for videos (MP4/MOV); the Content-Type: audio/ gate is gone, so audio metadata is read from any file.

The libregraph bump lands here as the foundation for the follow-up motion-photo and live-photo PRs (kept separate, they are non-trivial), so those stay smaller and don't have to re-review the vendor changes.

@codacy-production

codacy-production Bot commented Jul 29, 2026

Copy link
Copy Markdown

Not up to standards ⛔

🟢 Coverage 100.00% diff coverage · +0.03% coverage variation

Metric Results
Coverage variation +0.03% coverage variation (-1.00%)
Diff coverage 100.00% diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (a1b0535) 82984 19541 23.55%
Head commit (c7c0551) 83018 (+34) 19577 (+36) 23.58% (+0.03%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#3198) 169 169 100.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@dschmidt dschmidt changed the title feat(search): more tika facets (audio bitrate/drm/counts, copyright, composer, gps altitude) + libregraph bump feat(search): extract more data from tika Jul 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant