- الدليل بقى 75 صفحة: اتضاف ملحق بيشرح أداة تكرار الرقم القومي (إزاي
  تستخدمها، وإمتى تدمج وإمتى تفرّغ الرقم بدل ما تدمج)، وملحق تاني بيوضّح
  إن «برسم التحصيل» مصطلح بنكي معناه «بغرض التحصيل» — مفيش رسم بيتخصم،
  وقيد تحصيل الشيك بيكون بقيمته كاملة.

- tools/pdf/fonts: خطوط Amiri اتحطّت جوّه المستودع. كانت في /tmp واتمسحت
  مع تنضيف السيرفر، فالدليل ما كانش ينفع يتبني تاني من غيرها.

- tools/pdf/build_addendum.py: بيولّد صفحات ملحق بنفس شكل الدليل ويلزقها
  في آخره — عشان أي أداة جديدة تتوثّق من غير ما نعيد تصوير الـ67 شاشة.

- tools/fix_player_duplicates.py: بينضّف التكرار من خلال نفس الـ routes
  بتاعة الشاشة (نفس CSRF ونفس الكنترولر)، وبيدمج بس المجموعات اللي كل
  الأسماء فيها متطابقة ويسيب الباقي للموظف.

المشكلة اللي اتبلّغت: نفس اللاعب متسجّل مرتين بنفس الرقم القومي — مرة
كعضو ومرة كغير عضو. اتأكدت من الداتا الحية: فعلاً فيه 7 أرقام قومية
مكررة، واحد منهم متكرر 4 مرات. شاشة تسجيل اللاعبين ما كانش فيها أي فحص
للتكرار، والـ index على العمود non-unique.

المنع:
- PlayerController::store() و ::update() بقوا يرفضوا أي رقم قومي مسجّل
  للاعب تاني، ويقولوا اسم اللاعب ورقمه عشان الموظف يفتح ملفه بدل ما
  يسجّله من جديد.

التنضيف (شاشة جديدة /sa/players/duplicates):
- بتعرض كل مجموعة مكررة وجوّاها اللاعبين، وقدّام كل نسخة عدد السجلات
  المرتبطة بيها (اشتراكات، حضور، بطاقات، مستندات...) عشان الموظف يعرف
  النسخة اللي عليها شغل فعلي.
- بتفرّق بين «نفس الاسم» و«أسماء مختلفة» وبتحذّر: مش كل تكرار هو نفس
  الشخص — أحياناً حد كتب رقم قومي غلط لشخص تاني.
- إجراءين: دمج (بينقل كل السجلات للنسخة المختارة ويأرشف الباقي)، أو
  تفريغ الرقم القومي لو اتضح إنهم أشخاص مختلفين.
- الدمج كله في transaction، ومفيش حذف نهائي — النسخ المدموجة بتتأرشف
  بملاحظة توضّح اتدمجت في مين.

ملاحظة مهمة في التنفيذ: في الداتابيز 45 عمود اسمه player_id، لكن كتير
منهم بيشاور على جدول players بتاع «شؤون اللاعبين» — جدول تاني خالص.
فالدمج بيشتغل على قايمة جداول مكتوبة بإيد ومتأكد منها، مش على اسم العمود،
عشان ما نخربش بيانات موديول تاني.

كمان: مسمّى «إيداع ورقة قبض برسم التحصيل» في قيود الأوراق التجارية اتغيّر
لـ «تسليم ورقة قبض للبنك للتحصيل» — «برسم التحصيل» مصطلح بنكي معناه
«عشان يتحصّل»، بس بتتقري كأن فيه رسم اتخصم.

mermaid keeps a pie title's quotes as literal text (other chart types strip
them), so the widget unquotes pie titles before rendering. Pie title, legend
and slice labels are sized up, since the chart is drawn wide and scaled down to
the panel.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

The "drop an unclosed code block" cleanup matched a chart's closing fence
whenever no backtick followed it, and deleted that fence and the rest of the
answer — the table note, the «ملحوظة» line. It now cuts only when the number of
fences is odd, and never a chart that is still being written. Found on the
first live end-to-end answer.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

A floating assistant for the super admin and senior managers (captain.use:
super_admin, board_member, general_manager, membership_director,
sports_director). It answers in Egyptian Arabic: step-by-step guides with the
exact on-screen labels and a link to the page, numbers from the live data with
tables and mermaid charts, and explanations of the page the user has open. It
never talks about code, and it cannot change anything.

The model runs on the DevPilot box under a restricted client token — Haiku at
low effort, only Read/Grep/Glob, confined to its own checkout (clubphp-captain),
enforced by DevPilot from the token. It never touches the database: it asks for
SELECTs in a [[QUERY]] block and QueryGuard runs them on a separate read-only
connection (READ ONLY transaction, one statement, 5 s cap), refusing other
schemas, credential tables and columns, and HR/accounting/treasury data the
user has no permission for; contact data is masked and every query is logged.
Verified against the live database before shipping, including a write with the
text filter bypassed, which MySQL itself refused (1792).

KnowledgeSync keeps .captain/schema.txt (live schema with the values actually
stored) and .captain/menu.txt (every sidebar path) current in that checkout.
Config sits in captain_settings, not system_config, so the token never appears
on the Settings screen; the widget only shows once a token is configured.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Terms the team says in English had been rendered in formal Arabic, which
read as machine translation — "معالج مُرشد" for wizard, "ماسح" for scanner,
"تشغيل تجريبي" for dry run, "الخادم" for server. They are now written as
used: Wizard, Scanner, Dry Run, Offline, Override, PIN, Push, SMS, Branding,
Audit Log, Mockups, Server. Two calques go too: "لا يُكتب شيء" → "لا يُحفظ
شيء", and "شاشات مُعادة الرسم" → "Mockups". The club's own terms and the
portal's screen names are untouched.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

The app screenshots came from a 16:9 emulator, so every phone in the
proposal looked squat. They are recaptured on a 1320x2868 @3x emulator — the
iPhone 16 Pro Max's own 440x956pt screen — so the frame now reads as a
modern phone.

The scope section listed what was out of scope, which read as the offer not
covering the club. It is replaced by an inventory of what is built and
working today, checked against both repos: 52 member-app features and 45
across the portal and the gate scanner, grouped into cards, under the four
pillars and the counts (51 app screens, 34 portal sections, 33 wizards,
6 roles). Items the guide implies but the app has no screen for — uploading
a transfer proof, requesting a plan, autopay — are left out.

Print is now true A4. One print stylesheet serves both the browser's Print
and generate-pdf.mjs: printer-safe margins, a cover page, dark bands printed
light, grids held at their desktop columns, every app and portal screenshot
in captioned galleries built into the page, page numbers in the footer, and
breaks that keep sections whole. 17 pages.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

The Sayd proposal carried a hand-built imitation of the app and a mocked
portal. Both are replaced with screenshots of the real thing: 17 screens
captured from the Flutter app (sayd-mobile-app-main) driving the live demo
API on an emulator, and 19 portal screens (sayd-mobile-portal-main) wearing
the club's own branding.

The phase framing is gone — no "first/second phase", no week-by-week plan.
Scope now lists what the system actually does (installments, fines and
appeals, credit balance, transfer proofs, medical certificates, attendance
and evaluations, events, audit log). Captions reuse the proposal's own
Arabic and the Arabic user guide's wording.

The PDF export prints both screenshot sets as captioned grids.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

دليل عملي واحد يغطي التلات مجالات المطلوبة بس:
- المحاسبة: الإعداد، دليل الحسابات، القيود، البنوك والشيكات، الاعتمادات
  والضمانات والقروض، وكل التقارير والقوائم المالية.
- الأصول الثابتة: السجل، الإضافة، الصيانة، العهدة، الموردين.
- الخزينة والكاشير: الورديات، التحصيل، التسويات، العهدة، الإيداعات البنكية.

كل شاشة ليها لقطة حقيقية من النظام الحي + شرح إيه اللي بتعمله وإزاي
تستخدمها والقواعد المهمة فيها.

أدوات التوليد اتحفظت تحت tools/pdf عشان الدليل يتبني تاني بعد أي تعديل.
اللقطات بتتفحص إنها مش صفحة تسجيل دخول قبل ما تتقبل — أول محاولة طلعت
65 لقطة من 67 صفحة لوجن من غير ما السكريبت ياخد باله.

السبب اللي خلّى شاشة تعديل اللاعب تبان كأنها «شاشة إضافة جديدة»:
old() في app/Core/Helpers.php مُعرّفة إنها ترجع (string) دايماً، يعني
بترجع '' مش null لما مفيش إدخال سابق. وكل الفورمات مكتوبة بالشكل ده:

    old('full_name_ar') ?? $player['full_name_ar']

و ?? بيشتغل على null بس — و'' مش null — فالنتيجة دايماً '' مهما كانت
البيانات المحفوظة. يعني كل خانات كل شاشات التعديل في النظام كانت بتطلع
فاضية، والمستخدم يفتكرها شاشة إضافة.

اتصلّح 345 موضع في 37 ملف بتحويل ?? إلى ?: في نداءات old() تحديداً،
فالقيمة المحفوظة بتظهر لما ما يكونش فيه إدخال سابق.
ده بيمس: HR (كل الفورمات)، SportsActivity، TrainingGroups، Coaches،
PlayerAffairs، Tournaments، PoolManagement، Achievements، Waiver،
Installments.

كمان فورمات بتبعت على routes مش موجودة (الحفظ كان بيدي 404 من غير ما
الشاشة تشتكي):
- /hr/overtime/store -> الصح /hr/overtime
- /hr/shifts/assign  -> الصح /hr/shifts/assignments

وحذف MirrorDisplayController وشاشتينه — مفيش أي route بيوصّلهم من ساعة
ما الميرور اتنقل لموديول FacilityGrids.

أدوات جديدة عشان النوع ده من الأعطال ما يوصلش للمستخدم تاني:
- tools/form_action_check.py: بيتأكد إن كل فورم POST بيبعت على route
  موجود فعلاً (692 route اتفحصوا). الفحص ده كان هيمسك بَجّي الـ /update.
- tools/ui_verify.js: فحص بالمتصفح بيدوس الأزرار زي المستخدم بالظبط
  ويفشل على أي استثناء في الصفحة — مش مجرد طلب GET على الـ route.

fix(accounting, sa): missing LC/LG show screens, debtors report repeating每 debtor, player link fix screen

- شاشتي عرض الاعتماد المستندي وخطاب الضمان مكانوش موجودين أصلاً (View not
  found) — أي حد يدوس «عرض» كان بياخد استثناء. اتعملوا كاملين: البيانات
  والغطاء والعمولة والمستندات، وكمان فورم تحديث الحالة اللي كان متعرّف في
  الكنترولر من غير أي واجهة توصّله (التجديد بيطلب تاريخ انتهاء جديد).

- تقرير المدينين كان بيطلع سطر لكل مطالبة، فالعضو الواحد يتكرر 53 مرة
  (1219 سطر لكل المدينين). بقى سطر واحد لكل مدين بإجمالي مديونيته وتوزيع
  التقادم، وبالضغط عليه بتتفتح تفاصيل مطالباته — من غير ما نخسر أي تفصيلة.

- «صلّح الربط» في شاشة الفجوات كان بيفتح فورم فيه خانة رقم مجرّدة، والأسوأ
  إن الفورم أصلاً بيبعت على /sa/players/{id}/update واللي مش موجود كـ
  route — يعني الحفظ ما كانش بيشتغل خالص. دلوقتي:
  • الفورم بيبعت على المسار الصح.
  • خانة «العضو المرتبط» بقت بحث بالاسم/رقم العضوية وبتختار من النتايج.
  • بتوضّح العضو المربوط حالياً، ولو الرقم المسجّل مش عضو حقيقي بتقول كده
    صراحة — وده بالظبط الحالة اللي الفجوة بتبلّغ عنها.
  • الكنترولر بقى يرفض أي member_id مش رقم عضو موجود، في الإضافة والتعديل —
    عشان الفجوة دي ما تتكررش من أصلها.

- tools/route_smoke.py: كان بيحط id=1 دايماً، والكنترولر بيعمل redirect لما
  الصف مش موجود، فالشاشة ما بتتفتحش والفحص بيعدّي وهو فاضي — وده اللي خلّى
  شاشات ناقصة view تعدّي الفحص وتقع عند المستخدم. بقى بيجيب ID حقيقي من
  قاعدة البيانات لكل نوع، وبيبلّغ عن أي شاشة ما اتفتحتش بدل ما يعتبرها نجاح.

fix(payments, cashier, treasury): void-type gaps, unpaid fines via queue, invisible guest payments, missing permission checks

Payments/Installments:
- PaymentLifecycleService::onPaymentVoided() ما كانتش بترجع حالة العضو
  إلا لـ membership_fee/down_payment بس — رغم إن foreign_membership_fee
  وsports_membership_fee وseasonal_fee كلهم بينشّطوا العضو، إلغاء أي واحدة
  منهم كان بيسيب العضو "فعال" للأبد من غير أي غطاء دفع حقيقي.
  MembershipPaymentGuard::deactivateMember() نفسها كانت أصلًا عارفة تتعامل
  مع الخمس أنواع صح — المشكلة كانت في القايمة اللي بتقرر تنادي عليها.
- غرامة بتترسل لطابور الدفع (fine) كانت بتتحصّل وتتطبع إيصال، لكن صف
  الغرامة في جدول fines فضل زي ما هو "غير مسدد" للأبد — مفيش حد كان بيسمع
  الحدث ده أصلًا. أي فحص رصيد أو أهلية كارنيه كان هيفضل يقول إن الغرامة
  لسه مستحقة حتى بعد سدادها فعليًا. اتضاف مستمع بيقفل الغرامة المحددة.
- GET /api/v1/payments كان مكسور تمامًا — بيفلتر بعمود is_archived مش
  موجود أصلًا في جدول payments (الصح is_voided).
- شاشة كل المدفوعات والتقرير اليومي كانوا بيستخدموا INNER JOIN مع
  members، فأي دفعة لعميل زائر (member_id فاضي — زي تسجيل رياضي لغير
  عضو) كانت تختفي تمامًا من الشاشتين دول رغم إنها محصّلة فعليًا
  (23 دفعة بحوالي 41,800 جنيه في البيانات الحية). اتحول لـ LEFT JOIN.
- زرار إلغاء الدفعة كان بيتحكم بصلاحية payment.void_receipt بينما الـ
  route بتاعه محتاج payment.void — محدش في الأدوار الحية عنده الصلاحية
  التانية، فكان عمليًا محدش غير super_admin يقدر يلغي دفعة من الشاشة دي.
- عضوية موسمية بتتفعّل من كاشير مباشرة من غير ما تعدّي على
  activateIncludedDependents — يعني زوجة/أبناء العضو الموسمي المرفقين في
  نفس الرسم ما كانوش بيتفعّلوا معاه.
- حذف DefaultChecker.php الميتة تمامًا (صفر استدعاء) — نفس منطق فحص
  التعثر بالظبط موجود وشغّال فعليًا في cron/jobs/InstallmentDefaultJob.php.

Treasury/Cashier:
- شاشات موديول Treasury بالكامل كانت من غير أي فحص can() على الإطلاق —
  كل الأزرار بتظهر لأي حد شايف الشاشة بغض النظر عن صلاحياته الحقيقية.
  اتأكد إن ده مش نظري: أدوار حية زي auditor وtreasury_manager وmain_cashier
  عندهم جزء من صلاحيات الخزنة بس مش كلها، فكانوا بياخدوا 403 على أزرار
  شايفينها. كل زرار دلوقتي بيتحقق من نفس صلاحية الـ route بتاعه.
  نفس الحاجة اتصلحت في زرار فتح/قفل وردية كاشير، وزرار طباعة الإيصال اللي
  كان بيتحكم بصلاحية receipt.print بدل payment.view الحقيقية.
- عداد "طلبات معلقة" في لوحة الخزنة الفرعية كان بيستخدم قايمة أنواع دفع
  أقدم من القايمة الحقيقية المستخدمة في الطابور نفسه — ناقصة رسوم عضوية
  أجانب/رياضية وبعض أنواع الأنشطة الرياضية.

Plus extreme-detail reference docs for Payments/Installments, Cashier, and Treasury.

fix(facility-grids, members): cross-grid delete auth gap, blank names, wrong permissions, dead crons, FY string mismatch

FacilityGrids:
- ZoneTraineeController::remove()/clearZone()/move() ما كانوش بيتأكدوا إن
  المتدرب/المنطقة فعلاً تابعين لنفس الشبكة في الـ URL — أي حد عنده
  facility_grid.manage على أي شبكة كان يقدر يمسح متدرب أو يفرغ منطقة تابعة
  لشبكة تانية تمامًا لو خمّن الـ ID. دلوقتي بيتأكدوا من الملكية الأول.
- FacilityAttendanceService::sendPaymentReminders() والـ view بتاع شاشة
  الحضور كانوا لسه بيقروا name_ar بعد ما الكويري اتصلح لـ full_name_ar —
  فكل تذكير دفع كان بيتبعت باسم فاضي، وشاشة تسجيل الحضور كانت بتطلع كل
  أسماء المدربين/المتدربين فاضية وبتحفظ entity_name_cache فاضي في القاعدة.
- MonthlyPlanController::index() كان بيتحقق من صلاحية مختلفة عن اللي مسجلة
  على الـ route بتاعه (manage بدل view) — يعني حتى لو عندك صلاحية العرض
  بس، تاخد 403 فور ما تدخل شاشة الخطط الشهرية.
- زرار "إنشاء شبكة" و"خطة جديدة" كانوا بيتحكموا بصلاحية facility.manage
  (بتاعة موديول Facilities المختلف) بدل facility_grid.manage بتاع
  الموديول نفسه.
- زرار "تصدير PDF" في لوحة المالية كان بيسمي نفسه غلط — الملف اللي بينزل
  فعليًا HTML مش PDF.

