Skip to content

Ship the amico verb surface: package launcher + AGENTS.md bookkeeping-verbs section - #132

Draft
aarontrowbridge wants to merge 1 commit into
mainfrom
aaron/ship-amico-verbs
Draft

Ship the amico verb surface: package launcher + AGENTS.md bookkeeping-verbs section#132
aarontrowbridge wants to merge 1 commit into
mainfrom
aaron/ship-amico-verbs

Conversation

@aarontrowbridge

Copy link
Copy Markdown
Member

Stacked on #131 (base aaron/vault-mounts-verbs) — top of the b1→b3→#131 spine chain.

Layer 0 of the integration doctrine (spec-20260711-132200, personal vault): the session-study found the entire amico verb surface (catalog/vault/device/note) implemented but unreachable and undiscoverable — not staged into the .vsix, not on PATH, and never mentioned by AGENTS.md/scores/skills. Zero invocations across 6 fresh probe sessions; warm-starts were done by hand-globbing the pulse bank and results never recorded in-session.

Three changes:

  1. esbuild staging ships launcher/amico + dist/amico.js beside amico-run — one PATH prepend now resolves both bins. Verified: vsix contains both; staged amico vault status resolves the real 3-mount stack.
  2. AGENTS.md § Bookkeeping verbs: before authoring → catalog query (warm-start seeds) + vault query (prior art); after FINISHED → note write --from-run + catalog ingest --from-run (promotion refuses without verification.agree); vault status/resolve for mount questions. Plus the vault layout convention (amicode/{problems,pulses,memory}) — the invented-flat-path read-error class had 16 occurrences in session mining.
  3. Router fix: deprecated amicode_ask → native question (removes the mixed guidance in the spliced prompt).

Tests: extension 435 passed | 3 skipped (unchanged), typecheck clean, pnpm run package green with both bins in the artifact. CI red on the pre-existing private-fork token — unrelated.

🤖 Generated with Claude Code

@jack-champagne
jack-champagne changed the base branch from aaron/vault-mounts-verbs to main July 28, 2026 14:13
@jack-champagne

Copy link
Copy Markdown
Member

Retargeted from aaron/vault-mounts-verbs to main.

The old base is now fully merged into main, so the PR was comparing against a dead branch. Against main the real remaining change is 2 files, +8 — that's what actually needs review here.

(Part of a sweep over the stacked b*/aaron/* chains: #119, #125 and #131 turned out to be entirely in main already and were closed; this one has real work left.)

The skill tool routes every load through a per-name permission ask by
default. Our staged set is surface+entitlement filtered, so the ask is
friction without a boundary — and hangs headless sessions, same class
as the external_directory default.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@jack-champagne
jack-champagne force-pushed the aaron/ship-amico-verbs branch from b135875 to 6b8edc7 Compare July 29, 2026 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants