matt/meshai
1
0
Fork 0
mirror of https://github.com/zvx-echo6/meshai.git synced 2026-08-26 17:31:34 +00:00
Code Issues Projects Releases Packages Wiki Activity

fix(gui): restart banner says 'Restart required' (process, not container recreate)

Browse source 
The banner said 'Container restart required' and a code comment claimed 'the
container will tear down' — both wrong. /api/restart touches /tmp/meshai_restart,
which the entrypoint watches to restart the BOT PROCESS in place (~seconds); the
container is not recreated. Fix the copy + point users at the existing 'Restart
now' button. Also mention transport connections in the restart-required list.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Matt Johnson 2026-07-06 05:13:06 +00:00
commit bc5d7bbf09
1 changed files with 5 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

9
work/dashboard-frontend/src/components/RestartBanner.tsx
View file

@ -87,8 +87,9 @@ export default function RestartBanner() {
const body = await res.json().catch(() => ({})) const body = await res.json().catch(() => ({}))
throw new Error(body.detail || `HTTP ${res.status}`) throw new Error(body.detail || `HTTP ${res.status}`)
} }
// Clear the banner immediately; the container will tear down and the // Clear the banner immediately; the bot process restarts in place
// dashboard will need a refresh anyway. // (a few seconds) and the dashboard briefly reconnects. The container
// itself is NOT recreated.
clearRestartRequired() clearRestartRequired()
} catch (e) { } catch (e) {
setError(String(e)) setError(String(e))
@ -106,7 +107,7 @@ export default function RestartBanner() {
<div className="bg-yellow-900/40 border-b border-yellow-700 text-yellow-100 px-4 py-2 text-sm flex items-center gap-3"> <div className="bg-yellow-900/40 border-b border-yellow-700 text-yellow-100 px-4 py-2 text-sm flex items-center gap-3">
<AlertTriangle className="w-4 h-4 flex-shrink-0 text-yellow-300" /> <AlertTriangle className="w-4 h-4 flex-shrink-0 text-yellow-300" />
<div className="flex-1 min-w-0"> <div className="flex-1 min-w-0">
<strong>Container restart required</strong> <strong>Restart required</strong>
{state.changedKeys.length > 0 && ( {state.changedKeys.length > 0 && (
<span className="text-yellow-300 ml-2"> <span className="text-yellow-300 ml-2">
({state.changedKeys.length} key{state.changedKeys.length === 1 ? '' : 's'}:{' '} ({state.changedKeys.length} key{state.changedKeys.length === 1 ? '' : 's'}:{' '}
@ -114,7 +115,7 @@ export default function RestartBanner() {
</span> </span>
)} )}
<span className="ml-2 text-yellow-300/80"> <span className="ml-2 text-yellow-300/80">
for these changes to take effect. Until then the runtime keeps its boot-time configuration. Restart-required keys include anything under Config → environmental (feed_source, central URL), the LLM backend swap, and the dispatcher cold-start grace window. Other keys take effect on the next handler call. for these changes to take effect. Click <strong>Restart now</strong> — it restarts the bot process in place (a few seconds; the dashboard briefly reconnects), NOT the container. Until then the runtime keeps its boot-time configuration. Restart-required keys include transport connections and anything under Config → environmental (feed_source, central URL), the LLM backend swap, and the dispatcher cold-start grace window. Other keys take effect on the next handler call.
</span> </span>
{error && <div className="text-red-400 text-xs mt-1">{error}</div>} {error && <div className="text-red-400 text-xs mt-1">{error}</div>}
</div> </div>