stores-com / bloodhound

86%

main: 88%

LAST BUILD ON BRANCH remove-nock

branch: remove-nock

CHANGE BRANCH

x

Reset

• remove-nock
• USPS-404-handling
• USPS-client-migration
• add-xpo-carrier
• bump-4.0.2
• bump-4.1.0
• bump-dhl-ecommerce-solutions-1
• dependabot/github_actions/actions/setup-node-7
• dhl-backoff-429
• dhl-ecommerce-async-await
• dhl-ecommerce-solutions-0.5
• dhl-impb-ecs
• dhl-no-retry-on-4xx
• dhl-outer-retry-4xx
• dhl-processed-through-usps-sort
• document-v4-error-and-timezone-changes
• fedex-package
• fedex-request-to-fetch
• fix-404-detection-by-status
• fix-event-sort
• fix-publish-gate
• fix/usps-status-category-fallback
• four-small-fixes
• guard-ups-incomplete-timestamp
• main
• ontrac-mock-carrier-api
• promise-carriers
• remove-callbacks
• remove-geocoding
• remove-pitney-bowes-support
• remove-request-dependency
• remove-usps-fallback
• request-to-fetch
• skip-geocoding-and-default-timezone
• stringify-USPS-error
• test
• update-package-version
• update-pitney-bowes
• update-readme

Build # 30507354936

Build Type

Pull #175

github

Committed by freshlogic

Commit Message

chore: drop xml2js, which nothing has required since the USPS JSON migration

Audited every declared dependency against actual `require()` calls. xml2js is the only
dead one: its sole mention anywhere outside node_modules is its own package.json line.
The last XML consumer was the legacy USPS API, and carriers/usps.js now only carries a
comment referring to it — the v3r2 JSON API arrived in 3.3.0 via @stores.com/usps.

The other four asked about are all still load-bearing:

  memory-cache      carriers/fedEx.js — caches the OAuth token
  moment-timezone   dhl.js, dhlEcommerceSolutions.js, pitneyBowes.js, usps.js
  pitney-bowes      carriers/pitneyBowes.js
  async             six carriers, for async.retry and async.mapLimit

Scoping the negative claim: I grepped carriers/, util/, index.js and test/ for
require('xml2js'), for dynamic require patterns, and for parseString/Builder usage.
Nothing. I did not audit transitive dependents, which are free to depend on it
themselves.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #175: test: drop nock and add UPS fixtures for the response shapes we never covered

Coverage Stats

312 of 373 branches covered (83.65%)

Branch coverage included in aggregate %.

1494 of 1715 relevant lines covered (87.11%)

17.18 hits per line