Skip to content

KnpLabs/phpstan-rules

Repository files navigation

knplabs/phpstan-rules

CI PHP Packagist License

PHPStan rules shared across KnpLabs organization projects.

Requirements

  • PHP 8.2 or higher
  • PHPStan ^2.0

Installation

Using phpstan/extension-installer (recommended)

composer require --dev knplabs/phpstan-rules phpstan/extension-installer

If you don't use phpstan/extension-installer, include the extension in your phpstan.neon:

includes:
    - vendor/knplabs/phpstan-rules/extension.neon

Rules

clock.disallowDateTimeNow — PSR-20 Clock Abstraction

Enforces the PSR-20 recommendation to avoid instantiating DateTime or DateTimeImmutable with the current time directly. This makes code that depends on the current time testable and respects the clock abstraction.

Triggers on:

$a = new DateTime();
$b = new DateTime('now');
$c = new DateTimeImmutable();
$d = new DateTimeImmutable('now');

Does not trigger on (explicit non-"now" timestamps are fine):

$a = new DateTime('2023-01-01');
$b = new DateTimeImmutable('yesterday');

Recommended fix: inject Psr\Clock\ClockInterface and call $clock->now():

use Psr\Clock\ClockInterface;

final class MyService
{
    public function __construct(private readonly ClockInterface $clock) {}

    public function doSomething(): void
    {
        $now = $this->clock->now();
        // ...
    }
}

clock.disallowTimeFunctions — PSR-20 Clock Abstraction (functions)

Enforces the PSR-20 recommendation to avoid using time() or date() directly. This makes code that depends on the current time testable and respects the clock abstraction.

Triggers on:

$a = time();
$b = date('Y-m-d');
$c = date('Y-m-d', time());
$d = date('Y-m-d', 'now');

Does not trigger on (explicit non-"now" timestamps are fine):

$a = date('Y-m-d', 1672531200);
$b = date('Y-m-d', $someTimestamp);

Recommended fix: inject Psr\Clock\ClockInterface and call $clock->now():

use Psr\Clock\ClockInterface;

final class MyService
{
    public function __construct(private readonly ClockInterface $clock) {}

    public function doSomething(): void
    {
        $now = $this->clock->now();
        // ...
    }
}

Contributing

See CONTRIBUTING.md for the human contributor guide.

If you are working with an AI agent, refer to AGENTS.md — it contains the AI-facing instructions for this repository.

Release & Publishing

Releases are fully automated with release-please and published to Packagist as soon as a GitHub release goes out.

Prerequisites (one-time setup)

  1. Register the package on Packagist — submit the repository once at packagist.org/packages/submit.

  2. Add repository secrets — in GitHub → Settings → Secrets and variables → Actions, create two repository secrets:

    Secret name Value
    PACKAGIST_USERNAME Your Packagist account username
    PACKAGIST_API_TOKEN An API token generated on your Packagist profile page

How it works

  1. Every push to main runs the Release workflow (.github/workflows/release.yml), which calls GoogleCloudPlatform/release-please-action. It parses Conventional Commits (feat:, fix:, doc:, etc.) since the last release and keeps a release PR up to date with the bumped version and generated CHANGELOG.md entry.
  2. Merging the release PR makes release-please tag the repository and publish the corresponding GitHub release automatically — no manual "Draft a new release" step is needed anymore.
  3. That GitHub release publish event triggers the Publish workflow, which notifies Packagist via its REST API.
  4. The new version appears on Packagist within a few minutes.

Commit messages matter — release-please only picks up a version bump when commits follow the Conventional Commits format. Non-conforming commits are ignored when computing the next version.

Manual re-trigger

If the Publish workflow fails or you need to re-sync Packagist without creating a new release:

  1. Go to Actions → PublishRun workflow.
  2. Click Run workflow (no inputs required).

License

MIT — see LICENSE.

About

No description, website, or topics provided.

Resources

License

Contributing

Stars

1 star

Watchers

0 watching

Forks

Packages

 
 
 

Contributors