hubot-nextbus 2.3.0 → 2.4.0

package/.github/dependabot.yml

@@ -0,0 +1,8 @@
+version: 2
+updates:
+  - package-ecosystem: "npm"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    ignore:
+      - dependency-name: "eslint*"

package/.github/workflows/npm-publish.yml

@@ -17,7 +17,7 @@ jobs:
 
       - uses: actions/setup-node@v6
         with:
-          node-version: "20"
+          node-version: "24"
           registry-url: https://registry.npmjs.org
 
       - name: Update npm to latest version

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "hubot-nextbus",
   "description": "Shows when the next transit vehicle will arrive at a particular stop.",
-  "version": "2.3.0",
+  "version": "2.4.0",
   "author": "Stephen Yeargin <stephen@yearg.in>",
   "homepage": "https://github.com/transitnownash/hubot-nextbus",
   "license": "MIT",
@@ -24,18 +24,16 @@
     "hubot": ">=3 || 0.0.0-development"
   },
   "devDependencies": {
-    "chai": "^4.3.10",
+    "@vitest/coverage-v8": "^4.1.6",
     "eslint": "^8.56.0",
     "eslint-config-airbnb-base": "^15.0.0",
     "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-n": "^16.5.0",
     "eslint-plugin-promise": "^6.1.1",
-    "hubot-test-helper": "^1.9.0",
-    "husky": "^8.0.3",
-    "mocha": "^10.2.0",
-    "nock": "^13.4.0",
-    "sinon": "^17.0.1",
-    "sinon-chai": "^3.7.0"
+    "hubot": "^14.1.0",
+    "husky": "^9.1.7",
+    "nock": "^14.0.17",
+    "vitest": "^4.1.6"
   },
   "main": "index.js",
   "scripts": {
@@ -46,7 +44,10 @@
     "ascii-table": "^0.0.9",
     "moment": "^2.29.4"
   },
+  "engines": {
+    "node": ">=22"
+  },
   "volta": {
-    "node": "18.19.0"
+    "node": "22.22.3"
   }
 }

package/script/test

@@ -3,4 +3,4 @@
 # bootstrap environment
 source script/bootstrap
 
-mocha "test/**/*.js" --reporter spec --exit
+vitest run --coverage

package/src/nextbus.js

