skalpel 3.1.5 → 3.1.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skalpel",
-  "version": "3.1.5",
+  "version": "3.1.6",
   "description": "Skalpel — local proxy and TUI for coding agents (skalpel + skalpeld bundle).",
   "license": "Apache-2.0",
   "homepage": "https://skalpel.ai",
@@ -55,10 +55,10 @@
     "x64"
   ],
   "optionalDependencies": {
-    "@skalpelai/skalpel-darwin-arm64": "3.1.5",
-    "@skalpelai/skalpel-darwin-x64": "3.1.5",
-    "@skalpelai/skalpel-linux-arm64": "3.1.5",
-    "@skalpelai/skalpel-linux-x64": "3.1.5",
-    "@skalpelai/skalpel-win32-x64": "3.1.5"
+    "@skalpelai/skalpel-darwin-arm64": "3.1.6",
+    "@skalpelai/skalpel-darwin-x64": "3.1.6",
+    "@skalpelai/skalpel-linux-arm64": "3.1.6",
+    "@skalpelai/skalpel-linux-x64": "3.1.6",
+    "@skalpelai/skalpel-win32-x64": "3.1.6"
   }
 }

package/postinstall/lib/env-inject.js

@@ -45,8 +45,15 @@ function run({ dryRun, port }) {
   for (const rc of rcs) {
     const exists = fs.existsSync(rc.path);
     if (!exists) {
-      log.info(`env-inject: ${rc.shell} rc absent (${rc.path}) — skip`);
-      continue;
+      // BUG-0035: on Windows most users have no PowerShell $PROFILE yet,
+      // but the `claude` wrapper still must be installed — applyBlock
+      // creates the file (and parent dir). On POSIX, skip absent rc files
+      // (the present.length===0 fallback below bootstraps ~/.profile).
+      if (process.platform !== 'win32') {
+        log.info(`env-inject: ${rc.shell} rc absent (${rc.path}) — skip`);
+        continue;
+      }
+      log.info(`env-inject: ${rc.shell} rc absent (${rc.path}) — creating (Windows)`);
     }
     present.push(rc);
     if (dryRun) {

package/postinstall/lib/integrations.js

@@ -47,11 +47,11 @@ end`,
     powershell: `
 # Pre-launch status hint + fail-open for Claude Code.
 # Set $env:SKALPEL_NO_AGENT_WRAP=1 to disable.
-$_skalpelOrigClaude = Get-Command claude.exe -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
+$_skalpelOrigClaude = Get-Command claude -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
 $_skalpelStatusBin = Get-Command skalpel -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
 if (-not $env:SKALPEL_NO_AGENT_WRAP -and $_skalpelOrigClaude -and $_skalpelStatusBin) {
     function global:claude {
-        & $script:_skalpelStatusBin.Source status 1>&2
+        & $script:_skalpelStatusBin.Source status | Out-Host
         if ($LASTEXITCODE -eq 0) {
             & $script:_skalpelStatusBin.Source claude-exec @args
         } else {
@@ -134,11 +134,11 @@ end`,
     powershell: `
 # Codex CLI/App integration. Config overrides are native Codex -c knobs.
 # Set $env:SKALPEL_NO_AGENT_WRAP=1 to disable.
-$_skalpelOrigCodex = Get-Command codex.exe -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
+$_skalpelOrigCodex = Get-Command codex -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
 $_skalpelStatusBin = Get-Command skalpel -CommandType Application -ErrorAction SilentlyContinue | Select-Object -First 1
 if (-not $env:SKALPEL_NO_AGENT_WRAP -and $_skalpelOrigCodex -and $_skalpelStatusBin) {
     function global:codex {
-        & $script:_skalpelStatusBin.Source status 1>&2
+        & $script:_skalpelStatusBin.Source status | Out-Host
         if ($LASTEXITCODE -eq 0) {
             $_skalpelCodexHome = if ($env:CODEX_HOME) { $env:CODEX_HOME } else { Join-Path $HOME '.codex' }
             $_skalpelCodexAuthMode = ''

package/postinstall/lib/rc-edit.test.js

@@ -198,6 +198,26 @@ function run() {
     assert.ok(!block.includes('network.proxy_url'), 'Codex wrapper should not generic-proxy metadata routes');
   });
 
+  test('TestRcEdit_PowerShell_Wrapper_Valid_Syntax', () => {
+    // BUG-0036 regression: the PowerShell wrappers must not use `1>&2`
+    // (a PowerShell parse error → the whole profile block fails to load
+    // → the claude/codex functions are never defined → bare `claude`
+    // bypasses skalpel on Windows). And the original-binary lookup must
+    // target `claude`/`codex` (npm installs claude.cmd/.ps1 on Windows),
+    // not `claude.exe`/`codex.exe` (which don't exist → null → wrapper
+    // never engages).
+    const env = rc.envBlockValues(7878);
+    const block = rc.buildBlock('powershell', env);
+    assert.ok(!block.includes('1>&2'), 'PowerShell wrapper uses unsupported 1>&2 redirect');
+    assert.ok(!block.includes('claude.exe'), 'PowerShell wrapper looks up claude.exe (absent on npm Windows)');
+    assert.ok(!block.includes('codex.exe'), 'PowerShell wrapper looks up codex.exe (absent on npm Windows)');
+    assert.ok(
+      block.includes('Get-Command claude -CommandType Application'),
+      'PowerShell wrapper should resolve claude via Get-Command claude'
+    );
+    assert.ok(block.includes('| Out-Host'), 'PowerShell wrapper should route status output via | Out-Host');
+  });
+
   process.stdout.write(`\n  pass=${pass} fail=${fail}\n`);
   return fail === 0 ? 0 : 1;
 }