feat: add remote server/client mode over TCP - #20
Merged
Merged
Conversation
Every provider send() now logs: debug before request, error on HTTP failure (with URL, status, response body), error on stream read errors, and debug on stream start. Adds logging standard to CLAUDE.md. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Priority: PHX_LOG env var > project .phx/phx.json > global ~/.phx/phx.json > "info" default. Add "log_level" field to Config schema with merge support. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Extract duplicated HTTP calling code from all four providers (Anthropic, OpenAI, Google, Ollama) into a shared src/http/ module. Each provider now uses http::send_streaming() instead of manual reqwest calls, eliminating ~100 lines of duplicated client creation, status checking, and error logging. Add retry with exponential backoff (1,2,5,10,15,20,25,30,30,30s) for transient failures (429, 500, 502, 503, 504, connection errors). Max 10 attempts. Non-retryable errors (400, 401, 403) fail immediately. Retry status is shown inline in the TUI chat as system messages. The retry wait is interruptible via Esc/Ctrl-C. Changes: - New src/http/mod.rs with send_streaming, is_retryable, backoff_delay - Provider::send() now takes &SendOptions (was by-value) - Retry loop in session/conversation.rs and session/agent_loop.rs - ConvEvent::Retrying and ConvEvent::RetryRecovered for TUI feedback - Msg::ConvRetrying and Msg::ConvRetryRecovered with update handlers - Version bump to 0.6.0 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Extract shared SSE stream parser (SseParser) into src/http/sse.rs used by all four providers. Centralize Message-to-ProviderMessage conversion into session::message::to_provider_messages, removing duplicate mapping from conversation.rs and agent_loop.rs. Add ToolSchema::to_openai_json and to_simple_json to eliminate repeated tool JSON construction. Extract render_picker_list into picker.rs, consolidating modal_picker and command_completion rendering. Fix MockProvider to use non-retryable error so tests don't hang on retry backoff. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… per spec Previously, next_double() tried to parse each data: line individually as JSON. Per the SSE spec, multiple data: lines in a block should be joined with newlines before interpretation. This caused split JSON payloads to fall through as Raw instead of Data. Adds regression test: double_newline_concatenates_multiline_data Closes PHX-507
Add --host/--port flags to run phx as a headless TCP server, and --remote flag to connect the TUI to a remote server. Designed for running across tailscale-connected machines. Server mode reuses rpc::server::run over TCP with per-connection sessions. Client mode produces ConvEvent from the TCP stream so the TUI rendering layer is unchanged. - Add RemoteConfig to config schema (default host 127.0.0.1, port 4200) - Wire SESSION_RESUME handler and session_id param in rpc::server - New src/remote/ module with TCP server and RemoteClient with id-demuxing read task - Branch Cmd::StartConversation/ResumeSession on app.remote.is_some() - Add spawn_remote_conversation() and resume_remote_session() - Status bar shows "remote: <endpoint>" when connected remotely Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add ConvEvent::RemoteSessionId so the server's session_id event flows into app.remote_session_id (was only logged before) - Intercept /sessions and /resume in remote mode to query the server via session.list RPC instead of reading the local (empty) store - These are required for session resume across reconnects to work Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
# Conflicts: # Cargo.toml # src/config/schema.rs
- Remove eprintln! in remote server (use tracing only) - Enforce --host/--port co-requirement via clap `requires` - Extract restore_session helper to deduplicate rpc/server.rs - Add shared retry helpers (should_retry, format_retry_msg, format_recovered_msg) - Switch RemoteClient to bounded channel with backpressure - Notify pending requests on server disconnect instead of silent drop - Add HomeOverrideGuard for safer test home dir management - Sync Cargo.lock version to 0.7.0 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
--host+--port) — headless phx that accepts TCP connections and delegates to the existing RPC server--remote host:port) — TUI connects to a remote phx server over TCP instead of running locallysrc/http/session.resumemethod + remote/sessionscommandTest plan
just checkpasses (lint, build, test, lockfile)--hostrequires--portand vice versa--remoteconflicts with--host/--portconfig.get, gets response🤖 Generated with Claude Code