feat: implement Apple vcgp (Video Card Gamma & Primary) tag decoder

[harbik]

Summary

• Adds vcgp (Video Card Gamma and Primary) decoder — an Apple-private tag found in macOS display profiles, distinct from vcgt
• Decodes the two-section 56-byte payload:
  • Calibration section (bytes 8–31): per-channel gamma_in (native panel gamma, typically 3.0) and gamma_out (target EOTF gamma, typically 2.4) as S15.16 fixed-point
  • Chromaticity section (bytes 32–55): per-channel display primary primary_x / primary_y chromaticity coordinates, encoded as U32 normalized (value / 0xFFFF_FFFF → [0, 1]); uncalibrated profiles use placeholder values (x ≈ 0.0, y ≈ 0.2)
• Serializes to TOML as [vcgp.red], [vcgp.green], [vcgp.blue] sections with four fields each
• Graceful fallback to zeros on malformed or short input
• 4 unit tests: real profile values, calibrated sRGB chromaticity round-trip, zero payload, malformed short input
• CHANGELOG updated under [Unreleased]

Closes #20

Test plan

• cargo test — all 42 tests pass
• cargo clippy -- -D warnings — zero warnings
• cargo doc --no-deps — zero warnings

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Implements decoding for Apple’s private vcgp (Video Card Gamma and Primary) tag so it serializes into structured, per-channel TOML instead of falling back to raw hex.

Changes:

• Added a vcgp payload decoder with typed layout parsing and TOML-serializable structures.
• Registered vcgp in ParsedTag so it’s emitted as a first-class decoded tag.
• Updated CHANGELOG.md to document the new decoder.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
src/tag/tagdata/vcgp.rs	Adds vcgp binary layout parsing, TOML-serializable structs, and unit tests.
src/tag/parsed_tag.rs	Wires TagData::Vcgp into the ParsedTag serialization path.
CHANGELOG.md	Documents the new vcgp decoding behavior under Unreleased.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Layout::try_ref_from_bytes(data.0.as_slice()) will fail if the tag payload is larger than exactly 56 bytes (e.g., if a profile includes trailing padding/extra vendor bytes), causing an unnecessary all-zero fallback. Consider parsing only the first 56 bytes (e.g., via data.0.get(..56) before try_ref_from_bytes, or a *_from_prefix helper) so valid prefixes still decode.

Suggested change

	let Some(layout) = Layout::try_ref_from_bytes(data.0.as_slice()).ok() else {
	return fallback();
	};
	let Some(prefix) = data.0.get(..core::mem::size_of::<Layout>()) else {
	return fallback();
	};
	let Some(layout) = Layout::try_ref_from_bytes(prefix).ok() else {
	return fallback();
	};

[Copilot]

The CHANGELOG entry does not match the implementation: the decoder outputs per-channel gamma_in, gamma_out, primary_x, and primary_y, where only the gamma fields are S15.16 and the primaries are normalized U32. Please update the text to avoid mentioning min/max and “All four values are decoded as S15.16”.

Suggested change

	to structured TOML with per-channel fields (`gamma_in`, `gamma_out`, `min`,
	`max`) instead of a raw hex dump. All four values are decoded as S15.16
	fixed-point, consistent with the encoding used by the related `vcgt` formula
	type.
	to structured TOML with per-channel fields (`gamma_in`, `gamma_out`,
	`primary_x`, `primary_y`) instead of a raw hex dump. `gamma_in` and
	`gamma_out` are decoded as S15.16 fixed-point values, while `primary_x`
	and `primary_y` are decoded as normalized U32 values.

[Copilot]

The CHANGELOG description calls vcgp “Video Card Gamma Profile”, but this PR (and the new decoder) treats it as “Video Card Gamma and Primary”. Updating the wording will reduce confusion with vcgt and align with the tag’s purpose.

Suggested change

	* **`vcgp` decoder** — Apple `vcgp` (Video Card Gamma Profile) tag now decodes
	* **`vcgp` decoder** — Apple `vcgp` (Video Card Gamma and Primary) tag now decodes

[harbik]

Closing without merging. The vcgp field interpretation (gamma_in/gamma_out as S15.16, primary_x/y as U32-normalized) is entirely reverse-engineered with no Apple documentation to validate against. gamma_in=3.0 is plausible but unconfirmed; the chromaticity section shows identical values across all three channels (x≈0, y≈0.2) in every test profile, which is not consistent with real display primaries. Silently wrong structured output is worse than an honest hex dump. vcgp will continue to fall through to the Raw handler until we have better evidence of the actual encoding.