ask-coding-harness 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bb36f1a55f6b263fdbffd97383c1a744d9b4864a74d04d3003270134437bc3fd
-  data.tar.gz: f2a6fd12ffb36d5fff6baf1d59151c913dd5596db22a5d6abd366494d57be328
+  metadata.gz: 3fef50d92fb06108330d8976a625372fb12ec8edca50ea5e777317da3e36e2b4
+  data.tar.gz: e8da881f33b685ca384653d9d5fcdc825f2cf954aa117b14e544d3c39f247a54
 SHA512:
-  metadata.gz: 37ba3d010ae2ca1f044e919b2e82f3fc55e82bf96d792c65896e5e4141a225cecbcdc8e82c07d06eca24b12089d4e624b679b2a8fc5bd42d149b7e3047e37333
-  data.tar.gz: 3cd3c97b80c739cd6b9a6182241083856d514a2aee9f1ee6c8758f48671847377efe19f67a3a232c4eb412c725b51b3b3acd0b9f2df59381177bdf21e211643b
+  metadata.gz: cd3e2adfef79ac48479349187ea5f1f10ee3a6a2f041235a3466f021ce25c3d1f0ac7b23145d42895a09fb5a9c7694246fee98bb73474ad8c57c9b965269fae6
+  data.tar.gz: a9ca377914655989f1524cc06a46380b046b6d0a9e22b53bb8a7575066f2fb3375a436574d30877e37b72e7b8f0c2df5256e905dbeeabf11f35141553a16ab4c

data/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 ## [0.2.0] - 2026-08-10
 
