Skip to content

README: state that dependency advisory/CVE coverage is out of scope - #43

Open
Kaushik2210 wants to merge 1 commit into
cloudflare:mainfrom
Kaushik2210:docs/state-dependency-advisory-scope
Open

Kaushik2210 wants to merge 1 commit into
cloudflare:mainfrom
Kaushik2210:docs/state-dependency-advisory-scope

Conversation

@Kaushik2210

Copy link
Copy Markdown

Summary

Addresses gap 1 of #20 (the other gaps are design calls and are not touched here). Adds a short "Scope and limits" section to the README.

The limit is real but was implicit: no file in the workflow mentions advisory or CVE data (grep -ri "advisor\|CVE" finds only the "known-vulnerable dependency is not a finding by itself" rule in SUPPLY-CHAIN-AND-RELEASE.md), and hunters run with no external networking. So a disclosed vulnerability in a pinned dependency that postdates the model's knowledge cannot be found, and someone evaluating the skill for that coverage only learns it from a 0% result.

Change

README only, two bullets:

  • Known vulnerabilities in pinned dependencies are out of scope, and a known-vulnerable dependency is not a finding by itself; use a dependency scanner alongside.
  • Facts the audit cannot observe stay needs_validation with the fact named (restating existing behavior from the design principles and ATTACK-CLASSES.md).

No behavior change. Happy to reword if the maintainers want a different framing.

🤖 Generated with Claude Code

Nothing in the workflow fetches advisory data and hunters run with no
external networking, so disclosed vulnerabilities in pinned dependencies
are not found. The limit was implicit; a user evaluating the skill for
dependency coverage only learned it from a 0% result. State it up front,
alongside the existing needs_validation behavior for unobservable facts.

Addresses gap 1 of cloudflare#20.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant