qql-cli 0.4.2

Command-line interface, REPL, converter, and migration tools for QQL
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

//! `qql config edge` merge semantics (e2e, isolated `$HOME`).
//!
//! A config update must patch the persisted `edge.json` in place: keys it does
//! not set — known siblings and unknown/future keys alike — survive.

use std::path::PathBuf;
use std::process::Command;
use std::time::{SystemTime, UNIX_EPOCH};

fn qql() -> Command {
    Command::new(env!("CARGO_BIN_EXE_qql"))
}

fn temp_home(name: &str) -> PathBuf {
    let unique = SystemTime::now()
        .duration_since(UNIX_EPOCH)
        .expect("clock")
        .as_nanos();
    let home = std::env::temp_dir().join(format!(
        "qql-config-edge-it-{}-{unique}-{name}",
        std::process::id()
    ));
    let _ = std::fs::remove_dir_all(&home);
    std::fs::create_dir_all(home.join(".qql")).expect("create .qql");
    home
}

#[test]
fn config_update_preserves_sibling_and_unknown_keys() {
    let home = temp_home("merge");
    let config = home.join(".qql").join("edge.json");
    std::fs::write(
        &config,
        serde_json::to_string_pretty(&serde_json::json!({
            "wal_segment_mb": 8,
            "embedder": "http",
            "embed_url": "http://127.0.0.1:9/v1/embeddings",
            "embed_model": "custom-model",
            "future_knob": { "nested": true },
        }))
        .expect("serialize seed"),
    )
    .expect("seed edge.json");

    let output = qql()
        .env("HOME", &home)
        .args(["config", "edge", "--wal-segment-mb", "4"])
        .output()
        .expect("spawn qql");
    assert!(
        output.status.success(),
        "config update failed: {}",
        String::from_utf8_lossy(&output.stderr)
    );

    let merged: serde_json::Value =
        serde_json::from_str(&std::fs::read_to_string(&config).expect("read merged"))
            .expect("parse merged");
    // The patched key is updated...
    assert_eq!(merged["wal_segment_mb"], serde_json::json!(4));
    // ...and every sibling survives, known or not.
    assert_eq!(merged["embedder"], serde_json::json!("http"));
    assert_eq!(
        merged["embed_url"],
        serde_json::json!("http://127.0.0.1:9/v1/embeddings")
    );
    assert_eq!(merged["embed_model"], serde_json::json!("custom-model"));
    assert_eq!(merged["future_knob"], serde_json::json!({ "nested": true }));

    #[cfg(unix)]
    {
        use std::os::unix::fs::PermissionsExt;
        let mode = std::fs::metadata(&config)
            .expect("metadata")
            .permissions()
            .mode()
            & 0o777;
        assert_eq!(mode, 0o600, "edge.json must stay owner-only");
    }

    let _ = std::fs::remove_dir_all(&home);
}

#[test]
fn config_update_validates_the_merged_state() {
    let home = temp_home("validate");
    let config = home.join(".qql").join("edge.json");
    std::fs::write(
        &config,
        r#"{"embedder": "http", "embed_url": "http://x/v1"}"#,
    )
    .expect("seed edge.json");

    // The persisted embed_url satisfies the http requirement even though this
    // invocation does not repeat it.
    let output = qql()
        .env("HOME", &home)
        .args(["config", "edge", "--wal-segment-mb", "4"])
        .output()
        .expect("spawn qql");
    assert!(
        output.status.success(),
        "{}",
        String::from_utf8_lossy(&output.stderr)
    );

    // A zero WAL knob fails closed on the merged state.
    let output = qql()
        .env("HOME", &home)
        .args(["config", "edge", "--wal-segment-mb", "0"])
        .output()
        .expect("spawn qql");
    assert!(!output.status.success(), "wal 0 must fail closed");
    assert!(
        String::from_utf8_lossy(&output.stderr).contains("greater than zero"),
        "{}",
        String::from_utf8_lossy(&output.stderr)
    );

    let _ = std::fs::remove_dir_all(&home);
}