+### Added
+
+- **Universal workspaces.** The harness is no longer bound to one
+  directory: open any project from the UI (workspace switcher + Open
+  workspace dialog), switch between them, and every conversation runs
+  inside its own workspace with its own system prompt.
+  - `POST /api/workspaces` (open/register), `GET /api/workspaces`
+    (list with name/branch/counts), `GET /api/workspaces/:path/info`.
+  - `POST /api/chat` accepts a `workspace` param; conversations are
+    created and grouped per workspace.
+  - Turns execute inside their workspace directory (serialized via a
+    turn mutex, since the shell tools default to `Dir.pwd`).
+- **Pi-style system prompts.** `SystemPrompt` builds a composable prompt:
+  default or custom base, guidelines, `<project_context>` from
+  AGENTS.md/CLAUDE.md (walked from the workspace up to the root, like the
+  pi coding agent), an append section, and a `Current working directory:`
+  footer. Config: `system_prompt`, `system_prompt_append`,
+  `system_prompt_guidelines` (env `ACH_SYSTEM_PROMPT` still appends).
+- **ask-ui-kit sidebar/shell components** extracted from the popular
+  coding agents' UIs (openchamber, openwebui, t3code): `ask-dialog`,
+  `ask-menu`, `ask-search-input`, `ask-conversation-item`,
+  `ask-conversation-group`. The harness frontend now builds its switcher,
+  sidebar (search + grouped conversation items), and dialogs on them.
+
+### Fixed
+
+- `DemoAdapter#create_session` accepts the `system_prompt` keyword (the
+  universal runner passes it to every adapter).
+
+
 ### Added
 
 - **Demo adapter and `ach demo`** — a scripted coding agent (todos, tool

data/README.md

@@ -64,7 +64,7 @@ ach version
 | `ACH_PLAN_MODE` | off | plan mode (research first, then execute) |
 | `ACH_TODOS` | on | todo list tool |
 | `ACH_DB_PATH` | `./data/ask-coding-harness.db` | conversation database |
-| `ACH_SYSTEM_PROMPT` | — | extra system prompt lines |
+| `ACH_SYSTEM_PROMPT` | — | extra system prompt lines (append section) |
 
 Programmatic use:
 

data/lib/ask/coding_harness/agent_runner.rb

@@ -41,6 +41,7 @@ module Ask
         @adapter = nil
         @mutex = Mutex.new
         @adapter_mutex = Mutex.new
+        @turn_mutex = Mutex.new
         @sessions = {}
         @turns = {}
       end
@@ -127,7 +128,8 @@ module Ask
       private
 
       def run_turn(conversation, prompt, model: nil, &on_event)
-        sid = session_for(conversation, model: model)
+        workspace = conversation["directory"] || @config.workspace
+        sid = session_for(conversation, workspace, model: model)
 
         conv = @store.load(conversation["id"]) || conversation
         conv["messages"] << { "role" => "user", "content" => prompt, "created_at" => Time.now.iso8601 }
@@ -135,24 +137,37 @@ module Ask
 
         outcome = :completed
         accumulated = +""
-        adapter.send_and_stream(sid, prompt, turn_timeout: @config.turn_timeout) do |event|
-          translated = @translator.translate(event)
-          next unless translated
-
-          case translated[:type]
-          when "message.delta"
-            accumulated << translated[:data][:delta].to_s
-          when "turn.failed"
-            outcome = :failed
-          when "turn.aborted"
-            outcome = :aborted
+        with_workspace(workspace) do
+          adapter.send_and_stream(sid, prompt, turn_timeout: @config.turn_timeout) do |event|
+            translated = @translator.translate(event)
+            next unless translated
+
+            case translated[:type]
+            when "message.delta"
+              accumulated << translated[:data][:delta].to_s
+            when "turn.failed"
+              outcome = :failed
+            when "turn.aborted"
+              outcome = :aborted
+            end
+            emit(on_event, translated)
           end
-          emit(on_event, translated)
         end
 
         persist_outcome(conversation["id"], accumulated, outcome)
       end
 
+      # Run the block with the workspace as the process working directory.
+      # The shell tools default to Dir.pwd, so the turn must execute inside
+      # its workspace; a turn mutex keeps concurrent workspaces from racing
+      # on the process-global directory. Self-hosted single-user: turns
+      # serialize, which is both safe and simple.
+      def with_workspace(workspace)
+        @turn_mutex.synchronize do
+          Dir.chdir(workspace) { yield }
+        end
+      end
+
       # Persist the assistant turn outcome (response, error, or abort) so
       # the conversation reads correctly when reopened. Best-effort; the
       # streamed events are the source of truth for the live view.
@@ -175,16 +190,31 @@ module Ask
         nil
       end
 
-      # One adapter session per conversation, created on first use.
-      def session_for(conversation, model: nil)
+      # One adapter session per conversation, created on first use. Each
+      # session gets its workspace's system prompt (pi-style: default or
+      # custom base + guidelines + AGENTS.md/CLAUDE.md project context).
+      def session_for(conversation, workspace = nil, model: nil)
+        workspace ||= conversation["directory"] || @config.workspace
         @mutex.synchronize do
           @sessions[conversation["id"]] ||= adapter.create_session(
-            conversation["directory"] || @config.workspace,
-            model: model || @config.model
+            workspace,
+            model: model || @config.model,
+            system_prompt: system_prompt_for(workspace)
           )
         end
       end
 
+      # The system prompt for a workspace. Adapters that don't accept a
+      # per-session prompt (external ACP agents) ignore the extra keyword.
+      def system_prompt_for(workspace)
+        Ask::CodingHarness::SystemPrompt.build(
+          workspace: workspace,
+          custom: @config.system_prompt,
+          append: @config.system_prompt_append,
+          guidelines: @config.system_prompt_guidelines
+        )
+      end
+
       # Dispatch a control to the adapter, returning nil when the adapter
       # doesn't support it (e.g. external ACP agents without approvals).
       def send_adapter(conversation_id, method_name, *args)
@@ -217,8 +247,7 @@ module Ask
           approval: @config.approval,
           approval_required: @config.approval_policy_tools,
           plan_mode: @config.plan_mode,
-          todos: @config.todos,
-          system_prompt: default_system_prompt
+          todos: @config.todos
         )
         adapter.start
         # Tools operate relative to the process working directory; the
@@ -247,14 +276,6 @@ module Ask
         end
       end
 
