@hcu-lab.me/mcp-device 1.0.6 → 1.0.7

package/dist/utils/feature-flags.js

@@ -1,215 +1,21 @@
-import fs from 'fs/promises';
-import path from 'path';
-import { existsSync } from 'fs';
-import { CONFIG_FILE } from '../config.js';
-import { logger } from './logger.js';
 class FeatureFlagManager {
     constructor() {
         this.flags = {};
-        this.lastFetch = 0;
-        this.cacheMaxAge = 30 * 60 * 1000;
-        this.refreshInterval = null;
-        // Track fresh fetch status for A/B tests that need network flags
-        this.freshFetchPromise = null;
-        this.resolveFreshFetch = null;
-        this.loadedFromCache = false;
-        const configDir = path.dirname(CONFIG_FILE);
-        this.cachePath = path.join(configDir, 'feature-flags.json');
-        // Use production flags (v2 supports weighted variants)
-        this.flagUrl = process.env.DC_FLAG_URL ||
-            'https://desktopcommander.app/flags/v2/production.json';
-        // Set up promise for waiting on fresh fetch
-        this.freshFetchPromise = new Promise((resolve) => {
-            this.resolveFreshFetch = resolve;
-        });
     }
-    /**
-     * Initialize - load from cache and start background refresh
-     */
-    async initialize() {
-        try {
-            // Load from cache immediately (non-blocking)
-            await this.loadFromCache();
-            // Fetch in background (don't block startup)
-            this.fetchFlags().then(() => {
-                // Signal that fresh flags are now available
-                if (this.resolveFreshFetch) {
-                    this.resolveFreshFetch();
-                }
-            }).catch(err => {
-                logger.debug('Initial flag fetch failed:', err.message);
-                // Still resolve the promise so waiters don't hang forever
-                if (this.resolveFreshFetch) {
-                    this.resolveFreshFetch();
-                }
-            });
-            // Start periodic refresh every 5 minutes
-            this.refreshInterval = setInterval(() => {
-                this.fetchFlags().catch(err => {
-                    logger.debug('Periodic flag fetch failed:', err.message);
-                });
-            }, this.cacheMaxAge);
-            // Allow process to exit even if interval is pending
-            // This is critical for proper cleanup when MCP client disconnects
-            this.refreshInterval.unref();
-            logger.info(`Feature flags initialized (refresh every ${this.cacheMaxAge / 1000}s)`);
-        }
-        catch (error) {
-            logger.warning('Failed to initialize feature flags:', error);
-        }
-    }
-    /**
-     * Get a flag value
-     */
+    async initialize() { }
     get(flagName, defaultValue = false) {
         return this.flags[flagName] !== undefined ? this.flags[flagName] : defaultValue;
     }
-    /**
-     * Get all flags for debugging
-     */
     getAll() {
         return { ...this.flags };
     }
-    /**
-     * Manually refresh flags immediately (for testing)
-     */
     async refresh() {
-        try {
-            await this.fetchFlags();
-            return true;
-        }
-        catch (error) {
-            logger.error('Manual refresh failed:', error);
-            return false;
-        }
+        return true;
     }
-    /**
-     * Check if flags were loaded from cache (vs fresh fetch)
-     */
     wasLoadedFromCache() {
-        return this.loadedFromCache;
-    }
-    /**
-     * Wait for fresh flags to be fetched from network.
-     * Use this when you need to ensure flags are loaded before making decisions
-     * (e.g., A/B test assignments for new users who don't have a cache yet)
-     *
-     * Has a hard timeout to prevent blocking MCP startup if the fetch hangs.
-     * See: https://github.com/wonderwhy-er/DesktopCommanderMCP/issues/465
-     */
-    async waitForFreshFlags() {
-        if (this.freshFetchPromise) {
-            let safetyTimeoutHandle;
-            try {
-                const safetyTimeout = new Promise((resolve) => {
-                    safetyTimeoutHandle = setTimeout(resolve, 5000);
-                });
-                await Promise.race([this.freshFetchPromise, safetyTimeout]);
-            }
-            finally {
-                if (safetyTimeoutHandle) {
-                    clearTimeout(safetyTimeoutHandle);
-                }
-            }
-        }
-    }
-    /**
-     * Load flags from local cache
-     */
-    async loadFromCache() {
-        try {
-            if (!existsSync(this.cachePath)) {
-                logger.debug('No feature flag cache found');
-                this.loadedFromCache = false;
-                return;
-            }
-            const data = await fs.readFile(this.cachePath, 'utf8');
-            const config = JSON.parse(data);
-            if (config.flags) {
-                this.flags = config.flags;
-                this.lastFetch = Date.now();
-                this.loadedFromCache = true;
-                logger.debug(`Loaded ${Object.keys(this.flags).length} feature flags from cache`);
-            }
-        }
-        catch (error) {
-            logger.warning('Failed to load feature flags from cache:', error);
-            this.loadedFromCache = false;
-        }
-    }
-    /**
-     * Fetch flags from remote URL
-     */
-    async fetchFlags() {
-        const FETCH_TIMEOUT_MS = 3000;
-        const controller = new AbortController();
-        const abortTimeout = setTimeout(() => controller.abort(), FETCH_TIMEOUT_MS);
-        let hardTimeoutHandle;
-        try {
-            // Don't log here - runs async and can interfere with MCP clients
-            // Use Promise.race as a hard timeout safety net.
-            // On some platforms (Windows + Node 24 / undici 7.x), AbortController.abort()
-            // fails to interrupt an in-progress TCP connect — the fetch hangs until the
-            // OS-level TCP timeout (~30s on Windows). Promise.race guarantees we reject
-            // at the JS level regardless of AbortController behavior.
-            // See: https://github.com/wonderwhy-er/DesktopCommanderMCP/issues/465
-            const fetchPromise = fetch(this.flagUrl, {
-                signal: controller.signal,
-                headers: {
-                    'Cache-Control': 'no-cache',
-                }
-            });
-            const hardTimeout = new Promise((_, reject) => hardTimeoutHandle = setTimeout(() => reject(new Error('Feature flags fetch timed out')), FETCH_TIMEOUT_MS));
-            const response = await Promise.race([fetchPromise, hardTimeout]);
-            if (!response.ok) {
-                throw new Error(`HTTP ${response.status}: ${response.statusText}`);
-            }
-            const config = await response.json();
-            // Update flags
-            if (config.flags) {
-                this.flags = config.flags;
-                this.lastFetch = Date.now();
-                // Save to cache (silently - don't log during async operations
-                // as it can interfere with MCP clients that close quickly)
-                await this.saveToCache(config);
-            }
-        }
-        catch (error) {
-            logger.debug('Failed to fetch feature flags:', error.message);
-            // Continue with cached values
-        }
-        finally {
-            clearTimeout(abortTimeout);
-            if (hardTimeoutHandle) {
-                clearTimeout(hardTimeoutHandle);
-            }
-        }
-    }
-    /**
-     * Save flags to local cache
-     */
-    async saveToCache(config) {
-        try {
-            const configDir = path.dirname(this.cachePath);
-            if (!existsSync(configDir)) {
-                await fs.mkdir(configDir, { recursive: true });
-            }
-            await fs.writeFile(this.cachePath, JSON.stringify(config, null, 2), 'utf8');
-            // Don't log here - this runs async and can cause issues with MCP clients
-        }
-        catch (error) {
-            logger.warning('Failed to save feature flags to cache:', error);
-        }
-    }
-    /**
-     * Cleanup on shutdown
-     */
-    destroy() {
-        if (this.refreshInterval) {
-            clearInterval(this.refreshInterval);
-            this.refreshInterval = null;
-        }
+        return false;
     }
+    async waitForFreshFlags() { }
+    destroy() { }
 }
-// Export singleton instance
 export const featureFlagManager = new FeatureFlagManager();

package/dist/utils/files/docx.d.ts

@@ -5,10 +5,10 @@
  *
  * READ (default): Returns a text-bearing outline — skips shapes, drawings, SVG noise.
  *   Shows paragraphs with text, tables with cell content, style info, and image refs.
- *   Each element shows its raw XML tag context so Claude can target it for editing.
+ *   Each element shows its raw XML tag context so an agent can target it for editing.
  *
  * READ (with offset/length): Returns raw pretty-printed XML with line pagination,
- *   so Claude can drill into specific sections when the outline isn't enough.
+ *   so an agent can drill into specific sections when the outline isn't enough.
  *
  * EDIT (old_string/new_string): Find/replace on the pretty-printed XML, then
  *   compact and repack into valid DOCX. Works exactly like text file editing.

package/dist/utils/files/docx.js

@@ -5,10 +5,10 @@
  *
  * READ (default): Returns a text-bearing outline — skips shapes, drawings, SVG noise.
  *   Shows paragraphs with text, tables with cell content, style info, and image refs.
- *   Each element shows its raw XML tag context so Claude can target it for editing.
+ *   Each element shows its raw XML tag context so an agent can target it for editing.
  *
  * READ (with offset/length): Returns raw pretty-printed XML with line pagination,
