remove throw error on channel message fail

.env.example

@@ -0,0 +1,28 @@
+# Discord Bot Configuration
+# -----------------------
+# Required: Your Discord bot token
+TOKEN=your_discord_bot_token_here
+
+# API Configuration
+# -----------------------
+# Required: Password for admin API access
+PASSWORD=your_secure_password_here
+
+# Web Server Settings
+# -----------------------
+# Port for the API server
+PORT=3000
+
+# Scheduling
+# -----------------------
+# CRON schedule for when to post (default: every day at 9 AM)
+CRON_LEGICA=0 9 * * *
+# Timezone for the CRON job (e.g. 'Europe/Zagreb', 'America/New_York', etc.)
+TIMEZONE=utc
+
+# Content Settings
+# -----------------------
+# Date format used in post titles
+LEGICA_DATE_FORMAT=D.M.YYYY
+# Number of hourly retry attempts if date check fails
+RETRY_ATTEMPTS=3

README.md

@@ -2,6 +2,27 @@
 
 Discord bot that scrapes the https://sib.net.hr/legica-dana website and posts the latest legica-dana post to all discord text channels it has permissions to.
 
+## Features
+
+- Automatically posts new content from the website daily
+- Built-in retry mechanism if the post isn't available yet
+- Adds rating reactions (1-10) to each post
+- REST API for controlling the bot
+
+## Configuration
+
+The bot can be configured using environment variables:
+
+| Variable | Description | Default |
+|----------|-------------|---------|
+| TOKEN | Discord Bot Token | Required |
+| PASSWORD | Admin password for API | Required |
+| PORT | Port for the API server | 3000 |
+| CRON_LEGICA | CRON schedule for posting | 0 9 * * * |
+| TIMEZONE | Timezone for the CRON job | utc |
+| LEGICA_DATE_FORMAT | Date format used in posts | D.M.YYYY |
+| RETRY_ATTEMPTS | Number of hourly retries if date check fails | 3 |
+
 ## Documentation
 
 [Documentation](https://legica.jurmanovic.com/swagger)

package.json

@@ -1,9 +1,11 @@
 {
 	"name": "legica-dana",
-	"version": "2.0.6",
+	"version": "2.1.1",
 	"main": "src/app.ts",
 	"scripts": {
-		"start": "bun run src/app.ts"
+		"start": "bun run src/app.ts",
+		"build-script": "bun build src/app.ts --outdir dist --target bun",
+		"start-build": "bun run dist/app.js"
 	},
 	"author": "Fran Jurmanović <fjurma12@outlook.com>",
 	"license": "MIT",
@@ -18,6 +20,7 @@
 		"discord.js": "^12.5.1",
 		"dotenv": "^8.2.0",
 		"elysia": "^0.7.15",
+		"ffmpeg-static": "^5.2.0",
 		"lodash-es": "^4.17.21",
 		"minimatch": "^9.0.3",
 		"pino": "^8.15.4",

process-env.d.ts

@@ -7,6 +7,7 @@ declare global {
 			PASSWORD: string;
 			TIMEZONE: string;
 			LEGICA_DATE_FORMAT: string;
+			RETRY_ATTEMPTS: string;
 		}
 	}
 }

src/app.ts

@@ -25,7 +25,18 @@ async function jobRunner() {
 	try {
 		await sendNextMessage(client);
 	} catch (err) {
-		logger.error(err);
+		// Log detailed error information, including retry attempts
+		if (err instanceof Error && err.message.includes("Failed after")) {
+			logger.error({
+				msg: "All retry attempts failed in job runner",
+				error: err.message,
+			});
+		} else {
+			logger.error({
+				msg: "Error in job runner, no retries attempted",
+				error: err,
+			});
+		}
 	}
 }
 const botPlugin = new Elysia({ prefix: "/bot" })
@@ -93,6 +104,14 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 			store: {
 				cron: { job },
 			},
+		}: {
+			store: {
+				cron: {
+					job: {
+						resume: () => void;
+					};
+				};
+			};
 		}) => {
 			client.on("ready", (): void => {
 				job.resume();
@@ -105,6 +124,17 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 				cron: { job },
 			},
 			set,
+		}: {
+			store: {
+				cron: {
+					job: {
+						isStopped: () => boolean;
+					};
+				};
+			};
+			set: {
+				status: number;
+			};
 		}) => {
 			if (job.isStopped()) {
 				set.status = 400;
@@ -129,6 +159,16 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 			store: {
 				cron: { job },
 			},
+		}: {
+			store: {
+				cron: {
+					job: {
+						isRunning: () => boolean;
+						isStopped: () => boolean;
+						nextRun: () => Date | null;
+					};
+				};
+			};
 		}) => ({
 			running: job.isRunning() ?? false,
 			stopped: job.isStopped() ?? false,
@@ -147,6 +187,18 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 				cron: { job },
 			},
 			set,
+		}: {
+			store: {
+				cron: {
+					job: {
+						isRunning: () => boolean;
+						resume: () => void;
+					};
+				};
+			};
+			set: {
+				status: number;
+			};
 		}) => {
 			if (job.isRunning()) {
 				set.status = 400;
@@ -168,6 +220,18 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 				cron: { job },
 			},
 			set,
+		}: {
+			store: {
+				cron: {
+					job: {
+						isRunning: () => boolean;
+						pause: () => void;
+					};
+				};
+			};
+			set: {
+				status: number;
+			};
 		}) => {
 			if (!job.isRunning()) {
 				set.status = 400;
@@ -184,7 +248,17 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 	)
 	.post(
 		"/send",
-		async ({ set, body }) => {
+		async ({
+			set,
+			body,
+		}: {
+			set: {
+				status: number;
+			};
+			body?: {
+				url?: string;
+			};
+		}) => {
 			try {
 				const url = body?.url;
 				if (url) {
@@ -193,9 +267,10 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 					await sendNextMessage(client);
 				}
 				return true;
-			} catch (err) {
+			} catch (err: unknown) {
 				set.status = 400;
-				return err;
+				logger.error(err);
+				return false;
 			}
 		},
 		{
@@ -216,7 +291,7 @@ const taskPlugin = new Elysia({ prefix: "/job" })
 	});
 const app = new Elysia()
 	.error({ BASIC_AUTH_ERROR: BasicAuthError })
-	.onError(({ error, code }) => {
+	.onError(({ error, code }: { error: Error; code: string }) => {
 		switch (code) {
 			case "BASIC_AUTH_ERROR":
 				return new Response(error.message, {

src/common/getFirstHtml.ts

@@ -2,10 +2,10 @@ import { config } from "@constants";
 import axios from "axios";
 import cheerio from "cheerio";
 
-export async function getFirstHtml(): Promise<string> {
+export async function getFirstHtml(): Promise<string | undefined> {
 	const response = await axios.get(config.LEGICA_URL);
 	const html = response.data;
 	const $ = cheerio.load(html);
-	const { href } = $(".News-link.c-def")?.attr() || {};
+	const href = $(".legica-dana").first().find("a").attr("href");
 	return href;
 }

src/common/getImgTitle.ts

@@ -7,8 +7,9 @@ export async function getImgTitle(href: string): Promise<Legica> {
 	const html = response.data;
 	const $ = cheerio.load(html);
 
-	const title = $(".Article-inner > h1").text();
+	const title = $(".article-title-container > h1").text();
-	const { src: img } = $(".Article-media > img").attr() || {};
+	const src = $(".image-holder", ".article-content").find("img").attr("src");
+	if (!src) throw new Error(`Image not found at ${href}.`);
 
-	return { title, img };
+	return { title, img: src };
 }

src/common/sendDiscordMessage.ts

@@ -6,12 +6,28 @@ import { Client, MessageEmbed, TextChannel } from "discord.js";
 
 dayjs.extend(customParseFormat);
 
+/**
+ * Retry mechanism for failed date checks
+ *
+ * This implementation allows the bot to retry fetching and posting content
+ * when a date check fails. It will retry at hourly intervals for a number of attempts
+ * specified by the RETRY_ATTEMPTS environment variable (defaults to 3).
+ *
+ * This is useful when:
+ * 1. The website hasn't updated with today's post yet
+ * 2. There are temporary network issues
+ * 3. The website structure changed temporarily
+ */
+
+// Sleep function to delay between retry attempts
+const sleep = (ms: number): Promise<void> =>
+	new Promise((resolve) => setTimeout(resolve, ms));
+
 export async function sendDiscordMessage(
 	client: Client,
 	url: string,
 	dateCheck?: dayjs.Dayjs
 ): Promise<void> {
-	if (!url) return;
 	const { img, title } = await getImgTitle(url);
 
 	if (dateCheck) {
@@ -26,7 +42,7 @@ export async function sendDiscordMessage(
 			);
 	}
 
-	client.channels.cache.forEach(async (channel) => {
+	const promises = client.channels.cache.map(async (channel) => {
 		try {
 			if (channel.type !== "text") return null;
 			const embeddedMessage = new MessageEmbed().setTitle(title).setImage(img);
@@ -50,13 +66,66 @@ export async function sendDiscordMessage(
 					console.error(`Reaction ${reaction} to channel ${channel.id} failed.`);
 				}
 			}
-		} catch {
+		} catch (err) {
 			console.error(`Message to channel ${channel.id} failed.`);
 		}
 	});
+	await Promise.all(promises);
 }
 
+/**
+ * Fetches and sends the next legica-dana post to all Discord channels
+ *
+ * This function implements a retry mechanism that will:
+ * 1. Try to fetch and post the latest content
+ * 2. If it fails (especially due to date check), wait for 1 hour
+ * 3. Retry up to the number of times specified in RETRY_ATTEMPTS env var
+ *
+ * @param client The Discord client used to send messages
+ * @throws Error if all retry attempts fail
+ */
 export async function sendNextMessage(client: Client): Promise<void> {
-	const href = await getFirstHtml();
+	// Get max retry attempts from config
-	await sendDiscordMessage(client, href, dayjs());
+	const maxRetries = config.RETRY_ATTEMPTS;
+	let attempts = 0;
+	let lastError: Error | null = null;
+
+	// Keep trying until we've reached max attempts
+	while (attempts < maxRetries) {
+		try {
+			// Get the URL of the latest post
+			const href = await getFirstHtml();
+			if (!href) throw new Error("URL cannot be empty!");
+
+			// Try to send the message
+			await sendDiscordMessage(client, href, dayjs());
+
+			// If successful, return
+			return;
+		} catch (error: unknown) {
+			attempts++;
+			const typedError = error instanceof Error ? error : new Error(String(error));
+			lastError = typedError;
+
+			// Log the retry attempt
+			console.error(
+				`Attempt ${attempts}/${maxRetries} failed: ${typedError.message}`
+			);
+
+			// If we've reached max attempts, throw the last error
+			if (attempts >= maxRetries) {
+				throw new Error(
+					`Failed after ${attempts} attempts. Last error: ${
+						lastError?.message || "Unknown error"
+					}`
+				);
+			}
+
+			// Wait for 1 hour before retrying (3600000 ms)
+			console.log(
+				`Waiting 1 hour before retry attempt ${attempts + 1}/${maxRetries}...`
+			);
+			await sleep(3600000);
+		}
+	}
 }

src/constants/config.ts

@@ -5,6 +5,7 @@ dotenv();
 type Config = {
 	APP_VERSION: string;
 	LEGICA_URL: string;
+	RETRY_ATTEMPTS: number;
 };
 
 export type ProjectConfig = Config & NodeJS.ProcessEnv;
@@ -18,6 +19,7 @@ const config: ProjectConfig = {
 	LEGICA_URL: "https://sib.net.hr/legica-dana",
 	TIMEZONE: process.env.TIMEZONE || "utc",
 	LEGICA_DATE_FORMAT: process.env.LEGICA_DATE_FORMAT || "D.M.YYYY",
+	RETRY_ATTEMPTS: parseInt(process.env.RETRY_ATTEMPTS || "3", 10),
 };
 
 export { config };