Remove open dependency from AltTextSidebar useEffect to prevent redundant API calls

[Copilot]

The alt-text sidebar was re-fetching from the multinet API on every open/close toggle. The useEffect hook had open in its dependency array, causing unnecessary regeneration when only visibility changed.

Changes:

• Removed open from useEffect dependencies in AltTextSidebar.tsx
• Removed if (open) guard, allowing generation to occur regardless of sidebar visibility state
• Alt text now regenerates only when currState or generateAltText changes

The MUI Drawer with variant="persistent" keeps the component mounted when closed, so alt text persists across toggles without re-fetching.

// Before
useEffect(() => {
  setAltText(null);
  if (open) generate();
}, [currState, generateAltText, open]);

// After
useEffect(() => {
  setAltText(null);
  generate();
}, [currState, generateAltText]);

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• https://storage.googleapis.com/chrome-for-testing-public/143.0.7499.4/linux64/chrome-headless-shell-linux64.zip
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/upset2/upset2/node_modules/playwright-core/lib/server/registry/oopDownloadBrowserMain.js (http block)
• https://storage.googleapis.com/chrome-for-testing-public/143.0.7499.4/linux64/chrome-linux64.zip
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/upset2/upset2/node_modules/playwright-core/lib/server/registry/oopDownloadBrowserMain.js (http block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>alt-text panel toggle always triggers a re-request to the multinet API</issue_title>
<issue_description>Should probably use hidden instead to avoid unmounting the component</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes alt-text panel toggle always triggers a re-request to the multinet API #587

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[netlify]

✅ Deploy Preview for upset2 ready!

Name	Link
🔨 Latest commit	48151ce
🔍 Latest deploy log	https://app.netlify.com/projects/upset2/deploys/6966773b1753490008afc4ac
😎 Deploy Preview	https://deploy-preview-592--upset2.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.