workspaces 0.1.19 → 0.1.20

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1663319f23323ff66ace4b0744db8bd152448d78ed38d164299bf3c8e641ef8d
-  data.tar.gz: 44dea1893b4fc1299a7bb2c064b302e556b3574baebd2b7215fad8747c6b80c6
+  metadata.gz: b62d0530751f975a4e8604a691dce7bc133c3d7bd57a966e08e9b9e24c7088d9
+  data.tar.gz: 3fb3b5acff924444694d1390b9edd0f4dc2eaf5885da24b938552180b52a0b0e
 SHA512:
-  metadata.gz: c00c46b1ac312ae72bb239d74da78fe7c8b4094e0a04d6ae0ebf44232df1050c63f0d5c8a15232daef42d7b9ba526ca71cf86c70731728878db936d98a8b939d
-  data.tar.gz: d285c6a6d3f77f94da739295120d7538babc59de94c37fa538eeb0e49c6be6eaefaefe72e2524b9ad448a41032e676fa57f445d359adf15a0bdeba55046ccecc
+  metadata.gz: e860375ef8492aba63ef9340ff2ad29e22fc504353546d343b3e2c3dc31833b9df5bc77de4a8b367eeeb8e3423aef5918e0f61a51d4de26dd8ea9d916af7bec3
+  data.tar.gz: d8521f83fd632951c5ac551f52f1c4a72db8d22afd33dca483bad10e1f82456f01d8fcf4fa0d50fc00e93cb98da32eab34ae6c16454a8cb931e5654c9349ee19

data/DESIGN.md

@@ -38,7 +38,7 @@ Restrained tooling UI: a near-black page, one slightly lighter card surface with
 ## Component primitives
 
 - **Creation form**: four labelled fields in one wrapping row (`flex: 1 1 220px`, 16px gap), each with a real `<label for>`, then a `type="submit"` button on its own line.
-- **Workspace inventory**: a centered column up to 1180px. The page header carries "N managed worktrees · none running" (or "· K running"), counted from the data. The table sits in an `overflow-x: auto` wrapper with `min-width: 820px`, so a phone scrolls it sideways instead of cramming or hiding columns. Rows have 16px padding; workspace names are monospaced links with no underline until hover. Times read "Oct 27, 10:48", are UTC (the headers say so) and keep the full ISO value in a `title`. A missing start reads "Never" and a missing creation time "Unavailable", in the muted colour.
+- **Workspace inventory**: a centered column up to 1180px. Rows are ordered by last startup, newest first; workspaces that never started (or whose start time cannot be read) come last, and ties fall back to the id so the list never shuffles between loads. Moments are compared as moments, not text. The page header carries "N managed worktrees · none running" (or "· K running"), counted from the data. The table sits in an `overflow-x: auto` wrapper with `min-width: 820px`, so a phone scrolls it sideways instead of cramming or hiding columns. Rows have 16px padding; workspace names are monospaced links with no underline until hover. Times read "Oct 27, 10:48", are UTC (the headers say so) and keep the full ISO value in a `title`. A missing start reads "Never" and a missing creation time "Unavailable", in the muted colour.
 - **Status pill**: a dot and one word (13px, 600, fully rounded), never colour alone. A busy workspace adds "(busy)" and pulses its dot, which stops under reduced motion.
 - **Setup progress**: names and ordered states come from repository run steps; failures are highlighted. Restart shows the background launch and later readiness commands.
 - **Actions by state**: idle/stopped offers Start; active operations show progress without lifecycle buttons; failure offers Retry; ready offers Open workspace plus rebuild/restart/stop.
