Merged
Conversation
Reduce request timeout from 60s to 30s for faster failure detection. Add retry timeouts and delays for more resilient CDN fetching, plus a per-package timeout to prevent slow packages from blocking the overall load process.
Implement retry mechanism for CDN cache warming - first request triggers caching and subsequent retries hit the warm cache. Add per-package timeout cap to ensure slow packages don't block the entire batch.
…timeout - Remove per-package timeout race logic that was adding unnecessary complexity - Rename PER_PACKAGE_TIMEOUT to JSDELIVR_POOL_TIMEOUT (60s) for connection lifecycle - Add error logging for failed fetches to preserve observability - Add comprehensive retry tests for jsDelivr fallback behavior
Increase request timeout to 60s while shortening retry timeouts for faster fallback to npm registry. Add retry logic for transient HTTP errors (429, 5xx) and skip major version fetch when current major matches latest major version.
…ansient network issues
- Add in-flight request coalescing to prevent duplicate lookups for the same package - Handle transient network errors (ENOTFOUND, ECONNRESET, ETIMEDOUT, etc.) with retries - Improve version sorting to handle non-semver versions gracefully - Reduce noisy console output for expected transient errors - Add safer body consumption and JSON parsing with proper fallback - Refactor retry configuration to use validated timeout values
Add --debug CLI flag and INUP_DEBUG environment variable support for verbose logging to inup-debug-YYYY-MM-DD.log. Includes performance timing, registry request tracing, and error context capture across package detection and jsdelivr registry operations.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.