fix: port to current Perry toolchain (macOS)

The source targeted an older custom Perry fork; upstream @perryts/perry
(0.5.112x+) removed its bespoke helpers and tightened FFI/runtime
semantics, so the macOS build crashed on launch. Port off the
removed/changed surfaces (rebased onto the Windows-port work on main):

- fs.isDirectory removed -> src/fs-compat.ts (statSync().isDirectory())
- child_process.spawnBackground removed -> src/process-compat.ts; spawn
  with { detached:true }, platform-neutral (caller supplies the shell; no
  re-wrap), logFile redirected via fd, /dev/null|NUL discards
- execSync/spawnSync now return Buffers -> execText/spawnText wrappers in
  process-compat force { encoding:'utf8' }; all call sites renamed
- cross-device sync crypto intrinsics removed -> stubbed inline in
  sync-host/guest (sync disabled, deltas pass through)
- LSP FFI manifest: string params i64 -> "string"
- render.ts: define dropped t3/t5 timing vars; import find-bar match
  getters (unbound identifier reads now throw ReferenceError)

IDE launches: workbench, explorer, editor, tree-sitter highlighting.
Build requires PERRY_ALLOW_PERRY_FEATURES=1. See PERRY-MIGRATION-NOTES.md.

Author: proggeramlug

PERRY-MIGRATION-NOTES.md

@@ -0,0 +1,44 @@
+# Perry toolchain migration (macOS)
+
+Hone was originally built against a custom Perry fork. Upstream `@perryts/perry`
+(0.5.112x+) removed that fork's bespoke helpers and tightened FFI/runtime
+semantics, so the source needed porting. Summary of what changed and why, for
+anyone touching these areas (incl. the Windows port).
+
+## Compat shims (new files)
+- `src/fs-compat.ts` — `isDirectory()` (fork-only `fs` export, removed) reimplemented
+  via `statSync().isDirectory()`.
+- `src/process-compat.ts` — three shims:
+  - `execText` / `spawnText` — `execSync`/`spawnSync` now return a **Buffer** unless
+    `{ encoding: 'utf8' }` is passed (Node semantics). The codebase treats results as
+    strings everywhere, so all call sites route through these wrappers.
+  - `spawnBackground` — fork-only `child_process` helper, removed. Reimplemented over
+    `spawn` with `{ detached: true }`. **Platform-neutral:** the caller supplies the
+    shell (`shellBin`/`shellArg`), so the shim does NOT re-wrap; a real `logFile` is
+    redirected via an opened fd, `/dev/null`/`NUL`/empty discards.
+
+All `execSync(`/`spawnSync(` call sites were renamed to `execText(`/`spawnText(`;
+`isDirectory`/`spawnBackground` imports repointed to the shims. Native
+`execSync`/`spawnSync` are imported only inside `process-compat.ts`.
+
+## FFI manifest types (native libs)
+Upstream Perry strictly checks `i64` params and rejects NaN-boxed strings (the old
+fork extracted the pointer leniently). String params must be typed `"string"`, and
+string-pointer returns (`Rust -> i64`) typed `"i64_str"`. Fixed in
+`native/lsp/package.json` here, and in the `hone-editor` / `hone-terminal` repos.
+Find them by scanning Rust for `*const u8` params (incl. `pub unsafe extern`).
+
+## Other
+- Cross-device sync used custom crypto intrinsics (x25519/aes-gcm/hkdf), now gone —
+  **stubbed inline** in `sync-host.ts`/`sync-guest.ts`; sync is disabled (deltas pass
+  through as plaintext). Restore by reimplementing over Node `crypto`.
+- Unbound identifier reads now throw `ReferenceError` (old Perry read them as
+  `undefined`). Run `npx tsc --noEmit | grep 'TS2304\|TS2552'` after any port to catch
+  the whole class. (Fixed: dropped `t3`/`t5` timing vars + a missing find-bar import
+  in render.ts.)
+
+## Building
+Requires `PERRY_ALLOW_PERRY_FEATURES=1` (native-library allowlist gate), or add
+`"perry": { "allow": { "nativeLibrary": ["*"] } }` to package.json. Debug a runtime
+crash with `--debug-symbols` + `lldb -o "break set -n exit" -o run -o bt` — Perry's
+throw handler prints then exits without unwinding, so the crashing stack is intact.

native/lsp/package.json

