fix: Skip buildable images in pull command (#109)

* fix: Skip buildable images in pull command

Add --ignore-buildable flag to pull command, matching the behavior
of the update command. This prevents pull from failing when a stack
contains services with local build directives (no remote image).

* test: Fix flaky command palette close detection

Use state="hidden" instead of :not([open]) selector when waiting
for the command palette to close. The old approach failed because
wait_for_selector defaults to waiting for visibility, but a closed
<dialog> element is hidden by design.

.github/workflows/docs.yml

@@ -27,7 +27,7 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v6
         with:
           lfs: true
 
@@ -49,7 +49,7 @@ jobs:
 
       - name: Upload artifact
         if: github.event_name != 'pull_request'
-        uses: actions/upload-pages-artifact@v3
+        uses: actions/upload-pages-artifact@v4
         with:
           path: "./site"
 

src/compose_farm/cli/lifecycle.py

@@ -140,7 +140,7 @@ def pull(
     if service and len(stack_list) != 1:
         print_error("--service requires exactly one stack")
         raise typer.Exit(1)
-    cmd = f"pull {service}" if service else "pull"
+    cmd = f"pull --ignore-buildable {service}" if service else "pull --ignore-buildable"
     raw = len(stack_list) == 1
     results = run_async(run_on_stacks(cfg, stack_list, cmd, raw=raw))
     report_results(results)

src/compose_farm/web/templates/partials/containers.html

@@ -1,24 +1,26 @@
 {# Container list for a stack on a single host #}
 {% from "partials/icons.html" import terminal, rotate_ccw, scroll_text, square, play, cloud_download %}
 {% macro container_row(stack, container, host) %}
-<div class="flex items-center gap-2 mb-2">
-    {% if container.State == "running" %}
-        <span class="badge badge-success">running</span>
-    {% elif container.State == "unknown" %}
-        <span class="badge badge-ghost"><span class="loading loading-spinner loading-xs"></span></span>
-    {% elif container.State == "exited" %}
-        {% if container.ExitCode == 0 %}
-            <span class="badge badge-neutral">exited (0)</span>
+<div class="flex flex-col sm:flex-row sm:items-center gap-2 mb-3 p-2 bg-base-200 rounded-lg">
+    <div class="flex items-center gap-2 min-w-0">
+        {% if container.State == "running" %}
+            <span class="badge badge-success">running</span>
+        {% elif container.State == "unknown" %}
+            <span class="badge badge-ghost"><span class="loading loading-spinner loading-xs"></span></span>
+        {% elif container.State == "exited" %}
+            {% if container.ExitCode == 0 %}
+                <span class="badge badge-neutral">exited (0)</span>
+            {% else %}
+                <span class="badge badge-error">exited ({{ container.ExitCode }})</span>
+            {% endif %}
+        {% elif container.State == "created" %}
+            <span class="badge badge-neutral">created</span>
         {% else %}
-            <span class="badge badge-error">exited ({{ container.ExitCode }})</span>
+            <span class="badge badge-warning">{{ container.State }}</span>
         {% endif %}
-    {% elif container.State == "created" %}
-        <span class="badge badge-neutral">created</span>
-    {% else %}
-        <span class="badge badge-warning">{{ container.State }}</span>
-    {% endif %}
-    <code class="text-sm flex-1">{{ container.Name }}</code>
-    <div class="join">
+        <code class="text-sm truncate">{{ container.Name }}</code>
+    </div>
+    <div class="join sm:ml-auto shrink-0">
         <div class="tooltip tooltip-top" data-tip="View logs">
             <button class="btn btn-sm btn-outline join-item"
                     hx-post="/api/stack/{{ stack }}/service/{{ container.Service }}/logs"

tests/web/test_htmx_browser.py

@@ -669,8 +669,8 @@ class TestCommandPalette:
         page.locator("#cmd-input").fill("plex")
         page.keyboard.press("Enter")
 
-        # Palette should close
-        page.wait_for_selector("#cmd-palette:not([open])", timeout=SHORT_TIMEOUT)
+        # Palette should close (use state="hidden" since closed dialog is not visible)
+        page.wait_for_selector("#cmd-palette", state="hidden", timeout=SHORT_TIMEOUT)
 
         # Should navigate to plex stack page
         page.wait_for_url("**/stack/plex", timeout=TIMEOUT)
@@ -699,8 +699,8 @@ class TestCommandPalette:
 
         page.keyboard.press("Escape")
 
-        # Palette should close, URL unchanged
-        page.wait_for_selector("#cmd-palette:not([open])", timeout=SHORT_TIMEOUT)
+        # Palette should close, URL unchanged (use state="hidden" since closed dialog is not visible)
+        page.wait_for_selector("#cmd-palette", state="hidden", timeout=SHORT_TIMEOUT)
         assert page.url.rstrip("/") == server_url.rstrip("/")
 
     def test_fab_button_opens_palette(self, page: Page, server_url: str) -> None: