Page:
Coding-Standards
Pages
ARCHITECTURE
AUTO-BUMP
AUTO_CREATE_ORG_PROJECTS
Branching-Strategy
CLI_AUTOMATION
Coding-Standards
DEPLOY_SCRIPTS
DOLIBARR_MODULE_IDS
DRY_RUN_PATTERN
Documentation-Standards
File-Header-Standards
Home
JOOMLA_SYNC
LEGAL_DOC_GENERATOR_WEB_README
MANIFEST-STANDARD
MINIFICATION
MONITORING_SCRIPTS
NEW_SCRIPTS
QUICKSTART_ORG_PROJECTS
UPDATE-SERVER
WIKI_STANDARDS
WORKFLOW_STANDARDS
api-automation-index
api-definitions-default-index
api-definitions-sync-index
api-deploy-index
api-fix-index
api-index
api-maintenance-index.-
api-maintenance-index
api-plugin-index.-
api-plugin-index
api-tests-index.-
api-tests-sample-index.-
api-tests-sample-index
api-validate-index.-
automation-README.-
automation-branch-version-automation.-
automation-push-files.-
automation-repo-cleanup.-
client-repos.-.-
client-repos
standards-mokostandards-file-spec.-
standards-mokostandards-file-spec
templates-client-waas
templates-dolibarr
templates-generic
templates-mcp
unnamed
workflows-README.-
workflows-README
workflows-auto-release.-
workflows-auto-release
workflows-branch-protection.-
workflows-branch-protection
workflows-build-release.-
workflows-build-release
workflows-cascade-dev.-
workflows-cascade-dev
workflows-changelog-management.-
workflows-changelog-management
workflows-demo-deployment.-
workflows-demo-deployment
workflows-dev-branch-tracking.-
workflows-dev-branch-tracking
workflows-dev-deployment.-
workflows-dev-deployment
workflows-index.-
workflows-index
workflows-release-system.-
workflows-release-system
workflows-renovate.-
workflows-renovate
workflows-reusable-workflows.-
workflows-reusable-workflows
workflows-rs-deployment.-
workflows-rs-deployment
workflows-secret-scanning.-
workflows-secret-scanning
workflows-shared-workflows.-
workflows-shared-workflows
workflows-standards-compliance.-
workflows-standards-compliance
workflows-static-analysis.-
workflows-static-analysis
workflows-sub-issue-management.-
workflows-sub-issue-management
workflows-update-server.-
workflows-update-server
workflows-workflow-architecture.-
workflows-workflow-architecture
Clone
1
Coding-Standards
Jonathan Miller edited this page 2026-05-26 04:25:52 +00:00
Coding Standards
PHP Code Quality
| Tool | Level | Config | Enforcement |
|---|---|---|---|
| PHPCS | PSR-12 (errors only) | phpcs.xml | CI Gate 1 (blocks merge) |
| PHPStan | Level 6 | phpstan.neon + baseline | CI Gate 1 (blocks merge) |
| PHPUnit | 19 tests | phpunit.xml | CI Gate 2 (blocks merge) |
PHPDoc Standard
All public classes and methods must have PHPDoc blocks to support the auto-documentation engine.
File Header (navigation)
Every PHP file must have a file-level header for module grouping:
/* Copyright (C) 2026 Moko Consulting <hello@mokoconsulting.tech>
*
* SPDX-License-Identifier: GPL-3.0-or-later
*
* FILE INFORMATION
* DEFGROUP: moko-platform.CLI
* INGROUP: moko-platform
* REPO: https://git.mokoconsulting.tech/MokoConsulting/moko-platform
* PATH: /cli/my_tool.php
* VERSION: 01.00.00
* BRIEF: One-line description of what this file does
*/
Class Documentation
/**
* One-line summary of what the class does.
*
* Longer description if the class is complex. Explain the purpose,
* key design decisions, and how it fits into the architecture.
*
* @since 08.00.00
* @see RelatedClass
*/
class MyClass
Method Documentation
/**
* What this method does in one line.
*
* @param array<string, mixed> Configuration data
* @param string Human-readable name
* @return bool True if validation passed
* @throws \RuntimeException When the API is unreachable
*
* @since 08.00.00
* @see ConfigValidator::getErrors()
*
* @example
* = ->validate(, );
*/
public function validate(array , string ): bool
Required Tags
| Tag | Where | When |
|---|---|---|
| @param | Method | Every parameter |
| @return | Method | Every non-void return |
| @throws | Method | Every thrown exception |
| @since | Class, public method | Always |
| @see | Class, method | When referencing related code |
| @deprecated | Class, method | When marking for removal |
| @example | Method | High-value or complex methods |
Rules
- DO add PHPDoc to all public and protected methods
- DO add @since with the version that introduced the method
- DO add @throws for methods that throw exceptions
- DO keep descriptions concise -- one line for simple methods
- DO NOT add PHPDoc to trivial getters/setters (getName, getId)
- DO NOT duplicate type information already in the signature
- DO NOT add empty or boilerplate PHPDoc blocks
Interface Implementations
When a class implements an interface, use {@inheritdoc} or omit the PHPDoc entirely -- the interface documentation is authoritative:
// Option 1: explicit inheritdoc
/** {@inheritdoc} */
public function getPlatformName(): string
// Option 2: no doc (inherits from interface)
public function getPlatformName(): string
File Structure
CLI Tools (extend CliFramework)
#!/usr/bin/env php
<?php
/* File header with DEFGROUP/INGROUP/BRIEF */
declare(strict_types=1);
require_once __DIR__ . '/../vendor/autoload.php';
use MokoEnterprise\CliFramework;
/**
* What this tool does.
*
* @since 08.00.00
*/
class MyTool extends CliFramework
{
protected function configure(): void { ... }
protected function run(): int { ... }
}
= new MyTool();
exit(->execute());
Standalone CLI Tools (no framework)
#!/usr/bin/env php
<?php
/* File header */
declare(strict_types=1);
final class MyTool
{
public function run(): int { ... }
private function parseArgs(): void { ... }
private function log(string ): void { ... }
}
= new MyTool();
exit(->run());
PHPCS Exclusions
These PSR-12 rules are excluded in phpcs.xml by design:
| Rule | Reason |
|---|---|
| SideEffects | CLI scripts mix declarations and execution |
| MissingNamespace | CLI scripts and validators lack namespaces |
| MultipleClasses | Helper classes in same file is intentional |
| FileHeader.IncorrectOrder | Advisory only |
| TabsUsedHeredocCloser | PHP 7.3+ heredoc requires matching indent |
| EmptyStatement.DetectedCatch | Intentional error suppression |