From 30d558132b0abfa0780ddeaadbe77b3a1a6be3d2 Mon Sep 17 00:00:00 2001
From: thePR0M3TH3AN <53631862+PR0M3TH3AN@users.noreply.github.com>
Date: Fri, 6 Mar 2026 13:53:57 -0500
Subject: [PATCH] mulch: update expertise

---
 .mulch/expertise/architecture.jsonl | 1 +
 .mulch/expertise/seedpass.jsonl     | 2 ++
 .mulch/mulch.config.yaml            | 1 +
 3 files changed, 4 insertions(+)
 create mode 100644 .mulch/expertise/architecture.jsonl

diff --git a/.mulch/expertise/architecture.jsonl b/.mulch/expertise/architecture.jsonl
new file mode 100644
index 00000000..12e1eff4
--- /dev/null
+++ b/.mulch/expertise/architecture.jsonl
@@ -0,0 +1 @@
+{"classification":"tactical","recorded_at":"2026-03-06T18:47:40.029Z","tags":["merge-conflict"],"evidence":{"bead":"SeedPass-57f6"},"type":"pattern","name":"","description":"Merge conflict resolved at tier auto-resolve. Branch: overstory/builder-action-strip/SeedPass-57f6. Agent: builder-action-strip. Conflicting files: src/tests/test_tui_v3_parity.py.","id":"mx-ecd3cf"}
diff --git a/.mulch/expertise/seedpass.jsonl b/.mulch/expertise/seedpass.jsonl
index b8ccb700..e1c95347 100644
--- a/.mulch/expertise/seedpass.jsonl
+++ b/.mulch/expertise/seedpass.jsonl
@@ -16,3 +16,5 @@
 {"type":"convention","content":"tui-v3-worktree-pythonpath: When running pytest against worktree tests for v3, set PYTHONPATH with worktree src first: PYTHONPATH='.overstory/worktrees/builder-app-cmds/src:src'. Without this, the canonical repo src shadows the worktree and tests report AttributeError for new methods even though the edit was correct.","classification":"tactical","recorded_at":"2026-03-06T18:44:40.001Z","outcomes":[{"status":"success","agent":"builder-app-cmds"}],"id":"mx-0d5362"}
 {"type":"pattern","name":"action-bar-click-handlers","description":"Global action bar @click handlers must reference parameterless actions only. Actions requiring arguments (e.g. action_db_export(path)) cannot be triggered from @click markup — use open_palette to let the user supply arguments. Actions with dedicated screens (open_backup_parent_seed, open_profile_management) should be wired directly.","classification":"tactical","recorded_at":"2026-03-06T18:37:01.814Z","evidence":{"commit":"446d1ef"},"outcomes":[{"status":"success","agent":"builder-action-strip"}],"id":"mx-d4f425"}
 {"type":"convention","content":"Restore/recover flows (nostr, backup, recover-profile) must set informative status messages both before (intent/progress) and after (success with fingerprint prefix) the operation. Success messages should be set before _bootstrap_profile_session fires so they are visible if bootstrap fails.","classification":"tactical","recorded_at":"2026-03-06T18:37:04.389Z","evidence":{"commit":"446d1ef"},"outcomes":[{"status":"success","agent":"builder-action-strip"}],"id":"mx-363913"}
+{"type":"pattern","name":"action-bar-managed-qr-gap","description":"v3 action bar managed_account context still includes QR action — should be excluded since managed accounts have no standalone QR. Also action_bar.py has a bare except: on line 37 that should be except Exception:. Minor follow-ups from SeedPass-57f6.","classification":"tactical","recorded_at":"2026-03-06T18:53:47.583Z","evidence":{"bead":"SeedPass-57f6"},"id":"mx-b613a3"}
+{"type":"decision","title":"Cancel TUI v2, focus on v3","rationale":"TUI v2 app.py exceeded 6500 lines and was unmaintainable. v3 modular architecture is already the default UI with full parity. No reason to invest further in v2.","classification":"foundational","recorded_at":"2026-03-06T18:53:53.467Z","id":"mx-b8a6c4"}
diff --git a/.mulch/mulch.config.yaml b/.mulch/mulch.config.yaml
index 4e25b152..593b1382 100644
--- a/.mulch/mulch.config.yaml
+++ b/.mulch/mulch.config.yaml
@@ -1,6 +1,7 @@
 version: '1'
 domains:
   - seedpass
+  - architecture
 governance:
   max_entries: 100
   warn_entries: 150