@@ -49,18 +49,18 @@ Restrained tooling UI: a near-black page, one slightly lighter card surface with
   - **Shared with the settings page**: the card reuses the page's element ids and `app.js`, so polling needs no second script. `app.js` tolerates the elements the card lacks (actions, steps, logs, environment).
   - **On a preview host** (`data-base="/__workspaces"`, `data-reload-when-ready="true"`): the card is served from the workspace's own origin, so it is the smallest surface that can work. It polls `/__workspaces/status`, posts only to `/__workspaces/start|prepare|restart|resume`, shows a fixed failure line instead of error text, and reloads the current page once when the workspace is ready. Everything else (settings, environment, secrets, logs, removal) stays on the shared dashboard origin behind its same-origin check.
   - **Settings page**: the full page described below, with a *Back to workspace* link. It is where Update from PR, keep-awake, rebuild, restart, stop, environment editing and deletion live.
-- **Three layers** (the settings page; the card shows only the lead): the workspace page leads with the friendly, the deliberate, then the technical. Each layer is shown or collapsed by lifecycle state, driven by `Presentation::STATES`:
+- **Three layers** (the settings page; the card shows only the lead): the workspace page leads with the friendly, the deliberate, then the technical. Which layers are shown depends on lifecycle state, driven by `Presentation::STATES`. They are plain sections, never folded: the card at `/workspaces/ID` is what keeps the details out of the way, so the settings page has nothing to hide.
   1. **Lead** (always visible): a plain-language headline, one sentence, and at most one primary action (a large button). The setup steps never appear here.
-  2. **Settings** (a `<details>` titled *Change settings*, with a *Preset: name* badge in its header whenever the saved settings exactly match a preset, so the choice is visible while it is closed, or *Choose what to preview* on a brand new workspace): presets first, as buttons, then the environment variables. The one submit button names what it does (Start workspace, Save & restart, Save & start, Save & retry). It starts open only for a new workspace.
-  3. **Technical details** (one `<details>`): setup step logs with timings, the web server log, workspace controls (rebuild, restart, stop, update from PR), preview URL, branch and path, and the Danger Zone. It starts collapsed, and opens by itself while setup runs or after a failure so a cause is never hidden.
+  2. **Settings** (a section headed *Change settings*, with a *Preset: name* badge beside the heading whenever the saved settings exactly match a preset, or *Choose what to preview* on a brand new workspace): presets first, as buttons, then the environment variables. The one submit button names what it does (Start workspace, Save & restart, Save & start, Save & retry).
+  3. **Technical details** (a section): setup step logs with timings, the web server log, workspace controls (rebuild, restart, stop, update from PR), preview URL, branch and path, and the Danger Zone. It is shown whenever the state has something to show (while setup runs, after a failure, and once ready), so a cause is never hidden.
 - **Lead by state**:
-  - `idle`: "Let's get your workspace running", with the settings open and *Start workspace* in the form.
-  - `busy`: "Setting things up…", with the current step as a quiet line and the elapsed time. Technical details open so progress and the log can be watched.
-  - `ready`: "Your workspace is ready" with a large **Open workspace** button. Everything else is collapsed.
+  - `idle`: "Let's get your workspace running", with the settings shown and *Start workspace* in the form.
+  - `busy`: "Setting things up…", with the current step as a quiet line and the elapsed time. Technical details are shown so progress and the log can be watched.
+  - `ready`: "Your workspace is ready" with a large **Open workspace** button. Settings and technical details sit below it.
   - `stopped`: "Your workspace is paused" with a large **Start workspace** button.
-  - `error`/`interrupted`: a plain "Something didn't go to plan" with a large **Retry failed step** button (**Continue setup** after an interruption) that names the step in the summary, a subtle **Re-run all steps** beside it, the error shown, and technical details open. When the recorded steps cannot be resumed, the single **Retry** of the earlier design is shown instead.
-  - `unknown`: a minimal view with technical details open.
-- **Disclosure behavior**: the server renders the initial open state and the polling script re-applies it from `presentation.open`, but only when the state changes. A section the user opened or closed is left alone on later polls. Sections that are not meaningful in a state are hidden, never shown disabled.
+  - `error`/`interrupted`: a plain "Something didn't go to plan" with a large **Retry failed step** button (**Continue setup** after an interruption) that names the step in the summary, a subtle **Re-run all steps** beside it, the error shown, and technical details shown. When the recorded steps cannot be resumed, the single **Retry** of the earlier design is shown instead.
+  - `unknown`: a minimal view with technical details shown.
+- **Section visibility**: the server renders which sections the state shows, and the polling script reveals or hides them as the state changes. Sections that are not meaningful in a state are hidden, never shown disabled. Nothing on the page folds except the per-step log rows below, so polling never opens or closes anything the reader is looking at.
 - **Per-step re-run controls**: inside each step's accordion, small subtle **Re-run this step** and **Re-run from here** forms aimed at `/workspaces/ID/resume?only=KEY` and `?from=KEY`. The server decides which appear (`presentation.resume.step_modes`), so the page never offers something that would be refused; they are hidden while a run is in progress and are disabled with the other controls while busy.
 - **Sleeping**: a workspace asleep is its own state (`sleeping`), distinct from stopped and failed. The lead reads "Your workspace is asleep", says files and data are kept and that opening the preview wakes it, and offers one **Wake workspace** button. A ready workspace shows one muted line, "Sleeps after N min without preview traffic", or "Kept awake" when exempt. **Keep awake** / **Allow sleep** are subtle workspace controls. A waking workspace is simply busy.
 - **Environment rows**: the settings form is a list of rows, each a name input, a value input and a **Secret** checkbox, with **Add variable** and a per-row **Remove** (both revealed by script; without it the server-rendered rows still save). Rows post as `env[N][name|value|secret]`. A secret row's value is a password field that is never pre-filled; a stored secret shows its name, the placeholder "Stored. Enter a new value to replace it", and a locked checked checkbox (a hidden field carries it), so it can only be replaced or removed, not exposed. Ticking Secret on a new row switches its value to a password field. Presets replace only the visible rows and are matched against visible rows alone. A note under the rows shows why secrets are unavailable when the vault is not usable, and a line explains that secrets are write-only.
@@ -70,7 +70,7 @@ Restrained tooling UI: a near-black page, one slightly lighter card surface with
 - **Step logs (GitHub Actions style)**: setup output is split by recipe step. Each step is a `<details>` row with a status icon (ring, pulsing ring, check or cross), its name, how long it took and a chevron; opening it shows only that step's output. A step that is running or has failed starts open and a completed or waiting one starts closed. The script updates rows in place by step id, and applies the open preference only when a step's state changes, so a row the reader toggled is never reopened or reclosed and what they are reading is never rebuilt.
   - **Capture**: `StepLog` wraps the combined log. Every line still goes to the combined `<id>.setup.log`; while a step is current the same scrubbed text also goes to `logs/<id>.steps/<step-id>.log`. A step's file is truncated when it begins, the files are cleared when a run starts, and a failing step records its error message in its own file.
   - **Read**: the snapshot carries `step_logs` (the tail of each step that has output, with a `truncated` flag) for the setup view only, and `backend_log_available`.
-  - **Tabs**: *Setup steps* (the accordions) and *Web server log*. The server tab appears only once the backend log has content. The combined log stays reachable as a final, collapsed *Full setup log* row, which also covers workspaces set up before step files existed.
+  - **Tabs**: *Setup steps* (the step rows) and *Web server log*. The server tab appears only once the backend log has content. Switching tabs reloads the page, and the links end in `#log-title` so the reader lands back on the log rather than the top of the page. The combined log stays reachable as a final, collapsed *Full setup log* row, which also covers workspaces set up before step files existed.
 - **Build progress**: while a build is being prepared the lead is the headline, a progress bar (`role="progressbar"` with a text value), and one line saying *Step n of m · name*, the percentage, the elapsed time and an estimate. The server computes the numbers from the step states and per-step averages frozen when the run began (`Progress`), and the script only draws them; the elapsed clock ticks locally between polls. A step well past its usual time turns the bar amber and the estimate reads "Taking longer than usual". Without history there is no time estimate. The bar is hidden when no build is running.
 - **Build history**: every finished run is appended to `build-history.jsonl` (`BuildHistory`). Estimates are the mean of a step's last 10 successful completions; failed runs are recorded but never counted. Recording is best-effort and cannot fail a build.
 - **Bounded log panel**: monospace text, fixed max height, manual-scroll-safe updates

data/lib/workspaces/dashboard/app.js

@@ -244,7 +244,6 @@
       element.hidden = !sections[element.dataset.section];
     });
     renderLead(view);
-    applyDisclosureState(view);
     if (window.WorkspaceEnvironment)
       window.WorkspaceEnvironment.setSubmitLabel(view.environment_submit);
   }
@@ -325,22 +324,6 @@
     toggleBusyState(busy);
   }
 
-  // A disclosure opens by itself when its state says so (a failure must never be hidden,
-  // a fresh workspace needs its settings), but we never close one the user opened, and
-  // we only apply the state's preference when the state changes, not on every poll.
-  let lastDisclosureState = root.dataset.disclosureState || null;
-  function applyDisclosureState(view) {
-    if (view.state === lastDisclosureState) return;
-    lastDisclosureState = view.state;
-    const open = view.open || {};
-    [['environment-disclosure', open.environment], ['technical-disclosure', open.technical]].forEach(
-      ([id, shouldOpen]) => {
-        const element = node(id);
-        if (element) element.open = !!shouldOpen;
-      }
-    );
-  }
-
   // A card on a preview host stands in for the page the visitor asked for; reloading brings them to it.
   let reloading = false;
   function reloadWhenReady(view) {

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

@@ -1,4 +1,4 @@
-<main class="card-page" id="workspace" data-id="<%= @id %>" data-base="<%= action_base %>" data-reload-when-ready="<%= @preview ? 'true' : 'false' %>" data-status="<%= field('status') %>" data-started="<%= field('prepare_started_at') %>" data-completed="<%= field('completed_at') %>" data-busy="<%= @presentation['busy'] %>" data-disclosure-state="<%= escape(@presentation['state']) %>">
+<main class="card-page" id="workspace" data-id="<%= @id %>" data-base="<%= action_base %>" data-reload-when-ready="<%= @preview ? 'true' : 'false' %>" data-status="<%= field('status') %>" data-started="<%= field('prepare_started_at') %>" data-completed="<%= field('completed_at') %>" data-busy="<%= @presentation['busy'] %>">
   <section class="card" aria-labelledby="status">
     <header class="card__header">
       <p class="card__name"><span class="eyebrow">Workspace</span><span class="card__id"><%= @id %></span></p>

data/lib/workspaces/dashboard/styles.css

@@ -205,48 +205,20 @@ h1 {
 .lead__action:empty { display: none; }
 .button--large { min-block-size: 52px; padding-inline: var(--space-8); font-size: 16px; border-radius: var(--radius-control); }
 
-/* Disclosures: quiet by default, a clear label and a one-line hint of what is inside. */
-/* A disclosure has to read as a control. The native marker is lost once a summary is a flex
-   container, so draw our own chevron; the whole header is the click target, with a hover
-   fill and a keyboard focus ring. */
-/* Two classes so the narrow-screen panel padding below cannot re-add padding around the summary. */
-.status-panel.disclosure { padding: 0; }
-.disclosure > summary {
+/* Settings page sections: a heading, a one-line hint of what is inside, then the content. Nothing
+   folds; the card at /workspaces/ID is what keeps these details out of the way. The section is a grid
+   (.status-panel:not(details)), so the gap below the heading comes from there. */
+.section-heading {
   display: flex;
   flex-wrap: wrap;
   align-items: baseline;
   gap: var(--space-1) var(--space-3);
-  padding: var(--space-4) calc(var(--space-6) + var(--space-8)) var(--space-4) var(--panel-padding);
-  position: relative;
-  list-style: none;
-  color: var(--text);
-  font-weight: 600;
-  border-radius: var(--radius);
-  cursor: pointer;
-  user-select: none;
-  transition: background-color 120ms ease;
+  padding-block-end: var(--space-4);
+  border-block-end: 1px solid var(--border);
 }
-.disclosure > summary::-webkit-details-marker { display: none; }
-.disclosure > summary::after {
-  content: '';
-  position: absolute;
-  inset-inline-end: var(--panel-padding);
-  inset-block-start: 50%;
-  inline-size: 8px;
-  block-size: 8px;
-  margin-block-start: -6px;
-  border-inline-end: 2px solid var(--muted);
-  border-block-end: 2px solid var(--muted);
-  transform: rotate(45deg);
-  transition: transform 160ms ease, border-color 120ms ease;
-}
-.disclosure[open] > summary::after { margin-block-start: -2px; transform: rotate(225deg); }
-.disclosure > summary:hover { background: color-mix(in oklab, var(--accent) 8%, transparent); }
-.disclosure > summary:hover::after { border-color: var(--text); }
-.disclosure > summary:focus-visible { outline: 2px solid var(--accent); outline-offset: -2px; }
-.disclosure[open] > summary { border-end-start-radius: 0; border-end-end-radius: 0; border-block-end: 1px solid var(--border); margin-block-end: 0; }
-.disclosure > :not(summary) { margin: var(--panel-padding); }
-/* Which preset is active, readable without opening the section. Same tint as a pressed preset button. */
+.section-heading h2 { margin: 0; font-size: 1.125rem; }
+.section-heading p { margin: 0; font-size: 13px; }
+/* Which preset is active, readable at a glance. Same tint as a pressed preset button. */
 .badge {
   display: inline-flex;
   align-items: center;
@@ -262,14 +234,12 @@ h1 {
   white-space: nowrap;
 }
 .badge::before { content: ''; inline-size: 6px; block-size: 6px; border-radius: 999px; background: var(--success); }
-.disclosure__hint { color: var(--muted); font-size: 13px; font-weight: 400; }
-@media (prefers-reduced-motion: reduce) { .disclosure > summary, .disclosure > summary::after { transition: none; } }
-.disclosure .technical > section:not(.danger-zone) { padding: 0; border: 0; box-shadow: none; background: none; }
-.disclosure .danger-zone { display: grid; gap: var(--space-3); padding: var(--space-4); border: 1px solid var(--danger); border-radius: var(--radius); }
-.disclosure .log-panel { padding: 0; }
+#technical-section .technical > section:not(.danger-zone) { padding: 0; border: 0; box-shadow: none; background: none; }
+#technical-section .danger-zone { display: grid; gap: var(--space-3); padding: var(--space-4); border: 1px solid var(--danger); border-radius: var(--radius); }
+#technical-section .log-panel { padding: 0; }
 .log-panel > #log { margin-block-start: 0; }
-.disclosure .danger-zone h3 { color: var(--danger); font-size: 1rem; }
-.disclosure h3 { font-size: 1rem; }
+#technical-section .danger-zone h3 { color: var(--danger); font-size: 1rem; }
+#technical-section h3 { font-size: 1rem; }
 
 .status-row {
   display: grid;

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

@@ -1,4 +1,4 @@
-<main class="dashboard" id="workspace" data-id="<%= @id %>" data-status="<%= field('status') %>" data-started="<%= field('prepare_started_at') %>" data-completed="<%= field('completed_at') %>" data-busy="<%= @presentation['busy'] %>" data-disclosure-state="<%= escape(@presentation['state']) %>">
+<main class="dashboard" id="workspace" data-id="<%= @id %>" data-status="<%= field('status') %>" data-started="<%= field('prepare_started_at') %>" data-completed="<%= field('completed_at') %>" data-busy="<%= @presentation['busy'] %>">
   <header class="dashboard__header">
     <div class="dashboard__title"><p class="eyebrow">Managed workspace</p><h1 class="mono-title"><%= @id %></h1></div>
     <div class="header-actions"><a class="button button--subtle" href="/workspaces/<%= @id %>">Back to workspace</a><a class="button button--subtle" href="/workspaces">All workspaces</a></div>
@@ -26,9 +26,9 @@
     </section>
     <p id="connection" role="status"></p>
   </section>
-  <details class="status-panel disclosure" id="environment-disclosure" data-section="environment" <%= 'open' if @presentation.dig('open', 'environment') %> <%= 'hidden' unless section?('environment') %>>
-    <summary><span id="environment-title"><%= escape(@presentation['environment_title']) %></span><span class="badge" id="environment-preset-badge" <%= 'hidden' unless @presentation['current_preset'] %>><%= escape(preset_badge_text) %></span><span class="disclosure__hint">Presets and environment variables</span></summary>
-    <div class="workspace-settings" aria-labelledby="environment-title">
+  <section class="status-panel" id="environment-section" data-section="environment" aria-labelledby="environment-title" <%= 'hidden' unless section?('environment') %>>
+    <header class="section-heading"><h2 id="environment-title"><%= escape(@presentation['environment_title']) %></h2><span class="badge" id="environment-preset-badge" <%= 'hidden' unless @presentation['current_preset'] %>><%= escape(preset_badge_text) %></span><p class="muted">Presets and environment variables</p></header>
+    <div class="workspace-settings">
       <form method="post" action="/workspaces/<%= @id %>/environment" id="environment-form" class="stack" data-environment-form data-environment-presets="<%= escape(environment_presets_json) %>">
         <fieldset class="env-fieldset" data-environment-fieldset <%= 'disabled' if @presentation['busy'] %>>
           <legend class="env-legend">Workspace settings</legend>
@@ -66,9 +66,9 @@
         <p class="muted" id="environment-status" role="status" aria-live="polite"></p>
       </form>
     </div>
-  </details>
-  <details class="status-panel disclosure" id="technical-disclosure" data-section="technical" <%= 'open' if @presentation.dig('open', 'technical') %> <%= 'hidden' unless section?('technical') %>>
-    <summary><span>Technical details</span><span class="disclosure__hint">Setup steps, logs and workspace controls</span></summary>
+  </section>
+  <section class="status-panel" id="technical-section" data-section="technical" aria-labelledby="technical-title" <%= 'hidden' unless section?('technical') %>>
+    <header class="section-heading"><h2 id="technical-title">Technical details</h2><p class="muted">Setup steps, logs and workspace controls</p></header>
     <div class="technical stack stack--loose">
       <div class="workspace-actions-group stack stack--snug" data-section="actions" aria-labelledby="actions-heading" <%= 'hidden' unless section?('actions') %>>
         <h3 id="actions-heading">Workspace controls</h3>
@@ -80,8 +80,8 @@
       <section class="log-panel" data-section="log" aria-labelledby="log-title" <%= 'hidden' unless section?('log') %>>
         <div class="log-header"><h3 id="log-title">Live <%= field('log_source') == 'backend' ? 'backend' : 'setup' %> log</h3><time id="updated"><%= field('updated_at') %></time></div>
         <nav class="log-switch" aria-label="Log source">
-          <a href="?log=setup" data-log-source="setup" <%= 'aria-current="page"' if @snapshot['log_source'] != 'backend' %>>Setup steps</a>
-          <a href="?log=backend" id="backend-tab" data-log-source="backend" <%= 'aria-current="page"' if @snapshot['log_source'] == 'backend' %> <%= 'hidden' unless backend_tab? %>>Web server log</a>
+          <a href="?log=setup#log-title" data-log-source="setup" <%= 'aria-current="page"' if @snapshot['log_source'] != 'backend' %>>Setup steps</a>
+          <a href="?log=backend#log-title" id="backend-tab" data-log-source="backend" <%= 'aria-current="page"' if @snapshot['log_source'] == 'backend' %> <%= 'hidden' unless backend_tab? %>>Web server log</a>
         </nav>
         <div id="steps" class="step-logs" data-step-logs <%= 'hidden' if @snapshot['log_source'] == 'backend' %>>
           <% @presentation['steps'].each do |step| %><%= step_accordion(step) %><% end %>
@@ -107,6 +107,6 @@
         </form>
       </section>
     </div>
-  </details>
+  </section>
   <noscript><a href="/workspaces/<%= @id %>/settings">Refresh status</a></noscript>
 </main>

data/lib/workspaces/dashboard_view.rb

@@ -135,13 +135,31 @@ module Workspaces
         <table class="workspace-table">
           <thead><tr><th scope="col">Workspace</th><th scope="col">Status</th><th scope="col" class="workspace-table__date">Created (UTC)</th><th scope="col" class="workspace-table__date">Last startup (UTC)</th></tr></thead>
           <tbody>
-        #{workspaces.map { |workspace| inventory_row(workspace) }.join}
+        #{by_last_startup(workspaces).map { |workspace| inventory_row(workspace) }.join}
           </tbody>
         </table>
         </div>
       HTML
     end
 
+    # Most recently started first; workspaces that have never started (or whose start time cannot be read)
+    # come last. Moments are compared as moments, not as text, so a different UTC offset or fraction cannot
+    # misorder rows, and ties fall back to the id so the list never shuffles between loads.
+    def by_last_startup(workspaces)
+      workspaces.sort_by do |workspace|
+        started = startup_moment(workspace['started_at'])
+        [started ? 0 : 1, started ? -started.to_r : 0, workspace['id'].to_s]
+      end
+    end
+
+    def startup_moment(value)
+      return nil if value.nil? || value.to_s.empty?
+
+      Time.iso8601(value.to_s)
+    rescue ArgumentError
+      nil
+    end
+
     # "8 managed worktrees · none running", counted from the data on the page.
     def inventory_summary(workspaces)
       total = workspaces.length

data/lib/workspaces/presentation.rb

@@ -11,12 +11,12 @@ module Workspaces
                 'resume' => 'Retry failed step', 'keep-awake' => 'Keep awake',
                 'allow-sleep' => 'Allow sleep' }.freeze
 
-    # Three layers, from friendly to technical. `lead` is the headline, one sentence and at
-    # most one primary action. `environment` (presets and variables) is the area for
-    # deliberately changing something. `technical` is one disclosure holding setup steps,
-    # logs, workspace details, lifecycle controls and deletion; it is never the first thing
-    # a user reads, but it opens by itself when something needs attention. Setup steps are
-    # not a section of their own: each is an accordion with its own log inside `log`.
+    # The sections of the settings page, from friendly to technical. `lead` is the headline, one
+    # sentence and at most one primary action. `environment` (presets and variables) is the area for
+    # deliberately changing something. `technical` holds setup steps, logs, workspace details,
+    # lifecycle controls and deletion. All of them are plain sections, always open when shown: the
+    # card at /workspaces/ID is what keeps the details out of the way, so this page has nothing to
+    # fold. Setup steps are not a section of their own: each has its own log row inside `log`.
     SECTIONS = %w(lead actions environment technical log details danger).freeze
     TECHNICAL = %w(technical actions log details danger).freeze
 
@@ -32,66 +32,57 @@ module Workspaces
     # `environment_submit` labels the single environment-form button, which saves the
     # settings and (re)starts the workspace, so a brand new workspace needs only that.
     #
-    # Per state: `headline` and `summary` are plain language; `open_environment` and
-    # `open_technical` say which disclosures start expanded. A failure always opens the
-    # technical section so the cause is visible without hunting for it.
+    # Per state: `headline` and `summary` are plain language, `sections` says which sections are
+    # shown, and `environment_submit` / `environment_title` word the settings form.
     STATES = {
       'idle' => {
         sections: %w(lead environment),
         headline: "Let's get your workspace running",
         summary: 'Pick the version of the site you want to preview, then start. Setup usually takes a few minutes.',
-        environment_submit: 'Start workspace', environment_title: 'Choose what to preview',
-        open_environment: true, open_technical: false
+        environment_submit: 'Start workspace', environment_title: 'Choose what to preview'
       },
       'busy' => {
         sections: %w(lead technical log),
         headline: 'Setting things up…',
         summary: 'This page keeps itself up to date. Open the preview once it is ready.',
-        environment_submit: 'Save & restart', environment_title: 'Change settings',
-        open_environment: false, open_technical: true
+        environment_submit: 'Save & restart', environment_title: 'Change settings'
       },
       'ready' => {
         sections: %w(lead environment) + TECHNICAL,
         headline: 'Your workspace is ready',
         summary: 'Everything is set up and running. Open it to see your changes.',
-        environment_submit: 'Save & restart', environment_title: 'Change settings',
-        open_environment: false, open_technical: false
+        environment_submit: 'Save & restart', environment_title: 'Change settings'
       },
       'sleeping' => {
         sections: %w(lead environment) + TECHNICAL,
         headline: 'Your workspace is asleep',
         summary: 'It went quiet, so its server was stopped to free up resources. Your files and data are kept. ' \
                  'Opening the preview wakes it in a few seconds.',
-        environment_submit: 'Save & start', environment_title: 'Change settings',
-        open_environment: false, open_technical: false
+        environment_submit: 'Save & start', environment_title: 'Change settings'
       },
       'stopped' => {
         sections: %w(lead environment) + TECHNICAL,
         headline: 'Your workspace is paused',
         summary: 'Start it again to pick up where you left off.',
-        environment_submit: 'Save & start', environment_title: 'Change settings',
-        open_environment: false, open_technical: false
+        environment_submit: 'Save & start', environment_title: 'Change settings'
       },
       'error' => {
         sections: %w(lead environment) + TECHNICAL,
         headline: "Something didn't go to plan",
         summary: 'Setup stopped before it finished. You can try again, or change settings first.',
-        environment_submit: 'Save & retry', environment_title: 'Change settings',
-        open_environment: false, open_technical: true
+        environment_submit: 'Save & retry', environment_title: 'Change settings'
       },
       'interrupted' => {
         sections: %w(lead environment) + TECHNICAL,
         headline: 'Setup was interrupted',
         summary: 'It stopped before it finished. Try again to pick up from the start.',
-        environment_submit: 'Save & retry', environment_title: 'Change settings',
-        open_environment: false, open_technical: true
+        environment_submit: 'Save & retry', environment_title: 'Change settings'
       },
       'unknown' => {
         sections: %w(lead technical log details),
         headline: 'Workspace status unknown',
         summary: 'Open the technical details to see what the workspace is doing.',
-        environment_submit: 'Save & restart', environment_title: 'Change settings',
-        open_environment: false, open_technical: true
+        environment_submit: 'Save & restart', environment_title: 'Change settings'
       }
     }.freeze
 
@@ -107,8 +98,7 @@ module Workspaces
         'lead_action' => lead_action, 'total' => total, 'progress' => progress,
         'current_preset' => current_preset, 'sleep' => sleep_info,
         'environment_title' => definition[:environment_title],
-        'environment_submit' => definition[:environment_submit],
-        'open' => { 'environment' => definition[:open_environment], 'technical' => definition[:open_technical] } }
+        'environment_submit' => definition[:environment_submit] }
     end
 
     private

data/lib/workspaces/version.rb

@@ -1,3 +1,3 @@
 module Workspaces
-  VERSION = '0.1.19'
+  VERSION = '0.1.20'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workspaces
 version: !ruby/object:Gem::Version
-  version: 0.1.19
+  version: 0.1.20
 platform: ruby
 authors:
 - Workspaces contributors
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-10-28 00:00:00.000000000 Z
+date: 2026-10-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack