Skip to content

fix: cache-bust the nodeps nbextension bundles - #1170

Merged
maartenbreddels merged 1 commit into
masterfrom
fix/nbextension-nodeps-cache-busting
Jul 4, 2026
Merged

fix: cache-bust the nodeps nbextension bundles#1170
maartenbreddels merged 1 commit into
masterfrom
fix/nbextension-nodeps-cache-busting

Conversation

@maartenbreddels

Copy link
Copy Markdown
Contributor

Summary

The requirejs urlArgs cache-busting only matched */extension module ids, so the nodeps bundles — which contain the actual widget frontends and are loaded via the requirejs map config (jupyter-vuenbextensions/jupyter-vue/nodeps) — were requested without any version parameter. Browsers then keep serving a stale cached copy after the extension is updated, which breaks pages in ways that show no console error (e.g. a widget manager waiting forever on a model the old bundle doesn't have — the page just sits at "Loading app").

The extension-directory hash already covers the nodeps files (it hashes every file in the extension directories), so the fix is to normalize the module id: strip the nbextensions/ prefix variants and map <ext>/nodeps to <ext>/extension before the hash lookup.

Verified locally: nodeps.js is now requested as nodeps.js?<hash> and stale-cache pages recover on a plain reload.

Found while working on ES module support for ipyvue (#1169), where every frontend rebuild was silently masked by this.

🤖 Generated with Claude Code

The requirejs urlArgs hash lookup only matched */extension ids, so the
nodeps bundles (which contain the actual widget frontends, mapped via
requirejs config) were served without any version parameter - browsers
keep serving a stale cached copy after the extension updates, which
breaks pages in ways that show no console error. The extension directory
hash already covers the nodeps files; normalize the id so they resolve
to it.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@maartenbreddels
maartenbreddels temporarily deployed to fix/nbextension-nodeps-cache-busting - solara-stable PR #1170 July 4, 2026 14:04 — with Render Destroyed
@maartenbreddels
maartenbreddels merged commit 091e5bb into master Jul 4, 2026
33 of 34 checks passed
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.

1 participant