fix: wrap state-mutating ops in try/catch with contextual error logging

- convertToDraft re-throws on failure so the workflow fails loudly
- removeManagedLabels does not re-throw since draft conversion is the
  primary goal; a label-cleanup failure is logged via core.error but
  does not fail the run

Signed-off-by: Gourav NSS <gourav341111@gmail.com>

Author: NssGourav

.github/scripts/revision-guard/index.js

@@ -35,22 +35,35 @@ module.exports = async function revisionGuard({ github, context, core }) {
     return;
   }
 
-  await convertToDraft(github, pr.node_id);
-  core?.info?.(`Converted PR #${pr.number} to draft.`);
+  try {
+    await convertToDraft(github, pr.node_id);
+    core?.info?.(`Converted PR #${pr.number} to draft.`);
+  } catch (error) {
+    core?.error?.(`Failed to convert PR #${pr.number} to draft: ${error.message}`);
+    throw error;
+  }
 
   const labelsToRemove = getPresentManagedLabels(pr.labels);
   if (labelsToRemove.length === 0) {
     core?.info?.(`No managed labels to remove for PR #${pr.number}.`);
     return;
   }
 
-  await removeManagedLabels(github, {
-    owner: repo.owner,
-    repo: repo.repo,
-    issueNumber: pr.number,
-    labels: labelsToRemove,
-  });
-  core?.info?.(
-    `Removed managed labels from PR #${pr.number}: ${labelsToRemove.join(', ')}.`
-  );
+  try {
+    await removeManagedLabels(github, {
+      owner: repo.owner,
+      repo: repo.repo,
+      issueNumber: pr.number,
+      labels: labelsToRemove,
+    });
+    core?.info?.(
+      `Removed managed labels from PR #${pr.number}: ${labelsToRemove.join(', ')}.`
+    );
+  } catch (error) {
+    core?.error?.(
+      `Failed to remove labels from PR #${pr.number}: ${error.message}. ` +
+      `Labels to remove: ${labelsToRemove.join(', ')}.`
+    );
+    // Don't re-throw; draft conversion succeeded and is the primary goal
+  }
 };