coniadev / chuck

src/Server/CliRouter.php

<?php

// phpcs:ignoreFile

declare(strict_types=1);

require_once 'functions.php';

if (PHP_SAPI !== 'cli') {
    $uri = $_SERVER['REQUEST_URI'] ?? '';
    $publicDir = getenv('CONIA_DOCUMENT_ROOT');
    $url = urldecode(parse_url($uri, PHP_URL_PATH));

    $start = microtime(true);

    if ($publicDir) {
        // serve existing files as-is
        if (is_file($publicDir . $url)) {
            /** @psalm-suppress PossiblyInvalidArgument */
            serverEcho(http_response_code() ?: 0, $uri, microtime(true) - $start);

It seems like http_response_code() ?: 0 can also be of type true; however, parameter $statusCode of serverEcho() does only seem to accept integer, maybe add an additional type check?

            return false;
        }

        if (is_file($publicDir . rtrim($url, '/') . '/index.html')) {
            /** @psalm-suppress PossiblyInvalidArgument */
            serverEcho(http_response_code() ?: 0, $uri, microtime(true) - $start);

            return false;
        }

        if ($url === '/phpinfo') {
            echo phpinfo();
            /** @psalm-suppress PossiblyInvalidArgument */
            serverEcho(http_response_code() ?: 0, $uri, microtime(true) - $start);

            return true;
        }

        $_SERVER['SCRIPT_NAME'] = 'index.php';

        /** @psalm-suppress UnresolvableInclude, MixedAssignment */
        $response = require_once $publicDir . '/index.php';

        if ($response) {
            /** @psalm-suppress MixedMethodCall, MixedArgument */
            serverEcho($response->getStatusCode(), $uri, microtime(true) - $start);
        }

        return true;
    }

    return false;
}