Skip to content

feat: list plugins with version, activation and update state - #1

Open
ShunLuk wants to merge 2 commits into
mainfrom
feat/list-plugins
Open

feat: list plugins with version, activation and update state#1
ShunLuk wants to merge 2 commits into
mainfrom
feat/list-plugins

Conversation

@ShunLuk

@ShunLuk ShunLuk commented Sep 3, 2026

Copy link
Copy Markdown
Member

No description provided.

@ShunLuk
ShunLuk requested review from a team, Rovasch, SimonvanWijhe, dtakken, ictbeheer, mvdhoek1 and rivanuff and a lite review from Copilot and removed request for a team September 3, 2026 10:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new plugins data function currently has runtime-breaking issues (missing return for an array-typed function and unsafe transient dereference) that can fatal the REST endpoint.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds a new REST API endpoint to expose an inventory of installed plugins, including version, activation state (site/network), and whether updates are available, complementing the existing relay/v1/core diagnostics endpoint.

Changes:

  • Add relay_get_plugins() to collect plugin metadata plus activation and update availability details.
  • Register a new relay/v1/plugins GET endpoint and implement relay_rest_plugins() to return that data.
File summaries
File Description
includes/relay-data.php Adds plugin inventory collection logic (name/version/active/network active/update info).
includes/relay-api.php Registers the plugins REST route and implements the REST callback to return plugin data.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 4
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread includes/relay-data.php
Comment thread includes/relay-data.php Outdated
Comment thread includes/relay-data.php
Comment thread includes/relay-api.php Outdated
Comment thread includes/relay-data.php Outdated
Comment thread includes/relay-data.php Outdated
@SimonvanWijhe

Copy link
Copy Markdown
Member

Wil je de Features-lijst in readme.txt ook aanvullen met de pluginlijst?

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.

3 participants