Merge pull request #194 from stephenyeargin/feature/responder-send-error

feature: add sendError to Responder

.devcontainer/.profile

@@ -0,0 +1,4 @@
+# trust the repo
+# fixes:
+# - fatal: detected dubious ownership in repository at '/workspaces/bot-zero'.
+git config --global --add safe.directory "$PWD"

.devcontainer/Dockerfile

@@ -1,8 +1,15 @@
 FROM mcr.microsoft.com/devcontainers/typescript-node:1-20-bullseye
 
 # Install the npm packages globally
-RUN npm install -g npm@10.7.0 \
+RUN npm install -g npm@10.9.2 \
     && npm install -g npm-check-updates
 
 COPY ./startup.sh /
 RUN chmod +x /startup.sh
+
+# Append the profile to the current .bashrc and .zshrc files
+# this makes sure we keep the current behavior like colors and aliases
+COPY ./.profile /tmp/.profile
+RUN cat /tmp/.profile >> /home/node/.bashrc && \
+    cat /tmp/.profile >> /home/node/.zshrc && \
+    rm /tmp/.profile

.devcontainer/startup.sh

@@ -1,10 +1,5 @@
 #!/bin/sh
 
-# trust the repo
-# fixes:
-# - fatal: detected dubious ownership in repository at '/workspaces/bot-zero'.
-git config --global --add safe.directory "$PWD"
-
 # install NPM packages
 echo ""
 echo "Installing packages..."

src/Bot.js

@@ -125,7 +125,7 @@ class Bot {
    */
   sendError(message, context) {
     context.robot.logger.error(message);
-    context.send(message);
+    this.adapter.responder.sendError(context, message);
   }
 
   /**

src/adapters/Responder.js

@@ -10,6 +10,15 @@ class Responder {
   send(res, title, image, link) {
     res.send(`${title}: ${image} - ${link}`);
   }
+
+  /**
+   * Sends the error message to Hubot.
+   * @param {Hubot.Response} res the context.
+   * @param {string} message the error message.
+   */
+  sendError(res, message) {
+    res.send(message);
+  }
 }
 
 exports.Responder = Responder;