diff --git a/.agents/skills/PR_WORKFLOW.md b/.agents/skills/PR_WORKFLOW.md index 0af4d3231..f75414181 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 73b851edc..4d8f01f4c 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 2095f7f89..4bcd76333 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