openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-10-19 17:28:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

permissions: move workspace roots onto thread state

Browse Source
This commit is contained in:
Michael Bolin
2026-05-13 08:38:19 -07:00
parent 3d517fbd00
commit ada4bdc2dc
168 changed files with 3263 additions and 3357 deletions
Download Patch File Download Diff File Expand all files Collapse all files
codex-rs/thread-store/Cargo.toml
+1
View File
@@ -20,6 +20,7 @@ codex-protocol = { workspace = true }
codex-rollout = { workspace = true }
codex-state = { workspace = true }
codex-utils-path = { workspace = true }
codex-utils-absolute-path = { workspace = true }
serde = { workspace = true, features = ["derive"] }
serde_json = { workspace = true }
thiserror = { workspace = true }
codex-rs/thread-store/src/local/create_thread.rs
+1
View File
@@ -22,6 +22,7 @@ pub(super) async fn create_thread(
codex_home: store.config.codex_home.clone(),
sqlite_home: store.config.sqlite_home.clone(),
cwd,
workspace_roots: params.metadata.workspace_roots,
model_provider_id: params.metadata.model_provider.clone(),
generate_memories: matches!(params.metadata.memory_mode, ThreadMemoryMode::Enabled),
};
codex-rs/thread-store/src/local/list_threads.rs
+1
View File
@@ -44,6 +44,7 @@ pub(super) async fn list_threads(
codex_home: store.config.codex_home.clone(),
sqlite_home: store.config.sqlite_home.clone(),
cwd: store.config.codex_home.clone(),
workspace_roots: Vec::new(),
model_provider_id: store.config.default_model_provider_id.clone(),
generate_memories: false,
};
codex-rs/thread-store/src/local/live_writer.rs
+1
View File
@@ -62,6 +62,7 @@ pub(super) async fn resume_thread(
codex_home: store.config.codex_home.clone(),
sqlite_home: store.config.sqlite_home.clone(),
cwd,
workspace_roots: params.metadata.workspace_roots,
model_provider_id: params.metadata.model_provider.clone(),
generate_memories: matches!(params.metadata.memory_mode, ThreadMemoryMode::Enabled),
};
codex-rs/thread-store/src/local/mod.rs
+4
View File
@@ -526,6 +526,7 @@ mod tests {
include_archived: false,
metadata: ThreadPersistenceMetadata {
cwd: Some(home.path().to_path_buf()),
workspace_roots: Vec::new(),
model_provider: "different-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
},
@@ -581,6 +582,7 @@ mod tests {
include_archived: false,
metadata: ThreadPersistenceMetadata {
cwd: Some(home.path().to_path_buf()),
workspace_roots: Vec::new(),
model_provider: "different-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
},
@@ -795,6 +797,7 @@ mod tests {
include_archived: true,
metadata: ThreadPersistenceMetadata {
cwd: None,
workspace_roots: Vec::new(),
model_provider: "test-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
},
@@ -1025,6 +1028,7 @@ mod tests {
fn thread_metadata() -> ThreadPersistenceMetadata {
ThreadPersistenceMetadata {
cwd: Some(std::env::current_dir().expect("cwd")),
workspace_roots: Vec::new(),
model_provider: "test-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
}
codex-rs/thread-store/src/local/update_thread_metadata.rs
+1
View File
@@ -1510,6 +1510,7 @@ mod tests {
fn test_thread_metadata() -> ThreadPersistenceMetadata {
ThreadPersistenceMetadata {
cwd: Some(std::env::current_dir().expect("cwd")),
workspace_roots: Vec::new(),
model_provider: "test-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
}
codex-rs/thread-store/src/thread_metadata_sync.rs
+1
View File
@@ -533,6 +533,7 @@ mod tests {
include_archived: false,
metadata: ThreadPersistenceMetadata {
cwd: None,
workspace_roots: Vec::new(),
model_provider: "test-provider".to_string(),
memory_mode: ThreadMemoryMode::Enabled,
},
codex-rs/thread-store/src/types.rs
+4
View File
@@ -14,6 +14,7 @@ use codex_protocol::protocol::SessionSource;
use codex_protocol::protocol::ThreadMemoryMode as MemoryMode;
use codex_protocol::protocol::ThreadSource;
use codex_protocol::protocol::TokenUsage;
use codex_utils_absolute_path::AbsolutePathBuf;
use serde::Deserialize;
use serde::Deserializer;
use serde::Serialize;
@@ -59,6 +60,9 @@ pub struct ThreadPersistenceMetadata {
///
/// `None` means the thread has no filesystem/environment context.
pub cwd: Option<PathBuf>,
/// Thread-scoped workspace roots used to materialize permission profiles.
#[serde(default)]
pub workspace_roots: Vec<AbsolutePathBuf>,
/// Model provider associated with the thread.
pub model_provider: String,
/// Memory mode associated with the live thread.