feat(web): align memory plant/list/read to #177 (signed memory:<ns> service + query-aware engine) + web wire runbook#180
Closed
hanwencheng wants to merge 1 commit into
Closed
Conversation
…ery-aware engine) + web wire test runbook #177 made memory namespace-partitioned behind the gate (OpenViking ranking engine). The web front-end + web-flow docs now match the canonical model (arch.md §896): - memory cap/scope service is the SIGNED memory:<ns> (e.g. memory:travel); the worker keys storage per-namespace (memory:<ns>.enc); a bare memory fails cap-mint (service_not_in_scope). New memoryService(ns) helper in lib/constants. - reading is query-aware: the agent's pre_llm_call read is engine-ranked (OpenViking, else deterministic fallback) over the gate-bounded lines, never widening past granted namespaces; engine chosen at wire-time. Front-end functions: constants.ts (memoryService), client/core.ts (capMemoryPut/Get doc), web-core/broker.rs (CapRequest.service doc), client/types.ts (MasterMemoryEntry.ns doc + section), _components/memory.tsx (desc + planted banner). Docs: new web-wire-test-runbook.md (mirror of phase1-wire-demo) + README index row; aligned stage3-agent-usage.md + data-model.md from bare travel -> memory:travel (no-divergence fix). Daemon-mode /v1/master/memory[/plant] is structurally unchanged (master's own IAM-scoped store; #177 touched broker/cli/core/worker-creds, not the daemon). Verified: cargo fmt --check + clippy -D warnings + wasm-pack build + tsc --noEmit + next build all green.
hanwencheng
force-pushed
the
claude/web-memory-namespace-177
branch
from
4a8d01b to
f02f8f7
Compare
Member
Author
|
inclueded by #187 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
#177 made AgentKeys memory namespace-partitioned behind the gate (OpenViking ranking engine). This updates the parent-control web app's memory plant / list / read surface + the web-flow docs to match the canonical model (arch.md §896), and adds a test runbook mirroring
harness/phase1-wire-demo.sh.The CoreBackend / web-core cap path this documents merged with #172, so these changes sit cleanly on
main.Two #177 truths the front-end now reflects
memory:<ns>(e.g.memory:travel): the worker keys storage per-namespace (bots/<actor>/memory/memory:<ns>.enc), the broker hashes it forisServiceInScope, and a barememoryfails cap-mint (service_not_in_scope).pre_llm_callread is engine-ranked (OpenViking, else a deterministic fallback) over the gate-bounded lines — ranking reorders, never widens past granted namespaces. Engine chosen atwire-time.Front-end functions
lib/constants.ts— newmemoryService(ns) → "memory:${ns}"(single source for the convention).lib/client/types.ts—MasterMemoryEntry.ns+ memory-section docs point atmemory:<ns>/memoryService/ query-ranked read.lib/client/core.ts—capMemoryPut/Getdoc:req.servicemust bememory:<ns>.crates/agentkeys-web-core/src/broker.rs—CapRequest.servicedoc: namespace-qualified; barememory→service_not_in_scope.app/_components/memory.tsx— page desc + "planted" banner: agents read permemory:<ns>scope, engine-ranked per query.Docs (incl. a divergence fix)
docs/plan/web-flow/web-wire-test-runbook.md— the web mirror ofphase1-wire-demo.sh(+ a README File-map row).stage3-agent-usage.md+data-model.md— corrected baretravel→memory:travel(they'd drifted from the post-OpenViking memory engine behind the gate — adapter + query-aware hook + runbook (#147) #177wire-real-paths.md); per-namespaces3_key; query-ranked inject context.Not changed (deliberately)
Daemon-mode
/v1/master/memory[/plant]— the master's own IAM-scoped store; #177 touched broker/cli/core/worker-creds, not the daemon.empty.ts/pairing.tsxremain stubs.Verification
cargo fmt --check·cargo clippy -p agentkeys-web-core -- -D warnings·wasm-pack build·tsc --noEmit·next build— all green on themainbaseline (which now includes #177/#178).🤖 Generated with Claude Code