mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-07-31 19:16:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

refactor(desktop): drop the legacy localStorage queue migration

Browse source 
The old client-owned queue was ephemeral draft state; losing a queued
message across the one upgrade isn't worth carrying migration code
forever. The stale localStorage key is simply ignored.
This commit is contained in:
ethernet 2026-07-15 17:02:19 -04:00
parent bd70904c26
commit 5c8ae70d2e
2 changed files with 0 additions and 59 deletions
Download patch file Download diff file Expand all files Collapse all files

8
apps/desktop/src/app/chat/composer/hooks/use-composer-queue.ts
View file

@ -6,7 +6,6 @@ import { type ComposerAttachment } from '@/store/composer'
import {
$pendingSteersBySession,
$queuedPromptsBySession,
migrateLegacyQueue,
migrateQueuedPrompts,
promoteQueuedPrompt,
type QueuedPromptEntry,
@ -221,13 +220,6 @@ export function useComposerQueue({
migrateQueuedPrompts(prev, activeQueueSessionKey)
}, [activeQueueSessionKey, queueSessionKey])
// One-time legacy migration: entries stranded in the localStorage queue
// (from before the queue moved to the gateway) are pushed to
// session.queue.add and the storage key is cleared.
useEffect(() => {
migrateLegacyQueue(activeQueueSessionKey)
}, [activeQueueSessionKey])
// Queue-edit cleanup: on session swap the scope effect already stashed the
// edit snapshot; only restore into the composer when still on the same scope.
useEffect(() => {

51
apps/desktop/src/store/composer-queue.ts
View file

@ -36,10 +36,6 @@ export interface PendingSteerEntry {
type QueueState = Record<string, QueuedPromptEntry[]>
type SteerState = Record<string, PendingSteerEntry[]>
// Legacy localStorage key from the client-owned queue era. Read once by
// migrateLegacyQueue() to push stranded entries to the gateway, then cleared.
const LEGACY_STORAGE_KEY = 'hermes.desktop.composerQueue.v1'
export const $queuedPromptsBySession = atom<QueueState>({})
export const $pendingSteersBySession = atom<SteerState>({})
@ -295,53 +291,6 @@ export const migrateQueuedPrompts = (fromKey: string | null | undefined, toKey:
return true
}
/** One-time migration: push any entries stranded in the legacy localStorage
* queue (client-owned era) to the gateway, then clear the storage key. */
export const migrateLegacyQueue = (key: string | null | undefined) => {
const sid = sidOf(key)
if (!sid || typeof window === 'undefined') {
return
}
try {
const raw = window.localStorage.getItem(LEGACY_STORAGE_KEY)
if (!raw) {
return
}
const parsed: unknown = JSON.parse(raw)
if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) {
window.localStorage.removeItem(LEGACY_STORAGE_KEY)
return
}
const state = parsed as Record<string, { text?: string }[]>
const entries = state[sid]
if (entries?.length) {
for (const entry of entries) {
if (entry?.text?.trim()) {
void enqueueQueuedPrompt(sid, { text: entry.text })
}
}
}
delete state[sid]
if (Object.keys(state).length === 0) {
window.localStorage.removeItem(LEGACY_STORAGE_KEY)
} else {
window.localStorage.setItem(LEGACY_STORAGE_KEY, JSON.stringify(state))
}
} catch {
// Best-effort — a broken legacy blob shouldn't take down the composer.
}
}
// ── Pending steers ───────────────────────────────────────────────────
// A steer RPC is accepted instantly, but the text only reaches the model at
// the next tool-batch boundary. These helpers track that in-between state so