forgejo_admin/exifcleaner-web
1
0
Fork 0
Code Issues 5 Pull requests Projects Releases Packages Wiki Activity Actions

A2: Fragmented MP4 via mp4box.js #2

New issue
Closed
opened 2026-05-04 21:18:26 +00:00 by obuvuyoviz26-lab · 0 comments
obuvuyoviz26-lab commented 2026-05-04 21:18:26 +00:00 (Migrated from github.com)
Copy link

Part of v5 webapp migration — Phase A: Failing formats (priority #1)

Add VideoStrategy using mp4box.js. Rewrites the MP4 box/atom tree without metadata boxes (udta/meta/Xtra). Pure JS, no decoding, no quality loss. Handles fragmented MP4 (moof-based) — the case ExifTool cannot write.

New files: infrastructure/wasm/strategies/video_strategy.ts (~150 lines)
New dep: mp4box
Registers for: .mp4, .mov, .m4v, .m4a

Deliverable: drop the fragmented MP4 that currently fails → cleaned successfully.
Verify: yarn dev → drop failing MP4 → exiftool output.mp4 shows no metadata; play to confirm video intact.

See design doc docs/superpowers/specs/2026-05-05-webapp-migration-design.md §9.1

Part of v5 webapp migration — **Phase A: Failing formats (priority #1)** Add `VideoStrategy` using mp4box.js. Rewrites the MP4 box/atom tree without metadata boxes (`udta`/`meta`/`Xtra`). Pure JS, no decoding, no quality loss. Handles fragmented MP4 (`moof`-based) — the case ExifTool cannot write. **New files**: `infrastructure/wasm/strategies/video_strategy.ts` (~150 lines) **New dep**: `mp4box` **Registers for**: `.mp4`, `.mov`, `.m4v`, `.m4a` **Deliverable**: drop the fragmented MP4 that currently fails → cleaned successfully. **Verify**: `yarn dev` → drop failing MP4 → `exiftool output.mp4` shows no metadata; play to confirm video intact. See design doc `docs/superpowers/specs/2026-05-05-webapp-migration-design.md` §9.1
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
fix/192-audit-deps
fix/180-apk-safe-area
worktree-docs-direction-apk-standalone
worktree-poc-webperl-exiftool
fix/traf-blanking-36
fix/upload-artifact
feat/issue-133-mat2-png
feat/issue-132-mat2-jpeg
refactor/strategy-split
feat/arabic-default-language
fix/issue-141-office-review
refactor/video-diff-146
fix/101-status-bar-error-count
feat/office-structural-removals
worktree-fix+tags-removed-count
feat/video-diff-125
fix/office-structural-clean-status
feat/issue-124-office-diff
feat/pdf-forensic-mat2
feat/issue-121-office-phase3
refactor/xml-property-walker
feat/issue-131-xmp-enumeration
feat/issue-123-png-diff
feat/ci-merge-gate-hook
fix/master-test-errors
feat/metadata-diff
fix/35-timed-track-removal
feat/rebrand-metascrub
feat/phase-g-electron-retirement
feat/claude-hooks
feat/trim-empty-state-copy
docs/readme-web-first
worktree-claude-md-branch
worktree-vps-deploy
worktree-issue-83-zero-timestamps
feat/phase-d-d5-docs
feat/phase-d-shell-collapse
feat/office-forensic-verification
feat/video-forensic-verification
feat/phase-d-doc-updates
feat/phase-c-web-e2e
feat/phase-b8-png-hdr
ci/disable-auto-builds
feat/route-jpeg-pdf-wasm
chore/readme-cleanup
feat/phase-b-webapp
feat/phase-a2-fragmented-mp4
No results found.
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
e-copy

Phase E.3 — trust copy (empty-state, offline indicator)

  
e-features

Phase E.2 — feature additions (folder zip, metadata diff, Web Share Target)

  
e-mobile

Phase E.1 — mobile-critical UX (touch, picker, install prompt, size caps, messaging)

  
enhancement

New feature or request

  
f-coverage

Phase F.1 — format coverage (HEIC, future PNG/WebP/GIF/BMP/TIFF walkers)

  
f-forensic

Phase F.3 — forensic + invariant tests (no-network test, sentinel batteries)

  
f-perf

Phase F.4 — performance + privacy correctness (streaming, memory caps)

  
f-privacy

Phase F.2 — privacy-gap closure (Office Phase 2 hardening, etc.)

  
forensic

Forensic verification work — sentinel-based recovery testing per format-strategy-workflow.md

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
infra

Build/deploy/CI infrastructure work

  
invalid

This doesn't seem right

  
phase-a

Failing formats

  
phase-b

Webapp shell

  
phase-c

Testing

  
phase-d

Electron simplification

  
phase-e

Feature additions

  
phase-f

Depth and coverage hardening: format-strategy work, forensic verification, privacy correctness

  
phase-g

Electron retirement — web-only build; ships before Phase E/F feature work

  
phase-h

Deferred from Phase E — platform/mobile UX refinement

  
priority-1

Must ship first

  
priority-2

Ships second

  
priority-3

Polish and features

  
privacy

Privacy-correctness work, invariant alignment, gap documentation

  
question

Further information is requested

  
v5

Deferred to V5 — not blocking current work

  
v6

Deferred to v6 — not in v5 scope

  
video-hardening

Video metadata stripping hardening — formerly the D1.x sub-tasks

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
e-copy
e-features
e-mobile
enhancement
f-coverage
f-forensic
f-perf
f-privacy
forensic
good first issue
help wanted
infra
invalid
phase-a
phase-b
phase-c
phase-d
phase-e
phase-f
phase-g
phase-h
priority-1
priority-2
priority-3
privacy
question
v5
v6
video-hardening
wontfix
Milestone
Clear milestone
No items
No milestone
v5 — Webapp Migration
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: forgejo_admin/exifcleaner-web#2
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?