Docs/ast v2 update - #634
Merged
Merged
Conversation
douglaskurotaki
approved these changes
Aug 24, 2026
Conviso Security Gate — No Security Gate rule configuredNo 6 new findings introduced by this pull request: 🔴 6 high New findings
|
FelipePLima
reviewed
Sep 16, 2026
FelipePLima
approved these changes
Sep 16, 2026
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.
What this changes
Brings the Conviso AST documentation in line with the CLI that ships today, across the
product pages and every CI/CD integration guide. No page describes a "v2": the product is
Conviso AST, and the engines are
conviso sast,conviso sca,conviso iac,conviso sbom,conviso secretandconviso container.Command surface
conviso ast runis the documented entry point everywhere, replacing the bareconviso-astinvocation in the four orchestrator guides and their flow diagrams.--vulnerability-auto-closeis gone from every example. Closing fixed vulnerabilities isthe default behavior, and the flag is accepted only as a no-op — keeping it in the docs
taught people to pass something that does nothing.
--start-commit/--end-commitgive way toBASELINE_REF/BASELINE_COMMIT, which ishow a diff-aware scan is scoped now. The GitLab example also sets
GIT_DEPTH: "0",without which the baseline commit is not in the clone.
pages still promised "Code Review" and stopped at SAST + SCA.
Environment and image
CONVISO_APIKEY/CONVISO_BASE_URLbecomeCONVISO_API_KEY/CONVISO_API_URLin theorchestrator templates.
CONVISO_BASE_URLwas never read by anything — those pipelineswere falling back to the default API URL and the docs made that invisible.
convisoappsec/convisoast. This depends on the release side: the deployworkflow currently publishes
convisoappsec/convisoast_v2only, with the second namecommented out. Merging before that is restored points readers at a stale image.
Pages reworked
detection, environment variables, dry-run mode and troubleshooting.
on findings), that it evaluates one branch, that it never creates an asset, and that
conviso vulnerability assert-security-rulesis deprecated. The YAML-based section nolonger documents a legacy output format that the CLI stopped producing.
the asset from the remote, so a webhook from a new repository works with no pipeline edit.
from
conviso ast runorconviso sbom generate(not fromconviso sca run), thecontainer prerequisite is the Conviso AST image, and auto-close needs no flag.
--asset-nameis documented as being discontinued--asset-name/CONVISO_ASSET_NAMEnow carries a deprecation warning on the AST page, arow in the scan options table and a row in the environment variables table, all pointing at
--asset-id/CONVISO_ASSET_ID. The reasoning is in the page: a name is not an identity —two assets in the same company can share one, and a name that stops resolving sends the scan
to an asset nobody chose.
Validation
The last commit fixes what did not survive a check against the CLI source (conviso-ast-v2
4.0.0):
--branch-name.-b/--branchis not a spelling the parserknows: it is dropped with an "unknown argument" warning, and the scan silently reports to
whatever branch git resolves to. It was wrong in four places, including the security gate.
is stopped for being outdated. Both claims were replaced by what a pinned tag actually
costs.
CONVISO_BRANCH_NAMEwas undocumented despite outrankingCONVISO_BRANCHand every CIvariable.
yarn buildpasses, and the broken anchor this branch had introduced is gone. The eightanchors still reported are pre-existing on
mainand untouched here.