space-architect 9.3.0 → 9.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4fd0baee9b2db69fa1ef96c1b4993f75a8912b5b0898d012c13036a379e32a7f
-  data.tar.gz: 582172fa5c61e4a9419409cafdfa8701ca096a2621527141099944f3261c6592
+  metadata.gz: e768903f0762630e4c4c510a83621cf4c82be30ea7d95026d794f90d70205f44
+  data.tar.gz: 77c7b1faae07696200558929456ab481aa5f8f061f69a0ff923fac53a4957f2c
 SHA512:
-  metadata.gz: 59dd000d01b8fd0b42729d87f7f843f93690b0fbccf927e127426f352fa79aada6078a885b54e3342d3c901500fde93338be3198b77e871df85f7b28ef816f9e
-  data.tar.gz: 29f88c1fdec961ed7b53d9d9e4a3a3c156cadac2da157786bea28fca4b20b09cb786ab195d5eedf0877e11c901a2d140d66347bbee3fc52570a4a68225b78284
+  metadata.gz: 27e9c5abd1ae5a60b42aee5bed88640b3dd49352214b2af420384333b8d93c558369e14220f4f996729d1ccdcf4155552e05dbd59c0ffaf05ca81fe0ea4f9489
+  data.tar.gz: 5e86aa794e2ec154f8e797040cdadd6a59c43489bf98ffdb070f8e7d90a82572e73f19ba5cadedfebdd90fea29062fd0fa873e9bba08cf06f1277483a60e7ee0

data/CHANGELOG.md

@@ -5,6 +5,39 @@ All notable changes to this project are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [9.4.0] - 2026-10-17
+
+### Added
+
+- **The skill family.** `/architect` is now the inline working mode — work
+  outside the loop at a conversational cadence (UI, ops, small fixes), with
+  the space as the workspace (notes kept, `ARCHITECT.md` updated, commits
+  when the work calls for them) and the vocabulary glossary folded in.
+  **`/architect-discuss`** carries the judgment seat: exploration,
+  arbitration, judging finished runs, landing, iteration surgery.
+  **`/architect-status`** reports state read-only from CLI output and the
+  newest notes. The coordinator launch is agent-owned end to end:
+  **`/architect-action`** (renamed from `/architect-plan`) launches via the
+  new `architect_loop` pi-extension tool (jetpks/pi-extensions) — the
+  agent-side equivalent of `/loop`, sharing one spawn path so the two cannot
+  drift, awaiting the run id, then self-compacting the launching session
+  into a small on-call escalation seat.
+
+### Changed
+
+- The `install-skills` help text, `docs/DESIGN.md`, and
+  `docs/reference/project.md` describe the five-skill family; the
+  `/architect-discuss` routing section now describes the real dual
+  agent-file mechanism (per-space `.pi/agents/` generation with resolved
+  pins, plus user-level `install-agents` playbooks as the headless/
+  untrusted fallback) instead of claiming nothing generates into the space.
+
+### Removed
+
+- **`/architect-plan`** (renamed to `/architect-action`) and
+  **`/architect-vocabulary`** (folded into `/architect`); the `dispatch.md`
+  and `research.md` companion references moved to the discuss skill.
+
 ## [9.3.0] - 2026-10-17
 
 ### Removed

data/lib/space_architect/architect_project.rb

@@ -1713,7 +1713,7 @@ module Space::Architect
     # two paths looked at, and the default in use.
     def default_model_inform
       "model: no architect.yaml found (looked at #{Routing.space_path(space.path)} + #{Routing.global_path}) — " \
-        "using default '#{Harness::DEFAULT_MODEL}'"
+        "using default '#{Harness::DEFAULT_MODEL}' #{Routing::INIT_HINT}"
     end
 
     # --prompt: the caller authors the lane prompt anywhere (a fresh scratch file) and

data/lib/space_architect/cli/architect.rb

@@ -169,7 +169,9 @@ module Space::Architect
               # Legacy lanes store no model — resolve the config layer once so the
               # display shows what they would actually route, and mark the bare
               # constant as the last resort it is, not ordinary behavior.
-              config_model = Routing.for_space(sp.path)&.dig("builder", "model")
+              # Fully qualified: the Architect::Routing subcommand namespace
+              # (I09) lexically shadows Space::Architect::Routing inside this module.
+              config_model = Space::Architect::Routing.for_space(sp.path)&.dig("builder", "model")
 
               terminal.say "Project status:     #{block['status'] || '(none)'}"
               terminal.say "Current iteration:  #{block['current_iteration'] || '(none)'}"
@@ -1060,7 +1062,7 @@ module Space::Architect
       end
 
       class InstallSkills < BaseCommand
-        desc "Install bundled skills (architect, architect-research, architect-vocabulary) for a harness"
+        desc "Install bundled skills (architect, architect-discuss, architect-action, architect-status, architect-research) for a harness"
         phase 53, "Project"
         option :provider, default: "claude", desc: "Skill install target (default: claude; validated by the installer)"
         option :project, type: :boolean, default: false, desc: "Install to CWD instead of global"
@@ -1100,6 +1102,56 @@ module Space::Architect
         end
       end
 
+      module Routing
+        # I09: `architect routing init` — the self-service cure for the
+        # missing-config wall (default_model_inform's hint names this command).
+        # Writes Routing::TEMPLATE verbatim and round-trips the write through
+        # Routing.parse before claiming success; the routing machinery itself
+        # is untouched — the command only writes a file it already accepts.
+        class Init < BaseCommand
+          desc "Write a commented starter architect.yaml (space root by default; --global for the machine-wide config)"
+          phase 53, "Project"
+          argument :space, required: false, desc: "Space identifier (default: $PWD)"
+          option :global, type: :boolean, default: false,
+            desc: "Write ~/.config/space-architect/architect.yaml (the machine-wide config; SPACE_ARCHITECT_GLOBAL_CONFIG overrides the path) instead of the space root, creating parent directories as needed"
+
+          def call(space: nil, global: false, **opts)
+            setup_terminal(**opts.slice(:color, :colors))
+            handle_errors do
+              target =
+                if global
+                  raise Space::Core::Error, "--global and an explicit space argument are mutually exclusive" if space
+                  # Routing.global_path is the env-overridable canonical path — never re-derive it here.
+                  path = Space::Architect::Routing.global_path
+                  FileUtils.mkdir_p(path.dirname)
+                  path
+                else
+                  # The house walk-up: empty identifier looks for space.yaml from $PWD upward.
+                  found = store.find(space).value_or(nil)
+                  unless found
+                    raise Space::Core::Error,
+                      "no space.yaml found upward from #{Dir.pwd} — run `architect routing init --global` to scaffold the machine-wide config instead"
+                  end
+                  Space::Architect::Routing.space_path(found.path)
+                end
+
+              raise Space::Core::Error, "architect.yaml already exists: #{target} — routing init never overwrites" if target.exist?
+
+              File.write(target, Space::Architect::Routing::TEMPLATE)
+              # A scaffold that fails its own parser (or misses a required role)
+              # is a bug — validate the round-trip, let the error escape loud.
+              parsed = Space::Architect::Routing.parse(File.read(target), target)
+              missing = Space::Architect::Routing::REQUIRED_ROLES - parsed.keys
+              raise Space::Core::Error, "scaffolded #{target} is missing required roles: #{missing.join(', ')}" if missing.any?
+
+              terminal.say "Wrote starter routing config: #{terminal.path(target.to_s)}"
+              terminal.say "Edit the example/REPLACE_ME models; the next command or /loop picks the file up."
+              CLI.record_outcome(Outcome.new(exit_code: 0))
+            end
+          end
+        end
+      end
+
       module Worktree
         class Add < BaseCommand
           desc "Create a worktree for a lane"
@@ -1340,6 +1392,9 @@ Space::Architect::CLI::Registry.register "worktree" do |wt|
   wt.register "remove", Space::Architect::CLI::Architect::Worktree::Remove
   wt.register "list",   Space::Architect::CLI::Architect::Worktree::List
 end
+Space::Architect::CLI::Registry.register "routing" do |r|
+  r.register "init", Space::Architect::CLI::Architect::Routing::Init
+end
 Space::Architect::CLI::Registry.register "variant" do |v|
   v.register "add",     Space::Architect::CLI::Architect::Variant::Add
   v.register "promote", Space::Architect::CLI::Architect::Variant::Promote

data/lib/space_architect/routing.rb

@@ -22,6 +22,49 @@ module Space::Architect
     REQUIRED_ROLES = %w[spec builder judge coordinator narrator].freeze
     ENTRY_KEYS     = %w[model thinking].freeze
 
+    # I09: the `architect routing init` scaffold — the exact bytes the command
+    # writes, living beside the parsers that must accept it. Example models are
+    # loud REPLACE_MEs; the header comment carries both locations and the merge
+    # order so the file is self-documenting. Byte-exact contract: the extension
+    # lane's tests carry an inline copy, and the command round-trips the write
+    # through parse below — a template the machinery rejects is a bug.
+    TEMPLATE = <<~'YAML'.freeze
+      # architect.yaml — space-architect loop role routing, scaffolded by `architect routing init`.
+      # Locations (merged in this order; the space-local entry wins per role):
+      #   global:  ~/.config/space-architect/architect.yaml   (machine-wide)
+      #   local:   architect.yaml beside this space's space.yaml (this file)
+      # A local entry overrides the global entry for THAT ROLE only; after the
+      # merge every agent role must be routed: spec, builder, judge, coordinator,
+      # narrator (architect is optional).
+      # Replace every example/REPLACE_ME below with a real provider/model-id
+      # (e.g. zai/glm-5.3). thinking is optional: off|minimal|low|medium|high|xhigh|max.
+      # Once saved, the next /loop or architect command picks the file up.
+      version: 1
+      roles:
+        architect:
+          model: example/REPLACE_ME
+          thinking: high
+        spec:
+          model: example/REPLACE_ME
+          thinking: high
+        builder:
+          model: example/REPLACE_ME
+          thinking: high
+        judge:
+          model: example/REPLACE_ME
+          thinking: high
+        coordinator:
+          model: example/REPLACE_ME
+          thinking: high
+        narrator:
+          model: example/REPLACE_ME
+          thinking: low
+    YAML
+
+    # The one canonical cure line appended to default_model_inform's last-resort
+    # inform — self-punctuating, appended after a single space by the caller.
+    INIT_HINT = "Create one: `architect routing init` (beside space.yaml) or `architect routing init --global` (~/.config/space-architect/architect.yaml).".freeze
+
     def self.global_path
       env = ENV["SPACE_ARCHITECT_GLOBAL_CONFIG"]
       return Pathname.new(env) unless env.nil?

data/lib/space_architect/version.rb

@@ -2,6 +2,6 @@
 
 module Space
   module Architect
-    VERSION = "9.3.0"
+    VERSION = "9.4.0"
   end
 end