starred/orca
1
0
Fork 0
mirror of https://github.com/stablyai/orca.git synced 2026-09-28 16:02:45 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(agent-status): mint the canonical epoch on first use, not at construction

Browse Source 
The hook server's canonical store was built in an instance-member initializer, so
constructing AgentHookServer — which happens at import time for the module
singleton — demanded a live randomUUID. Any importer that stubs node:crypto threw
'Invalid agent status store epoch' before a single test ran.

The store is now created on first canonical access and reset by dropping it, so
construction owes nothing to a crypto implementation and the epoch still rotates
per authority incarnation.
This commit is contained in:
Brennan Benson
2026-09-16 00:45:53 -07:00
parent 69f14ab7ac
commit 65b9885ccf
1 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/main/agent-hooks/server/server-state.ts
+12 -6
View File
@@ -22,7 +22,7 @@ import type { AgentHookSource } from '../../../shared/agent-hook-relay'
import type { AgentStatusClearIpcPayload } from '../../../shared/agent-status-types'
import type { LegacyPaneKeyAliasEntry } from '../../../shared/persisted-state-types'
import type { SpoolRecord } from '../../../shared/agent-hook-spool'
import { createAgentStatusStore } from '../../../shared/agent-status-store'
import { createAgentStatusStore, type AgentStatusStore } from '../../../shared/agent-status-store'
import { AGENT_STATUS_2A_CURRENT_PRODUCER_MODE } from '../../../shared/agent-status-legacy-adapter'
import type { AgentStatusStructuredSessionSubject } from '../../../shared/agent-status-subject'
import type {
@@ -58,10 +58,16 @@ export abstract class AgentHookServerState {
)
}
protected canonicalStatusStore = createAgentStatusStore({
epoch: randomUUID(),
mode: 'authority'
})
// Why: the epoch is minted on first canonical use, so constructing the server — which happens at
// import time for the module singleton — owes nothing to a live crypto implementation.
private canonicalStatusStoreInstance: AgentStatusStore | null = null
protected get canonicalStatusStore(): AgentStatusStore {
this.canonicalStatusStoreInstance ??= createAgentStatusStore({
epoch: randomUUID(),
mode: 'authority'
})
return this.canonicalStatusStoreInstance
}
protected readonly canonicalListingOrder = new Map<string, number>()
protected readonly canonicalSubjectsByPane = new Map<
string,
@@ -71,7 +77,7 @@ export abstract class AgentHookServerState {
protected nextStatusListingOrder = (): number => ++this.statusListingOrder
protected resetCanonicalStatus(): void {
this.canonicalStatusStore = createAgentStatusStore({ epoch: randomUUID(), mode: 'authority' })
this.canonicalStatusStoreInstance = null
this.canonicalListingOrder.clear()
this.canonicalSubjectsByPane.clear()
}