fix: Add parseInt validation and remove unused environment variable

[hobostay]

Summary

This PR includes two bug fixes for the scripts:

1. Add NaN validation for parseInt operations in backfill-duplicate-comments.ts

   • Previously, invalid environment variable values (e.g., MAX_ISSUE_NUMBER="invalid") would silently become NaN, causing unexpected script behavior
   • Now validates that parsed numbers are valid and within reasonable ranges
   • Throws descriptive error messages for invalid inputs

2. Remove unused STATSIG_API_KEY from auto-close-duplicates.yml workflow

   • The environment variable was set but never used by the auto-close-duplicates.ts script
   • Removes confusion about unused configuration

Changes

scripts/backfill-duplicate-comments.ts

• Added validation after parseInt() operations to ensure values are not NaN
• Added validation to ensure MIN_ISSUE_NUMBER is at least 1
• Added validation to ensure MAX_ISSUE_NUMBER is greater than MIN_ISSUE_NUMBER
• Error messages now include the invalid value for easier debugging

.github/workflows/auto-close-duplicates.yml

• Removed STATSIG_API_KEY environment variable (unused)

Test plan

• Script now throws descriptive errors for invalid MAX_ISSUE_NUMBER or MIN_ISSUE_NUMBER values
• Script validates that MIN_ISSUE_NUMBER >= 1
• Script validates that MAX_ISSUE_NUMBER > MIN_ISSUE_NUMBER
• Workflow configuration no longer includes unused environment variable

Example Error Messages

MAX_ISSUE_NUMBER must be a valid number. Got: "invalid"
MIN_ISSUE_NUMBER must be at least 1. Got: 0
MAX_ISSUE_NUMBER must be greater than MIN_ISSUE_NUMBER. Got: MIN=100, MAX=50

🤖 Generated with Claude Code