Skip to content

Conversation

AtofStryker
Copy link
Contributor

@AtofStryker AtofStryker commented Oct 2, 2025

  • Closes

Additional details

Steps to test

How has the user experience changed?

PR Tasks


Note

Introduces @packages/network-tools for browser/shared networking utilities and refactors imports; updates @packages/network to CJS/ESM builds with async APIs, and adjusts tests/CI accordingly.

  • New package: @packages/network-tools
    • Exposes cors, uri, document-domain-injection, and related types for browser/shared use.
  • Refactors:
    • Replace imports from @packages/network with @packages/network-tools in driver, proxy, server, config, and tests (e.g., origin/cors/uri helpers, DocumentDomainInjection).
    • Update APIs where needed (e.g., origin, getSuperDomain*, url*Match, removeDefaultPort, policyFromConfig).
  • @packages/network updates:
    • Build outputs CJS/ESM; remove runtime ts-register shim; export CombinedAgent.
    • Modernize to async/await (e.g., connect.getAddress, CA/client cert handling); add typings and minor fixes.
    • README adjusted to reflect Node-only scope.
  • Tests:
    • Migrate Bluebird usages to native promises/util.promisify where appropriate; adjust spies (e.g., agent prototype) and fixtures.
  • CI/Docs:
    • CircleCI: build order tweaks and ensure @packages/network(-tools) built in binary check job.
    • ESM migration guide: mark packages/network and packages/network-tools as completed.

Written by Cursor Bugbot for commit 267df5b. This will update automatically on new commits. Configure here.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

Copy link

cypress bot commented Oct 2, 2025

cypress    Run #66159

Run Properties:  status check passed Passed #66159  •  git commit 267df5b599: chore: sync cloud validations after building packages as the network libs are ne...
Project cypress
Branch Review chore/network-bundle
Run status status check passed Passed #66159
Run duration 19m 51s
Commit git commit 267df5b599: chore: sync cloud validations after building packages as the network libs are ne...
Committer Bill Glesias
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 9
Tests that did not run due to a developer annotating a test with .skip  Pending 1102
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 26661
View all changes introduced in this branch ↗︎
UI Coverage  45.11%
  Untested elements 186  
  Tested elements 157  
Accessibility  97.76%
  Failed rules  4 critical   8 serious   2 moderate   2 minor
  Failed elements 110  

@AtofStryker AtofStryker force-pushed the chore/network-bundle branch from 21c3259 to 2b03c2c Compare October 3, 2025 01:38
cursor[bot]

This comment was marked as outdated.

@AtofStryker AtofStryker force-pushed the chore/network-bundle branch 2 times, most recently from a6ac039 to eab0745 Compare October 3, 2025 15:14
@AtofStryker AtofStryker self-assigned this Oct 3, 2025
cursor[bot]

This comment was marked as outdated.

@AtofStryker AtofStryker force-pushed the chore/network-bundle branch from 4ce54ef to 1c01f92 Compare October 3, 2025 16:32
… network-tools is expected to be used in simple environments, where network is intended to be used in the node context. additionally, makes these packages bundable and removes the ts-node entrypoint to make ESM possible.
@AtofStryker AtofStryker force-pushed the chore/network-bundle branch from 1c01f92 to 187cfd7 Compare October 3, 2025 18:11
@AtofStryker AtofStryker force-pushed the chore/network-bundle branch from 187cfd7 to 267df5b Compare October 3, 2025 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant