Skill Shell Gotchas

SKILL Shell & IPC Gotchas

Problem 1: sh() Returns t/nil, Not stdout

Symptom

ipcBeginProcess exits with state=127 (command not found). Variable is set to "t" instead of a file path.

Root Cause

sh() in SKILL returns t (success) or nil (failure) — it does NOT return stdout.

; WRONG — fromPath will be "t", not a path
fromPath = car(parseString(sh("which virtuoso-daemon 2>/dev/null") "\n\r"))
if(fromPath then  ; "t" is truthy → takes this branch with wrong value
    fromPath      ; returns "t", not a real path

; CORRECT — use isFile() to validate; skip which entirely
cargoPath = strcat(getShellEnvVar("HOME") "/.cargo/bin/virtuoso-daemon")
if(isFile(cargoPath) then cargoPath else "")

Fix

Never use sh() to capture command output. Use it only for side effects (mkdir, kill, etc.). For path resolution, use getShellEnvVar("RB_DAEMON_PATH") or check known fixed locations with isFile().

Problem 2: fprintf/outfile Writes 0-Byte Files

Symptom

outfile/fprintf calls succeed (return t) but the file is empty or not created.

Root Cause

SKILL's fprintf has buffering issues in Virtuoso's IPC context — the buffer is not flushed to disk reliably.

Fix

Use sh() with shell's printf to write files:

; WRONG
port = outfile("/tmp/session.json" "w")
fprintf(port "{\"id\":\"%s\",\"port\":%d}" sessionId portNum)

; CORRECT — delegate file writing to shell
sh(sprintf(nil "printf '{\"id\":\"%s\",\"port\":%d}' > \"%s\""
    sessionId portNum filename))

Problem 3: getpid() Is Undefined in SKILL

Symptom

*Error* eval: undefined function - getpid

Root Cause

SKILL has no getpid() function. There is no standard way to get the current process PID.

Fix

If PID is needed for session tracking, store 0 as a placeholder. Use TCP port reachability (TcpStream::connect_timeout) to check liveness instead of PID signals.

Problem 4: boundp Preserves Stale Values Across load()

Symptom

After fixing a bug in a variable's initialization, reloading the .il file doesn't pick up the fix because unless(boundp('Var)) skips re-initialization.

Root Cause

unless(boundp('RBDPath)) only runs when the variable is unbound. If a previous load set it to a bad value (e.g., "t"), it stays bad across reloads.

Fix

Add validity checks to the guard condition:

; WRONG — stale "t" value is preserved
unless(boundp('RBDPath)
    RBDPath = ...
)

; CORRECT — re-resolve if empty, nil, or not a valid file
when(!boundp('RBDPath) || RBDPath == "" || RBDPath == nil || !isFile(RBDPath)
    RBDPath = ...
)

Problem 5: ipcBeginProcess PATH Is Stripped

Symptom

ipcBeginProcess exits with state=127 even though the binary exists on the system.

Root Cause

Virtuoso's ipcBeginProcess launches via a stripped shell environment that does NOT inherit the user's PATH. Binaries in ~/.cargo/bin, /usr/local/bin, etc. may not be found via bare name.

Fix

Always pass the absolute path to the binary. Resolve it at load time:

; Priority resolution (highest to lowest):
; 1. RB_DAEMON_PATH env var (user override)
; 2. Known absolute install locations (isFile check)
RBDPath = let((fromEnv cargoPath)
    fromEnv = getShellEnvVar("RB_DAEMON_PATH")
    if(fromEnv then
        fromEnv
    else
        cargoPath = strcat(getShellEnvVar("HOME") "/.cargo/bin/virtuoso-daemon")
        if(isFile(cargoPath) then cargoPath else "")
    )
)

Problem 6: Multi-Expression SKILL String Without progn Silently Fails

Symptom

A SKILL string with two side-effectful calls (e.g. maeCloseSession(...) printf(...)) appears to succeed but only the second call runs; the first is silently skipped.

Root Cause

Inside a lambda body, mapcar, or certain callback contexts, SKILL parses f1() f2() as "apply the return value of f1() to the arguments of f2()" rather than two sequential calls. This is not an issue at the top level of evalstring, but it is an issue inside any expression context.

Fix

Wrap multiple expressions in progn(...):

; WRONG — close silently skipped; skill_ok() still returns t (printf returns t)
maeCloseSession("sess1" ?forceClose t) printf("done\n")

; CORRECT
progn(
    maeCloseSession("sess1" ?forceClose t)
    printf("done\n")
)

let((...) ...) is a safe alternative — its body accepts multiple forms and evaluates them in sequence:

let((result)
    result = maeCloseSession("sess1" ?forceClose t)
    printf("done\n")
    result
)

Problem 7: sprintf %L Adds an Extra Quote Layer to Paths

Symptom

A path returned through the bridge has extra backslash-quotes: \"path\" instead of path.

Root Cause

sprintf(nil "%L" someStringVar) in SKILL adds SKILL-style escaping to the string, turning "/home/user/run" into "\"/home/user/run\"". When this value is returned through the bridge and passed to output_unquoted(), one layer of quotes is stripped, but the inner \" escapes remain.

Fix

Use %s instead of %L when embedding a string variable in a sprintf result that will be read back in Rust:

; WRONG — produces extra quote layer
path = sprintf(nil "%L" getWorkingDir())

; CORRECT
path = sprintf(nil "%s" getWorkingDir())
; or just
path = getWorkingDir()

If receiving a %L-escaped value, strip both outer quotes and unescape inner \" pairs before using the path.

Problem 8: hiCreateAppForm / hiInsertBannerMenu IC23 Reload Traps

Symptom A — Cannot delete a form that is mapped

Calling hiCreateAppForm with the same ?name while that form is already open (visible on screen) throws an error and the reload fails.

Fix A

Guard the entire form-creation block with a boundp check so it only runs once per session. Separate the form guard from the menu guard so they can be managed independently:

; Form: created once per session
unless(boundp('RBMonInstalled)
    progn(
        hiCreateAppForm(?name 'RBMonitor ...)
        RBMonInstalled = t
    )
)

; Menu: also one-shot, but separate flag so the form guard doesn't conflate them
unless(boundp('RBMenuInstalled)
    progn(
        hiInsertBannerMenu(window(1) myMenu 3)
        RBMenuInstalled = t
    )
)

Symptom B — ?buttonLayout 'Apply causes an error

'Apply is not a valid ?buttonLayout enum value in IC23. The complete valid set is: 'Empty, 'OK, 'Close, 'OKCancel, 'OKCancelApply.

Fix B

Use 'OKCancelApply (or 'Empty + manual hiCreateButton) instead of 'Apply.

Symptom C — hiInsertBannerMenu is one-shot

Calling hiInsertBannerMenu a second time (on bridge reload) inserts a duplicate menu entry and cannot be undone without restarting Virtuoso.

Fix C

Always guard with a separate boundp flag (see Fix A above).

Summary Table