Skip to content

feat(makernote): add Sony MakerNote parser#25

Merged
rpuneet merged 1 commit intomainfrom
feat/sony-makernote
Feb 7, 2026
Merged

feat(makernote): add Sony MakerNote parser#25
rpuneet merged 1 commit intomainfrom
feat/sony-makernote

Conversation

@rpuneet
Copy link
Contributor

@rpuneet rpuneet commented Feb 7, 2026

Summary

  • Implement Sony MakerNote handler for 'SONY DSC ' and 'SONY CAM ' cameras
  • IFD parsing with proper offset calculation (OffsetAbsolute, little-endian)
  • Tag name mappings for 60+ Sony-specific tags
  • Comprehensive test coverage (7 test functions)

Key Features

  • Header detection for Sony cameras
  • Tag decoding: SonyModelID, LensID, SerialNumber, Quality, WhiteBalance, etc.
  • Proper integration with MakerNote registry

Test plan

  • Unit tests for Detect, Parse, TagName
  • Tests for single and multiple IFD entries
  • Edge cases: unknown tags, type sizes

Part of MakerNote Support Epic (Issue #9)
Implements work-009 (MNI-3)

🤖 Generated with Claude Code

Implement Sony MakerNote handler with:
- Header detection for 'SONY DSC ' and 'SONY CAM '
- IFD parsing with proper offset calculation
- Tag name mappings for 60+ Sony-specific tags
- Comprehensive test coverage (7 test functions)

Key tags decoded:
- SonyModelID (0xb001), LensID (0xb027)
- SerialNumber (0x2031), Quality (0x0102)
- WhiteBalance, CreativeStyle, ColorMode, etc.

Part of MakerNote Support Epic (Issue #9)
Implements work-009 (MNI-3)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Feb 7, 2026

Codecov Report

❌ Patch coverage is 37.81513% with 148 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/parser/tiff/makernote/sony/sony.go 37.02% 135 Missing and 13 partials ⚠️

📢 Thoughts on this report? Let us know!

@rpuneet rpuneet merged commit 7a48ad3 into main Feb 7, 2026
6 of 7 checks passed
@rpuneet rpuneet mentioned this pull request Feb 7, 2026
5 tasks
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