kaishin 0.6.3

Universal self-update library for Rust CLIs, extracted from rvpm and renri.
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
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
969
970
971
972
973
974
975
976
977
978
979
980
981
982
983
984
985
986
987
988
989
990
991
992
993
994
995
996
997
998
999
1000
1001
1002
1003
1004
1005
1006
1007
1008
1009
1010
1011
1012
1013
1014
1015
1016
1017
1018
1019
1020
1021
1022
1023
1024
1025
1026
1027
1028
1029
1030
1031
1032
1033
1034
1035
1036
1037
1038
1039
1040
1041
1042
1043
1044
1045
1046
1047
1048
1049
1050
1051
1052
1053
1054
1055
1056
1057
1058
1059
1060
1061
1062
1063
1064
1065
1066
1067
1068
1069
1070
1071
1072
1073
1074
1075
1076
1077
1078
1079
1080
1081
1082
1083
1084
1085
1086
1087
1088
1089
1090
1091
1092
1093
1094
1095
1096
1097
1098
1099
1100
1101
1102
1103
1104
1105
1106
1107
1108
1109
1110
1111
1112
1113
1114
1115
1116
1117
1118
1119
1120
1121
1122
1123
1124
1125
1126
1127
1128
1129
1130
1131
1132
1133
1134
1135
1136
1137
1138
1139
1140
1141
1142
1143
1144
1145
1146
1147
1148
1149
1150
1151
1152
1153
1154
1155
1156
1157
1158
1159
1160
1161
1162
1163
1164
1165
1166
1167
1168
1169
1170
1171
1172
1173
1174
1175
1176
1177
1178
1179
1180
1181
1182
1183
1184
1185
1186
1187
1188
1189
1190
1191
1192
1193
1194
1195
1196
1197
1198
1199
1200
1201
1202
1203
1204
1205
1206
1207
1208
1209
1210
1211
1212
1213
1214
1215
1216
1217
1218
1219
//! `kaishin` is a universal self-update library for Rust CLIs, extracted from `rvpm` and `renri`.
//!
//! It provides utilities to:
//! 1. Fetch the latest release information from GitHub.
//! 2. Detect how the current executable was installed (e.g., via `cargo install`).
//! 3. Perform a self-update by replacing the current binary.
//! 4. Manage background update check intervals to avoid frequent API calls via [`Checker`].
//! 5. Silently auto-update in the background — check, download, and replace the
//!    binary without prompting via [`Checker::auto_update`] /
//!    [`Checker::spawn_auto_update`]. The running process keeps the old binary;
//!    the new version takes effect on the next launch.

use anyhow::{Context, Result, anyhow};
use serde::{Deserialize, Serialize};
use std::path::{Path, PathBuf};
use std::time::{Duration, SystemTime};

/// Information about the latest release fetched from the GitHub API.
#[derive(Debug, Clone, Deserialize, Serialize, PartialEq, Eq)]
pub struct LatestRelease {
    /// The tag name of the release (e.g., `v3.31.4`).
    pub tag_name: String,
    /// The human-readable URL of the release page on GitHub.
    #[serde(default)]
    pub html_url: String,
}

/// The method by which the current executable was installed.
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
pub enum InstallMethod {
    /// Installed via `cargo install`. By default, updated by downloading the
    /// GitHub release binary; falls back to re-running `cargo install` if the
    /// download fails. See [`UpdateOptions::prefer_github_release`].
    CargoInstall,
    /// A development build found under a `target/` directory.
    DevBuild,
    /// A standalone binary.
    DirectBinary,
}

/// Persistent state for background update checks, used for throttling.
#[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Eq)]
pub struct UpdateCheckState {
    /// Unix timestamp of the last time a check was performed.
    pub last_checked_unix: u64,
    /// The tag name of the latest version found in the last check.
    pub last_known_latest: Option<String>,
    /// The URL of the latest version found in the last check.
    pub last_known_url: Option<String>,
}

/// Configuration options for `kaishin`.
#[derive(Debug, Clone)]
pub struct KaishinOptions {
    /// The GitHub owner (e.g., `yukimemi`).
    pub owner: String,
    /// The GitHub repository name (e.g., `rvpm`).
    pub repo: String,
    /// The binary name of the application (e.g., `rvpm`).
    ///
    /// Also drives self-update asset matching: GitHub-release assets must be
    /// named `<bin_name>-<target>.<ext>` (the pj-rust-cli kata layout) for the
    /// download path to single them out. Assets named otherwise won't match;
    /// only the interactive [`InstallMethod::CargoInstall`] flow may then fall
    /// back to `cargo install`, while `DirectBinary` and silent auto-update
    /// report failure / skip without rebuilding.
    pub bin_name: String,
    /// The current version of the application (usually `env!("CARGO_PKG_VERSION")`).
    pub current_version: String,
    /// The crates.io package name, when it differs from `bin_name`
    /// (e.g., package `kotonoha-server` ships the binary `kotonoha`).
    /// Used by the `cargo install` fallback; `None` means the package
    /// is named like the binary.
    pub crate_name: Option<String>,
}

impl KaishinOptions {
    /// Creates a new instance of `KaishinOptions`.
    pub fn new(owner: &str, repo: &str, bin_name: &str, current_version: &str) -> Self {
        Self {
            owner: owner.to_string(),
            repo: repo.to_string(),
            bin_name: bin_name.to_string(),
            current_version: current_version.to_string(),
            crate_name: None,
        }
    }

    /// Sets the crates.io package name for when it differs from the
    /// binary name (used by the `cargo install` fallback).
    pub fn crate_name(mut self, crate_name: &str) -> Self {
        self.crate_name = Some(crate_name.to_string());
        self
    }

    /// The package name to pass to `cargo install` — the explicit
    /// `crate_name` when set, the binary name otherwise.
    pub fn cargo_crate_name(&self) -> &str {
        self.crate_name.as_deref().unwrap_or(&self.bin_name)
    }
}

/// Options for the self-update process.
#[derive(Debug, Clone)]
pub struct UpdateOptions {
    /// Automatically answer "yes" to all prompts.
    pub yes: bool,
    /// Only check for updates and print status, don't perform the update.
    pub check_only: bool,
    /// Run in non-interactive mode. Bail if a prompt would be required and `yes` is false.
    pub non_interactive: bool,
    /// When the binary was installed via `cargo install`, prefer downloading the
    /// release binary from GitHub instead of running `cargo install` (which
    /// rebuilds from source and is slow). Defaults to `true`.
    ///
    /// If the GitHub release download fails (e.g., no matching asset for the
    /// current platform), the update falls back to `cargo install`. Set this
    /// to `false` to skip the GitHub release attempt entirely.
    pub prefer_github_release: bool,
}

impl Default for UpdateOptions {
    fn default() -> Self {
        Self {
            yes: false,
            check_only: false,
            non_interactive: false,
            prefer_github_release: true,
        }
    }
}

impl UpdateOptions {
    /// Creates a new instance of `UpdateOptions` with default values.
    pub fn new() -> Self {
        Self::default()
    }

    /// Sets the `yes` flag.
    pub fn yes(mut self, yes: bool) -> Self {
        self.yes = yes;
        self
    }

    /// Sets the `check_only` flag.
    pub fn check_only(mut self, check_only: bool) -> Self {
        self.check_only = check_only;
        self
    }

    /// Sets the `non_interactive` flag.
    pub fn non_interactive(mut self, non_interactive: bool) -> Self {
        self.non_interactive = non_interactive;
        self
    }

    /// Sets the `prefer_github_release` flag.
    pub fn prefer_github_release(mut self, prefer_github_release: bool) -> Self {
        self.prefer_github_release = prefer_github_release;
        self
    }
}

/// A high-level handler for managing background update checks.
///
/// It encapsulates JSON state persistence and update logic.
#[derive(Debug, Clone)]
pub struct Checker {
    opts: KaishinOptions,
    interval: Duration,
    state_path: PathBuf,
}

impl Checker {
    /// Creates a new `Checker` for the given application.
    ///
    /// By default, the state is stored in the system's data directory under `app_name`.
    pub fn new(app_name: &str, opts: KaishinOptions) -> Self {
        let state_path = default_state_path(app_name)
            .expect("failed to resolve default state path (dirs::data_dir() failed)");
        Self {
            opts,
            interval: Duration::from_secs(86400),
            state_path,
        }
    }

    /// Sets the interval between background update checks.
    pub fn interval(mut self, interval: Duration) -> Self {
        self.interval = interval;
        self
    }

    /// Sets a custom path for the persistent state file.
    pub fn state_path(mut self, path: impl Into<PathBuf>) -> Self {
        self.state_path = path.into();
        self
    }

    /// Determines if a background check should be performed now.
    pub fn should_check(&self) -> bool {
        let state = self.load_state();
        should_auto_check(state.as_ref(), self.interval, SystemTime::now())
    }

    /// Fetches the latest release, saves it to the state file, and returns the
    /// result *only when it actually outranks the running version*.
    ///
    /// The state file is updated regardless of the comparison result so the
    /// next [`Checker::should_check`] call throttles correctly. The return
    /// value mirrors [`Checker::cached_update`]: `Ok(Some(_))` if a newer
    /// release exists, `Ok(None)` if the running version is already up to date
    /// (or ahead), and `Err(_)` only if the GitHub request itself failed.
    ///
    /// Callers can therefore feed the return value straight into
    /// [`Checker::format_banner`] without a separate
    /// [`is_update_available`] check — the asymmetry that bit
    /// `renri`/`yui` in kaishin 0.3.x is gone.
    pub async fn check_and_save(&self) -> Result<Option<LatestRelease>> {
        let latest = check_latest_release(&self.opts).await?;
        let now_unix = SystemTime::now()
            .duration_since(SystemTime::UNIX_EPOCH)
            .map(|d| d.as_secs())
            .unwrap_or(0);
        let state = UpdateCheckState {
            last_checked_unix: now_unix,
            last_known_latest: Some(latest.tag_name.clone()),
            last_known_url: Some(latest.html_url.clone()),
        };
        let _ = self.save_state(&state);
        if is_update_available(&self.opts.current_version, &latest.tag_name)? {
            Ok(Some(latest))
        } else {
            Ok(None)
        }
    }

    /// Returns the cached latest release from the state file, if available and newer.
    pub fn cached_update(&self) -> Option<LatestRelease> {
        let state = self.load_state()?;
        let latest_tag = state.last_known_latest?;
        if is_update_available(&self.opts.current_version, &latest_tag).unwrap_or(false) {
            Some(LatestRelease {
                tag_name: latest_tag,
                html_url: state.last_known_url.unwrap_or_default(),
            })
        } else {
            None
        }
    }

    /// Formats an update banner for the given release.
    pub fn format_banner(&self, latest: &LatestRelease) -> String {
        format_update_banner(&self.opts, latest)
    }

    /// Silently checks for and installs a newer release in the background.
    ///
    /// This is the Claude-Code-style "auto-update" path: no prompts, no
    /// download progress, no stdout chatter. It is throttled by the same
    /// [`Checker::interval`] as [`Checker::should_check`], so it is safe to
    /// call unconditionally on every startup — it hits GitHub at most once per
    /// interval.
    ///
    /// Behaviour:
    /// - Returns `Ok(None)` immediately if the check interval hasn't elapsed,
    ///   or if another process already holds the update lock (see below).
    /// - Fetches the latest release and persists the check timestamp (so the
    ///   throttle advances even when nothing is installed).
    /// - If a newer release exists, replaces the binary **silently**:
    ///   - [`InstallMethod::DevBuild`] → skipped (returns `Ok(None)`); a dev
    ///     build is never clobbered.
    ///   - [`InstallMethod::CargoInstall`] and [`InstallMethod::DirectBinary`]
    ///     → download the matching GitHub release asset and self-replace. A
    ///     `cargo install` rebuild is **never** triggered on this path (it's
    ///     slow and noisy); if no release asset matches, the update simply
    ///     fails and is reported as `Err`.
    /// - On success returns `Ok(Some(release))` with the installed release.
    ///   The **running** process keeps the old binary in memory; the new
    ///   version takes effect on the next launch.
    ///
    /// Opt-out is intentionally left to the caller — decide whether to call
    /// this at all (e.g. gated behind your own env var or config) rather than
    /// relying on the library to read the environment.
    ///
    /// ## Concurrency
    ///
    /// An OS advisory lock on a lock file next to the state file serialises
    /// updates across processes so two concurrently-starting instances don't
    /// both self-replace the same binary (which races on Windows in
    /// particular). The loser returns `Ok(None)`. The kernel releases the lock
    /// when the holder exits — including on a crash — so a failed update can't
    /// disable auto-update permanently.
    ///
    /// ## Runtime
    ///
    /// The actual install runs on a detached OS thread with no Tokio context,
    /// for the same reason as [`run_self_update`] — `self_update`'s backend
    /// spins up (and drops) its own blocking runtime, which deadlocks inside an
    /// async executor.
    pub async fn auto_update(&self) -> Result<Option<LatestRelease>> {
        if !self.should_check() {
            return Ok(None);
        }

        // Serialise across processes; bail (not error) if someone else holds it.
        let lock_path = self.state_path.with_extension("lock");
        let Some(_lock) = UpdateLock::acquire(&lock_path) else {
            return Ok(None);
        };

        // Advances the throttle even when there's no newer release.
        let Some(latest) = self.check_and_save().await? else {
            return Ok(None);
        };

        let opts = self.opts.clone();
        let latest_for_thread = latest.clone();
        let (tx, rx) = tokio::sync::oneshot::channel();
        std::thread::Builder::new()
            .name(format!("{}-auto-update", opts.bin_name))
            .spawn(move || {
                let _ = tx.send(run_silent_update_blocking(&opts, &latest_for_thread));
            })
            .context("failed to spawn auto-update worker thread")?;
        let installed = rx
            .await
            .context("auto-update worker thread exited without reporting a result")??;

        Ok(installed.then_some(latest))
    }

    /// Fire-and-forget wrapper around [`Checker::auto_update`].
    ///
    /// Spawns a detached Tokio task and returns immediately so application
    /// startup isn't blocked on a network round-trip or a download. Any error
    /// (and the installed-version result) is discarded — use
    /// [`Checker::auto_update`] directly if you need to react to either.
    ///
    /// Must be called from within a Tokio runtime.
    pub fn spawn_auto_update(&self) {
        let this = self.clone();
        tokio::spawn(async move {
            let _ = this.auto_update().await;
        });
    }

    fn load_state(&self) -> Option<UpdateCheckState> {
        load_check_state(&self.state_path)
    }

    fn save_state(&self, state: &UpdateCheckState) -> Result<()> {
        save_check_state(&self.state_path, state)
    }
}

/// Returns the default interval between background update checks (24 hours).
pub fn default_interval() -> Duration {
    Duration::from_secs(86400)
}

/// Parses a duration string (e.g., "24h", "1d", "30m") into a `Duration`.
pub fn parse_interval(s: &str) -> Result<Duration> {
    Ok(humantime::parse_duration(s)?)
}

/// Detects the installation method of the executable at the given path.
pub fn detect_install_method(exe: &Path) -> InstallMethod {
    let s = exe.to_string_lossy().replace('\\', "/").to_lowercase();
    if s.contains("/target/debug/") || s.contains("/target/release/") {
        return InstallMethod::DevBuild;
    }
    let cargo_bin = std::env::var("CARGO_HOME")
        .ok()
        .map(PathBuf::from)
        .or_else(|| dirs::home_dir().map(|h| h.join(".cargo")))
        .map(|p| {
            p.join("bin")
                .to_string_lossy()
                .replace('\\', "/")
                .to_lowercase()
        });
    if let Some(bin) = cargo_bin {
        if s.starts_with(&format!("{}/", bin)) {
            return InstallMethod::CargoInstall;
        }
    }
    if s.contains("/.cargo/bin/") || s.contains("/cargo/bin/") {
        return InstallMethod::CargoInstall;
    }
    InstallMethod::DirectBinary
}

/// Compares the current version with a latest tag and returns `true` if an update is available.
pub fn is_update_available(current: &str, latest_tag: &str) -> Result<bool> {
    let cur = semver::Version::parse(current)
        .map_err(|e| anyhow!("invalid current version `{}`: {}", current, e))?;
    let lat_str = latest_tag.trim_start_matches('v');
    let lat = semver::Version::parse(lat_str)
        .map_err(|e| anyhow!("invalid latest tag `{}`: {}", latest_tag, e))?;
    Ok(lat > cur)
}

/// Fetches the latest release information for the repository specified in `opts` from GitHub.
pub async fn check_latest_release(opts: &KaishinOptions) -> Result<LatestRelease> {
    let url = format!(
        "https://api.github.com/repos/{}/{}/releases/latest",
        opts.owner, opts.repo
    );
    let client = reqwest::Client::builder()
        .user_agent(format!("{}/{}", opts.bin_name, opts.current_version))
        .timeout(Duration::from_secs(5))
        .build()?;
    let res = client.get(url).send().await?;
    if !res.status().is_success() {
        return Err(anyhow!("GitHub releases API returned {}", res.status()));
    }
    let release: LatestRelease = res.json().await?;
    Ok(release)
}

/// Returns the default path for the state file under the system's data directory.
pub fn default_state_path(app_name: &str) -> Option<PathBuf> {
    dirs::data_dir().map(|d| d.join(app_name).join("last_update_check.json"))
}

/// Loads the persistent update check state from the given path.
pub fn load_check_state(path: &Path) -> Option<UpdateCheckState> {
    let content = std::fs::read_to_string(path).ok()?;
    serde_json::from_str(&content).ok()
}

/// Saves the persistent update check state to the given path.
pub fn save_check_state(path: &Path, state: &UpdateCheckState) -> Result<()> {
    if let Some(parent) = path.parent() {
        std::fs::create_dir_all(parent)?;
        let json = serde_json::to_string(state)?;
        let mut tmp = tempfile::NamedTempFile::new_in(parent)?;
        use std::io::Write;
        tmp.write_all(json.as_bytes())?;
        tmp.persist(path)?;
    }
    Ok(())
}

