pointbreak 0.6.0

Durable terminal code review for changes humans and coding agents collaborate on together
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

mod support;

use support::inspect::{Inspector, representative_store};

fn served_app_css() -> String {
    let store = representative_store();
    Inspector::spawn(store.repo.path()).get_text("/app.css")
}

#[test]
fn index_html_is_one_master_detail_shell_not_four_views() {
    let store = representative_store();
    let html = Inspector::spawn(store.repo.path()).get_text("/");
    // One master pane + one detail pane (the list-detail skeleton), not four sections.
    assert!(
        html.contains("id=\"master\"") && html.contains("id=\"detail\""),
        "the shell is one master pane + one detail pane"
    );
    // The four parallel view sections are gone (collapsed into the lens dispatch).
    for old in [
        "id=\"view-timeline\"",
        "id=\"view-units\"",
        "id=\"view-revisions\"",
        "id=\"view-unit\"",
    ] {
        assert!(
            !html.contains(old),
            "the parallel `{old}` section is collapsed into the shell"
        );
    }
    // The master pane switches between the three lenses (durable data-attr values).
    for lens in ["timeline", "list", "attention"] {
        assert!(
            html.contains(&format!("data-lens=\"{lens}\"")),
            "the lens switcher offers the `{lens}` lens"
        );
    }
    // The threads lens is dissolved into the revision list; its tab never returns.
    assert!(
        !html.contains("data-lens=\"threads\""),
        "the retired threads lens is not offered"
    );
}

#[test]
fn served_assets_preserve_the_advisory_framing_and_competing_peers() {
    let store = representative_store();
    let html = Inspector::spawn(store.repo.path()).get_text("/");
    // The advisory / read-only framing now lives in the store-identity popover note
    // (issue #391 follow-up) rather than a persistent top-bar badge, but it stays in
    // the served markup as rendered text, not a `title`-only tooltip.
    assert!(
        html.contains("never gates writes") && html.contains("reader-relative"),
        "the served shell carries the read-only / advisory framing as rendered text"
    );
}

#[test]
fn served_topbar_uses_the_pointbreak_logo_mark() {
    let store = representative_store();
    let inspector = Inspector::spawn(store.repo.path());
    let html = inspector.get_text("/");
    assert!(
        html.contains("class=\"brand-mark\"")
            && html.contains("aria-hidden=\"true\"")
            && html.contains("Pointbreak<span class=\"brand-accent\">Review</span>"),
        "the served shell should keep readable text while rendering the decorative logo mark"
    );

    let logo = inspector.get_text("/pointbreak-logo-mono.svg");
    assert!(
        logo.contains("currentColor") && logo.contains("<svg"),
        "the served logo asset should be the currentColor mono Pointbreak mark"
    );
}

#[test]
fn served_shell_exposes_the_pointbreak_favicon() {
    let store = representative_store();
    let inspector = Inspector::spawn(store.repo.path());
    let html = inspector.get_text("/");
    assert!(
        html.contains(r#"<link rel="icon" href="/favicon.png" type="image/png" sizes="32x32" />"#)
            && html.contains(r#"<link rel="icon" href="/favicon-dark.png" type="image/png" sizes="32x32" media="(prefers-color-scheme: dark)" />"#)
            && !html.contains(r#"rel="icon" href="/favicon.svg""#),
        "the served shell should declare transparent PNG favicons instead of the Safari-problematic SVG favicon"
    );

    let (light_status, light_favicon) = inspector.raw_get("/favicon.png");
    let (dark_status, dark_favicon) = inspector.raw_get("/favicon-dark.png");
    assert!(
        light_status.contains("200 OK")
            && dark_status.contains("200 OK")
            && light_favicon.contains("PNG")
            && dark_favicon.contains("PNG"),
        "the served favicon routes should return PNG image bodies"
    );
}

#[test]
fn served_css_has_a_narrow_viewport_shell_contract() {
    let css = served_app_css();
    assert!(
        css.contains("@media") && css.contains("max-width"),
        "served CSS should carry an explicit narrow viewport breakpoint"
    );
    assert!(
        css.contains("grid-template-columns: minmax(0, 1fr)")
            || css.contains("grid-template-columns: 1fr"),
        "narrow shell should stop forcing two minmax(360px, 1fr) columns"
    );
    assert!(
        css.contains("#topbar")
            && css.contains("flex-wrap: wrap")
            && css.contains(".stats")
            && css.contains("justify-content: flex-start"),
        "topbar and stats should be able to wrap instead of causing narrow overflow"
    );
    // The narrow detail is a slide-over sheet, not a stacked half-height row:
    // full-height over the list, transform-driven so the open/close animates,
    // with the list's scroll position preserved beneath it.
    let narrow = css
        .split("@media (max-width: 760px)")
        .nth(1)
        .expect("narrow viewport media block exists");
    assert!(
        narrow.contains("#detail") && narrow.contains("position: fixed"),
        "narrow detail should be a full-height sheet over the list"
    );
    assert!(
        narrow.contains("translateX"),
        "the sheet should slide via a transform bound to the open state"
    );
    assert!(
        css.contains("prefers-reduced-motion"),
        "the sheet slide should honor prefers-reduced-motion"
    );
}

#[test]
fn served_css_has_a_narrow_diff_modal_contract() {
    let css = served_app_css();
    let narrow = css
        .split("@media (max-width: 760px)")
        .nth(1)
        .and_then(|tail| tail.split(".units").next())
        .expect("narrow viewport media block exists");
    assert!(
        narrow.contains(".diff-layout") && narrow.contains("flex-direction: column"),
        "narrow diff modal should stack navigator and body instead of forcing a side-by-side row"
    );
    assert!(
        narrow.contains(".diff-nav")
            && narrow.contains("border-bottom: 1px solid var(--border)")
            && narrow.contains("border-right: 0"),
        "stacked diff navigator should become a top region with bottom divider"
    );
}

#[test]
fn served_css_keeps_visible_focus_for_custom_interactive_surfaces() {
    let css = served_app_css();
    for selector in [
        ".diff-nav-file:focus-visible",
        ".diff-nav-fact:focus-visible",
        ".dag-node:focus-visible",
        ".ref[data-ref-kind]:focus-visible",
    ] {
        assert!(
            css.contains(selector),
            "served CSS should style visible focus for {selector}"
        );
    }

    let focus_visible_blocks = css
        .split('}')
        .filter(|block| block.contains(":focus-visible"))
        .collect::<Vec<_>>();
    assert!(
        focus_visible_blocks
            .iter()
            .all(|block| !block.contains("outline: none")),
        "focus-visible rules should not remove every outline without replacement"
    );
}