Rework history store

[chrisvxd]

Rework the history store to ensure intialHistories API works as expected. Previously, the historyStore would assume that data was the first item in history, and all history was additive. This PR reverses the logic, placing data at the end of the history to enable injection of custom histories.

Changes:

• Make initialHistories mandatory internally when using the useHistoryStore hook
• Set EMPTY_HISTORY_INDEX to 0 instead of -1, since some history always exists
• Set the last history item to the data prop
• Remove unnecessary id parameter in History type
• Fix an issue preventing viewports from resizing when changed programatically (encountered when testing history)

Before

Default history store:

[]

Injecting history via initialHistories prop API

[
  {
    data: {} // custom history as injected by `initialHistories ` prop
  }
]

After

Default history store

[
  {
    data: {} // `data` prop passed in via user
  } 
]

or when injecting via API

[
  {
    data: {} // custom history as injected by `initialHistories ` prop
  },
  {
    data: {} // `data` prop passed in via user
  } 
]

---

@mkilpatrick I think this should help with your challenges in #562. You can test this via 0.16.0-canary.464aa1e. I'd love to get this in ASAP as this API is now one of the things blocking release of v0.16.

[vercel[bot]]

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

Name	Status	Preview	Comments	Updated (UTC)
puck-demo	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 9, 2024 8:58pm
puck-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Sep 9, 2024 8:58pm