fix: resolve linter unused variable errors and improve generators

Author: Mearman

src/generation/generators/perfect-variants.ts

@@ -103,38 +103,26 @@ export const generatePtolemaicEdges = (
 	if (nodeCount === 0) return;
 	if (nodeCount === 1) return;
 
-	// Partition nodes into 2-4 blocks (cliques)
-	const numberBlocks = Math.min(nodeCount, rng.integer(2, 4));
-	const blockSize = Math.floor(nodeCount / numberBlocks);
-	const blocks: number[][] = [];
-
-	for (let index = 0; index < numberBlocks; index++) {
-		const start = index * blockSize;
-		const end = index === numberBlocks - 1 ? nodeCount : (index + 1) * blockSize;
-		blocks.push(Array.from({ length: end - start }, (_, index) => start + index));
+	// Simple approach: Create a path with some triangles
+	// This is ptolemaic (chordal + distance-hereditary)
+	// Build main path
+	for (let index = 0; index < nodeCount - 1; index++) {
+		edges.push({
+			source: nodes[index].id,
+			target: nodes[index + 1].id,
+		});
 	}
 
-	// Make each block a clique
-	for (const block of blocks) {
-		for (let index = 0; index < block.length; index++) {
-			for (let index_ = index + 1; index_ < block.length; index_++) {
-				edges.push({
-					source: nodes[block[index]].id,
-					target: nodes[block[index_]].id,
-				});
-			}
+	// Add some chords to make small blocks (but keep it ptolemaic)
+	// Every 3 nodes, close the triangle if we have enough nodes
+	for (let index = 0; index < nodeCount - 2; index += 3) {
+		if (index + 2 < nodeCount) {
+			edges.push({
+				source: nodes[index].id,
+				target: nodes[index + 2].id,
+			});
 		}
 	}
-
-	// Connect blocks in a tree structure (each block connects to next)
-	for (let index = 0; index < blocks.length - 1; index++) {
-		const u = blocks[index][rng.integer(0, blocks[index].length - 1)];
-		const v = blocks[index + 1][rng.integer(0, blocks[index + 1].length - 1)];
-		edges.push({
-			source: nodes[u].id,
-			target: nodes[v].id,
-		});
-	}
 };
 
 /**
@@ -194,7 +182,6 @@ export const generateQuasiLineEdges = (
 	} else {
 		// Generate complete bipartite K_{m,n}
 		const m = Math.floor(nodeCount / 2);
-		const n = nodeCount - m;
 
 		// Partition nodes
 		for (let index = 0; index < m; index++) {

src/utils/graph-adapters.ts

@@ -191,8 +191,8 @@ export const toSpecifiedGraph = (core: CoreGraph, spec: GraphSpec): SpecifiedGra
  * @returns CoreGraph without spec
  */
 export const fromSpecifiedGraph = (specified: SpecifiedGraph): CoreGraph => {
-	 
-	const { spec, ...core } = specified;
+
+	const { spec: _spec, ...core } = specified;
 	return core;
 };
 
@@ -217,8 +217,8 @@ export const toDocumentedGraph = (core: CoreGraph, meta: CoreGraphMeta): Documen
  * @returns CoreGraph without metadata
  */
 export const fromDocumentedGraph = (document: DocumentedGraph): CoreGraph => {
-	 
-	const { meta, ...core } = document;
+
+	const { meta: _meta, ...core } = document;
 	return core;
 };
 
@@ -342,8 +342,8 @@ export const toCoreGraph = (graph:
 	// Already CoreGraph
 	if ("nodes" in graph && "edges" in graph && !("vertices" in graph)) {
 		// Remove spec/meta if present
-		 
-		const { spec, meta, ...core } = graph as SpecifiedGraph &
+
+		const { spec: _spec, meta: _meta, ...core } = graph as SpecifiedGraph &
       DocumentedGraph &
       CoreGraph;
 		return core;
@@ -377,7 +377,7 @@ export const toCoreGraph = (graph:
  * @param node
  */
 const extractNodeAttributes = (node: CoreGraphNode): Record<string, unknown> | undefined => {
-	const { id, label, type, partition, ...rest } = node;
+	const { id: _id, label: _label, type: _type, partition: _partition, ...rest } = node;
 	return Object.keys(rest).length > 0 ? rest : undefined;
 };
 
@@ -386,7 +386,7 @@ const extractNodeAttributes = (node: CoreGraphNode): Record<string, unknown> | u
  * @param edge
  */
 const extractEdgeAttributes = (edge: CoreGraphEdge): Record<string, unknown> | undefined => {
-	const { source, target, weight, directed, type, ...rest } = edge;
+	const { source: _source, target: _target, weight: _weight, directed: _directed, type: _type, ...rest } = edge;
 	return Object.keys(rest).length > 0 ? rest : undefined;
 };
 
@@ -395,7 +395,7 @@ const extractEdgeAttributes = (edge: CoreGraphEdge): Record<string, unknown> | u
  * @param node
  */
 const extractGmlNodeAttributes = (node: GmlGraphNode): Record<string, unknown> | undefined => {
-	const { id, label, ...rest } = node;
+	const { id: _id, label: _label, ...rest } = node;
 	return Object.keys(rest).length > 0 ? rest : undefined;
 };
 
@@ -404,6 +404,6 @@ const extractGmlNodeAttributes = (node: GmlGraphNode): Record<string, unknown> |
  * @param edge
  */
 const extractGmlEdgeAttributes = (edge: GmlGraphEdge): Record<string, unknown> | undefined => {
-	const { source, target, weight, directed, ...rest } = edge;
+	const { source: _source, target: _target, weight: _weight, directed: _directed, ...rest } = edge;
 	return Object.keys(rest).length > 0 ? rest : undefined;
 };