space-architect 1.3.0 → 2.0.0.rc2

data/lib/space_architect/run_creator.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+require "async/http/client"
+require "async/http/endpoint"
+require "json"
+
+module Space::Architect
+  class RunCreator
+    def initialize(host, token, client: nil)
+      @host   = host.chomp("/")
+      @token  = token
+      @client = client
+    end
+
+    # POSTs to /runs and returns the integer run id.
+    # Raises Space::Core::Error on any failure — never returns nil.
+    def create
+      Sync do
+        if @client
+          response = @client.post("/runs", headers: headers, body: nil)
+          parse_response(response)
+        else
+          Async::HTTP::Client.open(Async::HTTP::Endpoint.parse(@host)) do |c|
+            response = c.post("/runs", headers: headers, body: nil)
+            parse_response(response)
+          end
+        end
+      end
+    end
+
+    private
+
+    def headers
+      [
+        ["authorization", "Bearer #{@token}"],
+        ["content-type",  "application/json"]
+      ]
+    end
+
+    def parse_response(response)
+      status = response.status
+      body   = response.read || ""
+      raise Space::Core::Error, "POST /runs failed (#{status}): #{body[0, 200]}" unless status == 201
+
+      parsed = JSON.parse(body)
+      id     = parsed["id"]
+      raise Space::Core::Error, "POST /runs: missing or non-integer id in response: #{body[0, 200]}" \
+        unless id.is_a?(Integer)
+
+      id
+    end
+  end
+end

data/lib/space_architect/skill_installer.rb

@@ -3,103 +3,105 @@
 require "fileutils"
 require "pathname"
 
-module SpaceArchitect
-  module SkillInstaller
-    PROVIDERS = %w[claude codex opencode pi].freeze
-
-    class << self
-      def source_root
-        Pathname.new(__dir__).parent.parent.join("skill")
-      end
-
-      def dest_root(provider, project:, env:, cwd: Dir.pwd)
-        case provider.to_s
-        when "claude"
-          base = project ? Pathname.new(cwd) : Pathname.new(XDG.home(env: env))
-          base.join(".claude", "skills")
-        when "codex"
-          base = project ? Pathname.new(cwd) : Pathname.new(XDG.home(env: env))
-          base.join(".agents", "skills")
-        when "opencode"
-          project ? Pathname.new(cwd).join(".opencode", "skills") : XDG.config_home(env: env).join("skills")
-        when "pi"
-          base = project ? Pathname.new(cwd) : Pathname.new(pi_agent_dir(env: env))
-          base.join("skills")
-        else
-          raise Error, "Unknown provider '#{provider}'. Expected one of: #{PROVIDERS.join(', ')}"
+module Space
+  module Architect
+    module SkillInstaller
+      PROVIDERS = %w[claude codex opencode pi].freeze
+
+      class << self
+        def source_root
+          Pathname.new(__dir__).parent.parent.join("skill")
         end
-      end
-
-      def install(provider, project:, force:, env:, cwd: Dir.pwd, dry_run: false)
-        validate_provider!(provider)
-        dest = dest_root(provider, project: project, env: env, cwd: cwd)
-        results = []
 
-        source_skills.each do |skill_dir|
-          name = skill_dir.basename.to_s
-          skill_dest = dest.join(name)
-          results << install_skill(skill_dir, skill_dest, force: force, dry_run: dry_run)
+        def dest_root(provider, project:, env:, cwd: Dir.pwd)
+          case provider.to_s
+          when "claude"
+            base = project ? Pathname.new(cwd) : Pathname.new(Space::Core::XDG.home(env: env))
+            base.join(".claude", "skills")
+          when "codex"
+            base = project ? Pathname.new(cwd) : Pathname.new(Space::Core::XDG.home(env: env))
+            base.join(".agents", "skills")
+          when "opencode"
+            project ? Pathname.new(cwd).join(".opencode", "skills") : Space::Core::XDG.config_home(env: env).join("skills")
+          when "pi"
+            base = project ? Pathname.new(cwd) : Pathname.new(pi_agent_dir(env: env))
+            base.join("skills")
+          else
+            raise Space::Core::Error, "Unknown provider '#{provider}'. Expected one of: #{PROVIDERS.join(', ')}"
+          end
         end
 
