SmartNull: don't answer SmartString methods when SmartStrings are off

- Before: ->or(), ->trim(), etc. on a missing key in a raw array quietly
  worked and returned an HTML-encoding SmartString - the one place a raw
  array handed back encoded output
- Now: throws the same "Call to undefined method" error as any unknown
  method, matching stored values (plain strings, so those calls always
  failed there)
- HTML mode unchanged - misses still chain
- Fallbacks on raw arrays use ?? instead

Author: daveedis

CHANGELOG.md

@@ -36,6 +36,7 @@
 - `orDie()` and `or404()` now exit with status 1 instead of 0, so shell scripts and cron jobs see the failure. Output is unchanged. Matches SmartString.
 - Developer-mistake exceptions (bad types, wrong context, misuse) now throw `CallerException`, which reports your file and line instead of the library's internals - the same class SmartString uses. It extends `InvalidArgumentException`, so existing catch blocks keep working, except six throws that previously used `RuntimeException`: `load()` misuse (no handler, non-callable handler, bad or empty field name, called on a record set), `orRedirect()` after headers sent, and writing to a `SmartNull`. See UPGRADING.md. `orThrow()` still throws `RuntimeException` by contract.
 - Clearer messages for two of those throws: `load()` with no handler now explains handlers come from the database layer (was "No loadHandler property is defined"), and writing to a `SmartNull` now says the value came from a missing key or empty result (was "Cannot set values on SmartNull"). The unsupported-type message from `set()` no longer prefixes the library's internal method name.
+- Raw-mode arrays no longer answer SmartString methods on missing keys: `$row->missing->or('n/a')` on a raw array throws the standard undefined-method Error instead of returning an HTML-encoding SmartString. Raw stored values never had these methods (chaining `->or()` on a stored string was already a fatal), so a miss was the one path that silently produced encoded output in a raw array. HTML mode is unchanged - SmartNull still delegates SmartString methods so chains through a missing key keep working. Raw fallbacks use `??`.
 - Unknown methods on `SmartNull` now throw the same `Error` as the rest of the library - method name, "did you mean" hint, caller's file and line - instead of `InvalidArgumentException("Method 'x' not found")`. Chains from a missing key now fail with the same message quality as everything else.
 
 ### Deprecated

src/SmartNull.php

@@ -252,7 +252,10 @@ private function throwCannotSet(): never
      */
     public function __call($name, array $arguments): mixed
     {
-        if (!method_exists(SmartArrayBase::class, $name) && method_exists(SmartString::class, $name)) {
+        // SmartString methods only delegate in HTML mode: raw values are plain scalars
+        // with no methods, so a miss answers SmartString calls the same way - with the
+        // standard undefined-method Error
+        if ($this->useSmartStrings && !method_exists(SmartArrayBase::class, $name) && method_exists(SmartString::class, $name)) {
             return SmartString::new(null)->$name(...$arguments);
         }
         return $this->useSmartStrings

tests/Unit/SmartNullTest.php

@@ -224,10 +224,9 @@ public function testDelegatedArraysCarryTheSourceMetadata(string $class): void
     //endregion
     //region Method delegation: SmartString methods
 
-    #[DataProvider('modeProvider')]
-    public function testSmartStringOnlyMethodsDelegateToANullSmartString(string $class): void
+    public function testSmartStringOnlyMethodsDelegateToANullSmartStringInHtmlMode(): void
     {
-        $smartNull = $this->smartNullFrom($class);
+        $smartNull = $this->smartNullFrom(SmartArrayHtml::class);
 
         $trimmed = $smartNull->trim();
         $this->assertInstanceOf(SmartString::class, $trimmed);
@@ -238,17 +237,32 @@ public function testSmartStringOnlyMethodsDelegateToANullSmartString(string $cla
         $this->assertSame('n/a', $fallback->value());
     }
 
-    #[DataProvider('modeProvider')]
-    public function testSmartStringTypeCastsReturnEmptyScalars(string $class): void
+    public function testSmartStringTypeCastsReturnEmptyScalarsInHtmlMode(): void
     {
-        $smartNull = $this->smartNullFrom($class);
+        $smartNull = $this->smartNullFrom(SmartArrayHtml::class);
 
         $this->assertSame('', $smartNull->string());
         $this->assertSame(0, $smartNull->int());
         $this->assertSame(0.0, $smartNull->float());
         $this->assertFalse($smartNull->bool());
     }
 
+    public function testSmartStringMethodsThrowInRawMode(): void
+    {
+        // Raw values are plain scalars with no methods, so a miss answers
+        // SmartString calls with the same Error as any unknown method
+        $smartNull = $this->smartNullFrom(SmartArray::class);
+
+        foreach (['trim', 'or', 'string'] as $method) {
+            try {
+                $smartNull->$method('n/a');
+                $this->fail("expected an Error from ->$method()");
+            } catch (Error $e) {
+                $this->assertStringStartsWith("Call to undefined method SmartArray->$method(), ", $e->getMessage());
+            }
+        }
+    }
+
     //endregion
     //region Method delegation: unknown methods