Add line numbers to file paths in the website editor structure docs (#3130)

MohdMohsin97 · Keavon · web-flow · commit b697cc8131a4 · 2026-02-15T23:59:27.000Z
* added the line number to existing path

* Update JS parser to use line numbers

---------

Co-authored-by: Keavon Chambers &lt;keavon@keavon.com&gt;
diff --git a/editor/src/utility_traits.rs b/editor/src/utility_traits.rs
@@ -49,11 +49,11 @@ pub trait HierarchicalTree {
 	fn build_message_tree() -> DebugMessageTree;
 
 	fn message_handler_data_str() -> MessageData {
-		MessageData::new(String::new(), Vec::new(), "")
+		MessageData::new(String::new(), Vec::new(), "", 0)
 	}
 
 	fn message_handler_str() -> MessageData {
-		MessageData::new(String::new(), Vec::new(), "")
+		MessageData::new(String::new(), Vec::new(), "", 0)
 	}
 
 	fn path() -> &'static str {
@@ -64,5 +64,6 @@ pub trait HierarchicalTree {
 pub trait ExtractField {
 	fn field_types() -> Vec<(String, usize)>;
 	fn path() -> &'static str;
+	fn line_number() -> usize;
 	fn print_field_types();
 }
diff --git a/editor/src/utility_types.rs b/editor/src/utility_types.rs
@@ -3,11 +3,12 @@ pub struct MessageData {
 	name: String,
 	fields: Vec<(String, usize)>,
 	path: &'static str,
+	line_number: usize,
 }
 
 impl MessageData {
-	pub fn new(name: String, fields: Vec<(String, usize)>, path: &'static str) -> MessageData {
-		MessageData { name, fields, path }
+	pub fn new(name: String, fields: Vec<(String, usize)>, path: &'static str, line_number: usize) -> MessageData {
+		MessageData { name, fields, path, line_number }
 	}
 
 	pub fn name(&self) -> &str {
@@ -21,6 +22,10 @@ impl MessageData {
 	pub fn path(&self) -> &'static str {
 		self.path
 	}
+
+	pub fn line_number(&self) -> usize {
+		self.line_number
+	}
 }
 
 #[derive(Debug)]
@@ -31,6 +36,7 @@ pub struct DebugMessageTree {
 	message_handler: Option<MessageData>,
 	message_handler_data: Option<MessageData>,
 	path: &'static str,
+	line_number: usize,
 }
 
 impl DebugMessageTree {
@@ -42,6 +48,7 @@ impl DebugMessageTree {
 			message_handler: None,
 			message_handler_data: None,
 			path: "",
+			line_number: 0,
 		}
 	}
 
@@ -53,6 +60,10 @@ impl DebugMessageTree {
 		self.path = path;
 	}
 
+	pub fn set_line_number(&mut self, line_number: usize) {
+		self.line_number = line_number
+	}
+
 	pub fn add_variant(&mut self, variant: DebugMessageTree) {
 		if let Some(variants) = &mut self.variants {
 			variants.push(variant);
@@ -81,6 +92,10 @@ impl DebugMessageTree {
 		self.path
 	}
 
+	pub fn line_number(&self) -> usize {
+		self.line_number
+	}
+
 	pub fn variants(&self) -> Option<&Vec<DebugMessageTree>> {
 		self.variants.as_ref()
 	}
diff --git a/proc-macros/src/extract_fields.rs b/proc-macros/src/extract_fields.rs
@@ -9,6 +9,8 @@ pub fn derive_extract_field_impl(input: TokenStream) -> syn::Result<TokenStream>
 	let generics = &input.generics;
 	let (impl_generics, ty_generics, where_clause) = generics.split_for_impl();
 
+	let line_number = struct_name.span().start().line;
+
 	let fields = match &input.data {
 		Data::Struct(data) => match &data.fields {
 			Fields::Named(fields) => &fields.named,
@@ -50,6 +52,11 @@ pub fn derive_extract_field_impl(input: TokenStream) -> syn::Result<TokenStream>
 			fn path() -> &'static str {
 				file!()
 			}
+
+			fn line_number() -> usize {
+				#line_number
+			}
+
 		}
 	};
 
diff --git a/proc-macros/src/hierarchical_tree.rs b/proc-macros/src/hierarchical_tree.rs
@@ -7,6 +7,8 @@ pub fn generate_hierarchical_tree(input: TokenStream) -> syn::Result<TokenStream
 	let input = parse2::<DeriveInput>(input)?;
 	let input_type = &input.ident;
 
+	let line_number = input_type.span().start().line;
+
 	let data = match &input.data {
 		Data::Enum(data) => data,
 		_ => return Err(syn::Error::new(Span::call_site(), "Tried to derive HierarchicalTree for non-enum")),
@@ -101,6 +103,8 @@ pub fn generate_hierarchical_tree(input: TokenStream) -> syn::Result<TokenStream
 
 				message_tree.set_path(file!());
 
+				message_tree.set_line_number(#line_number);
+
 				message_tree
 			}
 		}
diff --git a/proc-macros/src/message_handler_data_attr.rs b/proc-macros/src/message_handler_data_attr.rs
@@ -16,6 +16,8 @@ pub fn message_handler_data_attr_impl(attr: TokenStream, input_item: TokenStream
 
 	let input_type = path.segments.last().map(|s| &s.ident).unwrap();
 
+	let handler_line_number = input_type.span().start().line;
+
 	// Extract the message type from the trait path
 	let trait_path = match &impl_block.trait_ {
 		Some((_, path, _)) => path,
@@ -40,14 +42,16 @@ pub fn message_handler_data_attr_impl(attr: TokenStream, input_item: TokenStream
 								// Get just the base identifier (ToolMessageData) without generics
 								let type_name = &type_path.path.segments.first().unwrap().ident;
 
+								let handler_data_line_number = type_name.span().start().line;
+
 								quote! {
 									#input_item
 									impl #message_type {
 										pub fn message_handler_data_str() -> MessageData {
-											MessageData::new(format!("{}", stringify!(#type_name)), #type_name::field_types(), #type_name::path())
+											MessageData::new(format!("{}", stringify!(#type_name)), #type_name::field_types(), #type_name::path(), #type_name::line_number())
 										}
 										pub fn message_handler_str() -> MessageData {
-											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path())
+											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path(), #input_type::line_number())
 
 										}
 									}
@@ -57,7 +61,7 @@ pub fn message_handler_data_attr_impl(attr: TokenStream, input_item: TokenStream
 								#input_item
 								impl #message_type {
 										pub fn message_handler_str() -> MessageData {
-											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path())
+											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path(), #input_type::line_number())
 										}
 									}
 							},
@@ -67,16 +71,19 @@ pub fn message_handler_data_attr_impl(attr: TokenStream, input_item: TokenStream
 									syn::Type::Path(type_path) => &type_path.path.segments.first().unwrap().ident,
 									_ => return Err(syn::Error::new(type_reference.elem.span(), "Expected type path")),
 								};
+
+								let type_line_number = type_ident.span().start().line;
+
 								let tr = clean_rust_type_syntax(type_reference.to_token_stream().to_string());
 								quote! {
 									#input_item
 									impl #message_type {
 										pub fn message_handler_data_str() -> MessageData {
-											MessageData::new(format!("{}", #tr), #type_ident::field_types(), #type_ident::path())
+											MessageData::new(format!("{}", #tr), #type_ident::field_types(), #type_ident::path(), #type_ident::line_number())
 										}
 
 										pub fn message_handler_str() -> MessageData {
-											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path())
+											MessageData::new(format!("{}", stringify!(#input_type)), #input_type::field_types(), #input_type::path(), #input_type::line_number())
 
 										}
 									}
diff --git a/tools/editor-message-tree/src/main.rs b/tools/editor-message-tree/src/main.rs
@@ -6,7 +6,7 @@ fn main() {
 	let result = Message::message_tree();
 	std::fs::create_dir_all("../../website/generated").unwrap();
 	let mut file = std::fs::File::create("../../website/generated/hierarchical_message_system_tree.txt").unwrap();
-	file.write_all(format!("{} `{}`\n", result.name(), result.path()).as_bytes()).unwrap();
+	file.write_all(format!("{} `{}#L{}`\n", result.name(), result.path(), result.line_number()).as_bytes()).unwrap();
 	if let Some(variants) = result.variants() {
 		for (i, variant) in variants.iter().enumerate() {
 			let is_last = i == variants.len() - 1;
@@ -28,7 +28,8 @@ fn print_tree_node(tree: &DebugMessageTree, prefix: &str, is_last: bool, file: &
 	if tree.path().is_empty() {
 		file.write_all(format!("{}{}{}\n", prefix, branch, tree.name()).as_bytes()).unwrap();
 	} else {
-		file.write_all(format!("{}{}{} `{}`\n", prefix, branch, tree.name(), tree.path()).as_bytes()).unwrap();
+		file.write_all(format!("{}{}{} `{}#L{}`\n", prefix, branch, tree.name(), tree.path(), tree.line_number()).as_bytes())
+			.unwrap();
 	}
 
 	// Print children if any
@@ -64,7 +65,8 @@ fn print_tree_node(tree: &DebugMessageTree, prefix: &str, is_last: bool, file: &
 		if data.name().is_empty() && tree.name() != FRONTEND_MESSAGE_STR {
 			panic!("{}'s MessageHandler is missing #[message_handler_data]", tree.name());
 		} else if tree.name() != FRONTEND_MESSAGE_STR {
-			file.write_all(format!("{}{}{} `{}`\n", prefix, branch, data.name(), data.path()).as_bytes()).unwrap();
+			file.write_all(format!("{}{}{} `{}#L{}`\n", prefix, branch, data.name(), data.path(), data.line_number()).as_bytes())
+				.unwrap();
 
 			for (i, field) in data.fields().iter().enumerate() {
 				let is_last_field = i == len - 1;
@@ -81,7 +83,8 @@ fn print_tree_node(tree: &DebugMessageTree, prefix: &str, is_last: bool, file: &
 		if data.path().is_empty() {
 			file.write_all(format!("{}{}{}\n", prefix, "└── ", data.name()).as_bytes()).unwrap();
 		} else {
-			file.write_all(format!("{}{}{} `{}`\n", prefix, "└── ", data.name(), data.path()).as_bytes()).unwrap();
+			file.write_all(format!("{}{}{} `{}#L{}`\n", prefix, "└── ", data.name(), data.path(), data.line_number()).as_bytes())
+				.unwrap();
 		}
 		for (i, field) in data.fields().iter().enumerate() {
 			let is_last_field = i == len - 1;
diff --git a/website/.build-scripts/generate-editor-structure.ts b/website/.build-scripts/generate-editor-structure.ts
@@ -50,7 +50,7 @@ function buildHtmlList(nodes: Entry[], currentIndex: number, currentLevel: numbe
 		const hasDirectChildren = i + 1 < nodes.length && nodes[i + 1].level > node.level;
 		const hasDeeperChildren = hasDirectChildren && i + 2 < nodes.length && nodes[i + 2].level > nodes[i + 1].level;
 
-		const linkHtml = node.link ? `<a href="${node.link}" target="_blank">${path.basename(node.link)}</a>` : "";
+		const linkHtml = node.link ? `<a href="${node.link}" target="_blank">${path.basename(node.link.split("#L").join(":"))}</a>` : "";
 		const fieldPieces = node.text.match(/([^:]*):(.*)/);
 		let escapedText;
 		if (fieldPieces && fieldPieces.length === 3) {

Original file line number	Diff line number	Diff line change
`@@ -49,11 +49,11 @@ pub trait HierarchicalTree {`
`49`	`49`	`fn build_message_tree() -> DebugMessageTree;`
`50`	`50`
`51`	`51`	`fn message_handler_data_str() -> MessageData {`
`52`		`- MessageData::new(String::new(), Vec::new(), "")`
	`52`	`+ MessageData::new(String::new(), Vec::new(), "", 0)`
`53`	`53`	`}`
`54`	`54`
`55`	`55`	`fn message_handler_str() -> MessageData {`
`56`		`- MessageData::new(String::new(), Vec::new(), "")`
	`56`	`+ MessageData::new(String::new(), Vec::new(), "", 0)`
`57`	`57`	`}`
`58`	`58`
`59`	`59`	`fn path() -> &'static str {`
`@@ -64,5 +64,6 @@ pub trait HierarchicalTree {`
`64`	`64`	`pub trait ExtractField {`
`65`	`65`	`fn field_types() -> Vec<(String, usize)>;`
`66`	`66`	`fn path() -> &'static str;`
	`67`	`+ fn line_number() -> usize;`
`67`	`68`	`fn print_field_types();`
`68`	`69`	`}`
Original file line number	Diff line number	Diff line change
`@@ -3,11 +3,12 @@ pub struct MessageData {`
`3`	`3`	`name: String,`
`4`	`4`	`fields: Vec<(String, usize)>,`
`5`	`5`	`path: &'static str,`
	`6`	`+ line_number: usize,`
`6`	`7`	`}`
`7`	`8`
`8`	`9`	`impl MessageData {`
`9`		`- pub fn new(name: String, fields: Vec<(String, usize)>, path: &'static str) -> MessageData {`
`10`		`- MessageData { name, fields, path }`
	`10`	`+ pub fn new(name: String, fields: Vec<(String, usize)>, path: &'static str, line_number: usize) -> MessageData {`
	`11`	`+ MessageData { name, fields, path, line_number }`
`11`	`12`	`}`
`12`	`13`
`13`	`14`	`pub fn name(&self) -> &str {`
`@@ -21,6 +22,10 @@ impl MessageData {`
`21`	`22`	`pub fn path(&self) -> &'static str {`
`22`	`23`	`self.path`
`23`	`24`	`}`
	`25`	`+`
	`26`	`+ pub fn line_number(&self) -> usize {`
	`27`	`+ self.line_number`
	`28`	`+ }`
`24`	`29`	`}`
`25`	`30`
`26`	`31`	`#[derive(Debug)]`
`@@ -31,6 +36,7 @@ pub struct DebugMessageTree {`
`31`	`36`	`message_handler: Option<MessageData>,`
`32`	`37`	`message_handler_data: Option<MessageData>,`
`33`	`38`	`path: &'static str,`
	`39`	`+ line_number: usize,`
`34`	`40`	`}`
`35`	`41`
`36`	`42`	`impl DebugMessageTree {`
`@@ -42,6 +48,7 @@ impl DebugMessageTree {`
`42`	`48`	`message_handler: None,`
`43`	`49`	`message_handler_data: None,`
`44`	`50`	`path: "",`
	`51`	`+ line_number: 0,`
`45`	`52`	`}`
`46`	`53`	`}`
`47`	`54`
`@@ -53,6 +60,10 @@ impl DebugMessageTree {`
`53`	`60`	`self.path = path;`
`54`	`61`	`}`
`55`	`62`
	`63`	`+ pub fn set_line_number(&mut self, line_number: usize) {`
	`64`	`+ self.line_number = line_number`
	`65`	`+ }`
	`66`	`+`
`56`	`67`	`pub fn add_variant(&mut self, variant: DebugMessageTree) {`
`57`	`68`	`if let Some(variants) = &mut self.variants {`
`58`	`69`	`variants.push(variant);`
`@@ -81,6 +92,10 @@ impl DebugMessageTree {`
`81`	`92`	`self.path`
`82`	`93`	`}`
`83`	`94`
	`95`	`+ pub fn line_number(&self) -> usize {`
	`96`	`+ self.line_number`
	`97`	`+ }`
	`98`	`+`
`84`	`99`	`pub fn variants(&self) -> Option<&Vec<DebugMessageTree>> {`
`85`	`100`	`self.variants.as_ref()`
`86`	`101`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,6 +7,8 @@ pub fn generate_hierarchical_tree(input: TokenStream) -> syn::Result<TokenStream`
`7`	`7`	`let input = parse2::<DeriveInput>(input)?;`
`8`	`8`	`let input_type = &input.ident;`
`9`	`9`
	`10`	`+ let line_number = input_type.span().start().line;`
	`11`	`+`
`10`	`12`	`let data = match &input.data {`
`11`	`13`	`Data::Enum(data) => data,`
`12`	`14`	`_ => return Err(syn::Error::new(Span::call_site(), "Tried to derive HierarchicalTree for non-enum")),`
`@@ -101,6 +103,8 @@ pub fn generate_hierarchical_tree(input: TokenStream) -> syn::Result<TokenStream`
`101`	`103`
`102`	`104`	`message_tree.set_path(file!());`
`103`	`105`
	`106`	`+ message_tree.set_line_number(#line_number);`
	`107`	`+`
`104`	`108`	`message_tree`
`105`	`109`	`}`
`106`	`110`	`}`