fix(journal): TUI Full mode + drop auto-close + reject closed-task attaches (v0.3.1)

Three corrections to the auto-capture pipeline. Symptoms were
visible in the TUI: tasks closed without the user closing them,
events attached to unrelated old tasks, and compact detail view
hid the reasoning chain.

TUI:
- task_detail now uses pack::assemble(.., Full) instead of Compact.
  User opens a task expecting the full chain (open/hypothesis/
  decision/rejection/evidence with commit hashes/close), Compact's
  3-event summary made the screen look empty.

Classifier output safeguards (in IngestHook handler, between
classifier.classify() and event write):
- If kind == "Stop" AND event_type == Close → drop. The Stop hook
  fires on every Claude Code session end and the classifier was
  emitting Close events from session endings. Closes are reserved
  for explicit `task-journal close <id>`.
- If task_id_guess points at a non-existent or closed task → route
  to pending with a descriptive reason. Prevents old tasks from
  accumulating events that the classifier mistakenly attributes to
  them based on prompt similarity.

Added tj_core::db::task_status helper for the closed-task check.

Closes claude-memory-* (3 P1 beads opened earlier this session for
v0.3.1 polish).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

Author: Shahinyanm

.claude-plugin/marketplace.json

@@ -6,14 +6,14 @@
   },
   "metadata": {
     "description": "Task Journal — append-only reasoning chain memory for AI-coding tasks",
-    "version": "0.3.0"
+    "version": "0.3.1"
   },
   "plugins": [
     {
       "name": "task-journal",
       "source": "./plugin",
       "description": "Append-only journal of AI-coding task reasoning chains. Captures hypotheses, decisions, rejections, evidence — renders compact resume packs so an agent can pick up a 2-week-old task with full context.",
-      "version": "0.3.0",
+      "version": "0.3.1",
       "author": {
         "name": "Digital-Threads"
       },

CHANGELOG.md

@@ -7,6 +7,35 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.1] - 2026-05-08
+
+Three correctness fixes for the auto-capture pipeline. The journal
+was technically working but producing confusing output: events
+attached to the wrong tasks, sessions auto-closed tasks they had no
+business closing, and TUI's compact summary hid the reasoning chain
+the user actually wanted to see.
+
+### Fixed
+- TUI task detail now renders the **Full** pack instead of Compact —
+  every event, decisions, rejections, evidence (including commit
+  hashes), and close lines, in chronological order. Compact's three-
+  line "Active decisions / Recent events" summary made the detail
+  view look empty.
+- Stop hook no longer auto-closes tasks. The Stop hook fires every
+  time a Claude Code session ends, and the classifier was happily
+  emitting `Close` events from those endings. Sessions ending !=
+  task done. Closes are now reserved for explicit
+  `task-journal close <id>` calls.
+- Closed and missing tasks are no longer silently appended to. When
+  the classifier's `task_id_guess` points at a task that doesn't
+  exist or is already closed, the event is routed to `pending/`
+  instead of being attached. Old tasks ("Demo task", "Тест plugin"
+  in our case) stop accumulating events from unrelated work.
+
+### Added
+- `tj_core::db::task_status(&conn, task_id)` helper for the closed-
+  task safeguard above.
+
 ## [0.3.0] - 2026-05-08
 
 ### Changed

Cargo.lock

@@ -7,7 +7,7 @@ members = [
 ]
 
 [workspace.package]
-version = "0.3.0"
+version = "0.3.1"
 edition = "2021"
 rust-version = "1.88"
 license = "MIT"

Cargo.toml

@@ -16,7 +16,7 @@ name = "task-journal"
 path = "src/main.rs"
 
 [dependencies]
-tj-core = { package = "task-journal-core", version = "0.3.0", path = "../tj-core" }
+tj-core = { package = "task-journal-core", version = "0.3.1", path = "../tj-core" }
 anyhow = { workspace = true }
 clap = { workspace = true }
 tracing = { workspace = true }

crates/tj-cli/Cargo.toml

@@ -1326,6 +1326,44 @@ fn main() -> Result<()> {
                     let Some(tid) = out.task_id_guess else {
                         return Ok(());
                     };
+
+                    // Journal-integrity safeguards. The classifier sometimes
+                    // mis-attributes events to old or closed tasks (no fault
+                    // of the model — its prompt only sees recent_tasks). We
+                    // reject three patterns that produce confusing journals:
+                    //
+                    //   1. Stop-hook → Close event. The Stop hook fires at
+                    //      every Claude Code session end. Session ending
+                    //      != task done. Closes happen via explicit
+                    //      `task-journal close <id>` only.
+                    //   2. task_id_guess pointing at a non-existent task —
+                    //      route to pending so the user can decide later.
+                    //   3. task_id_guess pointing at a CLOSED task — same
+                    //      treatment; closed tasks must stay closed.
+                    use tj_core::event::EventType;
+                    if matches!(out.event_type, EventType::Close) && kind == "Stop" {
+                        return Ok(());
+                    }
+                    match tj_core::db::task_status(&conn, &tid)? {
+                        None => {
+                            persist_pending(
+                                &events_path,
+                                &text,
+                                &format!("task_id_guess `{tid}` not found"),
+                            )?;
+                            return Ok(());
+                        }
+                        Some(s) if s == "closed" => {
+                            persist_pending(
+                                &events_path,
+                                &text,
+                                &format!("task_id_guess `{tid}` is closed"),
+                            )?;
+                            return Ok(());
+                        }
+                        _ => {}
+                    }
+
                     (
                         out.event_type,
                         tid,

crates/tj-cli/src/main.rs

@@ -230,7 +230,10 @@ impl App {
         let project_hash = tj_core::project_hash::from_path(&cwd)?;
         let state_path = tj_core::paths::state_dir()?.join(format!("{project_hash}.sqlite"));
         let conn = tj_core::db::open(&state_path)?;
-        let pack = tj_core::pack::assemble(&conn, task_id, tj_core::pack::PackMode::Compact)?;
+        // Full mode: show the complete reasoning chain — every event,
+        // decisions, rejections, evidence with commit hashes — instead
+        // of the 3-line Compact summary the CLI default uses.
+        let pack = tj_core::pack::assemble(&conn, task_id, tj_core::pack::PackMode::Full)?;
         Ok(pack.text)
     }
 

crates/tj-cli/src/tui/app.rs

@@ -254,6 +254,15 @@ pub fn task_exists(conn: &Connection, task_id: &str) -> anyhow::Result<bool> {
     Ok(count > 0)
 }
 
+/// Status string for an existing task (e.g. "open", "closed"). Returns
+/// `None` when the task is unknown — caller decides whether that's a
+/// hard error or a route-to-pending case.
+pub fn task_status(conn: &Connection, task_id: &str) -> anyhow::Result<Option<String>> {
+    let mut stmt = conn.prepare("SELECT status FROM tasks WHERE task_id = ?1")?;
+    let mut rows = stmt.query(rusqlite::params![task_id])?;
+    Ok(rows.next()?.map(|r| r.get::<_, String>(0)).transpose()?)
+}
+
 /// Look up the most recent `event_id` we've ingested for this project.
 /// Returns `None` when the project has never been indexed (first call,
 /// or migration v002 just landed on an existing 0.1.x DB).

crates/tj-core/src/db.rs

@@ -16,7 +16,7 @@ name = "task-journal-mcp"
 path = "src/main.rs"
 
 [dependencies]
-tj-core = { package = "task-journal-core", version = "0.3.0", path = "../tj-core" }
+tj-core = { package = "task-journal-core", version = "0.3.1", path = "../tj-core" }
 anyhow = { workspace = true }
 tokio = { workspace = true }
 tracing = { workspace = true }

crates/tj-mcp/Cargo.toml

@@ -1,6 +1,6 @@
 {
   "name": "task-journal",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Append-only journal of AI-coding task reasoning chains: hypotheses, decisions, rejections, evidence. Renders compact resume packs so an agent can pick up a 2-week-old task with full context.",
   "author": {
     "name": "Mher Shahinyan"