docs: correct reachability (Go ships) + close out C4 re-evaluation#502
Merged
Conversation
…re-evaluation Fixes an accuracy defect: analysis-types.md (Wave 7), comparison.md, and data-sources.md all claimed reachability is planned / not shipped / 'the backend does not run govulncheck'. It ships: tasks/scan_reachability.py runs govulncheck (worker-image binary) as a best-effort post-source-scan enrichment, default-on REACHABILITY_ENABLED, stamping Go findings reachable/not-reachable/not-analysed, surfaced by the badge + ?reachable= filter + sort=reachable, and able to feed the policy gate (fully tested). Corrected EN+KO to: Go ships; other ecosystems not yet analysed (the multi-language commercial gap). Also records the C4 re-evaluation. BomLens (now v1.8.2) classifies reachability as the portal's domain and won't produce it, so the 'BomLens produces -> TRUSCA ingests' premise is dead. TRUSCA already ships Go reachability, so C4 is reframed to TRUSCA-owned multi-language reachability and deferred (no offline OSS call-graph tooling for Java/JS). Master plan §6 + parity-review updated (baseline -> v1.8.2+25, 8th review). Version currency/staleness adopted as parity #18 (implementation next); regulatory-crosswalk and copyleft-strength recorded as low-value defers.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two things, folded together because the second surfaced the first.
1. Fixes a real accuracy defect: the docs understated shipped reachability
Several docs claimed reachability is "planned / not shipped / the backend does not run govulncheck." That is false. Go reachability ships and is fully wired + tested (
tasks/scan_reachability.pyrunsgovulncheck— built into the worker image — as a best-effort follow-up after every source scan, default-onREACHABILITY_ENABLED; it stamps each Go finding reachable / not-reachable / not-analysed, surfaced by the badge, the?reachable=filter, andsort=reachable, and can feed the policy gate; tests:test_scan_reachability,test_govulncheck,test_vulnerability_reachability,test_policy_gate_reachability,ReachabilityBadge.test).Corrected across
reference/analysis-types.md(my Wave 7 page — the most wrong),comparison.md, andreference/data-sources.md(EN + KO mirrors): reachability ships for Go; other ecosystems (Java/JS/Python/…) are not yet analysed — and that multi-language gap is the real commercial differentiator, stated as such.2. Records the C4 (multi-language reachability) re-evaluation
Investigating BomLens (now v1.8.2) for the C4 collaboration track found its premise is dead: BomLens explicitly classifies reachability as "the portal's domain" (
bomlens-internal/improvement-roadmap.md) and won't change its stateless-local-tool identity — so "BomLens produces call-graph → TRUSCA ingests" cannot happen. Since TRUSCA already ships Go reachability, C4 is reframed to "TRUSCA-owned multi-language reachability" and deferred (no offline OSS call-graph tooling exists for Java/JS — the commercial moat).docs/completeness-master-plan.md§6 anddocs/bomlens-parity-review.mdupdated (parity baseline bumped to v1.8.2+25, 8th review recorded).Also from the parity review: version currency/staleness (behind-latest-patch / releases-behind — distinct from EOL) is adopted as parity #18 and queued for implementation; the regulatory-crosswalk and copyleft-strength items are recorded as low-value defers.
Verification
docs-sitebuild green for both EN and KO locales; no broken links/anchors (#reachability-detailpreserved)CI note
Touches
docs-site/**(notpaths-ignored), so the code checks run and this merges normally; thedocs/**roadmap updates ride along.