/// Determines whether an automatic update check should be performed based on the interval.
pub fn should_auto_check(
    state: Option<&UpdateCheckState>,
    interval: Duration,
    now: SystemTime,
) -> bool {
    let Some(state) = state else {
        return true;
    };
    let Ok(now_unix) = now.duration_since(SystemTime::UNIX_EPOCH) else {
        return true;
    };
    let elapsed = now_unix.as_secs().saturating_sub(state.last_checked_unix);
    elapsed >= interval.as_secs()
}

/// Formats a banner message intended for display when an update is available.
pub fn format_update_banner(opts: &KaishinOptions, latest: &LatestRelease) -> String {
    let tag = latest.tag_name.trim_start_matches('v');
    let mut s = format!(
        "\u{2699} {} {} available (current {}) — run `{} self-update` to upgrade",
        opts.bin_name, tag, opts.current_version, opts.bin_name
    );
    if !latest.html_url.is_empty() {
        s.push_str(&format!("\n  release notes: {}", latest.html_url));
    }
    s
}

/// Executes the self-update flow.
///
/// 1. Fetches the latest release from GitHub.
/// 2. Compares versions.
/// 3. If [`UpdateOptions::check_only`] is true, prints status and returns.
/// 4. If [`UpdateOptions::non_interactive`] is true and [`UpdateOptions::yes`] is false, bails if an update is available.
/// 5. Prompts the user (if [`UpdateOptions::yes`] is false and terminal is interactive).
/// 6. Detects the installation method and performs the update accordingly:
///    - `DevBuild`: errors out (refuses to overwrite a development binary).
///    - `CargoInstall`: if [`UpdateOptions::prefer_github_release`] is `true`
///      (the default), downloads the release binary from GitHub and only falls
///      back to `cargo install` when the GitHub release path fails. If `false`,
///      runs `cargo install` directly.
///    - `DirectBinary`: downloads the matching binary from the GitHub release.
pub async fn run_self_update(opts: &KaishinOptions, upd_opts: UpdateOptions) -> Result<()> {
    let latest = check_latest_release(opts)
        .await
        .context("failed to fetch latest release from GitHub")?;

    let available = is_update_available(&opts.current_version, &latest.tag_name)?;
    if !available {
        println!(
            "\u{2713} {} {} is already up to date.",
            opts.bin_name, opts.current_version
        );
        return Ok(());
    }

    let latest_clean = latest.tag_name.trim_start_matches('v');
    if upd_opts.check_only {
        println!(
            "\u{2699} {} {} available (current {}). Run `{} self-update` to install.",
            opts.bin_name, latest_clean, opts.current_version, opts.bin_name
        );
        if !latest.html_url.is_empty() {
            println!("  release notes: {}", latest.html_url);
        }
        return Ok(());
    }

    // The confirmation prompt and the install step both perform synchronous,
    // blocking I/O. In particular the `self_update` backend creates — and then
    // drops — its own blocking Tokio runtime. Doing that anywhere inside the
    // caller's async runtime deadlocks ("Cannot start a runtime from within a
    // runtime"); even a `spawn_blocking` pool thread is still runtime-attached
    // and instead trips "Cannot drop a runtime in a context where blocking is
    // not allowed". This is exactly what froze `shoka`/`renri` mid-update.
    //
    // Run the whole blocking tail on a detached OS thread that has *no* Tokio
    // context at all, and await its result over a oneshot channel. Works on
    // both multi-thread and current-thread runtimes.
    let opts = opts.clone();
    let (tx, rx) = tokio::sync::oneshot::channel();
    std::thread::Builder::new()
        .name(format!("{}-self-update", opts.bin_name))
        .spawn(move || {
            let _ = tx.send(run_self_update_blocking(&opts, &latest, upd_opts));
        })
        .context("failed to spawn self-update worker thread")?;
    rx.await
        .context("self-update worker thread exited without reporting a result")?
}

/// Windows console-mode guard for the confirmation prompt.
///
/// `read_line` on Windows reads the console via `ReadConsoleW`, which only
/// returns on Enter (and only echoes / honours Ctrl-C as a signal) when the
/// console input is in cooked mode — `ENABLE_LINE_INPUT | ENABLE_ECHO_INPUT |
/// ENABLE_PROCESSED_INPUT`. A program that put the console into raw mode
/// (e.g. a crossterm/ratatui TUI) and exited without restoring it leaves
/// those flags cleared *for the whole console*, so a later `self-update` in
/// the same terminal hangs at `[y/N]` with no echo and an inert Ctrl-C. We
/// can't trust the inherited mode, so set cooked input for the read and
/// restore the previous mode on drop.
#[cfg(windows)]
mod cooked_input {
    use windows_sys::Win32::Foundation::HANDLE;
    use windows_sys::Win32::System::Console::{
        CONSOLE_MODE, ENABLE_ECHO_INPUT, ENABLE_LINE_INPUT, ENABLE_PROCESSED_INPUT, GetConsoleMode,
        GetStdHandle, STD_INPUT_HANDLE, SetConsoleMode,
    };

    const COOKED: CONSOLE_MODE = ENABLE_PROCESSED_INPUT | ENABLE_LINE_INPUT | ENABLE_ECHO_INPUT;

    /// Restores the previous console input mode on drop, but only if `guard`
    /// actually changed it.
    pub(crate) struct CookedInput {
        handle: HANDLE,
        prev: CONSOLE_MODE,
        restore: bool,
    }

