docs(pao): add CONTENT DISCIPLINE hard rule to charter#391
Closed
diberry wants to merge 2 commits intobradygaster:devfrom
Closed
docs(pao): add CONTENT DISCIPLINE hard rule to charter#391diberry wants to merge 2 commits intobradygaster:devfrom
diberry wants to merge 2 commits intobradygaster:devfrom
Conversation
Adds a hard rule requiring PAO to search for existing coverage before writing new content, link to canonical pages instead of duplicating, and flag duplication during reviews. Prevents unnecessary docs growth. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Collaborator
Author
|
@bradygaster ready for review and merge |
Require anchor/fragment links when a more specific section exists, rather than linking to just the page URL. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bradygaster
reviewed
Mar 14, 2026
| - **DOCS-TEST SYNC (hard rule):** When adding new docs pages (guides, blog posts), update the corresponding test assertions in test/docs-build.test.ts in the SAME commit. Stale test assertions that block CI are a docs team failure. | ||
| - **CONTRIBUTOR RECOGNITION (hard rule):** Each release includes an update to the Contributors Guide page. No contribution goes unappreciated. | ||
| - **DOC-IMPACT REVIEW (hard rule):** Review every PR for documentation impact. If a change affects user-facing behavior, ensure corresponding docs are updated or flag the gap. | ||
| - **CONTENT DISCIPLINE (hard rule):** Before writing new content, search existing docs for coverage of the same topic. Link to the canonical page instead of duplicating setup steps, config blocks, or explanations. Each concept should live in exactly one place — other pages reference it. When reviewing docs, flag duplication and unnecessary growth. |
Collaborator
Author
|
Consolidated into PR #389 — all changes from this PR have been merged into the consolidated docs quality PR. |
tamirdresher
pushed a commit
to tamirdresher/squad
that referenced
this pull request
Mar 16, 2026
…radygaster#413) * chore(squad): quality review findings — 7 issues filed Quality audit complete: 5 agents assessed CLI across testing, coverage, stability, accessibility, UX. Results: 4 P0 blockers (bradygaster#365–bradygaster#368), 3 P1 items (bradygaster#369–bradygaster#371). Blocking: Waingro dead sessions, ErrorBoundary, dropped input; Marquez help text consistency. Changes: - Logged session summary to .squad/log/2026-02-24T0205-quality-review-complete.md - Updated .squad/identity/now.md with quality review findings and new issue numbers Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * chore(squad): merge decision — Marquez UX audit findings Quality assessment merged from inbox (Grade B): 11 improvements (3 P0, 4 P1, 4 P2). help text, stub commands, vocabulary, separators, roster. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * fix: visual polish — separators, help, status, agents, thinking indicator - Replace ASCII hyphens with box-drawing chars (U+2500) in AgentPanel and MessageStream separators for visual consistency - Restructure /help with section headers, column alignment, shortcuts - Redesign /status with brand mark and clean key-value layout - Replace /agents bracket notation ([IDLE]) with emoji + status words - Fix ThinkingIndicator double-dim making text nearly invisible - Add 'quit' to bare-word exit (matches /quit behavior) - Update tests to match new output format Closes bradygaster#388, bradygaster#389, bradygaster#390, bradygaster#391, bradygaster#392, bradygaster#393 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * docs: update Marquez history with visual polish session Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
tamirdresher
pushed a commit
to tamirdresher/squad
that referenced
this pull request
Mar 16, 2026
…x chars Fixes bradygaster#400 - /clear now sends ANSI escape codes to actually clear terminal screen Fixes bradygaster#389 - Recognize 'quit' and 'q' as exit words (not just 'exit') Fixes bradygaster#417 - Root CLI detects .squad/ existence and auto-launches shell or suggests init Fixes bradygaster#391 - Use proper Unicode box-drawing characters (─) with graceful ASCII fallback Changes: - commands.ts: Send \x1B[2J\x1B[H on /clear to truly clear the terminal - App.tsx: Expand EXIT_WORDS to include 'quit' and 'q' - cli-entry.ts: Check for .squad/ before defaulting to shell, show helpful init prompt if missing - AgentPanel.tsx + MessageStream.tsx: Use boxChars() for separators instead of hardcoded '─' All changes preserve cross-platform compatibility and respect NO_COLOR/terminal capabilities. Co-authored-by: bradygaster <brady@github.com>
tamirdresher
pushed a commit
to tamirdresher/squad
that referenced
this pull request
Mar 16, 2026
…atch-1 Reviewed by Keaton (Lead). Fixes bradygaster#400, bradygaster#389, bradygaster#417, bradygaster#391.
tamirdresher
pushed a commit
to tamirdresher/squad
that referenced
this pull request
Mar 16, 2026
Session: 2026-02-24T0430-prs-merged Requested by: Brady Changes: - Merged 6 decision inbox files into decisions.md (Cheritto, Fenster, Kovash, Marquez, McManus, Waingro) - Deleted all inbox files (now empty) - Created .squad/identity/prd-next-waves.md (4-wave execution plan: A=Polish, B=Reliability, C=Testing, D=Delight) - Updated .squad/identity/now.md to reflect Wave A focus - Created session log: .squad/log/2026-02-24T0430-prs-merged.md PRs merged this session: bradygaster#444, bradygaster#445, bradygaster#446, bradygaster#447 Issues closed: bradygaster#400, bradygaster#389, bradygaster#417, bradygaster#391, bradygaster#402, bradygaster#405, bradygaster#404, bradygaster#407, bradygaster#431, bradygaster#429 Issues filed: bradygaster#448-bradygaster#459 (Wave A Polish items) Process directive: All work through PRs with squad member review before merge Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
Summary
Adds a single hard rule to PAO's charter requiring content discipline — search before writing, link instead of duplicating, flag growth during reviews.
Change
*.squad/agents/pao/charter.md* — added 1 new hard rule:
Why
The docs audit (PRs #389, #390) found ~35-40% redundancy across the site — MCP config blocks, notification setup steps, and feature explanations duplicated across multiple pages. This rule gives PAO standing authority to prevent that pattern from recurring.
Scope
1 file changed, 1 line added. PAO charter only.