green while never having run.
That is not hypothetical: the former `pydantic-ai-ui-security-review` shipped this way and its
agent never fired once, reporting false-green on every
21k2d agoReviewMountCreate a free account to mount
reimplement or filter above it. A reimplemented
policy layer that drifts is a security bug, not a style problem: the realtime approval-bypass was
exactly this.
- When behavior differs from
21k2d agoReviewMountCreate a free account to mount
pull request ITSELF is the participant, and is named.**
GitHub CodeQL filing a security fix, Copilot Autofix, Dependabot raising a
dependency bump: nobody wrote those, so crediting a human would
21k36d agoReviewMountCreate a free account to mount
AGENTS.md
## Review Guidelines
- Focus only on critical security, reliability, performance, and scalability issues.
- Ignore style, performance micro-optimizations, and readability nits unless they are tied to a significant failure
### What
21k4mo agoReviewMountCreate a free account to mount
unavailable after one retry, or when satisfying the rule is technically impossible without increasing security or data-loss risk. The repository owner is the sole approver. Before proceeding, record
21k8mo agoReviewMountCreate a free account to mount
user-supplied URL goes through `safeFetch` / the SSRF helpers. This is a security boundary; do not bypass it.
- Server-side data access belongs in a controller under `apps/web/lib/api/controllers/`, client-side
Steady22 mentions in 2 weeks
No other project uses it yet
20k22mo agoReviewMountCreate a free account to mount
Repository
## Security reviews
For reviews of this repository's GitHub automation, CI, and release process, use the [GitHub repository threat model](agents/references/repository-threat-model.md). The `ruff` submodule pins ty's Rust source
20k32d agoReviewMountCreate a free account to mount
assumptions by writing syscall
tests to verify Linux behavior from a black-box perspective.
#### Security first
All changes are reviewed for security. gVisor's design and architecture ensures
that
20k6d agoReviewMountCreate a free account to mount
change guidelines
- Treat `Sandboxie/core` with extreme caution: bugs can cause BSODs or sandbox-escape security holes. Prefer minimal, surgical patches there over refactors.
- Verify builds, branches, and properties
20k2y agoReviewMountCreate a free account to mount
variables before plugin setup.
- LiteLLM and OpenAI pins must remain compatible. Retain shared dependency security floors and verify upgrades with installed-SDK HTTP tests in `tests/test_litellm_sdk.py` using the framework runtime
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
changing handler behavior.
- For auth, CSRF, upload/download, tunnel, or file endpoints, run the nearest security regression tests.
- Check file-level documentation coverage with a script or shell loop that verifies
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
repeatedly when the lifecycle point can fire multiple times.
- Secret masking, auth, security, and persistence extensions must not be bypassed by convenience changes.
## Work Guidance
- Keep extension code small
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
handlers.
- Keep local-only work separate from community publication and catalog discovery separate from security scanning.
- Preserve authentication, scope, user authorization, and unrelated runtime state.
## Work Guidance
- Prefer existing framework
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
behavior, scan expectations, and activation semantics current.
- Distinguish Index discovery from model-based security scans and installation; metadata is not verified capability or safety.
- Handle missing/null Index fields and check
20k8mo agoReviewMountCreate a free account to mount
handlers.
- Keep local-only work separate from community publication and catalog discovery separate from security scanning.
- Preserve authentication, scope, user authorization, and unrelated runtime state.
## Work Guidance
- Prefer existing framework
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
Tests DOX
## Purpose
- Own pytest regression, security, integration, and contract tests.
- Keep tests focused on behavior that should remain stable across framework changes.
## Ownership
- Test files live directly under `tests
Steady2 mentions in 2 weeks
No other project uses it yet
20k8mo agoReviewMountCreate a free account to mount
desktop/mobile inspection.
- Submit changes through a PR against `develop`. Report vulnerabilities
privately through GitHub Security Advisories. The community plugin registry
is retired; first-party catalog data lives in `packages/core/src/catalog/`.
Getting
No other project uses it yet
20k40d agoReviewMountCreate a free account to mount