@@ -38,2 +38,2 @@
-if (!doLogging) {
-	console.warn("WARN: No logging channel! No logs of any kind will be sent to slack!")
+// this is used instead of doLogging so typescript can coerce loggingChannel into just a string
+if (!loggingChannel) {

@@ -53,3 +58,2 @@
 
-const goalDate = new Date(!!process.env.DATE? process.env.DATE : "3027/01/01");
-if (Number.isNaN(goalDate.getDate())) throw new Error("DATE must be a valid date (e.g. yyyy/mm/dd)")
+const goalDate = new Date(process.env.DATE ? process.env.DATE : "3027/01/01");

@@ -140,0 +155,4 @@
+ * @param {Error} error the error object
+ */
+async function publishError(error) {
+	console.error("ERROR: ", error);

@@ -140,0 +156,4 @@
+ */
+async function publishError(error) {
+	console.error("ERROR: ", error);
+	if (error) console.error(error);

@@ -140,0 +160,4 @@
+	if (doLogging)
+		await publishMessage(
+			loggingChannel,
+			`ERROR: ${error.message}. Stack: ${error.stack}`,

@@ -198,0 +242,4 @@
+				new Error(
+					`Tried to post a duplicate '${emoji}' reaction to message ${ts} in ${channelId}`,
+				),
+			);

@@ -298,0 +372,4 @@
+		// this _should_ always pass, but the type checker doesn't know that
+		if (error instanceof Error) {
+			publishError(error);
+		}

@@ -352,1 +434,3 @@
-		publishError(String(error), error);
+		if (error instanceof Error) {
+			publishError(error);
+		}

@@ -370,1 +454,3 @@
-		publishError(String(error), error);
+		if (error instanceof Error) {
+			publishError(error);
+		}

@@ -499,1 +596,3 @@
-		publishError(String(error), error);
+		if (error instanceof Error) {
+			publishError(error);
+		}

@@ -32,3 +32,4 @@
 	const start =
 		regularStartQuotes[Math.floor(Math.random() * regularStartQuotes.length)];
 
+	/** @type {string | undefined} */