-      def default_system_prompt
-        base = "You are the coding agent for the workspace \"#{File.basename(@config.workspace)}\" " \
-               "at #{@config.workspace}. You can read, write, and edit files, search the codebase, " \
-               "and run shell commands to inspect and modify the project. Work autonomously: " \
-               "investigate, make changes, and verify your work."
-        extra = ENV["ACH_SYSTEM_PROMPT"]
-        extra ? "#{base}\n\n#{extra}" : base
-      end
     end
   end
 end

data/lib/ask/coding_harness/config.rb

@@ -30,7 +30,8 @@ module Ask
 
       attr_accessor :host, :workspace, :db_path, :model,
                     :max_turns, :turn_timeout, :approval_required,
-                    :plan_mode, :todos, :tools, :adapter, :adapter_opts
+                    :plan_mode, :todos, :tools, :adapter, :adapter_opts,
+                    :system_prompt, :system_prompt_append, :system_prompt_guidelines
 
       def initialize
         @host = ENV.fetch("ACH_HOST", "0.0.0.0")
@@ -48,6 +49,9 @@ module Ask
         @tools = DEFAULT_TOOLS.dup
         @adapter = ENV["ACH_ADAPTER"] || ENV["CODING_PROVIDER"] || "ask_agent"
         @adapter_opts = {}
+        @system_prompt = nil
+        @system_prompt_append = ENV["ACH_SYSTEM_PROMPT"]
+        @system_prompt_guidelines = []
         validate!
       end
 

data/lib/ask/coding_harness/demo_adapter.rb

@@ -39,7 +39,7 @@ module Ask
         @mutex.synchronize { @aborted = true; @cv.broadcast }
       end
 
-      def create_session(workspace_path, mode: nil, model: nil)
+      def create_session(workspace_path, mode: nil, model: nil, system_prompt: nil, **)
         sid = "demo_#{@sessions.size + 1}"
         @sessions[sid] = workspace_path
         sid

data/lib/ask/coding_harness/server.rb

@@ -88,6 +88,44 @@ module Ask
             workspace_info.to_json
           end
 
+          # GET /api/workspaces — known workspaces (registered + with
+          # conversations), enriched with name, branch, and counts
+          r.get "workspaces" do
+            store = harness_store
+            dirs = (store.workspaces + store.projects.map { |p| p["directory"] }).uniq
+            dirs.filter_map { |dir| workspace_info(dir, store) }.to_json
+          end
+
+          # POST /api/workspaces — open a workspace by path
+          r.post "workspaces" do
+            body = JSON.parse(r.body.read)
+            path = body["path"].to_s.strip
+            if path.empty?
+              response.status = 400
+              next { error: "path is required" }.to_json
+            end
+            unless File.directory?(path)
+              response.status = 404
+              next { error: "Not a directory: #{path}" }.to_json
+            end
+            dir = harness_store.register_workspace(path)
+            workspace_info(dir, harness_store).to_json
+          end
+
+          # GET /api/workspaces/:encoded_path/info
+          r.on "workspaces", String do |encoded|
+            r.get "info" do
+              dir = URI.decode_www_form_component(encoded)
+              info = workspace_info(dir, harness_store)
+              if info
+                info.to_json
+              else
+                response.status = 404
+                { error: "Not a directory: #{dir}" }.to_json
+              end
+            end
+          end
+
           # POST /api/chat — streaming turn
           r.post "chat" do
             body = JSON.parse(r.body.read)
@@ -101,8 +139,16 @@ module Ask
             end
 
             store = harness_store
+            workspace = body["workspace"].to_s.strip
+            workspace = harness_config.workspace if workspace.empty?
+            if !File.directory?(workspace)
+              response.status = 404
+              next { error: "Not a directory: #{workspace}" }.to_json
+            end
+            store.register_workspace(workspace)
+
             existing = conversation_id && store.load(conversation_id)
