scalix 1.1.0 → 1.2.0

package/PARITY-PLAN.md

@@ -0,0 +1,116 @@
+# Scalix Coder — Claude Code Parity Implementation Plan
+
+**Status:** In Progress
+**Target:** Full feature and UX parity with Claude Code
+**Estimated Total:** 12-15 days
+
+---
+
+## Phase 1: Security & Reliability (1-2 days) ✓ COMPLETE
+
+- [x] **1.1 Secure credential storage** — macOS Keychain / Linux libsecret via `keychain.ts`, auto-strips key from config file
+- [x] **1.2 Auto-retry with exponential backoff** — Already existed in agent.ts (backoffDelay + circuit breaker)
+- [x] **1.3 Non-streaming fallback** — `forceNonStreaming` flag auto-retries with `stream: false` on stream errors
+
+## Phase 2: Chat Experience Parity (3-4 days) ✓ COMPLETE
+
+- [x] **2.1 Syntax highlighting in code blocks** — `cli-highlight` integration for 100+ languages + render cache (128 entries)
+- [x] **2.2 Collapsible tool groups** — Auto-groups 3+ consecutive tool items, Ctrl+O toggles expansion
+- [x] **2.3 Multiline input** — Alt+Enter inserts newline (terminal limitation prevents Shift+Enter)
+- [x] **2.4 "Always Allow" in permission prompts** — [A] key auto-allows tool for session, shown in prompt UI
+- [x] **2.5 @-mention typeahead in terminal** — `@` triggers fuzzy file search from project index, Tab/Enter selects
+- [x] **2.6 Ctrl+R reverse history search** — Incremental reverse search with visual indicator, Enter/Esc to accept/cancel
+- [x] **2.7 External editor support** — Ctrl+X Ctrl+E opens $EDITOR with input buffer, loads edited content back
+- [x] **2.8 Markdown render cache** — Content-hash cache (128 entries) prevents re-rendering identical content
+- [x] **2.9 Rate limit warning banner** — Context % in status bar when >50%, rate limit warning on 429 errors, 2-min auto-clear
+- [x] **2.10 Image paste from clipboard** — Ctrl+V detects clipboard images (macOS pngpaste/osascript, Linux xclip), shows `[Image #N]` chips, sends to vision model
+
+## Phase 3: Permission & Hook System (2-3 days) ✓ COMPLETE
+
+- [x] **3.1 Scoped permissions** — `run_command(git:*)` prefix matching per tool, configurable allow/deny rules in `scopedPermissions` config
+- [x] **3.2 Enhanced hooks — 9 events** — Added Stop, SessionEnd, Notification events (now 9 total)
+- [x] **3.3 Prompt-type hooks** — `type: 'prompt'` hooks with `getPromptHooks()` API, separate from command hooks
+- [x] **3.4 Structured hook I/O** — JSON stdin with event/tool/args, JSON stdout parsing, exit code 2 = block with message
+- [x] **3.5 Matcher patterns** — Pipe-separated `run_command|edit_file`, wildcard `*`, regex `/pattern/` in hook match field
+- [x] **3.6 Permission memory** — "Always Allow" ([A] key) persisted per session, scoped rules in config, shown in `/permissions`
+- [x] **3.7 Recent denials view** — `/permissions` shows last 10 denied tools with timestamps and args
+
+## Phase 4: Command & Skills System (2-3 days) ✓ COMPLETE
+
+- [x] **4.1 Rich command frontmatter** — Parse `allowed-tools`, `model`, `argument-hint`, `disable-model-invocation` from .md commands
+- [x] **4.2 @file references in commands** — `@path/to/file` includes file content inline in command prompt
+- [x] **4.3 Inline bash execution** — `` !`command` `` syntax executes shell and injects stdout into command prompt
+- [x] **4.4 Positional arguments** — `$1`, `$2`, etc. in addition to existing `$ARGS`
+- [x] **4.5 Skills framework** — `SKILL.md` with frontmatter (name, description, version, paths), progressive 3-level disclosure
+- [x] **4.6 Skill discovery** — `/skills` command lists available skills with descriptions
+- [x] **4.7 Path-triggered skills** — `paths:` frontmatter auto-loads skill when working in matching directories
+- [x] **4.8 Skill resources** — `scripts/`, `references/`, `examples/`, `assets/` directories per skill
+- [x] **4.9 Built-in skills** — feature-dev (7-phase workflow), code-review, commit-push-pr
+
+## Phase 5: Session & Conversation (1-2 days) ✓ COMPLETE
+
+- [x] **5.1 Auto-save sessions** — Auto-saves crash state after each assistant turn via `saveCrashState()`
+- [x] **5.2 Session resume picker** — `/resume` tries crash recovery first, falls back to interactive session picker
+- [x] **5.3 Session forking** — `/fork [N]` branches conversation from message N in history
+- [x] **5.4 Session diff** — `getSessionDiff()` tracks file modifications (edit_file/write_file) per session
+- [x] **5.5 Crash recovery** — `.crash-recovery.json` auto-saved, detected on startup, resumable via `/resume`
+
+## Phase 6: Configuration & Settings (1 day) ✓ COMPLETE
+
+- [x] **6.1 Local settings overrides** — `.scalix/settings.local.json` (gitignored) for per-project model/permission/limits overrides
+- [x] **6.2 Managed/organization settings** — `managed-settings.json` for enterprise policy enforcement (permission mode, security, scoped permissions)
+- [x] **6.3 Keybinding customization** — Covered by existing keyboard shortcuts system + Ctrl+X Ctrl+E editor support
+- [x] **6.4 Theme support** — Covered by existing marked-terminal + cli-highlight integration (auto-adapts to terminal colors)
+- [x] **6.5 .scalixignore file** — `.scalixignore` and `.scalix/.ignore` files for excluding paths from project indexing
+
+## Phase 7: Advanced Features (1-2 days) ✓ COMPLETE
+
+- [x] **7.1 Worktree support** — `--worktree` CLI flag + `/worktree` command (list/create/remove/cleanup), backed by existing `worktree.ts`
+- [x] **7.2 CJK/RTL text support** — `text-utils.ts` with CJK/RTL detection, double-width char handling, `wrapText()`, `applyRTLMarks()`
+- [x] **7.3 IME composition support** — `stringWidth()` for accurate cursor positioning with CJK chars, combining mark handling
+- [x] **7.4 Idle session nudge** — 75-minute idle timer suggests `/clear` or `/compact` when conversation has 10+ messages
+- [x] **7.5 Terminal notification passthrough** — `terminal-notify.ts` with iTerm2/Kitty/Ghostty/tmux OSC support, auto-notify on task completion
+- [x] **7.6 Plugin management commands** — `/plugin install/uninstall/enable/disable/list` with git clone + local symlink support
+- [x] **7.7 Installation channels** — Homebrew formula, curl install script, PowerShell installer (all in `scripts/`)
+
+---
+
+## Summary
+
+| Phase | Items | Est. Days | Priority |
+|-------|-------|-----------|----------|
+| 1. Security & Reliability | 3 | 1-2 | Critical |
+| 2. Chat Experience | 10 | 3-4 | Critical |
+| 3. Permission & Hooks | 7 | 2-3 | Critical |
+| 4. Command & Skills | 9 | 2-3 | Important |
+| 5. Session & Conversation | 5 | 1-2 | Important |
+| 6. Configuration | 5 | 1 | Important |
+| 7. Advanced Features | 7 | 1-2 | Nice to have |
+| **Total** | **46** | **12-15** | |
+
+---
+
+## Already Ahead (Preserve These Advantages)
+
+These features already exceed Claude Code — protect them during implementation:
+
+- 40+ built-in tools (browser automation, multi-provider search, vision, voice)
+- Model-agnostic with multi-model complexity routing
+- Dynamic model discovery from API
+- Auto-convention detection and project memory
+- Injection detection with configurable sensitivity
+- Checkpoint/undo system (`/checkpoint`, `/undo`)
+- VS Code: inline chat, tab completion, context menu, 6 @-mention types
+- Session replay (`/replay`)
+- Message queueing during processing
+- Shell escape (`!cmd`) prefix
+- Input stashing (Ctrl+S)
+- Auto-verification after file edits (build/lint check)
+- Multiple output formats (text, json, json-schema, patch, markdown)
+- Gradient banner with ink-big-text
+- Informative status bar (model + tokens + tools + mode + elapsed time + version)
+- Box-drawn diff frames with addition/deletion summary
+- Smart error hints (pattern-matched suggestions)
+- Scroll indicators (↑ more above / ↓ more below)
+- Distinctive ❯ prompt with spinner animation
+- Dynamic placeholder text

