Package Security & Test Guidelines
**Last Updated:** August 18, 2026
## Archive Extraction Security
- Always validate ZIP entry names with a safe join. Reject absolute paths, Windows drive or volume paths
Steady2 mentions in 2 weeks
No other project uses it yet
40k3mo agoReviewMountCreate a free account to mount
comment, as `Test_Chain_NoAllocations`, `Test_Combined_DoesNotAllocate` and `Test_Security_Do_BlocksDNSRebinding` do.
- Do not hand-write the plumbing that nearly every test repeats; `internal/apptest` has helpers
40k8d agoReviewMountCreate a free account to mount
Library Guidelines
`api/` is the public API library: the extension model, service contracts, and security abstractions consumed by the application and by plugins. Changes here affect downstream compatibility.
## Scope
- Source
No other project uses it yet
40k4mo agoReviewMountCreate a free account to mount
requirement.
- Before adding a dependency, verify current upstream maintenance, release activity, community/support health, and security advisories and patch responsiveness. Do not introduce abandoned or unmaintained packages. Prefer a small, readable
Steady3 mentions in 2 weeks
No other project uses it yet
40k4y agoReviewMountCreate a free account to mount
available linked issues, module documentation, and review discussion alongside the diff. Prioritize correctness, security, regressions, compatibility, and reproducible validation of changed behavior.
- Ground each finding in a concrete trigger, affected
40k2y agoReviewMountCreate a free account to mount
stable release rather than copying a stale
version.
**Rationale**: Fewer, well-vetted dependencies reduce security
vulnerabilities, supply-chain risk, and long-term maintenance cost.
**Known exclusions** (to be fixed):
- Most
38k3mo agoReviewMountCreate a free account to mount
breaks. In Markdown files in the repository, put each sentence on its own line.
## Security vulnerabilities
Do not create public issues for security vulnerabilities. Follow the process in [SECURITY.md](SECURITY.md
38k3d agoReviewMountCreate a free account to mount
names the concrete defect and the fix.
- Recommend blocking only for concrete correctness, security, or agreed-contract
defects with a realistic failure scenario: state the defect, how it fails
37k3d agoReviewMountCreate a free account to mount
headers. `_notice` is reserved — open-map
infrastructure is in place for deprecation / version_skew / security notices;
the field is omitted until a producer is wired in v0.8. `profile` echoes
37k52d agoReviewMountCreate a free account to mount
Open/Closed Principle: provide extension hooks rather than bloating core with endless configuration toggles.
### 3. Security & Scope Boundaries
- **No Sanitization in Core**: Marked's single responsibility is translating Markdown into HTML
37k6y agoReviewMountCreate a free account to mount
Preserve backwards compatibility unless the task explicitly authorizes a breaking change.
- Do not weaken security checks, path hardening, sanitization, or generated-skill scanning to make an experiment pass.
## Execution loop
37k5mo agoReviewMountCreate a free account to mount
that summary. Conventional title types choose the category; `changelog:added|changed|fixed|deprecated|removed|security` labels override it, and `skip-changelog` excludes a PR. Internal-only title types are excluded
No other project uses it yet
37k20d agoReviewMountCreate a free account to mount
lives in **[review-code review guide](./.agents/skills/review-code/references/REVIEW_GUIDE.md)** — how changes are accepted or rejected, the security-first / performance-second priorities, the security checklist and advisory regression themes, and the test/changeset/version-coverage expectations
37k12d agoReviewMountCreate a free account to mount
pnpr/`.
**Read [`../AGENTS.md`](../AGENTS.md) and [`../PHILOSOPHY.md`](../PHILOSOPHY.md) first.**
They cover the monorepo-wide philosophy: security, performance, efficient disk
usage, predictability, and feature consolidation. They also cover PR workflow,
signing agent
37k12d agoReviewMountCreate a free account to mount
server enables hot-reload for quick iteration
- Production builds are optimized and minified
### Security Notes
- No sensitive data or credentials should be committed
- Use environment variables for any API keys
Steady49 mentions in 2 weeks
No other project uses it yet
37k14mo agoReviewMountCreate a free account to mount
server enables hot-reload for quick iteration
- Production builds are optimized and minified
### Security Notes
- No sensitive data or credentials should be committed
- Use environment variables for any API keys
Steady49 mentions in 2 weeks
No other project uses it yet
37k14mo agoReviewMountCreate a free account to mount