// Code review for UnoPim. Activates when reviewing code changes, checking standards compliance, flagging violations, or performing PR reviews; or when the user mentions review, standards, conventions, best practices, violations, or code quality.
| name | unopim-code-review |
| description | Code review for UnoPim. Activates when reviewing code changes, checking standards compliance, flagging violations, or performing PR reviews; or when the user mentions review, standards, conventions, best practices, violations, or code quality. |
| license | MIT |
| metadata | {"author":"unopim"} |
Review code changes against UnoPim coding standards and conventions.
Consult the unopim-backend-dev skill for detailed standards. Flag these violations:
Architecture & Structure:
Webkul\Core\Eloquent\Repositorypackages/Webkul/{Package}/src/ directorynew Model() instantiation instead of using repository patternNaming & Conventions:
{domain}.{entity}.{action}.{before|after} patternDocumentation:
@param/@return annotationsData Integrity:
$guarded = [] on modelsTesting:
beforeEach login setupassertDatabaseHas/assertDatabaseMissing)it()/describe() patternService Provider:
boot()register() (menu, acl, importers, exporters)<x-admin:: component usage (using raw HTML instead)@lang() / trans())<option> text, title="" tooltips, JS alert/error messages, and "Loading…" / "No results" status text. Every user-visible string must be a translation key (see unopim-backend-dev/coding-conventions.md → "Translations" section)en_US/app.php only — must propagate to all 32 other locales with natural translations (not English copies); verify with php artisan unopim:translations:check showing 100% per localeFor each violation found:
[Issue Type]: [Specific problem]
Location: [File path and line number]
Standard: [Link to relevant skill file]
Fix: [Brief explanation or example]
Backend PHP development for UnoPim. Activates when writing PHP code, creating classes, models, repositories, events, listeners, or tests; or when the user mentions model, repository, controller, service, event, listener, observer, Concord, proxy, or needs to write backend code. MUST be invoked before writing any PHP tests.
Development workflow for UnoPim. Activates when running tests, linting code, building assets, or debugging; or when the user mentions test, lint, pint, build, npm, pest, format, style, quality, coverage, or needs to verify code works.
Handle module attribute mapping and mapping history management in Unopim. Use this skill when creating module attribute mapping UI, mapping controller, mapping history, or restricting fields to module structure. Only Attribute Mapping and History tabs are allowed. Covers DB_PREFIX (auto-added by Laravel), Migration/ singular folder, HistoryTrait on models, flat ACL, cURL HTTP client, and JsonResponse controller pattern.
Day-1 checklist and step-by-step guide to build a complete Unopim third-party connector (WooCommerce, Shopify, Shopware, module, or any REST API) within one working day. Use this skill when starting a new Unopim integration, planning a connector from scratch, or needing the full ordered sequence of steps to create a production-ready connector module. Covers all required files, patterns, and critical checkpoints verified against the WooCommerce connector reference implementation.
Implement credential storage, API connection testing, secure field handling, history tracking, and full CRUD for Unopim third-party connector modules. Covers Credential model with HistoryTrait and extras JSON, Contract interface, CredentialRepository, CredentialController with JsonResponse, FormRequest validation, DataGrid, and migration with DB_PREFIX. Use this skill when building the credentials section of any Unopim connector (WooCommerce, Shopify, Shopware, module, etc.).
Implement Unopim DataGrid classes for listing records in admin panels with search, filter, sort, actions (edit/delete), and mass actions. Covers DataGrid subdirectory structure (DataGrids/Section/), prepareQueryBuilder with DB::table (prefix added by Laravel), addColumn with correct closure patterns, addAction with function callbacks for url, addMassAction, bouncer() permission checks, and Blade integration. Use this skill when creating a DataGrid for credentials, mappings, export history, or any Unopim module listing page.