Members:
- AutoFreezeService::freezeTemporaryAtAgeLimit() كانت شرطه بيدور على أكواد
  فئات (sisters_under_25 إلخ) مش موجودة في القاعدة أصلًا — الفئات الحقيقية
  المخزنة هي sister/stepchild/orphan، فالكرون كان ميت فعليًا ومفيش شقيقة
  ولا ابن زوج ولا يتيم بيتقفل تلقائيًا بعد سن 25.
- حالة permanently_dropped (اللي بيكتبها كرون إسقاط الأعضاء) ما كانتش في
  قائمة تسميات الحالات — فكانت بتظهر بالإنجليزي الخام في أي شاشة تعرض حالة
  العضو. اتضافت هي وterminated اللي كانت ناقصة في نفس القائمة.
- MemberController::show() كان بيبني السنة المالية الحالية بصيغة فيها شرطة
  (2025-2026) بينما كل مكان تاني في الكود بيكتبها بشرطة مايلة (2025/2026)
  زي ما فعليًا متخزن في subscriptions — فكانت شاشة بروفايل أي عضو دايمًا
  بتقول إن مفيش اشتراك للسنة الحالية حتى لو مدفوع فعلًا.

Plus extreme-detail reference docs for both modules.

يوثق كل شاشات الموديول (تقييمات، لياقة، إصابات، الملفات) وأهم نقطة فيه:
جدول players هنا منفصل تمامًا عن sa_players بتاع SportsActivity — مفيش
مفتاح مشترك ولا مزامنة. بيوثق كمان الإصلاحات اللي اتعملت قبل كده في نفس
الجلسة لصلاحيات الأزرار، وموضوع تضارب شاشتي اعتماد الشهادات الطبية.

fix(player-affairs): buttons gated by a different module's permission, and one by a permission marked deprecated

كل شاشات الموديول تقريبًا كانت بتتحكم في ظهور أزرار الإضافة/التعديل بصلاحية
sa.player.manage اللي أصلًا مسجّلة في SportsActivity مش هنا — فموظف عنده
صلاحياته الصحيحة (player.register، player.edit، player.evaluate،
player.manage_injuries، player.manage_fitness) ما كانش بيشوف زراره خالص.

وشاشة اعتماد الشهادات الطبية كانت بتتحكم بصلاحية sa.medical.approve
اللي متسجّلة في كود SportsActivity نفسه بعلامة '(ملغي) استخدم
medical.board.approve' — يعني حتى المصلحة الأصلية بتقول إنها ملغاة،
والصلاحية الحقيقية للـ route هي player.approve_medical. كل الأزرار
بقت بتتحكم بنفس الصلاحية اللي الـ route فعليًا محتاجها.

Disciplines, academies/contracts, programs, groups (enroll vs force-enroll,
the three disagreeing pause implementations, medical grace-period sweep),
coaches, facilities/units, pricing resolution algorithm, attendance,
player cards, and both SA reports. Documents the coach-cost fix's real
impact (9 of 25 active coaches previously cost 0 in the finance report)
and flags the card-renewal duration bug as a deliberate fix, not a
quick patch, since it touches both fee calculation and payment payload.

Classic hourly booking form, booking passes (confirmed orphaned gate-scan
API), daily/weekly schedule, blackout dates, conflict detection scope,
locker rentals, group schedule/waitlist/transfer (including the separate
TrainingGroups parallel system), makeup sessions, and Service Desk. Flags
a consistent cash-collected-outside-the-system pattern across three
screens for a deliberate finance decision rather than three silent fixes.

- تقرير الماليات (SaFinanceReportService): الـ switch بتاع تكلفة المدربين
  كان بيقارن بقيم زي 'monthly'/'hourly' مش موجودة في enum الحقيقي
  (monthly_fixed/per_session/per_player/hybrid/salary) — يعني أي مدرب
  مش per_session كان بيتحسب تكلفته صفر، وصافي الربح المعروض في التقرير
  كان مبالغ فيه بشكل منهجي. اتصلحت لتغطي الخمس نماذج كلهم، بنفس صيغة
  الحساب المستخدمة فعلًا في Coaches\Services\CoachPaymentService
  (مدرب الأكاديمية 'salary' متعمّد يفضل صفر — أجره بيتحسب من خلال تسوية
  عقد الأكاديمية مش كبند تكلفة مباشر).
- FacilityUnit::$fillable ما كانش فيه expected_capacity، فالحقل كان
  بيتقبل في الفورم ويتفلتر بصمت قبل ما يوصل لقاعدة البيانات.
- EnrollmentService::forceEnroll() كان بيثبّت is_full=1 دايمًا بدل ما
  يحسبه — تجاوز واحد في مجموعة سعتها 20 كان بيقفل التسجيل العادي على
  الـ 19 مكان الفاضيين اللي بعده.
