MC-19366: Adds GraphQL sniffs

Magento2/Sniffs/GraphQL/AbstractGraphQLSniff.php

@@ -0,0 +1,76 @@
+<?php
+/**
+ * NOTICE OF LICENSE
+ *
+ * Copyright (c) 2019 TechDivision GmbH <[email protected]> - TechDivision GmbH
+ * All rights reserved
+ *
+ * This product includes proprietary software developed at TechDivision GmbH, Germany
+ * For more information see https://www.techdivision.com/
+ *
+ * To obtain a valid license for using this software please contact us at [email protected]
+ */
+
+namespace Magento2\Sniffs\GraphQL;
+
+use PHP_CodeSniffer\Sniffs\Sniff;
+
+/**
+ * Defines an abstract base class for GraphQL sniffs.
+ */
+abstract class AbstractGraphQLSniff implements Sniff
+{
+    /**
+     * Defines the tokenizers that this sniff is using.
+     *
+     * @var array
+     */
+    public $supportedTokenizers = ['GraphQL'];
+
+    /**
+     * Returns whether <var>$name</var> starts with a lower case character and is written in camel case.
+     *
+     * @param string $name
+     * @return bool
+     */
+    protected function isCamelCase($name)
+    {
+        return (preg_match('/^[a-z][a-zA-Z0-9]+$/', $name) !== 0);
+    }
+
+    /**
+     * Returns whether <var>$name</var> is specified in snake case (either all lower case or all upper case).
+     *
+     * @param string $name
+     * @param bool $upperCase If set to <kbd>true</kbd> checks for all upper case, otherwise all lower case
+     * @return bool
+     */
+    protected function isSnakeCase($name, $upperCase = false)
+    {
+        $pattern = $upperCase ? '/^[A-Z][A-Z0-9_]*$/' : '/^[a-z][a-z0-9_]*$/';
+        return preg_match($pattern, $name);
+    }
+
+    /**
+     * Searches for the first token that has <var>$tokenCode</var> in <var>$tokens</var> from position
+     * <var>$startPointer</var> (excluded).
+     *
+     * @param mixed $tokenCode
+     * @param array $tokens
+     * @param int $startPointer
+     * @return bool|int If token was found, returns its pointer, <kbd>false</kbd> otherwise
+     */
+    protected function seekToken($tokenCode, array $tokens, $startPointer = 0)
+    {
+        $numTokens = count($tokens);
+
+        for ($i = $startPointer + 1; $i < $numTokens; ++$i) {
+            if ($tokens[$i]['code'] === $tokenCode) {
+                return $i;
+            }
+        }
+
+        //if we came here we could not find the requested token
+        return false;
+    }
+}

Magento2/Sniffs/GraphQL/ValidArgumentNameSniff.php

