refactor: convert exported functions to internal functions for improved encapsulation

Author: AndyLiner13

package.json

@@ -43,7 +43,6 @@
     "@types/debug": "^4.1.12",
     "@types/filesystem": "^0.0.36",
     "@types/node": "^25.0.0",
-    "@types/sinon": "^21.0.0",
     "@types/ws": "^8.18.1",
     "@types/yargs": "^17.0.33",
     "@typescript-eslint/eslint-plugin": "^8.43.0",
@@ -59,7 +58,6 @@
     "rollup": "4.57.1",
     "rollup-plugin-cleanup": "^3.2.1",
     "rollup-plugin-license": "^3.6.0",
-    "sinon": "^21.0.0",
     "typescript": "^5.9.2",
     "typescript-eslint": "^8.43.0",
     "yargs": "18.0.0",
@@ -70,7 +68,6 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "1.27.0",
-    "get-port": "^7.1.0",
     "jsonc-parser": "^3.3.0",
     "logpare": "^0.1.0",
     "ws": "^8.19.0"

pnpm-lock.yaml

@@ -526,7 +526,7 @@ export async function fetchAXTree(verbose: boolean): Promise<AXTreeResult> {
 /**
  * Get the frame ID for a given UID (for cross-frame interactions).
  */
-export function getFrameIdForUid(uid: string): string | undefined {
+function getFrameIdForUid(uid: string): string | undefined {
   return uidToFrameId.get(uid);
 }
 
@@ -587,7 +587,7 @@ export function getBackendNodeId(uid: string): number {
 /**
  * Resolve a UID to a CDP RemoteObjectId by calling DOM.resolveNode.
  */
-export async function resolveNodeToRemoteObject(uid: string): Promise<string> {
+async function resolveNodeToRemoteObject(uid: string): Promise<string> {
   const backendNodeId = getBackendNodeId(uid);
   const sessionId = getSessionIdForUid(uid);
   const opts = sessionId ? {sessionId} : undefined;

src/ax-tree.ts

@@ -208,7 +208,7 @@ function sendClientRequest(
 /**
  * Execute a VS Code command in the Client window.
  */
-export async function commandExecute(
+async function commandExecute(
   command: string,
   args?: unknown[],
 ): Promise<CommandExecuteResult> {
@@ -408,7 +408,7 @@ export async function codebaseGetOverview(
 /**
  * Get detailed exports from a module/file/directory.
  */
-export async function codebaseGetExports(
+async function codebaseGetExports(
   path: string,
   rootDir?: string,
   includeTypes?: boolean,
@@ -736,7 +736,7 @@ export interface UnifiedFileSymbol {
   modifiers?: string[];
 }
 
-export interface UnifiedFileResult {
+interface UnifiedFileResult {
   symbols: UnifiedFileSymbol[];
   content: string;
   totalLines: number;
@@ -1000,7 +1000,7 @@ export async function fileApplyCodeAction(
  * Extract orphaned content (imports, exports, comments) from TypeScript/JavaScript files.
  * Supplements VS Code's DocumentSymbol API which doesn't include these constructs.
  */
-export async function fileExtractOrphanedContent(
+async function fileExtractOrphanedContent(
   filePath: string,
   includeSymbols = true,
 ): Promise<OrphanedContentResult> {
@@ -1133,7 +1133,7 @@ export async function ensureClientAvailable(): Promise<void> {
 /**
  * Returns the fixed Client pipe path for this platform.
  */
-export function getClientPipePath(): string {
+function getClientPipePath(): string {
   return CLIENT_PIPE_PATH;
 }
 

src/client-pipe.ts

@@ -286,7 +286,7 @@ export function writeExtSourceFingerprint(extensionDir: string): void {
  * @param sessionStartedAtMs Debug session start timestamp in epoch ms
  * @deprecated Use isBuildStale instead for proper source vs build comparison
  */
-export function hasExtensionChangedSince(
+function hasExtensionChangedSince(
   extensionDir: string,
   sessionStartedAtMs: number,
 ): boolean {

src/extension-watcher.ts

@@ -200,7 +200,7 @@ export async function hotReloadRequired(params: McpReadyParams): Promise<HotRelo
 /**
  * Query the current state of the Host and Client.
  */
-export async function getStatus(): Promise<HostStatus> {
+async function getStatus(): Promise<HostStatus> {
   const result = await sendHostRequest('getStatus', {});
   return result as HostStatus;
 }
@@ -218,7 +218,7 @@ export async function teardown(): Promise<TeardownResult> {
  * Request a session takeover from the existing Host.
  * Used when a new MCP instance wants to control the session.
  */
-export async function requestTakeover(): Promise<TakeoverResult> {
+async function requestTakeover(): Promise<TakeoverResult> {
   const result = await sendHostRequest('takeover', {});
   return result as TakeoverResult;
 }
@@ -254,7 +254,7 @@ export async function showHostNotification(message: string): Promise<void> {
 /**
  * Check if the Host pipe is reachable via a system.ping.
  */
-export async function pingHost(): Promise<boolean> {
+async function pingHost(): Promise<boolean> {
   try {
     await sendHostRequest('system.ping', {}, 3_000);
     return true;
@@ -266,6 +266,6 @@ export async function pingHost(): Promise<boolean> {
 /**
  * Returns the fixed Host pipe path for this platform.
  */
-export function getHostPipePath(): string {
+function getHostPipePath(): string {
   return HOST_PIPE_PATH;
 }

src/host-pipe.ts

@@ -48,7 +48,7 @@ export function getIssueDescription(fileName: string): string | null {
   return issueDescriptions[fileName] ?? null;
 }
 
-export const ISSUE_UTILS = {
+const ISSUE_UTILS = {
   loadIssueDescriptions,
   getIssueDescription,
 };

src/issue-descriptions.ts

@@ -74,7 +74,7 @@ function buildLogpareOptions(format: LogFormat): CompressOptions {
  * Consolidate a raw text string (e.g., terminal or task output).
  * Returns LogPare's compressed format when meaningful compression is achievable.
  */
-export function consolidateOutput(
+function consolidateOutput(
   text: string,
   options?: ConsolidationOptions,
 ): ConsolidationResult {
@@ -157,7 +157,7 @@ function noCompression(text: string, lineCount: number): ConsolidationResult {
 /**
  * Convert a ConsolidationResult to a JSON-safe object for API responses.
  */
-export function toConsolidatedJson(
+function toConsolidatedJson(
   result: ConsolidationResult,
 ): Record<string, unknown> {
   return {

src/log-consolidator.ts

@@ -273,7 +273,7 @@ const cliArgs = parseArguments(VERSION);
 export const config: ResolvedConfig = loadConfig(cliArgs);
 
 // Legacy export for backwards compatibility
-export const args = cliArgs;
+const args = cliArgs;
 
 // ── MCP Server Hot-Reload Marker (detect post-restart) ───
 // Check this BEFORE initializing lifecycle service so we can pass the flag