    impl CookedInput {
        pub(crate) fn guard() -> Self {
            // SAFETY: plain FFI calls into the Win32 console API with a
            // borrowed std handle; no memory is aliased or freed.
            unsafe {
                let handle = GetStdHandle(STD_INPUT_HANDLE);
                let mut prev: CONSOLE_MODE = 0;
                // `GetConsoleMode` fails when stdin is redirected (a pipe or
                // file) — there's no console mode to repair, and the
                // non-interactive guard upstream already covers that path.
                if handle.is_null() || GetConsoleMode(handle, &mut prev) == 0 {
                    return Self {
                        handle,
                        prev: 0,
                        restore: false,
                    };
                }
                let cooked = prev | COOKED;
                // Only touch (and later restore) the mode when something was
                // actually cleared, so the common healthy case is a no-op.
                if cooked != prev && SetConsoleMode(handle, cooked) != 0 {
                    Self {
                        handle,
                        prev,
                        restore: true,
                    }
                } else {
                    Self {
                        handle,
                        prev,
                        restore: false,
                    }
                }
            }
        }
    }

    impl Drop for CookedInput {
        fn drop(&mut self) {
            if self.restore {
                // Best-effort: nothing actionable if the restore fails.
                // SAFETY: same borrowed console handle as in `guard`.
                unsafe {
                    SetConsoleMode(self.handle, self.prev);
                }
            }
        }
    }
}

/// The synchronous, blocking tail of [`run_self_update`]: prompt the user and
/// perform the actual install. Must run off the async executor (see the call
/// site) because `self_update`'s backend blocks on its own runtime.
fn run_self_update_blocking(
    opts: &KaishinOptions,
    latest: &LatestRelease,
    upd_opts: UpdateOptions,
) -> Result<()> {
    let latest_clean = latest.tag_name.trim_start_matches('v');

    if !upd_opts.yes {
        use std::io::IsTerminal;
        if upd_opts.non_interactive || !std::io::stdin().is_terminal() {
            anyhow::bail!(
                "non-interactive mode: use `--yes` to proceed with update to v{}",
                latest_clean
            );
        }

        eprint!("Update to v{}? [y/N] ", latest_clean);
        use std::io::Write;
        std::io::stderr().flush().ok();
        // A full-screen TUI (ratatui/crossterm, …) run earlier in the same
        // console can exit without restoring cooked mode, leaving stdin's
        // console with ENABLE_LINE_INPUT / ENABLE_ECHO_INPUT /
        // ENABLE_PROCESSED_INPUT cleared. `read_line` then blocks forever on
        // an Enter that never terminates the line, and Ctrl-C arrives as a
        // raw 0x03 byte instead of a signal — the "frozen at [y/N]" hang.
        // Force a sane mode for the read and restore it after; no-op off
        // Windows and when stdin isn't a console.
        #[cfg(windows)]
        let _cooked_input = cooked_input::CookedInput::guard();
        let mut answer = String::new();
        std::io::stdin().read_line(&mut answer)?;
        let answer = answer.trim().to_ascii_lowercase();
        if answer != "y" && answer != "yes" {
            eprintln!("aborted.");
            return Ok(());
        }
    }

    let exe = std::env::current_exe().context("failed to resolve current_exe()")?;
    let method = detect_install_method(&exe);
    match method {
        InstallMethod::DevBuild => {
            return Err(anyhow!(
                "\u{26a0} `{}` looks like a development build. Refusing to self-update.",
                exe.display()
            ));
        }
        InstallMethod::CargoInstall => {
            if upd_opts.prefer_github_release {
                match update_via_github_release(opts, latest, true) {
                    Ok(()) => return Ok(()),
                    Err(e) => {
                        eprintln!(
                            "GitHub release download failed: {e:#}. Falling back to `cargo install`."
                        );
                    }
                }
            }
            update_via_cargo_install(opts, latest_clean)?;
        }
        InstallMethod::DirectBinary => {
            update_via_github_release(opts, latest, true)?;
        }
    }
    Ok(())
}

/// The silent, blocking tail of [`Checker::auto_update`]: install the newer
/// release without any prompt or stdout output. Must run off the async
/// executor (see the call site) for the same reason as
/// [`run_self_update_blocking`].
///
/// Returns `Ok(true)` if the binary was replaced, `Ok(false)` if the install
/// was skipped because the running binary is a development build. Unlike the
/// interactive [`InstallMethod::CargoInstall`] path, a `cargo install` rebuild
/// is never attempted here: auto-update sticks to the GitHub release asset and
/// reports `Err` if none matches.
fn run_silent_update_blocking(opts: &KaishinOptions, latest: &LatestRelease) -> Result<bool> {
    let exe = std::env::current_exe().context("failed to resolve current_exe()")?;
    match detect_install_method(&exe) {
        // Never clobber a dev build, and never surface it as an error on the
        // background path — just decline.
        InstallMethod::DevBuild => Ok(false),
        InstallMethod::CargoInstall | InstallMethod::DirectBinary => {
            update_via_github_release(opts, latest, false)?;
            Ok(true)
        }
    }
}

/// Cross-process guard so two concurrently-starting instances don't both
/// self-replace the same binary. Acquisition is fail-open: if the lock can't
/// be taken, the caller simply skips the update this round.
///
/// Backed by an OS advisory file lock (`fs2`) on the lock file rather than a
/// hand-rolled "create_new + timestamp" scheme. The kernel releases the lock
/// automatically when the holder's file handle is closed —
/// including on a crash or kill — so there is no orphaned-lock state to reclaim
/// and no time-of-check/time-of-use window between a staleness check and the
/// acquire. The lock file itself is left in place (empty); its mere existence
/// means nothing, only the live advisory lock does.
struct UpdateLock {
    // Held only for its `Drop`, which closes the handle and releases the OS
    // lock. Never read directly.
    _file: std::fs::File,
}

impl UpdateLock {
    /// Tries to take the advisory lock at `path`. Returns `None` if another
    /// process currently holds it (the caller skips the update this round) or
    /// if the lock file can't be opened.
    fn acquire(path: &Path) -> Option<Self> {
        if let Some(parent) = path.parent() {
            let _ = std::fs::create_dir_all(parent);
        }
        let file = std::fs::OpenOptions::new()
            .create(true)
            .read(true)
            .write(true)
            .truncate(false)
            .open(path)
            .ok()?;
        // `try_lock_exclusive` is non-blocking: `Ok(())` means we hold the
        // exclusive advisory lock; any error (would-block or otherwise) means
        // we don't, so we decline rather than wait.
        use fs2::FileExt;
        match file.try_lock_exclusive() {
            Ok(()) => Some(Self { _file: file }),
            Err(_) => None,
        }
    }
}

fn update_via_cargo_install(opts: &KaishinOptions, latest_clean: &str) -> Result<()> {
    let tmp = tempfile::Builder::new()
        .prefix(&format!("{}-self-update-", opts.bin_name))
        .tempdir()?;
    let tmp_root = tmp.path().to_path_buf();
    // The cargo *package* name can differ from the binary name (e.g.
    // package `kotonoha-server`, binary `kotonoha`). The package name
    // goes to `cargo install`; the binary lookup below stays on
    // `bin_name`, because that's what cargo puts under `<root>/bin/`.
    println!(
        "running: cargo install {} --version {} --locked --force --root {}",
        opts.cargo_crate_name(),
        latest_clean,
        tmp_root.display()
    );
    let status = std::process::Command::new("cargo")
        .arg("install")
        .arg(opts.cargo_crate_name())
        .arg("--version")
        .arg(latest_clean)
        .arg("--locked")
        .arg("--force")
        .arg("--root")
        .arg(&tmp_root)
        .status()?;
    if !status.success() {
        anyhow::bail!("cargo install failed");
    }
    let bin_exe_name = if cfg!(windows) {
        format!("{}.exe", opts.bin_name)
    } else {
        opts.bin_name.clone()
    };
    let new_exe = tmp_root.join("bin").join(bin_exe_name);
    self_update::self_replace::self_replace(&new_exe)?;
    println!("\u{2713} {} v{} installed.", opts.bin_name, latest_clean);
    Ok(())
}

fn update_via_github_release(
    opts: &KaishinOptions,
    latest: &LatestRelease,
    show_progress: bool,
) -> Result<()> {
    // Try the compiled-in target first (self_update picks it up automatically).
    let err = match try_github_release_with_target(opts, latest, None, show_progress) {
        Ok(()) => return Ok(()),
        Err(e) => e,
    };

    // On Linux x86_64, try the alternate libc ABI (musl↔gnu) so that releases
    // can migrate between them without breaking self-update.
    #[cfg(all(target_os = "linux", target_arch = "x86_64"))]
    {
        if cfg!(target_env = "musl") {
            // musl binary falling back to gnu: only safe when glibc is present
            // on this system. On musl-only hosts (Alpine, etc.) a gnu binary
            // requires the glibc dynamic linker and won't execute.
            let has_glibc = std::path::Path::new("/lib/x86_64-linux-gnu/libc.so.6").exists()
                || std::path::Path::new("/lib64/ld-linux-x86-64.so.2").exists()
                || std::path::Path::new("/lib/ld-linux-x86-64.so.2").exists();
            if has_glibc {
                if let Ok(()) = try_github_release_with_target(
                    opts,
                    latest,
                    Some("x86_64-unknown-linux-gnu"),
                    show_progress,
                ) {
                    return Ok(());
                }
            }
        } else {
            // gnu binary falling back to musl: always safe — musl static
            // binaries carry their own libc and run on any Linux kernel.
            if let Ok(()) = try_github_release_with_target(
                opts,
                latest,
                Some("x86_64-unknown-linux-musl"),
                show_progress,
            ) {
                return Ok(());
            }
        }
    }

    // On Windows x86_64, try the alternate ABI (gnu↔msvc) for the same reason.
    // Most releases ship msvc; users who installed via cargo with the GNU
    // toolchain get a gnu binary that can still upgrade to an msvc release.
    #[cfg(all(target_os = "windows", target_arch = "x86_64"))]
    {
        let alt = if cfg!(target_env = "gnu") {
            "x86_64-pc-windows-msvc"
        } else {
            "x86_64-pc-windows-gnu"
        };
        if let Ok(()) = try_github_release_with_target(opts, latest, Some(alt), show_progress) {
            return Ok(());
        }
    }

    Err(err)
}

/// In-archive binary paths to try, in order. Release archives built by
/// the kata `pj-rust-cli` template contain the binary renamed with the
/// target-triple suffix (`<bin>-<target>[.exe]`, matching the archive
/// stem); older / hand-rolled releases ship the plain `<bin>[.exe]`.
///
/// These are *fully resolved literals*, deliberately **not** `self_update`'s
/// `{{ bin }}` / `{{ target }}` templates. `self_update`'s `bin_name()` forces
/// the platform exe suffix onto the name (`kanade` → `kanade.exe` on Windows)
/// and then expands `{{ bin }}` to that suffixed value — so
/// `{{ bin }}-{{ target }}.exe` expands to the never-matching
/// `kanade.exe-<target>.exe`, leaving every Windows self-update to fail
/// extraction and fall back to `cargo install`. Building the path ourselves
/// from the raw `bin_name` + the effective target sidesteps the suffix entirely
/// (`EXE_SUFFIX` is `.exe` on Windows, empty elsewhere). The alternate-ABI
/// retries only ever swap to a same-OS target, so the host `EXE_SUFFIX` always
/// matches the override target's OS.
///
/// Each failed attempt re-downloads the asset, so the common layout
/// (kata) goes first.
///
/// `bin_name` is expected already-bare (see [`strip_exe_suffix`]); the
/// host `EXE_SUFFIX` is the only suffix appended.
fn bin_path_in_archive_candidates(bin_name: &str, target: &str) -> [String; 2] {
    let exe = std::env::consts::EXE_SUFFIX;
    [
        format!("{bin_name}-{target}{exe}"),
        format!("{bin_name}{exe}"),
    ]
}

/// Strip a trailing platform exe suffix from `bin_name` (a no-op when `exe`
/// is empty, i.e. off Windows). A caller that baked `.exe` into its
/// configured `bin_name` would otherwise double it — `kanade.exe-<target>.exe`
/// for the archive path, `kanade.exe-<target>` for the asset identifier —
/// neither of which matches the kata layout. `exe` is a parameter rather than
/// read inline so the Windows behaviour is unit-testable on non-Windows CI.
fn strip_exe_suffix<'a>(bin_name: &'a str, exe: &str) -> &'a str {
    if exe.is_empty() {
        bin_name
    } else {
        bin_name.strip_suffix(exe).unwrap_or(bin_name)
    }
}

