fix(validate-pr): Remove draft enforcement from composite action (#159)

The `convertPullRequestToDraft` GraphQL mutation requires permissions
that the SDK Maintainer Bot app does not have, causing the action to
fail on non-draft PRs.

Remove the draft conversion and labeling steps entirely rather than
expanding the app's permissions.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: stephanie-anderson

validate-pr/README.md

@@ -4,10 +4,9 @@ Validates non-maintainer pull requests against contribution guidelines.
 
 ## What it does
 
-1. **Validates issue references** — Non-maintainer PRs must reference a GitHub issue where the PR author and a maintainer have discussed the approach. PRs that don't meet this requirement are automatically closed with a descriptive comment.
-2. **Enforces draft status** — All PRs must start as drafts. Non-draft PRs are automatically converted and labeled.
+**Validates issue references** — Non-maintainer PRs must reference a GitHub issue where the PR author and a maintainer have discussed the approach. PRs that don't meet this requirement are automatically closed with a descriptive comment.
 
-Maintainers (users with `admin` or `maintain` role) are exempt from the issue reference validation. Draft enforcement applies to everyone.
+Maintainers (users with `admin` or `maintain` role) are exempt from validation.
 
 ## Usage
 
@@ -61,10 +60,6 @@ A PR is valid if **any** referenced issue passes all checks:
 - If the issue has assignees, the PR author must be one of them
 - Both the PR author and a maintainer have participated in the issue discussion
 
-### Draft enforcement
-
-Non-draft PRs are converted to draft and labeled `converted-to-draft` with an informational comment.
-
 ## Labels
 
 The action creates these labels automatically (they don't need to exist beforehand):
@@ -73,4 +68,3 @@ The action creates these labels automatically (they don't need to exist beforeha
 - `missing-issue-reference` — PR body has no issue references
 - `missing-maintainer-discussion` — referenced issue lacks author + maintainer discussion
 - `issue-already-assigned` — referenced issue is assigned to someone else
-- `converted-to-draft` — PR was automatically converted to draft

validate-pr/action.yml

@@ -1,5 +1,5 @@
 name: 'Validate PR'
-description: 'Validates non-maintainer PRs against contribution guidelines and enforces draft status'
+description: 'Validates non-maintainer PRs against contribution guidelines'
 author: 'Sentry'
 
 inputs:
@@ -33,26 +33,3 @@ runs:
         script: |
           const script = require('${{ github.action_path }}/scripts/validate-pr.js');
           await script({ github, context, core });
-
-    - name: Convert PR to draft
-      if: >-
-        steps.validate.outputs.was-closed != 'true'
-        && steps.validate.outputs.skipped != 'true'
-        && github.event.pull_request.draft == false
-      shell: bash
-      env:
-        GH_TOKEN: ${{ steps.app-token.outputs.token }}
-        PR_URL: ${{ github.event.pull_request.html_url }}
-      run: gh pr ready "$PR_URL" --undo
-
-    - name: Label and comment on draft conversion
-      if: >-
-        steps.validate.outputs.was-closed != 'true'
-        && steps.validate.outputs.skipped != 'true'
-        && github.event.pull_request.draft == false
-      uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
-      with:
-        github-token: ${{ steps.app-token.outputs.token }}
-        script: |
-          const script = require('${{ github.action_path }}/scripts/enforce-draft.js');
-          await script({ github, context, core });