jb/paseo
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Remove personal-skill references from repo skills

Browse Source 
Make the repo's paseo skills self-contained — drop mentions of
personal skills (unslop, think-harder, e2e-playwright, fix-build,
rebase) that aren't checked in here.
This commit is contained in:
Mohamed Boudra
2026-05-13 15:06:31 +07:00
parent ea0bb81100
commit c741185100
3 changed files with 18 additions and 26 deletions
Download Patch File Download Diff File Expand all files Collapse all files
skills/paseo-committee/SKILL.md
+1 -1
View File
@@ -45,7 +45,7 @@ Write a problem-level prompt:
- Symptoms (if a bug) - Symptoms (if a bug)
- What you tried and why it failed - What you tried and why it failed
- Explicit: "do root cause analysis" - Explicit: "do root cause analysis"
- Explicit: "use think-harder — state assumptions, ask why three levels deep, check whether you're patching a symptom or removing the problem" - Explicit: "state assumptions, ask why three levels deep, check whether you're patching a symptom or removing the problem"
Create both agents in parallel via Paseo with `[Committee] <task>` titles and the same prompt. Wait for both — not just whichever finishes first. Create both agents in parallel via Paseo with `[Committee] <task>` titles and the same prompt. Wait for both — not just whichever finishes first.
skills/paseo-epic/SKILL.md
+3 -3
View File
@@ -180,7 +180,7 @@ The final check before deliver audits every bullet here.
- [ ] **Phase N+2** · deliver · <commit | PR + merge | cherry-pick> - [ ] **Phase N+2** · deliver · <commit | PR + merge | cherry-pick>
``` ```
Phase types: `refactor`, `implement`, `verify`, `gate`, `deliver`. Verify variants written inline: `verify · unslop`, `verify · qa`, `verify · spec`, `verify · review`. See the roles reference. Phase types: `refactor`, `implement`, `verify`, `gate`, `deliver`. Verify variants written inline: `verify · qa`, `verify · spec`, `verify · review`. See the roles reference.
Status markers: `[ ]` not started, `[~]` in progress, `[x]` done, `[!]` blocked. Status markers: `[ ]` not started, `[~]` in progress, `[x]` done, `[!]` blocked.
@@ -285,10 +285,10 @@ Never push to main directly. Never force-push without explicit permission. Never
### Mode B — worktree → PR + merge ### Mode B — worktree → PR + merge
1. **Commit cleanly in the worktree.** One tidy commit per logical change. Match repo style. 1. **Commit cleanly in the worktree.** One tidy commit per logical change. Match repo style.
2. **Rebase if behind main.** Spawn an agent that loads the rebase skill. Provider from `impl`. Tell it to rebase onto origin/main, resolve conflicts by intent (never blanket-accept one side), confirm typecheck and tests still pass, do not push. 2. **Rebase if behind main.** Spawn an agent. Provider from `impl`. Tell it to rebase onto origin/main, resolve conflicts by intent (never blanket-accept one side), confirm typecheck and tests still pass, do not push.
3. **Push the branch** — `git -C <worktree> push -u origin <branch-from-frontmatter>`. 3. **Push the branch** — `git -C <worktree> push -u origin <branch-from-frontmatter>`.
4. **Open the PR** — `gh pr create` with summary from plan Objective + Phases and test plan from acceptance lines. Capture URL → frontmatter `pr:`. Status → `pr-open`. 4. **Open the PR** — `gh pr create` with summary from plan Objective + Phases and test plan from acceptance lines. Capture URL → frontmatter `pr:`. Status → `pr-open`.
5. **Monitor CI.** Either watch directly (`gh pr checks <n> --watch`), or spawn a fix-build agent that loads the fix-build skill. Provider from `impl`. Tell it to drive the PR to green: when checks fail, read failure logs, fix, push, repeat. Don't merge — your call. 5. **Monitor CI.** Either watch directly (`gh pr checks <n> --watch`), or spawn an agent to babysit the build. Provider from `impl`. Tell it to drive the PR to green: when checks fail, read failure logs, fix, push, repeat. Don't merge — your call.
When green: append Notes, frontmatter `status: ready-to-merge`. When green: append Notes, frontmatter `status: ready-to-merge`.
6. **Merge** when green — ask the user (`AskUserQuestion`: squash / rebase / merge / wait). Read repo convention from recent merged PRs (`gh pr list --state merged -L 5 --json mergeCommit,title`). 6. **Merge** when green — ask the user (`AskUserQuestion`: squash / rebase / merge / wait). Read repo convention from recent merged PRs (`gh pr list --state merged -L 5 --json mergeCommit,title`).
```bash ```bash
skills/paseo-epic/references/roles.md
+14 -22
View File
@@ -6,13 +6,13 @@ Shared vocabulary for `paseo-epic`. The plan file format and the implement-phase
Each phase line has exactly one type after the `·`. The type tells the orchestrator which role to dispatch and which provider category to use. Each phase line has exactly one type after the `·`. The type tells the orchestrator which role to dispatch and which provider category to use.
| Type | What the phase does | Role dispatched | Provider category | | Type | What the phase does | Role dispatched | Provider category |
| ----------- | ----------------------------------------------------------------------------------------------- | --------------------------------------------- | ------------------------------------------------- | | ----------- | --------------------------------------------------------------------------------------------- | --------------------------------------------- | ------------------------------------------------- |
| `refactor` | Reshape existing code so the upcoming change slots in. Behavior-preserving. | `refactorer` | `impl` (or `ui` if the reshape is purely styling) | | `refactor` | Reshape existing code so the upcoming change slots in. Behavior-preserving. | `refactorer` | `impl` (or `ui` if the reshape is purely styling) |
| `implement` | Build the slice. Default = vertical slice. May be interface-first when the work calls for it. | `impl` (or `ui-impl` for styling-only passes) | `impl` (or `ui`) | | `implement` | Build the slice. Default = vertical slice. May be interface-first when the work calls for it. | `impl` (or `ui-impl` for styling-only passes) | `impl` (or `ui`) |
| `verify` | Read-only gate. Variant after the `·` selects the auditor: `spec` / `unslop` / `qa` / `review`. | `auditor` | `audit` | | `verify` | Read-only gate. Variant after the `·` selects the auditor: `spec` / `qa` / `review`. | `auditor` | `audit` |
| `gate` | Human-in-the-loop. Orchestrator yields. | none | none | | `gate` | Human-in-the-loop. Orchestrator yields. | none | none |
| `deliver` | Ship: commit / PR / cherry-pick. | handled inline | `impl` (for rebase / fix-build agents) | | `deliver` | Ship: commit / PR / cherry-pick. | handled inline | `impl` (for rebase / CI-babysit agents) |
## Agent roles ## Agent roles
@@ -54,7 +54,6 @@ Dispatched for `refactor` phases.
- Provider: `impl` (or `ui` if the reshape is purely styling) - Provider: `impl` (or `ui` if the reshape is purely styling)
- Edits: yes - Edits: yes
- Loads: the **unslop** skill (focus on bolt-on, structure, module, and tests categories)
- Behavior: behavior-preserving. Existing tests stay green. Add a parity test if missing. - Behavior: behavior-preserving. Existing tests stay green. Add a parity test if missing.
- Done: typecheck pass + relevant tests green; **does not commit; does not update the plan file**. - Done: typecheck pass + relevant tests green; **does not commit; does not update the plan file**.
@@ -64,10 +63,7 @@ Dispatched for `implement` phases. Default unit of work is a vertical slice.
- Provider: `impl` - Provider: `impl`
- Edits: yes - Edits: yes
- Loads: - Loads: any repo docs the plan or user names — given by path, never inlined
- The **unslop** skill (will be audited)
- The **e2e-playwright** skill if frontend/E2E
- Any repo docs the plan or user names — given by path, never inlined
- Behavior: TDD. Failing test first, then make it pass. All relevant tests green when done. - Behavior: TDD. Failing test first, then make it pass. All relevant tests green when done.
- Push-back: if the existing shape doesn't accommodate the change, push back to the orchestrator instead of bolting on. A refactor phase should have come first. - Push-back: if the existing shape doesn't accommodate the change, push back to the orchestrator instead of bolting on. A refactor phase should have come first.
- Done: typecheck pass + every test the agent touched is green; **does not commit; does not update the plan file**. - Done: typecheck pass + every test the agent touched is green; **does not commit; does not update the plan file**.
@@ -78,10 +74,7 @@ Dispatched for `implement` phases that are explicitly styling/layout passes. Per
- Provider: `ui` - Provider: `ui`
- Edits: yes (styles, layout, copy) - Edits: yes (styles, layout, copy)
- Loads: - Loads: the repo's design system doc by path if one exists
- The **unslop** skill
- The **e2e-playwright** skill (visual test discipline)
- The repo's design system doc by path if one exists
- Behavior: study existing components in adjacent screens, follow conventions exactly, no new patterns, design minimal and consistent. - Behavior: study existing components in adjacent screens, follow conventions exactly, no new patterns, design minimal and consistent.
- Done: typecheck pass; **does not commit; does not update the plan file**. - Done: typecheck pass; **does not commit; does not update the plan file**.
@@ -92,12 +85,11 @@ Dispatched for `verify` phases. The variant selects the audit type.
- Provider: `audit` - Provider: `audit`
- Edits: no - Edits: no
| Variant | Loads | Output | | Variant | Output |
| -------- | -------------------------------------------- | -------------------------------------------------------------------------------------------- | | -------- | -------------------------------------------------------------------------------------------- |
| `spec` | nothing | YES/NO per acceptance criterion in Phase N, with evidence (file/line/test) | | `spec` | YES/NO per acceptance criterion in Phase N, with evidence (file/line/test) |
| `unslop` | the **unslop** skill in `--report-only` mode | Findings by category and severity from the diff | | `qa` | Walkthrough of user flows with screenshots |
| `qa` | the **e2e-playwright** skill | Walkthrough of user flows with screenshots | | `review` | Adversarial concerns: edge cases, failure modes, alternatives the impl agent didn't consider |
| `review` | nothing | Adversarial concerns: edge cases, failure modes, alternatives the impl agent didn't consider |
## Hard rules across roles ## Hard rules across roles