fix(utils): preserve target URL query string in path merging

[pi0]

Summary

• When the proxy target URL contains a query string (e.g., http://example.com/api?key=val), it was silently dropped because setupOutgoing only used .pathname from the target URL
• Now the target's .search is extracted and properly merged with any request query string, using & as separator when both are present
• Only applies to URL instances (not string or ProxyTargetDetailed targets), which is correct since string targets are normalized to URL objects before reaching setupOutgoing

Upstream: http-party/node-http-proxy#1334

Test plan

• Added regression test: target with query string, no request query string
• Added regression test: target and request both have query strings (merged with &)
• Added regression test: target with query string only, request path is /
• All 247 existing tests pass with no regressions

[coderabbitai]

Warning

Rate limit exceeded

@pi0 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 58 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 14d1a235-994c-438e-8909-1b3a40e42848

📥 Commits

Reviewing files that changed from the base of the PR and between f30f90f and 2315fc3.

📒 Files selected for processing (2)

• src/_utils.ts
• test/_utils.test.ts

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/query-string-path-merge

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.15%. Comparing base (f30f90f) to head (2315fc3).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #117      +/-   ##
==========================================
+ Coverage   96.12%   96.15%   +0.02%     
==========================================
  Files           8        8              
  Lines         645      650       +5     
  Branches      243      246       +3     
==========================================
+ Hits          620      625       +5     
  Misses         24       24              
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.