Skip to content

Conversation

@sanjacornelius
Copy link
Contributor

@sanjacornelius sanjacornelius commented Oct 10, 2025

ci:package-smart-extract:revert-32-observation/FOUR-28662
ci:deploy

..


Note

Introduces Smart Extract HITL flow for manual document review and tightens error/UX handling around tasks and scripts.

  • Tasks: Detect Manual Document Review and, when smart-extract.hitl_enabled, render a Smart Extract iframe (tasks/partials/hitl-iframe.blade.php) instead of the standard <task> component; pass hitlEnabled and computed iframeSrc from TaskController@edit
  • Task listing/API: Include manual review tasks in non-system results when HITL is enabled; in Task resource, backfill process_request.case_number from parent when missing; frontend task list tolerates empty case numbers
  • Error handling: Jobs/ErrorHandling now extracts concise messages from microservice output (prefers output.error/stderr, trims to first line), still throws ScriptTimeoutException on timeouts; adds unit tests
  • Modeler UI: Default assignmentLock/allowReassignment to false and add safe JSON parsing utility in TaskAssignment.vue
  • Infra/views: Add @stack('scripts') to main layout; register broadcast channels; minor middleware docblock type updates
  • Dependencies: Add package-smart-extract and adjust package-signature version; new/updated tests for tasks and process assignment logic

Written by Cursor Bugbot for commit 4e016a8. This will update automatically on new commits. Configure here.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

2 similar comments
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

CarliPinell and others added 2 commits October 31, 2025 09:24
Task/FOUR-27119: Modify Task Edit to render iframe with smart extract URL after Operator clicks on HITL Manual Task
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

CarliPinell and others added 3 commits November 4, 2025 10:41
Observation/FOUR-27719: Iframe displays “No iframe source provided for Manual Edit” on non–Smart Extract tasks
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

mcraeteisha and others added 2 commits November 25, 2025 17:20
FOUR-27111 | Implement Backend Logic to Call API → Submit Document for Extraction
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

CarliPinell and others added 2 commits December 2, 2025 12:44
Task/FOUR-28207: Build iframeSrc in TaskController with fileId and documentToken received from Apply Model response
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

1 similar comment
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

1 similar comment
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

2 similar comments
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

Fix BPMN schema violation caused by Smart Extract HITL assignment properties
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

Remove console.logs; Fix unhandled JSON.parse errors that may crash the component.
Check that the dashboardUrl is not empty before building the URL
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

1 similar comment
@vladyrichter
Copy link

QA server K8S was successfully deployed https://ci-d6cfac9856.engk8s.processmaker.net

@processmaker-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed 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.

6 participants