feat: add readonly tuple type for AGENT_NAMES constant

Signed-off-by: leocavalcante <leo@cavalcante.dev>

Author: leocavalcante

src/paths.d.mts

@@ -31,8 +31,9 @@ export declare const OPENCODE_VERSION: string
 /**
  * List of expected agent names (without .md extension).
  * This is the single source of truth for agent filenames.
+ * Frozen to prevent accidental mutation.
  */
-export declare const AGENT_NAMES: string[]
+export declare const AGENT_NAMES: readonly ["opencoder", "opencoder-planner", "opencoder-builder"]
 
 /** Minimum character count for valid agent files */
 export declare const MIN_CONTENT_LENGTH: number

tests/index.test.ts

@@ -39,7 +39,7 @@ describe("index.ts exports", () => {
 		const files = readdirSync(agentsDir)
 		const mdFiles = files.filter((f) => f.endsWith(".md"))
 		const agentNamesFromFiles = mdFiles.map((f) => f.replace(/\.md$/, "")).sort()
-		const agentNamesFromExport = [...agents].sort()
+		const agentNamesFromExport = ([...agents] as string[]).sort()
 
 		expect(agentNamesFromExport).toEqual(agentNamesFromFiles)
 	})

tests/paths.test.ts

@@ -397,23 +397,23 @@ describe("paths.mjs exports", () => {
 		it("should prevent push to AGENT_NAMES", () => {
 			const originalLength = AGENT_NAMES.length
 			expect(() => {
-				;(AGENT_NAMES as string[]).push("new-agent")
+				;(AGENT_NAMES as unknown as string[]).push("new-agent")
 			}).toThrow()
 			expect(AGENT_NAMES.length).toBe(originalLength)
 		})
 
 		it("should prevent modification of AGENT_NAMES elements", () => {
 			const originalFirst = AGENT_NAMES[0]
 			expect(() => {
-				;(AGENT_NAMES as string[])[0] = "modified"
+				;(AGENT_NAMES as unknown as string[])[0] = "modified"
 			}).toThrow()
 			expect(AGENT_NAMES[0]).toBe(originalFirst)
 		})
 
 		it("should prevent pop from AGENT_NAMES", () => {
 			const originalLength = AGENT_NAMES.length
 			expect(() => {
-				;(AGENT_NAMES as string[]).pop()
+				;(AGENT_NAMES as unknown as string[]).pop()
 			}).toThrow()
 			expect(AGENT_NAMES.length).toBe(originalLength)
 		})