improve field accessor retrieval with nil checks and type assertions

[matthyx]

Summary by CodeRabbit

• Bug Fixes
  • Improved field accessor retrieval to robustly handle nil receivers, invalid cached values, and type assertion failures, preventing potential application crashes.
  • Enhanced caching logic with comprehensive validation checks and strengthened fallback mechanisms to ensure reliable field access throughout the application.
  • Increased overall application stability by eliminating edge cases that could cause unexpected behavior.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

The getFieldAccessor function in datasource_event.go is enhanced with robust nil and type-safety checks. It now validates nil receivers, performs two-stage cache resolution using sync.Map, type-asserts cached values, and verifies field accessors implement the correct interface, returning a fallback accessor on any validation failure.

Changes

Cohort / File(s)	Summary
Nil safety and type assertion improvements pkg/utils/datasource_event.go	Added nil receiver check, two-stage cache resolution with explicit type assertions, nil guard for missing datasource fields, and accessor type verification. Cache now uses per-type sync.Map with LoadOrStore for field resolution. Fallback to missingFieldAccessor on any validation failure.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A cache so clever, now holds its ground,
With nil-checks and assertions all around,
No panics shall spring from types unknown,
Defensive walls protect each stone! 🛡️✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately and concisely describes the main change: adding nil checks and type assertions to improve field accessor retrieval robustness.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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