Add the "Along Normals" parameter to the 'Jitter Points' node

[Keavon]

No description provided.

[gemini-code-assist]

Code Review

This pull request introduces an 'Along Normals' parameter to the 'Jitter Points' node, allowing points to be offset perpendicular to the path direction. The implementation includes new tangent calculation methods in SegmentDomain and a document migration for existing nodes. Review feedback suggests replacing the magic number 1e3 used for epsilon scaling with a named constant to improve maintainability.

[cubic-dev-ai]

1 issue found across 3 files

Confidence score: 3/5

• There is a concrete behavior bug: self-loop segments are counted once, so point_tangent returns an incorrect single-connection tangent for looped segments in node-graph/libraries/vector-types/src/vector/vector_attributes.rs.
• This is a medium-severity, user-visible geometry correctness issue, so it carries some risk despite being localized.
• Pay close attention to node-graph/libraries/vector-types/src/vector/vector_attributes.rs - verify handling of self-loop segment tangents.

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/vector-types/src/vector/vector_attributes.rs">

<violation number="1" location="node-graph/libraries/vector-types/src/vector/vector_attributes.rs:475">
P2: Self-loop segments (start == end) are only counted once here, so point_tangent returns a single-connection tangent instead of treating the segment as two connections. That yields the wrong tangent for looped segments.</violation>
</file>

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