@pmPaulis @henryjonathanquispe Here are my initial notes and questions:

• loader*.js files have a lot of duplication. You could extract to a common folder and wrap the differences in if statements and call it like this

import { initializeCommonLoader } from "../next/commonLoader";
import sharedComponents from "../next/libraries/sharedComponents";
import vueFormElements from "../next/libraries/vueFormElements";

initializeCommonLoader({
 includeScreenBuilder: true,
 includeModeler: false,
 includeMonaco: false,
});

• is accesibility.js being used anywhere?
• If we make any changes to the logic in bootstrap.js and app-layout.js, do we need to duplicate that logic in the next folder until the entire app is converted to use the new code?
• What’s the purpose of the badPractices.js file?
• EventBus, apilient, etc. should be in their own files that are imported as a modules instead of being passed in with globalInput in next/config/*.js files. Can we get rid of globalInput attribute everywhere? If you really need a global import and use the globalVariables.js helper.

@pmPaulis
I've created a PR: #7869
for the changes.

@nolanpro
Answered your questions:

Is accessibility.js being used anywhere?
Yes, it is used.

If we make any changes to the logic in bootstrap.js and app-layout.js, do we need to duplicate that logic in the next folder until the entire app is converted to use the new code?
Yes, duplication is necessary for now.

What’s the purpose of the badPractices.js file?
This file no longer exists.

Regarding EventBus, apiClient, etc.: These should now reside in their own files and be imported as modules instead of being passed in with globalInput in next/config/*.js files. Can we get rid of the globalInput attribute everywhere? If a global import is absolutely needed, use the globalVariables.js helper.
I've implemented these changes. The globalInput has been removed.

PR's Packages
https://github.com/ProcessMaker/package-webentry/pull/269
https://github.com/ProcessMaker/connector-docusign/pull/55

@henryjonathanquispe
the PR has conflicts with the following PR
https://github.com/ProcessMaker/processmaker/pull/7873/files

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@ryancooley @boliviacoca
This PR has the following Failing in the unit test, this is not related with the changes but we will review in a specific ticket.

Filter (Tests\Filter)
 ✘ Compare data integer
   │
   │ Failed asserting that actual size 2 matches expected size 1.
   │
   │ /opt/processmaker/tests/unit/ProcessMaker/FilterTest.php:88
   │ /opt/processmaker/vendor/laravel/framework/src/Illuminate/Foundation/Testing/TestCase.php:176
   │