Skip to content

feat(config): add optional versionId support across config iac - #462

Open
jcbsfilho wants to merge 4 commits into
mainfrom
feat/version-id-support
Open

feat(config): add optional versionId support across config iac#462
jcbsfilho wants to merge 4 commits into
mainfrom
feat/version-id-support

Conversation

@jcbsfilho

Copy link
Copy Markdown
Collaborator

This pull request introduces optional versionId support throughout the config package, ensuring that the versionId field is consistently handled in both directions (config <-> manifest) for applications and other process-config strategies. It also adds robust test coverage for the new field and previously untested strategies, and fixes issues where versionId was missing or misnamed in certain strategies.

- formatting-only change (no deploy required for this package),
unrelated to the versionId feature work.
- adds versionId (config) <-> version_id (manifest) round-trip handling to
application, connector, functions, firewall, network list, waf and
workload process-config strategies, fixing missing/broken mappings in
connector and network list; adds JSON schema support and full test
coverage for the new field and for the previously untested application,
cache, device groups, function instances and rules strategies.
@github-actions

Copy link
Copy Markdown
Contributor

SCC Complexity Analysis

  • Title: feat(config): add optional versionId support across config iac
  • Author: jcbsfilho
Metric Source Branch (feat/version-id-support) Target Branch (main) Difference
Complexity 2457 2452 5
Code 52391 50654 1737

@pkg-pr-new

pkg-pr-new Bot commented Jul 16, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@aziontech/builder@462
npm i https://pkg.pr.new/@aziontech/config@462
npm i https://pkg.pr.new/@aziontech/presets@462

commit: 92d8a60

@jcbsfilho
jcbsfilho marked this pull request as ready for review July 17, 2026 16:02
@jcbsfilho
jcbsfilho requested a review from a team as a code owner July 17, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants