Repository navigation
Align Viewer docs with mixed-media Storage - #657
Merged
Merged
Conversation
kvz
marked this pull request as ready for review
November 14, 2026 07:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
The published CLI now stores mixed-media originals, but the Viewer reference still described image-only catalogs, warn-only integrity mismatches and sync rejecting non-images. This misleads people following the website onboarding flow tracked in transloadit/content#7151.
Changes
storeImage()callers that deliberately accept transformed image bytes.transloaditREADME from the reviewed Node README; both are byte-identical.--core-onlyfor focused runs). Verify exact local package artifacts, absent React/React DOM/Next, version-pinned public URLs, synchronous server signing, and per-request authorized GET/HEAD redirects. Add the neutral browser entry to existing bundle-budget tests.Documentation and regression-test changes only; no runtime behavior, package version or release is changed.
November 14 follow-up validation
yarn checkpasses, and generated legacy README synchronization passes.e9122ae: all 14 executed checks pass, including Node 20/22/24/26, the packed Next.js fixture, Supabase Edge, E2E and release dry run. The exact-head watcher exited 0; no package was published.Earlier batch validation
corepack yarn check.yarn lint:transloadit-syncafter generating the legacy wrapper README. The first CI run caught this missing derived file; it was cancelled before the corrected push, not treated as passing. CI on the corrected head is green, including the published-package fixtures and ready-only checks.Shipment
Squashmerged under Kevin's shipping approval on November 15 at 11:20:36 UTC as
0ebefa40bd5aadcdc6d14c99236f89eb62f8663e, after all 14 executed checks passed on exact heade9122ae85dfb9c301442af97bfe7bc8a4274a018. The initial GraphQL request returned HTTP 502, and its retry reported “merge already in progress”; fresh readback still showed open/clean, after which the SHA-bound REST merge succeeded and independent PR readback confirmed the merge. No code or CI was changed for recovery.Companion public guide: https://github.com/transloadit/content/pull/7428. Its reference dependency is now on main; Content follows after its main-conflict resolution and green new CI under Kevin's separate admin-squash/website-deploy approval. No new package version or manual npm publication is needed.
Related: https://github.com/transloadit/content/issues/7151 · DAM-071.
🤖 Claimed by kvz on
studio1/content-clone-7; framework1/node-sdksince 2026-11-14 · resume:codex resume 019fa2ee-f994-79d0-9d50-071bf2cb7ad6