build: remove bootstrap and jquery from dev dependencies

Author: alan-agius4

package.json

@@ -123,7 +123,6 @@
     "autoprefixer": "10.4.19",
     "babel-loader": "9.1.3",
     "babel-plugin-istanbul": "6.1.1",
-    "bootstrap": "^4.0.0",
     "browser-sync": "3.0.2",
     "browserslist": "^4.21.5",
     "buffer": "6.0.3",
@@ -149,7 +148,6 @@
     "jasmine": "^5.0.0",
     "jasmine-core": "~5.1.0",
     "jasmine-spec-reporter": "~7.0.0",
-    "jquery": "^3.3.1",
     "jsonc-parser": "3.2.1",
     "karma": "~6.4.0",
     "karma-chrome-launcher": "~3.2.0",
@@ -176,7 +174,6 @@
     "patch-package": "^7.0.1",
     "picomatch": "4.0.2",
     "piscina": "4.4.0",
-    "popper.js": "^1.14.1",
     "postcss": "8.4.38",
     "postcss-loader": "8.1.1",
     "prettier": "^3.0.0",

packages/angular_devkit/build_angular/BUILD.bazel

@@ -357,9 +357,6 @@ LARGE_SPECS = {
         "extra_deps": [
             "@npm//@angular/animations",
             "@npm//@angular/material",
-            "@npm//bootstrap",
-            "@npm//jquery",
-            "@npm//popper.js",
         ],
     },
     "prerender": {},

packages/angular_devkit/build_angular/src/builders/browser/specs/styles_spec.ts

