question weird existing code to make new code high quality, as in:
- correct
- secure
- performant
- readable
- easy to maintain/change
- modern
In descending order of importance
When adding new features
No other project uses it yet
63k13d agoReviewMountCreate a free account to mount
only that branch reads the window's full cookie store, with exact IP, Path, Secure and expiry filtering. Keep normal hostnames and other platforms on `cookies_for_url`. Validate
No other project uses it yet
63k54d agoReviewMountCreate a free account to mount
Skipping this step is the #1 cause of incorrect or incomplete work.
---
## Auto-triggers
- **Security advisory URL pasted** — When the user pastes a URL matching
`github.com/penpot/penpot/security/advisories/GHSA-*`, extract the GHSA
Steady5 mentions in 2 weeks
No other project uses it yet
62k2mo agoReviewMountCreate a free account to mount
correct project spelling is "Pi-hole" (capital P, lowercase h, hyphen).
## Security considerations
- The installer and most scripts run as root on end-user systems. Quote variables, validate all user
No other project uses it yet
62k34d agoReviewMountCreate a free account to mount
Feature). `gh issue create` does not apply templates automatically.
Maintainer-directed work, urgent security fixes, release automation, and local or exploratory changes do not require a Ready issue.
## MCP Server
1 project uses a version of this
56k4mo agoReviewMountCreate a free account to mount
reuse existing patterns.
- Before adding complexity for a speculative or vanishingly unlikely race or security edge case, explain the concrete failure mode, likelihood, and complexity cost to the user
No other project uses it yet
55k15d agoReviewMountCreate a free account to mount
tools/tv.py --probe`, `getprop` and backups, quote only the model, Android version and firmware build.
- **Security problems are reported privately**, as [SECURITY.md](SECURITY.md) describes, never in a public issue.
## Commit
Steady21 mentions in 2 weeks
1 project uses a version of this
54k16d agoReviewMountCreate a free account to mount
invoked directly.
Next.js documentation explicitly states: "Treat Server Actions with the same security considerations as public-facing API endpoints, and verify if the user is allowed to perform a mutation
7 projects use a version of this
53k4d agoReviewMountCreate a free account to mount
loudly and get a human sign-off before breaking it.
## Additional tips
- Security is important, but should not be over-indexed on, especially for dev mode/maintainer-only features
No other project uses it yet
52k9d agoReviewMountCreate a free account to mount
domain concept rather than a mechanical step extracted only to shorten the caller.
### Authentication Security (OWASP Mandatory)
- Any implementation, modification, or review involving authentication-related flows MUST comply with
No other project uses it yet
52k56d agoReviewMountCreate a free account to mount
could not be inspected, say so when reporting the PR.
## Code Review Rules
For security reviews of Ruff and ty runtime changes, use the
[threat models](.agents/references/threat-models.md) to assess trust
No other project uses it yet
51k45d agoReviewMountCreate a free account to mount
during a local (non-CI) run, so `yarn test web_security --browser chrome` deletes the Firefox- and Electron-gated keys from `__snapshots__/web_security_spec.js`, and CI then fails those tests with
51k35d agoReviewMountCreate a free account to mount
Working on nanobot
The Python gateway owns agent execution, sessions, tools, memory, and security policy. WebUI and TUI share that runtime; keep execution and policy out of the clients.
## Task
No other project uses it yet
50k38d agoReviewMountCreate a free account to mount
senior Cal.diy engineer working in a Yarn/Turbo monorepo. You prioritize type safety, security, and small, reviewable diffs.
## Do
- Use `select` instead of `include` in Prisma queries for performance and security
1 project uses a version of this
50k17d agoReviewMountCreate a free account to mount
src/utils/http/axios/` wraps Axios with:
- Token injection through interceptors.
- MD5 request signing for API security.
- Multi-tenant header support.
- Standard response unwrapping through the `result` field.
API functions usually follow this
No other project uses it yet
48k13mo agoReviewMountCreate a free account to mount
imports, or execution flow.** Graph first; text search only for empty/`UNKNOWN`/literals.
- For security review, `explain({target: "fileOrSymbol"})` lists taint findings (source→sink flows; needs `analyze --pdg`).
- For control/data
Steady29 mentions in 2 weeks
No other project uses it yet
48k2mo agoReviewMountCreate a free account to mount