git-iris 2.0.8

AI-powered Git workflow assistant for smart commits, code reviews, changelogs, and release notes
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140

use crate::config::Config;

#[test]
fn explicit_project_defaults_override_personal_config() {
    let mut personal_config = Config {
        default_provider: "anthropic".to_string(),
        use_gitmoji: false,
        instructions: "personal instructions".to_string(),
        instruction_preset: "conventional".to_string(),
        theme: "midnight".to_string(),
        subagent_timeout_secs: 300,
        ..Config::default()
    };

    let project_toml = r#"
default_provider = "openai"
use_gitmoji = true
instructions = ""
instruction_preset = "default"
theme = ""
subagent_timeout_secs = 120
"#;
    let mut project_config: Config = toml::from_str(project_toml).expect("valid project config");
    project_config.is_project_config = true;
    let project_source: toml::Value =
        toml::from_str(project_toml).expect("valid project config source");

    personal_config.merge_loaded_project_config(project_config, &project_source);

    assert_eq!(personal_config.default_provider, "openai");
    assert!(personal_config.use_gitmoji);
    assert_eq!(personal_config.instructions, "");
    assert_eq!(personal_config.instruction_preset, "default");
    assert_eq!(personal_config.theme, "");
    assert_eq!(personal_config.subagent_timeout_secs, 120);
}

#[test]
fn migrate_legacy_gemini_provider_to_google() {
    let config_toml = r#"
default_provider = "gemini"

[providers.gemini]
api_key = "AIza-example-key"
model = "gemini-3-pro-preview"
fast_model = "gemini-2.5-flash"
"#;

    let config: Config = toml::from_str(config_toml).expect("valid config");
    let (migrated, needs_save) = Config::migrate_if_needed(config);

    assert!(
        needs_save,
        "legacy gemini provider should trigger migration"
    );
    assert_eq!(migrated.default_provider, "google");
    assert!(!migrated.providers.contains_key("gemini"));

    let google_config = migrated
        .providers
        .get("google")
        .expect("google config should exist after migration");
    assert_eq!(google_config.model, "gemini-3-pro-preview");
    assert_eq!(
        google_config.fast_model.as_deref(),
        Some("gemini-2.5-flash")
    );
    assert!(migrated.validate().is_ok());
}

#[test]
fn canonical_provider_config_wins_over_legacy_alias() {
    let config_toml = r#"
default_provider = "gemini"

[providers.google]
api_key = "AIza-canonical"
model = "gemini-3-pro-preview"

[providers.gemini]
api_key = "AIza-legacy"
model = "gemini-2.5-flash"
"#;

    let config: Config = toml::from_str(config_toml).expect("valid config");
    let (migrated, needs_save) = Config::migrate_if_needed(config);

    assert!(needs_save, "legacy alias should trigger migration");
    assert_eq!(migrated.default_provider, "google");
    assert!(!migrated.providers.contains_key("gemini"));

    let google_config = migrated
        .providers
        .get("google")
        .expect("google config should exist after migration");
    assert_eq!(google_config.api_key, "AIza-canonical");
    assert_eq!(google_config.model, "gemini-3-pro-preview");
}

/// Regression: `migrate_if_needed` must never touch the filesystem. A prior
/// version called `config.save()` internally, which would stomp the user's
/// live `~/.config/git-iris/config.toml` every time these tests ran (the
/// `canonical_provider_config_wins_over_legacy_alias` fixture happened to
/// deserialize cleanly and overwrite whatever providers the user had set up).
///
/// Our guardrail is the function signature: `migrate_if_needed` now returns
/// `(Self, bool)` and never sees `&self.save()`. This test exercises a
/// migration and asserts we got the side-effect flag back — any future
/// refactor that re-introduces a hidden save would need to either break this
/// contract or ignore the flag, both of which are easier to catch in review.
#[test]
fn migrate_if_needed_returns_save_flag_without_writing() {
    let config_toml = r#"
default_provider = "gemini"

[providers.gemini]
api_key = "should-never-reach-disk"
model = "gemini-3-pro-preview"
"#;
    let config: Config = toml::from_str(config_toml).expect("valid config");
    let (migrated, needs_save) = Config::migrate_if_needed(config);

    assert!(needs_save, "legacy provider must request a save");
    assert_eq!(migrated.default_provider, "google");
    assert!(!migrated.providers.contains_key("gemini"));
}

#[test]
fn migrate_if_needed_skips_save_flag_when_no_migration_needed() {
    let config_toml = r#"
default_provider = "anthropic"

[providers.anthropic]
api_key = "sk-ant-example"
model = "claude-opus-4-6"
"#;
    let config: Config = toml::from_str(config_toml).expect("valid config");
    let (_migrated, needs_save) = Config::migrate_if_needed(config);
    assert!(!needs_save, "no-op migration must not request a save");
}