admin/CC-Switch
1
0
Fork 0
mirror of https://github.com/farion1231/cc-switch.git synced 2026-07-24 12:44:18 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(codex): skip environment_context injection when extracting session title (#2439)

Browse Source 
Co-authored-by: xiaohan1 <xiaohan1@chinaums.com>
This commit is contained in:
eclipsehx
2026-04-30 12:45:42 +08:00
committed by GitHub
parent 15497b0e41
commit 1c69269405
1 changed files with 23 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src-tauri/src/session_manager/providers/codex.rs
+23 -1
View File
@@ -173,7 +173,10 @@ fn parse_session(path: &Path) -> Option<SessionMeta> {
{
let text = payload.get("content").map(extract_text).unwrap_or_default();
let trimmed = text.trim();
if !trimmed.is_empty() && !trimmed.starts_with("# AGENTS.md") {
if !trimmed.is_empty()
&& !trimmed.starts_with("# AGENTS.md")
&& !trimmed.starts_with("<environment_context>")
{
first_user_message = Some(trimmed.to_string());
}
}
@@ -354,6 +357,25 @@ mod tests {
assert!(parse_session(&path).is_none());
}
#[test]
fn parse_session_skips_environment_context_injection() {
let temp = tempdir().expect("tempdir");
let path = temp.path().join("session.jsonl");
std::fs::write(
&path,
concat!(
"{\"timestamp\":\"2026-03-06T21:50:12Z\",\"type\":\"session_meta\",\"payload\":{\"id\":\"test-id\",\"cwd\":\"/tmp/project\"}}\n",
"{\"timestamp\":\"2026-03-06T21:50:13Z\",\"type\":\"response_item\",\"payload\":{\"type\":\"message\",\"role\":\"user\",\"content\":\"<environment_context>\\n <cwd>/tmp/project</cwd>\\n</environment_context>\"}}\n",
"{\"timestamp\":\"2026-03-06T21:50:14Z\",\"type\":\"response_item\",\"payload\":{\"type\":\"message\",\"role\":\"user\",\"content\":\"Fix the login bug\"}}\n"
),
)
.expect("write");
let meta = parse_session(&path).unwrap();
// Should skip environment_context injection and use the real user message
assert_eq!(meta.title.as_deref(), Some("Fix the login bug"));
}
#[test]
fn parse_session_falls_back_to_dir_basename() {
let temp = tempdir().expect("tempdir");