chore(scripts): drop the script unit tests and the exports that served them

Author: waleedlatif1

package.json

@@ -31,7 +31,7 @@
     "check:cron-parity": "bun run scripts/check-cron-parity.ts",
     "check:api-validation:strict": "bun run scripts/check-api-validation-contracts.ts --check --enforce-boundary-baseline",
     "check:realtime-prune": "bun run scripts/check-realtime-prune-graph.ts",
-    "check:tool-request-boundary": "bun test scripts/check-tool-request-boundary.test.ts && bun run scripts/check-tool-request-boundary.ts",
+    "check:tool-request-boundary": "bun run scripts/check-tool-request-boundary.ts",
     "check:tool-registry-boundary": "bun run scripts/check-tool-registry-boundary.ts",
     "check:sql-date-binding": "bun run scripts/check-sql-date-binding.ts",
     "check:zustand-v5": "bun run scripts/check-zustand-v5-selectors.ts",

scripts/check-migrations-safety.test.ts

@@ -330,7 +330,7 @@ const ANNOTATE_GUIDANCE =
   'is a contract-phase op. Confirm the old code no longer reads/writes it (it must have shipped in an earlier deploy — not this same PR), then acknowledge with a `-- migration-safe: <reason>` comment on the line above.'
 
 /** Lint a single migration's SQL. Returns only actionable findings. */
-export function lintSql(content: string): Finding[] {
+function lintSql(content: string): Finding[] {
   const lines = content.split('\n')
   const statements = parseStatements(content)
   const createdTables = new Set<string>()

scripts/check-migrations-safety.ts

@@ -148,7 +148,7 @@ function isLikelyToolIdentifier(expression: SyntaxNode): boolean {
   )
 }
 
-export function findToolRequestBoundaryViolations(source: string, file = 'source.ts'): Violation[] {
+function findToolRequestBoundaryViolations(source: string, file = 'source.ts'): Violation[] {
   const extension = extname(file)
   const syntaxTree = parse(source, {
     sourceFilename: file,