@@ -0,0 +1,116 @@
+<?php
+/**
+ * Copyright © Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+
+namespace Magento2\Sniffs\GraphQL;
+
+use PHP_CodeSniffer\Files\File;
+
+/**
+ * Detects argument names that are not specified in <kbd>cameCase</kbd>.
+ */
+class ValidArgumentNameSniff extends AbstractGraphQLSniff
+{
+    /**
+     * @inheritDoc
+     */
+    public function register()
+    {
+        return [T_OPEN_PARENTHESIS];
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function process(File $phpcsFile, $stackPtr)
+    {
+        $tokens                  = $phpcsFile->getTokens();
+        $closeParenthesisPointer = $this->getCloseParenthesisPointer($stackPtr, $tokens);
+
+        //if we could not find the closing parenthesis pointer, we add a warning and terminate
+        if ($closeParenthesisPointer === false) {
+            $error = 'Possible parse error: Missing closing parenthesis for argument list in line %d';
+            $data  = [
+                $tokens[$stackPtr]['line'],
+            ];
+            $phpcsFile->addWarning($error, $stackPtr, 'UnclosedArgumentList', $data);
+            return;
+        }
+
+        $arguments = $this->getArguments($stackPtr, $closeParenthesisPointer, $tokens);
+
+        foreach ($arguments as $argument) {
+            $pointer = $argument[0];
+            $name    = $argument[1];
+
+            if (!$this->isCamelCase($name)) {
+                $type  = 'Argument';
+                $error = '%s name "%s" is not in CamelCase format';
+                $data  = [
+                    $type,
+                    $name,
+                ];
+
+                $phpcsFile->addError($error, $pointer, 'NotCamelCase', $data);
+                $phpcsFile->recordMetric($pointer, 'CamelCase argument name', 'no');
+            } else {
+                $phpcsFile->recordMetric($pointer, 'CamelCase argument name', 'yes');
+            }
+        }
+
+        //return stack pointer of closing parenthesis
+        return $closeParenthesisPointer;
+    }
+
+    /**
+     * Finds all argument names contained in <var>$tokens</var> in range <var>$startPointer</var> to
+     * <var>$endPointer</var>.
+     *
+     * @param int $startPointer
+     * @param int $endPointer
+     * @param array $tokens
+     * @return array[]
+     */
+    private function getArguments($startPointer, $endPointer, array $tokens)
+    {
+        $argumentTokenPointer = null;
+        $argument             = '';
+        $names                = [];
+        $skipTypes            = [T_COMMENT, T_WHITESPACE];
+
+        for ($i = $startPointer + 1; $i < $endPointer; ++$i) {
+            //skip some tokens
+            if (in_array($tokens[$i]['code'], $skipTypes)) {
+                continue;
+            }
+            $argument .= $tokens[$i]['content'];
+
+            if ($argumentTokenPointer === null) {
+                $argumentTokenPointer = $i;
+            }
+
+            if (preg_match('/^.+:.+$/', $argument)) {
+                list($name, $type) = explode(':', $argument);
+                $names[]              = [$argumentTokenPointer, $name];
+                $argument             = '';
+                $argumentTokenPointer = null;
+            }
+        }
+
+        return $names;
+    }
+
+    /**
+     * Seeks the next available token of type {@link T_CLOSE_PARENTHESIS} in <var>$tokens</var> and returns its pointer.
+     *
+     * @param int $stackPointer
+     * @param array $tokens
+     * @return bool|int
+     */
+    private function getCloseParenthesisPointer($stackPointer, array $tokens)
+    {
+        return $this->seekToken(T_CLOSE_PARENTHESIS, $tokens, $stackPointer);
+    }
+}

Magento2/Sniffs/GraphQL/ValidEnumValueSniff.php

@@ -0,0 +1,134 @@
+<?php
+/**
+ * Copyright © Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento2\Sniffs\GraphQL;
+
+use PHP_CodeSniffer\Files\File;
+
+/**
+ * Detects enum values that are not specified in <kbd>SCREAMING_SNAKE_CASE</kbd>.
+ */
+class ValidEnumValueSniff extends AbstractGraphQLSniff
+{
+
+    /**
+     * @inheritDoc
+     */
+    public function register()
+    {
+        return [T_CLASS];
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function process(File $phpcsFile, $stackPtr)
+    {
+        $tokens = $phpcsFile->getTokens();
+
+        //bail out if we're not inspecting an enum
+        if ($tokens[$stackPtr]['content'] !== 'enum') {
+            return;
+        }
+
+        $openingCurlyPointer = $this->getOpeningCurlyBracketPointer($stackPtr, $tokens);
+        $closingCurlyPointer = $this->getClosingCurlyBracketPointer($stackPtr, $tokens);
+
+        //if we could not find the closing curly bracket pointer, we add a warning and terminate
+        if ($openingCurlyPointer === false || $closingCurlyPointer === false) {
+            $error = 'Possible parse error: %s missing opening or closing brace';
+            $data  = [$tokens[$stackPtr]['content']];
+            $phpcsFile->addWarning($error, $stackPtr, 'MissingBrace', $data);
+            return;
+        }
+
+        $values = $this->getValues($openingCurlyPointer, $closingCurlyPointer, $tokens, $phpcsFile->eolChar);
+
+        foreach ($values as $value) {
+            $pointer = $value[0];
+            $name    = $value[1];
+
+            if (!$this->isSnakeCase($name, true)) {
+                $type  = 'Enum value';
+                $error = '%s "%s" is not in SCREAMING_SNAKE_CASE format';
+                $data  = [
+                    $type,
+                    $name,
+                ];
+
+                $phpcsFile->addError($error, $pointer, 'NotScreamingSnakeCase', $data);
+                $phpcsFile->recordMetric($pointer, 'SCREAMING_SNAKE_CASE enum value', 'no');
+            } else {
+                $phpcsFile->recordMetric($pointer, 'SCREAMING_SNAKE_CASE enum value', 'yes');
+            }
+        }
+
+        return $closingCurlyPointer;
+    }
+
+    /**
+     * Seeks the next available token of type {@link T_CLOSE_CURLY_BRACKET} in <var>$tokens</var> and returns its
+     * pointer.
+     *
+     * @param int $startPointer
+     * @param array $tokens
+     * @return bool|int
+     */
+    private function getClosingCurlyBracketPointer($startPointer, array $tokens)
+    {
+        return $this->seekToken(T_CLOSE_CURLY_BRACKET, $tokens, $startPointer);
+    }
+
+    /**
+     * Seeks the next available token of type {@link T_OPEN_CURLY_BRACKET} in <var>$tokens</var> and returns its
+     * pointer.
+     *
+     * @param $startPointer
+     * @param array $tokens
+     * @return bool|int
+     */
+    private function getOpeningCurlyBracketPointer($startPointer, array $tokens)
+    {
+        return $this->seekToken(T_OPEN_CURLY_BRACKET, $tokens, $startPointer);
+    }
+
+    /**
+     * Finds all enum values contained in <var>$tokens</var> in range <var>$startPointer</var> to
+     * <var>$endPointer</var>.
+     *
+     * @param int $startPointer
+     * @param int $endPointer
+     * @param array $tokens
+     * @param string $eolChar
+     * @return array[]
+     */
+    private function getValues($startPointer, $endPointer, array $tokens, $eolChar)
+    {
+        $valueTokenPointer = null;
+        $enumValue         = '';
+        $values            = [];
+        $skipTypes         = [T_COMMENT, T_WHITESPACE];
+
+        for ($i = $startPointer + 1; $i < $endPointer; ++$i) {
+            //skip some tokens
+            if (in_array($tokens[$i]['code'], $skipTypes)) {
+                continue;
+            }
+            $enumValue .= $tokens[$i]['content'];
+
+            if ($valueTokenPointer === null) {
+                $valueTokenPointer = $i;
+            }
+
+            if (strpos($enumValue, $eolChar) !== false) {
+                $values[]          = [$valueTokenPointer, rtrim($enumValue, $eolChar)];
+                $enumValue         = '';
+                $valueTokenPointer = null;
+            }
+        }
+
+        return $values;
+    }
+}

Magento2/Sniffs/GraphQL/ValidFieldNameSniff.php

@@ -0,0 +1,44 @@
+<?php
+/**
+ * Copyright © Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento2\Sniffs\GraphQL;
+
+use PHP_CodeSniffer\Files\File;
+
+/**
+ * Detects field names the are not specified in <kbd>snake_case</kbd>.
+ */
+class ValidFieldNameSniff extends AbstractGraphQLSniff
+{
+    /**
+     * @inheritDoc
+     */
+    public function register()
+    {
+        return [T_VARIABLE];
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function process(File $phpcsFile, $stackPtr)
+    {
+        $tokens = $phpcsFile->getTokens();
+        $name   = $tokens[$stackPtr]['content'];
+
+        if (!$this->isSnakeCase($name)) {
+            $type  = 'Field';
+            $error = '%s name "%s" is not in snake_case format';
+            $data  = [
+                $type,
+                $name,
+            ];
+            $phpcsFile->addError($error, $stackPtr, 'NotSnakeCase', $data);
+            $phpcsFile->recordMetric($stackPtr, 'SnakeCase field name', 'no');
+        } else {
+            $phpcsFile->recordMetric($stackPtr, 'SnakeCase field name', 'yes');
+        }
+    }
+}