Embedded struct fields, field tag fix, code improvements

.php-cs-fixer.dist.php

@@ -4,12 +4,19 @@
 
 const CONFIG = new PhpCsFixer\Config();
 const RULES = [
-    '@PSR12' => true,
+    '@PER' => true,
     'strict_param' => true,
-    'braces' => false,
     'ternary_operator_spaces' => false,
     'no_break_comment' => false,
     'array_syntax' => ['syntax' => 'short'],
+    'no_unused_imports' => true,
+    'single_line_empty_body' => true,
+    'statement_indentation' => false,
+    'global_namespace_import' => [
+        'import_classes' => true,
+        'import_constants' => true,
+        'import_functions' => true,
+    ],
 ];
 
 CONFIG->setRules(RULES);

README.md

@@ -29,13 +29,13 @@ $errs = $parser->getErrors();
 ```
 Supported language level: Go 1.18 (generics).
 
-Parser is able to recover itself if a parse error occurs, in this case it will continue parsing at the closest node it is able to recognise.
+The parser is capable of recovering itself if a parse error occurs. In such cases, it will continue parsing at the closest node it can recognise.
 
-The resulting AST will be as full as possible, and you have to check `getErrors()` to see errors.
+The resulting Abstract Syntax Tree (AST) will be as complete as possible. You need to check `getErrors()` to identify any errors.
 
 ## Single declaration parsing
 
-Parser can also handle a single declaration only (e.g. a single function), instead of a fully defined Go program:
+The parser can also handle a single declaration (e.g., a single function) instead of an entire Go program:
 ```php
 use GoParser\{Parser, ParseMode};
 
@@ -51,11 +51,11 @@ $decl = $parser->parseSingleDecl();
 
 ## Abstract Syntax Tree
 
-Parsing results in an Abstract Syntax Tree result. See `src/Ast`.
+Parsing results in an Abstract Syntax Tree. Refer to `src/Ast` for details.
 
-For the most part the AST nodes structure follows closely the official Golang [specification][1].
+For the most part, the structure of AST nodes closely follows the official Golang [specification][1].
 
-Some Nodes may also have a bit different name (e.g. `ExpressionList` vs `ExprList`), but mostly the names are either the same or easily recognisable.
+Some nodes may have slightly different names (e.g., `ExpressionList` instead of `ExprList`), but in most cases, the names are the same or easily recognisable.
 
 ## CLI
 Package comes with a CLI command:

bin/go-parser

@@ -83,7 +83,7 @@ function parse_argv(array $argv): array
                 $mode = ParseMode::SingleDecl;
                 break;
             case '--node-dumper':
-                // default for now
+                // default
                 break;
             case '-d':
             case '--var-dump':

src/Ast/AstNode.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast;
 
-interface AstNode
-{
-}
+interface AstNode {}

src/Ast/CaseClause.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast;
 
-interface CaseClause extends AstNode
-{
-}
+interface CaseClause extends AstNode {}

src/Ast/CaseLabel.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast;
 
-interface CaseLabel extends AstNode
-{
-}
+interface CaseLabel extends AstNode {}

src/Ast/EmbeddedFieldDecl.php

@@ -0,0 +1,18 @@
+<?php
+
+declare(strict_types=1);
+
+namespace GoParser\Ast;
+
+use GoParser\Ast\Expr\PointerType;
+use GoParser\Ast\Expr\RawStringLit;
+use GoParser\Ast\Expr\StringLit;
+use GoParser\Ast\Expr\TypeName;
+
+final class EmbeddedFieldDecl implements AstNode
+{
+    public function __construct(
+        public readonly TypeName|PointerType $type,
+        public readonly StringLit|RawStringLit|null $tag,
+    ) {}
+}

src/Ast/Expr/Expr.php

@@ -6,6 +6,4 @@
 
 use GoParser\Ast\AstNode;
 
-interface Expr extends AstNode
-{
-}
+interface Expr extends AstNode {}

src/Ast/Expr/Literal.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface Literal extends Operand
-{
-}
+interface Literal extends Operand {}

src/Ast/Expr/Operand.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface Operand extends PrimaryExpr
-{
-}
+interface Operand extends PrimaryExpr {}

