T38: review round 1: herdr's records take herdr: before the pane, a colon no OSC 7501 id holds, so a program's record named like theirs, herdr/build among them, stays through herdr's snapshots and its leaving

src/term/herdr/tests.rs

@@ -141,21 +141,21 @@ fn herdr_s_agents_whose_state_it_knows_are_records_by_pane() {
         said,
         [
             (
-                "herdr/w1:p1",
+                "herdr:w1:p1",
                 State::Working,
                 Some("codex"),
                 Some("review"),
                 Some("Reading files")
             ),
             (
-                "herdr/w1:p2",
+                "herdr:w1:p2",
                 State::Blocked,
                 Some("claude"),
                 Some("Claude Code"),
                 Some("claude")
             ),
-            ("herdr/w1:p5", State::Done, None, None, None),
-            ("herdr/w1:p6", State::Idle, None, None, None),
+            ("herdr:w1:p5", State::Done, None, None, None),
+            ("herdr:w1:p6", State::Idle, None, None, None),
         ]
     );
     assert!(

src/term/status.rs

@@ -57,8 +57,8 @@ const ID: usize = 128;
 const SEGMENT: usize = 32;
 const LEVELS: usize = 8;
 
-/// The ids herdr's agents take, one by pane.
-pub(super) const HERDR: &str = "herdr/";
+/// The ids herdr's agents take, one by pane: no OSC 7501 id holds a colon, so none is taken for theirs.
+pub(super) const HERDR: &str = "herdr:";
 
 /// Records a terminal keeps; past them, the one updated longest ago goes.
 pub const RECORDS: usize = 256;

src/term/status/tests.rs

@@ -144,8 +144,27 @@ fn records_replace_and_clear_as_the_spec_says() {
 fn herdr_s_agents_replace_their_own_records_alone() {
     let mut records = Records::default();
     records.report("build".into(), record(State::Working, None));
+    // A program's id ends at a colon, so `herdr:w1:p1` names its record `herdr`.
+    for body in [
+        &b"state=done:id=herdr/build"[..],
+        b"state=done:id=herdr:w1:p1",
+    ] {
+        let Some(Status::Report(id, done)) = said(body) else {
+            panic!("a program's report");
+        };
+        records.report(id, done);
+    }
+    let ids = |records: &Records| -> Vec<String> {
+        records.list.iter().map(|(id, ..)| id.clone()).collect()
+    };
+    records.herdr(&[]);
+    assert_eq!(
+        ids(&records),
+        ["build", "herdr/build", "herdr"],
+        "an empty snapshot takes no program's record"
+    );
     let codex = [(
-        "herdr/w1:p1".to_string(),
+        "herdr:w1:p1".to_string(),
         record(State::Working, Some("codex")),
     )];
     records.herdr(&codex);
@@ -153,17 +172,27 @@ fn herdr_s_agents_replace_their_own_records_alone() {
     records.herdr(&codex);
     assert_eq!(records.clock, clock, "the same agents change nothing");
     let claude = [(
-        "herdr/w1:p2".to_string(),
+        "herdr:w1:p2".to_string(),
         record(State::Blocked, Some("claude")),
     )];
     records.herdr(&claude);
-    let ids: Vec<&str> = records.list.iter().map(|(id, ..)| id.as_str()).collect();
-    assert_eq!(ids, ["build", "herdr/w1:p2"]);
+    assert_eq!(
+        ids(&records),
+        ["build", "herdr/build", "herdr", "herdr:w1:p2"]
+    );
+    records.herdr(&[]);
+    assert_eq!(
+        ids(&records),
+        ["build", "herdr/build", "herdr"],
+        "herdr's leaving takes its own alone"
+    );
     records.reset();
     records.herdr(&claude);
-    assert_eq!(records.list.len(), 1, "the next snapshot puts them back");
-    records.herdr(&[]);
-    assert!(records.list.is_empty());
+    assert_eq!(
+        ids(&records),
+        ["herdr:w1:p2"],
+        "the next snapshot puts them back"
+    );
 }
 
 #[test]