Feature/dev setup

[turegjorup]

Link to ticket

https://leantime.itkdev.dk/TimeTable/TimeTable?showTicketModal=7041&fromDate=2026-03-23&toDate=2026-03-29#/tickets/showTicket/7041

Description

Summary

• Modernize dev environment with ITK Dev docker setup, Taskfile, and multi-PHP test matrix (8.3, 8.4, 8.5)
• Replace monolithic pr.yaml workflow with dedicated CI workflows (changelog, composer, PHP, markdown, YAML)
• Upgrade to PHPUnit 12 and bump minimum PHP to 8.3
• Fix all PHPStan errors at max level with proper type annotations
• Add tests for error paths and Token model, raising line coverage from ~71% to 94%

Changes

Dev environment

• Add Taskfile.yml with tasks for linting, testing, analysis, and a full CI matrix runner
• Update docker-compose.yml with PHP 8.3/8.4/8.5 services and per-version vendor volume isolation
• Add PHPStan config (phpstan.neon), markdownlint config, and LICENSE.md
• Update .php-cs-fixer.dist.php to preserve @var PHPDoc annotations

CI workflows

• Delete pr.yaml (used shivammathur/setup-php, outdated actions)
• Add individual workflows: changelog.yaml, composer.yaml, php.yaml, markdown.yaml, yaml.yaml, github_build_release.yml

Code quality

• Bump php requirement from >=8.2 to >=8.3 (required by PHPUnit 12)
• Upgrade phpunit/phpunit from ^11.3 to ^12.0 and update phpunit.xml schema
• Add dev dependencies: phpstan/phpstan, rector/rector, ergebnis/composer-normalize, symfony/runtime
• Fix 41 PHPStan errors: type-narrow cache returns, add @var annotations for json_decode, use JSON_THROW_ON_ERROR on json_encode

Tests

• Add TokenTest covering isExpired(), used(), usesLeft()
• Add 8 error-path tests to VaultTest: HTTP client failures, vault error responses, missing keys, cache refresh bypass, version parameter
• Line coverage: 94% (80/85 lines)

Test plan

• task test:matrix — all 6 combinations pass (PHP 8.3/8.4/8.5 × lowest/stable)
• task analyze:php — PHPStan max level, no errors
• task lint:php — php-cs-fixer clean
• task test:coverage — 94.12% line coverage

Screenshot of the result

If your change affects the user interface, you should include a screenshot of the result with the pull request.

Checklist

• My code is covered by test cases.
• My code passes our test (all our tests).
• My code passes our static analysis suite.
• My code passes our continuous integration process.

If your code does not pass all the requirements on the checklist, you have to add a comment explaining why this change
should be exempt from the list.

Additional comments or questions

If you have any further comments or questions for the reviewer, please add them here.

[codecov-commenter]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️