openclacky 1.2.4 → 1.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 80e1b5549b0e55760ec4e0e905eea3ab50bc9a66bd8c256e628a1afdb8531ba0
-  data.tar.gz: 54142ef66c3b74d3998214202fa73618dd9b22ebdb06586c8d6896aafcb56d9d
+  metadata.gz: 234f676c7efe1f2803eeb4f262b20014e2a4b967442a03af70dc3e04e81b4f6b
+  data.tar.gz: 6e9161983ebec82c44467a28e92ff25636718942bec953ebdc7ef6593b515c7e
 SHA512:
-  metadata.gz: d69d02391cc1929d26f2cfc2a260d27cb6b1f6840c254ae67f7ad358f621ba16eb2476fd5cee28308f7eaf2e0eb21d5983b49de4f84c6905863292625644641a
-  data.tar.gz: 54dbeb6d31f187edc30ea4e6030776e988cf50ee7ed44a2177a77473d00c47ae358df25dfd9bd526ea93126b94733feaca5a63048cde8657201c4c1e7cd05edd
+  metadata.gz: afd3471fa15b77e3921aa640fbfd7cf937030478d834c06d0ad2d52faa6ada82561929ce7aa4ac2b075a71b1cf97bd5295ec80af4d2903dbf55c983bb4263e1f
+  data.tar.gz: 7713e3b3a0aa9775a4a993b651a536552963320787f072ea6f79dbde0bf345a1d680b3cc0a07eec1d5667da69899f770344ba376b52d44d12360ac3ec60402a4

data/CHANGELOG.md

@@ -5,6 +5,15 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.5] - 2026-05-28
+
+### Fixed
+- Bash tool stream output not rendering in real-time
+- Multi-line shell output being incorrectly stripped
+
+### More
+- Remove WSL2 network doctor script for better compatibility
+
 ## [1.2.4] - 2026-05-27
 
 ### Added

data/lib/clacky/default_skills/browser-setup/SKILL.md

@@ -111,96 +111,6 @@ chrome-devtools-mcp --version 2>/dev/null
 
 If still missing after user confirms, stop with error message.
 
-### Step 2.5 — WSL networking setup (only when session context shows `OS: WSL/Windows`)
-
-**Skip this entire step on macOS / Linux.** Look at the session context line that begins with `[Session context: ...]` — only run this step if it includes `OS: WSL/Windows`.
-
-#### Background (read this so you know what to do)
-
-The browser tool runs inside WSL but Chrome/Edge runs on Windows. By default WSL2 uses NAT networking, which means `127.0.0.1` inside WSL **cannot** reach Windows' Chrome debug port. The fix is to enable WSL2 **mirrored networking** (`networkingMode=mirrored` in `%USERPROFILE%\.wslconfig`), which makes WSL share Windows' network stack so `127.0.0.1` works directly.
-
-We have a helper script that handles all the Windows-side details:
-
-```
-~/.clacky/scripts/wsl_network_doctor.ps1
-```
-
-It exposes three subcommands:
-
-| Subcommand | What it does | Exit code |
-|---|---|---|
-| `status` | Check whether mirrored is configured (auto-passes on WSL1) | `0` OK / `10` NEED_ENABLE |
-| `enable` | Write `networkingMode=mirrored` to `.wslconfig` (does NOT shut down WSL) | `0` success / `1` fail |
-| `repair` | Restart Windows Host Network Service (HNS) via UAC prompt | `0` launched / `1` fail |
-
-Invoke it from WSL like this:
-
-```bash
-powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$(wslpath -w ~/.clacky/scripts/wsl_network_doctor.ps1)" <subcommand>
-```
-
-#### Step 2.5.1 — Check status
-
-```bash
-powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$(wslpath -w ~/.clacky/scripts/wsl_network_doctor.ps1)" status
-```
-
-- Exit `0` (output starts with `OK:`) → either mirrored is configured (WSL2) or
-  Ubuntu is running on WSL1 (which shares the Windows network stack and needs no
-  config). Either way, proceed to Step 3.
-- Exit `10` (output starts with `NEED_ENABLE:`) → continue to Step 2.5.2.
-- Any other failure → show the output to the user and ask them to retry. Stop here.
-
-#### Step 2.5.2 — Enable mirrored (only when NEED_ENABLE)
-
-Tell the user what's about to happen (in their language):
-
-> WSL doesn't have mirrored networking enabled yet — the browser tool needs it to reach Chrome on Windows.
-> I'll add one line to `%USERPROFILE%\.wslconfig`. Your current WSL session will NOT be restarted.
-
-Run:
-
-```bash
-powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$(wslpath -w ~/.clacky/scripts/wsl_network_doctor.ps1)" enable
-```
-
-If the script exits `0`:
-
-> ✅ `.wslconfig` updated. Tell the user (in their language):
->
-> The config takes effect only after WSL restarts, but we can't restart WSL from inside WSL.
-> Please:
->
-> 1. Open **PowerShell** on Windows
-> 2. Run: `wsl --shutdown`
-> 3. Reopen the Clacky terminal
-> 4. Run `/browser-setup` again
->
-> Stop here. Wait for the user to come back in a new session.
-
-If the script exits non-zero, show the output to the user and stop. Do NOT proceed to Step 3 — without mirrored networking the browser tool will not work.
-
-#### Step 2.5.3 — When to run repair
-
-Do NOT run `repair` proactively. Only run it later if **all** of the following are true:
-
-- `status` returned `OK` (mirrored is configured)
-- The user has restarted WSL since the config was written
-- Step 3's `browser(action="status")` still fails with a "Chrome/Edge is not running or remote debugging is not enabled" error
-
-In that situation, tell the user (in their language):
-
-> The config looks correct but the browser still can't connect. Windows Host Network Service may be stuck — I'll restart it.
-> **A Windows User Account Control (UAC) prompt will appear shortly. Please click "Yes".**
-
-Then run:
-
-```bash
-powershell.exe -NoProfile -ExecutionPolicy Bypass -File "$(wslpath -w ~/.clacky/scripts/wsl_network_doctor.ps1)" repair
-```
-
-After it returns, tell the user to run `wsl --shutdown` in PowerShell and reopen Clacky. Stop and wait.
-
 ### Step 3 — Verify Chrome/Edge is running with remote debugging
 
 **CRITICAL**: Do NOT attempt `browser()` calls yet. First check if the browser is reachable using the API:
