feat(search): video facet

[dschmidt]

Stacked on #3198 (tika facets).

Adds the video facet (dimensions, duration, codec, audio stream details) to the search index, the search API and the graph driveItem.

Maps what the released Tika 4.0.0 exposes for MP4/MOV: width, height, duration, bitrate (video:bitrate, TIKA-4802), frameRate (video:frame-rate, TIKA-4800), audioSamplesPerSecond, audioBitsPerSample (audio:bits-per-sample), audioChannels (numeric audio:channels, enum fallback). Verified against a live apache/tika:4.0.0-full with a generated MP4; against a Tika that does not emit these keys the facet simply stays empty.

Tika stores the video track dimensions under the same tiff:ImageWidth/tiff:ImageLength keys it uses for images (via Metadata.IMAGE_WIDTH), so image and video would otherwise both match. getImage is now gated on the image/ content type and getVideo on video/, inside the respective facet function, so a file gets exactly one of the two.

Still open: the codec FourCCs. video:fourcc (fourCC) and audio:fourcc (audioFormat) land with Tika 4.1 (TIKA-4838) and are already wired up here; until then both stay empty. The xmpDM:videoCompressor name ("AVC Coding") is no FourCC, so it is deliberately not used as a stand-in.

Needs a separate reva change, not includable here (same as #3200): the WebDAV PROPFIND handler must know the video keys to expose the facet over WebDAV. It lives in vendored reva, so it needs its own reva PR + dependency bump. The required diff:

diff --git a/vendor/github.com/opencloud-eu/reva/v2/internal/http/services/owncloud/ocdav/propfind/propfind.go b/vendor/github.com/opencloud-eu/reva/v2/internal/http/services/owncloud/ocdav/propfind/propfind.go
index d86367a2c5..8e4af07848 100644
--- a/vendor/github.com/opencloud-eu/reva/v2/internal/http/services/owncloud/ocdav/propfind/propfind.go
+++ b/vendor/github.com/opencloud-eu/reva/v2/internal/http/services/owncloud/ocdav/propfind/propfind.go
@@ -107,6 +107,18 @@ var (
 		"orientation",
 		"takenDateTime",
 	}
+	videoKeys = []string{
+		"audioBitsPerSample",
+		"audioChannels",
+		"audioFormat",
+		"audioSamplesPerSecond",
+		"bitrate",
+		"duration",
+		"fourCC",
+		"frameRate",
+		"height",
+		"width",
+	}
 )
 
 type countingReader struct {
@@ -887,6 +899,8 @@ func metadataKeys(pf XML) ([]string, []string) {
 					metadataKeys = append(metadataKeys, metadataKeysWithPrefix("libre.graph.image", imageKeys)...)
 				case "http://owncloud.org/ns/photo":
 					metadataKeys = append(metadataKeys, metadataKeysWithPrefix("libre.graph.photo", photoKeys)...)
+				case "http://owncloud.org/ns/video":
+					metadataKeys = append(metadataKeys, metadataKeysWithPrefix("libre.graph.video", videoKeys)...)
 				default:
 					metadataKeys = append(metadataKeys, key)
 				}
@@ -944,7 +958,7 @@ func requiresExplicitFetching(n *xml.Name) bool {
 		}
 	case net.NsOwncloud:
 		switch n.Local {
-		case "favorite", "share-types", "checksums", "size", "tags", "audio", "location", "image", "photo":
+		case "favorite", "share-types", "checksums", "size", "tags", "audio", "location", "image", "photo", "video":
 			return true
 		default:
 			return false
@@ -1292,6 +1306,7 @@ func mdToPropResponse(ctx context.Context, pf *XML, md *provider.ResourceInfo, p
 			appendMetadataProp(k, "oc", "location", "libre.graph.location", locationKeys)
 			appendMetadataProp(k, "oc", "image", "libre.graph.image", imageKeys)
 			appendMetadataProp(k, "oc", "photo", "libre.graph.photo", photoKeys)
+			appendMetadataProp(k, "oc", "video", "libre.graph.video", videoKeys)
 		}
 
 		if md.Type == provider.ResourceType_RESOURCE_TYPE_CONTAINER {
@@ -1579,6 +1594,10 @@ func mdToPropResponse(ctx context.Context, pf *XML, md *provider.ResourceInfo, p
 					if k := md.GetArbitraryMetadata().GetMetadata(); k != nil {
 						appendMetadataProp(k, "oc", "photo", "libre.graph.photo", photoKeys)
 					}
+				case "video":
+					if k := md.GetArbitraryMetadata().GetMetadata(); k != nil {
+						appendMetadataProp(k, "oc", "video", "libre.graph.video", videoKeys)
+					}
 				case "name":
 					appendToOK(prop.Escaped("oc:name", md.Name))
 				case "shareid":

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 1 duplication

Metric	Results
Duplication	1

View in Codacy

🟢 Coverage 40.30% diff coverage · +0.04% coverage variation

Metric	Results
Coverage variation	✅ +0.04% coverage variation (-1.00%)
Diff coverage	✅ 40.30% diff coverage

View coverage diff in Codacy

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (f60e2e5)	87708	20427	23.29%
Head commit (727c986)	87901 (+193)	20510 (+83)	23.33% (+0.04%)

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 (#3201)	201	81	40.30%

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.}