Skip to content

fix(acp): preserve failed-turn reload and external ids#147

Open
MoozLee wants to merge 1 commit intoxintaofei:mainfrom
MoozLee:pr/failed-turn-reload-and-external-id-upstream
Open

fix(acp): preserve failed-turn reload and external ids#147
MoozLee wants to merge 1 commit intoxintaofei:mainfrom
MoozLee:pr/failed-turn-reload-and-external-id-upstream

Conversation

@MoozLee
Copy link
Copy Markdown
Contributor

@MoozLee MoozLee commented May 3, 2026

Closes #146

Summary

  • preserve an existing conversation external_id when session/load falls back to session/new
  • keep failed optimistic user turns recoverable across reload without hiding persisted history
  • restore missing Gemini parser test imports so current Rust tests compile on this branch

Test plan

  • pnpm build
  • targeted frontend eslint for changed runtime files
  • cargo clean --manifest-path src-tauri/Cargo.toml
  • cargo check --manifest-path src-tauri/Cargo.toml
  • cargo check --manifest-path src-tauri/Cargo.toml --bin codeg-server --no-default-features
  • cargo test --manifest-path src-tauri/Cargo.toml gemini --no-run
  • manual verification: failed turn reload keeps last user message visible
  • manual verification: session/load fallback no longer overwrites existing external_id

Keep failed user turns recoverable across reload without hiding persisted history, and avoid overwriting an existing conversation external_id when session/load falls back to session/new. Also restore missing Gemini test imports so the current Rust test suite compiles again.
@MoozLee MoozLee force-pushed the pr/failed-turn-reload-and-external-id-upstream branch from a061ddd to 6bd6648 Compare May 3, 2026 04:45
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.

bug: session/load 回退时 external_id 被覆盖,导致数小时会话日志丢失

1 participant