feat: ui accept tgz / tar.gz

Author: Dunky13

ui/app.py

@@ -127,16 +127,25 @@ def _shell(body: str) -> bytes:
 button.secondary {{ background: white; color: var(--ink); border-color: var(--line); }}
 .actions {{ display: flex; flex-wrap: wrap; gap: 10px; align-items: end; }}
 .actions > * {{ width: auto; }}
-table {{ width: 100%; border-collapse: collapse; font-size: 14px; }}
-th, td {{ border-bottom: 1px solid var(--line); padding: 9px; text-align: left; vertical-align: top; }}
-code {{ overflow-wrap: anywhere; }}
+.device-grid {{ display: grid; grid-template-columns: repeat(auto-fit, minmax(min(100%, 360px), 1fr)); gap: 12px; }}
+.device-card {{ border: 1px solid var(--line); border-radius: 8px; padding: 12px; min-width: 0; }}
+.device-card-head {{ display: flex; gap: 10px; align-items: center; justify-content: space-between; min-width: 0; }}
+.pick-row {{ display: flex; grid-template-columns: none; flex-direction: row; align-items: center; gap: 8px; min-width: 0; color: var(--ink); }}
+.pick-row input {{ width: auto; min-height: auto; }}
+.pick-row strong {{ overflow-wrap: anywhere; }}
+.device-meta {{ display: grid; grid-template-columns: repeat(2, minmax(0, 1fr)); gap: 10px; margin: 12px 0 0; }}
+.device-meta div {{ min-width: 0; }}
+.device-meta .wide {{ grid-column: 1 / -1; }}
+.device-meta dt {{ color: var(--muted); font-size: 12px; margin-bottom: 3px; }}
+.device-meta dd {{ margin: 0; min-width: 0; }}
+code {{ display: inline-block; max-width: 100%; overflow-wrap: anywhere; word-break: break-word; }}
 .muted {{ color: var(--muted); }}
 .badge {{ border: 1px solid var(--line); border-radius: 999px; padding: 2px 8px; color: var(--ok); }}
 .notice {{ border-left: 4px solid var(--warn); padding: 10px 12px; background: #fff7ed; }}
 pre {{ max-height: 360px; overflow: auto; padding: 12px; background: #111827; color: #d1fae5; border-radius: 8px; }}
 iframe {{ width: 100%; min-height: 420px; border: 1px solid var(--line); border-radius: 8px; background: white; }}
 @media (max-width: 720px) {{
-  table {{ display: block; overflow-x: auto; }}
+  .device-meta {{ grid-template-columns: 1fr; }}
   .actions > * {{ width: 100%; }}
 }}
 </style>
@@ -197,7 +206,7 @@ def render_home(query: dict[str, list[str]], notice: str = "") -> bytes:
   <h2>Release Bundle</h2>
   <p class="muted">{html.escape(bundle_text)}</p>
   <form method="post" action="/import?workspace={html.escape(workspace_id)}" enctype="multipart/form-data">
-    <label>Tarball<input type="file" name="tarball" accept=".tar,.tar.gz,.tgz"></label>
+    <label>Tarball<input type="file" name="tarball" accept=".tar,.tar.gz,.tgz,application/gzip,application/x-tar"></label>
     <div class="actions"><button {"disabled" if not workspace else ""}>Import tarball</button></div>
   </form>
 </section>
@@ -233,10 +242,7 @@ def render_home(query: dict[str, list[str]], notice: str = "") -> bytes:
 <section>
   <h2>Generated Devices</h2>
   <form method="post" action="/flash?workspace={html.escape(workspace_id)}">
-    <table>
-      <thead><tr><th></th><th>Serial</th><th>VID / PID</th><th>Manual code</th><th>QR payload</th><th>Status</th></tr></thead>
-      <tbody>{render_device_rows(devices, selected_serial)}</tbody>
-    </table>
+    <div class="device-grid">{render_device_rows(devices, selected_serial)}</div>
     <div class="grid">
       <label>Serial port<select name="port">{render_port_options(ports, selected_port)}</select></label>
       <label>Options<span><input type="checkbox" name="erase" value="1"> Erase first</span><span><input type="checkbox" name="monitor" value="1"> Monitor after flash</span></label>

ui/components/device_table.py

@@ -7,19 +7,22 @@
 
 def render_device_rows(devices: list[DeviceRow], selected_serial: str | None) -> str:
     if not devices:
-        return '<tr><td colspan="6" class="muted">No generated devices yet.</td></tr>'
+        return '<p class="muted">No generated devices yet.</p>'
     rows = []
     for device in devices:
         checked = " checked" if device.serial_num == selected_serial else ""
         rows.append(
-            "<tr>"
-            f'<td><input type="radio" name="serial_num" value="{escape(device.serial_num)}"{checked}></td>'
-            f"<td>{escape(device.serial_num)}</td>"
-            f"<td>{escape(device.vendor_id)} / {escape(device.product_id)}</td>"
-            f"<td><code>{escape(device.manualcode)}</code></td>"
-            f"<td><code>{escape(device.qrcode)}</code></td>"
-            f'<td><span class="badge">{escape(device.flash_status)}</span></td>'
-            "</tr>"
+            '<article class="device-card">'
+            '<div class="device-card-head">'
+            f'<label class="pick-row"><input type="radio" name="serial_num" value="{escape(device.serial_num)}"{checked}> '
+            f'<strong>{escape(device.serial_num)}</strong></label>'
+            f'<span class="badge">{escape(device.flash_status)}</span>'
+            "</div>"
+            '<dl class="device-meta">'
+            f"<div><dt>VID / PID</dt><dd>{escape(device.vendor_id)} / {escape(device.product_id)}</dd></div>"
+            f"<div><dt>Manual code</dt><dd><code>{escape(device.manualcode)}</code></dd></div>"
+            f'<div class="wide"><dt>QR payload</dt><dd><code>{escape(device.qrcode)}</code></dd></div>'
+            "</dl>"
+            "</article>"
         )
     return "\n".join(rows)
-

ui/services/release_bundle.py

@@ -18,6 +18,11 @@ class ReleaseBundle:
     provisioning_supported: bool
 
 
+def is_supported_tarball(path: Path) -> bool:
+    name = path.name.lower()
+    return name.endswith((".tar", ".tar.gz", ".tgz"))
+
+
 def _sha256(path: Path) -> str:
     digest = hashlib.sha256()
     with path.open("rb") as file:
@@ -47,7 +52,7 @@ def _find_first(root: Path, filename: str) -> Path | None:
 
 def import_release_tarball(workspace: Workspace, tarball_path: str | Path) -> ReleaseBundle:
     source = Path(tarball_path).expanduser().resolve()
-    if source.suffix not in {".tar", ".gz", ".tgz"} and not source.name.endswith(".tar.gz"):
+    if not is_supported_tarball(source):
         raise ValueError("Release bundle must be .tar, .tar.gz, or .tgz")
     if not source.is_file():
         raise FileNotFoundError(f"Release tarball not found: {source}")

ui/tests/test_release_bundle.py

@@ -8,7 +8,7 @@
 from unittest import mock
 
 from ui.services import workspace_store
-from ui.services.release_bundle import import_release_tarball
+from ui.services.release_bundle import import_release_tarball, is_supported_tarball
 
 
 def write_tar(path: Path, members: dict[str, bytes]) -> None:
@@ -20,6 +20,12 @@ def write_tar(path: Path, members: dict[str, bytes]) -> None:
 
 
 class ReleaseBundleTests(unittest.TestCase):
+    def test_tarball_extension_filter_accepts_release_formats(self) -> None:
+        self.assertTrue(is_supported_tarball(Path("release.tar")))
+        self.assertTrue(is_supported_tarball(Path("release.tar.gz")))
+        self.assertTrue(is_supported_tarball(Path("release.tgz")))
+        self.assertFalse(is_supported_tarball(Path("release.gz")))
+
     def test_rejects_path_traversal_member(self) -> None:
         with tempfile.TemporaryDirectory() as temp_dir:
             root = Path(temp_dir)
@@ -55,4 +61,3 @@ def test_detects_flasher_args_build_dir(self) -> None:
 
 if __name__ == "__main__":
     unittest.main()
-