Skip to content

P0006 (accept + execute): vodka boundary enumeration as spec convention#171

Merged
klappy merged 1 commit intomainfrom
claude/p0006-vodka-boundary-enumeration
May 5, 2026
Merged

P0006 (accept + execute): vodka boundary enumeration as spec convention#171
klappy merged 1 commit intomainfrom
claude/p0006-vodka-boundary-enumeration

Conversation

@klappy
Copy link
Copy Markdown
Owner

@klappy klappy commented May 5, 2026

What this PR does

Combined acceptance + execution of promotion P0006 — sharpens vodka-architecture from philosophy to a concrete spec-shape requirement.

Acceptance (1 file)

  • docs/promotions/P0006-vodka-boundary-enumeration-as-spec-convention.md
    • promotion_status: proposedaccepted
    • Tags array "proposed""accepted"
    • Status section header → accepted (2026-05-05)
    • Review Notes filled

Execution (1 file, +27 lines)

  • canon/principles/vodka-architecture.md — new section before ## See Also:
    • ## Spec Convention — The Boundary Must Be Enumerated
    • Mandates three enumerated sections in any vodka-compliant spec:
      • ## What This Server Knows
      • ## What This Server Does NOT Know ← the vodka boundary, written down
      • ## What This Server Is NOT ← non-goals to rebut for scope expansion
    • Why-Enumeration: written boundary survives author rotation; rebutting an enumerated non-goal is asymmetrically harder than arguing for a useful-sounding addition
    • Failure Mode: implicit boundaries drift; "small additions" cumulatively violate the original vodka frame
    • Receipt: PTXprint v1.2 §1

Position in the 8-proposal sweep

# ID Status
1 P0009 PR #167
2 P0001 PR #168
3 P0008 PR #169
4 P0007 PR #170
5 P0006 this PR ← last of the small append-style PRs
6 P0003 next — reframe-before-trimming (NEW canon doc)
7-8 P0004, P0005 queued (also new docs)

DoD

  • Proposal frontmatter promotion_status flipped
  • Review Notes filled
  • Canon edit text matches P0006 §"Proposed Language" verbatim
  • Insertion at the proposed location ("near the end of the existing doc, before any 'See Also' / footer sections")
  • No other canon docs touched

Note

Low Risk
Docs-only change that introduces a new spec-convention requirement; risk is limited to process/workflow expectations for future vodka-compliant specs.

Overview
Accepts P0006 by flipping its promotion metadata/status to accepted and recording the review decision/date.

Updates canon/principles/vodka-architecture.md with a new spec convention that mandates three explicit sections in any vodka-compliant server spec: ## What This Server Knows, ## What This Server Does NOT Know (the written boundary), and ## What This Server Is NOT (enumerated non-goals), plus rationale and a reference receipt.

Reviewed by Cursor Bugbot for commit 04e9c63. Bugbot is set up for automated code reviews on this repo. Configure here.

Acceptance:
- Flip docs/promotions/P0006 promotion_status: proposed -> accepted
- Update tags array ("proposed" -> "accepted")
- Fill Review Notes with operator decision (klappy, 2026-05-05)

Execution:
- canon/principles/vodka-architecture.md: append new section
  "## Spec Convention — The Boundary Must Be Enumerated" before "## See Also"
- Section requires three enumerated sections in any vodka-compliant spec:
  - ## What This Server Knows (bullet list)
  - ## What This Server Does NOT Know (the vodka boundary, written down)
  - ## What This Server Is NOT (non-goals to rebut for scope expansion)
- Includes Why Enumeration Matters, Failure Mode, Receipts (PTXprint v1.2 §1)

Last of the append-style amendments. Fifth of 8 stuck proposals.
Previous: P0009 (#167), P0001 (#168), P0008 (#169), P0007 (#170).
Next: P0003 — reframe-before-trimming method (NEW canon doc, larger surface).
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

Canon Quality — oddkit_audit

No dead klappy:// references or legacy link patterns found in writings/. 39 files scanned.

Spec: klappy://docs/oddkit/specs/oddkit-audit · Workflow: .github/workflows/canon-quality.yml · Run: #42

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