From 0dbff79357d252b96428c1cd7c611c0421219905 Mon Sep 17 00:00:00 2001
From: Rosio Reyes <rosioreyes@Rosios-MBP.lan1>
Date: Wed, 11 May 2022 12:34:42 -0700
Subject: [PATCH] remove bower, install js dependencies with npm using
 bower-lite script to copy them into components folder. this is applying the
 changes made by @minrk in 
 https://github.com/jupyter/notebook/pull/6213/files

---
 .bowerrc                          |  3 ---
 MANIFEST.in                       |  1 +
 bower-lite                        | 37 +++++++++++++++++++++++++++++++
 bower.json                        | 30 -------------------------
 nbclassic/templates/page.html     | 14 ++++++------
 nbclassic/templates/terminal.html |  2 +-
 package.json                      | 32 ++++++++++++++++++++++----
 setupbase.py                      | 36 +++++++++++++++---------------
 tools/build-main.js               |  6 ++---
 9 files changed, 95 insertions(+), 66 deletions(-)
 delete mode 100644 .bowerrc
 create mode 100644 bower-lite
 delete mode 100644 bower.json

diff --git a/.bowerrc b/.bowerrc
deleted file mode 100644
index 25442973e..000000000
--- a/.bowerrc
+++ /dev/null
@@ -1,3 +0,0 @@
-{
-  "directory": "nbclassic/static/components"
-}
\ No newline at end of file
diff --git a/MANIFEST.in b/MANIFEST.in
index 5004858d5..753237768 100644
--- a/MANIFEST.in
+++ b/MANIFEST.in
@@ -1,4 +1,5 @@
 include LICENSE
+include bower-lite
 
 # added by check-manifest
 include *.md
diff --git a/bower-lite b/bower-lite
new file mode 100644
index 000000000..eec33864a
--- /dev/null
+++ b/bower-lite
@@ -0,0 +1,37 @@
+#!/usr/bin/env python3
+# Copyright (c) Jupyter Development Team.
+# Distributed under the terms of the Modified BSD License.
+"""
+bower-lite
+Since Bower's on its way out,
+stage frontend dependencies from node_modules into components
+"""
+import json
+import os
+import shutil
+from os.path import join
+
+HERE = os.path.abspath(os.path.dirname(__file__))
+
+
+components = join(HERE, "nbclassic", "static", "components")
+node_modules = join(HERE, "node_modules")
+
+if os.path.exists(components):
+    shutil.rmtree(components)
+os.mkdir(components)
+
+with open(join(HERE, "package.json")) as f:
+    package_json = json.load(f)
+
+renames = {
+    "jquery-ui-dist": "jquery-ui",
+}
+
+dependencies = package_json["dependencies"]
+for dep in dependencies:
+    src = join(node_modules, dep)
+    dest_name = renames.get(dep, dep)
+    dest = join(components, dest_name)
+    print(f"{src} -> {dest}")
+    shutil.copytree(src, dest)
\ No newline at end of file
diff --git a/bower.json b/bower.json
deleted file mode 100644
index 5f72f6ff0..000000000
--- a/bower.json
+++ /dev/null
@@ -1,30 +0,0 @@
-{
-  "name": "jupyter-notebook-deps",
-  "version": "0.0.1",
-  "dependencies": {
-    "backbone": "components/backbone#~1.2",
-    "bootstrap": "bootstrap#~3.4",
-    "bootstrap-tour": "0.9.0",
-    "codemirror": "components/codemirror#5.56.0+components1",
-    "create-react-class": "https://cdn.jsdelivr.net/npm/create-react-class@15.6.3/create-react-class.min.js",
-    "es6-promise": "~1.0",
-    "font-awesome": "components/font-awesome#~4.7.0",
-    "google-caja": "5669",
-    "jed": "~1.1.1",
-    "jquery": "components/jquery#~3.5.0",
-    "jquery-typeahead": "~2.10.6",
-    "jquery-ui": "components/jqueryui#~1.12",
-    "marked": "~0.7",
-    "MathJax": "^2.7.4",
-    "moment": "~2.19.3",
-    "react": "~16.0.0",
-    "requirejs": "~2.2",
-    "requirejs-text": "~2.0.15",
-    "requirejs-plugins": "~1.0.3",
-    "text-encoding": "~0.1",
-    "underscore": "components/underscore#~1.8.3",
-    "xterm.js": "https://unpkg.com/xterm@~3.1.0/dist/xterm.js",
-    "xterm.js-css": "https://unpkg.com/xterm@~3.1.0/dist/xterm.css",
-    "xterm.js-fit": "https://unpkg.com/xterm@~3.1.0/dist/addons/fit/fit.js"
-  }
-}
diff --git a/nbclassic/templates/page.html b/nbclassic/templates/page.html
index 223731903..35ec629ba 100644
--- a/nbclassic/templates/page.html
+++ b/nbclassic/templates/page.html
@@ -7,7 +7,7 @@
     <title>{% block title %}Jupyter Notebook{% endblock %}</title>
     {% block favicon %}<link id="favicon" rel="shortcut icon" type="image/x-icon" href="{{static_url("base/images/favicon.ico") }}">{% endblock %}
     <meta http-equiv="X-UA-Compatible" content="IE=edge" />
