Skip to content

Conversation

@Boy132
Copy link
Member

@Boy132 Boy132 commented Dec 23, 2025

Summary by CodeRabbit

Release Notes

  • Chores
    • Added automated validation for plugin configuration files in the CI/CD pipeline. Configuration validation now runs on every push to main and pull request, with detailed error reporting for any issues found.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 23, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A new validation system is introduced to automatically check plugin.json files in the repository. The system consists of a PHP script that recursively discovers and validates all plugin.json files, ensuring they meet structural requirements, combined with a GitHub Actions workflow that executes this validation on push to main and on pull requests.

Changes

Cohort / File(s) Summary
Validation Script
.github/scripts/validatePluginJson.php
Adds recursive directory traversal to locate all plugin.json files and validates each one. Implements findPluginJsonFiles() for file discovery and validateJsonFile() for JSON parsing and structure validation. Includes error handling for unreadable files and JSON parse errors. Exits with code 0 on success or 1 on validation failure.
CI/CD Workflow
.github/workflows/validate-plugin-json.yml
Introduces a new GitHub Actions workflow that triggers on push to main and pull requests. Sets up PHP 8.4 environment and executes the validation script to enforce plugin.json compliance.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hop, hop, JSON we check,
Plugins lined up—what a trek!
Scripts and workflows, oh so keen,
Validating files unseen. 🏗️
Errors caught before they're through,
Here's to standards tried and true!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch meta-workflow

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c255bea and 20759dd.

📒 Files selected for processing (2)
  • .github/scripts/validatePluginJson.php
  • .github/workflows/validate-plugin-json.yml

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.

@Boy132 Boy132 merged commit 73644b7 into main Dec 23, 2025
2 of 3 checks passed
@Boy132 Boy132 deleted the meta-workflow branch December 23, 2025 23:25
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.

2 participants