[breaking] feat: Query::insertGetId throws error now instead of retur…

…n last value

this method is only called if `incrementing = true` which should never be true for Spanner databases

phpstan.neon

@@ -20,12 +20,6 @@ parameters:
           path: src/Colopl/Spanner/Connection.php
         - message: '#^Cannot cast mixed to int\.$#'
           path: src/Colopl/Spanner/Eloquent/Model.php
-        - message: '#^Return type \(string\) of method Colopl\\Spanner\\Query\\Builder::insertGetId\(\) should be compatible with return type \(int\) of method Illuminate\\Database\\Query\\Builder::insertGetId\(\)$#'
-          path: src/Colopl/Spanner/Query/Builder.php
-        - message: '#^Method Colopl\\Spanner\\Query\\Builder::insertGetId\(\) should return string but returns mixed\.$#'
-          path: src/Colopl/Spanner/Query/Builder.php
-        - message: '#^Cannot access offset string\|null on mixed\.$#'
-          path: src/Colopl/Spanner/Query/Builder.php
         - message: '#^Method Colopl\\Spanner\\Schema\\Builder::createBlueprint\(\) should return Illuminate\\Database\\Schema\\Blueprint but returns mixed\.$#'
           path: src/Colopl/Spanner/Schema/Builder.php
         - message: '#^Property Illuminate\\Database\\Schema\\Builder::\$resolver \(Closure\) in isset\(\) is not nullable.$#'

src/Colopl/Spanner/Query/Builder.php

@@ -52,18 +52,14 @@ public function insert(array $values)
      * This will return the last value used since Spanner does not have the
      * feature to return the last inserted ID.
      *
-     * @param  array        $values
-     * @param  string|null  $sequence the name of primary key
-     * @return string
-     * @throws InvalidArgumentException
+     * @param array $values
+     * @param string|null $sequence the name of primary key
+     * @return int
      */
     public function insertGetId(array $values, $sequence = null)
     {
-        $values = $this->prepareInsertForDml($values);
-        $this->insert($values);
-
-        $lastValue = array_values(array_slice($values, -1))[0];
-        return $lastValue[$sequence] ?? null;
+        $this->markAsNotSupported('insertGetId');
+        return -1;
     }
 
     /**