- زرار سحب اللاعب من المجموعة كان بيتفعّل بصلاحية sa.group.enroll بينما
  الـ route نفسه محتاج sa.group.manage — تضارب بين الاتنين حسب الدور.

Chart of accounts, cost centers/budgets, dimensions, revenue mapping
manual screens, posting chains, accruals, gaps, opening entries/
reclassification, bank reconciliation, loans/documentary credits/
guarantees, settlements/billing, fiscal years/journal types/branch
fees, and the exact calculation logic behind every report screen
including all 5 financial statements — plus a status table for every
defect found and fixed or flagged along the way.

نفس فئة الغلط بتاعة نوع القرض: القائمة كانت بتعرض initial/final/advance
بينما الـ ENUM الحقيقي في القاعدة tender/performance/advance_payment/
maintenance/customs/other — فأي اختيار غير maintenance كان يفشل الحفظ
بـ Data truncated for column 'guarantee_type'.

بعد إصلاح أسماء الحقول، اختبرت الشاشات الثلاثة على السيرفر الحي ولقيت:

- إنشاء اعتماد مستندي: تاريخ الانتهاء NOT NULL في القاعدة بس مش required
  في الفورم — الحفظ كان بيقع بـ 500. بقى required.
- إنشاء خطاب ضمان: حقل رقم الخطاب كان required في الـ HTML بس الكنترولر
  بيتجاهله ويولّد رقمه لوحده — يعني المتصفح نفسه كان بيمنع الحفظ من غير
  ما توصل السيرفر أصلًا. بقى للعرض بس زي رقم القرض.
- bank_loans.interest_rate و letters_of_guarantee.commission_rate كانوا
  DECIMAL(5,4) — أقصى قيمة 9.9999%. أي سعر فايدة حقيقي (15%، 22%...
  معدلات مصرية عادية) كان يفشل بـ Out of range. اتوسّعوا لـ DECIMAL(7,4).

اكتشفت أثناء توثيق الشاشات المتبقية إن تلات فورمات معطلة فعليًا:

- إنشاء اعتماد مستندي: أسماء الحقول (credit_number، supplier_id،
  bank_account_id، currency) ما تطابقش اللي الكنترولر بيقراه
  (issuing_bank_id، beneficiary_supplier_id، currency_code)، ومفيش حقل
  أصلًا لـ beneficiary_name رغم إنه NOT NULL في القاعدة — يعني أي محاولة
  حفظ كانت بتطلّع 500 من قاعدة البيانات.
- خطاب ضمان جديد: نفس المشكلة بالظبط — beneficiary بدل beneficiary_name
  NOT NULL، فأي حفظ كان بيقع.
- قرض بنكي جديد: خانة الضمان اسمها collateral بس الكنترولر بيقرا
  collateral_description، فأي نص كان بيتكتب فيها بيتفقد بصمت. وقائمة
  نوع القرض فيها قيم (term_loan، equipment) مش موجودة في enum القاعدة
  أصلًا (term، revolving، overdraft، mortgage) — اختيارها كان هيفشل
  الحفظ أو يبوّظ العمود. رقم القرض واسم البنك كانا حقول قابلة للتعديل
  بالمصادفة بس الكنترولر بيتجاهلهم دايمًا — بقوا معروضين للقراءة بس.

Member retroactive entry, SA player registration, SA hourly booking,
swimming pool reservations, and the accounting revenue-mapping wizard.
Every field, validation rule, permission key, button, DB write, and
error message, sourced from reading the actual controller/service code
rather than the UI — plus a list of real defects found along the way
(transaction leaks, dead endpoints, schema mismatches, silent failures)
flagged for a deliberate fix pass rather than folded in unannounced.

اتضافت خطوة البحث في دليل الحسابات (٧٣٣ حساب) اللي بتوضّح إن البحث
بيشتغل من غير همزة وبالأرقام العربية ومع الأخطاء المطبعية.

سكربتات الشاشات بتشتغل وقت تحميل الصفحة، يعني قبل سكربتات آخرها. فدليل
الحسابات كان بيقع على الطريقة القديمة: «إيراد» بترجّع ١٢ و«ايراد» ٤،
و«١٢٠٦» بالأرقام العربية صفر. النواة دلوقتي في الـ head قبل أي شاشة.

سبع شاشات كانت بتعمل بحث بـ toLowerCase().indexOf — يعني «احمد» ما
بتلاقيش «أحمد» و«المطابقه» ما بتلاقيش «المطابقة». أهمهم دليل الحسابات
(٧٣٣ حساب) والمدينون والدائنون والحسابات البنكية ومراكز التكلفة.

دلوقتي كلهم على ArabicSearch: تطبيع الهمزات والتشكيل والأرقام العربية،
ومطابقة تقريبية بتتحمّل الأخطاء المطبعية. ودليل الحسابات بيطبّع نص كل
صف مرة واحدة بدل كل ضغطة زرار.

- شاشتين وميثودين وداليتين قراءة بقوا ميتين بعد توحيد كشوف الحسابات.
  AccountStatementService فضلت لتسجيل الحركات (١٦ مكان بيستخدمها).
- البحث بالحروف المتتابعة كان فضفاض: «recon» كانت بتلاقي «شئون العضوية»
  لأن حروفها متفرقة في النص. اتضاف شرط تقارب — الحروف لازم تكون في مدى
  ضعف طول اللي اتكتب.

الفصل الجديد بيغطي تنظيم قائمة المالية الجديد، كشوف الحسابات الموحّدة
بالمطابقة، البحث والترتيب في الجداول، وفحص جاهزية الإقفال. وفصل البنوك
اتشال منه لقطة القائمة القديمة بعد ما التنظيم اتغيّر.

الـ view اتدفع وفيه خطأ صياغة لأن git commit ما بيلنتش. tools/php_lint.sh
بيفحص الـ ٢٣٩٠ ملف PHP في ثواني، وده اللي كان لازم يتشغّل قبل الدفع.

الإقفال كان بيتحقق من حاجة واحدة (القيود المسودة) وبعدين يقفل. أسوأ حاجة
في الإقفال إن المحاسب يقفل ويطلّع القوائم وبعدين يكتشف ناقص ويضطر يفتح تاني.

الشاشة دلوقتي بتعمل سبع فحوصات قبل الإقفال، كل واحد بلينك يوصّل للشاشة
اللي تحلّه: القيود المسودة، توازن الميزان، الشهر السابق، العمليات غير
المقيّدة، الفلوس الواقفة في حسابات وسيطة، الشيكات المستحقة، والمطابقة
البنكية.

المسودة وعدم التوازن موانع بتوقف الإقفال — الباقي تنبيهات والقرار للمحاسب.
والفحص بيتعاد في الكنترولر مش بس في الشاشة عشان حد ما يعديه بـ POST مباشر.

أول ما شغّلت الشاشة على عضو حقيقي طلّعت «فرق ٣٦٣ ألف» على عضو حسابه سليم.
السبب: القيد بيتعلّم برقم العضو على الطرفين (مدين الصندوق / دائن الإيراد)،
فجمع كل سطوره بيطلّع صفر دايمًا والفرق مالوش معنى.

المطابقة الصح بتقارن رصيد العضو في حسابات الذمم (١٢٠٣/١٢٠٤ للعملاء،
٢٣٠٦ للموردين) بالحساب المساعد. والعضو اللي بيدفع نقدي وقت العملية
مالوش حساب ذمم أصلًا، فبقت الشاشة بتقول كده صريح بدل تحذير غلط.

٣١٦ شاشة فيها جدول من غير أي خانة بحث، والمستخدم بيلف بعينه على الصفحة.
بدل ما نضيف خانة في كل شاشة (شغل ما بيخلصش وبيتنسي في الجديد)، مكوّن
واحد بيشتغل لوحده على أي جدول أكتر من ٨ صفوف:

- خانة بحث بتفلتر وانت بتكتب، بنفس نواة التطبيع العربي بتاعة النظام.
- ترتيب بالضغط على عنوان العمود، بيفهم الأرقام والفلوس والتواريخ فـ
  «1,200.00 ج.م» بتترتّب كرقم مش كنص.
- عدّاد «كام من كام»، ورسالة واضحة لما مفيش نتيجة.
- بيتخطى الشاشات اللي فيها بحث أصلًا، وبيتعطّل بـ data-no-filter.

القائمة:
- «المحاسبة والدفتر العام» بقت «المحاسبة المالية»، وتحتها مجموعات:
  المؤشرات المالية، حسابات البنوك، المراجعة الداخلية والإقفال،
  القوائم المالية، والإعدادات المالية.
- مراكز التكلفة والموازنات خرجوا لقائمة «المحاسبة الإدارية» — دول أدوات
  قرار مش جزء من دورة القيد والترحيل.
- الشريط الجانبي كان بيدعم مستويين بس، فاتعمل يدعم أي عمق: الترشيح
  بالصلاحيات، وحالة «مفتوح»، والبحث كلهم بيمشوا على الشجرة كلها.

