Remove requirejs as dependency (shiny now includes it and provides a shim for define()ing anonymous modules)

Author: cpsievert

ipyshiny/_dependencies.py

@@ -22,15 +22,6 @@
 # TODO: scripts/static_download.R should produce/update these
 def libembed_dependency() -> List[HTMLDependency]:
     return [
-        # Load 3rd party widget dependencies at runtime via requirejs. One of the benefits of doing it
-        # this way is that for whatever reason, if we can't find the widgets statically assets locally,
-        # the requireLoader we use client side will fallback to loading the dependencies from a CDN.
-        HTMLDependency(
-            name="requirejs",
-            version="2.3.4",
-            source={"package": "ipyshiny", "subdir": "static"},
-            script={"src": "require.min.js"},
-        ),
         # Jupyter Notebook/Lab both come "preloaded" with several @jupyter-widgets packages
         # (i.e., base, controls, output), all of which are bundled into this extension.js file
         # provided by the widgetsnbextension package, which is a dependency of ipywidgets.

ipyshiny/static/require.min.js

@@ -1,11 +1,6 @@
 
 static_dir <- file.path(getwd(), "ipyshiny", "static")
 if (!dir.exists(static_dir)) dir.create(static_dir)
-# From https://github.com/jupyter-widgets/ipywidgets/blob/fbdbd005/python/ipywidgets/ipywidgets/embed.py#L32
-download.file(
-  "https://cdnjs.cloudflare.com/ajax/libs/require.js/2.3.4/require.min.js",
-  file.path(static_dir, "require.min.js")
-)
 # From https://github.com/jupyter-widgets/ipywidgets/blob/fbdbd005/python/ipywidgets/ipywidgets/embed.py#L62
 # Note that we also grab libembed-amd, not embed-amd, because the Shiny bindings handle the actual rendering aspect
 # https://github.com/jupyter-widgets/ipywidgets/blob/fbdbd00/packages/html-manager/scripts/concat-amd-build.js#L6