Would supporting batchStart and batchEnd methods to save multiple modifications as a single history snapshot be a good solution?

[tolerance-go]

I am encountering issues with managing historical states. I want to merge several modifications into a single operation history. My solution is to support batchStart and batchEnd methods to save multiple modifications as a single history snapshot. What do you all think?

// replace the following line
// import { proxyWithHistory } from 'valtio/utils';

import { proxyWithHistory } from 'valtio-history';
import { useSnapshot } from 'valtio';

const state = proxyWithHistory({ count: 0 });
console.log(state.value); // ---> { count: 0 }
state.batchStart();
state.value.count += 1;
state.value.count += 1;
state.batchEnd();
console.log(state.value); // ---> { count: 2 }
state.undo();
console.log(state.value); // ---> { count: 0 }
state.redo();
console.log(state.value); // ---> { count: 2 }

// React example
export default function App() {
  const {
    value,
    undo,
    redo,
    history,
    isUndoEnabled,
    isRedoEnabled,
    currentChangeDate,
    remove,
    replace,
} = useSnapshot(state);

  ...
}

[lwhiteley]

hi @tolerance-go ,

can you please give a reproduction of your issue.

As far as i know, it is possible to batch changes already once all the changes happen in the same scope.