Fix proxy response handling and improve error handling#173
Open
chriswritescode-dev wants to merge 24 commits intomainfrom
Open
Fix proxy response handling and improve error handling#173chriswritescode-dev wants to merge 24 commits intomainfrom
chriswritescode-dev wants to merge 24 commits intomainfrom
Conversation
- Introduce <promise>DONE</promise> completion signal for architect and code agents - Require verification steps in all plans with mandatory test/lint/type-check commands - Add session status fetching to CLI status command - Improve session output formatting with recent activity display - Fix v2 client session status calls to include worktree directory parameter - Bump version to 0.0.28
- Add useWorkspaces config option for loop worktree management - Implement SQLite-based loop state persistence in TUI - Update build script to use Bun.build with solid plugin - Improve loop state handling for workspace vs worktree detection - Add click-to-select functionality for loops in sidebar - Update tests to use wrk_loop_ workspace ID prefix
…lling - Replace zodResolver with standardSchemaResolver in 10 frontend files to fix zod version incompatibility - Add @standard-schema/spec dev dependency - Improve loop polling logic in tui.tsx - Add error logging for toast failures in loop.ts
Extract tool definitions from 1554-line index.ts into focused modules under tools/: - tools/types.ts: Shared ToolContext interface and types - tools/memory.ts: Memory CRUD tools (read, write, edit, delete) - tools/kv.ts: Key-value store tools - tools/health.ts: Health check and reindex tools - tools/plan-execute.ts: Plan execution tool - tools/loop.ts: Memory loop tools with setup orchestration - tools/plan-approval.ts: Plan approval directives and loop blocking hooks - tools/index.ts: Barrel export with createTools factory Results: - index.ts reduced from 1554 to 321 lines (79% reduction) - Factory pattern: createXxxTools(ctx: ToolContext) for each category - ToolContext interface consolidates all dependencies - No runtime behavior changes - pure refactoring - All 295 tests passing, lint and build successful
- Convert response body to text before creating new Response in proxy services - Use Set for header filtering to improve performance - Update error handling in useSendPrompt to use FetchError - Add dev-only logging for SSE initial session data failures
- Replace backtick-wrapped tool names with "tool" suffix for clarity - Add memory-kv-delete to librarian's excluded tools list - Update auditor agent instructions for memory KV operations - Fix loop service review instructions for consistency
Move memory plugin installation from Dockerfile build-time to entrypoint runtime. Add INSTALL_MEMORY_PLUGIN env var (defaults to true) to control installation. Update Dockerfile, docker-compose.yml, entrypoint script, and .env.example.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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
Why
Type Of Change
Checklist