Skip to content

Four 32412 - #8946

Open
henryjonathanquispe wants to merge 73 commits into
VITE_INBOXfrom
FOUR-32412
Open

Four 32412#8946
henryjonathanquispe wants to merge 73 commits into
VITE_INBOXfrom
FOUR-32412

Conversation

@henryjonathanquispe

@henryjonathanquispe henryjonathanquispe commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

Describe the issue this ticket solves and describe how to reproduce the issue (please attach any fixtures used to reproduce the issue).

Solution

  • List the changes you've introduced to solve the issue.

How to Test

Describe how to test that this solution works.

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:deploy
ci:package-data-sources:VITE_INBOX
ci:package-variable-finder:VITE_INBOX

@henryjonathanquispe
henryjonathanquispe changed the base branch from VITE_INBOX to develop July 28, 2026 21:20
@gitguardian

gitguardian Bot commented Jul 28, 2026

Copy link
Copy Markdown

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@henryjonathanquispe
henryjonathanquispe changed the base branch from develop to VITE_INBOX July 28, 2026 21:20
@henryjonathanquispe
henryjonathanquispe force-pushed the VITE_INBOX branch 3 times, most recently from 29fc119 to cac46ac Compare July 29, 2026 18:16
@processmaker-sonarqube

Copy link
Copy Markdown

Quality Gate passed 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

@cursor

cursor Bot commented Sep 9, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Wide-ranging frontend build and script-order changes across many authenticated routes; regressions are likely in modeler, screen builder, and package addon timing rather than server-side security.

Overview
This PR continues the Laravel Mix → Vite migration across a large slice of the app: admin (users, groups, settings, logs, DevLink, Customize UI, etc.), designer/processes flows, cases, requests, inbox rules, processes catalogue, modeler/screen builder, and several auth layouts.

Blade / bootstrapping: Pages switch to layouts.layoutnextvite (or new mobilenextvite / previewnextvite) and load @vite entries instead of mix('js/...'). Boot data moves to window.temporal (and sometimes window.config / globals) so ESM bundles can read what inline Blade scripts used to define. Shared loader modules (loaderMinimal.js, loaderMonaco.js, feature-specific loaders) call setupMain() and copy package lists onto ProcessMaker.packages. layoutnextvite now seeds temporal.packages and defers addon Mix scripts; Vue roots often mount on window load so package scripts after @yield('js') run first.

JS refactors: Inline Vue instances move into entry files; SharedComponents barrel imports become explicit relative .vue paths; assets use public URLs instead of require(); TinyMCE/CSS override gets Vite-friendly imports; jQuery is exposed as a callable on window; i18n sets window.__; setupMain dispatches app-bootstrapped. Modeler registers components via a shallow-copy helper for non-extensible ESM namespaces; catalogue uses ProcessMaker.EventBus instead of a local EventBus.

Backend (small): Screen builder and request preview views receive an uppercased type for the screen (FORM default).

Docs: resources/js/vite/AGENTS.md is expanded with migration checklist, route table, and loader patterns.

Reviewed by Cursor Bugbot for commit 14e572f. Bugbot is set up for automated code reviews on this repo. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8f18ccd. Configure here.

Comment thread resources/views/layouts/previewnextvite.blade.php
Comment thread resources/jscomposition/cases/casesDetail/loaderCasesDetail.js
Comment thread resources/js/processes/screens/edit.js
@decisions-sonarqube

Copy link
Copy Markdown

Quality Gate passed 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant