Skip to content

Conversation

BrianHung
Copy link
Contributor

feat: add getRecentDiffs API with TypeScript interfaces

  • Add getRecentDiffs() method to expose recent changes without queue modification
  • Include WASM and Node.js bindings with proper serialization
  • Add comprehensive TypeScript interfaces for all diff types
  • Include tests for various diff scenarios and queue integrity

BrianHung added a commit to BrianHung/IronCalc that referenced this pull request Jun 13, 2025
Copy link
Member

@nhatcher nhatcher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting. The problem I see is that this introduces code but it is not clear the purpose. Please see comments in:

#371 (comment)

@BrianHung
Copy link
Contributor Author

The purpose of this PR is to let UI performantly listen to changes in the calculation engine, and only re-draw what is minimally necessary.

Re-drawing the entire grid every time we get a change is not a scalable pattern that works with other data grid UIs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants