Skip to content
This repository was archived by the owner on Apr 5, 2026. It is now read-only.

Commit b48a5d1

Browse files
HerbHallclaude
andauthored
docs: ingest autolearn patterns AP#133-134 from Samverk session (#401) (#408)
Add two learned patterns: - AP#133: Prefer dynamic MCP discovery over static prompt files - AP#134: Issue specs must reference verified exported API Both discovered during Samverk dispatch protocol implementation. Closes #401 Co-authored-by: Claude <noreply@anthropic.com>
1 parent b3e268e commit b48a5d1

1 file changed

Lines changed: 18 additions & 1 deletion

File tree

claude/rules/autolearn-patterns.md

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
22
description: Learned patterns from past sessions. Read when encountering similar situations.
33
tier: 2
4-
entry_count: 69
4+
entry_count: 71
55
last_updated: "2026-03-17"
66
---
77

@@ -695,3 +695,20 @@ MUI Popper needs `anchorEl` during render. `useRef` + `ref.current` triggers Rea
695695
**Category:** ci-config
696696
**Context:** Release-please generates CHANGELOG.md with asterisk list markers (`*`) and double blank lines. These violate MD004 and MD012 but cannot be fixed -- the file is regenerated on each release.
697697
**Fix:** Add `"CHANGELOG.md"` to the `ignores` array in `.markdownlint-cli2.jsonc`. Do not attempt `replace_all` on asterisk-space -- it corrupts bold markers (`**text:**`) inside list items.
698+
699+
## 133. Prefer Dynamic MCP Discovery Over Static Prompt Files
700+
701+
**Added:** 2026-03-17 | **Source:** Samverk | **Status:** active
702+
703+
**Category:** process-pattern
704+
**Context:** Hand-written `.samverk/prompts/*.md` files listed issues as critical that were already implemented. MCP calls (`get_digest`, `list_open_prs`) at session start already provide live state. Static prompts duplicate and contradict.
705+
**Fix:** Use dynamic MCP discovery for session orientation. If static prompts are used, they should be generated (not hand-written) and include a staleness warning with a generation timestamp.
706+
**See also:** AP#85 (roadmap drift)
707+
708+
## 134. Issue Specs Must Reference Verified Exported API
709+
710+
**Added:** 2026-03-17 | **Source:** Samverk | **Status:** active
711+
712+
**Category:** process-pattern
713+
**Context:** Issue spec said "call into existing `dispatcher.Claim()`" but no such public method existed, and the caller and callee were in separate processes. A 2-minute codebase check would have caught both problems.
714+
**Fix:** Before writing implementation specs that reference internal methods, verify: (1) the method exists and is exported, (2) the caller and callee are in the same process. Extends AP#47 (check existing assets before scoping) and AP#83 (sprint scope reduction via exploration).

0 commit comments

Comments
 (0)