query without prepare
$wpdb->query("UPDATE {$table} SET status = 'active'");
// ✅ GOOD: Use prepare for security
$wpdb->query(
$wpdb->prepare(
"UPDATE {$table} SET status = %s",
'active'
)
);
```
### 3. Missing Nonce Verification
7672mo agoReviewMountCreate a free account to mount
code review, always check for these issues. Flag any violations
as high severity.
### Security (all xlet types)
- Check for command/shell injection opportunities: Functions that take an argument
array variant should
7654mo agoReviewMountCreate a free account to mount
sacrifice grammar for brevity. Every comment MUST include an importance level prefix:
- **[CRITICAL]** — Bugs, security issues, data loss. Must fix.
- **[HIGH]** — Correctness, perf, or maintainability problems. Should fix.
- **[NIT]** — Style
7343d agoReviewMountCreate a free account to mount
Never minimal about:** input validation at trust boundaries, error handling
that prevents data loss, security, accessibility, anything explicitly requested.
Always on once installed. Deactivate: "stop chisle" / "normal mode
71416d agoReviewMountCreate a free account to mount
server/auth.ts`.
5. **Input Validation**: Validate requests with express-validator middleware in `server/middleware.ts`.
6. **Security**: Use `safeFetch()` from `server/ssrf.ts` for any outbound HTTP requests to prevent SSRF.
7. **Environment Variables
7087mo agoReviewMountCreate a free account to mount
asset must reside inside one of the eight categories under `assets/`:
- `m365-and-copilot`
- `security-and-governance`
- `landing-zones-and-infra`
- `agent-runtime`
- `ai-gateway`
- `orchestration-and-workflows`
- `agentops
69211d agoReviewMountCreate a free account to mount
types, or changed `ZenkoVersion` CR shape must be called out — they break deployed operators. |
| **Security** | OWASP-relevant issues for web/API code in tests; overly permissive RBAC, hostPath/hostNetwork usage, privileged containers
6723mo agoReviewMountCreate a free account to mount
fast development iteration
- Memory management optimizations for large codebase
- Plugin federation for modular architecture
## Security and Compliance
### Security Measures
- Regular dependency updates via Renovate
- Snyk security scanning and vulnerability auditing
66712mo agoReviewMountCreate a free account to mount
Keep fixes narrow and reviewable.
- Do not modify `.github/**`, CI, release, secret, funding, or security configuration unless the task is specifically about those files.
- Do not install new dependencies unless
6664mo agoReviewMountCreate a free account to mount
tests are added for new features
- [ ] Documentation is updated for API changes
- [ ] No security vulnerabilities introduced
- [ ] Performance considerations are addressed
- [ ] Error handling is comprehensive
- [ ] Code is readable and maintainable
6516mo agoReviewMountCreate a free account to mount
interface — do not add direct imports from `webviewProvider.ts`. This prevents circular dependencies.
## Security Best Practices
Follow OWASP Top 10 principles. Specific patterns enforced in this codebase:
- **Timing-safe comparison
6347mo agoReviewMountCreate a free account to mount
IronCurtain Copilot Review Instructions
IronCurtain is a secure agent runtime that mediates between an AI agent and MCP servers. Every tool call passes through a trusted process that evaluates
6153mo agoReviewMountCreate a free account to mount
bullets. explain/teach -- only when user says "why" or "explain".
Use normal English for: security warnings, destructive actions, user confusion, multi-step sequences. Resume terse after.
## Memory routing
Session start: call
6065mo agoReviewMountCreate a free account to mount
Rule validation logic
- `tests/` - Test suite
- `docs/` - Sphinx documentation
## Important Considerations
- This is a security-focused library - code quality and correctness are critical
- Changes should maintain backward compatibility when possible
60112mo agoReviewMountCreate a free account to mount