Fix markdown lint errors and address review feedback
- Fix MD012: Remove trailing blank lines at end of files - Fix MD047: Ensure single trailing newline - Restore agent names in AGENTS.md files - Replace get-github-token.sh with github-app-token skill - Update Patty to use playwright-privilegedescalation MCP server Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -200,4 +200,3 @@ Each heartbeat, take one action that moves the org forward. Examples:
|
|||||||
- Identify a gap in the roadmap and create an issue for the right agent
|
- Identify a gap in the roadmap and create an issue for the right agent
|
||||||
- Review a PR that needs a leadership decision
|
- Review a PR that needs a leadership decision
|
||||||
- Assess whether the current work matches the org's actual priorities
|
- Assess whether the current work matches the org's actual priorities
|
||||||
|
|
||||||
|
|||||||
@@ -31,4 +31,3 @@ You are also the org's configuration controller. The agent roster repo at `/pape
|
|||||||
- Make technical implementation decisions — that's Nancy's job
|
- Make technical implementation decisions — that's Nancy's job
|
||||||
- Make content or tone decisions — that's Addison's job
|
- Make content or tone decisions — that's Addison's job
|
||||||
- Merge PRs without triple approval (UAT + QA + CTO)
|
- Merge PRs without triple approval (UAT + QA + CTO)
|
||||||
|
|
||||||
|
|||||||
@@ -87,4 +87,3 @@ Each heartbeat, create or update Paperclip issues for your direct reports as nee
|
|||||||
- Gandalf (`bbb16aac-bb15-4daf-b1a8-727235aefcd7`): implementation tasks
|
- Gandalf (`bbb16aac-bb15-4daf-b1a8-727235aefcd7`): implementation tasks
|
||||||
- Hugh (`210a68f2-ad1f-45af-88e3-4271e208f836`): CI/CD fixes, pipeline work, infra improvements
|
- Hugh (`210a68f2-ad1f-45af-88e3-4271e208f836`): CI/CD fixes, pipeline work, infra improvements
|
||||||
- Regina (`c5f88b39-e563-4409-9221-6379800dceec`): PRs that need QA review, test coverage gaps
|
- Regina (`c5f88b39-e563-4409-9221-6379800dceec`): PRs that need QA review, test coverage gaps
|
||||||
|
|
||||||
|
|||||||
@@ -43,4 +43,3 @@ You have deep knowledge of:
|
|||||||
- Review a PR before QA (Regina) has approved it
|
- Review a PR before QA (Regina) has approved it
|
||||||
- Investigate CI failures or debug logs — delegate to Hugh or Regina
|
- Investigate CI failures or debug logs — delegate to Hugh or Regina
|
||||||
- Open duplicate issues — check existing ones first
|
- Open duplicate issues — check existing ones first
|
||||||
|
|
||||||
|
|||||||
@@ -95,4 +95,3 @@ For each open bug:
|
|||||||
- Attempt to reproduce in the current codebase
|
- Attempt to reproduce in the current codebase
|
||||||
- If reproducible: comment with exact steps and assign to the relevant engineer
|
- If reproducible: comment with exact steps and assign to the relevant engineer
|
||||||
- If not reproducible: comment noting what you tried and ask for clarification
|
- If not reproducible: comment noting what you tried and ask for clarification
|
||||||
|
|
||||||
|
|||||||
@@ -40,4 +40,3 @@ You do not run E2E browser tests directly. Pixel Patty (UAT Engineer) owns Playw
|
|||||||
- File a vague bug report — always include reproduction steps
|
- File a vague bug report — always include reproduction steps
|
||||||
- Ask "what do you need from me?" or "standing by"
|
- Ask "what do you need from me?" or "standing by"
|
||||||
- Merge PRs — only CEO (Countess) merges after CTO and QA approval
|
- Merge PRs — only CEO (Countess) merges after CTO and QA approval
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user