fix(mcpb): expand template variables in config paths

jqnatividad · claude · jqnatividad · commit 3177cfe1e32c · 2026-01-07T16:05:10.000-05:00
Fixed issue where paths like ${HOME}/Downloads were used literally instead of being expanded to actual home directory paths. Changes to config.ts: - Added expandTemplateVars() function to expand template variables: * ${HOME} → User's home directory (Unix/Mac) * ${USERPROFILE} → User's home directory (Windows) * ${DESKTOP} → Desktop directory * ${DOCUMENTS} → Documents directory * ${DOWNLOADS} → Downloads directory * ${TEMP}, ${TMPDIR} → Temporary directory - Updated getStringEnv() to expand variables in single paths - Updated getStringArrayEnv() to expand variables in path arrays This allows Claude Desktop extension configuration values like "${HOME}/Downloads" to properly resolve to "/Users/username/Downloads" at runtime. Also updated manifest.json author from personal to company name. Regenerated qsv-mcp-server.mcpb with fixes. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
diff --git a/.claude/skills/manifest.json b/.claude/skills/manifest.json
@@ -4,8 +4,8 @@
   "version": "13.0.0",
   "description": "Complete CSV data-wrangling toolkit with 66 commands for transforming, analyzing, and validating tabular data. Process local CSV, Excel, and JSONL files without uploading.",
   "author": {
-    "name": "Joel Natividad",
-    "url": "https://github.com/dathere/qsv"
+    "name": "datHere, Inc.",
+    "url": "https://dathere.com"
   },
   "repository": {
     "type": "git",
diff --git a/.claude/skills/qsv-mcp-server.mcpb b/.claude/skills/qsv-mcp-server.mcpb
diff --git a/.claude/skills/src/config.ts b/.claude/skills/src/config.ts
@@ -5,6 +5,44 @@
  * Supports both legacy MCP server and Desktop Extension modes.
  */
 
+import { homedir, tmpdir } from 'os';
+import { join } from 'path';
+
+/**
+ * Expand template variables in strings
+ * Supports: ${HOME}, ${USERPROFILE}, ${DESKTOP}, ${DOCUMENTS}, ${DOWNLOADS}, ${TEMP}, ${TMPDIR}
+ */
+function expandTemplateVars(value: string): string {
+  if (!value) return value;
+
+  const home = homedir();
+
+  // Get platform-specific special directories
+  const desktop = process.platform === 'win32'
+    ? join(home, 'Desktop')
+    : join(home, 'Desktop');
+
+  const documents = process.platform === 'win32'
+    ? join(home, 'Documents')
+    : join(home, 'Documents');
+
+  const downloads = process.platform === 'win32'
+    ? join(home, 'Downloads')
+    : join(home, 'Downloads');
+
+  const temp = tmpdir();
+
+  // Replace template variables
+  return value
+    .replace(/\$\{HOME\}/g, home)
+    .replace(/\$\{USERPROFILE\}/g, home)
+    .replace(/\$\{DESKTOP\}/g, desktop)
+    .replace(/\$\{DOCUMENTS\}/g, documents)
+    .replace(/\$\{DOWNLOADS\}/g, downloads)
+    .replace(/\$\{TEMP\}/g, temp)
+    .replace(/\$\{TMPDIR\}/g, temp);
+}
+
 /**
  * Parse integer from environment variable with validation
  */
@@ -59,19 +97,25 @@ function parseFloatEnv(envVar: string, defaultValue: number, min?: number, max?:
 
 /**
  * Get string from environment variable with default
+ * Expands template variables like ${HOME}, ${USERPROFILE}, etc.
  * Uses nullish coalescing (??) to allow empty strings while falling back for undefined/null
  */
 function getStringEnv(envVar: string, defaultValue: string): string {
-  return process.env[envVar] ?? defaultValue;
+  const value = process.env[envVar] ?? defaultValue;
+  return expandTemplateVars(value);
 }
 
 /**
  * Get string array from environment variable (split by delimiter)
+ * Expands template variables in each path
  */
 function getStringArrayEnv(envVar: string, defaultValue: string[], delimiter: string): string[] {
   const value = process.env[envVar];
   if (!value) return defaultValue;
-  return value.split(delimiter).filter(s => s.length > 0);
+  return value
+    .split(delimiter)
+    .filter(s => s.length > 0)
+    .map(s => expandTemplateVars(s));
 }
 
 /**
