From c27b03794afa19128c92430ced70ddac194b0842 Mon Sep 17 00:00:00 2001 From: Gustavo Madeira Santana Date: Sat, 7 Feb 2026 21:46:55 -0500 Subject: [PATCH] chore: updated PR review skills and workflow info on tests + fake timers --- .agents/skills/PR_WORKFLOW.md | 1 + .agents/skills/merge-pr/SKILL.md | 4 ++-- .agents/skills/review-pr/SKILL.md | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/.agents/skills/PR_WORKFLOW.md b/.agents/skills/PR_WORKFLOW.md index 0af4d32314..f75414181b 100644 --- a/.agents/skills/PR_WORKFLOW.md +++ b/.agents/skills/PR_WORKFLOW.md @@ -88,6 +88,7 @@ Is the code properly scoped? Is the code properly typed? Is the code hardened? Do we have enough tests? +Are tests using fake timers where relevant? (e.g., debounce/throttle, retry backoff, timeout branches, delayed callbacks, polling loops) Do not add performative tests, ensure tests are real and there are no regressions. Take your time, fix it properly, refactor if necessary. Do you see any follow-up refactors we should do? diff --git a/.agents/skills/merge-pr/SKILL.md b/.agents/skills/merge-pr/SKILL.md index 73b851edc3..4d8f01f4cf 100644 --- a/.agents/skills/merge-pr/SKILL.md +++ b/.agents/skills/merge-pr/SKILL.md @@ -49,7 +49,7 @@ Create a checklist of all merge steps, print it, then continue and execute the c Use an isolated worktree for all merge work. ```sh -cd ~/Development/openclaw +cd ~/dev/openclaw # Sanity: confirm you are in the repo git rev-parse --show-toplevel @@ -167,7 +167,7 @@ gh pr view --json state --jq .state Run cleanup only if step 6 returned `MERGED`. ```sh -cd ~/Development/openclaw +cd ~/dev/openclaw git worktree remove ".worktrees/pr-" --force diff --git a/.agents/skills/review-pr/SKILL.md b/.agents/skills/review-pr/SKILL.md index 2095f7f891..4bcd76333b 100644 --- a/.agents/skills/review-pr/SKILL.md +++ b/.agents/skills/review-pr/SKILL.md @@ -51,7 +51,7 @@ Create a checklist of all review steps, print it, then continue and execute the Use an isolated worktree for all review work. ```sh -cd ~/Development/openclaw +cd ~/dev/openclaw # Sanity: confirm you are in the repo git rev-parse --show-toplevel