Skip to content

Commit 088b167

Browse files
committed
fix: add grep command to check 1 and clarify check 3A import exclusion
1 parent 539f708 commit 088b167

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

  • plugin/skills/pipedrive-review-marketplace-readiness

plugin/skills/pipedrive-review-marketplace-readiness/SKILL.md

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,14 @@ Collect all failures before reporting. Do not stop at the first failure.
3939

4040
Scan source files for `fetch(` or `axios` calls (including `axios.get`, `axios.post`, etc.) that include an Authorization header with a token value.
4141

42+
Run:
43+
44+
```bash
45+
grep -rn "Authorization\|Bearer\|access_token" src --include="*.ts" | grep -v "src/pipedrive/client.ts"
46+
```
47+
48+
Review each match to determine the token source (see below).
49+
4250
For each match:
4351

4452
1. Is it inside `src/pipedrive/client.ts`? → Skip. The SDK wrapper handles token refresh automatically via `onTokenUpdate`.
@@ -116,7 +124,7 @@ Run:
116124
grep -n "NextFunction" src/app.ts
117125
```
118126

119-
Look for a 4-argument middleware with the signature `(err: Error, _req: Request, res: Response, _next: NextFunction)`. If no such handler exists, this sub-check fails.
127+
This will match both the import line and any usage. Look specifically for a 4-argument middleware with the signature `(err: Error, _req: Request, res: Response, _next: NextFunction)` — ignore any lines that are just import statements. If no such handler exists (only the import line matches), this sub-check fails.
120128

121129
**Report if sub-check A failing:**
122130

0 commit comments

Comments
 (0)