Skip to content

feat: add bundle-release notification inputs to slack-notify-webhook#84

Open
originalix wants to merge 2 commits intoOneKeyHQ:mainfrom
originalix:feat/bundle-release-notification
Open

feat: add bundle-release notification inputs to slack-notify-webhook#84
originalix wants to merge 2 commits intoOneKeyHQ:mainfrom
originalix:feat/bundle-release-notification

Conversation

@originalix
Copy link
Contributor

Summary

  • Add 3 optional inputs to slack-notify-webhook action: artifact-bundle-version, artifact-skip-gpg, artifact-build-source
  • These fields are passed through to the eas-notice webhook payload so Slack notifications can differentiate between daily-build and bundle-release workflows
  • All new inputs are optional with sensible defaults, fully backward compatible

Test plan

  • Verify npm run build produces correct dist/index.js
  • Verify existing workflows calling this action are unaffected (new inputs are optional)
  • After deploying eas-notice changes, trigger a bundle-release workflow and verify Slack notification includes bundle version and skip GPG info

Add 3 optional inputs to support differentiated Slack notifications
for bundle-release vs daily-build workflows.
@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedaxios@​1.6.7986010096100
Added@​vercel/​ncc@​0.34.09910010082100
Added@​actions/​core@​1.10.19910010090100
Addedneedle@​2.9.19910010090100

View full report

@revan-zhang
Copy link
Collaborator

revan-zhang commented Mar 16, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

2 participants