@@ -27,11 +27,12 @@ module.exports = (robot) => {
     robot.logger.debug(url);
     robot.http(url)
       .get()((err, res, body) => {
-        const response = JSON.parse(body);
         if (err) {
-          msg.send(err);
+          const detail = err.errors ? err.errors.map((e) => e.message).join(', ') : err.message;
+          msg.send(`Error fetching NextBus data: ${detail || err}`);
           return;
         }
+        const response = JSON.parse(body);
         if (response.error) {
           msg.send(response.error);
           return;
@@ -84,6 +85,24 @@ module.exports = (robot) => {
     return alertLines.join('\n');
   };
 
+  const tripTableHeading = ['Time', 'Route', 'ETA'];
+
+  const formatSlackTripTable = (heading, rows) => ({
+    blocks: [
+      {
+        type: 'header',
+        text: { type: 'plain_text', text: heading, emoji: true },
+      },
+      {
+        type: 'table',
+        rows: [tripTableHeading, ...rows].map((row) => row.map((cell) => ({
+          type: 'raw_text',
+          text: String(cell ?? ''),
+        }))),
+      },
+    ],
+  });
+
   const queryStopById = (stopId, msg) => getAPIResponse('agencies.json', msg, (agencies) => {
     // Override timezone for moment() calls
     process.env.TZ = agencies.data[0].agency_timezone;
@@ -118,31 +137,31 @@ module.exports = (robot) => {
         return;
       }
 
-      const table = new AsciiTable();
-      nextTripsData.slice(0, 5).forEach((tripData) => {
+      const rows = nextTripsData.slice(0, 5).map((tripData) => {
         const tripTime = formatTripTimeAsMoment(tripData.stop_time.arrival_time);
         const realtimeStatus = getRealTimeStatus(tripData.stop_time);
         const hasVehicle = vehiclePositions
           && vehiclePositions.some((vp) => vp.trip && vp.trip.trip_id === tripData.trip.trip_gid);
         const busIndicator = hasVehicle ? ' 🚌' : '';
         const timeUntilText = realtimeStatus ? `${tripTime.fromNow()} (${realtimeStatus})` : tripTime.fromNow();
-        const columns = [
-          formatTripTimeAsMoment(tripData.stop_time.arrival_time).format('LT'),
+        return [
+          tripTime.format('LT'),
           `#${tripData.trip.route_gid} - ${tripData.trip.trip_headsign}${busIndicator}`,
           timeUntilText,
         ];
-        table.addRow(columns);
       });
       const adapterName = robot.adapterName ?? robot.adapter?.name ?? '';
-      table.removeBorder();
-      const tableOutput = table.toString().split('\n').map((line) => line.trimEnd()).join('\n');
-
-      const heading = `🚏 *${stop.stop_name}*`;
 
       if (/slack/i.test(adapterName)) {
-        msg.send(`${heading}\n\`\`\`\n${tableOutput}\n\`\`\``);
+        msg.send(formatSlackTripTable(`🚏 ${stop.stop_name}`, rows));
         return;
       }
+
+      const table = new AsciiTable();
+      rows.forEach((row) => table.addRow(row));
+      table.removeBorder();
+      const tableOutput = table.toString().split('\n').map((line) => line.trimEnd()).join('\n');
+      const heading = `🚏 *${stop.stop_name}*`;
       msg.send(`${heading}\n${tableOutput}`);
     });
   });
@@ -156,7 +175,7 @@ module.exports = (robot) => {
 
     getAPIResponse(`stops/near/${latlon}/1000.json?per_page=5`, msg, (stops) => {
       if (stops.total > 0) {
-        queryStopById(stops.data[0].stop_gid, msg);
+        queryStopById(stops.data[0].stop_code, msg);
         return;
       }
       msg.send(`No stops found near ${latlon}`);
@@ -165,16 +184,20 @@ module.exports = (robot) => {
 
   // get a list of nearby stops
   robot.respond(/(?:bus|nextbus) stops$/i, (msg) => getAPIResponse(`stops/near/${latlon}/1000.json?per_page=5`, msg, (stops) => {
+    if (stops.total === 0) {
+      msg.send(`No stops found near ${latlon}`);
+      return;
+    }
     msg.send('List of nearby stops:');
     const output = [];
     stops.data.forEach((stop) => {
-      output.push(`- [${stop.stop_gid}] ${stop.stop_name}`);
+      output.push(`- \`#${stop.stop_code}\` - ${stop.stop_name}`);
     });
     msg.send(output.join('\n'));
   }));
 
   // get a particular stop's next bus
-  robot.respond(/(?:bus|nextbus) stop ([A-Z0-9_]+)$/i, (msg) => {
+  robot.respond(/(?:bus|nextbus) stop #?([A-Z0-9_]+)$/i, (msg) => {
     const stopId = msg.match[1];
     robot.logger.debug(stopId);
     queryStopById(stopId, msg);

package/test/fixtures/agencies.json

@@ -4,7 +4,7 @@
   "per_page": 100,
   "data": [
     {
-      "id": 1,
+      "id": 178,
       "agency_gid": "WeGo Public Transit",
       "agency_name": "WeGo Public Transit",
       "agency_url": "http://www.wegotransit.com",
@@ -13,8 +13,8 @@
       "agency_phone": "615-862-8833",
       "agency_fare_url": null,
       "agency_email": null,
-      "created_at": "2022-08-02T21:14:03.545-05:00",
-      "updated_at": "2022-08-02T21:14:03.545-05:00"
+      "created_at": "2026-08-30T04:05:14.056-05:00",
+      "updated_at": "2026-08-30T04:05:14.056-05:00"
     }
   ]
 }