Use latest SDK tooling for regeneration - #10
Merged
Conversation
|
✅ Clean PR, @calvin-archastro! No blocking findings on
Reply |
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 Changed
This keeps the Python SDK regeneration automation on the latest published ArchAstro tooling packages instead of pinning stale package specs in
package.json.@archastro/sdk-generatornow uses the npmlatestdist-tag inpackage.jsonand resolves to0.3.2inpackage-lock.json.@archastro/channel-harnessnow uses the npmlatestdist-tag inpackage.jsonand resolves to0.1.2inpackage-lock.json.Regenerate SDKworkflow now refreshes both packages fromlatest, then commitspackage.jsonandpackage-lock.jsoninto the generated PR with the SDK output.No generated SDK/spec files are included in this PR.
Diagrams
This is a CI config and dependency-lock update only. It does not add or change runtime/control flow in the SDK and does not add or restructure application types, so there is no meaningful sequence or class diagram to render.
Scope
Tooling and CI only.
Risk Assessment
Low to medium. The change is limited to regeneration tooling, but future manual regeneration runs will intentionally resolve the latest published ArchAstro generator and harness before creating the SDK update PR. The generated PR still records the exact resolved tarballs and integrity hashes in
package-lock.jsonfor review and repeatable CI installs.User Impact
No direct SDK runtime impact in this PR. The next regeneration run will use the latest generator package by default, which is needed for the Python keyword-handling fixes published in
@archastro/sdk-generator@0.3.2.Testing
npm ci --ignore-scripts --registry=https://registry.npmjs.org --@archastro:registry=https://registry.npmjs.orgnpm audit --audit-level=moderatenode_modules:@archastro/sdk-generator@0.3.2@archastro/channel-harness@0.1.2YAML.load_filegit diff --check HEAD~1 HEADpassedFollow-ups and Known Issues
Run the manual
Regenerate SDKworkflow after this merges to generate the actual SDK/spec changes with@archastro/sdk-generator@0.3.2.