diff --git a/codex-rs/core/src/session_share.rs b/codex-rs/core/src/session_share.rs index 4677130078..851415c4be 100644 --- a/codex-rs/core/src/session_share.rs +++ b/codex-rs/core/src/session_share.rs @@ -146,18 +146,9 @@ pub async fn upload_rollout_with_owner( upload_meta(&store, &meta_key, &updated).await?; } (true, None) => { - // Recover from a previous metadata upload failure by restoring metadata - // and overwriting the rollout blob. - let meta = SessionShareMeta { - owner: owner.to_string(), - created_at: now, - updated_at: now, - }; - upload_meta(&store, &meta_key, &meta).await?; - store - .put_object(&key, data, "application/x-ndjson") - .await - .with_context(|| format!("failed to upload rollout for id {session_id}"))?; + return Err(anyhow::anyhow!( + "remote session already exists but has no metadata; refusing to overwrite" + )); } (false, Some(meta)) => { if meta.owner != owner { diff --git a/codex-rs/utils/cargo-bin/src/lib.rs b/codex-rs/utils/cargo-bin/src/lib.rs index 67539f41b0..b0be2fe79a 100644 --- a/codex-rs/utils/cargo-bin/src/lib.rs +++ b/codex-rs/utils/cargo-bin/src/lib.rs @@ -126,7 +126,7 @@ fn cargo_target_dir() -> Result { fn absolutize_from_buck_or_cwd(path: PathBuf) -> Result { path.absolutize() - .map(|abs| abs.into_owned()) + .map(std::borrow::Cow::into_owned) .map_err(|source| CargoBinError::CurrentDir { source }) }