feat(ci): add Webex bot to comment SDK version and changelog on merged PRs by vamshigovardhana · Pull Request #4737 · webex/webex-js-sdk

vamshigovardhana · 2026-02-23T13:50:07Z

COMPLETES #https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-761506

This pull request addresses

Adds a new GitHub Actions workflow to send release notifications to a Webex Space

Triggers after Deploy CD workflow completes successfully
Uses yarn package-tools list --recursive for accurate package detection
Posts SDK version, PR link, and changelog URL to the configured Webex Space

by making the following changes

Workflow triggers after Deploy CD completes on next branch
Extracts version and PR number from the latest git tag
Uses package-tools to detect affected packages (with recursive dependency detection)
Sends a formatted message to the Webex Space via Webex API

Change Type

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)
Documentation update
Tooling change
Internal code refactor

The following scenarios were tested

The GAI Coding Policy And Copyright Annotation Best Practices

I certified that

I have read and followed contributing guidelines
I discussed changes with code owners prior to submitting this pull request
I have not skipped any automated checks
All existing and new tests passed
I have updated the documentation accordingly

Make sure to have followed the contributing guidelines before submitting.

Sends release notifications to a Webex Space after Deploy CD completes. Uses yarn package-tools for recursive package detection. Co-authored-by: Cursor <cursoragent@cursor.com>

Shreyas281299

Have we tested this?
Do we have a new bot created for this? And which webex space are we using for this?

Shreyas281299 · 2026-03-02T04:20:13Z

.github/workflows/webex-space-notification.yml

+    inputs:
+      test_version:
+        description: 'Test version number (e.g., 3.10.0)'
+        required: false
+        default: '3.10.0'
+      test_pr_number:
+        description: 'Test PR number (e.g., 123)'
+        required: false
+        default: ''


I dont think this is required

@Shreyas281299 Yah I will remove it

I still see these changes

Shreyas281299 · 2026-03-02T04:21:54Z

.github/workflows/webex-space-notification.yml

+          WEBEX_BOT_TOKEN: ${{ secrets.WEBEX_BOT_TOKEN }}
+          WEBEX_ROOM_ID: ${{ secrets.WEBEX_ROOM_ID }}


Have we added these secrets?

@Shreyas281299 Yes I have added that secrets

vamshigovardhana · 2026-03-02T05:26:55Z

Have we tested this?

Do we have a new bot created for this? And which webex space are we using for this?

@Shreyas281299 Yes I have tested these locally and I have created new space for this in webex and added webex bot in to that space

Shreyas281299 · 2026-03-11T04:05:50Z

We are printing UUIDs in the logs please remove it

Shreyas281299 · 2026-03-11T04:06:21Z

.github/workflows/webex-space-notification.yml

+    inputs:
+      test_version:
+        description: 'Test version number (e.g., 3.10.0)'
+        required: false
+        default: '3.10.0'
+      test_pr_number:
+        description: 'Test PR number (e.g., 123)'
+        required: false
+        default: ''


I still see these changes

Shreyas281299 · 2026-03-11T04:13:15Z

.github/workflows/webex-space-notification.yml

+          # Calculate stable version (strip -next.X suffix)
+          STABLE_VERSION=$(echo "$VERSION_NUMBER" | sed 's/-next\..*//')
+
+          # Build changelog URL
+          CHANGELOG_URL="https://web-sdk.webex.com/changelog/?stable_version=${STABLE_VERSION}"
+
+          # Add package parameter
+          if [ -n "${PRIMARY_PACKAGE}" ]; then
+            ENCODED_PACKAGE=$(node -e "console.log(encodeURIComponent(process.argv[1]))" "${PRIMARY_PACKAGE}")
+            CHANGELOG_URL="${CHANGELOG_URL}&package=${ENCODED_PACKAGE}"
+          fi
+
+          # Add version parameter
+          if [ -n "${VERSION_NUMBER}" ]; then
+            CHANGELOG_URL="${CHANGELOG_URL}&version=${VERSION_NUMBER}"
+          fi
+
+          echo "🔗 Changelog URL: ${CHANGELOG_URL}"


Since we are changing how the URL is created this would again become an issue, and we would end up creating a wrong URL.

By this time our deploy pipeline would have commented the correct cahngelog link. we can fetch that comment from the PR and use it. What do you think?

@Shreyas281299 if i implement this logic "The Webex space notification workflow fetches the changelog URL from the PR comment posted by the comment-on-prs job in deploy.yml. But since both are triggered after Deploy CD completes, what if the Webex bot runs before the PR comment is posted? Should we add a retry/polling mechanism, or is there a better way to ensure the comment exists first?"

Shreyas281299 · 2026-03-11T04:14:52Z

Please post the github run that you did for testing here in the PR

vamshigovardhana · 2026-03-11T08:25:31Z

.github/workflows/webex-space-notification.yml

Git-hub run:https://github.com/vamshigovardhana/webex-js-sdk/actions/runs/22941175851/job/66582913230

vamshigovardhana requested a review from a team as a code owner February 23, 2026 13:50

vamshigovardhana force-pushed the webex-space-bot branch from 22af426 to e36b017 Compare February 23, 2026 14:15

vamshigovardhana closed this Feb 23, 2026

vamshigovardhana force-pushed the webex-space-bot branch from e36b017 to be161d6 Compare February 23, 2026 15:48

ci(bot): Add Webex Space release notification workflow

fc08e37

Sends release notifications to a Webex Space after Deploy CD completes. Uses yarn package-tools for recursive package detection. Co-authored-by: Cursor <cursoragent@cursor.com>

vamshigovardhana reopened this Feb 23, 2026

vamshigovardhana changed the title ~~ci(bot): Add Webex Space release notification workflow~~ feat(ci): add Webe bot to comment SDK version and changelog on merged PRs Feb 23, 2026

vamshigovardhana changed the title ~~feat(ci): add Webe bot to comment SDK version and changelog on merged PRs~~ feat(ci): add Webex bot to comment SDK version and changelog on merged PRs Feb 23, 2026

Shreyas281299 added the validated If the pull request is validated for automation. label Feb 24, 2026

Shreyas281299 reviewed Mar 2, 2026

View reviewed changes

Shreyas281299 requested changes Mar 11, 2026

View reviewed changes

vamshigovardhana added 4 commits March 11, 2026 12:15

ci: add webex space notification workflow

fb56677

ci: remove old workflow file after rename

dd9258f

ci: suppress curl response output to hide UUIDs from logs

027e4be

ci: remove workflow_dispatch inputs and suppress curl output

b2abd1e

vamshigovardhana commented Mar 11, 2026

View reviewed changes

		WEBEX_BOT_TOKEN: ${{ secrets.WEBEX_BOT_TOKEN }}
		WEBEX_ROOM_ID: ${{ secrets.WEBEX_ROOM_ID }}

Conversation

vamshigovardhana commented Feb 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

COMPLETES #https://jira-eng-gpk2.cisco.com/jira/browse/SPARK-761506

This pull request addresses

by making the following changes

Change Type

The following scenarios were tested

The GAI Coding Policy And Copyright Annotation Best Practices

I certified that

Uh oh!

Shreyas281299 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

vamshigovardhana commented Mar 2, 2026

Uh oh!

Shreyas281299 commented Mar 11, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Shreyas281299 commented Mar 11, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vamshigovardhana commented Feb 23, 2026 •

edited

Loading