Implement Readme Parser

composer.json

@@ -8,7 +8,8 @@
     "composer/installers": "^v1.12.0 || ^2.2",
     "dealerdirect/phpcodesniffer-composer-installer": "^0.4.1 || ^0.5 || ^0.6.2 || ^0.7 || ^1.0",
     "wp-coding-standards/wpcs": "^3.0.0",
-    "automattic/vipwpcs": "^3.0.0"
+    "automattic/vipwpcs": "^3.0.0",
+    "afragen/wordpress-plugin-readme-parser": "dev-master"
   },
   "require-dev": {
     "wp-cli/extension-command": "^2.1",

includes/Checker/Checks/Plugin_Readme_Check.php

@@ -12,6 +12,7 @@
 use WordPress\Plugin_Check\Traits\Amend_Check_Result;
 use WordPress\Plugin_Check\Traits\Find_Readme;
 use WordPress\Plugin_Check\Traits\Stable_Check;
+use WordPressdotorg\Plugin_Directory\Readme\Parser;
 
 /**
  * Check the plugins readme file and contents.
@@ -69,42 +70,48 @@
 			return;
 		}
 
+		$readme_file = reset( $readme );
+
+		$parser = new Parser( $readme_file );
+
 		// Check the readme file for default text.
-		$this->check_default_text( $result, $readme );
+		$this->check_default_text( $result, $readme_file, $parser );
 
 		// Check the readme file for a valid license.
-		$this->check_license( $result, $readme );
+		$this->check_license( $result, $readme_file, $parser );
 
 		// Check the readme file for a valid version.
-		$this->check_stable_tag( $result, $readme );
+		$this->check_stable_tag( $result, $readme_file, $parser );
+
+		// Check the readme file for warnings.
+		$this->check_for_warnings( $result, $readme_file, $parser );
 	}
 
 	/**
 	 * Checks the readme file for default text.
 	 *
 	 * @since n.e.x.t
 	 *
-	 * @param Check_Result $result The Check Result to amend.
-	 * @param array        $files  Array of plugin files.
+	 * @param Check_Result $result      The Check Result to amend.
+	 * @param string       $readme_file Readme file.
+	 * @param Parser       $parser      The Parser object.
 	 */
-	private function check_default_text( Check_Result $result, array $files ) {
-		$default_text_patterns = array(
-			'Here is a short description of the plugin.',
-			'Tags: tag1',
-			'Donate link: http://example.com/',
-		);
+	private function check_default_text( Check_Result $result, string $readme_file, Parser $parser ) {
+		$short_description = $parser->short_description;
+		$tags              = $parser->tags;
+		$donate_link       = $parser->donate_link;
 
-		foreach ( $default_text_patterns as $pattern ) {
-			$file = self::file_str_contains( $files, $pattern );
-			if ( $file ) {
-				$this->add_result_warning_for_file(
-					$result,
-					__( 'The readme appears to contain default text.', 'plugin-check' ),
-					'default_readme_text',
-					$file
-				);
-				break;
-			}
+		if (
+			in_array( 'tag1', $tags, true )
+			|| str_contains( $short_description, 'Here is a short description of the plugin.' )
+			|| str_contains( $donate_link, '//example.com/' )
+		) {
+			$this->add_result_warning_for_file(
+				$result,
+				__( 'The readme appears to contain default text.', 'plugin-check' ),
+				'default_readme_text',
+				$readme_file
+			);
 		}
 	}
 
@@ -113,25 +120,20 @@
 	 *
 	 * @since n.e.x.t
 	 *
-	 * @param Check_Result $result The Check Result to amend.
-	 * @param array        $files  Array of plugin files.
+	 * @param Check_Result $result      The Check Result to amend.
+	 * @param string       $readme_file Readme file.
+	 * @param Parser       $parser      The Parser object.
 	 */
-	private function check_license( Check_Result $result, array $files ) {
-		$matches = array();
-		// Get the license from the readme file.
-		$file = self::file_preg_match( '/(License:|License URI:)\s*(.+)*/i', $files, $matches );
-
-		if ( empty( $matches ) ) {
-			return;
-		}
+	private function check_license( Check_Result $result, string $readme_file, Parser $parser ) {
+		$license = $parser->license;
 
 		// Test for a valid SPDX license identifier.
-		if ( ! preg_match( '/^([a-z0-9\-\+\.]+)(\sor\s([a-z0-9\-\+\.]+))*$/i', $matches[2] ) ) {
+		if ( ! empty( $license ) && ! preg_match( '/^([a-z0-9\-\+\.]+)(\sor\s([a-z0-9\-\+\.]+))*$/i', $license ) ) {
 			$this->add_result_warning_for_file(
 				$result,
 				__( 'Your plugin has an invalid license declared. Please update your readme with a valid SPDX license identifier.', 'plugin-check' ),
 				'invalid_license',
-				$file
+				$readme_file
 			);
 		}
 	}
@@ -141,25 +143,19 @@
 	 *
 	 * @since n.e.x.t
 	 *
-	 * @param Check_Result $result The Check Result to amend.
-	 * @param array        $files  Array of plugin files.
+	 * @param Check_Result $result      The Check Result to amend.
+	 * @param string       $readme_file Readme file.
+	 * @param Parser       $parser      The Parser object.
 	 */
-	private function check_stable_tag( Check_Result $result, array $files ) {
-		$matches = array();
-		// Get the Stable tag from readme file.
-		$file = self::file_preg_match( '/Stable tag:\s*([a-z0-9\.]+)/i', $files, $matches );
-		if ( ! $file ) {
-			return;
-		}
-
-		$stable_tag = isset( $matches[1] ) ? $matches[1] : '';
+	private function check_stable_tag( Check_Result $result, string $readme_file, Parser $parser ) {
+		$stable_tag = $parser->stable_tag;
 
 		if ( 'trunk' === $stable_tag ) {
 			$this->add_result_error_for_file(
 				$result,
 				__( "It's recommended not to use 'Stable Tag: trunk'.", 'plugin-check' ),
 				'trunk_stable_tag',
-				$file
+				$readme_file
 			);
 		}
 
@@ -174,7 +170,51 @@
 				$result,
 				__( 'The Stable Tag in your readme file does not match the version in your main plugin file.', 'plugin-check' ),
 				'stable_tag_mismatch',
-				$file
+				$readme_file
+			);
+		}
+	}
+
+	/**
+	 * Checks the readme file warnings.
+	 *
+	 * @since n.e.x.t
+	 *
+	 * @param Check_Result $result      The Check Result to amend.
+	 * @param string       $readme_file Readme file.
+	 * @param Parser       $parser      The Parser object.
+	 */
+	private function check_for_warnings( Check_Result $result, string $readme_file, Parser $parser ) {
+		$warnings = $parser->warnings ? $parser->warnings : array();
+
+		$warning_keys = array_keys( $warnings );
+
+		$ignored_warnings = array(
+			'contributor_ignored',
+		);
+
+		/**
+		 * Filter the list of ignored readme parser warnings.
+		 *
+		 * @since n.e.x.t
+		 *
+		 * @param array  $ignored_warnings Array of ignored warning keys.
+		 * @param Parser $parser           The Parser object.
+		 */
+		$ignored_warnings = (array) apply_filters( 'wp_plugin_check_ignored_readme_warnings', $ignored_warnings, $parser );
+
+		$warning_keys = array_diff( $warning_keys, $ignored_warnings );
+
+		if ( ! empty( $warning_keys ) ) {
+			$this->add_result_warning_for_file(
+				$result,
+				sprintf(
+					/* translators: list of warnings */
+					esc_html__( 'The following readme parser warnings were detected: %s', 'plugin-check' ),
+					esc_html( implode( ', ', $warning_keys ) )
+				),
+				'readme_parser_warnings',
+				$readme_file
 			);
 		}
 	}

phpstan.neon.dist

@@ -19,5 +19,6 @@ parameters:
       message: '/^Function str_contains not found.$/'
       paths:
         - includes/Checker/Checks/Abstract_File_Check.php
+        - includes/Checker/Checks/Plugin_Readme_Check.php
         - includes/Traits/Find_Readme.php
         - includes/Traits/File_Editor_URL.php

tests/phpunit/testdata/plugins/test-plugin-plugin-readme-parser-warnings/load.php

@@ -0,0 +1,16 @@
+<?php
+/**
+ * Plugin Name: Test Plugin Readme Errors (Parser Warnings)
+ * Plugin URI: https://github.com/WordPress/plugin-check
+ * Description: Test plugin for the Readme check.
+ * Requires at least: 6.0
+ * Requires PHP: 5.6
+ * Version: 1.0.0
+ * Author: WordPress Performance Team
+ * Author URI: https://make.wordpress.org/performance/
+ * License: GPLv2 or later
+ * License URI: https://www.gnu.org/licenses/old-licenses/gpl-2.0.html
+ * Text Domain: test-plugin-check-errors-parser-warnings
+ *
+ * @package test-plugin-check-errors-parser-warnings
+ */

tests/phpunit/testdata/plugins/test-plugin-plugin-readme-parser-warnings/readme.txt

@@ -0,0 +1,13 @@
+
+=== Test Plugin Readme Errors Parser Warnings ===
+
+Contributors:      plugin-check
+Requires at least: 6.0
+Tested up to:      6.1
+Requires PHP:      PHP 5.6
+Stable tag:        1.0.0
+License:           GPLv2 or later
+License URI:       https://www.gnu.org/licenses/gpl-2.0.html
+Tags:              testing, security
+
+Plugin description.