Skip to content

Fix empty PluginConfig.xml causing dialog error#3

Merged
simons-plugins merged 1 commit into
mainfrom
fix/pluginconfig-empty-field
Feb 24, 2026
Merged

Fix empty PluginConfig.xml causing dialog error#3
simons-plugins merged 1 commit into
mainfrom
fix/pluginconfig-empty-field

Conversation

@simons-plugins

@simons-plugins simons-plugins commented Feb 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add a label field to PluginConfig.xml so Indigo's config dialog has at least one <Field> element
  • Fixes PAXDialogControllerError -- Dialogs must contain at least one <Field> element on plugin enable/configure
  • Bumps version to 2026.0.3

Test plan

  • Install plugin in Indigo
  • Open plugin config dialog — should show info label without error
  • Verify plugin starts and endpoints respond normally

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores

    • Updated plugin version to 2026.0.3
  • Documentation

    • Added informational message in plugin configuration explaining the plugin's JSON API exposure capabilities.

Add a label field to PluginConfig.xml so Indigo's config dialog has at
least one Field element. Fixes PAXDialogControllerError on plugin enable.

Bump version to 2026.0.3.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Feb 24, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 04a3464 and d476f0c.

📒 Files selected for processing (2)
  • LogsOverReflector.indigoPlugin/Contents/Info.plist
  • LogsOverReflector.indigoPlugin/Contents/Server Plugin/PluginConfig.xml

📝 Walkthrough

Walkthrough

Version bump from 2026.0.2 to 2026.0.3 and addition of an informational UI label to the plugin configuration explaining LogsOverReflector's functionality and absence of configuration options.

Changes

Cohort / File(s) Summary
Version Metadata Update
LogsOverReflector.indigoPlugin/Contents/Info.plist
Updated PluginVersion and CFBundleVersion from 2026.0.2 to 2026.0.3.
Plugin Configuration UI
LogsOverReflector.indigoPlugin/Contents/Server Plugin/PluginConfig.xml
Added static informational Field element explaining that LogsOverReflector has no configuration options and exposes the Indigo event log as a JSON API over the Reflector.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop, a skip, the version does climb,
From point-two to three, such a fine time!
A label arrives, so helpful and bright,
"No config needed," it states just right!
LogsOverReflector hops through the night! 🌙

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/pluginconfig-empty-field

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.

❤️ Share

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

@simons-plugins simons-plugins merged commit 3d6e148 into main Feb 24, 2026
2 of 3 checks passed
@simons-plugins simons-plugins deleted the fix/pluginconfig-empty-field branch February 24, 2026 19:15
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