workspaces 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 501f96e0f82b4dc9f30ec6682feb64329301b3ee888561e84e65941cff084ae7
-  data.tar.gz: 4ae2d5e24099d7638b349a332daaadf89813a96e37996c4f03c41c7baf2f7636
+  metadata.gz: f3771d28dd38a7503c4ca8189e4ce6419d8ad5989c8ef8618fe5420b3572909a
+  data.tar.gz: 6dda806083d5074afeb3428d64eb9b05d67b0a1b84b5c485156d7d087606db5f
 SHA512:
-  metadata.gz: 67865a47386639bfcff1218218bcf8f055c0b563d295d355d9667e1d8410da7a4981def707bc0f219f4179215b4839a689ec36bfa8bba90738ea92d7771b8ad7
-  data.tar.gz: 4be62e33d0f29d0c19751b61bd3b9d0dab8cf334314ce62bc5bcfe5f59569235c960b10647cf5bb7060705068db6c1489052b59a97b543c662750f66df3e6f4d
+  metadata.gz: 9bfe8edb9cd5d06f9f10adf028da25cec903b5f3429971ef5521a99dd74142454985da667977d113c50129de27e5086cd040e8a6afaf111fbcfce406cb755634
+  data.tar.gz: a278582fd63bfae4d3f0f27afd68d36418854b6513e0fe50bcd96e30c73cacc9d8b3b008770d5c5b32dfd1b6abadb8ace1e60034b12909790b38fa169001adeb

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/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.4'
 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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: workspaces
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Workspaces contributors