feat: strict checks for required inputs (shubhaamgupta11#24)

* feat: safe checks added

* feat: version bumup

Author: shubhaamgupta11

README.md

@@ -35,7 +35,7 @@ jobs:
         with:
           node-version: '20'
       - name: Monitor New Issues
-        uses: shubhaamgupta11/[email protected].1
+        uses: shubhaamgupta11/[email protected].2
         with:
           # required inputs
           task: "monitor-issues"
@@ -73,7 +73,7 @@ jobs:
         with:
           node-version: '20'
       - name: Monitor New PRs
-        uses: shubhaamgupta11/[email protected].1
+        uses: shubhaamgupta11/[email protected].2
         with:
           # required inputs
           task: "monitor-prs"

action/index.js

@@ -24,6 +24,12 @@ async function run() {
     const discordIDType = core.getInput("discord_id_type");
     const discordIDs = core.getInput("discord_ids");
 
+        // Check if git_secret is provided
+    if (!gitToken) {
+      core.setFailed("Error: 'git_secret' is a mandatory input. Please provide a valid GitHub token.");
+      return;
+    }
+
     switch (task) {
       case "monitor-issues":
         await monitorIssues({

action/monitor-new-issue.js

@@ -90,6 +90,12 @@ async function monitorIssues({
       slackIDType,
       slackIDs,
     } = slackConfig;
+
+    if (!slackToken) {
+      console.log("No Slack token provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Slack for issues:",
       issues.map((issue) => issue.title)
@@ -109,13 +115,19 @@ async function monitorIssues({
       discordIDType,
       discordIDs,
     } = discordConfig;
+
+    if (!discordWebhookUrl) {
+      console.log("No Discord webhook URL provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Discord for issues:",
       issues.map((issue) => issue.title)
     );
     await sendDiscordNotification(discordWebhookUrl, issues, repo, "issue", discordIDType, discordIDs);
   } else {
-    throw new Error("Unsupported notifier. Use 'slack' or 'discord'.");
+    console.log("No notifier selected. Skipping notification.");
   }
 }
 

action/monitor-new-pr.js

@@ -101,6 +101,12 @@ async function monitorPRs({
       "🔔 Sending notifications to Slack for PRs:",
       prs.map((pr) => pr.title)
     );
+    
+    if (!slackToken) {
+      console.log("No Slack token provided. Skipping notification.");
+      return;
+    }
+
     await sendSlackNotification(
       slackToken,
       slackChannel,
@@ -117,13 +123,18 @@ async function monitorPRs({
       discordIDs,
     } = discordConfig;
 
+    if (!discordWebhookUrl) {
+      console.log("No Discord webhook URL provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Discord for PRs:",
       prs.map((pr) => pr.title)
     );
     await sendDiscordNotification(discordWebhookUrl, prs, repo, "pr", discordIDType, discordIDs);
   } else {
-    throw new Error("Unsupported notifier. Use 'slack' or 'discord'.");
+    console.log("No notifier selected. Skipping notification.");
   }
 }
 

dist/index.js

@@ -226,6 +226,12 @@ async function monitorIssues({
       slackIDType,
       slackIDs,
     } = slackConfig;
+
+    if (!slackToken) {
+      console.log("No Slack token provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Slack for issues:",
       issues.map((issue) => issue.title)
@@ -245,13 +251,19 @@ async function monitorIssues({
       discordIDType,
       discordIDs,
     } = discordConfig;
+
+    if (!discordWebhookUrl) {
+      console.log("No Discord webhook URL provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Discord for issues:",
       issues.map((issue) => issue.title)
     );
     await sendDiscordNotification(discordWebhookUrl, issues, repo, "issue", discordIDType, discordIDs);
   } else {
-    throw new Error("Unsupported notifier. Use 'slack' or 'discord'.");
+    console.log("No notifier selected. Skipping notification.");
   }
 }
 
@@ -365,6 +377,12 @@ async function monitorPRs({
       "🔔 Sending notifications to Slack for PRs:",
       prs.map((pr) => pr.title)
     );
+    
+    if (!slackToken) {
+      console.log("No Slack token provided. Skipping notification.");
+      return;
+    }
+
     await sendSlackNotification(
       slackToken,
       slackChannel,
@@ -381,13 +399,18 @@ async function monitorPRs({
       discordIDs,
     } = discordConfig;
 
+    if (!discordWebhookUrl) {
+      console.log("No Discord webhook URL provided. Skipping notification.");
+      return;
+    }
+
     console.log(
       "🔔 Sending notifications to Discord for PRs:",
       prs.map((pr) => pr.title)
     );
     await sendDiscordNotification(discordWebhookUrl, prs, repo, "pr", discordIDType, discordIDs);
   } else {
-    throw new Error("Unsupported notifier. Use 'slack' or 'discord'.");
+    console.log("No notifier selected. Skipping notification.");
   }
 }
 
@@ -39295,6 +39318,12 @@ async function run() {
     const discordIDType = core.getInput("discord_id_type");
     const discordIDs = core.getInput("discord_ids");
 
+        // Check if git_secret is provided
+    if (!gitToken) {
+      core.setFailed("Error: 'git_secret' is a mandatory input. Please provide a valid GitHub token.");
+      return;
+    }
+
     switch (task) {
       case "monitor-issues":
         await monitorIssues({

dist/index.js.map

@@ -1,6 +1,6 @@
 {
   "name": "repo-monitor",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "A GitHub Action to monitor a repository for new issues and PRs and send notifications to different channels.",
   "main": "index.js",
   "scripts": {