fix: convert revision guard PRs to draft via GraphQL

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

Author: NssGourav

.github/scripts/revision-guard/helpers/draft.js

@@ -7,13 +7,20 @@ function isDraft(pr) {
   return pr?.draft === true;
 }
 
-async function convertToDraft(github, { owner, repo, pullNumber }) {
-  await github.rest.pulls.update({
-    owner,
-    repo,
-    pull_number: pullNumber,
-    draft: true,
-  });
+async function convertToDraft(github, { pullRequestId }) {
+  await github.graphql(
+    `
+      mutation ConvertPullRequestToDraft($pullRequestId: ID!) {
+        convertPullRequestToDraft(input: { pullRequestId: $pullRequestId }) {
+          pullRequest {
+            id
+            isDraft
+          }
+        }
+      }
+    `,
+    { pullRequestId }
+  );
 }
 
 module.exports = {

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

@@ -18,6 +18,7 @@ module.exports = async function revisionGuard({ github, context, core }) {
     !repo?.repo ||
     !pr ||
     typeof pr.number !== 'number' ||
+    typeof pr.node_id !== 'string' ||
     reviewState !== 'changes_requested'
   ) {
     core?.info?.('Skipping revision guard due to missing or non-matching payload data.');
@@ -36,9 +37,7 @@ module.exports = async function revisionGuard({ github, context, core }) {
 
   try {
     await convertToDraft(github, {
-      owner: repo.owner,
-      repo: repo.repo,
-      pullNumber: pr.number,
+      pullRequestId: pr.node_id,
     });
     core?.info?.(`Converted PR #${pr.number} to draft.`);
   } catch (error) {

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

@@ -6,33 +6,41 @@ function freshRequire() {
   const helpersPath = require.resolve('./helpers/index.js');
   const labelsPath = require.resolve('./helpers/labels.js');
   const constantsPath = require.resolve('./helpers/constants.js');
+  const draftPath = require.resolve('./helpers/draft.js');
 
   delete require.cache[indexPath];
   delete require.cache[helpersPath];
   delete require.cache[labelsPath];
   delete require.cache[constantsPath];
+  delete require.cache[draftPath];
 
   return require('./index.js');
 }
 
 function createGithubMock() {
   const removedLabels = [];
-  const pullUpdates = [];
+  const graphqlCalls = [];
 
   return {
     removedLabels,
-    pullUpdates,
+    graphqlCalls,
+    graphql: async (query, variables) => {
+      graphqlCalls.push({ query, variables });
+      return {
+        convertPullRequestToDraft: {
+          pullRequest: {
+            id: variables.pullRequestId,
+            isDraft: true,
+          },
+        },
+      };
+    },
     rest: {
       issues: {
         removeLabel: async ({ name }) => {
           removedLabels.push(name);
         },
       },
-      pulls: {
-        update: async (params) => {
-          pullUpdates.push(params);
-        },
-      },
     },
   };
 }
@@ -74,9 +82,9 @@ describe('revision-guard index', () => {
 
     await handler({ github, context, core: { info() {} } });
 
-    assert.deepEqual(github.pullUpdates, [
-      { owner: 'hiero-ledger', repo: 'hiero-sdk-python', pull_number: 42, draft: true },
-    ]);
+    assert.equal(github.graphqlCalls.length, 1);
+    assert.match(github.graphqlCalls[0].query, /convertPullRequestToDraft/);
+    assert.deepEqual(github.graphqlCalls[0].variables, { pullRequestId: 'PR_node_42' });
     assert.deepEqual(github.removedLabels, [
       'queue:committers',
       'status: ready-to-merge',
@@ -98,7 +106,7 @@ describe('revision-guard index', () => {
 
     await handler({ github, context, core: { info() {} } });
 
-    assert.equal(github.pullUpdates.length, 0);
+    assert.equal(github.graphqlCalls.length, 0);
     assert.equal(github.removedLabels.length, 0);
   });
 
@@ -117,7 +125,7 @@ describe('revision-guard index', () => {
 
     await handler({ github, context, core: { info() {} } });
 
-    assert.equal(github.pullUpdates.length, 0);
+    assert.equal(github.graphqlCalls.length, 0);
     assert.equal(github.removedLabels.length, 0);
   });
 
@@ -142,9 +150,8 @@ describe('revision-guard index', () => {
     await handler({ github, context, core: { info() {} } });
 
     // Draft conversion must also fire for configurable-label scenarios.
-    assert.deepEqual(github.pullUpdates, [
-      { owner: 'hiero-ledger', repo: 'hiero-sdk-python', pull_number: 45, draft: true },
-    ]);
+    assert.equal(github.graphqlCalls.length, 1);
+    assert.deepEqual(github.graphqlCalls[0].variables, { pullRequestId: 'PR_node_45' });
     // Custom labels AND the matching default (queue:committers) must both be removed.
     assert.deepEqual(github.removedLabels, ['queue:committers', 'custom: one', 'custom: two']);
   });