Embed default config and attempt to fix tests#29
Conversation
Signed-off-by: AlexMikhalev <alex@metacortex.engineer>
Signed-off-by: AlexMikhalev <alex@metacortex.engineer>
| } else { | ||
| log::warn!("File does not exist"); | ||
| std::fs::copy("default/settings.toml", &filename)?; | ||
| // FIXME: the logic of settings that each app - desktop or server have their own default settings.toml |
There was a problem hiding this comment.
@mre I want your opinion on how to design it better - see FIXME: comment.
There was a problem hiding this comment.
I think we could have an environment variable for the config path that could be overwritten.
Then we could use it here.
TERRAPHIM_CONFIG_PATH=...It could be set to a sane default.
Just an idea.
There was a problem hiding this comment.
What is the path variable for default cargo run and cargo test? We can put the default config into that path, and it will pick it up automatically for run and test, but then subfolders in the Axum server and desktop can have their own when run.
mre
left a comment
mre
left a comment
There was a problem hiding this comment.
lgtm.
Good work with solid improvements. I vote for merging that and iterating on it from the main branch.
…erTier enum Add provider, fallback_provider, fallback_model, and provider_tier fields to AgentDefinition for subscription-based model routing (ADR-002, ADR-003). Add ProviderTier enum (Quick/Deep/Implementation/Oracle) with per-tier timeout values. Add opencode CLI support in spawner arg inference. All new fields are Optional with serde(default) for backward compatibility. Fixes #28 Refs #29 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2 participants
All tests are running using earthly +pipeline on earthly satellite.
Strangely, create article API tests are failing - the server can't add articles to the index.
I also changed default config - as a quick fix, we need to think how to make it in a sane way.