@@ -278,6 +188,12 @@ If still failing:
 >
 > Run `/browser-setup doctor` to diagnose the issue in detail.
 
+Fetch the online troubleshooting guide for additional help:
+
+```
+web_fetch(url="https://www.openclacky.com/docs/browser-troubleshooting")
+```
+
 Stop here and suggest running doctor.
 
 #### Case B: Other errors (MCP handshake timeout, daemon crash, etc.)
@@ -514,3 +430,13 @@ After fixing these issues, run `/browser-setup` again to verify.
 
 The browser tool is ready to use.
 ```
+
+### Online Troubleshooting Guide
+
+If any ❌ items are found, or the user reports a problem you cannot resolve with the above scenarios, fetch the latest troubleshooting guide:
+
+```
+web_fetch(url="https://www.openclacky.com/docs/browser-troubleshooting")
+```
+
+Use the content from that page to provide up-to-date diagnosis and resolution steps. This online document is maintained separately and may contain newer solutions not listed here.

data/lib/clacky/tools/terminal.rb

@@ -1166,20 +1166,26 @@ module Clacky
           # the literal printf tail `"$__clacky_ec"`. Until we see that, we
           # accumulate; once we do, we strip the whole wrapper out and only
           # emit whatever real output came after it.
+          #
+          # However, when stty -echo is active (the normal case for our
+          # persistent sessions), the wrapper is never echoed — so the tail
+          # marker never appears. We detect this by checking: if we have a
+          # complete line (\n present) and it does NOT contain the wrapper
+          # fingerprint, echo was suppressed and we can start streaming
+          # immediately.
           unless wrapper_swallowed
             tail_marker = '"$__clacky_ec"'
             tail_idx = stream_pending.index(tail_marker)
             if tail_idx
-              # Strip from start through end-of-line of the printf tail.
               eol_after = stream_pending.index("\n", tail_idx) || (stream_pending.bytesize - 1)
               stream_pending.replace(stream_pending.byteslice(eol_after + 1, stream_pending.bytesize - eol_after - 1).to_s)
               wrapper_swallowed = true
             elsif force_partial
-              # End of stream and we never saw the wrapper tail — give up
-              # on swallowing and emit what we have, run normal stripping.
+              wrapper_swallowed = true
+            elsif stream_pending.include?("\n") && !stream_pending.include?("__clacky_ec")
+              # stty -echo suppressed the wrapper echo; real output is arriving.
               wrapper_swallowed = true
             else
-              # Still hunting; keep buffering. Emit nothing yet.
               return
             end
           end
@@ -1206,7 +1212,12 @@ module Clacky
               ln.include?("__clacky_pc") ||
               ln.match?(/\A\s*\}\s*>\s*\/dev\/null\s+2>&1;?\s*\z/)
           end.join
-          on_output.call(cleaned) unless cleaned.empty?
+          # Collapse runs of 3+ blank lines into a single blank line so
+          # PTY noise (cursor-positioning codes cleaned to empty lines)
+          # doesn't produce a wall of whitespace in the streaming UI.
+          cleaned = cleaned.gsub(/\n{3,}/, "\n\n")
+          cleaned = cleaned.lstrip if cleaned.match?(/\A\n+\z/)
+          on_output.call(cleaned) unless cleaned.empty? || cleaned.match?(/\A\s*\z/)
         rescue StandardError
           # Streaming is best-effort — never let a UI bug abort the command.
         end

data/lib/clacky/utils/scripts_manager.rb

@@ -21,7 +21,6 @@ module Clacky
         install_browser.sh
         install_system_deps.sh
         install_rails_deps.sh
-        wsl_network_doctor.ps1
       ].freeze
 
       # Copy bundled scripts to ~/.clacky/scripts/ if missing or outdated.

data/lib/clacky/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Clacky
-  VERSION = "1.2.4"
+  VERSION = "1.2.5"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openclacky
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.5
 platform: ruby
 authors:
 - windy
@@ -596,7 +596,6 @@ files:
 - scripts/install_rails_deps.sh
 - scripts/install_system_deps.sh
 - scripts/uninstall.sh
-- scripts/wsl_network_doctor.ps1
 - sig/clacky.rbs
 homepage: https://github.com/clacky-ai/openclacky
 licenses:

data/scripts/wsl_network_doctor.ps1

@@ -1,196 +0,0 @@
-# wsl_network_doctor.ps1 — diagnose & repair WSL2 mirrored networking for the browser tool.
-#
-# Designed to be invoked from inside WSL via:
-#   powershell.exe -NoProfile -ExecutionPolicy Bypass -File <win-path-to-this-script> <subcommand>
-#
-# Subcommands:
-#   status   Check whether mirrored networking is configured.
-#   enable   Write networkingMode=mirrored to %USERPROFILE%\.wslconfig.
-#   repair   Restart Windows Host Network Service (HNS) via UAC elevation.
-#
-# Exit codes (status only):
-#   0   OK              — mirrored configured, OR running on WSL1 (no config needed)
-#   10  NEED_ENABLE     — mirrored not configured, run `enable`
-#   20  NEED_REPAIR     — configured but suspected broken, run `repair`
-#   1   unexpected error
-#
-# `enable` and `repair` exit 0 on success, 1 on failure.
-
-param(
-    [Parameter(Position = 0)]
-    [ValidateSet('status', 'enable', 'repair')]
-    [string]$Command
-)
-
-$ErrorActionPreference = 'Stop'
-
-# ---------------------------------------------------------------------------
-# Helpers
-# ---------------------------------------------------------------------------
-
-function Get-WslConfigPath {
-    return (Join-Path $env:USERPROFILE '.wslconfig')
-}
-
-function Test-MirroredConfigured {
-    $cfg = Get-WslConfigPath
-    if (-not (Test-Path $cfg)) { return $false }
-    $content = Get-Content $cfg -Raw -ErrorAction SilentlyContinue
-    if ($null -eq $content) { return $false }
-    return ($content -match '(?im)^\s*networkingMode\s*=\s*mirrored\s*$')
-}
-
-# Returns 1 or 2 if Ubuntu is registered, $null otherwise.
-# Parses `wsl.exe -l -v` output (UTF-16, may contain a star marker on default distro).
-function Get-UbuntuWslVersion {
-    try {
-        $raw = & wsl.exe -l -v 2>$null
-    } catch {
-        return $null
-    }
-    if (-not $raw) { return $null }
-
-    foreach ($line in $raw) {
-        $clean = ($line -replace '\s+', ' ').Trim().TrimStart('*').Trim()
-        if ($clean -match '^Ubuntu(?:-[\w\.]+)?\s+\S+\s+(\d+)\s*$') {
-            return [int]$matches[1]
-        }
-    }
-    return $null
-}
-
-# ---------------------------------------------------------------------------
-# Subcommand: status
-# ---------------------------------------------------------------------------
-
-function Invoke-Status {
-    $wslVer = Get-UbuntuWslVersion
-    if ($wslVer -eq 1) {
-        Write-Host "OK: Ubuntu is running on WSL1 — shares the Windows network stack directly."
-        Write-Host "No mirrored configuration needed. The browser tool can connect to 127.0.0.1 as-is."
-        exit 0
-    }
-
-    if (Test-MirroredConfigured) {
-        Write-Host "OK: mirrored networking is configured in .wslconfig."
-        Write-Host "If the browser tool still cannot connect, run: wsl_network_doctor.ps1 repair"
-        exit 0
-    }
-
-    Write-Host "NEED_ENABLE: mirrored networking is not configured."
-    Write-Host "Run: wsl_network_doctor.ps1 enable"
-    exit 10
-}
-
-# ---------------------------------------------------------------------------
-# Subcommand: enable
-# ---------------------------------------------------------------------------
-
-function Invoke-Enable {
-    if (Test-MirroredConfigured) {
-        Write-Host "OK: already enabled. No changes needed."
-        Write-Host "If the browser tool still cannot connect, run: wsl_network_doctor.ps1 repair"
-        exit 0
-    }
-
-    $cfg = Get-WslConfigPath
-    Write-Host "Writing networkingMode=mirrored to $cfg ..."
-
-    if (-not (Test-Path $cfg)) {
-        New-Item -ItemType File -Path $cfg -Force | Out-Null
-    }
-
-    $content = Get-Content $cfg -Raw -ErrorAction SilentlyContinue
-    if ($null -eq $content) { $content = '' }
-
-    if ($content -match '(?im)^\s*networkingMode\s*=') {
-        $new = [regex]::Replace($content, '(?im)^\s*networkingMode\s*=.*$', 'networkingMode=mirrored')
-        Set-Content -Path $cfg -Value $new -NoNewline
-    } else {
-        if ($content -notmatch '(?im)^\[wsl2\]') {
-            if ($content.Length -gt 0 -and -not $content.EndsWith([char]10)) {
-                Add-Content -Path $cfg -Value ''
-            }
-            Add-Content -Path $cfg -Value '[wsl2]'
-        }
-        Add-Content -Path $cfg -Value 'networkingMode=mirrored'
-    }
-
-    Write-Host "WROTE: .wslconfig updated."
-    Write-Host ""
-    Write-Host "Next step (cannot be done from inside WSL):"
-    Write-Host "  1. Open Windows PowerShell"
-    Write-Host "  2. Run: wsl --shutdown"
-    Write-Host "  3. Reopen Clacky and run /browser-setup again"
-    exit 0
-}
-
-# ---------------------------------------------------------------------------
-# Subcommand: repair
-# ---------------------------------------------------------------------------
-# Restart Windows Host Network Service (HNS). Requires admin → triggers UAC.
-# Does NOT call `wsl --shutdown` here — the user must run it manually after
-# the elevated window finishes, otherwise our own WSL session would be killed.
-
-function Invoke-Repair {
-    Write-Host "Repairing Windows Host Network Service (HNS) ..."
-    Write-Host ""
-    Write-Host "A Windows User Account Control (UAC) dialog will appear."
-    Write-Host "Please click 'Yes' to allow the repair script to run."
-    Write-Host ""
-
-    $inner = @'
-try {
-    Stop-Service hns -Force -ErrorAction SilentlyContinue
-    Start-Service hns -ErrorAction Stop
-    Write-Host "HNS restarted successfully."
-} catch {
-    Write-Host "Repair failed: $_"
-    Start-Sleep 5
-    exit 1
-}
-Write-Host ""
-Write-Host "Repair complete. Please run 'wsl --shutdown' in PowerShell, then reopen Clacky."
-Start-Sleep 4
-'@
-
-    $encoded = [Convert]::ToBase64String([Text.Encoding]::Unicode.GetBytes($inner))
-
-    try {
-        Start-Process powershell -Verb RunAs -ArgumentList '-NoProfile', '-EncodedCommand', $encoded
-    } catch {
-        Write-Host "FAILED: could not trigger UAC prompt: $_"
-        Write-Host ""
-        Write-Host "You can run the repair manually:"
-        Write-Host "  1. Open PowerShell as Administrator"
-        Write-Host "  2. Run: net stop hns; net start hns"
-        Write-Host "  3. Run: wsl --shutdown"
-        Write-Host "  4. Reopen Clacky"
-        exit 1
-    }
-
-    Write-Host "Repair script launched in an elevated PowerShell window."
-    Write-Host ""
-    Write-Host "After the elevated window finishes:"
-    Write-Host "  1. Run in regular PowerShell: wsl --shutdown"
-    Write-Host "  2. Reopen Clacky and run /browser-setup again"
-    exit 0
-}
-
-# ---------------------------------------------------------------------------
-# Dispatch
-# ---------------------------------------------------------------------------
-
-switch ($Command) {
-    'status' { Invoke-Status }
-    'enable' { Invoke-Enable }
-    'repair' { Invoke-Repair }
-    default {
-        Write-Host "Usage: wsl_network_doctor.ps1 {status|enable|repair}"
-        Write-Host ""
-        Write-Host "  status  Check whether WSL2 mirrored networking is configured."
-        Write-Host "  enable  Write networkingMode=mirrored to %USERPROFILE%\.wslconfig."
-        Write-Host "  repair  Restart Windows Host Network Service (HNS) via UAC."
-        exit 2
-    }
-}