(Fix): Handle CircleCI PR number parsing properly

[TylerJang27]

As noted in the docs, CIRCLE_PR_NUMBER is only set on forked pull requests

We'll need to update our ref for this to take effect

[trunk-io]

😎 Merged directly without going through the merge queue, as the queue was empty and the PR was up to date with the target branch - details.

[TylerJang27]

@claude review

[trunk-staging-io]

     

View Full Report ↗︎ ⋅ Docs

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.01%. Comparing base (10de8fd) to head (e83aadc).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1084      +/-   ##
==========================================
+ Coverage   81.74%   82.01%   +0.27%     
==========================================
  Files          69       69              
  Lines       14907    14917      +10     
==========================================
+ Hits        12185    12234      +49     
+ Misses       2722     2683      -39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[trunk-io]

     

View Full Report ↗︎ ⋅ Docs

[dfrankland]

lgtm