-    <link rel="stylesheet" href="{{static_url("components/jquery-ui/themes/smoothness/jquery-ui.min.css") }}" type="text/css" />
+    <link rel="stylesheet" href="{{static_url("components/jquery-ui-themes/themes/smoothness/jquery-ui.min.css") }}" type="text/css" />
     <link rel="stylesheet" href="{{static_url("components/jquery-typeahead/dist/jquery.typeahead.min.css") }}" type="text/css" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0">
     
@@ -15,10 +15,10 @@
     <link rel="stylesheet" href="{{ static_url("style/style.min.css") }}" type="text/css"/>
     {% endblock %}
     <link rel="stylesheet" href="{{ base_url }}custom/custom.css" type="text/css" />
-    <script src="{{static_url("components/es6-promise/promise.min.js")}}" type="text/javascript" charset="utf-8"></script>
-    <script src="{{static_url('components/react/react.production.min.js')}}" type="text/javascript"></script>
-    <script src="{{static_url('components/react/react-dom.production.min.js')}}" type="text/javascript"></script>
-    <script src="{{static_url('components/create-react-class/index.js')}}" type="text/javascript"></script>
+    <script src="{{static_url("components/es6-promise/dist/promise-1.0.0.min.js")}}" type="text/javascript" charset="utf-8"></script>
+    <script src="{{static_url('components/react/umd/react.production.min.js')}}" type="text/javascript"></script>
+    <script src="{{static_url('components/react-dom/umd/react-dom.production.min.js')}}" type="text/javascript"></script>
+    <script src="{{static_url('components/create-react-class/create-react-class.js')}}" type="text/javascript"></script>
     <script src="{{static_url('components/requirejs/require.js') }}" type="text/javascript" charset="utf-8"></script>
     <script>
       require.config({
@@ -34,7 +34,7 @@
             underscore : 'components/underscore/underscore-min',
             backbone : 'components/backbone/backbone-min',
             jed: 'components/jed/jed',
-            jquery: 'components/jquery/jquery.min',
+            jquery: 'components/jquery/dist/jquery.min',
             json: 'components/requirejs-plugins/src/json',
             text: 'components/requirejs-text/text',
             bootstrap: 'components/bootstrap/dist/js/bootstrap.min',
@@ -42,7 +42,7 @@
             'jquery-ui': 'components/jquery-ui/jquery-ui.min',
             moment: 'components/moment/min/moment-with-locales',
             codemirror: 'components/codemirror',
-            termjs: 'components/xterm.js/xterm',
+            termjs: 'components/xterm/dist/xterm',
             typeahead: 'components/jquery-typeahead/dist/jquery.typeahead.min',
           },
           map: { // for backward compatibility
diff --git a/nbclassic/templates/terminal.html b/nbclassic/templates/terminal.html
index b61c13b22..c2cf24620 100644
--- a/nbclassic/templates/terminal.html
+++ b/nbclassic/templates/terminal.html
@@ -15,7 +15,7 @@
 {% block stylesheet %}
 {{super()}}
 <link rel="stylesheet" href="{{ static_url("terminal/css/override.css") }}" type="text/css" />
-<link rel="stylesheet" href="{{static_url("components/xterm.js-css/index.css") }}" type="text/css" />
+<link rel="stylesheet" href="{{static_url("components/xterm/dist/xterm.css") }}" type="text/css" />
 {% endblock %}
 
 {% block headercontainer %}
diff --git a/package.json b/package.json
index 9f905bbd7..b2ad47d92 100644
--- a/package.json
+++ b/package.json
@@ -10,11 +10,11 @@
     "url": "https://github.com/jupyter/nbclassic.git"
   },
   "scripts": {
-    "bower": "bower install",
+    "bower": "python3 bower-lite",
     "build": "python setup.py js css",
     "build:webpack": "webpack --mode production",
     "build:watch": "npm run watch",
-    "watch": "onchange 'nbclassic/static/**/!(*.min).js' 'nbclassic/static/**/*.less' 'bower.json' -- npm run build"
+    "watch": "onchange 'nbclassic/static/**/!(*.min).js' 'nbclassic/static/**/*.less' -- npm run build"
   },
   "devDependencies": {
     "@babel/core": "^7.15.0",
@@ -22,12 +22,36 @@
     "@jupyterlab/apputils": "^3.1.3",
     "babel-loader": "^8.2.2",
     "babel-polyfill": "^6.26.0",
-    "bower": "^1.8.8",
     "less": "~2",
     "onchange": "^6.0.0",
     "po2json": "^0.4.5",
-    "requirejs": "^2.3.6",
     "webpack": "^5.46.0",
     "webpack-cli": "^4.7.2"
+  },
+  "dependencies": {
+    "backbone": "~1.2",
+    "bootstrap": "~3.4",
+    "bootstrap-tour": "^0.10.0",
+    "codemirror": "5.56.0",
+    "create-react-class": "^15.6.3",
+    "es6-promise": "~1.0",
+    "font-awesome": "~4.7.0",
+    "google-caja": "^0.0.2",
+    "jed": "~1.1.1",
+    "jquery": "~3.5.0",
+    "jquery-typeahead": "~2.10.6",
+    "jquery-ui-dist": "^1.12.0",
+    "jquery-ui-themes": "^1.12.0",
+    "marked": "^0.7.0",
+    "mathjax": "^2.7.4",
+    "moment": "~2.19.3",
+    "react": "~16.0.0",
+    "react-dom": "~16.0.0",
+    "requirejs": "^2.3.6",
+    "requirejs-plugins": "^1.0.2",
+    "requirejs-text": "~2.0.15",
+    "text-encoding": "~0.7.0",
+    "underscore": "~1.8.3",
+    "xterm": "~3.1.0"
   }
 }