fn try_github_release_with_target(
    opts: &KaishinOptions,
    latest: &LatestRelease,
    target_override: Option<&str>,
    show_progress: bool,
) -> Result<()> {
    // The effective target must mirror what self_update itself uses: the
    // override when set, otherwise its compiled-in target.
    let effective_target = target_override.unwrap_or_else(|| self_update::get_target());
    // self_update force-appends EXE_SUFFIX to bin_name internally; if a caller
    // also baked `.exe` into `opts.bin_name`, strip it here so neither the
    // identifier nor the in-archive path doubles the suffix.
    let bin_name = strip_exe_suffix(&opts.bin_name, std::env::consts::EXE_SUFFIX);
    // Disambiguate sibling binaries with an `<bin>-<target>` identifier (see
    // `asset_identifier`); both the identifier and the in-archive paths key off
    // the same effective target.
    let identifier = asset_identifier(bin_name, effective_target);
    let mut last_err: Option<anyhow::Error> = None;
    for bin_path in bin_path_in_archive_candidates(bin_name, effective_target) {
        match try_github_release_once(
            opts,
            latest,
            target_override,
            show_progress,
            &identifier,
            &bin_path,
        ) {
            Ok(()) => return Ok(()),
            Err(e) => last_err = Some(e),
        }
    }
    // The loop always runs at least once, so `last_err` is Some here.
    Err(last_err.unwrap_or_else(|| anyhow!("no bin-path-in-archive candidates")))
}

/// The `identifier` handed to `self_update` to disambiguate which asset to
/// download when a release ships several binaries from the same crate.
///
/// `self_update` `contains`-matches this against each asset name, so anchoring
/// the binary name directly to the target triple (`<bin>-<target>`) is what
/// separates `kanade-<target>.zip` from a sibling `kanade-agent-<target>.zip`:
/// the latter contains `kanade` and the target, but not `kanade-<target>`.
fn asset_identifier(bin_name: &str, target: &str) -> String {
    format!("{bin_name}-{target}")
}

fn try_github_release_once(
    opts: &KaishinOptions,
    latest: &LatestRelease,
    target_override: Option<&str>,
    show_progress: bool,
    identifier: &str,
    bin_path_in_archive: &str,
) -> Result<()> {
    let mut builder = self_update::backends::github::Update::configure();
    builder
        .repo_owner(&opts.owner)
        .repo_name(&opts.repo)
        .bin_name(&opts.bin_name)
        .identifier(identifier)
        .bin_path_in_archive(bin_path_in_archive)
        .show_download_progress(show_progress)
        // `show_output` (defaults true) controls self_update's own status
        // chatter ("Checking latest version...", "Downloading...", …),
        // independent of the progress bar above. Keep it in sync so the silent
        // background auto-update path stays quiet — without this a plain
        // `shoka cd` leaks a full self-update transcript mid-command.
        .show_output(show_progress)
        .current_version(&opts.current_version)
        .target_version_tag(&latest.tag_name)
        .no_confirm(true);
    if let Some(t) = target_override {
        builder.target(t);
    }
    let status = builder
        .build()
        .context("build")?
        .update()
        .context("update")?;
    // Stay silent on the background auto-update path (`show_progress == false`);
    // the interactive flows keep their confirmation output.
    if show_progress {
        match status {
            self_update::Status::UpToDate(v) => {
                println!("\u{2713} {} {} is already up to date.", opts.bin_name, v)
            }
            self_update::Status::Updated(v) => {
                println!("\u{2713} {} v{} installed.", opts.bin_name, v)
            }
        }
    }
    Ok(())
}

#[cfg(test)]
mod tests {
    use super::*;
    use tempfile::tempdir;

    #[test]
    fn test_detect_install_method() {
        let p = PathBuf::from("/home/u/.cargo/bin/kaishin");
        assert_eq!(detect_install_method(&p), InstallMethod::CargoInstall);

        let p = PathBuf::from(
            r"C:\Users\yukimemi\src\github.com\yukimemi\kaishin\target\debug\kaishin.exe",
        );
        assert_eq!(detect_install_method(&p), InstallMethod::DevBuild);

        let p = PathBuf::from("/opt/kaishin-bin/kaishin");
        assert_eq!(detect_install_method(&p), InstallMethod::DirectBinary);
    }

    #[test]
    fn test_is_update_available() {
        assert!(is_update_available("0.1.0", "v0.1.1").unwrap());
        assert!(!is_update_available("0.1.1", "v0.1.1").unwrap());
        assert!(!is_update_available("0.1.2", "v0.1.1").unwrap());
        // No 'v' prefix
        assert!(is_update_available("0.1.0", "0.1.1").unwrap());
    }

    #[test]
    fn test_cargo_crate_name_defaults_to_bin_name() {
        let opts = KaishinOptions::new("u", "r", "app", "1.0.0");
        assert_eq!(opts.cargo_crate_name(), "app");
    }

    #[test]
    fn test_cargo_crate_name_uses_explicit_crate_name() {
        let opts = KaishinOptions::new("yukimemi", "kotonoha", "kotonoha", "0.3.0")
            .crate_name("kotonoha-server");
        assert_eq!(opts.cargo_crate_name(), "kotonoha-server");
        // The binary name is untouched — `<root>/bin/<bin_name>` lookup
        // after `cargo install` must keep using it.
        assert_eq!(opts.bin_name, "kotonoha");
    }

    #[test]
    fn test_asset_identifier_disambiguates_sibling_binaries() {
        // Regression for the kanade self-update falling back to `cargo install`:
        // self_update `contains`-matches the identifier against asset names, so
        // the identifier must single out `<bin>-<target>` over a sibling
        // `<bin>-agent-<target>` that also contains the bare bin name + target.
        let target = "x86_64-pc-windows-msvc";
        let id = asset_identifier("kanade", target);

        let right = format!("kanade-{target}.zip");
        let sibling = format!("kanade-agent-{target}.zip");

        // self_update keeps assets whose name `contains` the id (which itself
        // embeds the target, so `contains(target)` is implied).
        assert!(right.contains(&id));
        assert!(sibling.contains(target));
        assert!(
            !sibling.contains(&id),
            "sibling `{sibling}` must not match identifier `{id}`"
        );
    }

