Skip to content

Add changeset for openjph-wasm switch (PR #65); fix dev.mjs import paths - #69

Merged
xinaesthete merged 1 commit into
mainfrom
add-openjph-wasm-changeset
Jun 29, 2026
Merged

Add changeset for openjph-wasm switch (PR #65); fix dev.mjs import paths#69
xinaesthete merged 1 commit into
mainfrom
add-openjph-wasm-changeset

Conversation

@xinaesthete

@xinaesthete xinaesthete commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What

  • Adds the missing changeset for PR #65 which switched the HTJ2K codec from @cornerstonejs/codec-openjph to openjph-wasm and added z>1 multi-component chunk support.
  • Fixes incorrect relative import paths in packages/vis/scripts/dev.mjs (paths had one ../ too few, breaking the dev server script).

Changeset summary

Package Bump Reason
zarrextra minor New Htj2kPlane export; behavioral change — z-planes now encoded as multi-component codestreams
@spatialdata/vis patch Dependency swap only (openjph-wasm replaces cornerstone codec)

Reviewer notes

The changeset was simply forgotten before merging #65 — no code changes beyond the import path fix. The bump levels can be adjusted if you disagree with minor vs patch for zarrextra.

Summary by CodeRabbit

  • New Features

    • Improved HTJ2K handling for volumetric data, preserving all components during lossless round-tripping.
    • Enhanced chunking for multi-component z>1 data to better encode z-planes within a single codestream.
    • Exposed Htj2kPlane for broader package use.
  • Bug Fixes

    • Resolved silent loss of extra components during HTJ2K decoding, improving data fidelity.
  • Chores

    • Updated package release version intents for related packages.

- Add missing changeset for PR #65 (switch HTJ2K codec to openjph-wasm,
  z>1 multi-component chunks, Htj2kPlane export)
- Fix relative import paths in packages/vis/scripts/dev.mjs (two dots
  too few, broke the dev server script)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ce8c1e26-4063-4b05-8f6f-815177efd448

📥 Commits

Reviewing files that changed from the base of the PR and between 113c6ae and c00faa3.

📒 Files selected for processing (2)
  • .changeset/switch-openjph-wasm.md
  • packages/vis/scripts/dev.mjs

📝 Walkthrough

Walkthrough

Adds a changeset file documenting the HTJ2K codec switch from @cornerstonejs/codec-openjph to openjph-wasm, multi-component z-plane encoding changes, and Htj2kPlane export. Fixes two relative import paths in packages/vis/scripts/dev.mjs to resolve one directory level higher.

Changes

Codec changeset and dev script path fix

Layer / File(s) Summary
Changeset and dev script import paths
.changeset/switch-openjph-wasm.md, packages/vis/scripts/dev.mjs
Changeset marks zarrextra as minor and @spatialdata/vis as patch, documenting the openjph-wasm codec switch and Htj2kPlane export. Dev script import paths for dev-process-utils.mjs and fixture-server-port.mjs are corrected to ../../../scripts/....

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐇 A codec was swapped, a path set right,
The openjph-wasm hops into the light,
Z-planes as components, no loss in the stream,
A changeset was written, or so it would seem.
Small fixes, big hops — the rabbit's delight! 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the two actual changes: the new changeset for the openjph-wasm switch and the dev.mjs import path fix.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-openjph-wasm-changeset

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@xinaesthete
xinaesthete merged commit e343a72 into main Jun 29, 2026
2 checks passed
@xinaesthete
xinaesthete deleted the add-openjph-wasm-changeset branch June 29, 2026 12:51
@github-actions github-actions Bot mentioned this pull request Jun 29, 2026
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.

1 participant