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
71417d 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
69212d 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
6725d 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
60412mo agoReviewMountCreate a free account to mount
guidelines are defined in [`AGENTS.md`](../AGENTS.md).
Additional references:
- REST API design: https://wso2.com/whitepapers/wso2-rest-apis-design-guidelines/
- Secure coding: https://security.docs.wso2.com/en/latest/security-guidelines/secure-engineering-guidelines/secure-coding-guidlines/general-recommendations-for-secure-coding/
- Documentation guidelines: [`docs/AGENTS.md`](../docs/AGENTS.md
5986mo agoReviewMountCreate a free account to mount
Authentication code is in `pkg/sqlcmd/azure_auth.go`
- Supports multiple authentication methods: DefaultAzureCredential, Password, Interactive, ManagedIdentity, ServicePrincipal
## Security Considerations
- Never commit secrets or credentials
- Use environment variables or secure credential stores for sensitive
5955mo agoReviewMountCreate a free account to mount
pickles4evaaaaDeletes or force-pushesCopilot instructions
memory registry then query `MATCH (j:ImportJob {id: ' '}) RETURN j` for persisted record.
### 9. Security & Multi-User
* Isolation via relationships—never return a Book unless verified via an owning `OWNS
59512mo agoReviewMountCreate a free account to mount