@@ -442,26 +442,6 @@ describe('Browser Builder styles', () => {
     4 * 60 * 1000,
   );
 
-  it(`supports bootstrap@4 with full path`, async () => {
-    const bootstrapPath = dirname(require.resolve('bootstrap/package.json'));
-
-    const overrides = {
-      styles: [bootstrapPath + '/dist/css/bootstrap.css'],
-      scripts: [bootstrapPath + '/dist/js/bootstrap.js'],
-    };
-
-    await browserBuild(architect, host, target, overrides);
-  });
-
-  it(`supports bootstrap@4 with package reference`, async () => {
-    const overrides = {
-      styles: ['bootstrap/dist/css/bootstrap.css'],
-      scripts: ['bootstrap/dist/js/bootstrap.js'],
-    };
-
-    await browserBuild(architect, host, target, overrides);
-  });
-
   it(`supports inline javascript in less`, async () => {
     const overrides = { styles: [`src/styles.less`] };
     host.writeMultipleFiles({

packages/angular_devkit/build_angular/src/builders/browser/tests/options/allowed-common-js-dependencies_spec.ts

@@ -18,7 +18,7 @@ describeBuilder(buildWebpackBrowser, BROWSER_BUILDER_INFO, (harness) => {
           aot ? 'AOT' : 'JIT'
         } Mode`, async () => {
           // Add a Common JS dependency
-          await harness.appendToFile('src/app/app.component.ts', `import 'bootstrap';`);
+          await harness.appendToFile('src/app/app.component.ts', `import 'ajv';`);
 
           harness.useTarget('build', {
             ...BASE_OPTIONS,
@@ -32,13 +32,13 @@ describeBuilder(buildWebpackBrowser, BROWSER_BUILDER_INFO, (harness) => {
           expect(logs).toContain(
             jasmine.objectContaining<logging.LogEntry>({
               message: jasmine.stringMatching(
-                /Warning: .+app\.component\.ts depends on 'bootstrap'\. CommonJS or AMD dependencies/,
+                /Warning: .+app\.component\.ts depends on 'ajv'\. CommonJS or AMD dependencies/,
               ),
             }),
           );
           expect(logs).not.toContain(
             jasmine.objectContaining<logging.LogEntry>({
-              message: jasmine.stringMatching('jquery'),
+              message: jasmine.stringMatching('require-from-string'),
             }),
             'Should not warn on transitive CommonJS packages which parent is also CommonJS.',
           );
@@ -51,14 +51,14 @@ describeBuilder(buildWebpackBrowser, BROWSER_BUILDER_INFO, (harness) => {
       await harness.appendToFile(
         'src/app/app.component.ts',
         `
-        import 'bootstrap';
+        import 'ajv';
         import 'zone.js';
       `,
       );
 
       harness.useTarget('build', {
         ...BASE_OPTIONS,
-        allowedCommonJsDependencies: ['bootstrap', 'zone.js'],
+        allowedCommonJsDependencies: ['ajv', 'zone.js'],
       });
 
       const { result, logs } = await harness.executeOnce();
@@ -76,7 +76,7 @@ describeBuilder(buildWebpackBrowser, BROWSER_BUILDER_INFO, (harness) => {
       await harness.appendToFile(
         'src/app/app.component.ts',
         `
-        import 'bootstrap';
+        import 'ajv';
         import 'zone.js';
       `,
       );

yarn.lock

@@ -5466,11 +5466,6 @@ boolbase@^1.0.0:
   resolved "https://registry.yarnpkg.com/boolbase/-/boolbase-1.0.0.tgz#68dff5fbe60c51eb37725ea9e3ed310dcc1e776e"
   integrity sha512-JZOSA7Mo9sNGB8+UjSgzdLtokWAky1zbztM3WRLCbZ70/3cTANmQmOdR7y2g+J0e2WXywy1yS468tY+IruqEww==
 
-bootstrap@^4.0.0:
-  version "4.6.2"
-  resolved "https://registry.yarnpkg.com/bootstrap/-/bootstrap-4.6.2.tgz#8e0cd61611728a5bf65a3a2b8d6ff6c77d5d7479"
-  integrity sha512-51Bbp/Uxr9aTuy6ca/8FbFloBUJZLHwnhTcnjIeRn2suQWsWzcuJhGjKDB5eppVte/8oCdOL3VuwxvZDUggwGQ==
-
 brace-expansion@^1.1.7:
   version "1.1.11"
   resolved "https://registry.yarnpkg.com/brace-expansion/-/brace-expansion-1.1.11.tgz#3c7fcbf529d87226f3d2f52b966ff5271eb441dd"
@@ -9042,11 +9037,6 @@ jju@~1.4.0:
   resolved "https://registry.yarnpkg.com/jju/-/jju-1.4.0.tgz#a3abe2718af241a2b2904f84a625970f389ae32a"
   integrity sha512-8wb9Yw966OSxApiCt0K3yNJL8pnNeIv+OEq2YMidz4FKP6nonSRoOXc80iXY4JaN2FC11B9qsNmDsm+ZOfMROA==
 
-jquery@^3.3.1:
-  version "3.7.1"
-  resolved "https://registry.yarnpkg.com/jquery/-/jquery-3.7.1.tgz#083ef98927c9a6a74d05a6af02806566d16274de"
-  integrity sha512-m4avr8yL8kmFN8psrbFFFmB/If14iN5o9nw/NgnnM+kybDJpRsAynV2BsfpTYrTRysYUdADVD7CkUUizgkpLfg==
-
 js-base64@^3.7.7:
   version "3.7.7"
   resolved "https://registry.yarnpkg.com/js-base64/-/js-base64-3.7.7.tgz#e51b84bf78fbf5702b9541e2cb7bfcb893b43e79"
@@ -11344,11 +11334,6 @@ pluralize@^8.0.0:
   resolved "https://registry.yarnpkg.com/pluralize/-/pluralize-8.0.0.tgz#1a6fa16a38d12a1901e0320fa017051c539ce3b1"
   integrity sha512-Nc3IT5yHzflTfbjgqWcCPpo7DaKy4FnpB0l/zCAW0Tc7jxAiuqSxHasntB3D7887LSrA93kDJ9IXovxJYxyLCA==
 
-popper.js@^1.14.1:
-  version "1.16.1"
-  resolved "https://registry.yarnpkg.com/popper.js/-/popper.js-1.16.1.tgz#2a223cb3dc7b6213d740e40372be40de43e65b1b"
-  integrity sha512-Wb4p1J4zyFTbM+u6WuO4XstYx4Ky9Cewe4DWrel7B0w6VVICvPwdOpotjzcf6eD8TsckVnIMNONQyPIUFOUbCQ==
-
 portfinder@^1.0.32:
   version "1.0.32"
   resolved "https://registry.yarnpkg.com/portfinder/-/portfinder-1.0.32.tgz#2fe1b9e58389712429dc2bea5beb2146146c7f81"