Reimplement mafsStateToInteractiveGraph

[benchristel]

This change was previously implemented in PR #1654, then reverted in #1663 due
to a bug. This PR reimplements the change without the bug. The original PR
summary follows.

---

Previously, we were passing the entire Mafs graph state to onChange. Since
the onChange callback data is used by the Interactive Graph widget editor to
set the correct answer for the graph, a bunch of extra data like
"hasBeenInteractedWith": true was getting saved in the correct field of
the Widget data.

This extra data is not used, so its presence in datastore is potentially
confusing to future maintainers trying to understand the data. It might
also cause bugs in the future - e.g. if some code merges it into another
object in which those properties are meaningful.

This PR ensures that we only save relevant data in the correct field of
interactive graph widgets.

Issue: none

Test plan:

yarn dev; open http://localhost:5173

Play around with the graphs on the dev UI gallery page. In particular:

• Unlimited point
• Polygons with angle and side labels
• Polygons with side snapping

You should not observe any issues.

Edit, save and publish an interactive graph widget in an exercise. As a
learner, you should be able to complete the exercise successfully.

[khan-actions-bot]

Gerald

Required Reviewers

• @Khan/perseus for changes to .changeset/eleven-elephants-trade.md, packages/perseus/src/types.ts, packages/perseus/src/widgets/interactive-graph.tsx, packages/perseus/src/widgets/interactive-graphs/mafs-state-to-interactive-graph.test.ts, packages/perseus/src/widgets/interactive-graphs/mafs-state-to-interactive-graph.ts, packages/perseus/src/widgets/interactive-graphs/stateful-mafs-graph.tsx

---

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

[github-actions]

npm Snapshot: Published

Good news!! We've packaged up the latest commit from this PR (21fc055) and published it to npm. You
can install it using the tag PR1664.

Example:

yarn add @khanacademy/perseus@PR1664

If you are working in Khan Academy's webapp, you can run:

./dev/tools/bump_perseus_version.sh -t PR1664

[github-actions]

Size Change: +359 B (+0.04%)

Total Size: 863 kB

Filename	Size	Change
packages/perseus/dist/es/index.js	418 kB	+359 B (+0.09%)

ℹ️ View Unchanged

Filename	Size
packages/kas/dist/es/index.js	38.3 kB
packages/keypad-context/dist/es/index.js	760 B
packages/kmath/dist/es/index.js	4.27 kB
packages/math-input/dist/es/index.js	78 kB
packages/math-input/dist/es/strings.js	1.79 kB
packages/perseus-core/dist/es/index.js	1.48 kB
packages/perseus-editor/dist/es/index.js	279 kB
packages/perseus-linter/dist/es/index.js	22.2 kB
packages/perseus/dist/es/strings.js	3.4 kB
packages/pure-markdown/dist/es/index.js	3.66 kB
packages/simple-markdown/dist/es/index.js	12.4 kB

_{compressed-size-action}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.07%. Comparing base (1642ad9) to head (1b238b8).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1664      +/-   ##
==========================================
+ Coverage   70.87%   71.07%   +0.19%     
==========================================
  Files         561      588      +27     
  Lines      108250   112800    +4550     
  Branches     7979    12076    +4097     
==========================================
+ Hits        76727    80170    +3443     
- Misses      31401    32630    +1229     
+ Partials      122        0     -122     

see 1149 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1642ad9...1b238b8. Read the comment docs.

[jeremywiebe]

This looks correct to me.

[jeremywiebe]

Well, removing this is a sign that things are moving in the right direction! :)