Evandro Fonseca Junior
bb3fe21766
The vendored SKILL.md files were byte-identical to upstream, but the playbooks, templates and schemas they reference were left behind in the original repos, so a loading skill pointed at a file that does not exist anywhere in this repository. Vendored 1:1 into each skill directory (so they survive `cp -r skills/* ~/.hermes/skills/`): - plays/ for api-security-review, code-review-security, sca-audit, secrets-scan, web-security-review (OWASP/secure-agent-playbook) - templates/finding.md for the four OWASP skills that cite it - schemas/finding.schema.json for cve-triage, patch-prioritization, dependency-scanning (UnitOneAI/SecuritySkills) - docs/sarif-output.md, docs/fixer-policy.md and docs/normalized-json-output.md for dependency-scanning Path fixes, the only edit to vendored text: - the three UnitOneAI skills referenced ../../../schemas/ and ../../../docs/ which assumed upstream's skills/<category>/<skill>/ depth; this repo is flat, so the links now resolve inside the skill directory - safedeps told the agent to run <skill_dir>/../../check_deps.py while the script is vendored next to SKILL.md; corrected to <skill_dir>/check_deps.py and documented the script's exit codes (1 = finding, not a crash) - NOTICE.md records the new files, the path edits, and the remaining known gaps (hermaguard's kensei-pipeline references, the ASVS dataset citations, and hermaguard being behind upstream) Verified: every backticked path in every SKILL.md now resolves; check_deps.py runs from the installed location and returns live OSV.dev data ([email protected] -> 2 advisories); hermaguard's 46 tests still pass.
49 lines
1.9 KiB
Markdown
49 lines
1.9 KiB
Markdown
# Finding Template
|
|
|
|
Use this structure for all security findings produced by skills.
|
|
|
|
```markdown
|
|
### [SEVERITY] Title
|
|
|
|
- **ID**: NDC-YYYY-NNN (auto-incrementing)
|
|
- **CWE**: CWE-XXX (if applicable)
|
|
- **CVE**: CVE-YYYY-NNNNN (if applicable)
|
|
- **OpenCRE**: [CRE-ID](https://www.opencre.org/cre/CRE-ID) — requirement name (see data/opencre/)
|
|
- **OWASP Ref**: Top 10 A01, ASVS V4.1.1, MASVS-STORAGE-1, MASWE-0001, MASTG-TEST-0200, LLM01, etc.
|
|
- **Location**: file_path:line_number or component name
|
|
- **Impact**: What an attacker can achieve (1-2 sentences)
|
|
- **Evidence**: Code snippet, command output, or proof-of-concept
|
|
- **Remediation**: Specific fix with code example
|
|
- **Confidence**: HIGH | MEDIUM | LOW (how certain is this finding)
|
|
```
|
|
|
|
### Resolving OpenCRE Links
|
|
|
|
When you have a CWE, look up its OpenCRE mapping via the API:
|
|
|
|
```
|
|
GET https://www.opencre.org/rest/v1/standard/CWE/sectionid/{cwe-number}
|
|
```
|
|
|
|
This returns the CRE ID and linked standards (ASVS, WSTG, NIST 800-53, cheat sheets). Include the CRE link in findings to give readers one-click access to all related standards and remediation guidance.
|
|
|
|
Common mappings are pre-populated in `data/opencre/README.md`.
|
|
|
|
## Severity Definitions
|
|
|
|
| Severity | CVSS Range | Criteria |
|
|
|----------|-----------|----------|
|
|
| CRITICAL | 9.0-10.0 | Remote code execution, full system compromise, mass data breach |
|
|
| HIGH | 7.0-8.9 | Privilege escalation, significant data exposure, auth bypass |
|
|
| MEDIUM | 4.0-6.9 | Limited data exposure, requires user interaction or specific conditions |
|
|
| LOW | 0.1-3.9 | Information disclosure, minor misconfigurations |
|
|
| INFORMATIONAL | N/A | Best practice recommendations, defense-in-depth suggestions |
|
|
|
|
## Confidence Levels
|
|
|
|
| Level | Meaning |
|
|
|-------|---------|
|
|
| HIGH | Confirmed via code path analysis, PoC, or tool output |
|
|
| MEDIUM | Strong indicators but not fully confirmed — needs manual verification |
|
|
| LOW | Heuristic match or pattern-based — may be false positive |
|