Add missing GetHashCode methods

[myieye]

I'm basically just doing this, because the dotnet build warnings were bugging me 😄

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

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

📝 Walkthrough

Walkthrough

Added GetHashCode overrides for RichString and RichSpan classes to ensure hash code equality aligns with equality implementations. Updated tests to verify hash code correctness across multiple seeded iterations and per-property equality scenarios.

Changes

Cohort / File(s)	Summary
GetHashCode Implementation backend/FwLite/MiniLcm/Models/RichString.cs	Implemented GetHashCode overrides for RichString (aggregates span hash codes) and RichSpan (hashes 60+ style and content properties including text, tags, colors, borders, and formatting attributes).
Test Enhancements backend/FwLite/MiniLcm.Tests/RichMultiStringTests.cs	Removed unused directive, added RichString hash code equality tests (standard and 200 seeded iterations), enhanced RichSpan equality tests with hash assertions, introduced GenerateDifferentValue helper for reliable per-property tripwire validation.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

🐰 With hash codes dancing, spans align,
Properties bundled, refined and fine,
Through fifty tests our paths we trace,
Equality's truth in every space! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and directly describes the main change: adding GetHashCode method overrides to RichString and RichSpan classes.
Description check	✅ Passed	The description, while informal, is related to the changeset—it explains the motivation for adding the GetHashCode methods to resolve dotnet build warnings.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch add-missing-get-hash-code-methods

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

UI unit Tests

  1 files  ±0   50 suites  ±0   22s ⏱️ -1s
138 tests ±0  138 ✅ ±0  0 💤 ±0  0 ❌ ±0 
203 runs  ±0  203 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 0f0246b. ± Comparison against base commit 1d20393.

♻️ This comment has been updated with latest results.

[argos-ci]

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build	Status	Details	Updated (UTC)
default (Inspect)	✅ No changes detected	-	Feb 10, 2026, 2:54 PM