Hare/yoi
1
1
Fork 0
Code Actions Packages Releases Activity

fix: remove tests that pin mutable profile defaults

Browse Source
This commit is contained in:
Hare
2026-10-06 13:31:37 +09:00
parent c57e236768
commit 3029e77c7a
2 changed files with 0 additions and 140 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/manifest/src/builtin_profile.rs
-1
View File
@@ -337,7 +337,6 @@ mod tests {
.expect("evaluate built-in standalone")
.expect("standalone exists");
assert_eq!(value["slug"], "standalone");
assert_eq!(value["model"]["ref"], "codex-oauth/gpt-6.1-sol");
assert_eq!(value["scope"]["intent"], "workspace_write");
assert_eq!(value["scope"]["symlink_policy"], "logical");
assert_eq!(value["delegation_scope"]["intent"], "workspace_write");
crates/manifest/src/profile.rs
-139
View File
@@ -1255,22 +1255,6 @@ mod tests {
)
.unwrap();
assert_eq!(resolved.manifest.worker.name, "role-worker");
let compaction = resolved.manifest.compaction.as_ref().unwrap();
assert_eq!(compaction.worker_max_turns, Some(100), "{}", entry.name);
assert_eq!(
compaction.worker_context_max_tokens, 100_000,
"{}",
entry.name
);
assert_eq!(
compaction
.model
.as_ref()
.and_then(|model| model.ref_.as_deref()),
Some("codex-oauth/gpt-6-luna"),
"{}",
entry.name
);
}
}
@@ -1332,125 +1316,6 @@ mod tests {
assert!(!legacy_consolidator.feature.subjektiv.profile.enabled);
}
#[test]
fn builtin_profiles_pin_role_models_and_reasoning() {
use crate::model::{ReasoningControl, ReasoningEffort};
let tmp = TempDir::new().unwrap();
let resolve = |name: &str| {
ProfileResolver::new()
.with_workspace_base(tmp.path())
.resolve(
&ProfileSelector::source_named(ProfileRegistrySource::Builtin, name),
ProfileResolveOptions::with_worker_name(format!("{name}-worker")),
)
.unwrap()
};
for name in [
"default",
"standalone",
"intake",
"orchestrator",
"companion",
"coder",
"reviewer",
"backend-job",
"memory-consolidation",
"subjektiv-memory-consolidation",
] {
let role = resolve(name);
assert_eq!(
role.manifest
.compaction
.as_ref()
.map(|compaction| compaction.prune_enabled),
Some(false),
"{name} must keep pruning disabled for the experiment"
);
assert_eq!(
role.manifest
.compaction
.as_ref()
.and_then(|compaction| compaction.model.as_ref())
.and_then(|model| model.ref_.as_deref()),
Some("codex-oauth/gpt-6-luna"),
"{name} must use Luna for compaction"
);
}
let companion = resolve("companion");
assert_eq!(
companion.manifest.model.ref_.as_deref(),
Some("codex-oauth/gpt-6-astra")
);
assert_eq!(
companion.manifest.engine.reasoning,
Some(ReasoningControl::Effort(ReasoningEffort::High))
);
for name in [
"default",
"standalone",
"intake",
"orchestrator",
"coder",
"reviewer",
] {
let role = resolve(name);
assert_eq!(
role.manifest.model.ref_.as_deref(),
Some("codex-oauth/gpt-6.1-sol")
);
assert_eq!(
role.manifest.engine.reasoning,
Some(ReasoningControl::Effort(ReasoningEffort::High))
);
let extraction = &role.manifest.feature.subjektiv.profile.extraction;
assert_eq!(
extraction
.model
.as_ref()
.and_then(|model| model.ref_.as_deref()),
Some("codex-oauth/gpt-6-luna")
);
assert_eq!(
extraction.reasoning,
Some(ReasoningControl::Effort(ReasoningEffort::Medium))
);
}
for name in [
"backend-job",
"memory-consolidation",
"subjektiv-memory-consolidation",
] {
let consolidation = resolve(name);
assert_eq!(
consolidation.manifest.model.ref_.as_deref(),
Some("codex-oauth/gpt-6-luna")
);
assert_eq!(
consolidation.manifest.engine.reasoning,
Some(ReasoningControl::Effort(ReasoningEffort::Medium))
);
if name == "memory-consolidation" {
assert_eq!(
consolidation
.manifest
.feature
.memory
.profile
.extraction
.model
.as_ref()
.and_then(|model| model.ref_.as_deref()),
Some("codex-oauth/gpt-6-luna")
);
}
}
}
#[test]
fn subjektiv_memory_consolidation_profile_is_lifecycle_and_capability_isolated() {
let tmp = TempDir::new().unwrap();
@@ -1595,10 +1460,6 @@ mod tests {
..
} if name == "standalone" && provenance.starts_with("profiles/standalone.dcdl#sha256:")
));
assert_eq!(
resolved.manifest.model.ref_.as_deref(),
Some("codex-oauth/gpt-6.1-sol")
);
assert!(resolved.manifest.feature.task.enabled);
assert!(resolved.manifest.feature.web.enabled);
assert!(resolved.manifest.feature.image.enabled);