docs: document affected functions for vulnerable dependency findings OD-297 - #2716
Conversation
Adds an "Affected functions" section to Security and risk management findings, and a matching section to the Codacy Cloud CLI docs covering issues/issue/findings/finding/pull-request output. Related: OD-297
|
Overall readability score: 53.92 (🔴 -0.02)
View detailed metrics🟢 - Shows an increase in readability
Averages:
View metric targets
|
Up to standards ✅🟢 Issues
|
The placeholder screenshot referenced a file that doesn't exist yet, which mkdocs' strict mode treats as a build-breaking warning. Move the image tag inside the TODO comment so it's added back (uncommented) once the real screenshot lands.
There was a problem hiding this comment.
Pull Request Overview
The pull request successfully updates the documentation for the new 'affected functions' SCA feature across the web UI and CLI. While Codacy results are up to standards, there is a significant blocker: the screenshot for the web UI is currently a placeholder, which fails a key acceptance criterion.
Additionally, there is a gap in the CLI documentation regarding specific command examples for 'findings', 'finding', and 'pull-request'. The PR also documents UI changes that have not yet been released to production, and a 'Learn more' link in the application will require post-deployment validation once this documentation is live.
About this PR
- This PR documents features currently in development for the web application (codacy-spa). Ensure the documentation release is synchronized with the application deployment to avoid user confusion.
- Note that the 'Learn more' link in the web UI currently points to an empty URL; this must be verified once these docs are published.
Test suggestions
- Found recommended test scenario: Documentation reflects the 'Advisory Information' tab rename
- Found recommended test scenario: CLI documentation provides examples for both list (summary) and detail (full block) views
- Found recommended test scenario: Documentation explains how to use the AI prompt for reachability checks
- Missing recommended test scenario: CLI documentation includes specific command examples for 'findings', 'finding', and 'pull-request'
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Missing recommended test scenario: CLI documentation includes specific command examples for 'findings', 'finding', and 'pull-request'
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Points to the codacy-cloud-cli skill instead of raw CLI/jq commands, so the workflow stays in sync with the skill's own instructions rather than duplicating command syntax in two places.
…into docs-affected-functions-od-297
…ndings issues/issue/findings/finding now surface the affected functions of a linked advisory (CVE or GHSA), where known. Document the compact and detail-view formats and add a workflow for checking reachability before upgrading a dependency or ignoring a finding as NotExploitable. Related: OD-296, OD-297 (codacy/docs#2716)
Summary
Documents the new "affected functions" capability shipping under the Expose Advisory entrypoint function project (OD-297): for vulnerable dependency findings linked to an advisory (CVE or GHSA) with known affected functions, Codacy now shows exactly which functions are affected and lets you generate a coding-agent prompt to check reachability in your own code.
issues/issue/findings/finding/pull-request.Notes for reviewers
tempAffectedFunctionsflag and an in-app announcement tooltip (OD-315) hasn't shipped yet.images/security-risk-management-affected-functions.png) with a TODO comment inmanaging-security-and-risk.md— needs a real screenshot before merging.articleUrl=""inAffectedFunctionsModal.tsx, codacy-spa#3071). Once this page is live, it should point toorganizations/managing-security-and-risk/#affected-functions.Test plan
mkdocs buildlocally to confirm no broken links/build warnings