-            conversation = existing || store.build(directory: harness_config.workspace)
+            conversation = existing || store.build(directory: workspace)
             new_conversation = existing.nil?
             conversation = store.save(conversation) if new_conversation
 
@@ -292,18 +338,23 @@ module Ask
         raise NotImplementedError, "built via Server.build"
       end
 
-      def workspace_info
-        cfg = harness_config
-        root = cfg.workspace
+      def workspace_info(root = nil, store = nil)
+        root ||= harness_config.workspace
+        return nil unless File.directory?(root)
         branch = nil
         head = File.join(root, ".git", "HEAD")
         if File.file?(head)
           ref = File.read(head).strip
           branch = ref.split("/").last if ref.start_with?("ref:")
         end
-        { name: File.basename(root), root: root, gitBranch: branch }
+        count = 0
+        if store
+          conv = store.list_for_directory(root)
+          count = conv.length
+        end
+        { name: File.basename(root), root: root, gitBranch: branch, conversation_count: count }
       rescue SystemCallError
-        { name: File.basename(root), root: root, gitBranch: nil }
+        nil
       end
 
       def models

data/lib/ask/coding_harness/store.rb

@@ -15,7 +15,9 @@ module Ask
     # conversation records; the server and runner only read/write through it.
     class Store
       CONVERSATIONS_KEY = "__conversations__"
+      WORKSPACES_KEY = "__workspaces__"
       MAX_CONVERSATIONS = 500
+      MAX_WORKSPACES = 100
 
       # @param db_path [String] path to the SQLite database file
       def initialize(db_path:)
@@ -86,6 +88,22 @@ module Ask
         end.map { |dir, count| { "directory" => dir, "name" => File.basename(dir), "conversation_count" => count } }
       end
 
+      # ── Workspace registry ──
+
+      # Register a workspace path (idempotent). Workspaces the server has
+      # explicitly opened, even before any conversation exists.
+      def register_workspace(path)
+        dir = File.expand_path(path)
+        existing = db.list_range(WORKSPACES_KEY, 0, -1)
+        db.list_append(WORKSPACES_KEY, dir, max_length: MAX_WORKSPACES) unless existing.include?(dir)
+        dir
+      end
+
+      # Explicitly registered workspace paths.
+      def workspaces
+        db.list_range(WORKSPACES_KEY, 0, -1)
+      end
+
       def close
         @db&.close
         @db = nil

