Skip to content

fix(build): default to webpack worker resolving - #1852

Open
Antreesy wants to merge 1 commit into
mainfrom
fix/noid/e2ee-calls-fix
Open

fix(build): default to webpack worker resolving#1852
Antreesy wants to merge 1 commit into
mainfrom
fix/noid/e2ee-calls-fix

Conversation

@Antreesy

@Antreesy Antreesy commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

☑️ Resolves

🖼️ Screenshots

🏚️ Before 🏡 After
B A
2026-08-07_12h35_01 2026-08-07_12h34_15

- after build worker file was sitting at 'talk_desktop__dist/assets/'
- webpack now resolves new Worker() from relative path and can not find a worker

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Maksim Sukharev <antreesy.web@gmail.com>
@Antreesy Antreesy added this to the ☀️ 26 Summer milestone Aug 7, 2026
@Antreesy
Antreesy requested a review from ShGKme August 7, 2026 10:41
@Antreesy Antreesy self-assigned this Aug 7, 2026
@Antreesy Antreesy added bug Something isn't working regression AI assisted This PR contains AI-assisted commits labels Aug 7, 2026
@ShGKme

ShGKme commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Could you elaborate the solution?

I understand that worker-loader isn't needed in modern setup and likely should be removed anyway.
But I don't understand how the change fixes the bug and why relative path is the issue...

Before/After screenshots are from the web-client, not the Desktop client.
Is it a screenshot of a call with a user from the desktop client?

@Antreesy

Antreesy commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

Before/After screenshots are from the web-client, not the Desktop client.
Is it a screenshot of a call with a user from the desktop client?

Yes, incoming video stream is from desktop client

From my understanding, worker is loaded (otherwise we'd see an error), but post and listen messages between worker and e2ee module are incorrectly established

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI assisted This PR contains AI-assisted commits bug Something isn't working regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blank screen without audio in desktop app, but working in talk web page

2 participants