Fix: fix url component parsing bug

[lslvxy]

When a Url is constructed from a string containing Nunjucks template tags ({{ }} / {% %}), urlObject is intentionally left undefined to avoid the native URL parser percent-encoding the tags. This caused getHost(), getPath(), and getRemote() to always return '' for any templated URL.

Changes

• urls.ts — Added regex fallbacks in getHost(), getPath(), and getRemote() that extract the relevant segment directly from this.origin when this.urlObject is undefined. Patterns extracted into three named module-level constants for clarity:

const URL_HOST_PATTERN  = /^(?:https?:\/\/)?([^\/:]+)/i;        // hostname, no port
const URL_PATH_PATTERN  = /^(?:https?:\/\/[^\/]+)(\/[^?#]*)/i;  // path, no query/hash
const URL_REMOTE_PATTERN = /^(?:https?:\/\/)?([^\/]+)/i;        // host + port

Example — previously all three returned '':

const url = new Url('http://{{ host }}:8080/api/v1');
url.getHost();   // '{{ host }}'
url.getPath();   // '/api/v1'
url.getRemote(); // '{{ host }}:8080'

• urls.test.ts — New test covering host-only tags, path tags, host+port tags, Nunjucks block tags ({% %}), and no-path URLs.

Link Issue #9746

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ lslvxy
❌ Copilot
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[Copilot]

Pull request overview

Fixes Url.getHost(), getPath(), and getRemote() returning empty strings when Url parsing is intentionally skipped for templated URLs (e.g. Nunjucks {{ }} / {% %}), by adding regex-based fallbacks that extract host/path/remote from this.origin.

Changes:

• Add module-level regex constants and fallback parsing in getHost(), getPath(), and getRemote() when urlObject is undefined.
• Add tests covering templated host, templated path, host+port, block tags, and no-path URLs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
packages/insomnia-scripting-environment/src/objects/urls.ts	Adds regex fallbacks to extract host/path/remote directly from the original string when native URL parsing is skipped.
packages/insomnia-scripting-environment/src/objects/tests/urls.test.ts	Adds unit tests validating fallback behavior for common templated URL shapes.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[lslvxy]

@copilot open a new pull request to apply changes based on the comments in this thread

[lslvxy]

@copilot open a new pull request to apply changes based on the comments in this thread