package/bin/scalix.js

@@ -13,6 +13,7 @@ const needsHeadless =
   args.includes('--pipe') || args.includes('-p') ||
   args.includes('--ipc') ||
   args.includes('--help') || args.includes('-h') ||
+  args.includes('--worktree') || args.includes('-w') ||
   !process.stdin.isTTY;
 
 const entry = resolve(__dirname, '..', 'dist', needsHeadless ? 'headless.js' : 'cli.js');

package/dist/agent.js

@@ -649,6 +649,7 @@ export class Agent extends EventEmitter {
         let retryAttempt = 0;
         let emptyTurnCount = 0;
         let disableTools = false;
+        let forceNonStreaming = false;
         while (turnCount < this.config.maxSteps) {
             if (this._aborted) {
                 reqLog.info('Agent aborted by user');
@@ -704,7 +705,7 @@ export class Agent extends EventEmitter {
                 const apiMessages = messagesToApi(this.messages, effectiveInstructions, vision);
                 const thinkingLevel = this.config.thinkingMode !== 'off' ? this.config.thinkingMode : undefined;
                 const hasToolMessages = this.messages.some(m => m.role === 'tool' || (m.role === 'assistant' && m.tool_calls && m.tool_calls.length > 0));
-                const useStreaming = !hasToolMessages;
+                const useStreaming = !hasToolMessages && !forceNonStreaming;
                 if (useStreaming) {
                     const stream = await this.client.chatComplete({
                         model: currentModel,
@@ -1107,8 +1108,18 @@ export class Agent extends EventEmitter {
             catch (err) {
                 this._abortController = null;
                 this.recordModelFailure(currentModel);
+                const errMsg = (err?.message || '').toLowerCase();
+                const isStreamError = errMsg.includes('stream') || errMsg.includes('sse') || errMsg.includes('no response body');
+                // On stream failure, retry once with non-streaming before giving up
+                if (isStreamError && !forceNonStreaming) {
+                    forceNonStreaming = true;
+                    reqLog.warn('Streaming failed, retrying with non-streaming mode', { error: err.message });
+                    await sleep(500);
+                    continue;
+                }
                 if (this.shouldFallbackToNext(err) && retryAttempt < this.config.maxRetries && turnCount < this.config.maxSteps) {
                     retryAttempt++;
+                    forceNonStreaming = false;
                     const delay = backoffDelay(retryAttempt);
                     reqLog.warn(`Retrying after ${Math.round(delay)}ms (attempt ${retryAttempt}/${this.config.maxRetries})`, {
                         model: currentModel,