refactor: migrate platform constants from EventConstant to Platform enum

- Move DEFAULT_PLATFORM, WEBHOOK_EVENT_HEADER, event separators,
  and PLATFORM_EVENT_SEPARATOR into Platform enum methods
- Add Platform::DEFAULT const for use in default parameter values
- Remove platformEnum() static helper (use Platform::from() directly)
- EventConstant now only contains callback serialization constants
- SettingConstant uses hardcoded separator strings (const context)
- All 16 tests passing

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: tanhongit

src/Bot.php

@@ -2,7 +2,7 @@
 
 namespace CSlant\TelegramGitNotifier;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 use CSlant\TelegramGitNotifier\Exceptions\ConfigFileException;
 use CSlant\TelegramGitNotifier\Interfaces\BotInterface;
 use CSlant\TelegramGitNotifier\Models\Event;
@@ -42,7 +42,7 @@ public function __construct(
         ?Telegram $telegram = null,
         ?string $chatBotId = null,
         ?Event $event = null,
-        ?string $platform = EventConstant::DEFAULT_PLATFORM,
+        ?string $platform = Platform::DEFAULT,
         ?string $platformFile = null,
         ?Setting $setting = null,
         ?string $settingFile = null,

src/Constants/EventConstant.php

@@ -2,34 +2,11 @@
 
 namespace CSlant\TelegramGitNotifier\Constants;
 
-use CSlant\TelegramGitNotifier\Enums\Platform;
-
 final class EventConstant
 {
-    public const DEFAULT_PLATFORM = 'github';
-
-    public const WEBHOOK_EVENT_HEADER = [
-        'github' => 'HTTP_X_GITHUB_EVENT',
-        'gitlab' => 'HTTP_X_GITLAB_EVENT',
-    ];
-
     public const EVENT_PREFIX = SettingConstant::SETTING_CUSTOM_EVENTS . '.evt.';
 
-    public const GITHUB_EVENT_SEPARATOR = 'gh.';
-
-    public const GITLAB_EVENT_SEPARATOR = 'gl.';
-
     public const EVENT_HAS_ACTION_SEPARATOR = 'atc.';
 
     public const EVENT_UPDATE_SEPARATOR = '.eus';
-
-    public const PLATFORM_EVENT_SEPARATOR = [
-        'github' => self::GITHUB_EVENT_SEPARATOR,
-        'gitlab' => self::GITLAB_EVENT_SEPARATOR,
-    ];
-
-    public static function platformEnum(string $platform): Platform
-    {
-        return Platform::from($platform);
-    }
 }

src/Constants/SettingConstant.php

@@ -16,9 +16,9 @@ final class SettingConstant
 
     public const string SETTING_CUSTOM_EVENTS = self::SETTING_PREFIX . 'cus';
 
-    public const string SETTING_GITHUB_EVENTS = self::SETTING_CUSTOM_EVENTS . EventConstant::GITHUB_EVENT_SEPARATOR;
+    public const string SETTING_GITHUB_EVENTS = self::SETTING_CUSTOM_EVENTS . 'gh.';
 
-    public const string SETTING_GITLAB_EVENTS = self::SETTING_CUSTOM_EVENTS . EventConstant::GITLAB_EVENT_SEPARATOR;
+    public const string SETTING_GITLAB_EVENTS = self::SETTING_CUSTOM_EVENTS . 'gl.';
 
     public const int BTN_LINE_ITEM_COUNT = 2;
 

src/Enums/Platform.php

@@ -7,6 +7,8 @@ enum Platform: string
     case GITHUB = 'github';
     case GITLAB = 'gitlab';
 
+    public const DEFAULT = 'github';
+
     public function eventSeparator(): string
     {
         return match ($this) {

src/Interfaces/EventInterface.php

@@ -2,7 +2,7 @@
 
 namespace CSlant\TelegramGitNotifier\Interfaces;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 use CSlant\TelegramGitNotifier\Exceptions\ConfigFileException;
 use CSlant\TelegramGitNotifier\Trait\ActionEventTrait;
 use Symfony\Component\HttpFoundation\Request;
@@ -28,7 +28,7 @@ public function getActionOfEvent(object $payload): string;
      * @return void
      * @see EventTrait::setPlatFormForEvent()
      */
-    public function setPlatFormForEvent(?string $platform = EventConstant::DEFAULT_PLATFORM, ?string $platformFile = null): void;
+    public function setPlatFormForEvent(?string $platform = Platform::DEFAULT, ?string $platformFile = null): void;
 
     /**
      * Set event config and get event name

src/Interfaces/SettingInterface.php

@@ -2,7 +2,7 @@
 
 namespace CSlant\TelegramGitNotifier\Interfaces;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 use CSlant\TelegramGitNotifier\Exceptions\BotException;
 use CSlant\TelegramGitNotifier\Exceptions\InvalidViewTemplateException;
 use CSlant\TelegramGitNotifier\Exceptions\MessageIsEmptyException;
@@ -20,7 +20,7 @@ interface SettingInterface
      * @return array<int, array<int, mixed>>
      * @see EventSettingTrait::eventMarkup()
      */
-    public function eventMarkup(?string $parentEvent = null, string $platform = EventConstant::DEFAULT_PLATFORM): array;
+    public function eventMarkup(?string $parentEvent = null, string $platform = Platform::DEFAULT): array;
 
     /**
      * Get callback data for markup

src/Models/Event.php

@@ -2,7 +2,6 @@
 
 namespace CSlant\TelegramGitNotifier\Models;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
 use CSlant\TelegramGitNotifier\Enums\Platform;
 
 class Event
@@ -14,7 +13,7 @@ class Event
 
     private string $lastLoadedFile = '';
 
-    public string $platform = EventConstant::DEFAULT_PLATFORM;
+    public string $platform = Platform::DEFAULT;
 
     public string $platformFile = '' {
         get {
@@ -39,7 +38,7 @@ public function getEventConfig(): array
      */
     public function setEventConfig(?string $platform = null): void
     {
-        $newPlatform = $platform ?? EventConstant::DEFAULT_PLATFORM;
+        $newPlatform = $platform ?? Platform::DEFAULT;
         $this->platform = $newPlatform;
 
         // Skip re-reading if same file is already loaded

src/Notifier.php

@@ -2,8 +2,8 @@
 
 namespace CSlant\TelegramGitNotifier;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
 use CSlant\TelegramGitNotifier\DTOs\ChatTarget;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 use CSlant\TelegramGitNotifier\Exceptions\ConfigFileException;
 use CSlant\TelegramGitNotifier\Interfaces\EventInterface;
 use CSlant\TelegramGitNotifier\Interfaces\Structures\NotificationInterface;
@@ -32,7 +32,7 @@ public function __construct(
         ?Telegram $telegram = null,
         ?string $chatBotId = null,
         ?Event $event = null,
-        ?string $platform = EventConstant::DEFAULT_PLATFORM,
+        ?string $platform = Platform::DEFAULT,
         ?string $platformFile = null,
         ?Client $client = null,
     ) {

src/Structures/Notification.php

@@ -2,7 +2,7 @@
 
 namespace CSlant\TelegramGitNotifier\Structures;
 
-use CSlant\TelegramGitNotifier\Constants\EventConstant;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 use CSlant\TelegramGitNotifier\Exceptions\MessageIsEmptyException;
 use CSlant\TelegramGitNotifier\Exceptions\SendNotificationException;
 use GuzzleHttp\Exception\GuzzleException;
@@ -40,7 +40,7 @@ public function setPayload(Request $request, string $event): ?object
     {
         $content = match ($this->event->platform) {
             'gitlab' => $request->getContent(),
-            EventConstant::DEFAULT_PLATFORM => $request->request->get('payload'),
+            Platform::DEFAULT => $request->request->get('payload'),
             default => null,
         };
 

src/Trait/EventSettingTrait.php

@@ -4,6 +4,7 @@
 
 use CSlant\TelegramGitNotifier\Constants\EventConstant;
 use CSlant\TelegramGitNotifier\Constants\SettingConstant;
+use CSlant\TelegramGitNotifier\Enums\Platform;
 
 trait EventSettingTrait
 {
@@ -12,7 +13,7 @@ trait EventSettingTrait
      */
     public function eventMarkup(
         ?string $parentEvent = null,
-        string $platform = EventConstant::DEFAULT_PLATFORM,
+        string $platform = Platform::DEFAULT,
         ?string $platformFile = null
     ): array {
         $replyMarkup = $replyMarkupItem = [];
@@ -56,9 +57,7 @@ public function getCallbackData(
         array|bool $value = false,
         ?string $parentEvent = null
     ): string {
-        $platformSeparator = $platform === EventConstant::DEFAULT_PLATFORM
-            ? EventConstant::GITHUB_EVENT_SEPARATOR
-            : EventConstant::GITLAB_EVENT_SEPARATOR;
+        $platformSeparator = Platform::from($platform)->eventSeparator();
         $prefix = EventConstant::EVENT_PREFIX . $platformSeparator;
 
         if (is_array($value)) {
@@ -136,14 +135,14 @@ public function getPlatformFromCallback(
         }
 
         if ($callback) {
-            return match (true) {
-                str_contains($callback, EventConstant::GITHUB_EVENT_SEPARATOR) => 'github',
-                str_contains($callback, EventConstant::GITLAB_EVENT_SEPARATOR) => 'gitlab',
-                default => EventConstant::DEFAULT_PLATFORM,
-            };
+            foreach (Platform::cases() as $p) {
+                if (str_contains($callback, $p->eventSeparator())) {
+                    return $p->value;
+                }
+            }
         }
 
-        return EventConstant::DEFAULT_PLATFORM;
+        return Platform::DEFAULT;
     }
 
     public function sendSettingEventMessage(
@@ -175,11 +174,16 @@ public function getEventFromCallback(?string $callback): string
             return '';
         }
 
-        return str_replace([
-            EventConstant::EVENT_PREFIX,
-            EventConstant::GITHUB_EVENT_SEPARATOR,
-            EventConstant::GITLAB_EVENT_SEPARATOR,
-        ], '', $callback);
+        $separators = array_map(
+            fn (Platform $p) => $p->eventSeparator(),
+            Platform::cases(),
+        );
+
+        return str_replace(
+            [EventConstant::EVENT_PREFIX, ...$separators],
+            '',
+            $callback,
+        );
     }
 
     public function handleEventWithActions(
@@ -235,7 +239,7 @@ public function eventUpdateHandle(
         $this->event->updateEvent($event, $action);
         $this->eventHandle(
             $action
-                ? EventConstant::PLATFORM_EVENT_SEPARATOR[$platform]
+                ? Platform::from($platform)->eventSeparator()
                 . EventConstant::EVENT_HAS_ACTION_SEPARATOR . $event
                 : null,
             $platform