Skip to content

PLT-1616 : aws-params-env-action GHA to use node 20 and updated packages#428

Open
mianava wants to merge 20 commits intomainfrom
feature/GitHub-Actions-AWS-Params-Node-Upgrade-To-20
Open

PLT-1616 : aws-params-env-action GHA to use node 20 and updated packages#428
mianava wants to merge 20 commits intomainfrom
feature/GitHub-Actions-AWS-Params-Node-Upgrade-To-20

Conversation

@mianava
Copy link
Copy Markdown
Contributor

@mianava mianava commented Mar 27, 2026

🎫 Ticket

https://jira.cms.gov/browse/...

🛠 Changes

This PR modernizes the aws-params-env-action Action to address dependency security vulnerabilities, update deprecated tooling, and ensure compatibility with Node 20 and ARM64 runners.

ℹ️ Context

The existing action used aws-sdk v2 (via @aws-sdk/client-ssm pinned to an older version), which pulled in a vulnerable version of fast-xml-parser through its dependency chain.
@vercel/ncc was on an outdated version, and broke the dist file singularity.
ESLint was using the legacy .eslintrc.json config format, which is no longer supported in ESLint 9

The project needed to be verified as buildable on ARM64 runners

🧪 Validation

These changes are validated by the test run, and will be confirmed in a subscriber repo dedicated branch.

@mianava mianava marked this pull request as ready for review March 27, 2026 13:56
@mianava mianava requested a review from a team as a code owner March 27, 2026 13:56
Copy link
Copy Markdown
Contributor

@gfreeman-navapbc gfreeman-navapbc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You touched the aws-params-env-action! Congratulations on your 30 THOUSAND lines changed! 😂

jasonvinson
jasonvinson previously approved these changes Mar 27, 2026
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add description to the workflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants