Emit model-alias deltas in AWF config and merge builtins at runtime#33875
Closed
Copilot wants to merge 4 commits into
Closed
Emit model-alias deltas in AWF config and merge builtins at runtime#33875Copilot wants to merge 4 commits into
Copilot wants to merge 4 commits into
Conversation
Co-authored-by: pelikhan <[email protected]>
Co-authored-by: pelikhan <[email protected]>
Co-authored-by: pelikhan <[email protected]>
Copilot
AI
changed the title
Store model alias mapping as runtime delta and merge at setup time
Emit model-alias deltas in AWF config and merge builtins at runtime
May 21, 2026
Copilot created this pull request from a session on behalf of
pelikhan
May 21, 2026 22:15
View session
Collaborator
|
@copilot do not add a step, merge on demand from the env var |
Co-authored-by: pelikhan <[email protected]>
Contributor
Author
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.
Compiled workflows were embedding the full builtin model alias map into
awf-config.json, inflating lock output and coupling runtime behavior to heavyweight serialized config. This change stores only workflow/import alias deltas and reconstructs the effective mapping at runtime by merging against the bundled base alias file.Compiler output: serialize delta, not full map
BuildModelAliasesDelta(...)inpkg/workflow/model_aliases.go.BuildAWFConfigJSONnow writesapiProxy.modelsonly when aliases differ from builtins (new keys or overrides).apiProxy.modelsentirely.Runtime reconstruction in setup action
actions/setup/js/merge_model_aliases.cjs..../actions/model_aliases.jsonawf-config.json -> apiProxy.modelsawf-config.jsonwith the merged alias map used by AWF.Setup wiring and bundled data
actions/setup/js/model_aliases.jsoninto setup assets.pkg/workflow/awf_helpers.go) to invoke merge helper before AWF starts.Behavioral contract updates