refactor: improve MIN_COMPONENT_LINES implementation

- Changed MIN_COMPONENT_LINES constant to use getter/setter functions
- Updated references to use getMinComponentLines() function
- Modified test helper to set value to 0 during tests
- This establishes a single source of truth while making testing easier

Signed-off-by: Eric Wheeler <[email protected]>

Author: Eric Wheeler

src/services/tree-sitter/__tests__/helpers.ts

@@ -1,10 +1,9 @@
 import { jest } from "@jest/globals"
-import { parseSourceCodeDefinitionsForFile } from ".."
+import { parseSourceCodeDefinitionsForFile, setMinComponentLines } from ".."
 import * as fs from "fs/promises"
 import * as path from "path"
 import Parser from "web-tree-sitter"
 import tsxQuery from "../queries/tsx"
-
 // Mock setup
 jest.mock("fs/promises")
 export const mockedFs = jest.mocked(fs)
@@ -74,6 +73,9 @@ export async function testParseSourceCodeDefinitions(
 		extKey?: string
 	} = {},
 ): Promise<string | undefined> {
+	// Set minimum component lines to 0 for tests
+	setMinComponentLines(0)
+
 	// Set default options
 	const wasmFile = options.wasmFile || "tree-sitter-tsx.wasm"
 	const queryString = options.queryString || tsxQuery

src/services/tree-sitter/index.ts

@@ -6,7 +6,25 @@ import { fileExistsAtPath } from "../../utils/fs"
 import { parseMarkdown } from "./markdownParser"
 import { RooIgnoreController } from "../../core/ignore/RooIgnoreController"
 
-const MIN_COMPONENT_LINES = 4
+// Private constant
+const DEFAULT_MIN_COMPONENT_LINES_VALUE = 4
+
+// Getter function for MIN_COMPONENT_LINES (for easier testing)
+let currentMinComponentLines = DEFAULT_MIN_COMPONENT_LINES_VALUE
+
+/**
+ * Get the current minimum number of lines for a component to be included
+ */
+export function getMinComponentLines(): number {
+	return currentMinComponentLines
+}
+
+/**
+ * Set the minimum number of lines for a component (for testing)
+ */
+export function setMinComponentLines(value: number): void {
+	currentMinComponentLines = value
+}
 
 const extensions = [
 	"tla",
@@ -287,7 +305,7 @@ function processCaptures(captures: any[], lines: string[], language: string): st
 		const lineCount = endLine - startLine + 1
 
 		// Skip components that don't span enough lines
-		if (lineCount < MIN_COMPONENT_LINES) {
+		if (lineCount < getMinComponentLines()) {
 			return
 		}
 
@@ -325,7 +343,7 @@ function processCaptures(captures: any[], lines: string[], language: string): st
 				const contextSpan = contextEnd - node.parent.startPosition.row + 1
 
 				// Only include context if it spans multiple lines
-				if (contextSpan >= MIN_COMPONENT_LINES) {
+				if (contextSpan >= getMinComponentLines()) {
 					// Add the full range first
 					const rangeKey = `${node.parent.startPosition.row}-${contextEnd}`
 					if (!processedLines.has(rangeKey)) {