fix(inspector): kill demo emitter — show real data only or "(no events)"

User asked twice to kill the demo mock data and I left a fallback in
place that kept silently activating because of subtle is_terminal()
discrepancies between default_command (stdin-tty check) and app::run
(separate stdin-tty check). The cockpit displayed synthetic
hydra.veto.fired / sylph.destructive.veto / shell.exec events that
literally do not exist in the real JSONL hook stream — they were
coming from src/demo.rs's synthetic emitter, masking real-data
issues.

Fix: remove the demo-mode branch from app::run entirely. If
Source::Stdin is reached with no piped data, the cockpit shows
"(no events)" honestly. No more silent fallback to synthetic
animation.

Also: lib.rs gains an eprintln! at startup naming the resolved
hook-jsonl path + tail-vs-no-hooks decision so users can confirm
which mode the cockpit picked without grepping the tracing log.

src/demo.rs stays in the codebase as legacy code (unused) — could
be revived for tests / fixtures but is no longer wired into the
default user-facing flow.

Author: klaiderman

inspector/src/app.rs

@@ -39,32 +39,27 @@ pub async fn run(config: Config) -> anyhow::Result<()> {
     let terminal = &mut guard.terminal;
 
     // Demo-mode trigger: stdin source AND no pipe (terminal stdin) → skip
-    // the real transport and run the built-in synthetic emitter so the
-    // dashboard always shows life on a bare `enchanter.exe` launch.
-    let demo_mode = matches!(config.source, Source::Stdin) && io::stdin().is_terminal();
-
-    // Build the unified event receiver — either from real transport or demo.
-    // The control writer is connected ONLY for `Source::SocketControl`; every
-    // other source path returns a disconnected writer so `send_control` errors
-    // visibly when the user tries to approve/veto on a read-only stream.
-    let (mut event_rx, control_writer): (mpsc::Receiver<Event>, ControlWriter) = if demo_mode {
-        let (tx, rx) = mpsc::channel::<Event>(1024);
-        crate::demo::spawn_demo_emitter(tx);
-        (rx, ControlWriter::disconnected())
-    } else {
-        match config.source.clone() {
-            Source::SocketControl(_) => {
-                // try_spawn opens the socket eagerly so we surface failure
-                // before the TUI swallows the error.
-                let transport = Transport::try_spawn(map_source(config.source), 1024).await?;
-                let writer = transport.writer();
-                (transport.into_receiver(), writer)
-            }
-            _ => (
-                Transport::spawn(map_source(config.source), 1024).into_receiver(),
-                ControlWriter::disconnected(),
-            ),
+    // Demo mode REMOVED. Earlier versions auto-launched the synthetic
+    // emitter when stdin was a TTY + Source::Stdin — that path was masking
+    // real-data issues by silently filling the cockpit with fake events.
+    // The user explicitly asked to kill it. If a Source::Stdin is reached
+    // here with no piped input, the cockpit will simply sit at "(no events)"
+    // until the user pipes something in or quits — honest, non-deceiving.
+    let demo_mode = false;
+
+    let (mut event_rx, control_writer): (mpsc::Receiver<Event>, ControlWriter) = match config
+        .source
+        .clone()
+    {
+        Source::SocketControl(_) => {
+            let transport = Transport::try_spawn(map_source(config.source), 1024).await?;
+            let writer = transport.writer();
+            (transport.into_receiver(), writer)
         }
+        _ => (
+            Transport::spawn(map_source(config.source), 1024).into_receiver(),
+            ControlWriter::disconnected(),
+        ),
     };
 
     // Keyboard: poll-on-blocking-thread, forward Crossterm events over a channel.

inspector/src/lib.rs

@@ -184,7 +184,9 @@ fn claude_code_hook_jsonl() -> PathBuf {
 /// no longer routes there.
 fn default_command() -> Command {
     use std::io::IsTerminal;
-    if !std::io::stdin().is_terminal() {
+    let stdin_tty = std::io::stdin().is_terminal();
+    if !stdin_tty {
+        eprintln!("[enchanter] stdin is not a TTY → reading JSONL from stdin (pipe mode)");
         return Command::Inspect(InspectArgs::default());
     }
 
@@ -193,12 +195,17 @@ fn default_command() -> Command {
     // since install), but the parent dir does, and `--tail` waits up to 30s
     // for the file to appear. That's the "boom, real data" UX.
     let hook_jsonl = claude_code_hook_jsonl();
-    let hooks_wired_up = hook_jsonl.exists()
-        || hook_jsonl.parent().map(|p| p.is_dir()).unwrap_or(false);
+    let exists = hook_jsonl.exists();
+    let parent_exists = hook_jsonl.parent().map(|p| p.is_dir()).unwrap_or(false);
+    let hooks_wired_up = exists || parent_exists;
+    eprintln!(
+        "[enchanter] checking hooks: path={} exists={} parent_dir={} → {}",
+        hook_jsonl.display(),
+        exists,
+        parent_exists,
+        if hooks_wired_up { "TAIL" } else { "no-hooks" }
+    );
     if hooks_wired_up {
-        // Log so users can grep ~/.cache/enchanter/inspector.log to confirm
-        // which mode bare `enchanter` picked — useful when troubleshooting
-        // "is this real or showcase data" without restarting the binary.
         tracing::info!(
             path = %hook_jsonl.display(),
             "default_command: hooks wired up — tailing real Claude Code stream"