From 53c4e5fe9c62748d0fd30cc57657406176f67754 Mon Sep 17 00:00:00 2001
From: LukeGus <bugattiguy527@gmail.com>
Date: Wed, 21 Oct 2026 11:55:50 -0500
Subject: [PATCH] fix: let the repair screen turn the layout plugin back on

---
 src/ui/locales/en.json                  |  2 ++
 src/ui/look/RepairView.tsx              | 12 ++++++++++
 src/ui/look/layout-status.ts            |  4 +++-
 src/ui/tests/look/RepairView.test.tsx   | 30 ++++++++++++++++++++++++-
 src/ui/tests/look/layout-status.test.ts | 13 +++++++++++
 5 files changed, 59 insertions(+), 2 deletions(-)

diff --git a/src/ui/locales/en.json b/src/ui/locales/en.json
index 2d19355f9..8bbfc961b 100644
--- a/src/ui/locales/en.json
+++ b/src/ui/locales/en.json
@@ -1098,6 +1098,8 @@
       "resetEveryone": "Reset the default for everyone",
       "update": "Update {{name}} to {{version}}",
       "disable": "Turn off {{name}}",
+      "enable": "Turn on {{name}}",
+      "startAgain": "Start {{name}} again",
       "install": "Find layouts and themes",
       "installTitle": "Layouts and themes",
       "installButton": "Install",
diff --git a/src/ui/look/RepairView.tsx b/src/ui/look/RepairView.tsx
index 72cbfc70d..e4487a91a 100644
--- a/src/ui/look/RepairView.tsx
+++ b/src/ui/look/RepairView.tsx
@@ -81,6 +81,18 @@ function AdminTools({
           })}
         </Button>
       )}
+      {entry?.installed && !entry.enabled && (
+        <Button
+          size="sm"
+          onClick={() => void manager.toggle(entry).then(onRetry)}
+          disabled={manager.busy.has(entry.id)}
+        >
+          <Power />
+          {status.reason === "disabled"
+            ? t("looks.repair.enable", { name: entry.name })
+            : t("looks.repair.startAgain", { name: entry.name })}
+        </Button>
+      )}
       {entry?.installed && entry.enabled && !entry.managedByServer && (
         <Button
           size="sm"
diff --git a/src/ui/look/layout-status.ts b/src/ui/look/layout-status.ts
index a80545e03..76bc0f451 100644
--- a/src/ui/look/layout-status.ts
+++ b/src/ui/look/layout-status.ts
@@ -79,13 +79,15 @@ export function layoutStatus(input: LayoutStatusInput): LayoutStatus {
   const owner = input.owner;
   if (!owner) return repair("missing");
   const summary = owner.summary;
-  if (!summary.enabled) return repair("disabled");
+  // Before the enabled check: the server reports a plugin that failed to
+  // start as not enabled, which hid the real error behind "turned off".
   if (owner.frontend === "blocked" || summary.state === "blocked") {
     return repair("blocked", { error: owner.error ?? summary.lastError });
   }
   if (owner.frontend === "failed" || summary.state === "failed") {
     return repair("failed", { error: owner.error ?? summary.lastError });
   }
+  if (!summary.enabled) return repair("disabled");
   if (owner.frontend === "loading") return { kind: "loading" };
   // Just chosen: the runtime is about to load its code.
   if (
diff --git a/src/ui/tests/look/RepairView.test.tsx b/src/ui/tests/look/RepairView.test.tsx
index b2ef96136..65b087269 100644
--- a/src/ui/tests/look/RepairView.test.tsx
+++ b/src/ui/tests/look/RepairView.test.tsx
@@ -53,6 +53,8 @@ vi.mock("@/look/use-look-theme", () => ({
   useChooseTheme: () => look.chooseTheme,
 }));
 
+const fancy = vi.hoisted(() => ({ enabled: true }));
+
 const manager = vi.hoisted(() => ({
   requestUpdate: vi.fn(),
   requestInstall: vi.fn(),
@@ -72,7 +74,7 @@ vi.mock("@/plugins/use-plugins-manager", () => ({
         id: "fancy-layout",
         name: "Fancy",
         installed: true,
-        enabled: true,
+        enabled: fancy.enabled,
         updateAvailable: true,
         latestVersion: "1.1.0",
         managedByServer: false,
@@ -115,6 +117,7 @@ const onRetry = vi.fn();
 
 beforeEach(() => {
   permissions.isAdmin = true;
+  fancy.enabled = true;
   for (const fn of [
     ...Object.values(look),
     ...Object.values(manager),
@@ -190,6 +193,31 @@ describe("RepairView", () => {
     );
   });
 
+  it("lets an admin turn the plugin back on and then tries again", async () => {
+    fancy.enabled = false;
+    manager.toggle.mockResolvedValue(undefined);
+    render(
+      <RepairView
+        status={{ ...status, reason: "disabled", error: null }}
+        onRetry={onRetry}
+      />,
+    );
+    expect(screen.queryByText(/looks\.repair\.disable/)).toBeNull();
+    fireEvent.click(screen.getByText('looks.repair.enable:{"name":"Fancy"}'));
+    expect(manager.toggle).toHaveBeenCalledWith(
+      expect.objectContaining({ id: "fancy-layout", enabled: false }),
+    );
+    await waitFor(() => expect(onRetry).toHaveBeenCalled());
+  });
+
+  it("offers to start a failed plugin again", () => {
+    fancy.enabled = false;
+    render(<RepairView status={status} onRetry={onRetry} />);
+    expect(
+      screen.getByText('looks.repair.startAgain:{"name":"Fancy"}'),
+    ).toBeTruthy();
+  });
+
   it("lets an admin restore the bundled copy and then tries again", async () => {
     render(<RepairView status={status} onRetry={onRetry} />);
     fireEvent.click(screen.getByText('looks.repair.restore:{"name":"Fancy"}'));
diff --git a/src/ui/tests/look/layout-status.test.ts b/src/ui/tests/look/layout-status.test.ts
index bf9c75962..5516766b4 100644
--- a/src/ui/tests/look/layout-status.test.ts
+++ b/src/ui/tests/look/layout-status.test.ts
@@ -120,6 +120,19 @@ describe("layoutStatus", () => {
     ).toMatchObject({ reason: "failed", error: "server" });
   });
 
+  it("says failed, not disabled, when the server could not start it", () => {
+    expect(
+      layoutStatus(
+        input({
+          owner: owner(
+            { enabled: false, state: "failed", lastError: "bad bundle" },
+            "none",
+          ),
+        }),
+      ),
+    ).toMatchObject({ reason: "failed", error: "bad bundle" });
+  });
+
   it("says incompatible when the server skipped its plugin", () => {
     expect(
       layoutStatus(