space-architect 2.0.2 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7251e017d780c7fc4accd3d4ee367d2b5063bb4d0c1692c0c69bc81a228c9fa7
-  data.tar.gz: ad25da0cf6025716953685d7483e86898368170967a29465625335337f8427a3
+  metadata.gz: 135a6282cfe3e3073265b71867e7d228673cbfb9b7253feaaba5ad67a180b7cb
+  data.tar.gz: 0d0406a99feadadebb24a6167c11837de25bede933e752ef6507524208f4e5a7
 SHA512:
-  metadata.gz: baf43e94bcd35987bb133f3958709bfc4083851732ed2d47ef72b25b92e042cfb72c2594f1067189e9581c554140cd2d6145dc7d607f9137931250522ba445dc
-  data.tar.gz: 0ecd90d05ea7dc4c3832fea90f7af76650b1c8279ced9e7e48242b825aaeef8e7291d6dc9b4dacd6bfac00ad647ecca887a2f27abb3823181287885ccede7210
+  metadata.gz: c8b161eff4f4b5a788c2e3c9b423bcc31a495915a076afb70445f6b0cdaf1cc2573ee381ce0c3e09ca712a8398b70ca091b015260b6b4e9c24914ece4a0f5539
+  data.tar.gz: f61cdd897881530ad2e81df5bd846c4e644fb85ac6847c4afd147e643b6045b59c7ecb89fd12b83f9a7b5c2ef12813c5e88e7634449a919aac2e4bc75052a6b0

data/CHANGELOG.md

@@ -5,6 +5,37 @@ 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).
 
+## [3.0.0] - 2026-07-01
+
+### Added
+
+- **`architect bug-report`** — zero-friction bug-filing command. Gathers
+  diagnostics (gem version, Ruby version/platform, and when run inside a space:
+  space id, title, and iteration list with verdicts), writes a prefilled GitHub
+  issue-body template to `<space>/build/bug-report/architect-bug-report-YYYYMMDD-HHMMSS.md`
+  (or `./architect-bug-report-YYYYMMDD-HHMMSS.md` outside a space — timestamped so
+  back-to-back runs in the same session never overwrite each other), and prints —
+  never executes — the `gh issue create -R jetpks/space-architect` invocation to run.
+
+### Removed
+
+- **BREAKING — `architect land` removed** (added in 2.0.0) — landing is the architect
+  skill's procedure: the architect writes the PR body and presents the push +
+  PR command. The command as shipped never produced a runnable block (#25) and
+  authored content the CLI has no context for (#24).
+
+### Fixed
+
+- **`architect bug-report` — `~`-contracted paths in printed commands.**
+  The `gh issue create` invocation printed to stdout renders the
+  `--body-file` path as `~/…` instead of the user's expanded `$HOME`.
+  `Space::Core::Paths.contract` is the single home-contraction helper; `Terminal#path`
+  delegates to it.
+- **Command wrapping for narrow terminals.** The `gh issue create` command is
+  now rendered with trailing ` \` continuations broken at `--flag` boundaries,
+  continuation lines indented two spaces. The wrapped output is valid shell
+  (`bash -n` clean). `Space::Core::Commands.wrap` is the single wrapping helper.
+
 ## [2.0.2] - 2026-07-01
 
 ### Added

data/README.md

@@ -302,7 +302,7 @@ architect evidence <it> --lane <lane>       # transcribe the builder's report ve
 architect gate <iteration>                  # run the frozen gate commands, stream raw output
 architect merge <it> <lane>                 # integrate ONE judged-passing lane (--no-ff)
 architect integrate <it> --lanes a,b        # integrate a set of passing lanes, in order
-architect land                              # end-of-project PR command (no push, no gh)
+# landing (PR body + push/PR command) is the architect's end-of-project procedure (see skill)
 architect status                            # project state (read-only)
 architect variant add|compare|promote …     # competing (harness, model) lanes over one frozen spec
 architect research dispatch|status|wait …   # parallel read-only research lanes (see below)
@@ -321,7 +321,7 @@ architect evidence my-feature --lane lane-a      # transcribe raw evidence
 architect gate my-feature                        # run the frozen gates yourself
 # … read the diff against the spec, then write the Verdict …
 architect integrate my-feature --lanes lane-a    # merge passing lanes → project/<slug>
-architect land                                   # print gh pr create at project end
+# landing (PR body + push/PR command) is the architect's end-of-project procedure (see skill)
 ```
 
 ### Streaming builder output 📡

data/lib/space_architect/architect_project.rb

@@ -415,43 +415,6 @@ module Space::Architect
       merged
     end
 
-    # Generate the end-of-project PR command(s) for each integrated repo.
-    # Writes a PR body to build/land/<repo>-pr-body.md and returns, per repo:
-    # { repo:, integration_branch:, body_file:, command:, context: }.
-    # Raises Space::Core::Error if nothing has been integrated yet.
-    # Side-effect-free: no git write, no push, no gh.
-    def land
-      b = space.data["project"] || {}
-      integration_branch = project_integration_branch
-
-      integrated_lanes = (b["iterations"] || []).flat_map do |s|
-        (s["lanes"] || []).filter_map { |l| { iteration: s, lane: l } if l["integration_branch"] }
-      end
-
-      raise Space::Core::Error, "nothing integrated yet — integrate a lane before landing" if integrated_lanes.empty?
-
-      repos = integrated_lanes.map { |e| e[:lane]["repo"] }.uniq
-
-      repos.map do |repo|
-        body_dir = space.path.join("build", "land")
-        FileUtils.mkdir_p(body_dir)
-        body_path = body_dir.join("#{repo}-pr-body.md")
-
-        iterations = b["iterations"] || []
-        body = +"# #{space.title}\n\nMerges `#{integration_branch}` → `main`.\n\n## Iterations\n\n"
-        iterations.each do |s|
-          nn = format("%02d", s["ordinal"])
-          verdict = s["verdict"] || "—"
-          body << "- I#{nn} #{s["name"]} — #{verdict}\n"
-        end
-        body_path.write(body)
-
-        cmd = %(gh pr create --base main --head #{integration_branch} --title "#{space.title}" --body-file #{body_path})
-        context = "# Run from repos/#{repo} on branch #{integration_branch} (gh pushes it)"
-        { repo: repo, integration_branch: integration_branch, body_file: body_path.to_s, command: cmd, context: context }
-      end
-    end
-
     # Run the iteration's frozen Acceptance Criteria gate commands. Each gate is
     # executed in the resolved cwd (per-gate `cwd` overrides the base dir), under
     # a hard timeout, and evaluated against its `expect` block. Returns an array

data/lib/space_architect/bug_report.rb

@@ -0,0 +1,93 @@
+# frozen_string_literal: true
+
+require "fileutils"
+require "pathname"
+
+module Space
+  module Architect
+    module BugReport
+      REPO = "jetpks/space-architect"
+
+      class << self
+        def generate(space: nil, env: ENV, cwd: Dir.pwd, now: Time.now)
+          body_path = resolve_body_path(space, cwd, now)
+          FileUtils.mkdir_p(body_path.dirname)
+          body = build_body(space)
+          body_path.write(body)
+          contracted = Space::Core::Paths.contract(body_path, env: env)
+          command = Space::Core::Commands.wrap(
+            %(gh issue create -R #{REPO} --title "<one-line summary>" --body-file #{contracted})
+          )
+          { body_path: body_path, command: command, body: body }
+        end
+
+        private
+
+        def resolve_body_path(space, cwd, now)
+          filename = "architect-bug-report-#{now.strftime('%Y%m%d-%H%M%S')}.md"
+          if space
+            space.path.join("build", "bug-report", filename)
+          else
+            Pathname.new(cwd).join(filename)
+          end
+        end
+
+        def build_body(space)
+          body = +template_header
+          body << diagnostics_section
+          body << space_section(space) if space
+          body
+        end
+
+        def template_header
+          <<~MD
+            <!-- Title: <one-line summary> -->
+
+            **Kind:** <!-- process / tooling / both -->
+
+            ## Summary
+
+            <!-- One sentence describing the bug. -->
+
+            ## What happened
+
+            <!-- Describe what you observed. -->
+
+            ## What was expected
+
+            <!-- Describe what you expected to happen. -->
+
+            ## Repro steps
+
+            <!-- Numbered steps to reproduce. -->
+
+          MD
+        end
+
+        def diagnostics_section
+          <<~MD
+            ## Diagnostics
+
+            - space-architect: #{Space::Core::VERSION}
+            - ruby: #{RUBY_VERSION} (#{RUBY_PLATFORM})
+          MD
+        end
+
+        def space_section(space)
+          iterations = Array(space.data.dig("project", "iterations"))
+          iter_lines = iterations.map do |s|
+            nn = format("%02d", s["ordinal"])
+            verdict = s["verdict"] || "—"
+            "- I#{nn} #{s["name"]} — #{verdict}"
+          end.join("\n")
+
+          +"\n## Space context\n\n" \
+            "- Space id: #{space.id}\n" \
+            "- Space title: #{space.title}\n" \
+            "\n### Iterations\n\n" \
+            "#{iter_lines.empty? ? "(none)" : iter_lines}\n"
+        end
+      end
+    end
+  end
+end

data/lib/space_architect/cli/architect.rb

@@ -389,27 +389,6 @@ module Space::Architect
         end
       end
 
-      class Land < BaseCommand
-        desc "Generate the end-of-project PR command (no push, no gh — prints gh pr create)"
-        argument :space, required: false, desc: "Space identifier (default: $PWD)"
-
-        def call(space: nil, **opts)
-          setup_terminal(**opts.slice(:color, :colors))
-          handle_errors do
-            render(store.find(space)) do |sp|
-              project = ArchitectProject.new(space: sp)
-              results = project.land
-              results.each do |r|
-                terminal.say r[:context]
-                terminal.say r[:command]
-                terminal.say "Body: #{terminal.path(r[:body_file])}"
-              end
-              CLI.record_outcome(Outcome.new(exit_code: 0))
-            end
-          end
-        end
-      end
-
       class Gate < BaseCommand
         desc "Run the frozen Acceptance Criteria gate commands and report PASS/FAIL"
         argument :iteration, required: true,  desc: "Iteration name"
@@ -439,6 +418,29 @@ module Space::Architect
         end
       end
 
+      class BugReport < BaseCommand
+        desc "Generate a prefilled GitHub issue template for filing bugs against space-architect"
+
+        def call(**opts)
+          setup_terminal(**opts.slice(:color, :colors))
+          handle_errors do
+            space = store.find.value_or(nil)
+            result = Space::Architect::BugReport.generate(
+              space: space,
+              env: project_config.env
+            )
+            terminal.say "Fill the placeholders in #{terminal.path(result[:body_path].to_s)}, then run:"
+            terminal.say result[:command]
+            terminal.say ""
+            terminal.say "Diagnostics:"
+            terminal.say "  space-architect #{Space::Core::VERSION}"
+            terminal.say "  ruby #{RUBY_VERSION} (#{RUBY_PLATFORM})"
+            terminal.say "  space: #{space.id} — #{space.title}" if space
+            CLI.record_outcome(Outcome.new(exit_code: 0))
+          end
+        end
+      end
+
       class InstallSkills < BaseCommand
         desc "Install bundled skills (architect, architect-research, architect-vocabulary) for a harness"
         option :provider, default: "claude", desc: "Harness: claude, codex, opencode, pi"
@@ -652,9 +654,9 @@ Space::Architect::CLI::Registry.register "verdict",   Space::Architect::CLI::Arc
 Space::Architect::CLI::Registry.register "evidence",  Space::Architect::CLI::Architect::Evidence
 Space::Architect::CLI::Registry.register "merge",     Space::Architect::CLI::Architect::Merge
 Space::Architect::CLI::Registry.register "integrate", Space::Architect::CLI::Architect::Integrate
-Space::Architect::CLI::Registry.register "land",      Space::Architect::CLI::Architect::Land
 Space::Architect::CLI::Registry.register "gate",      Space::Architect::CLI::Architect::Gate
 Space::Architect::CLI::Registry.register "install-skills", Space::Architect::CLI::Architect::InstallSkills
+Space::Architect::CLI::Registry.register "bug-report",     Space::Architect::CLI::Architect::BugReport
 Space::Architect::CLI::Registry.register "brief" do |b|
   b.register "new", Space::Architect::CLI::Architect::Brief::New
 end

data/lib/space_architect.rb

@@ -10,5 +10,6 @@ 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/bug_report"
 require_relative "space_architect/research"
 require_relative "space_architect/cli"

data/lib/space_core/commands.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module Space::Core
+  module Commands
+    module_function
+
+    # Render a multi-flag command with trailing " \" continuations broken at
+    # "--flag" boundaries, continuation lines indented two spaces. Commands
+    # without "--" flags are returned unchanged.
+    def wrap(command)
+      parts = command.split(/(?= --)/)
+      return command if parts.size <= 1
+
+      parts.each_with_index.map do |part, i|
+        segment = i.zero? ? part.rstrip : "  #{part.lstrip}"
+        i < parts.size - 1 ? "#{segment} \\" : segment
+      end.join("\n")
+    end
+  end
+end

data/lib/space_core/paths.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module Space::Core
+  module Paths
+    module_function
+
+    def contract(path, env: ENV)
+      value = path.to_s
+      home  = XDG.home(env: env)
+      [home, realpath_or_nil(home)].compact.uniq.each do |h|
+        return "~"  if value == h
+        return "~#{value.delete_prefix(h)}" if value.start_with?("#{h}/")
+      end
+      value
+    end
+
+    def realpath_or_nil(path)
+      File.realpath(path)
+    rescue SystemCallError
+      nil
+    end
+  end
+end

data/lib/space_core/terminal.rb

@@ -48,13 +48,7 @@ module Space::Core
     end
 
     def path(path)
-      value = path.to_s
-      homes.each do |home|
-        return "~" if value == home
-        return "~#{value.delete_prefix(home)}" if value.start_with?("#{home}/")
-      end
-
-      value
+      Paths.contract(path, env: config.env)
     end
 
     def with_spinner(message)
@@ -108,17 +102,6 @@ module Space::Core
       end
     end
 
-    def homes
-      home = XDG.home(env: config.env)
-      [home, realpath_or_nil(home)].compact.uniq
-    end
-
-    def realpath_or_nil(path)
-      File.realpath(path)
-    rescue SystemCallError
-      nil
-    end
-
     def table_row(headers, row, column_widths, header: false)
       row.each_with_index.map do |cell, index|
         raw = cell.to_s

data/lib/space_core/version.rb

@@ -2,6 +2,6 @@
 
 module Space
   module Core
-    VERSION = "2.0.2"
+    VERSION = "3.0.0"
   end
 end

data/lib/space_core.rb

@@ -10,6 +10,8 @@ require "space_core/warnings"
 Space::Core::Warnings.disable_experimental!
 require "space_core/atomic_write"
 require "space_core/xdg"
+require "space_core/paths"
+require "space_core/commands"
 require "space_core/config"
 require "space_core/state"
 require "space_core/slugger"

data/skill/architect/SKILL.md

@@ -110,6 +110,7 @@ loop.
 8. **Stop conditions:** failing verification you can't root-cause, instructions
    conflicting with project docs, irreversible/destructive calls, or scope
    growth beyond the iteration → checkpoint to the handoff and ask the human.
+   For bugs in the architect tooling or process itself, run `architect bug-report` — it prints a prefilled issue template and the `gh` command to file it.
 
 ## Procedure
 
@@ -373,10 +374,15 @@ cited BRIEF §sections (per §2), then write the **Verdict** (`architect verdict
 <iteration> continue|kill --from <file>`): disagreement rulings, per-AC
 PASS/FAIL/INVALID, the KILL/CONTINUE call.
 
-At project end, `architect land` prints the single `gh pr create --base main
---head project/<slug>` command per touched repo and writes a PR body to
-`build/land/` — no push, no `gh` call; the human runs it from the repo when
-the project is ready to ship.
+At project end, landing is yours, not the CLI's — the PR body is judgment
+output, the same class as a Verdict. Per touched repo: write the PR body
+yourself — from the iteration verdicts, the integrated diff, and the BRIEF —
+to `build/land/<repo>-pr-body.md`, then present the paste-and-run block to the
+human: `cd` to the repo checkout, `git push -u origin project/<slug>`, and
+`gh pr create --base main --head project/<slug> --title … --body-file …` —
+paths `~`-contracted, the multi-flag command broken with trailing ` \` at flag
+boundaries. No push, no `gh` call by you or the CLI; the human runs the block
+when the project is ready to ship.
 
 ## Maintenance
 

data/skill/architect/dispatch.md

@@ -112,7 +112,9 @@ lanes pass; the CLI does the git mechanics. Canonical path:
 architect integrate <iteration> --lanes <passing-set>   # e.g. --lanes lane-a,lane-b
 architect gate <iteration>                              # integration smoke (raw output; verdict stays yours)
 architect integrate <iteration> --lanes <passing-set> --teardown   # or remove worktrees + lane branches after
-architect land                                          # end of project: prints gh pr create --base main --head project/<slug>
+# end of project: landing is the architect's, not a CLI command — write the PR
+# body to build/land/<repo>-pr-body.md yourself, then present the paste-and-run
+# block (cd, git push -u origin project/<slug>, gh pr create) — see SKILL.md §6
 ```
 
 `architect integrate` commits each named lane on its branch and merges it
@@ -138,8 +140,8 @@ git -C repos/<repo> merge --no-ff lane/<iteration>-<lane>
 <run the gate commands>          # integration smoke after every merge
 architect worktree remove <iteration> <lane>
 git -C repos/<repo> branch -d lane/<iteration>-<lane>
-# at project end:
-architect land                   # prints gh pr create --base main --head project/<slug>
+# at project end there is no CLI step: the architect writes
+# build/land/<repo>-pr-body.md and presents the push + gh pr create block
 ```
 
 ### Parallel + fast-follow

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: space-architect
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 3.0.0
 platform: ruby
 authors:
 - Eric Jacobs
@@ -240,6 +240,7 @@ files:
 - exe/src
 - lib/space_architect.rb
 - lib/space_architect/architect_project.rb
+- lib/space_architect/bug_report.rb
 - lib/space_architect/cli.rb
 - lib/space_architect/cli/architect.rb
 - lib/space_architect/cli/research.rb
@@ -281,6 +282,7 @@ files:
 - lib/space_core/cli/show.rb
 - lib/space_core/cli/status.rb
 - lib/space_core/cli/use.rb
+- lib/space_core/commands.rb
 - lib/space_core/config.rb
 - lib/space_core/errors.rb
 - lib/space_core/git_client.rb
@@ -288,6 +290,7 @@ files:
 - lib/space_core/oci_builder.rb
 - lib/space_core/oci_packer.rb
 - lib/space_core/oci_runner.rb
+- lib/space_core/paths.rb
 - lib/space_core/repo_reference.rb
 - lib/space_core/repo_resolver.rb
 - lib/space_core/shell_integration.rb