Skip to content

Open Olympus ORI and Sinar STI RAW files - #93

Merged
roprgm merged 2 commits into
mainfrom
raw-extensions
Oct 13, 2026
Merged

roprgm merged 2 commits into
mainfrom
raw-extensions

Conversation

@roprgm

@roprgm roprgm commented Oct 12, 2026 •

Copy link
Copy Markdown
Owner

Change

.ori (Olympus and OM System) and .sti (Sinar) files now open as RAW. LibRaw already decodes both; only OpenLight's extension list turned them away with "Unsupported image".

Verification

  • bun run check, bun run build, and bun run test pass.
  • Opened real samples from raw.pixls.us in the app: .ori from the OM-1, OM-1 II, OM-5, OM-5 II, E-M1 II, E-M1 III, and E-M5 II, and .sti from a Sinarback eVolution 75 all render. OM-3 .ori files still fail, because LibRaw has no color matrix for that camera; raw-webgpu will fix that.

🤖 Generated with Claude Code


Devin Review

Summary by CodeRabbit

  • New Features
    • Added support for .ori and .sti RAW image files. These extensions are now recognized and handled by the existing RAW image decoder, alongside the other supported RAW formats. Existing RAW file support is unchanged. This expands the range of RAW file extensions the app can process.

LibRaw already decodes both; only the extension list turned them away.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vercel

vercel Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
openlight Ready Ready Preview Oct 13, 2026 9:55pm UTC

Request Review

@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 89a456e5-86df-4f89-b2f0-2804aaf97ef6
📥 Commits

Reviewing files that changed from the base of the PR and between 2ba9336 and 613e1c9.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: d14bf10d-0ab2-492b-96b8-7c94e1c4df11
📥 Commits

Reviewing files that changed from the base of the PR and between 51e9aea and 2ba9336.

📒 Files selected for processing (1)
  • src/core/image/decode/index.ts

Included review availability: This review used your included allowance. 3 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📝 Walkthrough

The RAW decoder’s recognized extensions now include ori and sti. Files with these extensions are routed through the existing RAW decoder.

Changes: RAW extension recognition

Layer / File(s) Summary
Add RAW extensions
src/core/image/decode/index.ts
The RAW extension list now includes ori and sti.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 2ba93

This change enables two additional RAW extensions, and no concrete merge-blocking issue is identified.

Pre-merge checks | Passed 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check Passed The description includes both required sections. It explains the .ori and .sti changes, the prior unsupported-image behavior, verification commands, tested camera samples, and the known OM-3 limit…
Title check Passed The title clearly and concisely identifies the primary change: support for Olympus ORI and Sinar STI RAW files.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 Devin Review: 1 flag

Not posted on this PR by your GitHub settings — view it in Devin Review. (Configure)

Devin Review

@roprgm
roprgm merged commit e317d3b into main Oct 13, 2026
5 checks passed

This branch was successfully deployed

1 active deployment
Preview — 613e1c90 Deployed Oct 13, 2026 by vercel[bot]
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