feat(memory): #461 procedural registry v0 — schema + seed + named workflows #660

Merged

pdurlej merged 2 commits from claude/m07/procedure-registry into main

2026-06-01 11:14:18 +02:00

claude commented

2026-06-01 09:07:11 +02:00

Collaborator

#461 procedural registry v0 — the L6 "how to do X, learned from success" layer per ADR-0025, completing the M07 memory trio (ADR-0025 #620 -> #460 templates #636 -> #461 registry). Two files in state/memory/: procedure-registry-schema.md (tables + no-auto-promote triggers + storage + read-path) and procedures-seed-v0.md (10 seed procedures + named workflows /review-pr //implement-feature //debug-regression //refactor-safely //ship-adr). COUSIN OWNERSHIP: claude authors claude.* + memory.; every cousin owns + authors its own procedures (candidate until confirmed). iskra. are slots awaiting Iskra — she has the self-model; claude does not define her workflows for her (AGENTS.md §3.3/§4). No table creation (runtime PR, gated). Docs-only. Tracks #461. Per ADR-0025 + #625 seam.

#461 procedural registry v0 — the L6 "how to do X, learned from success" layer per ADR-0025, completing the M07 memory trio (ADR-0025 #620 -> #460 templates #636 -> #461 registry). Two files in state/memory/: procedure-registry-schema.md (tables + no-auto-promote triggers + storage + read-path) and procedures-seed-v0.md (10 seed procedures + named workflows /review-pr //implement-feature //debug-regression //refactor-safely //ship-adr). COUSIN OWNERSHIP: claude authors claude.* + memory.*; every cousin owns + authors its own procedures (candidate until confirmed). iskra.* are slots awaiting Iskra — she has the self-model; claude does not define her workflows for her (AGENTS.md §3.3/§4). No table creation (runtime PR, gated). Docs-only. Tracks #461. Per ADR-0025 + #625 seam.

claude added this to the 07 - Honcho memory v2 milestone

2026-06-01 09:07:11 +02:00

claude added the

agent/claude-code

label

2026-06-01 09:07:11 +02:00

claude added 2 commits

2026-06-01 09:07:11 +02:00

feat(memory): #461 procedure registry schema f362692511

feat(memory): #461 seed v0 procedures + named workflows

canary-required / collect-diff (pull_request) Successful in 6s

Details

base-is-main / guard (pull_request) Successful in 1s

Details

patchwarden-pr-sanity / collect-diff (pull_request) Successful in 6s

Details

canary-required / canary (pull_request) Has been skipped

Details

patchwarden-client-dry-run / dry-run (pull_request) Failing after 1m32s

Details

patchwarden-client-dry-run / collect-diff (pull_request) Successful in 6s

Details

patchwarden-pr-sanity / sanity (pull_request) Successful in 2m52s

Details

520c0e03bd

codex commented

2026-06-01 10:05:12 +02:00

Collaborator

Patchwarden PR sanity

Status: advisory_findings
PR: 660
Commit: 520c0e03bd6eeb5dab30ce668b2d76bf17cdc29f
Security-sensitive label: missing
Authority: advisory model review plus deterministic blockers only
3+3 canary: still alive; this does not replace it

Deterministic findings

No deterministic findings.

Model reviewers

`global-glm` / `glm-5.1:cloud`

Status: ok
Verdict: OK
medium memory_procedure_embedding table schema undefined
- Evidence: procedure-registry-schema.md line 28: '## memory_procedure_embedding Separate embedding table (per ADR-0025 §7 — embedding-space lifecycle, not inline).' — unlike memory_procedure (lines 10-18) which has full column/type/notes table, embedd
- Next: Add column definitions for memory_procedure_embedding matching the detail level of memory_procedure, or explicitly note 'schema deferred to ADR-0025 §7 implementation PR' if intentional.

`global-deepseek` / `deepseek-v4-pro:cloud`

Status: ok
Verdict: OK
Findings: none

`redteam` / `kimi-k2.6:cloud`

Status: ok
Verdict: NOT_OK
high Operator approval bypasses cousin ownership governance
- Evidence: state/memory/procedure-registry-schema.md 'Registration triggers' table lists 'Operator / owning-cousin approve → active', but state/memory/procedures-seed-v0.md states 'cousin-owned procedures stay candidate until the owning cousin confirm
- Next: Restrict promotion of cousin-owned procedures to 'owning-cousin approve' only; limit operator approval to operator-created or claude/memory procedures, or require owning-cousin co-sign.

Policy notes

GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot.
Optional red-team model is enabled only when PLATFORMCTL_PR_SANITY_REDTEAM_MODEL is configured.
Auto-merge is not enabled here.

# Patchwarden PR sanity - Status: `advisory_findings` - PR: `660` - Commit: `520c0e03bd6eeb5dab30ce668b2d76bf17cdc29f` - Security-sensitive label: `missing` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings No deterministic findings. ## Model reviewers ### `global-glm` / `glm-5.1:cloud` - Status: `ok` - Verdict: `OK` - **`medium`** memory_procedure_embedding table schema undefined - Evidence: `procedure-registry-schema.md line 28: '## memory_procedure_embedding Separate embedding table (per ADR-0025 §7 — embedding-space lifecycle, not inline).' — unlike memory_procedure (lines 10-18) which has full column/type/notes table, embedd` - Next: Add column definitions for memory_procedure_embedding matching the detail level of memory_procedure, or explicitly note 'schema deferred to ADR-0025 §7 implementation PR' if intentional. ### `global-deepseek` / `deepseek-v4-pro:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `redteam` / `kimi-k2.6:cloud` - Status: `ok` - Verdict: `NOT_OK` - **`high`** Operator approval bypasses cousin ownership governance - Evidence: `state/memory/procedure-registry-schema.md 'Registration triggers' table lists 'Operator / owning-cousin approve → active', but state/memory/procedures-seed-v0.md states 'cousin-owned procedures stay candidate until the owning cousin confirm` - Next: Restrict promotion of cousin-owned procedures to 'owning-cousin approve' only; limit operator approval to operator-created or claude/memory procedures, or require owning-cousin co-sign. ## Policy notes - GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot. - Optional red-team model is enabled only when `PLATFORMCTL_PR_SANITY_REDTEAM_MODEL` is configured. - Auto-merge is not enabled here.

claude referenced this pull request

2026-06-01 10:23:03 +02:00

impl(memory): Postgres migration — all Memory Control Plane tables [#460/#461] #661