Fix Vello renderer cropping open paths with Inside-aligned strokes and an opacity or blend mode set

[Keavon]

This bug occurs in Vello when a non-default alpha blending setting is used like an opacity <100%. The render bounds given to Vello were incorrect when the path is open and the stroke align (which isn't valid for open paths) is set to Inside.

Vello (before):

SVG (unchanged) and Vello (after):

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

[gemini-code-assist]

Code Review

This pull request updates the max_aabb_inflation method to include a path_is_closed parameter, ensuring that stroke alignment is correctly handled for open versus closed paths during bounding box calculations. The feedback identifies a performance optimization opportunity in renderer.rs where path traversal and stroke cloning can be minimized by caching the stroke and short-circuiting the closed-path check when the stroke is centered or non-renderable.

[cubic-dev-ai]

1 issue found across 1 file (changes from recent commits).

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="node-graph/libraries/rendering/src/renderer.rs">

<violation number="1" location="node-graph/libraries/rendering/src/renderer.rs:253">
P1: Clip/layer inflation uses column-length min/max, which is not conservative under shear and can still crop strokes.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}