thirdshift 0.16.3

Turn a GitHub issue into a ready-for-review pull request, unattended
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

//! Prefer a Harness's stream metadata; otherwise read its CLI version once
//! per owning Command. The locked cache is shared by parallel child Runs.

use std::fs::{File, OpenOptions};
use std::io::{self, Read, Seek, SeekFrom, Write};
use std::path::{Path, PathBuf};
use std::process::Command;

use serde_json::Value;

use super::{Adapter, Harness, process};

/// Reuse detection from the owning Command, without querying the CLI again.
/// Missing or unreadable evidence costs attribution, never Delivery.
pub(crate) fn detected(harness: Harness, command_log: Option<&Path>) -> String {
    let read = || -> io::Result<String> {
        let path = cache_path(command_log.ok_or(io::ErrorKind::NotFound)?, harness.name());
        let mut file = File::open(path)?;
        file.lock_shared()?;
        let mut version = String::new();
        file.read_to_string(&mut version)?;
        Ok(version)
    };
    read()
        .ok()
        .and_then(|version| nonempty(&version).map(str::to_owned))
        .unwrap_or_else(|| "unknown".into())
}

fn cache_path(command_log: &Path, harness: &str) -> PathBuf {
    command_log.with_extension(format!("harness-{harness}.version"))
}

pub(crate) struct Version {
    adapter: &'static dyn Adapter,
    cache: Option<PathBuf>,
    command: Command,
    current: Option<String>,
}

impl Version {
    pub fn new(
        adapter: &'static dyn Adapter,
        command_log: Option<&Path>,
        session: &Command,
    ) -> Self {
        let mut command = Command::new(session.get_program());
        command.arg("--version");
        if let Some(dir) = session.get_current_dir() {
            command.current_dir(dir);
        }
        for (key, value) in session.get_envs() {
            if let Some(value) = value {
                command.env(key, value);
            } else {
                command.env_remove(key);
            }
        }
        Self {
            adapter,
            cache: command_log.map(|log| cache_path(log, adapter.name())),
            command,
            current: None,
        }
    }

    /// A lookup failure costs attribution, never the Session. Retain even an
    /// unknown result so a broken version command is not retried per event.
    pub fn observe(&mut self, event: &Value) -> &str {
        if let Some(version) = self.adapter.stream_version(event).and_then(nonempty) {
            if self.current.as_deref() != Some(version) {
                let _ = self.resolve(Some(version));
                self.current = Some(version.into());
            }
        } else if self.current.is_none() {
            self.current = Some(self.resolve(None).unwrap_or_else(|_| "unknown".into()));
        }
        self.current.as_deref().unwrap()
    }

    fn resolve(&mut self, reported: Option<&str>) -> io::Result<String> {
        let Some(path) = &self.cache else {
            return Ok(reported.map(str::to_owned).unwrap_or_else(|| self.query()));
        };
        let mut file = OpenOptions::new()
            .create(true)
            .truncate(false)
            .read(true)
            .write(true)
            .open(path)?;
        file.lock()?;
        let mut cached = String::new();
        file.read_to_string(&mut cached)?;
        if reported.is_none()
            && let Some(cached) = nonempty(&cached)
        {
            return Ok(cached.into());
        }
        let version = reported.map(str::to_owned).unwrap_or_else(|| self.query());
        file.seek(SeekFrom::Start(0))?;
        file.set_len(0)?;
        file.write_all(version.as_bytes())?;
        Ok(version)
    }

    fn query(&mut self) -> String {
        let Ok(output) = process::output(self.adapter, &mut self.command, None) else {
            return "unknown".into();
        };
        if !output.status.success() {
            return "unknown".into();
        }
        let stdout = String::from_utf8_lossy(&output.stdout);
        let stderr = String::from_utf8_lossy(&output.stderr);
        nonempty(&stdout)
            .or_else(|| nonempty(&stderr))
            .unwrap_or("unknown")
            .into()
    }
}

fn nonempty(text: &str) -> Option<&str> {
    let text = text.trim();
    (!text.is_empty()).then_some(text)
}