- *   so Claude can drill into specific sections when the outline isn't enough.
+ *   so an agent can drill into specific sections when the outline isn't enough.
  *
  * EDIT (old_string/new_string): Find/replace on the pretty-printed XML, then
  *   compact and repack into valid DOCX. Works exactly like text file editing.

package/dist/utils/usageTracker.js

@@ -162,18 +162,6 @@ class UsageTracker {
         if (!feedbackEnabled) {
             return false;
         }
-        // Check if client is desktop-commander (disable for this client)
-        //return true;
-        try {
-            const { currentClient } = await import('../server.js');
-            if (currentClient?.name === 'desktop-commander') {
-                return false;
-            }
-        }
-        catch (error) {
-            // If we can't import server, continue with other checks
-            console.log('[FEEDBACK DEBUG] Could not check client name, continuing...');
-        }
         const stats = await this.getStats();
         // Don't prompt if feedback already given (check top-level config)
         const feedbackGiven = await configManager.getValue('feedbackGiven');
@@ -339,7 +327,6 @@ class UsageTracker {
         // within seconds, so when the flag is ON onboarding still fires on a
         // later call while the user is new. Deliberately no waiting here to keep
         // tool calls latency-free.
-        // See: https://github.com/wonderwhy-er/DesktopCommanderMCP/issues/538
         const onboardingEnabled = featureFlagManager.get('onboarding_injection', false);
         if (!onboardingEnabled) {
             return false;
@@ -353,17 +340,6 @@ class UsageTracker {
         if (global.disableOnboarding) {
             return false;
         }
-        // Check if client is desktop-commander (disable for this client)
-        try {
-            const { currentClient } = await import('../server.js');
-            if (currentClient?.name === 'desktop-commander') {
-                return false;
-            }
-        }
-        catch (error) {
-            // If we can't import server, continue with other checks
-            console.log('[ONBOARDING DEBUG] Could not check client name, continuing...');
-        }
         const stats = await this.getStats();
         const onboardingState = await this.getOnboardingState();
         const now = Date.now();

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "1.0.6";
+export declare const VERSION = "1.0.7";

package/dist/version.js

@@ -1 +1 @@
-export const VERSION = '1.0.6';
+export const VERSION = '1.0.7';

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@hcu-lab.me/mcp-device",
-  "version": "1.0.6",
-  "description": "MCP Device securely connects a user-owned computer to an authenticated Model Context Protocol gateway.",
+  "version": "1.0.7",
+  "description": "MCP Device runtime.",
   "license": "MIT",
   "author": "Hirohimeow",
   "homepage": "https://github.com/Hiroshimeow/agent-mcp-device",
@@ -33,11 +33,11 @@
     "bump": "node scripts/sync-version.js --bump",
     "bump:minor": "node scripts/sync-version.js --bump --minor",
     "bump:major": "node scripts/sync-version.js --bump --major",
-    "build": "shx rm -rf dist/remote-device && shx rm -f dist/device/desktop-commander-integration.js dist/device/desktop-commander-integration.d.ts dist/device/windows-dpapi.js dist/device/windows-dpapi.d.ts && tsc && shx chmod +x dist/*.js && shx cp src/device/update-helper.cjs dist/device/update-helper.cjs && shx mkdir -p dist/data && shx cp src/data/onboarding-prompts.json dist/data/ && node scripts/copy-official-ca.cjs && node scripts/build-ui-runtime.cjs",
+    "build": "shx rm -rf dist && tsc && shx chmod +x dist/*.js && shx cp src/device/update-helper.cjs dist/device/update-helper.cjs && shx mkdir -p dist/data && shx cp src/data/onboarding-prompts.json dist/data/ && node scripts/copy-official-ca.cjs && node scripts/build-ui-runtime.cjs",
     "watch": "tsc --watch",
     "start": "node dist/mcp-device.js",
     "start:debug": "node --inspect-brk=9229 dist/mcp-device.js",
-    "prepack": "node scripts/verify-production-trust.cjs && shx rm -rf dist/remote-device && shx rm -f dist/device/desktop-commander-integration.js dist/device/desktop-commander-integration.d.ts dist/device/windows-dpapi.js dist/device/windows-dpapi.d.ts dist/hcu-device.js dist/hcu-device.d.ts dist/track-installation.js dist/setup-claude-server.js dist/uninstall-claude-server.js dist/npm-scripts/setup.js dist/npm-scripts/setup.d.ts dist/npm-scripts/uninstall.js dist/npm-scripts/uninstall.d.ts dist/npm-scripts/verify-ripgrep.js dist/npm-scripts/verify-ripgrep.d.ts",
+    "prepack": "node scripts/verify-production-trust.cjs",
     "prepare": "npm run build",
     "clean": "shx rm -rf dist",
     "test": "npm run build && node test/run-all-tests.js",
@@ -57,7 +57,6 @@
     "mcp",
     "model-context-protocol",
     "terminal",
-    "claude",
     "ai",
     "command-line",
     "process-management",
@@ -73,7 +72,7 @@
     "file-operations"
   ],
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.9.0",
+    "@modelcontextprotocol/sdk": "1.30.0",
     "@opendocsg/pdf2md": "^0.2.2",
     "@tiptap/core": "^3.31.3",
     "@tiptap/extension-image": "^3.31.3",

package/dist/tools/feedback.d.ts

@@ -1,8 +0,0 @@
-import { ServerResult } from '../types.js';
-interface FeedbackParams {
-}
-/**
- * Open feedback form in browser with optional pre-filled data
- */
-export declare function giveFeedbackToDesktopCommander(params?: FeedbackParams): Promise<ServerResult>;
-export {};

package/dist/tools/feedback.js

@@ -1,132 +0,0 @@
-import { usageTracker } from '../utils/usageTracker.js';
-import { capture } from '../utils/capture.js';
-import { configManager } from '../config-manager.js';
-import { exec } from 'child_process';
-import { promisify } from 'util';
-import * as os from 'os';
-const execAsync = promisify(exec);
-/**
- * Open feedback form in browser with optional pre-filled data
- */
-export async function giveFeedbackToDesktopCommander(params = {}) {
-    try {
-        // Get usage stats for context
-        const stats = await usageTracker.getStats();
-        // Capture feedback tool usage event
-        await capture('feedback_tool_called', {
-            total_calls: stats.totalToolCalls,
-            successful_calls: stats.successfulCalls,
-            failed_calls: stats.failedCalls,
-            days_since_first_use: Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24)),
-            total_sessions: stats.totalSessions,
-            platform: os.platform(),
-        });
-        // Build Tally.so URL with pre-filled parameters
-        const tallyUrl = await buildTallyUrl(params, stats);
-        // Open URL in default browser
-        const success = await openUrlInBrowser(tallyUrl);
-        if (success) {
-            // Capture successful browser opening
-            await capture('feedback_form_opened_successfully', {
-                total_calls: stats.totalToolCalls,
-                platform: os.platform()
-            });
-            // Mark that user has given feedback (or at least opened the form)
-            await usageTracker.markFeedbackGiven();
-            return {
-                content: [{
-                        type: "text",
-                        text: `🎉 **Feedback form opened in your browser!**\n\n` +
-                            `Thank you for taking the time to share your experience with MCP Device. ` +
-                            `Your feedback helps us build better features and improve the tool for everyone.\n\n` +
-                            `The form has been pre-filled with the information you provided. ` +
-                            `You can modify or add any additional details before submitting.\n\n` +
-                            `**Form URL**: ${tallyUrl.length > 100 ? tallyUrl.substring(0, 100) + '...' : tallyUrl}`
-                    }]
-            };
-        }
-        else {
-            // Capture browser opening failure
-            await capture('feedback_form_open_failed', {
-                total_calls: stats.totalToolCalls,
-                platform: os.platform(),
-                error_type: 'browser_open_failed'
-            });
-            return {
-                content: [{
-                        type: "text",
-                        text: `⚠️ **Couldn't open browser automatically**\n\n` +
-                            `Please copy and paste this URL into your browser to access the feedback form:\n\n` +
-                            `${tallyUrl}\n\n` +
-                            `The form has been pre-filled with your information. Thank you for your feedback!`
-                    }]
-            };
-        }
-    }
-    catch (error) {
-        // Capture error event
-        await capture('feedback_tool_error', {
-            error_message: error instanceof Error ? error.message : String(error),
-            error_type: error instanceof Error ? error.constructor.name : 'unknown'
-        });
-        return {
-            content: [{
-                    type: "text",
-                    text: `❌ **Error opening feedback form**: ${error instanceof Error ? error.message : String(error)}\n\n` +
-                        `You can still access our feedback form directly at: https://tally.so/r/mKqoKg\n\n` +
-                        `We appreciate your willingness to provide feedback!`
-                }],
-            isError: true
-        };
-    }
-}
-/**
- * Build Tally.so URL with pre-filled parameters
- */
-async function buildTallyUrl(params, stats) {
-    const baseUrl = 'https://tally.so/r/mKqoKg';
-    const urlParams = new URLSearchParams();
-    // Only auto-filled hidden fields remain
-    urlParams.set('tool_call_count', stats.totalToolCalls.toString());
-    // Calculate days using
-    const daysUsing = Math.floor((Date.now() - stats.firstUsed) / (1000 * 60 * 60 * 24));
-    urlParams.set('days_using', daysUsing.toString());
-    // Add platform info
-    urlParams.set('platform', os.platform());
-    // Add client_id from analytics config
-    try {
-        const clientId = await configManager.getValue('clientId') || 'unknown';
-        urlParams.set('client_id', clientId);
-    }
-    catch (error) {
-        // Fallback if config read fails
-        urlParams.set('client_id', 'unknown');
-    }
-    return `${baseUrl}?${urlParams.toString()}`;
-}
-/**
- * Open URL in default browser (cross-platform)
- */
-async function openUrlInBrowser(url) {
-    try {
-        const platform = os.platform();
-        let command;
-        switch (platform) {
-            case 'darwin': // macOS
-                command = `open "${url}"`;
-                break;
-            case 'win32': // Windows
-                command = `start "" "${url}"`;
-                break;
-            default: // Linux and others
-                command = `xdg-open "${url}"`;
-                break;
-        }
-        await execAsync(command);
-        return true;
-    }
-    catch (error) {
-        console.error('Failed to open browser:', error);
-        return false;
-    }
-}