Walkthrough

The plugin configuration loader in server/plugin_utils.py now wraps per-plugin config.json parsing in try/except. On FileNotFoundError, JSONDecodeError, or other Exceptions, it logs the error and skips that plugin. The selection logic for which plugins to consider and the final sorting/return behavior remain the same.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant S as Server
  participant PU as PluginUtils.get_plugins_configs
  participant FS as FileSystem
  participant J as JSON Parser
  participant L as Logger

  S->>PU: get_plugins_configs()
  loop For each candidate plugin
    PU->>FS: Read plugin/config.json
    alt File present
      FS-->>PU: File contents
      PU->>J: Parse JSON
      alt JSON valid
        J-->>PU: Config object
        PU-->>PU: Append to pluginsList
      else JSONDecodeError
        J-->>PU: Error
        PU->>L: Log JSON decode error
        PU-->>PU: Skip plugin
      end
    else FileNotFoundError
      FS-->>PU: Error
      PU->>L: Log missing config
      PU-->>PU: Skip plugin
    end
    opt Unexpected exception
      PU->>L: Log unexpected error
      PU-->>PU: Skip plugin
    end
  end
  PU-->>S: Sorted pluginsList

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hopped through configs, nimble and bright,
When files go missing, I don’t take fright.
If JSON is tangled, I log, then I glide—
Skipping the snares with a whiskered pride.
Now plugins align, neat as a row—
Onward I bound, ready to go! 🐇✨

Pre-merge checks and finishing touches

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}