data/lib/ask/coding_harness/system_prompt.rb

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+
+require "pathname"
+
+module Ask
+  module CodingHarness
+    # Builds the agent system prompt, in the style of the pi coding agent:
+    # a composable base (custom or default), tool-aware guidelines, project
+    # context files injected as <project_context> (AGENTS.md / CLAUDE.md
+    # walked from the workspace up to the filesystem root), an append
+    # section, and the working directory as a closing footer.
+    #
+    #   SystemPrompt.build(workspace: "/path/to/project")
+    #   SystemPrompt.build(workspace: "/p", custom: "...", append: "...",
+    #                      guidelines: ["Always run tests"])
+    class SystemPrompt
+      # Files picked up per directory when building project context.
+      CONTEXT_FILE_CANDIDATES = %w[AGENTS.md AGENTS.MD CLAUDE.md CLAUDE.MD].freeze
+
+      # Context files larger than this are skipped (token hygiene).
+      MAX_CONTEXT_FILE_SIZE = 64 * 1024
+
+      # Guidelines always present, regardless of configuration.
+      DEFAULT_GUIDELINES = [
+        "Be concise in your responses",
+        "Show file paths clearly when working with files",
+        "Use absolute paths rooted at your working directory for all file and command operations",
+        "Investigate before you act: read the relevant files before editing them",
+        "Verify your work: run tests or checks after changing code",
+        "Never claim an action you did not take"
+      ].freeze
+
+      DEFAULT_PROMPT = <<~PROMPT.chomp
+        You are an expert coding assistant operating inside ask-coding-harness, a coding agent
+        harness for the ask-rb ecosystem. You help users by reading files, executing commands,
+        editing code, and writing new files. Work autonomously: investigate the project, make
+        changes, and verify your work.
+      PROMPT
+
+      # Build the full system prompt for a workspace.
+      #
+      # @param workspace [String] the working directory (footer + context walk)
+      # @param custom [String, nil] custom base prompt (replaces the default;
+      #   project context, append, and footer are still added)
+      # @param append [String, nil] extra text appended after project context
+      # @param guidelines [Array<String>] extra guideline bullets
+      # @param context_files [Array<Hash>, :auto, nil] explicit project context
+      #   files ({path:, content:}), auto-discovered from the workspace walk,
+      #   or none
+      # @return [String]
+      def self.build(workspace:, custom: nil, append: nil, guidelines: [],
+                     context_files: :auto)
+        files =
+          case context_files
+          when :auto then load_project_context(workspace)
+          when nil then []
+          else context_files
+          end
+
+        parts = []
+        parts << (custom.nil? || custom.empty? ? DEFAULT_PROMPT : custom)
+        parts << render_guidelines(guidelines)
+        parts << render_project_context(files) unless files.empty?
+        parts << append unless append.nil? || append.empty?
+        parts << "Current working directory: #{workspace.to_s.tr("\\", "/")}"
+        parts.reject { |p| p.nil? || p.empty? }.join("\n\n")
+      end
+
+      # Walk from the workspace up to the filesystem root, collecting the
+      # nearest AGENTS.md / CLAUDE.md in each directory (nearest first,
+      # deduplicated by realpath). Mirrors the pi coding agent's project
+      # context discovery.
+      #
+      # @param workspace [String]
+      # @return [Array<Hash>] [{path:, content:}]
+      def self.load_project_context(workspace)
+        files = []
+        seen = {}
+
+        dir = File.expand_path(workspace)
+        loop do
+          candidate = context_file_in(dir)
+          if candidate && !seen.key?(candidate[:realpath])
+            seen[candidate[:realpath]] = true
+            files << candidate[:file]
+          end
+
+          parent = File.dirname(dir)
+          break if parent == dir
+          dir = parent
+        end
+
+        # Nearest first: the walk starts at the workspace.
+        files
+      end
+
+      # The first existing context file in a directory, or nil.
+      def self.context_file_in(dir)
+        CONTEXT_FILE_CANDIDATES.each do |name|
+          path = File.join(dir, name)
+          next unless File.file?(path)
+          size = File.size(path)
+          next if size > MAX_CONTEXT_FILE_SIZE
+          content = File.read(path, encoding: "UTF-8")
+          return { file: { path: path, content: content }, realpath: File.realpath(path) }
+        rescue SystemCallError, EncodingError
+          next
+        end
+        nil
+      end
+
+      def self.render_guidelines(extra)
+        list = DEFAULT_GUIDELINES.dup
+        Array(extra).each do |g|
+          normalized = g.to_s.strip
+          list << normalized unless normalized.empty? || list.include?(normalized)
+        end
+        "Guidelines:\n" + list.map { |g| "- #{g}" }.join("\n")
+      end
+
+      def self.render_project_context(files)
+        body = files.map do |f|
+          "<project_instructions path=\"#{f[:path]}\">\n#{f[:content]}\n</project_instructions>"
+        end.join("\n\n")
+        "<project_context>\n\nProject-specific instructions and guidelines:\n\n#{body}\n</project_context>"
+      end
+    end
+  end
+end

data/lib/ask/coding_harness/version.rb

@@ -2,6 +2,6 @@
 
 module Ask
   module CodingHarness
-    VERSION = "0.1.0"
+    VERSION = "0.2.0"
   end
 end

data/lib/ask-coding-harness.rb

@@ -58,5 +58,6 @@ require "ask-state-providers"
 require "ask/coding_harness/config"
 require "ask/coding_harness/store"
 require "ask/coding_harness/event_translator"
+require "ask/coding_harness/system_prompt"
 require "ask/coding_harness/agent_runner"
 require "ask/coding_harness/runner"