Issue to PR Planner — independently scanned and version-tracked by SaferSkills.
SaferSkills independently audited Issue to PR Planner (Agent Skill) and scored it 100/100 (green). The audit ran 55 deterministic rules across Security, Supply Chain, Maintenance, Transparency, and Community; it found 0 high-severity and 0 lower-severity findings. The full rule-by-rule trace and per-finding evidence are below. Free, methodology-open.
Findings & checks · 0 flagged
Every scanned point with the score it earned and what moved between them.
First recorded scan — no prior version to compare against.
The primary manifest — the file an agent reads to learn what this artifact does.
This skill takes a GitHub issue (or any feature request / bug report) and turns it into a concrete, step-by-step implementation plan. It reads the codebase to identify the relevant files, understands the existing patterns, and produces an ordered list of changes with enough detail that a developer — or an agent executing the plan — can work through it without ambiguity.
Use this at the start of a feature or bug fix to align on exactly what needs to change before writing code, or to hand off implementation work to another developer or AI agent.
Copy this file to .agents/skills/issue-to-pr-planner/SKILL.md in your project root.
Then ask:
Provide the full issue text including the title, description, and any acceptance criteria or comments.
Add the instructions below to your .cursorrules or paste them into the Cursor AI pane. Paste the issue text before asking for the plan.
Paste the issue text and provide the directory structure or key file list. Ask Codex to follow the instructions below.
When asked to plan an implementation from an issue, follow these steps:
Read the issue thoroughly. Identify:
If the issue is ambiguous, note the ambiguity and state the assumption you're making to proceed.
Read the relevant parts of the codebase:
Note: for large codebases, focus on the most relevant 3–5 files. Don't read everything.
List every change needed:
For each change, note the specific function, class, or section that needs to change — not just the file.
Order the changes from first to last. The order should follow a natural dependency graph — create types before using them, create database migrations before writing queries, write tests before or alongside implementation.
Format each step as:
### Step N — [Action verb] [what]
**File**: `path/to/file.ts`
**Change**: [Specific description of what to add, modify, or remove]
**Why**: [Why this step is needed]
**Notes**: [Any gotchas, patterns to follow, or decisions the implementer needs to make]If there are design decisions the issue doesn't answer, list them. Don't block the plan — make a recommendation and flag that it was a judgment call.
## Implementation Plan for: [Issue Title]
### Issue Type
[Bug / Feature / Enhancement / Refactor]
### Goal
[One sentence]
### Acceptance Criteria
- [ ] [Criterion 1]
- [ ] [Criterion 2]
...
### Files to change
- `path/to/file.ts` — [reason]
- `path/to/new-file.ts` — [create new]
---
### Implementation Steps
[Steps as described above]
---
### Open Questions / Decisions
1. [Question] — **Recommendation**: [your recommendation]
### Estimated Complexity
[Small (< 2hrs) / Medium (half day) / Large (1–2 days) / XL (needs to be broken into sub-issues)]Input to Agent:
"Use the Issue to PR Planner skill.
>
Issue #88: Add rate limiting to the login endpoint
>
Currently the POST /api/auth/login endpoint has no rate limiting. Attackers can brute-force passwords. We should limit login attempts to 5 per IP per 15 minutes. After 5 failed attempts, return a 429 with a message telling the user when they can retry."Output from Agent:
## Implementation Plan for: Add rate limiting to the login endpoint
>
### Issue Type Security enhancement
>
### Goal Prevent brute-force attacks on the login endpoint by limiting failed attempts per IP.
>
### Acceptance Criteria - [ ] After 5 failed login attempts from the same IP within 15 minutes, the endpoint returns 429 - [ ] The 429 response body includes a retryAfter field with seconds until the window resets - [ ] Successful logins do not count against the limit - [ ] Rate limit state resets after the 15-minute window>
### Files to change -server/middleware/rateLimit.ts— create new rate limiting middleware -server/routes/auth.ts— apply middleware to login route -server/routes/auth.test.ts— add tests for rate limiting behavior
>
---
>
### Implementation Steps
>
#### Step 1 — Install and configure a rate limiter File:package.jsonChange: Addexpress-rate-limitas a dependency (npm install express-rate-limit) Why: Provides a battle-tested sliding window implementation so we don't build this from scratch
>
#### Step 2 — Create the login rate limit middleware File:server/middleware/rateLimit.tsChange: Create aloginRateLimitmiddleware usingexpress-rate-limitconfigured for 5 attempts per 15 minutes, keyed by IP. Return a 429 with{ error: 'Too many attempts', retryAfter: N }on limit exceeded. Why: Separating the middleware into its own file keeps the route file clean and makes the limiter independently testable Notes: UseskipSuccessfulRequests: trueso only failed logins count. Check if a Redis store is needed for multi-instance deployments.
>
#### Step 3 — Apply middleware to the login route File:server/routes/auth.tsChange: ImportloginRateLimitand add it as a middleware before the login handler:router.post('/login', loginRateLimit, loginHandler)Why: Applies the limit only to login, not all auth routes
>
#### Step 4 — Write tests File:server/routes/auth.test.tsChange: Add tests: (a) 5 failed attempts → 6th returns 429 with retryAfter, (b) successful login does not increment counter, (c) counter resets after window Notes: Usejest.useFakeTimers()to simulate window expiry without waiting 15 real minutes
>
### Open Questions 1. Single-server vs. multi-instance — In-memory rate limiting won't work correctly if there are multiple server instances. Recommendation: Use in-memory for now (single EC2), but note that a Redis store will be needed before horizontal scaling.
>
### Estimated Complexity Small (< 2 hours)
~30 seconds. Free. No account. Every finding cites a rule and a line of evidence.