feat(cli): add oclite UI components for task panel #53
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for your contribution! This PR doesn't have a linked issue. All PRs must reference an existing issue. Please:
See CONTRIBUTING.md for details. |
- Fixes context initialization in UI components - Hardens ANSI injection protection - Fixes UTF-8 truncation issues - Ensures safeNum consistency across modules
This was referenced Jan 27, 2026
…odel (#73) Research into AIChat and Aider showed neither production AI chat CLI uses DECSTBM scroll regions. Both use append-only stdout with a dedicated line editor. Reverting to the simpler, proven architecture.
, #95, #96) - Show spinner immediately for custom commands before blocking work - Add elapsed time ticker for running subagent tasks - Show nested child tool status below running tasks - Add theme-based color coding (completed tools dimmed, tasks cyan) - Fix memory leaks in Bus subscription and task tracking maps
…ovide - Add await before input.fn() to keep Promise within context.provide scope - Fix type signatures: fn: () => Promise<R>, init?: () => Promise<void> - Update call sites in worktree and tests to use async functions Fixes startup crash: "No context found for instance"
…load crash State.create() was eagerly evaluating root() at module import time, but root() calls Instance.directory which requires AsyncLocalStorage context. This caused "No context found for instance" crashes. Now root() is only called when the returned factory function is invoked, which happens inside Instance.provide() context.
style.dim alone wasn't rendering visibly. Tools now show: - Completed: green icon + dim gray text - Running: yellow icon + dim gray text - Error: red icon + red text
#102) - Flush line buffer with newline before tool_start to prevent inline rendering - Reuse tool ID for repeated calls, show (×N) counter instead of spam
- Todos hidden by default, show '(N tasks - ctrl+t to show)' indicator - Press ctrl+t to toggle full todo list with icons and priorities - Centralized state in liveblock to prevent desync after reset - Added toggleTasksVisible() and getTasksVisible() methods
…#100) - Auto-search relevant memories before task dispatch - Auto-store task outcomes on successful completion - Graceful fallback when remory unavailable (TTL-cached availability) - Metadata serialized into text content for searchability - Empty description validation prevents unnecessary searches - Removed unused remory_persist_thinking config stub
- Show init progress: "Loading plugins" → "Loading providers" → etc. - Fix task rendering: tasks now show inline with other tools - Fix taskToChildSession memory leak: clear on stream reset - Remove premature taskToChildSession.delete() that broke child tool routing
Previously showed all possible models (300-1000+) filling the screen. Now only shows models from providers with API keys configured.
…104) - Add red ✗ visual for permission-denied tools with '(permission denied)' suffix - Fix subagent output timing: populate taskToChildSession at tool_start - Fix 7 failing tests: add missing provider key to mocks - Add type-safe validation for error, sessionId, agent, description fields - Extract PERMISSION_DENIED_PATTERN constant with case-insensitive regex - Remove duplicate child session extraction from tool_end
- Change rg tool to use single 'rg' permission instead of 'glob'/'grep' - Update TOOL display mappings in github.ts and run.ts - Update explore agent and ACP agent permission configs - Add legacy migration for existing glob/grep configs - Update Permission schema in config.ts - Update all related tests One tool = one permission. Simpler, more intuitive.
#109) - Add auto-wakeup: enable Session.enableAutoWakeup() when session starts - Fix child session linking: use Bus subscription only (remove race condition) - Fix memory leak: clear maps on Esc cancel and streamResponse start - Add partIdToToolId map to track tool callIDs for metadata updates Users now see task progress immediately and results auto-deliver on completion.
- Replace shell-based realpath with fs/promises realpath (security fix) - Remove static directory placeholders from bash tool description - Directory info now comes only from system prompt (dynamic per session) Prevents subagents from hallucinating parent session's working directory.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Implements comprehensive UI components for the oclite task panel interface, including task display, activity status, and system information.
Changes
taskpanel.ts): Main task display component with status icons and task listingstatusline.ts): Activity status display showing current operationsbottombar.ts): System information bar displaying permissions and file countstask.ts): Updated with improved task limit (5 → 10 concurrent tasks)Security
✅ ANSI injection protection for all user-visible output
✅ Input validation on all data sources
✅ Safe string rendering with escape sequences
Testing
bun run typecheckFixes
Related Issues
Task limit increased from 5 to 10 for better concurrent task handling.