diff --git a/setupbase.py b/setupbase.py
index 8cd7a48be..e4303bf25 100644
--- a/setupbase.py
+++ b/setupbase.py
@@ -133,21 +133,21 @@ def find_package_data():
         pjoin(components, "bootstrap", "dist", "js", "bootstrap.min.js"),
         pjoin(components, "bootstrap-tour", "build", "css", "bootstrap-tour.min.css"),
         pjoin(components, "bootstrap-tour", "build", "js", "bootstrap-tour.min.js"),
-        pjoin(components, "create-react-class", "index.js"),
+        pjoin(components, "create-react-class", "create-react-class.min.js"),
         pjoin(components, "font-awesome", "css", "*.css"),
-        pjoin(components, "google-caja", "html-css-sanitizer-minified.js"),
-        pjoin(components, "es6-promise", "*.js"),
+        pjoin(components, "google-caja", "sanitizer.js"),
+        pjoin(components, "es6-promise", "dist", "*.min.js"),
         pjoin(components, "font-awesome", "fonts", "*.*"),
         pjoin(components, "jed", "jed.js"),
-        pjoin(components, "jquery", "jquery.min.js"),
+        pjoin(components, "jquery", "dist", "jquery.min.js"),
         pjoin(components, "jquery-typeahead", "dist", "jquery.typeahead.min.js"),
         pjoin(components, "jquery-typeahead", "dist", "jquery.typeahead.min.css"),
         pjoin(components, "jquery-ui", "jquery-ui.min.js"),
-        pjoin(components, "jquery-ui", "themes", "smoothness", "jquery-ui.min.css"),
-        pjoin(components, "jquery-ui", "themes", "smoothness", "images", "*"),
+        pjoin(components, "jquery-ui-themes", "themes", "smoothness", "jquery-ui.min.css"),
+        pjoin(components, "jquery-ui-themes", "themes", "smoothness", "images", "*"),
         pjoin(components, "marked", "lib", "marked.js"),
