TS
Fix/#188
tscircuit/cli#207

Description

This PR fixes an issue where the CLI incorrectly reports failure while actually succeeding in pushing packages. The root cause was improper handling of success/failure states in the package push operation.

Changes

  • Added proper error handling for file uploads using try/catch blocks
  • Fixed the issue where success was reported even when file uploads failed
  • Added explicit success return with correct exit code
  • Improved error messages for better user feedback
  • Added break statement to exit upload loop on first error to prevent partial uploads

Testing

The changes have been tested to ensure:

  1. Success is only reported when all operations complete successfully
  2. Failure is properly reported when any part of the push operation fails
  3. Users receive clear error messages about what went wrong
  4. Process exits with correct status code (0 for success, 1 for failure)

Impact

This fix improves the reliability of the package push operation and provides better feedback to users about the actual state of their package push.

Related Issues

Fixes #188 - “pushing packages from cli succeeds while telling the user it failed” /claim #188

Claim

Total prize pool $18
Total paid $0
Status Pending
Submitted May 29, 2025
Last updated May 29, 2025

Contributors

AA

Aayush Khopade

@Apocalypse96

100%

Sponsors

TS

tscircuit

@tscircuit

$18