src/Ast/Expr/PrimaryExpr.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface PrimaryExpr extends Expr
-{
-}
+interface PrimaryExpr extends Expr {}

src/Ast/Expr/SliceExpr.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface SliceExpr extends PrimaryExpr
-{
-}
+interface SliceExpr extends PrimaryExpr {}

src/Ast/Expr/StructType.php

@@ -4,14 +4,15 @@
 
 namespace GoParser\Ast\Expr;
 
+use GoParser\Ast\EmbeddedFieldDecl;
 use GoParser\Ast\FieldDecl;
 use GoParser\Ast\Keyword;
 use GoParser\Ast\Punctuation;
 
 final class StructType implements TypeLit
 {
     /**
-     * @param FieldDecl[] $fieldDecls
+     * @param list<FieldDecl|EmbeddedFieldDecl> $fieldDecls
      */
     public function __construct(
         public readonly Keyword $keyword,

src/Ast/Expr/Type.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface Type extends TypeTerm, Expr
-{
-}
+interface Type extends TypeTerm, Expr {}

src/Ast/Expr/TypeLit.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface TypeLit extends Type
-{
-}
+interface TypeLit extends Type {}

src/Ast/Expr/TypeName.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface TypeName extends Type
-{
-}
+interface TypeName extends Type {}

src/Ast/Expr/TypeTerm.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Expr;
 
-interface TypeTerm
-{
-}
+interface TypeTerm {}

src/Ast/Expr/UnderlyingType.php

@@ -5,7 +5,6 @@
 namespace GoParser\Ast\Expr;
 
 use GoParser\Ast\Operator;
-use GoParser\Lexer\Position;
 
 final class UnderlyingType implements TypeTerm, Expr
 {

src/Ast/FieldDecl.php

@@ -11,8 +11,8 @@
 final class FieldDecl implements AstNode
 {
     public function __construct(
-        public readonly ?IdentList $identList,
-        public readonly ?Type $type,
+        public readonly IdentList $identList,
+        public readonly Type $type,
         public readonly StringLit|RawStringLit|null $tag,
     ) {}
 }

src/Ast/IdentList.php

@@ -4,12 +4,14 @@
 
 namespace GoParser\Ast;
 
+use GoParser\Ast\Expr\Type;
 use GoParser\Exception\InvalidArgument;
 use GoParser\Ast\Expr\Expr;
 use GoParser\Ast\Expr\Ident;
-use GoParser\Ast\Expr\Type;
 use GoParser\Ast\Expr\SingleTypeName;
 
+use function array_map;
+
 final class IdentList implements AstNode
 {
     /**
@@ -21,7 +23,7 @@ public function __construct(
 
     public static function fromExprList(ExprList $list): self
     {
-        return new self(\array_map(
+        return new self(array_map(
             static fn (Expr $expr): Ident => $expr instanceof Ident
                 ? $expr
                 : throw new InvalidArgument('Cannot create IdentList from an arbitrary expression list'),
@@ -31,7 +33,7 @@ public static function fromExprList(ExprList $list): self
 
     public static function fromTypeList(TypeList $list): self
     {
-        return new self(\array_map(
+        return new self(array_map(
             static fn (Type $type): Ident => $type instanceof SingleTypeName
                 ? $type->name
                 : throw new InvalidArgument('Cannot create IdentList from an arbitrary type list'),

src/Ast/SpecType.php

@@ -4,7 +4,9 @@
 
 namespace GoParser\Ast;
 
-enum SpecType implements \JsonSerializable
+use JsonSerializable;
+
+enum SpecType implements JsonSerializable
 {
     case Import;
     case Var;

src/Ast/Stmt/ConstDecl.php

@@ -10,14 +10,16 @@
 use GoParser\Ast\Keyword;
 use GoParser\Ast\SpecType;
 
+use function sprintf;
+
 final class ConstDecl implements Decl
 {
     public function __construct(
         public readonly Keyword $keyword,
         public readonly GroupSpec|ConstSpec $spec,
     ) {
         if ($spec->type() !== SpecType::Const) {
-            throw new InvalidArgument(\sprintf('Cannot create a ConstDecl with Spec of type %s', $spec->type()->name));
+            throw new InvalidArgument(sprintf('Cannot create a ConstDecl with Spec of type %s', $spec->type()->name));
         }
     }
 }

src/Ast/Stmt/Decl.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Stmt;
 
-interface Decl extends Stmt
-{
-}
+interface Decl extends Stmt {}

src/Ast/Stmt/IfStmt.php

@@ -14,14 +14,14 @@
 final class IfStmt implements Stmt
 {
     public function __construct(
-        public readonly Keyword $if,
+        public readonly Keyword $keyword,
         public readonly ?SimpleStmt $init,
         public readonly Expr $condition,
         public readonly BlockStmt $ifBody,
-        public readonly ?Keyword $else,
+        public readonly ?Keyword $elseKeyword,
         public readonly BlockStmt|self|null $elseBody,
     ) {
-        if ((bool) $else !== (bool) $elseBody) {
+        if ((bool) $elseKeyword !== (bool) $elseBody) {
             throw new InvalidArgument('Both "else" keyword and the body must be present or neither.');
         }
     }

src/Ast/Stmt/ImportDecl.php

@@ -10,14 +10,16 @@
 use GoParser\Ast\Keyword;
 use GoParser\Ast\SpecType;
 
+use function sprintf;
+
 final class ImportDecl implements Decl
 {
     public function __construct(
         public readonly Keyword $keyword,
         public readonly GroupSpec|ImportSpec $spec,
     ) {
         if ($spec->type() !== SpecType::Import) {
-            throw new InvalidArgument(\sprintf('Cannot create a ImportDecl with Spec of type %s', $spec->type()->name));
+            throw new InvalidArgument(sprintf('Cannot create a ImportDecl with Spec of type %s', $spec->type()->name));
         }
     }
 }

src/Ast/Stmt/SimpleStmt.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Stmt;
 
-interface SimpleStmt extends Stmt
-{
-}
+interface SimpleStmt extends Stmt {}

src/Ast/Stmt/Stmt.php

@@ -6,6 +6,4 @@
 
 use GoParser\Ast\AstNode;
 
-interface Stmt extends AstNode
-{
-}
+interface Stmt extends AstNode {}

src/Ast/Stmt/SwitchStmt.php

@@ -4,6 +4,4 @@
 
 namespace GoParser\Ast\Stmt;
 
-interface SwitchStmt extends Stmt
-{
-}
+interface SwitchStmt extends Stmt {}

src/Ast/Stmt/TypeDecl.php

@@ -10,14 +10,16 @@
 use GoParser\Ast\SpecType;
 use GoParser\Ast\TypeSpec;
 
+use function sprintf;
+
 final class TypeDecl implements Decl
 {
     public function __construct(
         public readonly Keyword $keyword,
         public readonly GroupSpec|TypeSpec $spec,
     ) {
         if ($spec->type() !== SpecType::Type) {
-            throw new InvalidArgument(\sprintf('Cannot create a TypeDecl with Spec of type %s', $spec->type()->name));
+            throw new InvalidArgument(sprintf('Cannot create a TypeDecl with Spec of type %s', $spec->type()->name));
         }
     }
 }

src/Ast/Stmt/VarDecl.php

@@ -10,14 +10,16 @@
 use GoParser\Ast\SpecType;
 use GoParser\Ast\VarSpec;
 
+use function sprintf;
+
 final class VarDecl implements Decl
 {
     public function __construct(
         public readonly Keyword $keyword,
         public readonly GroupSpec|VarSpec $spec,
     ) {
         if ($spec->type() !== SpecType::Var) {
-            throw new InvalidArgument(\sprintf('Cannot create a VarDecl with Spec of type %s', $spec->type()->name));
+            throw new InvalidArgument(sprintf('Cannot create a VarDecl with Spec of type %s', $spec->type()->name));
         }
     }
 }

src/Exception/InvalidArgument.php

@@ -4,7 +4,9 @@
 
 namespace GoParser\Exception;
 
-final class InvalidArgument extends \InvalidArgumentException
+use InvalidArgumentException;
+
+final class InvalidArgument extends InvalidArgumentException
 {
     public function __construct(string $message)
     {