@@ -13,8 +13,8 @@
       "abiVersion": "0.5",
       "module": "@honeide/lsp-bridge",
       "functions": [
-        { "name": "hone_lsp_start", "params": ["i64", "i64", "i64"], "returns": "f64" },
-        { "name": "hone_lsp_send", "params": ["f64", "i64"], "returns": "f64" },
+        { "name": "hone_lsp_start", "params": ["string", "string", "string"], "returns": "f64" },
+        { "name": "hone_lsp_send", "params": ["f64", "string"], "returns": "f64" },
         { "name": "hone_lsp_poll", "params": ["f64"], "returns": "i64" },
         { "name": "hone_lsp_is_alive", "params": ["f64"], "returns": "f64" },
         { "name": "hone_lsp_stop", "params": ["f64"], "returns": "f64" }

src/fs-compat.ts

@@ -0,0 +1,13 @@
+import { statSync } from 'fs';
+
+// Perry's `fs` module dropped its top-level `isDirectory(path)` helper
+// (upstream @perryts/perry). Shim it via statSync so existing call sites
+// keep their "false for missing/non-dir" semantics.
+export function isDirectory(path: string): boolean {
+  try {
+    const st = statSync(path);
+    return st.isDirectory();
+  } catch (e: any) {
+    return false;
+  }
+}

src/process-compat.ts

@@ -0,0 +1,58 @@
+import { spawn, execSync, spawnSync } from 'child_process';
+import { openSync } from 'fs';
+
+// Compatibility shims for upstream @perryts/perry, which diverged from the
+// custom fork hone was originally built against:
+//
+//  - execSync/spawnSync now follow Node semantics and return a Buffer unless
+//    `{ encoding: 'utf8' }` is passed. The codebase treats the result as a
+//    string everywhere, so route every call through these wrappers.
+//  - child_process.spawnBackground (a fork-only helper) was removed; reimplement
+//    it over the standard `spawn` with { detached: true }.
+
+export function execText(cmd: string): string {
+  return execSync(cmd, { encoding: 'utf8' }) as unknown as string;
+}
+
+export function spawnText(cmd: string, args: string[]): { stdout: string; stderr: string; status: number } {
+  return spawnSync(cmd, args, { encoding: 'utf8' }) as unknown as { stdout: string; stderr: string; status: number };
+}
+
+// Fire-and-forget background spawn. The CALLER supplies the platform shell
+// (e.g. cmd.exe /c vs /bin/sh -c), so we do NOT re-wrap in a shell here —
+// `command`/`args` are spawned directly, which keeps this cross-platform.
+//
+// The third argument matches the old signature: a string (output-redirect
+// target — '/dev/null' or 'NUL' mean discard) or an options object
+// { cwd?, logFile? }. When a real logFile is given, output is redirected to it
+// via an opened fd (no shell needed); otherwise output is discarded.
+export function spawnBackground(command: string, args: string[], options?: string | object): { pid: number; handleId: number } {
+  let logFile = '';
+  let cwd = '';
+  if (typeof options === 'string') {
+    const s = options as string;
+    if (s.length > 0 && s !== '/dev/null' && s !== 'NUL') logFile = s;
+  } else if (options !== null && options !== undefined && typeof options === 'object') {
+    const o: any = options;
+    if (o.logFile !== undefined && o.logFile !== null) logFile = o.logFile;
+    if (o.cwd !== undefined && o.cwd !== null) cwd = o.cwd;
+  }
+
+  let stdio: any = 'ignore';
+  if (logFile.length > 0) {
+    const fd: any = openSync(logFile, 'a');
+    stdio = ['ignore', fd, fd];
+  }
+
+  const spawnOpts: any = { detached: true, stdio: stdio };
+  if (cwd.length > 0) spawnOpts.cwd = cwd;
+
+  const child: any = spawn(command, args, spawnOpts);
+  let pid = 0;
+  if (child !== null && child !== undefined && child.pid !== undefined && child.pid !== null) {
+    pid = child.pid;
+  }
+  try { child.unref(); } catch (e: any) {}
+
+  return { pid: pid, handleId: 0 };
+}

src/workbench/paths.ts

@@ -9,7 +9,7 @@
  */
 
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'fs';
-import { execSync } from 'child_process';
+import { execText } from '../process-compat';
 
 declare const __platform__: number;
 
@@ -91,7 +91,7 @@ export function getHomeDir(): string {
       if (envHome && envHome.length > 0) { _homeDir = envHome; return _homeDir; }
     } catch (_e: any) {}
     try {
-      const result = execSync('/bin/echo $HOME') as unknown as string;
+      const result = execText('/bin/echo $HOME') as unknown as string;
       const dir = trimNewline(result);
       if (dir.length > 0) { _homeDir = dir; return _homeDir; }
     } catch (e: any) {}
@@ -105,7 +105,7 @@ export function getHomeDir(): string {
       if (envUP && envUP.length > 0) { _homeDir = envUP; return _homeDir; }
     } catch (_e: any) {}
     try {
-      const result = execSync('echo %USERPROFILE%') as unknown as string;
+      const result = execText('echo %USERPROFILE%') as unknown as string;
       const dir = trimNewline(result);
       if (dir.length > 0) { _homeDir = dir; return _homeDir; }
     } catch (e: any) {}
@@ -267,13 +267,13 @@ export function getCwd(): string {
   // that prints the working directory when called with no args).
   if (__platform__ === 3) {
     try {
-      const result = execSync('cd') as unknown as string;
+      const result = execText('cd') as unknown as string;
       const dir = trimNewline(result);
       if (dir.length > 0) return dir;
     } catch (e: any) {}
   } else {
     try {
-      const result = execSync('pwd') as unknown as string;
+      const result = execText('pwd') as unknown as string;
       const dir = trimNewline(result);
       if (dir.length > 0) return dir;
     } catch (e: any) {}

src/workbench/render.ts

@@ -53,10 +53,10 @@ import {
 } from './theme/theme-colors';
 import type { LayoutMode } from '../platform';
 import { getWorkbenchSettings, updateSettings, onSettingsChange, getSettingsVersion, getLastOpenTabs, getLastActiveTab, getLastPinnedTabs, applyWorkspaceOverlay, setNumberSetting } from './settings';
-import { readFileSync, writeFileSync, readdirSync, isDirectory, existsSync, unlinkSync } from 'fs';
+import { readFileSync, writeFileSync, readdirSync, existsSync, unlinkSync } from 'fs';
 import { join } from 'path';
-import { spawnBackground } from 'child_process';
-import { execSync, spawnSync } from 'child_process';
+import { execText, spawnText, spawnBackground } from '../process-compat';
+import { isDirectory } from '../fs-compat';
 import { spawn } from 'perry/thread';
 import { getTempDir, getCwd, getHomeDir, getAppDataDir } from './paths';
 import { getPlatformContext, isWebPlatform } from '../platform';
@@ -151,7 +151,7 @@ import { initWorkspaceTrust, isWorkspaceTrusted, trustWorkspace, revokeWorkspace
 import { renderReferencesPeek, showReferencesFromJson, setReferencesJumpHandler } from './views/references-peek/references-peek';
 import { renderTasksPanel, runDefaultBuildTask, runTaskByLabel, setTasksWorkspaceRoot, setTasksAppDataDir, setOnTaskRunStart, setOnTaskRunDone } from './views/tasks/tasks-panel';
 import { initRecentItems, addRecentFile, addRecentFolder, getRecentPath, getRecentType } from './views/recent/recent-store';
-import { createFindBar, setFindEditorCallbacks, showFindBar, showFindBarWithReplace, hideFindBar, isFindBarVisible } from './views/find/find-bar';
+import { createFindBar, setFindEditorCallbacks, showFindBar, showFindBarWithReplace, hideFindBar, isFindBarVisible, getFindMatchCount, getFindCurrentMatch, getFindMatchLine, getFindMatchCol, getFindMatchLen } from './views/find/find-bar';
 import { setLspWorkspaceRoot, initLspBridge, triggerDiagnostics, getCompletions, setDiagnosticsStatusUpdater } from './views/lsp/lsp-bridge';
 import { setDiagnosticsFileOpener } from './views/lsp/diagnostics-panel';
 import { createAutocompletePopup, setAutocompleteAcceptHandler } from './views/lsp/autocomplete-popup';
@@ -489,7 +489,7 @@ function toggleSidebarLocation(): void {
 function copyBranchNameToClipboard(): void {
   if (workspaceRoot.length === 0) return;
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'HEAD']);
+    const r = spawnText('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'HEAD']);
     if (r.status !== 0) return;
     let s = r.stdout;
     let end = s.length;
@@ -1839,7 +1839,7 @@ function promptCloseDirtyTabMacos(filePath: string): number {
   script += '  return "2"\n';
   script += 'end try\n';
   try {
-    const r = spawnSync('osascript', ['-e', script]);
+    const r = spawnText('osascript', ['-e', script]);
     if (r.status !== 0) return 2;
     const out = r.stdout.length > 0 ? r.stdout.charAt(0) : '';
     if (out === '0') return 0;
@@ -1859,7 +1859,7 @@ function promptCloseDirtyTabWindows(filePath: string): number {
   ps += '\'Save changes to ' + name + '?\',\'Hone\',\'YesNoCancel\',\'Question\');';
   ps += 'if ($r -eq \'Yes\') { [Console]::Out.Write(\'0\') } elseif ($r -eq \'No\') { [Console]::Out.Write(\'1\') } else { [Console]::Out.Write(\'2\') }';
   try {
-    const r = spawnSync('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', ps]);
+    const r = spawnText('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', ps]);
     if (r.status !== 0) return 2;
     const out = r.stdout.length > 0 ? r.stdout.charAt(0) : '';
     if (out === '0') return 0;
@@ -2055,7 +2055,7 @@ function promptForRenameMacos(): string {
   script += '  return ""\n';
   script += 'end try\n';
   try {
-    const r = spawnSync('osascript', ['-e', script]);
+    const r = spawnText('osascript', ['-e', script]);
     if (r.status !== 0) return '';
     let out = r.stdout;
     let end = out.length;
@@ -2074,7 +2074,7 @@ function promptForRenameWindows(): string {
   ps = '[void][System.Reflection.Assembly]::LoadWithPartialName(\'Microsoft.VisualBasic\');' + ps;
   ps += '[Console]::Out.Write($r)';
   try {
-    const r = spawnSync('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', ps]);
+    const r = spawnText('powershell.exe', ['-NoProfile', '-NonInteractive', '-Command', ps]);
     if (r.status !== 0) return '';
     let out = r.stdout;
     let end = out.length;
@@ -2421,12 +2421,12 @@ function onGenerateCommitMessageImpl(): void {
   // Prefer staged diff; fall back to working tree if nothing is staged.
   let diff = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--cached', '--no-color', '--stat=80,80']);
+    const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--cached', '--no-color', '--stat=80,80']);
     if (r.status === 0) diff = r.stdout;
   } catch (_e: any) {}
   if (diff.length < 5) {
     try {
-      const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--no-color', '--stat=80,80']);
+      const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--no-color', '--stat=80,80']);
       if (r.status === 0) diff = r.stdout;
     } catch (_e: any) {}
   }
@@ -2437,12 +2437,12 @@ function onGenerateCommitMessageImpl(): void {
   // Include the patch body too, capped at ~6KB so we don't blow the context.
   let body = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--cached', '--no-color', '-U2']);
+    const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--cached', '--no-color', '-U2']);
     if (r.status === 0) body = r.stdout;
   } catch (_e: any) {}
   if (body.length < 5) {
     try {
-      const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--no-color', '-U2']);
+      const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--no-color', '-U2']);
       if (r.status === 0) body = r.stdout;
     } catch (_e: any) {}
   }
@@ -2478,18 +2478,18 @@ function onGeneratePRDescriptionImpl(): void {
   // Resolve the base ref.
   let baseRef = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'rev-parse', '--verify', 'main']);
+    const r = spawnText('git', ['-C', workspaceRoot, 'rev-parse', '--verify', 'main']);
     if (r.status === 0) baseRef = 'main';
   } catch (_e: any) {}
   if (baseRef.length === 0) {
     try {
-      const r = spawnSync('git', ['-C', workspaceRoot, 'rev-parse', '--verify', 'master']);
+      const r = spawnText('git', ['-C', workspaceRoot, 'rev-parse', '--verify', 'master']);
       if (r.status === 0) baseRef = 'master';
     } catch (_e: any) {}
   }
   if (baseRef.length === 0) {
     try {
-      const r = spawnSync('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'origin/HEAD']);
+      const r = spawnText('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'origin/HEAD']);
       if (r.status === 0) {
         // Output is e.g. "origin/main\n" — slice off the "origin/" prefix.
         let s = r.stdout;
@@ -2508,7 +2508,7 @@ function onGeneratePRDescriptionImpl(): void {
   // Current branch — used in the prompt header.
   let branch = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'HEAD']);
+    const r = spawnText('git', ['-C', workspaceRoot, 'rev-parse', '--abbrev-ref', 'HEAD']);
     if (r.status === 0) {
       branch = r.stdout;
       let end = branch.length;
@@ -2533,7 +2533,7 @@ function onGeneratePRDescriptionImpl(): void {
 
   let log = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'log', '--no-color', '--pretty=%h %s', range]);
+    const r = spawnText('git', ['-C', workspaceRoot, 'log', '--no-color', '--pretty=%h %s', range]);
     if (r.status === 0) log = r.stdout;
   } catch (_e: any) {}
   if (log.length < 1) {
@@ -2543,13 +2543,13 @@ function onGeneratePRDescriptionImpl(): void {
 
   let stat = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--no-color', '--stat=80,80', range]);
+    const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--no-color', '--stat=80,80', range]);
     if (r.status === 0) stat = r.stdout;
   } catch (_e: any) {}
 
   let body = '';
   try {
-    const r = spawnSync('git', ['-C', workspaceRoot, 'diff', '--no-color', '-U2', range]);
+    const r = spawnText('git', ['-C', workspaceRoot, 'diff', '--no-color', '-U2', range]);
     if (r.status === 0) body = r.stdout;
   } catch (_e: any) {}
   if (body.length > 8000) body = body.slice(0, 8000) + '\n[…truncated]';
@@ -3089,11 +3089,13 @@ function displayFileContent(filePath: string): void {
   const t1 = Date.now();
   updateBreadcrumb();
   const t2 = Date.now();
+  const t3 = Date.now();
   updateStatusBarLanguageImpl(filePath);
   const t4 = Date.now();
   if (editorReady < 1) return;
   const lang = detectLanguage(filePath);
   editorInstance.setLanguage(lang);
+  const t5 = Date.now();
   const content = safeReadFile(filePath);
   // Strip a leading UTF-8 BOM (U+FEFF) before it reaches the editor. Windows
   // tools (Notepad, older Visual Studio, PowerShell `Out-File`/`>` default)
@@ -3909,7 +3911,7 @@ function syncInlineBlame(): void {
     const range = String(lineNum) + ',' + String(lineNum);
     let output = '';
     try {
-      const r = spawnSync('git', ['blame', '-L', range, '--porcelain', '--', filePath]);
+      const r = spawnText('git', ['blame', '-L', range, '--porcelain', '--', filePath]);
       if (r.status === 0) output = r.stdout;
     } catch (e) {
       return '';
@@ -5740,7 +5742,7 @@ function onClaudeRelayRequestFromGuest(guestId: string, prompt: string, wsRoot:
   let claudeBin = '';
   try {
     const cmd = __platform__ === 3 ? 'where' : 'which';
-    const r = spawnSync(cmd, ['claude']);
+    const r = spawnText(cmd, ['claude']);
     if (r.status === 0 && r.stdout.length > 0) {
       for (let i = 0; i < r.stdout.length; i++) {
         const ch = r.stdout.charCodeAt(i);
@@ -5768,9 +5770,9 @@ function onClaudeRelayRequestFromGuest(guestId: string, prompt: string, wsRoot:
     try {
       const pidStr = String(claudeRelayPid);
       if (__platform__ === 3) {
-        spawnSync('taskkill', ['/F', '/PID', pidStr]);
+        spawnText('taskkill', ['/F', '/PID', pidStr]);
       } else {
-        spawnSync('kill', [pidStr]);
+        spawnText('kill', [pidStr]);
       }
     } catch (e) {}
   }
@@ -5909,11 +5911,11 @@ function claudeRelayPollTick(): void {
       try {
         const pidStr = String(claudeRelayPid);
         if (__platform__ === 3) {
-          const r = spawnSync('tasklist', ['/FI', 'PID eq ' + pidStr, '/NH']);
+          const r = spawnText('tasklist', ['/FI', 'PID eq ' + pidStr, '/NH']);
           if (r.status !== 0) gone = 1;
           else if (r.stdout.length >= 5 && r.stdout.charCodeAt(0) === 73 && r.stdout.charCodeAt(1) === 78) gone = 1;
         } else {
-          const r = spawnSync('kill', ['-0', pidStr]);
+          const r = spawnText('kill', ['-0', pidStr]);
           if (r.status !== 0) gone = 1;
         }
       } catch (e) {
@@ -6148,9 +6150,9 @@ function onClaudeRelayStopFromGuest(guestId: string, sessionId: string): void {
     try {
       const pidStr = String(claudeRelayPid);
       if (__platform__ === 3) {
-        spawnSync('taskkill', ['/F', '/PID', pidStr]);
+        spawnText('taskkill', ['/F', '/PID', pidStr]);
       } else {
-        spawnSync('kill', [pidStr]);
+        spawnText('kill', [pidStr]);
       }
     } catch (e) {}
     claudeRelayPid = 0;