@zalom/plastic 1.0.0-alpha.9 → 1.0.0-beta.2

package/scripts/hook-session-start

@@ -7,6 +7,9 @@ require "json"
 require "date"
 require "yaml"
 require_relative "lib/bridge"
+require_relative "lib/boot_banner"
+require_relative "lib/qmd_sync"
+require_relative "doctor"
 
 index_path, store_root, mode, plugin_root = ARGV
 exit 0 unless index_path && store_root && mode
@@ -117,15 +120,23 @@ if File.exist?(projects_path)
   end
 end
 
+# --- Load PLASTIC.md conventions ---
+
+plastic_md_path = "#{store_root}/PLASTIC.md"
+plastic_md = File.exist?(plastic_md_path) ? File.read(plastic_md_path).strip : nil
+
 # --- Load deprecations ---
 
+dep_file = if plugin_root && !plugin_root.empty?
+  "#{plugin_root}/deprecations.yml"
+else
+  "#{store_root}/deprecations.yml"
+end
+
 deprecations = []
-if plugin_root && !plugin_root.empty?
-  dep_file = "#{plugin_root}/deprecations.yml"
-  if File.exist?(dep_file)
-    dep_data = YAML.safe_load(File.read(dep_file)) rescue {}
-    deprecations = dep_data["deprecations"] || []
-  end
+if File.exist?(dep_file)
+  dep_data = YAML.safe_load(File.read(dep_file)) rescue {}
+  deprecations = dep_data["deprecations"] || []
 end
 
 dismissed_json = `"#{read_config}" deprecations_dismissed`.strip
@@ -136,7 +147,10 @@ rescue
 end
 
 current_version = nil
-if plugin_root && !plugin_root.empty?
+version_file = "#{store_root}/VERSION"
+if File.exist?(version_file)
+  current_version = File.read(version_file).strip
+elsif plugin_root && !plugin_root.empty?
   plugin_json_path = "#{plugin_root}/.claude-plugin/plugin.json"
   if File.exist?(plugin_json_path)
     pj = JSON.parse(File.read(plugin_json_path)) rescue {}
@@ -157,53 +171,98 @@ cache_file = "#{store_root}/.cache/update-check.json"
 if File.exist?(cache_file)
   cache = JSON.parse(File.read(cache_file)) rescue {}
   if cache["updateAvailable"]
-    update_notice = "Plastic update available: #{cache["current"]} -> #{cache["latest"]} — run /plastic:update"
+    update_notice = "Plastic update available: #{cache["current"]} -> #{cache["latest"]} — run /plastic-update"
   end
 end
 
-# --- Early exit if nothing to show ---
+# --- Run core health in-process (intent 36a) ---
+# Reuse Doctor's own --core checks so there is one source of truth for "core
+# health" (no second process spawn, no duplicated check list). Never blocks the
+# session: any failure or exception degrades to a banner and we continue.
+
+core_health = begin
+  Doctor.new(plastic_home: store_root).run_core_checks("claude")
+rescue
+  nil
+end
+core_banner = BootBanner.render(health: core_health, version: current_version)
+
+# --- Assemble session context ---
 
 all_active = active + project_active
 all_future = future + project_future
-exit 0 if all_active.empty? && stale.empty? && all_future.empty? && active_deprecations.empty? && update_notice.nil?
 
 # --- Build context ---
+# PLASTIC.md conventions and intent context render only when conventions are
+# installed. Deprecation warnings and update notices render regardless, so a
+# partial install (or a missing PLASTIC.md) still surfaces critical warnings.
 
 parts = []
 
-if current_project
-  slug = current_project["slug"]
-  banner = "Project: #{slug} | Store: ~/.plastic/projects/#{slug}/store/"
-  if project_active.any? && project_active.first =~ /\[([^\]]+)\].*store\/([\w-]+)\//
-    intent_name, dir_name = $1, $2
-    intent_id = dir_name.split("--").first
-    banner += "\nActive: [#{intent_id} — #{intent_name}] | Artifacts → store/#{dir_name}/"
+# Boot banner first: every session start surfaces that Plastic loaded, with the
+# version (clean) or the first failing core check (degraded). Owned by the hook
+# so it runs by construction — the plastic-continuing skill no longer does this.
+parts << core_banner
+parts << ""
+
+# --- QMD search status (intent 45a, READ-ONLY) ---
+# Report-only line for the model (additionalContext), never systemMessage and
+# never the exit code. QmdSync.status shells out to `qmd collection list`, so the
+# whole block is guarded: a 2s timeout caps any hang, and rescue-all guarantees a
+# slow/broken/missing qmd appends nothing and the hook continues cleanly.
+begin
+  require "timeout"
+  qmd_status = Timeout.timeout(2) { QmdSync.status(plastic_home: store_root) }
+  if qmd_status[:present]
+    if qmd_status[:all_registered]
+      parts << "QMD: #{qmd_status[:registered].size} Plastic collections indexed (search with the qmd skill)."
+    else
+      parts << "QMD detected — run `qmd-sync register --all` to index your Plastic stores for search."
+    end
   end
-  parts << banner + "\n"
-else
-  parts << "PLASTIC — Global store loaded from ~/.plastic/\n"
+rescue Exception
+  # Any failure (timeout, missing binary, parse error) — stay silent, never crash.
 end
 
-if all_active.any?
-  parts << "Active intents:\n"
-  all_active.each { |a| parts << a }
-  if bridge_data
-    stage = bridge_data["build"]["stage"]
-    missing = bridge_data["build"]["missing"]
-    missing_str = missing.empty? ? "none" : missing.join(", ")
-    parts << "Stage: #{stage} | Next: #{missing_str}"
+if plastic_md
+  # Conventions always loaded first
+  parts << plastic_md
+  parts << "\n---\n"
+
+  if current_project
+    slug = current_project["slug"]
+    banner = "Project: #{slug} | Store: ~/.plastic/projects/#{slug}/store/"
+    if project_active.any? && project_active.first =~ /\[([^\]]+)\].*store\/([\w-]+)\//
+      intent_name, dir_name = $1, $2
+      intent_id = dir_name.split("--").first
+      banner += "\nActive: [#{intent_id} — #{intent_name}] | Artifacts → store/#{dir_name}/"
+    end
+    parts << banner + "\n"
+  else
+    parts << "PLASTIC — Global store loaded from ~/.plastic/\n"
   end
-  parts << ""
-else
-  parts << "No active intents. #{future.length} future intents available.\n"
-end
 
-if stale.any?
-  parts << "\nStale future intents (untouched for days):\n"
-  stale.each do |s|
-    parts << "- #{s[:name]} (#{s[:age]} days) — consider: activate, abandon, or defer to agent"
+  if all_active.any?
+    parts << "Active intents:\n"
+    all_active.each { |a| parts << a }
+    if bridge_data
+      stage = bridge_data["build"]["stage"]
+      missing = bridge_data["build"]["missing"]
+      missing_str = missing.empty? ? "none" : missing.join(", ")
+      parts << "Stage: #{stage} | Next: #{missing_str}"
+    end
+    parts << ""
+  else
+    parts << "No active intents. #{future.length} future intents available.\n"
+  end
+
+  if stale.any?
+    parts << "\nStale future intents (untouched for days):\n"
+    stale.each do |s|
+      parts << "- #{s[:name]} (#{s[:age]} days) — consider: activate, abandon, or defer to agent"
+    end
+    parts << "\nWhen appropriate, ask the user what to do with stale intents."
   end
-  parts << "\nWhen appropriate, ask the user what to do with stale intents."
 end
 
 if active_deprecations.any?
@@ -237,10 +296,19 @@ if update_notice
   parts.unshift("! #{update_notice}\n")
 end
 
+# Emit nothing when there is genuinely nothing to surface (no conventions,
+# no deprecations, no update notice).
+exit 0 if parts.join.strip.empty?
+
 payload = {
   "hookSpecificOutput" => {
     "hookEventName" => "SessionStart",
     "additionalContext" => parts.join("\n")
-  }
+  },
+  # Intent 54: additionalContext is model-only, so the banner stays invisible to
+  # the human. The top-level systemMessage channel is rendered in the user's
+  # terminal (and re-fires on /clear). Reuse the same BootBanner line so the
+  # visible banner and the model-facing banner cannot drift.
+  "systemMessage" => core_banner
 }
 puts JSON.generate(payload)