[github-actions]

Bug fix — oversized-number-crash

Proposed fix for issue #5194, applying the
remediation from the bug assessment.

Verdict: valid · Severity: low

Summary

The Python extension branch-creation script now validates --number against a bounded integer domain before conversion, so oversized numeric inputs fail with a clear argument error instead of crashing at int().

Changes

File	Change	Notes
extensions/git/scripts/python/create_new_feature_branch.py	modified	Added int64-style bound validation (0..2**63-1) for --number and a defensive conversion guard with user-facing error text.
tests/extensions/git/test_git_extension_python_parity.py	modified	Added boundary and regression tests for max, max+1, and oversized digit-only --number inputs; asserts no traceback on rejection.

Tests Added or Updated

• tests/extensions/git/test_git_extension_python_parity.py::TestCreateFeatureBranchParity::test_explicit_number_max_int64_accepted — verifies the maximum supported explicit number is accepted.
• tests/extensions/git/test_git_extension_python_parity.py::TestCreateFeatureBranchParity::test_explicit_number_above_max_int64_rejected — pins rejection for max+1 with expected error message.
• tests/extensions/git/test_git_extension_python_parity.py::TestCreateFeatureBranchParity::test_explicit_number_oversized_digits_rejected_without_traceback — verifies long digit strings are rejected cleanly without traceback.

Local Verification

• Commands run: python3 extensions/git/scripts/python/create_new_feature_branch.py --json --dry-run --number 42 "some feature" → pass (exit 0)
• Commands run: python3 extensions/git/scripts/python/create_new_feature_branch.py --json --dry-run --number 9223372036854775808 "some feature" → expected validation failure (exit 1, bounded-range error)
• Commands run: python3 extensions/git/scripts/python/create_new_feature_branch.py --json --dry-run --number <5000-digit number> "some feature" → expected validation failure (exit 1, bounded-range error, no traceback)
• python3 -m pytest ... could not run in this runner because pytest is unavailable in the base Python environment.

Deviations from Assessment

• Added Python-only parity tests that explicitly codify current divergence from bash for oversized --number handling, rather than changing bash/powershell behavior in this fix-stage patch.

Risks & Review Notes

• Bash/PowerShell extension twins currently do not enforce the same explicit upper bound for --number; this PR intentionally scopes remediation to the Python crash path identified in the assessment.

Refs #5194 · cc @soarkit

• Fixes The Python branch-creation port accepts an oversized --number and crashes at int() instead of rejecting it #5194

Generated by 🛠️ Fix Bug from Labeled Issue for #5194 · copilot · gpt53codex · 49.1 AIC · ⌖ 0.695 AIC · ⊞ 25.1K · ◷