Conversation
…/patterns/ subdir)
Acceptance:
- Flip docs/promotions/P0004 promotion_status: proposed -> accepted
- Update tags array ("proposed" -> "accepted")
- Fill Review Notes with operator decision (klappy, 2026-05-05)
Execution:
- canon/patterns/docs-proxy-canon-as-tool.md (NEW): tier-2 pattern doc
- This introduces a NEW canon subdirectory: canon/patterns/
(existing subdirs: bootstrap, case-studies, constraints, decisions,
defaults, definitions, diagnostics, instructions, meta, methods,
observations, principles, resonance, values)
- Frontmatter declares: derives_from vodka-architecture +
consistency-same-pattern-every-time + dry-canon-says-it-once;
complements doing-less-enables-more
- Codifies the docs-proxy tool: action MCP servers expose docs(query, ...)
that forwards to the canon-server (oddkit) with the action server's
repo URL pinned as knowledge_base_url; consumers wire one MCP, get both
surfaces; vodka boundary preserved (zero domain semantics in the proxy)
Receipts: PTXprint v1.2 §3 docs tool (added 2026-04-29, reversing v1.0's
no-retrieval decision).
Seventh of 8 stuck proposals. Previous: P0009 (#167), P0001 (#168),
P0008 (#169), P0007 (#170), P0006 (#171), P0003 (#172). Last: P0005
(async-by-default principle, new doc).
Canon Quality —
|
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
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 this PR does
Combined acceptance + execution of promotion P0004 — adds a tier-2 pattern doc for the docs-proxy tool that lets MCP server consumers wire one MCP instead of two.
Acceptance (1 file)
docs/promotions/P0004-docs-proxy-canon-as-tool.mdpromotion_status: proposed→accepted"proposed"→"accepted"accepted(2026-05-05)Execution (1 file, NEW)
canon/patterns/docs-proxy-canon-as-tool.md(new tier-2 pattern doc)tier: 2,audience: canon,status: activederives_from:vodka-architecture,consistency-same-pattern-every-time,dry-canon-says-it-oncecomplements:doing-less-enables-more⚠ New canon subdirectory:
canon/patterns/This is the first doc under
canon/patterns/. Existing canon subdirectories are:bootstrap,case-studies,constraints,decisions,defaults,definitions,diagnostics,instructions,meta,methods,observations,principles,resonance,values.The proposal explicitly names
canon/patterns/docs-proxy-canon-as-tool.mdas the target. If you'd rather the doc live undermethods/orprinciples/, the move is onegit mv+ a one-lineuri:edit in the frontmatter — no downstream consumers yet.Position in the 8-proposal sweep
DoD
promotion_statusflippedfrontmatter-schemaNote
Low Risk
Low risk documentation-only change; main risk is introducing a new
canon/patterns/directory that could affect nav/build discovery if tooling assumes a fixed canon tree.Overview
Accepts P0004 by updating
docs/promotions/P0004-docs-proxy-canon-as-tool.mdfromproposedtoaccepted, including frontmatter tags/status and filled review notes.Executes the promotion by adding a new tier-2 canon pattern doc at
canon/patterns/docs-proxy-canon-as-tool.md, establishing the newcanon/patterns/section and specifying the expecteddocs(query, audience?, depth?)proxy tool shape, return contract, and graceful failure behavior.Reviewed by Cursor Bugbot for commit 1df616c. Bugbot is set up for automated code reviews on this repo. Configure here.