LearnNewsExamplesServices
Frontmatter
id9585
titleBug: LivePreview popout windowUrl generates incorrectly in dist environments
stateClosed
labels
bugai
assigneestobiu
createdAtMar 27, 2026, 3:31 PM
updatedAtMar 27, 2026, 8:45 PM
githubUrlhttps://github.com/neomjs/neo/issues/9585
authortobiu
commentsCount1
parentIssuenull
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtMar 27, 2026, 4:09 PM

Bug: LivePreview popout windowUrl generates incorrectly in dist environments

Closed v12.1.0 bugai
tobiu
tobiu commented on Mar 27, 2026, 3:31 PM

Resolving the webpack basePath against location.href inside dist/development generates an incorrect absolute URL for windowUrl because the AppWorker is only 2 folders deep, whereas the HTML document is 4 folders deep, resulting in a 404 for MicroLoader.mjs.

We should return the relative URL directly and allow window.open natively to resolve it against the main thread's HTML document path. We also need to use Neo.windowConfigs?.[this.windowId] || Neo.config for basePath and useSharedWorkers instead of reading the global Neo.config.

tobiu added the bug label on Mar 27, 2026, 3:31 PM
tobiu added the ai label on Mar 27, 2026, 3:31 PM
tobiu referenced in commit 12b701f - "fix(code): Delete beforeSetWindowUrl manual resolution to leverage native browser relative URLs (#9585)" on Mar 27, 2026, 4:09 PM
tobiu
tobiu Mar 27, 2026, 4:09 PM

Input from Gemini 3.1 Pro (Antigravity):

✦ Fix implemented natively via commit 12b701ff8. The manual beforeSetWindowUrl string-math logic has been deleted.

We've confirmed that without this method, the Main thread evaluates relative URLs inherently based on the true physical window location, completely bypassing the SharedWorker basePath mapping complexities that caused the environment-stripping anomalies in dist/development. Initial macOS testing confirms that pop-outs from both the Home and Learn app routes resolve flawlessly.

To guarantee no recursive popout regressions or hidden OS edge cases resurface from the original implementation reason (#8074), an expansive review matrix has been scheduled in Analysis Ticket #9586.

tobiu closed this issue on Mar 27, 2026, 4:09 PM
tobiu assigned to @tobiu on Mar 27, 2026, 8:45 PM