From e9b22084e03880f43d1f1daaf5d9f45c717b77c4 Mon Sep 17 00:00:00 2001 From: Enrico Carlesso Date: Fri, 24 Jul 2026 12:05:20 -0700 Subject: feat: add grok cli integration for native session restore (#1807) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * feat: add grok cli integration for native session restore tl;dr: adds `herdr integration install grok` (session-identity tier: a SessionStart hook reports the grok session id for native restore; state stays screen-detected via the bundled manifest). Grok merges every ~/.grok/hooks/*.json, so install writes a self-contained hooks/herdr.json registering a SessionStart hook next to hooks/herdr-agent-state.sh, and never touches other hook files. The hook prefers $GROK_SESSION_ID (injected into every grok hook process) and reports through the socket API; herdr:grok is a reserved-native-state source so screen detection stays authoritative. Restore uses `grok --resume `. Additive JSON API change; no protocol bump; the schema artifact is regenerated. refs #1800 * fix: honor GROK_HOME and validate grok hook config in status Review follow-ups: grok_dir now resolves GROK_CONFIG_DIR (herdr test seam) before GROK_HOME (the grok CLI's own config-home override) before ~/.grok, so hook installs land where grok actually looks. Integration status now also validates the herdr-owned hooks/herdr.json next to the hook script — missing, corrupt, or no-longer-referencing configs report the install as outdated instead of masking a hook grok never invokes. refs #1800 * fix: validate the exact grok session hook command in status The config validator matched on a filename substring, so a SessionStart command like `echo herdr-agent-state.sh`, or one invoking the script without the `session` action, still reported the integration as current while grok never delivered a session id. Status now compares against the exact command the installer writes, via a shared builder, and the degradation tests cover both nonfunctional shapes. refs #1800 * fix: require a command-type grok session hook in status validation A config entry carrying the exact generated command under a non-command hook type (e.g. "http") is never executed by grok, so it must not report the integration as current. The validator now also requires type = "command", with a degradation test for the mismatched-type shape. refs #1800 * fix: validate the complete grok hook config refs #1800 --------- Co-authored-by: Ogulcan Celik --- src/agent_resume.rs | 15 +++++++++++++++ 1 file changed, 15 insertions(+) (limited to 'src/agent_resume.rs') diff --git a/src/agent_resume.rs b/src/agent_resume.rs index 1cb0f9dd..f1389b54 100644 --- a/src/agent_resume.rs +++ b/src/agent_resume.rs @@ -88,6 +88,7 @@ pub fn is_reserved_native_state_source(source: &str, agent: &str) -> bool { | ("herdr:droid", "droid") | ("herdr:qodercli", "qodercli") | ("herdr:cursor", "cursor") + | ("herdr:grok", "grok") ) } @@ -186,6 +187,9 @@ pub fn plan(source: &str, agent: &str, session_ref: &AgentSessionRef) -> Option< session_ref.value.clone(), ] } + ("herdr:grok", "grok", AgentSessionRefKind::Id) => { + vec!["grok".into(), "--resume".into(), session_ref.value.clone()] + } _ => return None, }; @@ -220,6 +224,7 @@ pub(crate) fn is_official_agent_source(source: &str, agent: &str) -> bool { | ("herdr:qodercli", "qodercli") | ("herdr:kilo", "kilo") | ("herdr:cursor", "cursor") + | ("herdr:grok", "grok") ) } @@ -402,6 +407,16 @@ mod tests { .argv, vec!["cursor-agent", "--resume", "cursor-session"] ); + assert_eq!( + plan( + "herdr:grok", + "grok", + &AgentSessionRef::id("grok-session").unwrap() + ) + .unwrap() + .argv, + vec!["grok", "--resume", "grok-session"] + ); } #[test] -- cgit v1.2.3