Skip to content

add resilient AnimView fallback and cache bust query param#82

Open
mmsqe wants to merge 2 commits intowill62794:masterfrom
mmsqe:fix_anim
Open

add resilient AnimView fallback and cache bust query param#82
mmsqe wants to merge 2 commits intowill62794:masterfrom
mmsqe:fix_anim

Conversation

@mmsqe
Copy link
Contributor

@mmsqe mmsqe commented Mar 1, 2026

  • add AnimView fallback is to prevent Animation tab from a hard Error evaluating animation view expression: 'AnimView'. to renders a fallback message and logs debug details like nodePos, nodeText, fingerprint.
  • add cache busting (__ts) is to avoid stale spec/anim fetches during local edits, so UI evaluates the latest file content instead of cached old text.
  • local server query-path fix is to ensure /local_dir/*.tla?__ts=... resolves correctly by stripping query params before filesystem mapping in serve.py, while remote spec URLs are unaffected which are fetched directly by browser from remote host.

@will62794
Copy link
Owner

Can you explain the motivation behind this change more clearly? The bullet points are a bit vague to me.

@mmsqe
Copy link
Contributor Author

mmsqe commented Mar 3, 2026

Can you explain the motivation behind this change more clearly? The bullet points are a bit vague to me.

Sure @will62794, just updated the description, we encountered local specs where AnimView throws during evaluation, adding a fallback plus diagnostics should make debugging easier.

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