    #[test]
    fn test_bin_path_candidates_try_kata_layout_first() {
        let target = "x86_64-pc-windows-msvc";
        let [first, second] = bin_path_in_archive_candidates("kanade", target);
        let exe = std::env::consts::EXE_SUFFIX;

        // Target-suffixed (kata pj-rust-cli layout) before plain, both fully
        // resolved literals — no `{{ … }}` templates leak through.
        assert_eq!(first, format!("kanade-{target}{exe}"));
        assert_eq!(second, format!("kanade{exe}"));
        assert!(!first.contains("{{") && !second.contains("{{"));
    }

    #[test]
    fn test_strip_exe_suffix_prevents_double_suffix() {
        // Platform-independent regression for the Windows self-update fallback
        // to `cargo install`: feeding the *bare* name through the candidate
        // builder must produce `kanade-<target>.exe`, never the doubled-suffix
        // `kanade.exe-<target>.exe`. Driven with an explicit `.exe` so the
        // Windows path is provably exercised even on Linux CI runners (where
        // `EXE_SUFFIX` is empty and `cfg!(windows)` would skip it).
        let target = "x86_64-pc-windows-msvc";

        // A caller-baked `.exe` is stripped back to the bare name…
        assert_eq!(strip_exe_suffix("kanade.exe", ".exe"), "kanade");
        assert_eq!(strip_exe_suffix("kanade", ".exe"), "kanade");
        // …and an empty suffix (Unix) is a no-op.
        assert_eq!(strip_exe_suffix("kanade", ""), "kanade");
        assert_eq!(strip_exe_suffix("kanade.exe", ""), "kanade.exe");

        // The doubled-suffix path must never be produced from a bare name.
        let bare = strip_exe_suffix("kanade.exe", ".exe");
        let kata = format!("{bare}-{target}.exe");
        assert_eq!(kata, format!("kanade-{target}.exe"));
        assert!(!kata.contains(".exe-"), "must not embed `.exe` mid-path");
    }

    #[test]
    fn test_should_auto_check() {
        let now = SystemTime::now();
        let now_unix = now
            .duration_since(SystemTime::UNIX_EPOCH)
            .unwrap()
            .as_secs();

        // No state
        assert!(should_auto_check(None, Duration::from_secs(86400), now));

        // Recent state
        let state = UpdateCheckState {
            last_checked_unix: now_unix - 3600,
            last_known_latest: None,
            last_known_url: None,
        };
        assert!(!should_auto_check(
            Some(&state),
            Duration::from_secs(86400),
            now
        ));

        // Old state
        let state = UpdateCheckState {
            last_checked_unix: now_unix - 100000,
            last_known_latest: None,
            last_known_url: None,
        };
        assert!(should_auto_check(
            Some(&state),
            Duration::from_secs(86400),
            now
        ));
    }

    #[test]
    fn test_update_options_defaults() {
        let opts = UpdateOptions::new();
        assert!(!opts.yes);
        assert!(!opts.check_only);
        assert!(!opts.non_interactive);
        assert!(opts.prefer_github_release);
    }

    #[test]
    fn test_update_options_prefer_github_release_builder() {
        let opts = UpdateOptions::new().prefer_github_release(false);
        assert!(!opts.prefer_github_release);
        let opts = opts.prefer_github_release(true);
        assert!(opts.prefer_github_release);
    }

    #[test]
    fn test_format_update_banner() {
        let opts = KaishinOptions::new("u", "r", "app", "1.0.0");
        let release = LatestRelease {
            tag_name: "v1.1.0".to_string(),
            html_url: "https://example.com".to_string(),
        };
        let banner = format_update_banner(&opts, &release);
        assert!(banner.contains("app 1.1.0 available"));
        assert!(banner.contains("(current 1.0.0)"));
        assert!(banner.contains("run `app self-update`"));
        assert!(banner.contains("https://example.com"));
    }

    #[test]
    fn test_checker_state_management() {
        let tmp = tempdir().unwrap();
        let state_path = tmp.path().join("state.json");
        let opts = KaishinOptions::new("u", "r", "app", "1.0.0");
        let checker = Checker::new("app", opts).state_path(&state_path);

        // Initial state
        assert!(checker.should_check());
        assert!(checker.cached_update().is_none());

        // Save state manually for testing
        let now_unix = SystemTime::now()
            .duration_since(SystemTime::UNIX_EPOCH)
            .unwrap()
            .as_secs();
        let state = UpdateCheckState {
            last_checked_unix: now_unix,
            last_known_latest: Some("v1.2.0".to_string()),
            last_known_url: Some("https://rel".to_string()),
        };
        checker.save_state(&state).unwrap();

        // Check again
        assert!(!checker.should_check()); // within 24h
        let cached = checker.cached_update().unwrap();
        assert_eq!(cached.tag_name, "v1.2.0");
        assert_eq!(cached.html_url, "https://rel");

        // Test banner from checker
        let banner = checker.format_banner(&cached);
        assert!(banner.contains("app 1.2.0 available"));
    }

    #[test]
    fn test_update_lock_mutual_exclusion() {
        let tmp = tempdir().unwrap();
        let path = tmp.path().join("nested").join("update.lock");

        // First acquire wins and creates the (nested) lock file.
        let lock = UpdateLock::acquire(&path).expect("first acquire should win");
        assert!(path.exists());

        // A second acquire from an independent handle is refused while the
        // advisory lock is held (flock/LockFileEx conflict across handles).
        assert!(UpdateLock::acquire(&path).is_none());

        // Dropping the guard closes the handle, so the OS releases the lock...
        drop(lock);

        // ...and a later acquire succeeds again. The lock file itself is left
        // in place; only the live advisory lock gates acquisition.
        let lock = UpdateLock::acquire(&path).expect("acquire after release");
        drop(lock);
    }
}