Remove the crate dependency graph website generation intermediate generation step

Author: Keavon

.github/workflows/build.yml

@@ -176,7 +176,7 @@ jobs:
         if: github.event_name == 'push'
         run: |
           mkdir -p website/generated-new
-          cargo run -p crate-hierarchy-viz -- website/generated-new/crate_hierarchy.dot
+          cargo run -p crate-hierarchy-viz -- website/generated-new/crate_hierarchy.svg
           cargo run -p editor-message-tree -- website/generated-new/hierarchical_message_system_tree.txt
 
       - name: 💿 Obtain cache of auto-generated code docs artifacts, to check if they've changed

.github/workflows/website.yml

@@ -64,15 +64,14 @@ jobs:
           rustup update stable
           echo "🦀 Latest updated version of Rust:"
           rustc --version
-          cargo run -p crate-hierarchy-viz -- website/generated/crate_hierarchy.dot
+          cargo run -p crate-hierarchy-viz -- website/generated/crate_hierarchy.svg
           cargo run -p editor-message-tree -- website/generated/hierarchical_message_system_tree.txt
 
-      - name: 🔧 Build auto-generated code docs artifacts into HTML/SVG
+      - name: 🔧 Build auto-generated code docs artifacts into HTML
         run: |
           cd website
           npm ci
           npm run generate-editor-structure
-          npm run generate-crate-hierarchy
 
       - name: 📃 Generate node catalog documentation
         run: cargo run -p node-docs -- website/content/learn/node-catalog

tools/crate-hierarchy-viz/src/main.rs

@@ -2,7 +2,9 @@ use anyhow::{Context, Result};
 use serde::Deserialize;
 use std::collections::{HashMap, HashSet};
 use std::fs;
+use std::io::Write;
 use std::path::PathBuf;
+use std::process::Command;
 
 #[derive(Debug, Deserialize)]
 struct WorkspaceToml {
@@ -173,17 +175,76 @@ fn main() -> Result<()> {
 
 	remove_transitive_dependencies(&mut crates);
 
-	// Generate DOT format and write to output file
+	// Generate DOT format, convert to SVG, and write to output file
 	let dot_content = generate_dot(&crates);
+	let svg_content = dot_to_svg(&dot_content)?;
 
 	if let Some(parent) = output_path.parent() {
 		fs::create_dir_all(parent).with_context(|| format!("Failed to create directory {:?}", parent))?;
 	}
-	fs::write(&output_path, &dot_content).with_context(|| format!("Failed to write to {:?}", output_path))?;
+	fs::write(&output_path, &svg_content).with_context(|| format!("Failed to write to {:?}", output_path))?;
 
 	Ok(())
 }
 
+/// Convert a DOT graph string to SVG by shelling out to @viz-js/viz via Node.js
+fn dot_to_svg(dot: &str) -> Result<String> {
+	let temp_dir = std::env::temp_dir().join("crate-hierarchy-viz");
+	fs::create_dir_all(&temp_dir).with_context(|| "Failed to create temp directory")?;
+
+	// Install @viz-js/viz into the temp directory if not already present
+	let node_modules = temp_dir.join("node_modules").join("@viz-js");
+	if !node_modules.exists() {
+		let npm = if cfg!(target_os = "windows") { "npm.cmd" } else { "npm" };
+		let status = Command::new(npm)
+			.args(["install", "--prefix", &temp_dir.to_string_lossy(), "@viz-js/viz"])
+			.stdout(std::process::Stdio::null())
+			.stderr(std::process::Stdio::piped())
+			.status()
+			.with_context(|| "Failed to run `npm install`. Is Node.js installed?")?;
+		if !status.success() {
+			anyhow::bail!("Executing `npm install @viz-js/viz` failed");
+		}
+	}
+
+	// Write a small script that reads DOT from stdin and outputs SVG
+	let script_path = temp_dir.join("convert.mjs");
+	fs::write(
+		&script_path,
+		r#"
+		import { instance } from "@viz-js/viz";
+		let dot = "";
+		for await (const chunk of process.stdin) dot += chunk;
+		const viz = await instance();
+		process.stdout.write(viz.renderString(dot, { format: "svg" }));
+		"#
+		.trim(),
+	)?;
+
+	let mut child = Command::new("node")
+		.arg(&script_path)
+		.stdin(std::process::Stdio::piped())
+		.stdout(std::process::Stdio::piped())
+		.stderr(std::process::Stdio::piped())
+		.spawn()
+		.with_context(|| "Failed to spawn `node`. Is Node.js installed?")?;
+
+	// Write DOT content to stdin then close the pipe
+	child.stdin.take().unwrap().write_all(dot.as_bytes()).with_context(|| "Failed to write DOT content to stdin")?;
+
+	let output = child.wait_with_output().with_context(|| "Failed to wait for `node` process")?;
+
+	// Clean up the temp script (node_modules is intentionally kept as a cache)
+	let _ = fs::remove_file(&script_path);
+
+	if !output.status.success() {
+		let stderr = String::from_utf8_lossy(&output.stderr);
+		anyhow::bail!("DOT to SVG conversion failed (exit code {:?}):\n{}", output.status.code(), stderr);
+	}
+
+	String::from_utf8(output.stdout).with_context(|| "SVG output was not valid UTF-8")
+}
+
 fn generate_dot(crates: &[CrateInfo]) -> String {
 	let mut out = String::new();
 	out.push_str("digraph CrateHierarchy {\n");

website/.build-scripts/generate-crate-hierarchy.ts

@@ -13,7 +13,6 @@
 	"scripts": {
 		"postinstall": "node .build-scripts/install.ts",
 		"generate-editor-structure": "node .build-scripts/generate-editor-structure.ts generated/hierarchical_message_system_tree.txt generated/hierarchical_message_system_tree.html",
-		"generate-crate-hierarchy": "node .build-scripts/generate-crate-hierarchy.ts generated/crate_hierarchy.dot generated/crate_hierarchy.svg",
 		"check": "tsc --noEmit && eslint",
 		"fix": "eslint --fix"
 	},
@@ -22,7 +21,6 @@
 		"@eslint/eslintrc": "^3.3.3",
 		"@eslint/js": "^9.39.2",
 		"@types/node": "^25.0.9",
-		"@viz-js/viz": "^3.25.0",
 		"eslint": "^9.39.2",
 		"eslint-config-prettier": "^10.1.8",
 		"eslint-import-resolver-typescript": "^4.4.4",

website/package-lock.json

@@ -57,8 +57,6 @@ <h2 class="headline"><a href="{{ article.permalink | safe }}">{{ article.title }
 
 TO TEST IT LOCALLY, FROM THE ROOT OF THE PROJECT, RUN:
 
-cargo run -p crate-hierarchy-viz -- website/generated/crate_hierarchy.dot
-cd website
-npm run generate-crate-hierarchy</pre>" -%}
+cargo run -p crate-hierarchy-viz -- website/generated/crate_hierarchy.svg</pre>" -%}
 {{ content | default(value = fallback) | safe }}
 {% endmacro crate_hierarchy %}