Segment IDs are wrong when no Path node exists, causing Path tool to fail before Path node is added

[0HyperCube]

Graphite/editor/src/messages/portfolio/document/utility_types/network_interface.rs

Lines 3047 to 3068 in b4fc3d0

	pub fn compute_modified_vector(&self, layer: LayerNodeIdentifier) -> Option<Vector> {
	let graph_layer = graph_modification_utils::NodeGraphLayer::new(layer, self);
	if let Some(path_node) = graph_layer.upstream_visible_node_id_from_name_in_layer("Path")
	&& let Some(vector) = self.document_metadata.vector_modify.get(&path_node)
	{
	let mut modified = vector.clone();
	let path_node = self.document_network().nodes.get(&path_node);
	let modification_input = path_node.and_then(|node: &DocumentNode| node.inputs.get(1)).and_then(|input| input.as_value());
	if let Some(TaggedValue::VectorModification(modification)) = modification_input {
	modification.apply(&mut modified);
	}
	return Some(modified);
	}
	self.document_metadata
	.click_targets
	.get(&layer)
	.map(|click| click.iter().map(ClickTarget::target_type))
	.map(|target_types| Vector::from_target_types(target_types, true))
	}

It tries to get a path node and then falls back on the document_metadata.click_targets. SegmentIds are not stored in the click targets. Therefore this code will just incorrectly assume incremental SegmentIds (the actual generation of SegmentIds occurs inside Vector::from_target_types in vector.append_subpath).

This means that trying to delete a segment or modify a handle when the SegmentIds are not contiguous (e.g. after a boolean operation) will result in confusing results (targeting the wrong segment or no segment at all). From this Discord report.

See reproduction steps: #3481 (comment)

[ayush80942]

A clean fix would be to ensure a Path node is always created before any geometry editing occurs. If no Path node exists, create it eagerly from the layer’s true vector geometry, and always derive the modified vector from the Path node thereafter. This preserves stable SegmentIds and avoids reconstructing geometry from UI-only data.

[Ayush2k02]

hey, @0HyperCube . Are there any steps to replicating this ?

[Ayush2k02]

Screen.Recording.2026-02-06.at.12.48.03.AM.mov

Seems to be working ideally so far

[0HyperCube]

I'd suggest you try the original example that was linked in the post.

• Draw circle
• Draw rectangle such that it covers the middle two control points.
• Do the boolean subtract back
• Ensure no path node is added.
• Try and delete a segment / move a handle on the left hand side.

segment_ids_incorrect.mp4

[Ayush2k02]

Hey, thanks for the repro steps