-        pjoin(components, "react", "react.production.min.js"),
-        pjoin(components, "react", "react-dom.production.min.js"),
+        pjoin(components, "react", "umd",  "react.production.min.js"),
+        pjoin(components, "react-dom", "umd",  "react-dom.production.min.js"),
         pjoin(components, "requirejs", "require.js"),
         pjoin(components, "requirejs-plugins", "src", "json.js"),
         pjoin(components, "requirejs-text", "text.js"),
@@ -155,9 +155,9 @@ def find_package_data():
         pjoin(components, "underscore", "underscore-min.js"),
         pjoin(components, "moment", "moment.js"),
         pjoin(components, "moment", "min", "*.js"),
-        pjoin(components, "xterm.js", "index.js"),
-        pjoin(components, "xterm.js-css", "index.css"),
-        pjoin(components, "xterm.js-fit", "index.js"),
+        pjoin(components, "xterm", "dist", "xterm.js"),
+        pjoin(components, "xterm", "dist", "xterm.css"),
+        pjoin(components, "xterm", "dist", "addons", "fit", "fit.js"),
         pjoin(components, "text-encoding", "lib", "encoding.js"),
     ])
 
@@ -168,7 +168,7 @@ def find_package_data():
                 static_data.append(pjoin(parent, f))
 
     # Trim mathjax
-    mj = lambda *path: pjoin(components, 'MathJax', *path)
+    mj = lambda *path: pjoin(components, 'mathjax', *path)
     static_data.extend([
         mj('MathJax.js'),
         mj('config', 'TeX-AMS-MML_HTMLorMML-full.js'),
@@ -360,10 +360,10 @@ def run(self):
                     ])
 
 class Bower(Command):
-    description = "fetch static client-side components with bower"
+    description = "fetch static client-side components with npm"
 
     user_options = [
-        ('force', 'f', "force fetching of bower dependencies"),
+        ('force', 'f', "force fetching of JavaScript dependencies"),
     ]
 
     def initialize_options(self):
@@ -384,7 +384,7 @@ def should_run(self):
         if not os.path.exists(self.sanitizer_dir):
             return True
 
-        bower_stale = mtime(self.bower_dir) < mtime(pjoin(repo_root, 'bower.json'))
+        bower_stale = mtime(self.bower_dir) < mtime(pjoin(repo_root, 'package.json'))
         if bower_stale:
             return True
 
@@ -400,7 +400,7 @@ def should_run_npm(self):
 
     def run(self):
         if not self.should_run():
-            print("bower dependencies up to date")
+            print("JS dependencies up to date")
             return
 
         if self.should_run_npm():
@@ -413,12 +413,12 @@ def run(self):
 
         try:
             run(
-                ['bower', 'install', '--allow-root', '--config.interactive=false'],
+                [sys.executable, pjoin(repo_root, "bower-lite")],
                 cwd=repo_root,
                 env=env
             )
         except OSError as e:
-            print("Failed to run bower: %s" % e, file=sys.stderr)
+            print("Failed to run bower-lite: %s" % e, file=sys.stderr)
             print("You can install js dependencies with `npm install`", file=sys.stderr)
             raise
         # self.npm_components()
diff --git a/tools/build-main.js b/tools/build-main.js
index 973868680..dcfa39c5b 100644
--- a/tools/build-main.js
+++ b/tools/build-main.js
@@ -16,7 +16,7 @@ var rjs_config = {
     underscore : 'components/underscore/underscore-min',
     backbone : 'components/backbone/backbone-min',
     jed: 'components/jed/jed',
-    jquery: 'components/jquery/jquery.min',
+    jquery: 'components/jquery/dist/jquery.min',
     json: 'components/requirejs-plugins/src/json',
     text: 'components/requirejs-text/text',
     bootstrap: 'components/bootstrap/dist/js/bootstrap.min',
@@ -24,8 +24,8 @@ var rjs_config = {
     "jquery-ui": 'components/jquery-ui/jquery-ui.min',
     moment: 'components/moment/min/moment-with-locales',
     codemirror: 'components/codemirror',
-    xterm: 'components/xterm.js/index',
-    "xtermjs-fit": 'components/xterm.js-fit/index',
+    xterm: 'components/xterm/dist/xterm',
+    "xtermjs-fit": 'components/xterm/dist/addons/fit/fit',
     "jquery-typeahead": 'components/jquery-typeahead/dist/jquery.typeahead.min',
     contents: 'empty:',
     custom: 'empty:',