Skip to content

Conversation

@pull
Copy link

@pull pull bot commented Nov 27, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

nivedin and others added 25 commits November 24, 2025 14:02
Bump dependencies and account for breaking changes.

---------

Co-authored-by: James George <[email protected]>
This updates Hoppscotch Agent dependencies to align with
 `v2025.11.0` security patch and other dependency chain.
This updates Hoppscotch Desktop (Shell) dependencies to align with
 `v2025.11.0` security patch and other dependency chain.

---------

Co-authored-by: James George <[email protected]>
Co-authored-by: mirarifhasan <[email protected]>
Co-authored-by: James George <[email protected]>
Fixes agent interceptor registration broken by dependency update

---------

Co-authored-by: James George <[email protected]>
…ach tab (#5544)

Co-authored-by: Nivedin <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: nivedin <[email protected]>
Refined network retry logic to distinguish between transient infrastructure
failures and intentional test errors, preventing incorrect test skips in
JUnit validation scenarios.

1. Network Error Detection (utils.ts)
   - Renamed `hasNetworkError` → `hasLowLevelNetworkError` for clarity
   - Removed REQUEST_ERROR from retry patterns (too generic, matches intentional bad URLs)
   - Now only retries on unambiguous TCP/DNS errors: ECONNRESET, EAI_AGAIN,
     ENOTFOUND, ETIMEDOUT, ECONNREFUSED
   - Preserved TEST_SCRIPT_ERROR detection when concurrent with REQUEST_ERROR
     (the actual CI failure mode from undefined response objects)
   - Added comprehensive JSDoc explaining when to use vs plain runCLI

2. JUnit XML Validation (test.spec.ts, 4 locations)
   - Removed REQUEST_ERROR and TEST_SCRIPT_ERROR from XML retry patterns
   - Only retry when low-level errors corrupt XML structure
   - Prevents skipping tests with intentional errors in collections
     (test-junit-report-export-coll.json has intentional invalid-url and
     script reference errors for validation)

3. Test Corrections
   - Fixed: "Fails to display console logs..." test now uses plain runCLI
     (test expects errors from legacy sandbox, shouldn't use retry)
   - Added: Environment version tests (v0, v1, v2) now use runCLIWithNetworkRetry
     (use echo.hoppscotch.io, expect success, benefit from retry)
   - Removed: Obsolete SKIP_EXTERNAL_TESTS env var check (retry logic handles this)
@pull pull bot locked and limited conversation to collaborators Nov 27, 2025
@pull pull bot added the ⤵️ pull label Nov 27, 2025
@pull pull bot merged commit ab52efc into Uncodedtech:main Nov 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants