chore(deps-dev): bump electron from 43.1.1 to 43.2.0 in /frontend

[dependabot]

Bumps electron from 43.1.1 to 43.2.0.

Release notes

Sourced from electron's releases.

electron v43.2.0

Release Notes for v43.2.0

Features

• Added net.WebSocket, a WHATWG-compatible WebSocket client for the main process that routes through Chromium's network stack. #52344 (Also in 42)

Fixes

• Fixed app.disableHardwareAcceleration() not fully disabling GPU hardware usage on Windows starting from Electron 38. #52369 (Also in 41, 42, 44)
• Fixed globalShortcut not working on GNOME Wayland by enabling Chromium's GlobalShortcutsPortalPreferredTrigger feature by default on Linux. #52228
• Fixed a crash when showing Linux message boxes while another Linux UI implementation was active. #52410 (Also in 42, 44)
• Fixed a window being marked hidden and background-throttled on macOS when covered by a transparent or click-through window. #52389 (Also in 44)
• Fixed unnecessary autofill popup creation for fields without datalist suggestions, which could cause input latency on macOS. #52321 (Also in 41, 42, 44)
• Fixed webContents.print() returning "Invalid printer settings" when print options are provided. #52381 (Also in 44)

Other Changes

• Updated Chromium to 150.0.7871.129. #52355

Commits

• 9b58e96 fix: use GTK UI theme for Linux message boxes (#52410)
• 281d885 fix: invalid printer settings when settings passed (#52381)
• eddd87c fix: don't treat transparent overlays as occluders on macOS (#52389)
• 2a35b07 test: make webContents.clone() process-id test offline (#52377)
• 4675b83 chore: bump chromium to 150.0.7871.129 (43-x-y) (#52355)
• 8d1cc07 feat: add net.WebSocket (#52344)
• f7b6b2c fix: append --disable-gpu switch in app.disableHardwareAcceleration() (#52369)
• 6a44af0 refactor: migrate electron::api::DesktopCapturer to cppgc (#52372)
• edf129f fix: enable GlobalShortcutsPortalPreferredTrigger by default on Linux (#52228)
• e04df3b build: add missing pdf resources dep in chromium_src (#52340)
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[nvidia-code-review-bot]

🤖 Automated Code Review

Summary: 🟡 2 warnings · 🔵 3 suggestions

---

Code Review

Overall, the codebase looks well-structured and clean. However, I've identified a few issues that need attention:

[WARNING] backend/app/services/agent_sandbox.py:123
Issue: The ensure_sandbox function does not handle the case where the Docker daemon is unavailable, but the teardown function does. This inconsistency might lead to unexpected behavior.
Why it matters: Inconsistent error handling can cause issues in production.
Fix:

async def ensure_sandbox(user_id: int, agent_id: int | None) -> Optional[str]:
    try:
        # ...
    except DockerUnavailable as exc:
        log.info("agent_sandbox: docker unavailable, host fallback: %s", exc)
        return None
    except Exception as exc:
        log.warning("agent_sandbox: ensure failed, host fallback: %s", exc)
        return None

[WARNING] backend/app/tools/terminal_tools.py:105
Issue: The execute_terminal_command function does not handle the case where the approved variable is None.
Why it matters: Unhandled None values can cause unexpected behavior.
Fix:

if approved is None:
    return "Terminal Execution Request was REJECTED by user."

[SUGGESTION] backend/app/services/agent_sandbox.py:283
Issue: The agent_sandbox module has a lot of duplicated code for handling Docker errors. Consider extracting a separate function for error handling.
Why it matters: Duplicated code makes maintenance harder.
Fix:

def _handle_docker_error(exc: Exception) -> None:
    log.warning("agent_sandbox: docker error: %s", exc)

# ...

try:
    # ...
except DockerUnavailable as exc:
    _handle_docker_error(exc)

[SUGGESTION] backend/tests/test_auth.py:16
Issue: The test_register_user function does not check for the presence of the refresh_token in the response.
Why it matters: Incomplete testing can lead to issues in production.
Fix:

assert "refresh_token" in data

[SUGGESTION] backend/tests/test_tools.py:166
Issue: The test_forget_tool_parameters function does not test the case where the tool_parameters are None.
Why it matters: Incomplete testing can lead to issues in production.
Fix:

def test_forget_tool_parameters_none():
    # ...
    tool_parameters = None
    # ...

Overall, the codebase looks good, and these issues are minor. However, addressing them will improve the overall quality and reliability of the code.

---

_{Powered by NVIDIA NIM · meta/llama-3.1-70b-instruct}

[dependabot]

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an ignore condition with the desired update_types to your config file.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.