-        { dest_root: dest, skills: results, dry_run: dry_run }
-      end
-
-      def source_skills
-        source_root.children.select(&:directory?)
-      end
+        def install(provider, project:, force:, env:, cwd: Dir.pwd, dry_run: false)
+          validate_provider!(provider)
+          dest = dest_root(provider, project: project, env: env, cwd: cwd)
+          results = []
 
-      private
-
-      def validate_provider!(provider)
-        return if PROVIDERS.include?(provider.to_s)
+          source_skills.each do |skill_dir|
+            name = skill_dir.basename.to_s
+            skill_dest = dest.join(name)
+            results << install_skill(skill_dir, skill_dest, force: force, dry_run: dry_run)
+          end
 
-        raise Error, "Unknown provider '#{provider}'. Expected one of: #{PROVIDERS.join(', ')}"
-      end
+          { dest_root: dest, skills: results, dry_run: dry_run }
+        end
 
-      def pi_agent_dir(env:)
-        Pathname.new(env.fetch("PI_CODING_AGENT_DIR", File.join(XDG.home(env: env), ".pi", "agent")))
-      end
+        def source_skills
+          source_root.children.select(&:directory?)
+        end
 
-      def install_skill(source, dest, force:, dry_run:)
-        name = source.basename.to_s
+        private
 
-        if dest.exist?
-          if same_content?(source, dest)
-            return { name: name, action: :unchanged, path: dest }
-          end
+        def validate_provider!(provider)
+          return if PROVIDERS.include?(provider.to_s)
 
-          unless force
-            return { name: name, action: :conflict, path: dest } if dry_run
+          raise Space::Core::Error, "Unknown provider '#{provider}'. Expected one of: #{PROVIDERS.join(', ')}"
+        end
 
-            raise Error,
-              "Refusing to overwrite existing skill at #{dest}. Re-run with --force."
-          end
+        def pi_agent_dir(env:)
+          Pathname.new(env.fetch("PI_CODING_AGENT_DIR", File.join(Space::Core::XDG.home(env: env), ".pi", "agent")))
+        end
 
-          unless dry_run
-            FileUtils.rm_rf(dest)
-            FileUtils.cp_r(source, dest)
-          end
-          { name: name, action: dry_run ? :would_update : :updated, path: dest }
-        else
-          unless dry_run
-            FileUtils.mkdir_p(dest.parent)
-            FileUtils.cp_r(source, dest)
+        def install_skill(source, dest, force:, dry_run:)
+          name = source.basename.to_s
+
+          if dest.exist?
+            if same_content?(source, dest)
+              return { name: name, action: :unchanged, path: dest }
+            end
+
+            unless force
+              return { name: name, action: :conflict, path: dest } if dry_run
+
+              raise Space::Core::Error,
+                "Refusing to overwrite existing skill at #{dest}. Re-run with --force."
+            end
+
+            unless dry_run
+              FileUtils.rm_rf(dest)
+              FileUtils.cp_r(source, dest)
+            end
+            { name: name, action: dry_run ? :would_update : :updated, path: dest }
+          else
+            unless dry_run
+              FileUtils.mkdir_p(dest.parent)
+              FileUtils.cp_r(source, dest)
+            end
+            { name: name, action: dry_run ? :would_install : :installed, path: dest }
           end
-          { name: name, action: dry_run ? :would_install : :installed, path: dest }
         end
-      end
 
-      def same_content?(source, dest)
-        return false unless dest.directory?
+        def same_content?(source, dest)
+          return false unless dest.directory?
 
-        source_files = Dir.glob("#{source}/**/*").reject { |f| File.directory?(f) }
-        dest_files = Dir.glob("#{dest}/**/*").reject { |f| File.directory?(f) }
+          source_files = Dir.glob("#{source}/**/*").reject { |f| File.directory?(f) }
+          dest_files = Dir.glob("#{dest}/**/*").reject { |f| File.directory?(f) }
 
-        return false if source_files.length != dest_files.length
+          return false if source_files.length != dest_files.length
 
-        source_files.sort.zip(dest_files.sort).all? do |sf, df|
-          rel = sf.sub("#{source}/", "")
-          df.end_with?(rel) && File.read(sf) == File.read(df)
+          source_files.sort.zip(dest_files.sort).all? do |sf, df|
+            rel = sf.sub("#{source}/", "")
+            df.end_with?(rel) && File.read(sf) == File.read(df)
+          end
         end
       end
     end

data/lib/space_architect/templates/architect.md.erb

@@ -2,11 +2,11 @@
 
 > Cross-iteration table of contents for the Architect Loop. Per-iteration detail lives in
 > architecture/I<NN>-<iteration>.md — this file only indexes the iterations and carries
-> mission-wide state. Keep it short (~150 lines): the next session must grok it
+> project-wide state. Keep it short (~150 lines): the next session must grok it
 > in under a minute. Not in the committed architecture = didn't happen.
 >
-> Durable mission contract: `architecture/BRIEF.md` (numbered §sections, cited as BRIEF §N).
-> Create it with `architect brief new`. Edits to a §section are mission-scope decisions —
+> Durable project contract: `architecture/BRIEF.md` (numbered §sections, cited as BRIEF §N).
+> Create it with `architect brief new`. Edits to a §section are project-scope decisions —
 > log them in the Decisions log below, never as silent per-iteration drift.
 
 ## TL;DR (keep current)
@@ -34,13 +34,24 @@
 
 ## Iteration index
 
-<!-- Add a row per iteration. Create iterations with `architect new <name>`. -->
+<!-- Rows are added here only for SCAFFOLDED iterations. `architect new <name>` allocates
+     the ordinal at spec-time and scaffolds architecture/I<NN>-<name>.md. Do NOT
+     pre-number planned work — un-numbered planned items live in the Backlog below. -->
 
 | I# | Iteration | Status | freeze_sha | Integration branch | Verdict | File |
 |----|-----------|--------|-----------|--------------------|---------|------|
 
 Status values: speccing → frozen → dispatched → in-flight → awaiting-verdict → done.
 
+## Backlog (ordered; no ordinal until `architect new`)
+
+<!-- Next-up first. Each item gets its ordinal only when it is about to be specced:
+     run `architect new <name>` at that point to allocate the ordinal at spec-time and
+     scaffold the iteration file. Pre-numbering planned work forces renumber churn every
+     time priorities reshuffle. -->
+
+- _[next planned item]_
+
 ## Open items for the human / architect
 
 <!-- Blocking items: unresolved disagreements (which iteration), scope questions,

data/lib/space_architect/templates/brief.md.erb

@@ -1,11 +1,11 @@
 # BRIEF — <%= @_title %>
 
-> Durable mission contract for the Architect Loop. The numbered §sections below are the
+> Durable project contract for the Architect Loop. The numbered §sections below are the
 > stable, cross-iteration address space: every iteration cites them as **BRIEF §N** in its
 > Grounds, Specification, Acceptance Criteria, and Verdict (e.g. `(BRIEF §3.1)`), the way each
-> gate addresses its intent back to one frozen reference. Frozen early; edits are mission-scope
+> gate addresses its intent back to one frozen reference. Frozen early; edits are project-scope
 > decisions logged in ARCHITECT.md's Decisions log, never silent per-iteration drift. Optional —
-> a discovery mission may defer this and cite per-iteration Grounds until the shape stabilizes,
+> a discovery project may defer this and cite per-iteration Grounds until the shape stabilizes,
 > then promote the consolidated picture here once.
 
 ## 1. Goal & non-goals
@@ -38,6 +38,6 @@ frozen proof. An iteration's Specification cites its row here (e.g. "BRIEF §5 S
 <!-- Known risks and the things a builder's PHASE 0 should challenge — cited from specs as
 "BRIEF §6". -->
 
-## 7. Definition of done (whole mission)
+## 7. Definition of done (whole project)
 
-<!-- The mission-level DoD. Iteration gates diff against "BRIEF §7 DoD". -->
+<!-- The project-level DoD. Iteration gates diff against "BRIEF §7 DoD". -->

data/lib/space_architect/templates/iteration.md.erb

@@ -32,16 +32,27 @@ Write + commit: `architect section <%= @_name %> specification --from <file>`. -
 
 ## Acceptance Criteria
 
-<!-- PROOF. Exact gate commands + thresholds. `architect freeze <%= @_name %>`
-commits this file and records its SHA as freeze_sha. Read-only afterward — any
-change to Grounds/Specification/Acceptance Criteria = automatic iteration FAIL. -->
+<!-- PROOF. Write the prose conditions of correctness (AC1, AC2, …) that the
+architect judges against. Runnable checks live in the fenced ```gates block below
+(parsed at freeze time — absent or empty is allowed; malformed fails freeze).
+`architect freeze <%= @_name %>` commits this file and records its SHA as
+freeze_sha. Read-only afterward — any change to Grounds/Specification/Acceptance
+Criteria = automatic iteration FAIL. -->
 
 > Gate-pass is necessary, not sufficient: the architect also reads the diff against the
 > cited **BRIEF §sections** intent and the §1 cardinal invariant before the verdict.
 
-| AC# | Command | Threshold | Brief § |
-|-----|---------|-----------|---------|
-|     |         |           |         |
+**AC1.** ...
+
+```gates
+# Each gate backs one prose AC above. Remove the comment markers to activate.
+# - id: suite-green        # unique slug within the iteration (required)
+#   ac: AC1                # which prose AC this gate backs (required)
+#   cwd: repos/my-repo     # run dir, relative to space root (optional; consumed by I03)
+#   cmd: bundle exec rake test  # shell command (required, non-empty)
+#   expect:                # at least one of: exit_code, stdout_match, threshold
+#     exit_code: 0
+```
 
 ## Builder Prompt
 

data/lib/space_architect.rb

@@ -1,27 +1,14 @@
 # frozen_string_literal: true
 
-$LOAD_PATH.unshift File.expand_path("../vendor/repo-tender/lib", __dir__)
-require "space_architect/pristine"
+require "space_core"
+require "space_src"
 
-require_relative "space_architect/version"
-require_relative "space_architect/errors"
-require_relative "space_architect/warnings"
-SpaceArchitect::Warnings.disable_experimental!
-require_relative "space_architect/atomic_write"
-require_relative "space_architect/xdg"
-require_relative "space_architect/config"
-require_relative "space_architect/state"
-require_relative "space_architect/slugger"
-require_relative "space_architect/space"
-require_relative "space_architect/repo_reference"
-require_relative "space_architect/repo_resolver"
-require_relative "space_architect/git_client"
-require_relative "space_architect/mise_client"
-require_relative "space_architect/space_store"
-require_relative "space_architect/shell_integration"
-require_relative "space_architect/skill_installer"
-require_relative "space_architect/terminal"
 require_relative "space_architect/harness"
+require_relative "space_architect/run_creator"
 require_relative "space_architect/dispatcher"
-require_relative "space_architect/architect_mission"
+require_relative "space_architect/gate_lint"
+require_relative "space_architect/gate_evaluator"
+require_relative "space_architect/architect_project"
+require_relative "space_architect/skill_installer"
+require_relative "space_architect/research"
 require_relative "space_architect/cli"

data/lib/{space_architect → space_core}/atomic_write.rb

@@ -2,7 +2,7 @@
 
 require "fileutils"
 
-module SpaceArchitect
+module Space::Core
   module AtomicWrite
     module_function
 

data/lib/space_core/cli/base_command.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require "dry/cli"
+
+module Space::Core::CLI
+  # Base for every space/architect command. dry-cli (>= 0.7.0) copies a
+  # superclass's options to its subclasses, so the global colour options are
+  # declared once here and inherited everywhere instead of being mixed in per
+  # command. Helpers (terminal/store/render) ride along by inheritance too.
+  #
+  # The `src` binary has its own output-mode system (--plain/--json) and does
+  # NOT inherit from this base.
+  class BaseCommand < Dry::CLI::Command
+    include Helpers
+
+    option :color,  type: :string, default: "auto", desc: "Color output: auto, always, never"
+    option :colors, type: :string, desc: "Alias for --color"
+  end
+end

data/lib/space_core/cli/build.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+require_relative "../oci_packer"
+require_relative "../oci_builder"
+
+module Space::Core::CLI
+class Build < BaseCommand
+  desc "Build (and tag) the OCI image for the current space"
+
+  def call(**opts)
+    setup_terminal(**opts.slice(:color, :colors))
+    handle_errors do
+      result = store.current.bind do |space|
+        out_dir = space.path.join("build", "oci")
+        Space::Core::OciPacker.new(space: space, output_dir: out_dir).generate.bind do
+          Space::Core::OciBuilder.new(space: space, output_dir: out_dir).command
+        end
+      end
+      render(result) do |argv|
+        terminal.say "Building: #{argv.join(' ')}"
+        out.flush
+        Kernel.exec(*argv)
+      end
+    end
+  end
+end
+end

data/lib/space_core/cli/config.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+module Space::Core::CLI
+module Config
+  class Show < BaseCommand
+    desc "Show current config"
+
+    def call(**opts)
+      setup_terminal(**opts.slice(:color, :colors))
+      handle_errors do
+        rows = Space::Core::Config::EDITABLE_KEYS.map do |key|
+          value = project_config.data[key]
+          [key, value.nil? ? "" : value.to_s]
+        end
+        terminal.say terminal.table(%w[Key Value], rows)
+        CLI.record_outcome(Outcome.new(exit_code: 0))
+      end
+    end
+  end
+
+  class ConfigPath < BaseCommand
+    desc "Print the config file path"
+
+    def call(**opts)
+      setup_terminal(**opts.slice(:color, :colors))
+      handle_errors do
+        terminal.say terminal.path(project_config.path)
+        CLI.record_outcome(Outcome.new(exit_code: 0))
+      end
+    end
+  end
+
+  class Set < BaseCommand
+    desc "Set a config key"
+    argument :key, required: true, desc: "Config key"
+    argument :value, required: true, desc: "Config value"
+
+    def call(key:, value:, **opts)
+      setup_terminal(**opts.slice(:color, :colors))
+      handle_errors do
+        project_config.set(key, value)
+        stored = project_config.data[key]
+        terminal.success "Set #{key}=#{stored.nil? ? '' : stored.to_s}"
+        CLI.record_outcome(Outcome.new(exit_code: 0))
+      end
+    end
+  end
+end
+end

data/lib/space_core/cli/current.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+module Space::Core::CLI
+class Current < BaseCommand
+  desc "Show the current space"
+
+  def call(**opts)
+    setup_terminal(**opts.slice(:color, :colors))
+    render(store.find) do |space|
+      terminal.say space.id.to_s
+      terminal.say terminal.path(space.path)
+      CLI.record_outcome(Outcome.new(exit_code: 0))
+    end
+  end
+end
+end

data/lib/space_core/cli/help.rb

@@ -0,0 +1,123 @@
+# frozen_string_literal: true
+
+require "dry/cli"
+require "pastel"
+
+module Space::Core::CLI
+  # Colourful replacement for dry-cli's plain `Usage` listing — the "global
+  # help" shown by `space`, `architect`, and every bare namespace (`space repo`,
+  # `worktree`, ...). Per-command help still flows through dry-cli's Banner,
+  # whose content we like; only the listing is reskinned.
+  #
+  # We reopen Dry::CLI::Usage.call (below) to delegate here, so BOTH the
+  # intercepted top-level help and dry-cli's own bare-namespace path get the
+  # same treatment from one place. Colour follows CLI.help_pastel, set once per
+  # invocation from the output stream's tty-ness and --color, so piped and test
+  # output stay plain. The `src` binary never loads space_core, so its own plain
+  # Usage is untouched.
+  module Help
+    # One tagline per binary — the root header is served for both `space` and
+    # `architect` from here, so we pick by $PROGRAM_NAME. The default covers the
+    # test runner and any other invocation where the binary can't be identified.
+    TAGLINES = {
+      "space"     => "date-prefixed workspaces; repos provisioned on fibers at copy-on-write speed",
+      "architect" => "the Architect Loop: structured judgment plus a fleet of headless AI builders"
+    }.freeze
+    DEFAULT_TAGLINE = TAGLINES.fetch("space")
+
+    module_function
+
+    def call(result, pastel: CLI.help_pastel)
+      rows  = listing(result)
+      width = rows.map { |label, _| label.length }.max || 0
+
+      lines = rows.map do |label, description|
+        painted = pastel.cyan(label.ljust(width))
+        description ? "  #{painted}   #{pastel.dim("# #{description}")}" : "  #{painted}"
+      end
+
+      [header(result, pastel), pastel.bold("Commands:"), *lines, footer(result, pastel)]
+        .compact.join("\n")
+    end
+
+    # The richer header only makes sense at the true root (`space` / `architect`),
+    # not on every sub-namespace listing.
+    def header(result, pastel)
+      return unless result.names.empty?
+
+      "#{pastel.bold.cyan("space-architect")} #{pastel.dim(Space::Core::VERSION)} " \
+        "#{pastel.dim("— #{tagline}")}\n"
+    end
+
+    # The tagline for the binary in hand: `space` and `architect` each get their
+    # own, keyed off $PROGRAM_NAME (the same signal program_prefix reads).
+    def tagline
+      TAGLINES.fetch(File.basename($PROGRAM_NAME), DEFAULT_TAGLINE)
+    end
+
+    def footer(result, pastel)
+      "\n#{pastel.dim("Run `#{program_prefix(result)} <command> --help` for details on a command.")}"
+    end
+
+    # "space" at the root, "space repo" inside a namespace. The `space`/`architect`
+    # binaries inject their name into ARGV, so $PROGRAM_NAME and the leading
+    # namespace segment can collide ("space space ..."); drop the duplicate.
+    def program_prefix(result)
+      prog  = File.basename($PROGRAM_NAME)
+      names = result.names.dup
+      names.shift if names.first == prog
+      [prog, *names].join(" ")
+    end
+
+    # [[label_with_banner, description_or_nil], ...] sorted by command name.
+    def listing(result)
+      result.children.sort_by { |name, _| name }.filter_map do |name, node|
+        next if node.hidden
+
+        [label(result, name, node), description(node)]
+      end
+    end
+
+    def label(result, name, node)
+      "#{program_prefix(result)} #{name}#{banner(node)}"
+    end
+
+    def banner(node)
+      if node.command && node.leaf? && node.children?
+        " [ARGUMENT|SUBCOMMAND]"
+      elsif node.leaf?
+        arguments(node.command)
+      else
+        " [SUBCOMMAND]"
+      end
+    end
+
+    def arguments(command)
+      return "" unless command.respond_to?(:required_arguments)
+
+      names = command.required_arguments.map { |arg| arg.name.to_s.upcase }
+      names += command.optional_arguments.map { |arg| "[#{arg.name.to_s.upcase}]" }
+      names.empty? ? "" : " #{names.join(" ")}"
+    end
+
+    def description(node)
+      return unless node.leaf? && node.command.respond_to?(:description)
+
+      node.command.description
+    end
+  end
+end
+
+# Route dry-cli's plain namespace/root listing through our colourful renderer.
+# We replace Usage.call wholesale and depend only on the LookupResult/Node API
+# (children, command, leaf?/children?/hidden, names) rather than copying Usage's
+# internals — see notes/ruby-cli-gems-report.md.
+module Dry
+  class CLI
+    module Usage
+      def self.call(result)
+        Space::Core::CLI::Help.call(result)
+      end
+    end
+  end
+end