From 7687ffbd2be5fb083d5eb234e91a3ee3cf26c860 Mon Sep 17 00:00:00 2001
From: "Maksym Hryzodub [DREAM]" <maksym.hryzodub@dreamvention.com>
Date: Fri, 18 Sep 2026 20:57:03 +0300
Subject: [PATCH] fix(admin): drop the duplicate restart banner from the Peers
 tab (CLEAN-98)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The agent page already raises a restart banner above every tab, and the peer
store is what raises it — connect, import, refresh and remove all call
markPendingRestart. The copy inside the Peers tab added a second button for
the same job, wired to a different code path, and it did not clear the page
banner when pressed, so the tab kept telling the operator to restart an agent
that had just restarted.

What the page banner cannot know stays where it was: the armed badge next to
"Peer network" still reports whether the RUNNING pod holds this peer set.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SZ9dCrzZqd5SwR4erLYe12
---
 .../slices/agent/peer/components/peer/Tab.vue | 45 +++----------------
 1 file changed, 6 insertions(+), 39 deletions(-)

diff --git a/admin/slices/agent/peer/components/peer/Tab.vue b/admin/slices/agent/peer/components/peer/Tab.vue
index 5aaebfd8..19b29021 100644
--- a/admin/slices/agent/peer/components/peer/Tab.vue
+++ b/admin/slices/agent/peer/components/peer/Tab.vue
@@ -1,7 +1,6 @@
 <script setup lang="ts">
 import { IconPlus } from '@tabler/icons-vue';
 import { toast } from 'vue-sonner';
-import { useAgentStore } from '#agent/stores/agent';
 import type { IAgentData } from '#agent/domain';
 import { usePeerStore, type IAgentPeer } from '#peer/stores/peer';
 import { cardAddress } from '#peer/domain';
@@ -18,7 +17,6 @@ import { cardAddress } from '#peer/domain';
 const props = defineProps<{ agent: IAgentData }>();
 
 const store = usePeerStore();
-const agentStore = useAgentStore();
 
 const adding = ref(false);
 const busyPeerId = ref<string | null>(null);
@@ -31,15 +29,11 @@ const peerFilter = ref<{ id: string; name: string } | null>(null);
 const peers = computed(() => store.peers(props.agent.id));
 const ownCard = computed(() => store.card(props.agent.id));
 const delegations = computed(() => store.delegations(props.agent.id));
-const pendingRestart = computed(() =>
-  agentStore.isPendingRestart(props.agent.id),
-);
 const peersState = computed(() => store.peersState(props.agent.id));
 /** null while unknown — the chip only renders on a definite answer. */
 const armed = computed(() =>
   peers.value.length && peersState.value ? peersState.value.armed : null,
 );
-const restarting = ref(false);
 
 const address = computed(() => cardAddress(ownCard.value));
 const nothingAdvertised = computed(
@@ -118,19 +112,6 @@ watch(
   },
 );
 
-async function restartNow() {
-  restarting.value = true;
-  try {
-    await agentStore.restart(props.agent.id);
-  } catch (err) {
-    toast.error(
-      err instanceof Error ? err.message : 'Could not restart the agent',
-    );
-  } finally {
-    restarting.value = false;
-  }
-}
-
 let copiedTimer: ReturnType<typeof setTimeout> | undefined;
 onUnmounted(() => clearTimeout(copiedTimer));
 
@@ -338,26 +319,12 @@ function goToKnowledge() {
       </div>
     </div>
 
-    <!-- A pod reads its tool list once, at boot. Without this banner an
-         operator connects a peer, asks a question, gets nothing, and has
-         no way to know why. The button is the one click that arms it. -->
-    <div
-      v-if="pendingRestart || armed === false"
-      class="flex flex-wrap items-center justify-between gap-3 rounded-lg border border-amber-500/30 bg-amber-500/10 px-3 py-2"
-    >
-      <p class="text-sm text-amber-600 dark:text-amber-500">
-        Restart the agent to apply: it reads the list of peers when it starts.
-      </p>
-      <Button
-        size="sm"
-        class="h-7 flex-none rounded-full"
-        :disabled="restarting"
-        @click="restartNow"
-      >
-        {{ restarting ? 'Restarting…' : 'Restart now' }}
-      </Button>
-    </div>
-
+    <!-- A pod reads its tool list once, at boot, so a peer change needs a
+         restart. That banner lives on the agent page above every tab and is
+         raised by the peer store itself (markPendingRestart) — a second copy
+         here only offered a second button to press (CLEAN-98). What the page
+         banner cannot know stays: the armed badge on Peer network, which
+         reports what the RUNNING pod holds. -->
     <Separator />
 
     <div class="grid items-start gap-8 xl:grid-cols-2">