Split common vendor chunk

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

Makefile

@@ -28,39 +28,7 @@ lint-fix-watch:
 
 # Cleaning
 clean:
-	rm -rf apps/accessibility/js/
-	rm -rf apps/comments/js/
-	rm -rf apps/dashboard/js/
-	rm -rf apps/dav/js/
-	rm -rf apps/files/js/dist/
-	rm -rf apps/files_sharing/js/dist/
-	rm -rf apps/files_trashbin/js/
-	rm -rf apps/files_versions/js/
-	rm -rf apps/oauth2/js/
-	rm -rf apps/settings/js/vue-*
-	rm -rf apps/systemtags/js/systemtags.*
-	rm -rf apps/twofactor_backupcodes/js
-	rm -rf apps/updatenotification/js/updatenotification.*
-	rm -rf apps/user_status/js/
-	rm -rf apps/weather_status/js/
-	rm -rf apps/workflowengine/js/
-	rm -rf core/js/dist
+	rm -rf dist
 
 clean-git: clean
-	git checkout -- apps/accessibility/js/
-	git checkout -- apps/comments/js/
-	git checkout -- apps/dashboard/js/
-	git checkout -- apps/dav/js/
-	git checkout -- apps/files/js/dist/
-	git checkout -- apps/files_sharing/js/dist/
-	git checkout -- apps/files_trashbin/js/
-	git checkout -- apps/files_versions/js/
-	git checkout -- apps/oauth2/js/
-	git checkout -- apps/settings/js/vue-*
-	git checkout -- apps/systemtags/js/systemtags.*
-	git checkout -- apps/twofactor_backupcodes/js
-	git checkout -- apps/updatenotification/js/updatenotification.*
-	git checkout -- apps/user_status/js/
-	git checkout -- apps/weather_status/js/
-	git checkout -- apps/workflowengine/js/
-	git checkout -- core/js/dist
+	git checkout -- dist

apps/comments/lib/Listener/LoadSidebarScripts.php

@@ -52,6 +52,6 @@ class LoadSidebarScripts implements IEventListener {
 		// TODO: make sure to only include the sidebar script when
 		// we properly split it between files list and sidebar
 		Util::addScript(Application::APP_ID, 'comments');
-		Util::addScript(Application::APP_ID, 'comments-tab');
+		Util::addScript(Application::APP_ID, 'comments-tab', 'files');
 	}
 }

apps/files/lib/Controller/ViewController.php

@@ -196,10 +196,8 @@ class ViewController extends Controller {
 
 		// Load the files we need
 		\OCP\Util::addStyle('files', 'merged');
-		\OCP\Util::addScript('files', 'merged-index');
-		\OCP\Util::addScript('files', 'templates');
-		\OCP\Util::addScript('files', 'files-app-settings');
-		\OCP\Util::addScript('files', 'legacy-filelist-search');
+		\OCP\Util::addScript('files', 'merged-index', 'files');
+		\OCP\Util::addScript('files', 'main');
 
 		// mostly for the home storage's free space
 		// FIXME: Make non static

apps/files/src/main.js

@@ -0,0 +1,3 @@
+import './files-app-settings'
+import './templates'
+import './legacy/filelistSearch'

build/files-checker.php

@@ -20,8 +20,8 @@
  */
 
 $expectedFiles = [
-	'.',
 	'..',
+	'.',
 	'.codecov.yml',
 	'.devcontainer',
 	'.drone.yml',
@@ -63,6 +63,7 @@ $expectedFiles = [
 	'COPYING',
 	'core',
 	'cron.php',
+	'dist',
 	'index.html',
 	'index.php',
 	'lib',
@@ -73,8 +74,8 @@ $expectedFiles = [
 	'ocs',
 	'package-lock.json',
 	'package.json',
-	'psalm.xml',
 	'psalm-ocp.xml',
+	'psalm.xml',
 	'public.php',
 	'README.md',
 	'remote.php',
@@ -88,6 +89,7 @@ $expectedFiles = [
 	'version.php',
 	'webpack.common.js',
 	'webpack.dev.js',
+	'webpack.modules.js',
 	'webpack.prod.js',
 ];
 $actualFiles = [];