Fix compass button undefined error

[Lucaci-Andrei]

Summary by CodeRabbit

• Bug Fixes
  • Fixed a TypeError in the my-position component that occurred when the compass button style was accessed before the element was fully rendered.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

Added a null/undefined guard in mi-my-position's setCompassStyle to avoid accessing the compass button's style before the element exists; updated two changelogs documenting the fix (components v13.32.1, map-template v1.89.6).

Changes

Cohort / File(s)	Summary
Bug fix (component) packages/components/src/components/my-position/my-position.tsx	Added a safety check ensuring compassButton is defined before reading/setting style.transform, preventing a TypeError when the element isn't rendered yet.
Changelog updates packages/components/CHANGELOG.md, packages/map-template/CHANGELOG.md	Added entries for components v13.32.1 and map-template v1.89.6 noting the fix for the mi-my-position compass button TypeError.

Sequence Diagram(s)

(Skipped — change is a small defensive check; no new multi-component control flow introduced.)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

• matbmapspeople
• ammapspeople

Poem

🐰 I poked a button not yet grown,

style unseen, a TypeError sown.
A tiny guard now holds the gate,
No more crashes — tidy fate. ✨

Pre-merge checks

✅ 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 'Fix compass button undefined error' directly and clearly summarizes the main change: fixing an undefined error related to the compass button by adding a safety check before accessing the element.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c0e84b8 and 3e44644.

📒 Files selected for processing (2)

• packages/components/CHANGELOG.md
• packages/map-template/CHANGELOG.md

🚧 Files skipped from review as they are similar to previous changes (2)

• packages/components/CHANGELOG.md
• packages/map-template/CHANGELOG.md

---

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.}

[matbmapspeople]

LGTM. Did you run npm i && npx lerna run build? Just to make sure all is up-to-date

[coderabbitai]

Actionable comments posted: 2

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

packages/map-template/CHANGELOG.md (1)

8-19: Resolve duplicate version number conflict.

There are two separate entries for version 1.89.5 with different dates and different changes. This violates semantic versioning principles.

If version 1.89.5 was already released on 2025-12-16, the new fix should be versioned as 1.89.6 (or the next appropriate version). If 1.89.5 hasn't been released yet, consolidate these entries into a single version.

🤖 Fix all issues with AI Agents

In @packages/components/CHANGELOG.md:
- Line 8: Update the version date on the changelog header "## [13.32.1] -
2025-01-06" to the correct year by changing it to "## [13.32.1] - 2026-01-06" so
the entry is correctly ordered after the 2025-12-11 entry.

In @packages/map-template/CHANGELOG.md:
- Line 8: Update the release header line "## [1.89.5] - 2025-01-06" to use the
correct year by changing the date to "2026-01-06"; additionally, to avoid a
duplicate version entry, consider bumping the version token in that header to
"1.89.6" (i.e., change "## [1.89.5] - 2025-01-06" to either "## [1.89.5] -
2026-01-06" if the version is correct, or "## [1.89.6] - 2026-01-06" if you need
a new release entry).

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 97f265b and c0e84b8.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (3)

• packages/components/CHANGELOG.md
• packages/components/src/components/my-position/my-position.tsx
• packages/map-template/CHANGELOG.md

🔇 Additional comments (1)

packages/components/src/components/my-position/my-position.tsx (1)

406-410: LGTM! Good defensive programming.

The null check prevents the TypeError when compassButton hasn't been rendered yet. This handles edge cases where setCompassStyle might be called before the ref is assigned.