fix: make GeoPoint portable through json

[fwal]

Make GeoPoint as portable as the other types

Summary by CodeRabbit

• New Features

  • Added Firestore GeoPoint model support for select, insert, update, and JSON operations.
  • GeoPoint values now round-trip safely between native Firestore instances and JSON-compatible latitude/longitude objects.
  • Exported GeoPoint helpers through the Firestore API.

• Tests

  • Added coverage for GeoPoint encoding, decoding, and JSON serialization scenarios.

• Documentation

  • Improved formatting of the built-in field helpers documentation.

[coderabbitai]

📝 Walkthrough

Walkthrough

Adds Firestore GeoPoint model support with database and JSON representations, exports the new helper through public Firestore modules, tests its transformations, and reformats helper documentation. Package manifest edits only adjust end-of-file formatting.

Changes

Firestore GeoPoint support

Layer / File(s)	Summary
GeoPoint schema and model mapping packages/effect-firebase/src/lib/firestore/schema/geopoint.ts, packages/effect-firebase/src/lib/firestore/model/geopoint.ts, packages/effect-firebase/src/lib/firestore/model/geopoint.spec.ts	Defines GeoPoint instance validation, database mappings, JSON representations, and tests for select, insert, and JSON round trips.
Public exports and helper documentation packages/effect-firebase/src/lib/firestore/model/index.ts, packages/effect-firebase/src/lib/firestore/firestore.ts, packages/effect-firebase/README.md	Re-exports the GeoPoint helper and reformats the built-in field helpers table without changing its described behaviors.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

I’m a bunny with points in a cloud,
Latitude, longitude—coordinates proud.
Through JSON I hop, then return as new,
A Firestore GeoPoint, sturdy and true.
Barrels now share what my models can do!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly matches the main change: making GeoPoint portable through JSON.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

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

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@packages/effect-firebase/README.md`:
- Around line 37-46: Add the new Model.GeoPoint helper to the built-in field
helpers table in the README, including a concise description of its Firestore
geographic-point behavior consistent with the surrounding entries.

In `@packages/effect-firebase/src/lib/firestore/model/geopoint.ts`:
- Around line 1-4: Remove the unused Schema import from geopoint.ts while
preserving the Model, VariantSchema, and FirestoreSchema imports.

🪄 Autofix (Beta)

✅ Autofix completed

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7d41de7a-c976-4ee5-8d8c-c7aad1d969d0

📥 Commits

Reviewing files that changed from the base of the PR and between dd31795 and ea3615f.

📒 Files selected for processing (10)

• example/app/package.json
• example/backend/package.json
• example/shared/package.json
• package.json
• packages/effect-firebase/README.md
• packages/effect-firebase/src/lib/firestore/firestore.ts
• packages/effect-firebase/src/lib/firestore/model/geopoint.spec.ts
• packages/effect-firebase/src/lib/firestore/model/geopoint.ts
• packages/effect-firebase/src/lib/firestore/model/index.ts
• packages/effect-firebase/src/lib/firestore/schema/geopoint.ts

[coderabbitai]

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Fixes Applied Successfully

Fixed 1 file(s) based on 1 unresolved review comment.

Files modified:

• packages/effect-firebase/README.md

Commit: 0af27fa138ac24d5065de1694415544791de3173

The changes have been pushed to the fwal/portable-geopoint branch.

Time taken: 3m 59s