fix: mutable remote refs

[nadilas]

TL;DR

Fix a bug in restoring from history in the Ogre repository.

What changed?

Added a test for restoring from history and a test for remoteRefs not changing on commit. Also added utility functions immutableMapCopy and mutableMapCopy.

How to test?

Run the new tests added and verify the correct behavior.

Why make this change?

Since maps store objects by reference, remoteRefs were mutated during commits so they became unusable.

---

https://macarthur.me/posts/maps-store-objects-by-reference/

📦 Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @dotinc/[email protected]
npm install @dotinc/[email protected]
# or 
yarn add @dotinc/[email protected]
yarn add @dotinc/[email protected]

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
ogre	⬜️ Ignored (Inspect)	Visit Preview		Apr 4, 2024 6:50pm

[nadilas]

• fix: mutable remote refs #171 👈
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @nadilas and the rest of your teammates on Graphite

[codecov]

Codecov Report

Attention: Patch coverage is 93.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.59%. Comparing base (5a08e4b) to head (7a45731).
Report is 2 commits behind head on main.

❗ Current head 7a45731 differs from pull request most recent head 5f575db. Consider uploading reports for the commit 5f575db to get more accurate results

Files	Patch %	Lines
packages/ogre/src/repository.ts	66.66%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #171      +/-   ##
==========================================
+ Coverage   76.83%   77.59%   +0.75%     
==========================================
  Files           6        6              
  Lines         354      366      +12     
  Branches       78       80       +2     
==========================================
+ Hits          272      284      +12     
+ Misses         55       54       -1     
- Partials       27       28       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

🚀 PR was released in @dotinc/[email protected], @dotinc/[email protected], @dotinc/[email protected] 🚀