refactor: remove psalm immutability

inc/functions.php

@@ -12,7 +12,6 @@
 /**
  * Creates a new column term.
  *
- * @psalm-pure
  * @param string|ColumnTerm|Table|TableRef $arg1 The column name or term if only 1 arg is given. Otherwise, the table
  *                                               name, instance, or reference.
  * @param string|null $arg2 The column name if the 1st arg is a table name, instance, or reference.
@@ -44,7 +43,6 @@ function col($arg1, ?string $arg2 = null): ColumnTerm
 /**
  * Creates a data source for a table. This is an alias for the {@link TableRef} constructor.
  *
- * @psalm-pure
  * @param string $name The table name.
  * @param string|null $alias Optional alias.
  * @return TableRef The created data source.
@@ -84,7 +82,6 @@ function all($source): ColumnTerm
  *
  * Note that this creates a join with an empty table name. Be sure to call {@link Join::with()} to set the table name.
  *
- * @psalm-pure
  * @param string $type The join type.
  * @param DataSource|null $with The data source to join with.
  * @param ExprInterface|null $on The condition to join on.
@@ -98,7 +95,6 @@ function using(string $type, ?DataSource $with = null, ?ExprInterface $on = null
 /**
  * Creates an ascending order instance for a column.
  *
- * @psalm-pure
  * @param string|ColumnTerm $column The column name or term.
  * @return Order|Group The order instance.
  */
@@ -110,7 +106,6 @@ function asc($column): Order
 /**
  * Creates a descending order instance for a column.
  *
- * @psalm-pure
  * @param string|ColumnTerm $column The column name or term.
  * @return Order|Group The order instance.
  */
@@ -126,7 +121,6 @@ function desc($column): Order
  * three arguments, this function creates a binary expression using the first argument as the left-hand side, the second
  * argument as the operator, and the third argument as the right-hand side.
  *
- * @psalm-pure
  * @param mixed $value The value to create the term from.
  * @param string|null $operator Optional operator to create a binary expression.
  * @param mixed|null $value2 Optional second value to create a binary expression.
@@ -148,7 +142,6 @@ function expr($value, ?string $operator = null, $value2 = null): ExprInterface
  * Creates a boolean NOT unary expression. This is an alias for calling the {@link ExprInterface::not()} method on the
  * passed argument.
  *
- * @psalm-pure
  * @param ExprInterface $expr The expression to negate.
  * @return ExprInterface The created expression.
  */
@@ -161,7 +154,6 @@ function not(ExprInterface $expr): ExprInterface
  * Creates a number negation unary expression. This is an alias for calling the {@link ExprInterface::not()} method on
  * the passed argument.
  *
- * @psalm-pure
  * @param ExprInterface $expr The expression to negate.
  * @return ExprInterface The created expression.
  */
@@ -173,7 +165,6 @@ function neg(ExprInterface $expr): ExprInterface
 /**
  * Creates a distinct column term. This is an alias for calling the {@link ColumnTerm::distinct()} method on the column.
  *
- * @psalm-pure
  * @param ColumnTerm $col The column term.
  * @return ColumnTerm The distinct column term.
  */
@@ -185,7 +176,6 @@ function distinct(ColumnTerm $col): ColumnTerm
 /**
  * ORs a list of expressions into a single expression.
  *
- * @psalm-pure
  * @param iterable<ExprInterface|null> $exprs The expressions to OR.
  * @return ExprInterface|null The created expression.
  */
@@ -209,7 +199,6 @@ function orAll(iterable $exprs): ?ExprInterface
 /**
  * ANDs a list of expressions into a single expression.
  *
- * @psalm-pure
  * @param iterable<ExprInterface|null> $exprs The expressions to AND.
  * @return ExprInterface|null The created expression.
  */

src/DataSource.php

@@ -12,7 +12,6 @@ interface DataSource
     /**
      * Creates a copy with an alias.
      *
-     * @psalm-mutation-free
      * @param string|null $alias The string alias or null for no alias.
      * @return static The new instance.
      */
@@ -21,15 +20,13 @@ public function as(?string $alias): self;
     /**
      * Retrieves the data source's alias, if it has one.
      *
-     * @psalm-mutation-free
      * @return string|null The string alias or null if the data source has no alias.
      */
     public function getAlias(): ?string;
 
     /**
      * Compiles the source into an SQL fragment.
      *
-     * @psalm-mutation-free
      * @return string The compiled SQL fragment.
      * @throws SqlCompileException If an error occurred while compiling.
      */

src/DatabaseAdapter.php

@@ -4,11 +4,7 @@
 
 use RebelCode\Atlas\Exception\DatabaseException;
 
-/**
- * An adapter for a database connection that allows queries created by Atlas to be executed.
- *
- * @psalm-immutable
- */
+/** An adapter for a database connection that allows queries created by Atlas to be executed. */
 interface DatabaseAdapter
 {
     /**

src/Expression/BaseExpr.php

@@ -2,7 +2,6 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/** @psalm-immutable */
 abstract class BaseExpr implements ExprInterface
 {
     protected ?string $alias = null;

src/Expression/BetweenExpr.php

@@ -2,7 +2,6 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/** @psalm-immutable */
 class BetweenExpr extends BaseExpr
 {
     const BETWEEN = 'BETWEEN';

src/Expression/BinaryExpr.php

@@ -2,7 +2,6 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/** @psalm-immutable */
 class BinaryExpr extends BaseExpr
 {
     const EQ = '=';

src/Expression/ColumnTerm.php

@@ -2,11 +2,7 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/**
- * An expression that represents a column name.
- *
- * @psalm-immutable
- */
+/** An expression that represents a column name. */
 class ColumnTerm extends BaseExpr
 {
     protected ?string $table;

src/Expression/ExprInterface.php

@@ -2,7 +2,6 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/** @psalm-immutable */
 interface ExprInterface
 {
     /**

src/Expression/FnExpr.php

@@ -2,11 +2,7 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/**
- * Represents a SQL function expression.
- *
- * @psalm-immutable
- */
+/** Represents a SQL function expression. */
 class FnExpr extends BaseExpr
 {
     protected string $name;

src/Expression/Term.php

@@ -5,7 +5,6 @@
 use InvalidArgumentException;
 use Traversable;
 
-/** @psalm-immutable */
 class Term extends BaseExpr
 {
     public const NUMBER = 0;
@@ -77,8 +76,6 @@ protected function toBaseString(): string
     /**
      * Creates a term from a value, automatically detecting the type.
      *
-     * @psalm-mutation-free
-     *
      * @param mixed $value
      * @return ExprInterface
      */
@@ -103,10 +100,7 @@ public static function create($value): ExprInterface
         return new self($type, $value);
     }
 
-    /**
-     * @psalm-mutation-free
-     * @psalm-return Term::*
-     */
+    /** @psalm-return Term::* */
     public static function detectType($value): int
     {
         $type = gettype($value);

src/Expression/UnaryExpr.php

@@ -2,7 +2,6 @@
 
 namespace RebelCode\Atlas\Expression;
 
-/** @psalm-immutable */
 class UnaryExpr extends BaseExpr
 {
     public const NOT = '!';

src/Join.php

@@ -4,7 +4,6 @@
 
 use RebelCode\Atlas\Expression\ExprInterface;
 
-/** @psalm-immutable */
 class Join
 {
     const INNER = 'INNER';

src/Order.php

@@ -4,7 +4,6 @@
 
 use RebelCode\Atlas\Expression\ColumnTerm;
 
-/** @psalm-immutable */
 class Order
 {
     public const ASC = 'ASC';

src/Query.php

@@ -6,7 +6,6 @@
 use RebelCode\Atlas\Exception\DatabaseException;
 use RebelCode\Atlas\Exception\QuerySqlException;
 
-/** @psalm-immutable */
 abstract class Query
 {
     protected ?DatabaseAdapter $adapter;
@@ -39,7 +38,6 @@ protected function getAdapter(): DatabaseAdapter
     /**
      * Compiles the query into an SQL string.
      *
-     * @psalm-mutation-free
      * @return string The compiled SQL string.
      * @throws QuerySqlException If an error occurred while compiling the SQL.
      */

src/Query/CompoundQuery.php

@@ -9,8 +9,6 @@
  *
  * This is intended to allow multiple queries to be executed in a single call to the database adapter. Rendering the
  * queries to SQL will concatenate the SQL of each query and a semicolon after each query.
- *
- * @psalm-immutable
  */
 class CompoundQuery extends Query
 {

src/Query/CreateIndexQuery.php

@@ -10,7 +10,6 @@
 use RebelCode\Atlas\Schema\Index;
 use Throwable;
 
-/** @psalm-immutable */
 class CreateIndexQuery extends Query
 {
     protected string $table;
@@ -37,10 +36,7 @@ public function __construct(
         $this->index = $index;
     }
 
-    /**
-     * @inheritDoc
-     * @psalm-mutation-free
-     */
+    /** @inheritDoc */
     public function toSql(): string
     {
         try {

src/Query/CreateTableQuery.php

@@ -10,7 +10,6 @@
 use Throwable;
 use UnexpectedValueException;
 
-/** @psalm-immutable */
 class CreateTableQuery extends Query
 {
     protected string $name;
@@ -41,10 +40,7 @@ public function __construct(
         $this->collate = $collate;
     }
 
-    /**
-     * @inheritDoc
-     * @psalm-mutation-free
-     */
+    /** @inheritDoc */
     public function toSql(): string
     {
         try {
@@ -75,8 +71,6 @@ public function toSql(): string
     /**
      * Compiles the schema for a table.
      *
-     * @psalm-mutation-free
-     *
      * @param Schema $schema The table schema.
      * @return string
      */

src/Query/DeleteQuery.php

@@ -10,7 +10,6 @@
 use RebelCode\Atlas\Query;
 use Throwable;
 
-/** @psalm-immutable */
 class DeleteQuery extends Query
 {
     use Query\Traits\HasWhereTrait;
@@ -45,7 +44,6 @@ public function __construct(
     /**
      * Creates a copy with a different FROM clause.
      *
-     * @psalm-mutation-free
      * @param string $from The table name.
      * @return static The new instance.
      */
@@ -56,10 +54,7 @@ public function from(string $from): self
         return $clone;
     }
 
-    /**
-     * @inheritDoc
-     * @psalm-mutation-free
-     */
+    /** @inheritDoc */
     public function toSql(): string
     {
         try {

src/Query/DropTableQuery.php

@@ -8,7 +8,6 @@
 use RebelCode\Atlas\Query;
 use Throwable;
 
-/** @psalm-immutable */
 class DropTableQuery extends Query
 {
     protected string $table;
@@ -35,10 +34,7 @@ public function __construct(
         $this->cascade = $cascade;
     }
 
-    /**
-     * @inheritDoc
-     * @psalm-mutation-free
-     */
+    /** @inheritDoc */
     public function toSql(): string
     {
         try {