Add Agent Ready Content integration loader - #7250
shekharnwagh wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Add the WordPress-version guard and success-path test coverage.
Get a fresh assessment by requesting another Copilot review.
Pull request overview
Adds an Agent Ready Content integration loader that selects the latest release, honors customer-loaded copies, and remains inactive when unavailable.
Changes:
- Registers the conditional integration loader.
- Implements release discovery and loading precedence.
- Adds focused integration tests.
File summaries
| File | Summary |
|---|---|
vip-integrations.php |
Includes and registers the integration loader. |
tests/integrations/test-agent-ready-content.php |
Tests loaded and unavailable states; the success path lacks coverage (nit, 1 vote). |
integrations/agent-ready-content.php |
Implements release loading; requires a WordPress-version guard before loading 6.8+ releases (critical, 2 votes). |
Review details
Suppressed comments (1)
integrations/agent-ready-content.php:49
- The added tests cover only the already-loaded and unavailable branches; they never exercise this success path. A wrong entry-file path or a failure to require the latest available release would therefore still pass the suite. Please add a valid-release fixture/mock and assert that the entry file is loaded and the integration remains active.
require_once $load_path;
- Files reviewed: 3/3 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #7250 +/- ##
=============================================
+ Coverage 37.65% 37.69% +0.03%
- Complexity 5477 5484 +7
=============================================
Files 303 304 +1
Lines 21875 21899 +24
=============================================
+ Hits 8238 8255 +17
- Misses 13637 13644 +7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Direct loading bypasses WordPress's plugin requirement checks and can load Agent Ready Content on unsupported versions. Mark the integration inactive before loading when WordPress is older than 6.8.
41de993 to
8d62cef
Compare
|



Description
Adds Agent Ready Content to the VIP integration framework. The loader selects the latest available release, gives precedence to customer-loaded copies, and stays inactive when the release files are missing.
Changelog Description
Added
Pre-review checklist
Pre-deploy checklist
Steps to Test
Use WordPress 6.8+ and PHP 8.2+, with no customer-loaded copy of Agent Ready Content. Replace the placeholders below with your checkout paths and environment slug.
Check out this PR in
vip-go-mu-pluginsand copy the released plugin fromvip-go-mu-plugins-ext:Configure your dev-env to use this checkout, then start it. For a new environment, replace
updatewithcreate.Add this entry to
~/.local/share/vip/dev-environment/<env-slug>/integrations-config/integrations.json, preserving other integration entries:{ "agent-ready-content": { "env": { "status": "enabled" } } }If
XDG_DATA_HOMEis set, use that directory instead of~/.local/share. Changes apply on the next request.Verify that
is_activeandis_loadedare bothtrue:Open Settings → Agent Ready Content in WordPress Admin. Verify that a published post serves Markdown through
.md,/markdown, and?markdown=true, and that/llms.txtloads. The normal post URL should still serve HTML.Change this integration's
env.statusto"disabled"inintegrations.json. Repeat the status command and confirm that both fields arefalseand the settings page is unavailable.Run the focused tests from the
vip-go-mu-pluginscheckout: