Conversation
…ur ALMs The merge gate compared the merged branch to a single name with exact equality, so the guides could only describe two setups: one branch, or every branch. It is now a regular expression resolved at two levels, which the "Merge target branch" section had no way to express. That section is replaced with "AST branch pattern" in the GitHub, GitLab, Bitbucket and Azure DevOps guides: where each level is configured, the resolution order (repository -> integration -> orchestrator ref as a legacy fallback -> every branch), how a pattern is matched and validated, worked examples, and the branch picker now on the on-demand Run AST dialog. Anchoring and the fail-closed behaviour are stated explicitly because both are observable: `main` does not match `maintenance`, and a pattern that errors or times out while a merge is evaluated skips the scan rather than dispatching it. Values already stored keep their exact meaning, so the guides say plainly that there is nothing to migrate. Ref keeps its own entry and a note saying it is the orchestrator's branch, not a branch policy -- it stays the filter only when nothing else is set. Also updates the configuration step, validation checklist, troubleshooting table and frontmatter of each guide, and the AST row of the GitLab repositories validation table. Backend: convisoappsec/platform-backend#14364 Frontend: convisoappsec/platform-frontend#3094 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Conviso Security Gate — No Security Gate rule configuredNo No new findings were introduced by this pull request. |
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
The merge gate that dispatches the AST orchestrator used to compare the merged PR's base branch to a single name, with exact equality. Only two setups were expressible: one branch, or every branch. It is now a regular expression — the AST branch pattern — resolved at two levels.
The "Merge target branch" section had no way to describe that, so it is replaced by an "AST branch pattern" section in all four ALM orchestrator guides: GitHub, GitLab, Bitbucket and Azure DevOps.
What each guide now documents
maindoes not matchmaintenance;|lists branches; a plain branch name behaves exactly as before.main,main|develop,release/.*,main|release/.*.Anchoring and fail-closed are stated explicitly because both are observable behaviour, not implementation detail.
Ref keeps its own entry plus a note saying it is the orchestrator's branch — where
ast.ymllives — and not a branch policy. It remains the filter only when nothing else is set.Migration
None. Every value already stored keeps its exact meaning, and the guides say so plainly.
Also updated
Configuration step, validation checklist, troubleshooting table and frontmatter (description + keywords) of each of the four guides, and the AST row of the GitLab repositories validation table.
Related
🤖 Generated with Claude Code