Greptile Summary

This PR hardens the sidebar plugin loading logic in src/sidebarApps/extensions/index.js by fixing two silent bugs and adding defensive error handling throughout.

Key changes:

• Bug fix — missing await on helpers.checkAPIStatus() in both searchPlugin and loadExplore: without await, the returned Promise object was always truthy, so if (!status) never fired and the API error UI was never shown to the user. Adding await makes the check work as intended.
• Hardened listInstalledPlugins: each plugin is now processed inside its own try/catch; unreadable plugin.json files log a warning and return null (filtered out at the end) instead of crashing the entire listing. Icon URI resolution is also independently guarded so a single broken icon does not abort the whole plugin entry.
• Improved error visibility in loadExplore with an added console.error on failure.

Confidence Score: 5/5

• Safe to merge — all changes are targeted bug fixes and defensive hardening with no new logic paths.
• The two await additions fix a clear, demonstrable bug (Promise is always truthy → API check was silently skipped). The listInstalledPlugins refactor is a strict improvement that converts a crash-on-any-error path into a warn-and-skip path. There is one minor follow-up (the outer lsDir call is still unguarded), but it is pre-existing and does not affect correctness of the new code.
• No files require special attention.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[searchPlugin / loadExplore triggered] --> B{await checkAPIStatus}
    B -- false --> C[Show api_error UI and return]
    B -- true --> D[Fetch plugins from API]
    D --> E[Call listInstalledPlugins]
    E --> F[await lsDir — unguarded]
    F --> G{For each item}
    G --> H{Read plugin.json}
    H -- error --> I[warn + return null]
    H -- ok --> J{plugin.icon exists?}
    J -- no --> K[Set installed=true, return plugin]
    J -- yes --> L{toInternalUri}
    L -- error --> M[warn, keep original icon path]
    L -- ok --> N[Set resolved icon]
    M --> K
    N --> K
    G --> G
    E --> O[filter out nulls]
    O --> P[Render plugin list items]

_{Reviews (1): Last reviewed commit: "fix(plugins): harden sidebar plugin load..." | Re-trigger Greptile}