refactor: env, cartUrl optional, other consistencies

Signed-off-by: Jef LeCompte <jeffreylec@gmail.com>

src/config.ts

@@ -21,20 +21,20 @@ const notifications = {
 		carrier: process.env.PHONE_CARRIER ?? '',
 		number: process.env.PHONE_NUMBER ?? ''
 	},
-	slack: {
-		channel: process.env.SLACK_CHANNEL ?? '',
-		token: process.env.SLACK_TOKEN ?? ''
-	},
+	playSound: process.env.PLAY_SOUND ?? 'false',
 	pushover: {
 		token: process.env.PUSHOVER_TOKEN,
 		user: process.env.PUSHOVER_USER
 	},
+	slack: {
+		channel: process.env.SLACK_CHANNEL ?? '',
+		token: process.env.SLACK_TOKEN ?? ''
+	},
 	telegram: {
-		botToken: process.env.TELEGRAM_ACCESS_TOKEN ?? '',
+		accessToken: process.env.TELEGRAM_ACCESS_TOKEN ?? '',
 		chatId: process.env.TELEGRAM_CHAT_ID ?? ''
 	},
-	test: process.env.NOTIFICATION_TEST ?? 'false',
-	playSound: process.env.PLAY_SOUND ?? 'false'
+	test: process.env.NOTIFICATION_TEST ?? 'false'
 };
 
 const page = {
@@ -45,12 +45,9 @@ const page = {
 	userAgent: 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/78.0.3904.108 Safari/537.36'
 };
 
-const rateLimitTimeout = Number(process.env.RATE_LIMIT_TIMEOUT) ?? 5000;
-
-const stores = process.env.STORES ?? 'nvidia';
-
 const openBrowser = process.env.OPEN_BROWSER ?? 'true';
-
+const rateLimitTimeout = Number(process.env.RATE_LIMIT_TIMEOUT) ?? 5000;
+const stores = process.env.STORES ? process.env.STORES.split(',') : ['nvidia'];
 const showOnlyBrands = process.env.SHOW_ONLY_BRANDS ? process.env.SHOW_ONLY_BRANDS.split(',') : [];
 
 export const Config = {

src/notification/email.ts

@@ -3,8 +3,8 @@ import Mail from 'nodemailer/lib/mailer';
 import {Config} from '../config';
 import {Logger} from '../logger';
 
-const subject = 'NVIDIA - BUY NOW';
 const email = Config.notifications.email;
+const subject = 'NVIDIA - BUY NOW';
 
 const transporter = nodemailer.createTransport({
 	service: 'gmail',

src/notification/notification.ts

@@ -3,34 +3,36 @@ import {sendEmail} from './email';
 import {sendSMS} from './sms';
 import {playSound} from './sound';
 import {sendSlackMessage} from './slack';
-import sendPushoverNotification from './pushover';
+import {sendPushoverNotification} from './pushover';
 import {sendTelegramMessage} from './telegram';
 
+const notifications = Config.notifications;
+
 export function sendNotification(cartUrl: string) {
-	if (Config.notifications.email.username && Config.notifications.email.password) {
+	if (notifications.email.username && notifications.email.password) {
 		sendEmail(cartUrl);
 	}
 
-	if (Config.notifications.slack.channel && Config.notifications.slack.token) {
+	if (notifications.slack.channel && notifications.slack.token) {
 		sendSlackMessage(cartUrl);
 	}
 
-	if (Config.notifications.telegram.botToken && Config.notifications.telegram.chatId) {
+	if (notifications.telegram.accessToken && notifications.telegram.chatId) {
 		sendTelegramMessage(cartUrl);
 	}
 
-	if (Config.notifications.phone.number) {
-		const carrier = Config.notifications.phone.carrier.toLowerCase();
-		if (carrier && Config.notifications.phone.availableCarriers.has(carrier)) {
+	if (notifications.phone.number) {
+		const carrier = notifications.phone.carrier.toLowerCase();
+		if (carrier && notifications.phone.availableCarriers.has(carrier)) {
 			sendSMS(cartUrl);
 		}
 	}
 
-	if (Config.notifications.pushover.token && Config.notifications.pushover.user) {
+	if (notifications.pushover.token && notifications.pushover.user) {
 		sendPushoverNotification(cartUrl);
 	}
 
-	if (Config.notifications.playSound !== 'false') {
+	if (notifications.playSound === 'true') {
 		playSound();
 	}
 }

src/notification/pushover.ts

@@ -1,18 +1,19 @@
+import Push from 'pushover-notifications';
 import {Config} from '../config';
 import {Logger} from '../logger';
-import Push = require('pushover-notifications');
 
-const p = new Push({
-	user: Config.notifications.pushover.user,
-	token: Config.notifications.pushover.token
+const pushover = Config.notifications.pushover;
+const push = new Push({
+	user: pushover.user,
+	token: pushover.token
 });
 
-export default function sendPushoverNotification(text: string) {
+export function sendPushoverNotification(text: string) {
 	const message = {
 		message: text
 	};
 
-	p.send(message, (err: Error, result: string) => {
+	push.send(message, (err: Error, result: string) => {
 		if (err) {
 			Logger.error(err);
 		} else {

src/notification/telegram.ts

@@ -2,18 +2,17 @@ import {Config} from '../config';
 import {Logger} from '../logger';
 import {TelegramClient} from 'messaging-api-telegram';
 
-const chatId = Config.notifications.telegram.chatId;
-const accessToken = Config.notifications.telegram.botToken;
+const telegram = Config.notifications.telegram;
 
 const client = new TelegramClient({
-	accessToken
+	accessToken: telegram.accessToken
 });
 
 export function sendTelegramMessage(text: string) {
 	(async () => {
 		try {
-			await client.sendMessage(chatId, text);
-			Logger.info(`✔ telegram message sent to '${chatId}': ${text}`);
+			await client.sendMessage(telegram.chatId, text);
+			Logger.info(`✔ telegram message sent to '${telegram.chatId}': ${text}`);
 		} catch (error) {
 			Logger.error(error);
 		}

src/store/lookup.ts

@@ -4,7 +4,7 @@ import {Logger} from '../logger';
 import open from 'open';
 import {Store} from './model';
 import {sendNotification} from '../notification';
-import {isOutOfStock as includesLabels} from './out-of-stock';
+import {includesLabels} from './out-of-stock';
 
 /**
  * Returns true if the brand should be checked for stock

src/store/model/amazon-ca.ts

@@ -1,74 +1,84 @@
 import {Store} from './store';
 
-export const amazonca: Store = {
-	cartUrl: '',
+export const AmazonCa: Store = {
 	links: [
 		{
 			brand: 'msi',
 			model: 'gaming trio',
 			url: 'https://www.amazon.ca/MSI-GeForce-RTX-3080-10G/dp/B08HR7SV3M?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'evga',
 			model: 'ftw3gaming',
 			url: 'https://www.amazon.ca/EVGA-GeForce-Technology-Backplate-10G-P5-3895-KR/dp/B08HR3DPGW?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'evga',
 			model: 'ftw3ultra',
 			url: 'https://www.amazon.ca/EVGA-GeForce-Technology-Backplate-10G-P5-3897-KR/dp/B08HR3Y5GQ?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'evga',
 			model: 'xc3ultra',
 			url: 'https://www.amazon.ca/EVGA-GeForce-Cooling-Backplate-10G-P5-3885-KR/dp/B08HR55YB5?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'evga',
 			model: 'xc3gaming',
 			url: 'https://www.amazon.ca/EVGA-GeForce-Cooling-Backplate-10G-P5-3883-KR/dp/B08HR4RJ3Q?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'evga',
 			model: 'xc3black',
 			url: 'https://www.amazon.ca/EVGA-GeForce-Gaming-Cooling-10G-P5-3881-KR/dp/B08HR6FMF3?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'gigabyte',
 			model: 'windforce',
 			url: 'https://www.amazon.ca/GIGABYTE-GeForce-Graphics-WINDFORCE-GV-N3080GAMING/dp/B08HJTH61J?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'gigabyte',
 			model: 'windforce eagle',
 			url: 'https://www.amazon.ca/GIGABYTE-GeForce-Graphics-WINDFORCE-GV-N3080EAGLE/dp/B08HJS2JLJ?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'asus',
 			model: 'tuf',
 			url: 'https://www.amazon.ca/Asus-90YV0FB0-M0AM00-TUF-RTX3080-10G-GAMING/dp/B08HHDP9DW?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'asus',
 			model: 'tufoc',
 			url: 'https://www.amazon.ca/Asus-90YV0FB1-M0AM00-TUF-RTX3080-O10G-GAMING/dp/B08HH5WF97?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		},
 		{
 			brand: 'msi',
 			model: 'ventus',
 			url: 'https://www.amazon.ca/MSI-GeForce-RTX-3080-10G/dp/B08HR5SXPS?ref_=ast_sto_dp',
-			oosLabels: ['currently unavailable', 'enter the characters you see below']
+			oosLabels: ['currently unavailable'],
+			captchaLabels: ['enter the characters you see below']
 		}
 	],
-	name: 'amazonca'
+	name: 'amazonCa'
 };

src/store/model/amazon.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const Amazon: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'pny',

src/store/model/bandh.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const BAndH: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'gigabyte',

src/store/model/bestbuy.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const BestBuy: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'asus',

src/store/model/evga.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const Evga: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'evga',

src/store/model/index.ts

@@ -6,11 +6,11 @@ import {Amazon} from './amazon';
 import {MicroCenter} from './microcenter';
 import {Config} from '../../config';
 import {Nvidia} from './nvidia';
-import {amazonca} from './amazonca';
+import {AmazonCa} from './amazon-ca';
 
 const masterList = new Map([
 	['amazon', Amazon],
-	['amazonca', amazonca],
+	['amazonca', AmazonCa],
 	['bestbuy', BestBuy],
 	['bandh', BAndH],
 	['evga', Evga],
@@ -21,9 +21,7 @@ const masterList = new Map([
 
 const list = new Map();
 
-const storeArray = Config.stores.split(',');
-
-for (const name of storeArray) {
+for (const name of Config.stores) {
 	list.set(name, masterList.get(name));
 }
 

src/store/model/microcenter.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const MicroCenter: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'evga',

src/store/model/newegg.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const NewEgg: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'asus',

src/store/model/nvidia.ts

@@ -1,7 +1,6 @@
 import {Store} from './store';
 
 export const Nvidia: Store = {
-	cartUrl: '',
 	links: [
 		{
 			brand: 'nvidia',

src/store/model/store.ts

@@ -7,7 +7,7 @@ interface Link {
 }
 
 export interface Store {
-	cartUrl: string;
+	cartUrl?: string;
 	links: Link[];
 	name: string;
 }

src/store/out-of-stock.ts

@@ -4,7 +4,7 @@
  * @param domText Complete DOM of website.
  * @param oosLabels Out-of-stock labels.
  */
-export function isOutOfStock(domText: string, oosLabels: string[]) {
+export function includesLabels(domText: string, oosLabels: string[]): boolean {
 	const domTextLowerCase = domText.toLowerCase();
 	return oosLabels.some(label => domTextLowerCase.includes(label));
 }