Skip to content

fix(quality): resolve cs-fixer formatting + Psalm PossiblyFalseOperand#17

Merged
walmir-silva merged 1 commit intomainfrom
develop
Mar 2, 2026
Merged

fix(quality): resolve cs-fixer formatting + Psalm PossiblyFalseOperand#17
walmir-silva merged 1 commit intomainfrom
develop

Conversation

@walmir-silva
Copy link
Contributor

cs-fixer: removed extra alignment spaces in array key-value pairs across 6 files (KcodeComposerGenerator, Devkit, InitCommand, ProcessExecutor, KcodeComposerGeneratorTest, ComposerResolverTest). No logic changes.

KcodeComposerGenerator.php:76:
json_encode() returns false|string; applied (string) cast to satisfy
Psalm PossiblyFalseOperand. The manifest is a static structure with no
un-serializable values so the false branch is unreachable in practice.

Result:
✓ cs-fixer passed
✓ phpstan passed
✓ psalm passed (0 errors)
✓ phpunit passed
✓ All 4 tool(s) passed

cs-fixer: removed extra alignment spaces in array key-value pairs across
6 files (KcodeComposerGenerator, Devkit, InitCommand, ProcessExecutor,
KcodeComposerGeneratorTest, ComposerResolverTest). No logic changes.

KcodeComposerGenerator.php:76:
  json_encode() returns false|string; applied (string) cast to satisfy
  Psalm PossiblyFalseOperand. The manifest is a static structure with no
  un-serializable values so the false branch is unreachable in practice.

Result:
  ✓ cs-fixer   passed
  ✓ phpstan    passed
  ✓ psalm      passed (0 errors)
  ✓ phpunit    passed
  ✓ All 4 tool(s) passed
@walmir-silva walmir-silva merged commit 898977e into main Mar 2, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant