@zalom/plastic 2.0.0-alpha.24 → 2.0.0-alpha.26

package/agents/plastic-advisor.md

@@ -9,7 +9,7 @@ description: |
   approaches and kill criteria for architecture, one-way doors, or deadlocks. Model is set by config
   (agents.models.claude.plastic-advisor); fable is the shipped default.
 model: fable
-effort: xhigh
+effort: medium
 ---
 
 You are the advisor, consulted for expensive reasoning per the shipped Advisor

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zalom/plastic",
-  "version": "2.0.0-alpha.24",
+  "version": "2.0.0-alpha.26",
   "description": "Intent-driven idea development system for AI coding agents",
   "type": "module",
   "bin": {

package/scripts/lib/agent_models.rb

@@ -102,6 +102,17 @@ module AgentModels
     models_section(global_config, harness: harness).merge(models_section(project_config, harness: harness))
   end
 
+  def effort_override_map(project_config: {}, global_config: {}, harness: "claude")
+    efforts_section(global_config, harness).merge(efforts_section(project_config, harness))
+  end
+
+  def efforts_section(config, harness)
+    agents = config.is_a?(Hash) ? config["agents"] : nil
+    efforts = agents.is_a?(Hash) ? agents["efforts"] : nil
+    section = efforts.is_a?(Hash) ? efforts[harness] : nil
+    section.is_a?(Hash) ? section : {}
+  end
+
   # The model_reasoning_effort for a Plastic tier alias, or nil for any value that is not
   # one of the three shipped aliases (the caller treats nil as a literal Codex model id).
   def effort_for(value)

package/scripts/lib/doctor_core.rb

@@ -412,7 +412,7 @@ class Doctor
         fix_hint: "Re-run the Plastic installer: npx @zalom/plastic@latest --claude (prunes " \
                   "stale launchers). The plastic- prefix is reserved for Plastic's own hooks: " \
                   "if one of these is yours, rename it (for example to ~/.claude/hooks/" \
-                  "writing-style) and re-register it in settings.json before re-running."
+                  "plain-writing) and re-register it in settings.json before re-running."
       )
     end
 
@@ -1257,7 +1257,7 @@ class Doctor
   # agent_model_drift, no store-content scanning).
   def run_core_checks(agent_key)
     all_checks = []
-    all_checks += check_agent_registration(agent_key)
+    all_checks += check_agent_registration(agent_key).reject { |c| c[:name] == "codex_hooks_trust" }
     all_checks += check_core_files(agent_key, include_drift: false)
     all_checks += check_manifest_sync(agent_key)
     all_checks += check_registered_project_paths

package/scripts/lib/installer_core.rb

@@ -1081,7 +1081,7 @@ class InstallerCore
     installed += install_skills_flat(skills_source, skills_root, exclude: skill_exclude) if File.directory?(skills_source)
 
     # Copy agent role files into <dir>/agents (manifest-tracked, pruned on update)
-    installed += install_agents(File.join(config[:dir], "agents"), models: agent_model_overrides, advisor_enabled: advisor_enabled?)
+    installed += install_agents(File.join(config[:dir], "agents"), models: agent_model_overrides, efforts: agent_effort_overrides, advisor_enabled: advisor_enabled?)
 
     # Write VERSION
     version_file = File.join(plastic_dir, "VERSION")
@@ -1314,7 +1314,7 @@ class InstallerCore
     skills_source = File.join(package_root, "skills")
     skill_exclude = advisor_enabled? ? [] : ["agent-advisor"]
     installed += install_skills_flat(skills_source, File.join(config[:dir], "skills"), exclude: skill_exclude) if File.directory?(skills_source)
-    installed += install_agents(File.join(config[:dir], "agents"), models: agent_model_overrides, advisor_enabled: advisor_enabled?)
+    installed += install_agents(File.join(config[:dir], "agents"), models: agent_model_overrides, efforts: agent_effort_overrides, advisor_enabled: advisor_enabled?)
 
     # Uniform per-agent record (intent 210, D2): write VERSION alongside the manifest,
     # the same shape install_claude already writes.
@@ -1365,7 +1365,7 @@ class InstallerCore
   # AgentModels::CONSULTATION_AGENTS file entirely (both plastic-advisor and
   # plastic-faux-advisor), so a user who declined the advisor never gets either
   # agent installed.
-  def install_agents(agents_root, models: {}, advisor_enabled: true)
+  def install_agents(agents_root, models: {}, efforts: {}, advisor_enabled: true)
     sources = Dir.glob(File.join(package_root, "agents", "*.md"))
     return [] if sources.empty?
 
@@ -1377,9 +1377,13 @@ class InstallerCore
     sources.map do |src|
       dest = File.join(agents_root, File.basename(src))
       basename = File.basename(src, ".md")
-      override = models[basename]
-      if override
-        File.write(dest, rewrite_model_line(File.read(src), override))
+      model = models[basename]
+      effort = efforts[basename]
+      if model || effort
+        content = File.read(src)
+        content = rewrite_model_line(content, model) if model
+        content = rewrite_effort_line(content, effort) if effort
+        File.write(dest, content)
       else
         FileUtils.cp(src, dest)
       end
@@ -1393,6 +1397,12 @@ class InstallerCore
     content.sub(/^model:[^\n]*$/, "model: #{model}")
   end
 
+  def rewrite_effort_line(content, effort)
+    return content.sub(/^effort:[^\n]*$/, "effort: #{effort}") if content.match?(/^effort:/)
+
+    content.sub(/^(model:[^\n]*)$/) { "#{Regexp.last_match(1)}\neffort: #{effort}" }
+  end
+
   # Resolve per-agent model overrides for this install: project config (when a
   # project dir is known) overlaid on global config, scoped to `harness`
   # ("claude" or "codex"). Defaults are NOT included, so unconfigured agents
@@ -1416,6 +1426,12 @@ class InstallerCore
     AgentModels.override_map(project_config: project_config, global_config: global_config, harness: harness)
   end
 
+  def agent_effort_overrides(project_dir = nil, harness: "claude")
+    global_config = load_config_yaml(File.join(plastic_home, "config.yml"))
+    project_config = project_dir ? load_config_yaml(File.join(project_dir, ".plastic_store", "config.yml")) : {}
+    AgentModels.effort_override_map(project_config: project_config, global_config: global_config, harness: harness)
+  end
+
   # advisor.enabled: project overlays global, missing or malformed counts as
   # enabled (fail-open). Harness-blind: false skips both advisor agents and the
   # agent-advisor skill on every installed harness.
@@ -1566,7 +1582,7 @@ class InstallerCore
     puts "  \u{2139}\u{fe0f}  Kept #{kept.size} hook(s) Plastic does not own, named with the reserved plastic- prefix:"
     kept.each { |cmd| puts "     - #{tilde(cmd.to_s)}" }
     puts "     The plastic- prefix is reserved for Plastic's own hooks. Rename yours (for"
-    puts "     example ~/.claude/hooks/writing-style) so a future update never mistakes it."
+    puts "     example ~/.claude/hooks/plain-writing) so a future update never mistakes it."
   end
 
   # --- settings.json merge (read-modify-write, never clobber) ---

package/skills/auto/references/human-report-contract.md

@@ -120,7 +120,7 @@ renders. The two never merge.
 
 ## Brevity: point, don't repeat
 
-Surface rules are owned by the `writing-style` skill. This contract does not restate them. Its
+Surface rules are owned by the `plain-writing` skill. This contract does not restate them. Its
 job is naming which screen prints when, not the wording inside it - `report-screen` derives
 every cell from the record (D14), so there is no prose left to style here.