Skip to content

Comments

fix(screenshots): correct plugin mount path for Lidarr plugins branch#329

Closed
RicherTunes wants to merge 5 commits intomainfrom
fix/screenshot-workflow-plugin-path
Closed

fix(screenshots): correct plugin mount path for Lidarr plugins branch#329
RicherTunes wants to merge 5 commits intomainfrom
fix/screenshot-workflow-plugin-path

Conversation

@RicherTunes
Copy link
Owner

Summary

  • Fixed plugin mount path from /config/plugins/Brainarr/ to /config/plugins/RicherTunes/Brainarr/
  • Updated API verification to use /api/v1/importlist/schema instead of /api/v1/system/plugins

Root Cause

Lidarr's plugins branch expects plugins at:

/config/plugins/[Owner]/[PluginName]/Lidarr.Plugin.*.dll

The previous mount path was missing the [Owner] directory level, causing plugins not to load.

Verification

Tested locally with Docker - plugin loads and registers correctly:

[Debug] Brainarr: Rate limiter configured for ollama: 30 requests per 60s
[Debug] Brainarr: LibraryAwarePromptBuilder instance created

Technical Details

  • PathExtensions.GetPluginAssemblies() scans: [AppDataFolder]/plugins/[Owner]/[PluginName]/Lidarr.Plugin.*.dll
  • /api/v1/system/plugins only shows API-installed plugins, not file-loaded ones
  • /api/v1/importlist/schema correctly shows file-loaded plugins

🤖 Generated with Claude Code

RicherTunes and others added 4 commits December 8, 2025 08:39
The Lidarr plugins branch expects plugins at:
  /config/plugins/[Owner]/[PluginName]/Lidarr.Plugin.*.dll

Previously mounted to /config/plugins/Brainarr/ (missing Owner directory)
Now mounts to /config/plugins/RicherTunes/Brainarr/

Also updated API verification to use /api/v1/importlist/schema
instead of /api/v1/system/plugins (which only shows API-installed plugins).

Verified locally with Docker - plugin loads and registers correctly.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
…sion conflicts

Updates to commit 415c6b1 which includes the ILRepack fix that merges
Microsoft.Extensions assemblies (DependencyInjection, Caching, Options, etc.)
to prevent version conflicts with Lidarr's older MS.Extensions versions.

This fixes the DryIoc resolution crashes caused by incompatible
MS.Extensions.DependencyInjection.Abstractions versions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Updates Common submodule to include FluentValidation in ILRepack merge
list, preventing version conflicts with Lidarr host.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@RicherTunes
Copy link
Owner Author

Stale — superseded by packaging-gates adoption (#397) and ecosystem standardization.

@RicherTunes RicherTunes closed this Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant