fix: resolve engineering practice violations (Rules 6, 7, 8)

docs-site/scripts/build-llms-txt.ts

@@ -32,11 +32,11 @@ const SUMMARY_TRUNCATE_STEP_CHARS = 25;
 const EXCLUDED_DIRS = ["tracking", "phases", "analysis", "plans", "test-reports"];
 
 // Priority definitions for summary file (lower number = higher priority)
-interface PriorityRule {
+type PriorityRule = {
   pattern: string | RegExp;
   priority: number;
   includeInSummary: boolean;
-}
+};
 
 const PRIORITY_RULES: PriorityRule[] = [
   { pattern: /^index\.md$/, priority: 1, includeInSummary: true },
@@ -82,7 +82,7 @@ const SECTION_ORDER = [
   "visual-content",
 ];
 
-interface DocFile {
+type DocFile = {
   relativePath: string;
   section: string;
   title: string;
@@ -91,12 +91,12 @@ interface DocFile {
   priority: number;
   includeInSummary: boolean;
   order: number;
-}
+};
 
-interface ProviderInfo {
+type ProviderInfo = {
   name: string;
   slug: string;
-}
+};
 
 /**
  * Strip unnecessary formatting from content

docs-site/scripts/create-version.ts

@@ -20,10 +20,10 @@ const ROOT_PACKAGE_JSON = path.resolve(__dirname, "../../package.json");
 const DOCS_SITE_DIR = path.resolve(__dirname, "..");
 const VERSIONS_FILE = path.join(DOCS_SITE_DIR, "versions.json");
 
-interface PackageJson {
+type PackageJson = {
   version: string;
   [key: string]: unknown;
-}
+};
 
 /**
  * Parse a semver version string into its components

docs-site/scripts/gitChangeDetector.ts

@@ -8,12 +8,12 @@ import { fileURLToPath } from "url";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export interface GitChangeInfo {
+export type GitChangeInfo = {
   addedFiles: Set<string>;
   modifiedFiles: Set<string>;
   deletedFiles: Set<string>;
   allChangedFiles: Set<string>;
-}
+};
 
 export class GitChangeDetector {
   /** @deprecated Use tag-based detection instead. Kept for backwards compatibility. */

docs-site/scripts/sync-docs.ts

@@ -138,16 +138,16 @@ const FILE_MAPPINGS: Record<string, string> = {
   "large-context-handling-design-doc.md": "development/large-context-design.md",
 };
 
-interface TransformResult {
+type TransformResult = {
   content: string;
   frontmatter: Record<string, unknown>;
-}
+};
 
-interface FileInfo {
+type FileInfo = {
   sourcePath: string;
   relativePath: string;
   targetPath: string;
-}
+};
 
 /**
  * Convert MkDocs admonitions to Docusaurus format

docs-site/scripts/validate-frontmatter.ts

@@ -53,27 +53,27 @@ const VALID_TWITTER_CARDS = ["summary", "summary_large_image", "app", "player"];
 // Validation issue types
 type IssueLevel = "error" | "warning" | "info";
 
-interface ValidationIssue {
+type ValidationIssue = {
   level: IssueLevel;
   file: string;
   message: string;
   field?: string;
-}
+};
 
-interface ValidationResult {
+type ValidationResult = {
   file: string;
   issues: ValidationIssue[];
   frontmatter: Record<string, unknown>;
-}
+};
 
-interface ValidationSummary {
+type ValidationSummary = {
   totalFiles: number;
   filesWithErrors: number;
   filesWithWarnings: number;
   totalErrors: number;
   totalWarnings: number;
   totalInfo: number;
-}
+};
 
 /**
  * Check if a path should be skipped

docs-site/src/hooks/useAlgoliaSearch.ts

@@ -4,7 +4,7 @@ import {
 } from "algoliasearch/lite";
 import { useCallback, useEffect, useRef, useState } from "react";
 
-export interface SearchResult {
+export type SearchResult = {
   objectID: string;
   title: string;
   url: string;
@@ -25,16 +25,16 @@ export interface SearchResult {
       lvl3?: { value: string };
     };
   };
-}
+};
 
-interface UseAlgoliaSearchOptions {
+type UseAlgoliaSearchOptions = {
   appId: string;
   searchApiKey: string;
   indexName: string;
   debounceMs?: number;
-}
+};
 
-export interface UseAlgoliaSearchReturn {
+export type UseAlgoliaSearchReturn = {
   query: string;
   setQuery: (query: string) => void;
   results: SearchResult[];
@@ -45,7 +45,7 @@ export interface UseAlgoliaSearchReturn {
   selectedIndex: number;
   setSelectedIndex: (index: number) => void;
   totalResults: number;
-}
+};
 
 // Simple LRU cache for recent searches
 class SearchCache {

docs-site/src/hooks/useLocalSearch.ts

@@ -2,7 +2,7 @@ import MiniSearch from "minisearch";
 import { useCallback, useEffect, useRef, useState } from "react";
 import type { SearchResult, UseAlgoliaSearchReturn } from "./useAlgoliaSearch";
 
-interface SearchDocument {
+type SearchDocument = {
   objectID: string;
   title: string;
   url: string;
@@ -13,7 +13,7 @@ interface SearchDocument {
     lvl2?: string;
     lvl3?: string;
   };
-}
+};
 
 let indexPromise: Promise<MiniSearch<SearchDocument>> | null = null;
 
@@ -62,9 +62,9 @@ function loadIndex(): Promise<MiniSearch<SearchDocument>> {
   return indexPromise;
 }
 
-interface UseLocalSearchOptions {
+type UseLocalSearchOptions = {
   debounceMs?: number;
-}
+};
 
 export function useLocalSearch({
   debounceMs = 200,

docs-site/src/theme/hooks/useNewDocs.ts

@@ -28,7 +28,7 @@ import { usePluginData } from "@docusaurus/useGlobalData";
 
 export type DocStatus = "new" | "updated" | "recent";
 
-export interface DocMetadata {
+export type DocMetadata = {
   docId: string;
   filePath: string;
   status: DocStatus;
@@ -39,9 +39,9 @@ export interface DocMetadata {
     message: string;
     date: string;
   } | null;
-}
+};
 
-export interface NewDocsData {
+export type NewDocsData = {
   newDocs: DocMetadata[];
   updatedDocs: DocMetadata[];
   recentDocs: DocMetadata[];
@@ -54,7 +54,7 @@ export interface NewDocsData {
   };
   baseTag: string;
   generatedAt: string;
-}
+};
 
 /**
  * Get all new docs data from the plugin

landing/src/lib/actions/reveal.ts

@@ -1,4 +1,4 @@
-interface RevealOptions {
+type RevealOptions = {
   y?: number;
   x?: number;
   scale?: number;
@@ -8,7 +8,7 @@ interface RevealOptions {
   ease?: string;
   start?: string;
   stagger?: number;
-}
+};
 
 export function reveal(node: HTMLElement, options: RevealOptions = {}) {
   if (window.matchMedia("(prefers-reduced-motion: reduce)").matches) {

landing/src/lib/lsystem.ts

@@ -1,4 +1,4 @@
-export interface LBranch {
+export type LBranch = {
   x1: number;
   y1: number;
   x2: number;
@@ -7,9 +7,9 @@ export interface LBranch {
   depth: number;
   opacity: number;
   isAxon: boolean;
-}
+};
 
-export interface LSystemOptions {
+export type LSystemOptions = {
   originX: number;
   originY: number;
   seed: number;
@@ -18,7 +18,7 @@ export interface LSystemOptions {
   axiom?: string;
   rules?: Record<string, string>;
   angle?: number;
-}
+};
 
 export function mulberry32(seed: number): () => number {
   return function () {
@@ -109,10 +109,10 @@ export function branchesToLines(
   return branches;
 }
 
-export interface NeuronPath {
+export type NeuronPath = {
   points: { x: number; y: number }[];
   length: number; // precomputed arc length
-}
+};
 
 export const cachedDendrites = new Map<string, LBranch[]>();
 export const cachedPaths = new Map<string, NeuronPath[]>();

landing/src/lib/stores/canvasState.ts

@@ -15,15 +15,15 @@ export const scrollProgress = writable<number>(0); // 0–1
 export const scrollVelocity = writable<number>(0); // pixels/frame, smoothed
 
 // Per-section canvas configuration
-export interface SectionCanvasConfig {
+export type SectionCanvasConfig = {
   intensity: number;
   particleSpeed: number;
   branchGrowth: number;
   signalDensity: number;
   dominantColor: string;
   glowNodes: boolean;
   spinalActivity: number;
-}
+};
 
 const SECTION_CONFIGS: Record<SectionId, SectionCanvasConfig> = {
   hero: {

landing/src/routes/api/og/templates.ts

@@ -1,12 +1,12 @@
 export type OGType = "home" | "docs" | "sdk" | "examples";
 
-interface OGParams {
+type OGParams = {
   type: OGType;
   title?: string;
   subtitle?: string;
   section?: string;
   method?: string;
-}
+};
 
 function escapeHtml(text: string): string {
   return text

neurolink-demo/create-comprehensive-demo-videos.ts

@@ -9,10 +9,10 @@ import { chromium, type Page } from "playwright";
 import fs from "fs";
 import path from "path";
 
-interface DemoAction {
+type DemoAction = {
   description: string;
   execute: (page: Page) => Promise<void>;
-}
+};
 
 const BASE_URL = "http://localhost:9876";
 const VIDEOS_DIR = "./videos";

neurolink-demo/mcp-examples.ts

@@ -21,11 +21,11 @@ import { fileURLToPath } from "url";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-interface CommandResult {
+type CommandResult = {
   success: boolean;
   output?: string;
   error?: string;
-}
+};
 
 /**
  * Execute NeuroLink MCP command and return result

neurolink-demo/mcp-helpers.ts

@@ -9,31 +9,31 @@ import { execSync, spawn, type ExecSyncOptions } from "child_process";
 import fs from "fs";
 import path from "path";
 
-interface MCPServerConfig {
+type MCPServerConfig = {
   command?: string;
   args?: string[];
   transport?: string;
   env?: Record<string, string>;
   cwd?: string;
   url?: string;
   name?: string;
-}
+};
 
-interface MCPConfig {
+type MCPConfig = {
   mcpServers: Record<string, MCPServerConfig>;
-}
+};
 
-interface CommandResult {
+type CommandResult = {
   success: boolean;
   output?: string;
   error?: string;
   command?: string;
-}
+};
 
-interface ServerStatus {
+type ServerStatus = {
   available: boolean;
   reason: string;
-}
+};
 
 // MCP configuration file path
 const MCP_CONFIG_FILE = path.join(process.cwd(), ".mcp-config.json");

neurolink-demo/mcp-workflow.ts

@@ -21,17 +21,17 @@ import { fileURLToPath } from "url";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-interface CommandResult {
+type CommandResult = {
   success: boolean;
   output?: string;
   error?: string;
-}
+};
 
-interface WorkflowResult {
+type WorkflowResult = {
   success: boolean;
   error?: string;
   [key: string]: unknown;
-}
+};
 
 /**
  * Execute NeuroLink MCP command safely