كشوف الحسابات:
- «كشف حساب عضو» و«كشف حساب عميل» كانوا شاشتين على نفس العضو. اتوحّدوا
  مع كشف المورد في شاشة واحدة بتختار فيها نوع الطرف، وبتعرض دفتر الأستاذ
  والحساب المساعد جنب بعض وتقارن بينهم — لأن اختلافهم هو بالظبط اللي
  المحاسب محتاج يشوفه. الروابط القديمة بتحوّل عليها بنفس الطرف والفترة.

البحث:
- نواة واحدة (arabic-search.js) بتطبّع الهمزات والتشكيل والأرقام العربية
  وبتتحمّل الأخطاء المطبعية وبترتّب بالأقرب. كان في تلات تطبيقات مختلفة،
  واحدة منهم بتعمل indexOf عادي فـ«احمد» ما كانتش بتلاقي «أحمد».
- أي قايمة فيها ١٢ خيار أو أكتر بتتحوّل لقايمة بحث تلقائيًا.

tools/menu_link_check.py بيتأكد إن كل بند قائمة وكل لينك في الشاشات له
route فعلي — ٢٨١ بند و٠ مكسور.

٨٠ صفحة، ١٥ فصل، ١٣٣ خطوة بلقطات حقيقية من النظام. الجديد فصلين:
حسابات البنوك (التجميع في القائمة، البحث في دليل الحسابات، وإقفال أوراق
القبض بالقيد اللي بيتولد منه) ودورة حياة الشيك بسجل حركته الكامل.

١. قايمة المالية كانت ٤٥ بند في ليستة واحدة. اتقسمت لمجموعات:
   المحاسبة والدفتر العام / حسابات البنوك / الإيرادات والتحصيل /
   التقارير المحاسبية / القوائم المالية.
   «حسابات البنوك» جمّعت الحسابات البنكية والشيكات والأوراق التجارية
   وإقفال أوراق الدفع والقبض والودائع والمطابقة والقروض والاعتمادات.

٢. البحث جوه القوايم المنسدلة: مكوّن مشترك بيتفعّل لوحده على أي قائمة
   فيها أكتر من ١٢ خيار (زي دليل الحسابات في قيد اليومية)، وبيدوّر
   بالاسم أو بالرقم. البحث بيتجاهل الهمزات والتشكيل عشان يلاقي بالعربي.

٣. «إقفال أوراق القبض» — المقابل الناقص لإقفال أوراق الدفع: بيأكّد تحصيل
   الشيكات الواردة من كشف حساب البنك.

٤. سجل حركة الشيك بقى بيعمل القيد المحاسبي فعلًا. كان بيسجّل الحركة بس
   من غير ترحيل، وده كان بيخلي الشاشة الجديدة تخالف قاعدة إن أي حركة
   مالية لازم تنعكس على الدفتر. كل حركة دلوقتي متربوطة بقيدها.

- ٦ شاشات كانت بتقع بـ View not found: طلب عروض الأسعار، أنواع العمل الإضافي،
  سجل عهدة الأصل، تقرير حضور المرفق، مندوب المبيعات، وأسعار العملاء الخاصة.
- مكوّن الترقيم المشترك بقى بيكمّل المفاتيح الناقصة لوحده، لأن ٢٠+ موديل
  بيبنوا مصفوفة الترقيم بإيدهم وبيبعتوا الأساسي بس.
- شاشة الملاعب بترجّع 404 لما الملعب مش موجود بدل ما تقع على null.
- تقرير الأبناء وسجل البوابة كانوا بيقروا مفاتيح مش موجودة.

اتنين أعطال بنيوية طلعوا من فحص الـ 830 شاشة:

1. الراوتر بيبعت بارامترات الـ URL كـ string، وفي ١٣ كنترولر معرّفينها int،
   ومع strict_types ده TypeError بيقع الشاشة. الراوتر دلوقتي بيقرا النوع
   المطلوب من الميثود نفسها ويحوّل ليه.

2. public/index.php كان لافف الـ dispatch والـ boot في try واحدة، فأي
   استثناء جوه الطلب بيطلّع «BOOT FAILURE» بـ 500 حتى لو كان 404 أو 403.
   الطلب دلوقتي ليه try لوحده بيسلّم لـ ExceptionHandler اللي بيحترم الكود.

كل رابط لسجل محذوف أو مش موجود كان بيطلّع صفحة 500 لأن findOrFail
بيرمي RuntimeException من غير code، والـ ExceptionHandler بيعتبر أي
code غير 401/403/404 عطل سيرفر.

tools/route_smoke.py بيفتح كل GET route في النظام (٨٣٠ شاشة) ويبلّغ
عن أي 500 برسالة الخطأ.