workspaces 0.1.3 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 501f96e0f82b4dc9f30ec6682feb64329301b3ee888561e84e65941cff084ae7
-  data.tar.gz: 4ae2d5e24099d7638b349a332daaadf89813a96e37996c4f03c41c7baf2f7636
+  metadata.gz: 9e8987b36ff8ba5cfd6c039c911012462661c5da01989e84843188e471c58171
+  data.tar.gz: 1bae9743875990e29ada15d6b26885d0881eaf84dbf8626bfef1bea779753833
 SHA512:
-  metadata.gz: 67865a47386639bfcff1218218bcf8f055c0b563d295d355d9667e1d8410da7a4981def707bc0f219f4179215b4839a689ec36bfa8bba90738ea92d7771b8ad7
-  data.tar.gz: 4be62e33d0f29d0c19751b61bd3b9d0dab8cf334314ce62bc5bcfe5f59569235c960b10647cf5bb7060705068db6c1489052b59a97b543c662750f66df3e6f4d
+  metadata.gz: 173beba79ca2237d5632e06f7abb7edbc0a46f3a17d283bfdead8e59469bb9383d1de2af97f90c8c9c51e24d2b014aa6a21c63dc83f80ac257c488121f7ae909
+  data.tar.gz: b98959ea514d65b64d454f928330b86959e417bbf1be83466790ea072cd350757b9daa5ff5c7724877684bc834618eeb6a6d6b20788879dcceba48ca8be32712

data/README.md

@@ -39,6 +39,17 @@ The returned JSON includes the workspace ID, dashboard URL, and preview URL. You
 create from a pull request with `create --pr NUMBER`, or create a new branch with
 `create --new-branch NAME --from REF`. Run `help` for lifecycle and other commands.
 
+### Step timings
+
+Per-step durations are persisted in seconds in workspace step status JSON, so they
+remain available after the command exits. Timings also appear in CLI progress,
+the workspace setup log, and the dashboard's step list.
+
+A background step measures only process launch, not the server's lifetime or
+readiness. The following readiness step measures its own wait separately.
+Interrupted steps have no recorded duration. Step timings are not the total
+workflow wall-clock time; orchestration overhead can fall outside the steps.
+
 ## Serve the dashboard and previews
 
 Run `serve` in a separate terminal using the **same project root and `WORKSPACES_HOME`**

data/lib/workspaces/backend.rb

@@ -4,6 +4,7 @@ require 'time'
 require_relative 'config'
 require_relative 'state_store'
 require_relative 'port_allocator'
+require_relative 'child_env'
 
 module Workspaces
   class Backend
@@ -61,8 +62,10 @@ module Workspaces
 
     def spawn_command(workspace, command, env)
       File.open(Config.backend_log_path(workspace_id), 'a') do |file|
-        Process.spawn(env, [command.first, command.first], *command.drop(1),
-                      chdir: workspace.path.to_s, in: File::NULL, out: file, err: file, pgroup: true)
+        child_env = ChildEnv.for_workspace(workspace_path: workspace.path, extra_env: env)
+        Process.spawn(child_env, [command.first, command.first], *command.drop(1),
+                      chdir: workspace.path.to_s, in: File::NULL, out: file, err: file, pgroup: true,
+                      unsetenv_others: true)
       end
     end
 

data/lib/workspaces/child_env.rb

@@ -0,0 +1,38 @@
+module Workspaces
+  module ChildEnv
+    module_function
+
+    def for_workspace(workspace_path:, extra_env: {}, parent_env: ENV.to_h)
+      env = parent_env.to_h.transform_keys(&:to_s).dup
+      if defined?(::Bundler)
+        clean = Bundler.unbundled_env
+        %w(PATH GEM_HOME GEM_PATH MANPATH RUBYOPT RUBYLIB RB_USER_INSTALL).each do |name|
+          clean.key?(name) ? env[name] = clean[name] : env.delete(name)
+        end
+      end
+      env.delete_if { |key, _| bundler_key?(key) }
+
+      gemfile = File.join(workspace_path.to_s, 'Gemfile')
+      env['BUNDLE_GEMFILE'] = gemfile
+      env['BUNDLE_LOCKFILE'] = File.join(workspace_path.to_s, 'Gemfile.lock')
+
+      extra_env.to_h.each do |name, value|
+        key = name.to_s
+        next if bundler_key?(key)
+
+        if value.nil?
+          env.delete(key)
+        else
+          env[key] = value
+        end
+      end
+
+      env
+    end
+
+    def bundler_key?(key)
+      %w(BUNDLE_GEMFILE BUNDLE_LOCKFILE BUNDLER_SETUP BUNDLER_VERSION).include?(key) ||
+        key.start_with?('BUNDLER_ORIG_')
+    end
+  end
+end

data/lib/workspaces/cli.rb

@@ -6,6 +6,7 @@ require_relative 'registry'
 require_relative 'coordinator'
 require_relative 'front_door'
 require_relative 'tls'
+require_relative 'child_env'
 
 module Workspaces
   class Cli
@@ -149,7 +150,9 @@ module Workspaces
       end
 
       registry.with_lock(id) do |workspace|
-        pid = Process.spawn([argv.first, argv.first], *argv.drop(1), chdir: workspace.path.to_s)
+        child_env = ChildEnv.for_workspace(workspace_path: workspace.path)
+        pid = Process.spawn(child_env, [argv.first, argv.first], *argv.drop(1),
+                            chdir: workspace.path.to_s, unsetenv_others: true)
         _, status = Process.wait2(pid)
         status.exitstatus || (128 + status.termsig)
       end

data/lib/workspaces/command_runner.rb

@@ -1,5 +1,6 @@
 require 'open3'
 require 'timeout'
+require_relative 'child_env'
 
 module Workspaces
   class CommandRunner
@@ -12,7 +13,9 @@ module Workspaces
 
     def run!(command, env:, timeout:)
       executable = [command.first, command.first]
-      Open3.popen2e(env, executable, *command.drop(1), chdir: @path.to_s, pgroup: true) do |input, output, child|
+      child_env = ChildEnv.for_workspace(workspace_path: @path, extra_env: env)
+      Open3.popen2e(child_env, executable, *command.drop(1), chdir: @path.to_s, pgroup: true,
+                    unsetenv_others: true) do |input, output, child|
         input.close
         output.set_encoding(Encoding::UTF_8, invalid: :replace, undef: :replace)
         run_child(output, child, timeout)

data/lib/workspaces/coordinator.rb

@@ -134,7 +134,7 @@ module Workspaces
     def publish_starting(id, restart)
       recipe = Recipe.new
       steps = recipe.for_operation(restart: restart).map do |step|
-        { 'key' => step['id'], 'label' => step['name'], 'state' => 'pending' }
+        { 'key' => step['id'], 'label' => step['name'], 'state' => 'pending', 'duration_seconds' => nil }
       end
       update(id, 'status' => 'preparing', 'message' => restart ? 'Restarting application' : 'Preparing workspace',
                  'steps' => steps,

data/lib/workspaces/dashboard/app.js

@@ -54,11 +54,25 @@
     label.textContent = step.label;
     const outcome = document.createElement('span');
     outcome.className = 'step-outcome';
-    outcome.textContent = step.state;
+    outcome.textContent = stepOutcome(step);
     item.append(bullet, label, outcome);
     return item;
   }
 
+  function stepOutcome(step) {
+    const state = step.state;
+    const rawDuration = step.duration_seconds;
+    if (typeof rawDuration !== 'number') return state;
+    const duration = rawDuration;
+    if (
+      (state === 'complete' || state === 'failed') &&
+      Number.isFinite(duration) &&
+      duration >= 0
+    )
+      return state + ' · ' + duration.toFixed(3) + 's';
+    return state;
+  }
+
   function updateLog(data) {
     if (data.log_source !== selectedLog) return;
     const log = node('log');

data/lib/workspaces/dashboard/template.html.erb

@@ -11,7 +11,7 @@
     <p id="message"><%= field('message') %></p>
     <ol class="stepper" id="progress" aria-label="Setup progress">
       <% @presentation['steps'].each do |step| %>
-        <li class="step" data-step-state="<%= step['state'] %>" <%= 'aria-current="step"' if step['state'] == 'active' %>><span class="step-bullet" aria-hidden="true"></span><span><%= escape(step['label']) %></span><span class="step-outcome"><%= step['state'] %></span></li>
+        <li class="step" data-step-state="<%= step['state'] %>" <%= 'aria-current="step"' if step['state'] == 'active' %>><span class="step-bullet" aria-hidden="true"></span><span><%= escape(step['label']) %></span><span class="step-outcome"><%= escape(step_outcome(step)) %></span></li>
       <% end %>
     </ol>
     <section class="error-panel setup-error" id="error-panel" role="alert" <%= 'hidden' unless @presentation['error'] %>>

data/lib/workspaces/dashboard_view.rb

@@ -128,5 +128,15 @@ module Workspaces
       "<form method=\"post\" action=\"/workspaces/#{@id}/#{action.fetch('key')}\">" \
         "<button class=\"button #{style}\">#{escape(action.fetch('label'))}</button></form>"
     end
+
+    def step_outcome(step)
+      state = step['state'].to_s
+      return state unless %w(complete failed).include?(state)
+
+      duration = step['duration_seconds']
+      return state unless duration.is_a?(Numeric) && duration.finite? && duration >= 0
+
+      "#{state} · #{format('%.3f', duration)}s"
+    end
   end
 end

data/lib/workspaces/version.rb

@@ -1,3 +1,3 @@
 module Workspaces
-  VERSION = '0.1.3'
+  VERSION = '0.1.5'
 end

data/lib/workspaces/workflow.rb

@@ -33,7 +33,7 @@ module Workspaces
 
     def publish_steps(steps)
       @state.set(@workspace.id, 'steps' => steps.map do |step|
-        { 'key' => step['id'], 'label' => step['name'], 'state' => 'pending' }
+        { 'key' => step['id'], 'label' => step['name'], 'state' => 'pending', 'duration_seconds' => nil }
       end)
     end
 
@@ -48,6 +48,7 @@ module Workspaces
     end
 
     def execute(step, recipe, context, runner, overrides)
+      started_at = Process.clock_gettime(Process::CLOCK_MONOTONIC)
       progress(step, 'active')
       @log.puts "==> #{step.fetch('name')}"
       command = recipe.command(step, context)
@@ -59,16 +60,24 @@ module Workspaces
         @backend.ensure_alive! if @server_started
       end
       @server_started = true if step['background']
-      progress(step, 'complete')
     rescue StandardError
-      progress(step, 'failed')
+      finish_step(step, 'failed', started_at)
       raise
+    else
+      finish_step(step, 'complete', started_at)
     end
 
-    def progress(step, status)
+    def finish_step(step, status, started_at)
+      duration = (Process.clock_gettime(Process::CLOCK_MONOTONIC) - started_at).round(3)
+      progress(step, status, duration_seconds: duration)
+      @log.puts format('==> %s %s (%.3fs)', step.fetch('name'), status, duration)
+    end
+
+    def progress(step, status, duration_seconds: nil)
       @state.transaction do |data|
         entry = data.fetch(@workspace.id)
-        entry['steps'].find { |item| item['key'] == step['id'] }['state'] = status
+        entry['steps'].find { |item| item['key'] == step['id'] }
+          .merge!('state' => status, 'duration_seconds' => duration_seconds)
         entry['message'] = step['name']
         entry['current_step'] = step['id']
         entry['updated_at'] = Time.now.utc.iso8601

data/lib/workspaces.rb

@@ -1,5 +1,6 @@
 require_relative 'workspaces/config'
 require_relative 'workspaces/state_store'
 require_relative 'workspaces/port_allocator'
+require_relative 'workspaces/child_env'
 require_relative 'workspaces/backend'
 require_relative 'workspaces/cli'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workspaces
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors:
 - Workspaces contributors
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-01 00:00:00.000000000 Z
+date: 2026-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack
@@ -71,6 +71,7 @@ files:
 - exe/workspaces
 - lib/workspaces.rb
 - lib/workspaces/backend.rb
+- lib/workspaces/child_env.rb
 - lib/workspaces/cli.rb
 - lib/workspaces/command_runner.rb
 - lib/workspaces/config.rb