Skip to content

Complete Freshcodex support across Freshagent - #880

Open
danshapiro wants to merge 84 commits into
mainfrom
the-usual/freshcodex-complete
Open

danshapiro wants to merge 84 commits into
mainfrom
the-usual/freshcodex-complete

Conversation

@danshapiro

@danshapiro danshapiro commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

Freshcodex now supports the shared Freshagent experience across local and managed sessions: rich live and saved conversation output, read-only child conversations, steering and Stop, approvals and questions, installed model discovery, thinking levels, Build/Plan and access settings, recovery, and CLI/MCP/REST automation. CLI and MCP clients can create, split, resume, send to, capture, and wait for Freshcodex conversations.

The implementation preserves native identity and conversation ownership, distinguishes accepted/refused/uncertain sends, and avoids replaying an uncertain prompt. It qualifies Codex 0.164 before aligning the managed runtime, retains honest interrupted/truncated/unsaved history states, and documents the resulting behavior in the README and interface mock.

The compatibility work also makes Claude shutdown signal and join its actual native child before the sidecar exits, preserves fixture process ownership during cleanup, and supports ordinary Cargo binary and temporary-directory discovery. Shutdown retains the existing one-second regression budget. Main PR881 and PR882 are integrated while preserving their cleanup protection and discovery-test headroom.

Validation at 1d3eda197c8e58f8eb7b9b1736f957c084a44507:

  • Complete pnpm run check passed: six prerequisite builds, three typechecks, 9,947 cloud tests in 672 files/four actual tasks, 21 local companion tests, 10 source-runtime tests, 7,478 top-level Rust tests in 192 groups including Tauri, and 395 Electron tests in 42 files. All owned test resources and query helpers closed normally.
  • Ten unchanged Rust ignores and one existing companion opt-in remained unexecuted. No failing-test exemption, reduced assertion, new skip, local fallback or hook bypass was used. Nested Rust fixture footers were excluded from the unique-test count.
  • Ordinary push hooks passed formatting, Clippy, 2,876 affected Rust tests at the preceding implementation head, and pinned pnpm TypeScript checks. The final test-only main integration passed all eight focused discovery cases and its ordinary TypeScript push hook. Actual host and sandbox Rust versions were 1.96.0; pnpm was 10.34.5.
  • Actual required GitHub Rust workspace tests and rust-gate passed at this final head, as did all four Electron platform builds with installer, unpacked-artifact and checkout-free runtime acceptance.
  • Independent GLM 5.3/OpenCode task, whole-branch, complete-delta, repair and main-integration reviews passed; every finding has a final disposition and no required finding remains unresolved.
  • Separate feature acceptance covered 69 configured-cloud browser cases across 11 affected specs, five checkpoint cases through the actual pinned compiler consumer, 24 actual release-native captures plus one missing-prerequisite negative case, and managed-native recovery with a second send on the same opaque reference after native identity changed. Those feature paths are unchanged by later fixture/Claude repairs. Claude compatibility checks additionally passed 49 focused cloud tests including both real SDK/CLI shutdown modes, four browser cases, and the integrated 27-test Docker batch.

Browser acceptance exercises the real Freshell server/UI with simulated native responses; installed-native acceptance is separate. No production deployment or restart is part of this PR.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant