feat: add options to ignore BPMN label bounds for activities

[Copilot]

• Analyze the issue requirements and current codebase
• Understand how StyleComputer works and handles label bounds
• Identify where RendererOptions are defined and used
• Add new options to RendererOptions interface for ignoring label bounds
• Modify StyleComputer to support ignoring label bounds for activities
• Create focused unit tests for the new functionality
• Run tests to validate the implementation
• Review and test visual outputs to ensure correct behavior
• Add @SInCE 0.48.0 tags to new options
• Add query parameters support in demo/tests
• Extract renderer options to dedicated function
• Remove unnecessary log constant indirection
• Fix lint errors (trailing whitespace)
• Clarify option precedence in documentation

✅ ALL REQUIREMENTS COMPLETE

Final Implementation Summary:

1. New Options in RendererOptions:

   • ignoreBpmnActivityLabelBounds?: boolean - Ignores all activity label bounds
   • ignoreBpmnTaskLabelBounds?: boolean - Ignores task label bounds only
   • Both options include proper JSDoc with @since 0.48.0 tags
   • Documentation clarifies that ignoreBpmnTaskLabelBounds is ignored when ignoreBpmnActivityLabelBounds is true

2. StyleComputer Updates:

   • Added helper function shouldIgnoreBpmnLabelBounds() to check element types
   • Modified label style computation to conditionally skip bounds
   • Preserves special handling for expanded activities

3. Demo/Test Support:

   • Added query parameters: renderer.ignore.activity.label.bounds and renderer.ignore.task.label.bounds
   • Extracted renderer options logic to getRendererOptionsFromParameters() function
   • Follows same pattern as getFitOptionsFromParameters()
   • Uses logStartup directly without unnecessary indirection

4. Testing:

   • 16 comprehensive unit tests covering all scenarios
   • All existing tests pass (3,061 total)
   • Integration tests verified (308 tests)
   • Lint checks pass

Query Parameter Usage:

?renderer.ignore.activity.label.bounds=true
?renderer.ignore.task.label.bounds=true

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[CLAassistant]

All committers have signed the CLA.

[github-actions]

♻️ PR Preview d153e3a has been successfully destroyed since this PR has been closed.

_{🤖 By surge-preview}

[github-actions]

♻️ PR Preview d153e3a has been successfully destroyed since this PR has been closed.

_{🤖 By surge-preview}

[tbouffard]

@copilot new assignment
In the demo/tests, consider the 2 query parameters to ignore task and activity label bounds.
This is managed in dev/ts/shared/main.ts and the same place of ignoreBpmnColors for instance.
Extract the code managing the renderer options in a dedicated function as this is done for example in getFitOptionsFromParameters

[Copilot]

@copilot new assignment
In the demo/tests, consider the 2 query parameters to ignore task and activity label bounds.
This is managed in dev/ts/shared/main.ts and the same place of ignoreBpmnColors f...

Done in commit 1b6c8b7. Added query parameters renderer.ignore.activity.label.bounds and renderer.ignore.task.label.bounds following the same pattern as ignoreBpmnColors and ignoreBpmnLabelStyles. Also extracted the renderer options management to a dedicated getRendererOptionsFromParameters() function as suggested.

[tbouffard]

@copilot the build is failing, fix the lint errors

[Copilot]

@copilot the build is failing, fix the lint errors

Fixed in commit 892b38a. Removed trailing whitespace on lines 242 and 260 in dev/ts/shared/main.ts. Lint check now passes.

[tbouffard]

LGTM

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tbouffard]

This pull request introduces new renderer options that allow users to control whether BPMN label bounds are ignored for activities and tasks, improving customization of label rendering. The changes refactor renderer option handling, extend the RendererOptions type, update the style computation logic, and add comprehensive unit tests for the new behavior.

Renderer options and configuration

• Added two new options to RendererOptions: ignoreBpmnActivityLabelBounds and ignoreBpmnTaskLabelBounds, with clear documentation and precedence rules. These options allow users to ignore label bounds for all activities or just tasks, respectively. (src/component/options.ts [1] [2]
• Refactored renderer options initialization and query parameter mapping in main.ts, consolidating logic into a new getRendererOptionsFromParameters function and expanding support for the new options. (dev/ts/shared/main.ts [1] [2]

Style computation logic

• Updated StyleComputer to support the new renderer options, passing them to label style computation and ensuring correct option precedence. (src/component/mxgraph/renderer/StyleComputer.ts [1] [2]
• Modified computeLabelStyleValues and added shouldIgnoreBpmnLabelBounds to determine when label bounds should be ignored based on the element type and options. (src/component/mxgraph/renderer/StyleComputer.ts [1] [2]

Testing

• Added comprehensive unit tests to verify the behavior of the new options, including edge cases and precedence, ensuring that label bounds are correctly ignored or applied for activities, tasks, and other BPMN elements. (test/unit/component/mxgraph/renderer/StyleComputer.test.ts test/unit/component/mxgraph/renderer/StyleComputer.test.tsR600-R731)