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
69222d 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
68612mo 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
67215d 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
6684mo 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
6526mo 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
6348mo 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
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
6076mo 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
60613mo 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
59712mo 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
5956mo agoReviewMountCreate a free account to mount
enforced mechanically when practical
Then review the implementation for bugs, regressions, missing validation, security issues, and maintainability risks.
## Harness-Specific Checks
- For harness changes, verify alignment with `docs/agent/governance.md`, `docs/agent/testing-strategy.md`, `docs/agent/feature-complete-checklist.md
5836mo agoReviewMountCreate a free account to mount
assembly)
- Pure Go versions: ` _generic.go` with build tag `//go:build purego || !(amd64 || ...)`
## Security Patterns
### Constant-Time Operations
- **Padding removal**: All padding schemes in `padding/` provide `ConstantTimeUnpad()` to prevent timing attacks
5695mo agoReviewMountCreate a free account to mount