MoonStack — إصلاح ثغرة الـ Seeder في التحديث

الخطة · 2026-06-17 · للمراجعة قبل التنفيذ — بتعدّل مسار تحديث كل العملاء (حسّاس)

1) المشكلة bug

التحديث بيشغّل migrate بس مابيشغّلش seeders. تعريفات الإعدادات (setting definitions) بتتزرع عبر SettingDefinitionSeeder وLabSettingDefinitionSeeder (مش migrations) — فالتعريفات الجديدة:

متوصلش للعملاء اللي بيعملوا تحديث.
وحتى التركيب الجديد ناقص: installer.seeders مفيهوش LabSettingDefinitionSeeder أصلًا.

النتيجة: SettingsService::get() بيرجّع null لأي مفتاح ملوش definition، وشاشة الإعدادات مابتعرضوش.

الدليل: المدينة على 2.2.8 (آخر نسخة) كانت عندها 14 من 53 تعريف LIS فقط.

2) الجذر

الخطوة	Install	Update
migrate	✅	✅
seed (تعريفات)	⚠️ جزئي (ناقص LIS)	❌ مفيش خالص

3) الحل 4 أجزاء

أضيف LabSettingDefinitionSeeder لـ config('moonstack.installer.seeders') → التركيب الجديد ياخد كل تعريفات LIS.

قايمة جديدة config('moonstack.updater.seeders') = seeders التعريفات الـ idempotent فقط (SettingDefinitionSeeder + LabSettingDefinitionSeeder).

في public/moonstack-update.php بعد الـ migrate وقبل caches: أشغّلهم best-effort / non-fatal داخل timeout 300 bash -c '…'.

Bridge migration بتشغّل نفس الـ seeders → العملاء الحاليين (على الـ updater القديم) يلحقوا التعريفات في أول تحديث، لأن migrate بيشتغل دايمًا.

4) ليه مش هنضيف `RolePermissionSeeder` مهم

صلاحياته idempotent (firstOrCreate)، لكنه كمان بيعيد تعيين أدوار لليوزرز (بيدّي owner/employee). تشغيله في كل تحديث ممكن يغيّر أدوار مستخدمين العميل — فمستبعد من القايمة. (لو احتجنا صلاحيات جديدة، نفصل جزء الصلاحيات لوحده لاحقًا.)

5) الأمان — الـ crash-safe محفوظ safe

الـ seeders كلها updateOrCreate → بتضيف/بتحدّث بس، مابتمسحش.
تشغيلها بعد نجاح الـ migrate، وفشلها non-fatal (مايعملش rollback لتحديث ناجح أصلًا) — بنسجّل الخطأ بس.
الـ migration الجسر بتشغّل نفس الكلاس → نفس الأمان.

6) الملفات اللي هتتعدّل

config/moonstack.php — installer.seeders (+LIS) + updater.seeders (جديد)
public/moonstack-update.php — خطوة seed بعد migrate
database/migrations/2026_..._moonstack_sync_reference_definitions.php — الجسر
(اختياري) أمر moonstack:sync-reference يقرا القايمة ويشغّلها — مصدر واحد للحقيقة للـ updater + الـ migration

7) خطة الاختبار

تشغيل SettingDefinitionSeeder/LabSettingDefinitionSeeder مرتين متتاليتين على sqlite — تأكيد idempotency (مفيش تكرار).
محاكاة: DB ناقصة تعريفات → migrate+seed → التعريفات كاملة، القيم الموجودة سليمة.
محاكاة فشل seed → التحديث يفضل ناجح (مفيش rollback) + الخطأ في اللوج.
المدينة (آمن — اتعمل يدوي بالفعل): تأكيد 58 تعريف.

8) الإطلاق

commit على lis/lab-updates + سطر في الـ what's-new، وبعد الموافقة release. العملاء ياخدوا الإصلاح في أول تحديث (الجسر يلحّقهم فورًا).

السياق الكامل: knowledge-base/topics/moonstack-update.md