Skip to content

Update Webex Dependency#111

Open
jeremywillans wants to merge 2 commits into
WebexCommunity:mainfrom
jeremywillans:update-webex
Open

Update Webex Dependency#111
jeremywillans wants to merge 2 commits into
WebexCommunity:mainfrom
jeremywillans:update-webex

Conversation

@jeremywillans

Copy link
Copy Markdown
Collaborator

🚨 Breaking Change: Requires Node 22 🚨

feat(breaking): node 22 mininium
feat: replace webex with webex-node
chore: update dependencies, overrides and tests

feat: replace webex with webex-node
chore: update dependencies, overrides and tests
@jeremywillans jeremywillans requested a review from jpjpjp June 14, 2026 15:45
@jeremywillans jeremywillans requested a review from adamweeks July 8, 2026 09:04

@jpjpjp jpjpjp left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hooray for @jeremywillans !

Comment thread lib/framework.js
var moment = require('moment');
var HttpsProxyAgent = require('https-proxy-agent');
var Webex = require('webex');
var Webex = require('webex-node');

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Interesting. Is there now a separate front-end SDK?

Comment thread package.json
{
"name": "webex-node-bot-framework",
"version": "2.5.1",
"version": "3.0.0",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Major milestone!

Comment thread package.json
"author": "JP Shipherd <jpbulk@gmail.com>",
"license": "MIT",
"engines": {
"node": ">=22",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Only 2 versions behind current. Not bad!

Comment thread README.md

### Node JS Bot Framework for Cisco Webex

> ⚠️ **Maintenance status:** This repository is maintained on a best-effort basis for security fixes, dependency and runtime updates, and low-risk compatibility work. Pull requests are still considered, but new feature development is not planned.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Smart.

Comment thread storage/mongo.js
this.memStore = {};

// this.connectUrl = encodeURI(`mongodb+srv://${config.mongoUser}:${config.mongoPass}@${config.mongoConnectionStringSuffix}`);
// this.connectUrl = encodeURI(`mongodb+srv://tropoBot:7M9rdmW6Goq1yAwg@cluster0-5rfnd.mongodb.net/new-cardSchool-dev?retryWrites=true&w=majority`);

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🫢

@adamweeks adamweeks left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Thanks @jeremywillans

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.

3 participants