test(tests): drop the E_DEPRECATED suppression from the .phpt files

[lisachenko]

What changed

Removed the third --INI-- line, error_reporting=E_ALL & ~E_DEPRECATED, from all 19 tests/Functional/*.phpt files. ffi.enable=1 and opcache.jit=off stay — those are load-bearing.

Docs updated to describe two mandatory --INI-- lines instead of three:

• CLAUDE.md — the "Running tests" section, the --INI-- block in the "Anatomy of a .phpt test" example, and the "Rules for a new test" bullet.
• .claude/agents/phpt-author.md — the template block and the matching rule.

.github/workflows/ci.yml and README.md were checked and never referenced the line: CI passes only the FFI/JIT pair as ini-values, so no change was needed there. No change to src/, composer.json or the operator handlers.

Why the suppression is now dead weight

The line existed for one reason: z-engine's development line declared implicitly nullable parameters (ZEngine\Type\OpLine::__construct()), which PHP reports as a deprecation. PHPUnit's .phpt runner forces display_errors=1, so that single deprecation was prepended to the captured output of every test and failed each --EXPECT-- block on noise from a dependency.

The stable releases now required (~8.4.2 || ~8.5.0, #20) declare those parameters ?Type — confirmed in the installed source on both lines:

// vendor/lisachenko/z-engine/src/Type/OpLine.php:99  (8.4.2 and 8.5.0 alike)
public function __construct(object $opline, ?ExecutionData $context = null)

So the line suppresses nothing today, and while it sits there it would silently swallow a real deprecation from a future dependency bump — turning a visible signal into a failure somewhere harder to find. The docs now say so explicitly rather than leaving the removal unexplained.

Verification

PHPUnit could not be installed in this environment (network egress reaches only lisachenko/*, so composer install without --no-dev fails). The suite was instead run through a minimal runner that reproduces what PHPUnit's PhptTestCase does: it writes each --FILE-- body next to the .phpt so the relative vendor/autoload.php include resolves, applies the file's own --INI--, forces display_errors=1, compares --EXPECT-- exactly and matches --EXPECTREGEX-- unanchored. CI runs the real composer test on both minors.

Baseline, before any edit, with error_reporting=E_ALL forced over each file's own --INI-- — i.e. proving the suppression was doing nothing:

PHP	z-engine	Result
8.4.19	8.4.2	19/19 pass
8.5.9	8.5.0	19/19 pass

After removing the line, each file running under its own two-line --INI--:

PHP	z-engine	Result
8.4.19	8.4.2	19/19 pass
8.5.9	8.5.0	19/19 pass

No segfaults, bus errors, skipped or incomplete tests in any of the four runs. z-engine was resolved per minor by re-running composer install with the matching binary after clearing vendor/ and composer.lock.

composer cs:fix was not run: php-cs-fixer is a dev dependency and is not installable here. Only .phpt and Markdown files are touched, so PER-CS2.0 does not apply to this diff.

Closes #21

---

Generated by Claude Code