改名

# Conflicts:
#	kiri-engine/FileListen/HotReload.php

function.php

@@ -72,7 +72,7 @@ if (!function_exists('checkPortIsAlready')) {
 
 		$serverPid = file_get_contents(storage('.swoole.pid'));
 		if (!empty($serverPid) && shell_exec('ps -ef | grep ' . $serverPid . ' | grep -v grep')) {
-			Process::kill($serverPid, SIGTERM);
+			Process::kill($serverPid, 0) && Process::kill($serverPid, SIGTERM);
 		}
 
 		exec('netstat -lnp | grep ' . $port . ' | grep "LISTEN" | awk \'{print $7}\'', $output);

kiri-engine/Abstracts/BaseObject.php

@@ -21,199 +21,223 @@ use Swoole\Coroutine;
 class BaseObject implements Configure
 {
 
-    /**
-     * BaseAbstract constructor.
-     *
-     * @param array $config
-     * @throws Exception
-     */
-    public function __construct(array $config = [])
-    {
-        if (!empty($config) && is_array($config)) {
-            Kiri::configure($this, $config);
-        }
-    }
+	/**
+	 * BaseAbstract constructor.
+	 *
+	 * @param array $config
+	 * @throws Exception
+	 */
+	public function __construct(array $config = [])
+	{
+		if (!empty($config) && is_array($config)) {
+			Kiri::configure($this, $config);
+		}
+	}
 
 
-    /**
-     * @throws Exception
-     */
-    public function init()
-    {
-    }
+	/**
+	 * @throws Exception
+	 */
+	public function init()
+	{
+	}
 
 
-    /**
-     * @param array|callable $callback
-     * @param object $scope
-     */
-    public function async_create(array|callable $callback, object $scope)
-    {
-        Coroutine::create($callback, $scope);
-    }
+	/**
+	 * @param array|callable $callback
+	 * @param object $scope
+	 */
+	public function async_create(array|callable $callback, object $scope)
+	{
+		Coroutine::create($callback, $scope);
+	}
 
 
-    /**
-     * @return string
-     */
-    #[Pure] public static function className(): string
-    {
-        return static::class;
-    }
+	/**
+	 * @return string
+	 */
+	#[Pure] public static function className(): string
+	{
+		return static::class;
+	}
 
-    /**
-     * @param $name
-     * @param $value
-     *
-     * @throws Exception
-     */
-    public function __set($name, $value)
-    {
-        $method = 'set' . ucfirst($name);
-        if (method_exists($this, $method)) {
-            $this->{$method}($value);
-        } else {
-            throw new Exception('The set name ' . $name . ' not find in class ' . static::class);
-        }
-    }
+	/**
+	 * @param $name
+	 * @param $value
+	 *
+	 * @throws Exception
+	 */
+	public function __set($name, $value)
+	{
+		$method = 'set' . ucfirst($name);
+		if (method_exists($this, $method)) {
+			$this->{$method}($value);
+		} else {
+			throw new Exception('The set name ' . $name . ' not find in class ' . static::class);
+		}
+	}
 
-    /**
-     * @param $name
-     *
-     * @return mixed
-     * @throws Exception
-     */
-    public function __get($name): mixed
-    {
-        $method = 'get' . ucfirst($name);
-        if (method_exists($this, $method)) {
-            return $this->$method();
-        } else {
-            throw new Exception('The get name ' . $name . ' not find in class ' . static::class);
-        }
-    }
+	/**
+	 * @param $name
+	 *
+	 * @return mixed
+	 * @throws Exception
+	 */
+	public function __get($name): mixed
+	{
+		$method = 'get' . ucfirst($name);
+		if (method_exists($this, $method)) {
+			return $this->$method();
+		} else {
+			throw new Exception('The get name ' . $name . ' not find in class ' . static::class);
+		}
+	}
 
 
-    /**
-     * @param $message
-     * @param string $model
-     * @return bool
-     * @throws Exception
-     */
-    public function addError($message, string $model = 'app'): bool
-    {
-        if ($message instanceof \Throwable) {
-            $this->error(jTraceEx($message));
-        } else {
-            if (!is_string($message)) {
-                $message = json_encode($message, JSON_UNESCAPED_UNICODE);
-            }
-            $this->error($message);
-        }
-        return FALSE;
-    }
+	/**
+	 * @param $message
+	 * @param string $model
+	 * @return bool
+	 * @throws Exception
+	 */
+	public function addError($message, string $model = 'app'): bool
+	{
+		if ($message instanceof \Throwable) {
+			$this->error(jTraceEx($message));
+		} else {
+			if (!is_string($message)) {
+				$message = json_encode($message, JSON_UNESCAPED_UNICODE);
+			}
+			$this->error($message);
+		}
+		return FALSE;
+	}
 
 
-    /**
-     * @return Logger
-     * @throws Exception
-     */
-    private function logger(): Logger
-    {
-        return Kiri::getDi()->get(Logger::class);
-    }
+	/**
+	 * @return Logger
+	 * @throws Exception
+	 */
+	private function logger(): Logger
+	{
+		return Kiri::getDi()->get(Logger::class);
+	}
 
 
-    /**
-     * @param mixed $message
-     * @param string $method
-     * @param string $file
-     * @throws Exception
-     */
-    public function debug(mixed $message, string $method = __METHOD__, string $file = __FILE__)
-    {
-        if (!is_string($message)) {
-            $message = print_r($message, true);
-        }
-        $message = "\033[35m[" . date('Y-m-d H:i:s') . '][DEBUG]: ' . $message . "\033[0m";
-
-        $this->logger()->debug(Logger::DEBUG, [$message, $method, $file]);
-    }
+	/**
+	 * @param mixed $message
+	 * @param string $method
+	 * @param string $file
+	 * @throws Exception
+	 */
+	public function debug(mixed $message, string $method = '', string $file = '')
+	{
+		if (!is_string($message)) {
+			$message = print_r($message, true);
+		}
+		$message = "\033[35m" . $message . "\033[0m";
 
 
-    /**
-     * @param mixed $message
-     * @param string $method
-     * @param string $file
-     * @throws Exception
-     */
-    public function info(mixed $message, string $method = __METHOD__, string $file = __FILE__)
-    {
-        if (!is_string($message)) {
-            $message = print_r($message, true);
-        }
-        $message = "\033[34m[" . date('Y-m-d H:i:s') . '][INFO]: ' . $message . "\033[0m";
+		$context = [];
+		if (!empty($method)) $context['method'] = $method;
+		if (!empty($file)) $context['file'] = $file;
 
-        $this->logger()->info(Logger::NOTICE, [$message, $method, $file]);
-    }
+		$this->logger()->debug($message, $context);
+	}
 
 
-    /**
-     * @param mixed $message
-     * @param string $method
-     * @param string $file
-     * @throws Exception
-     */
-    public function success(mixed $message, string $method = __METHOD__, string $file = __FILE__)
-    {
-        if (!is_string($message)) {
-            $message = print_r($message, true);
-        }
-
-        $message = "\033[36m[" . date('Y-m-d H:i:s') . '][SUCCESS]: ' . $message . "\033[0m";
-
-        $this->logger()->notice(Logger::NOTICE, [$message, $method, $file]);
-    }
+	/**
+	 * @param mixed $message
+	 * @param string $method
+	 * @param string $file
+	 * @throws Exception
+	 */
+	public function info(mixed $message, string $method = '', string $file = '')
+	{
+		if (!is_string($message)) {
+			$message = print_r($message, true);
+		}
+		$message = "\033[34m" . $message . "\033[0m";
 
 
-    /**
-     * @param mixed $message
-     * @param string $method
-     * @param string $file
-     * @throws Exception
-     */
-    public function warning(mixed $message, string $method = __METHOD__, string $file = __FILE__)
-    {
-        if (!is_string($message)) {
-            $message = print_r($message, true);
-        }
+		$context = [];
+		if (!empty($method)) $context['method'] = $method;
+		if (!empty($file)) $context['file'] = $file;
 
-        $message = "\033[33m[" . date('Y-m-d H:i:s') . '][WARNING]: ' . $message . "\033[0m";
-
-        $this->logger()->critical(Logger::NOTICE, [$message, $method, $file]);
-    }
+		$this->logger()->info($message, $context);
+	}
 
 
-    /**
-     * @param mixed $message
-     * @param null $method
-     * @param null $file
-     * @throws Exception
-     */
-    public function error(mixed $message, $method = null, $file = null)
-    {
-        if ($message instanceof \Throwable) {
-            $message = $message->getMessage() . " on line " . $message->getLine() . " at file " . $message->getFile();
-        }
-        $content = (empty($method) ? '' : $method . ': ') . $message;
+	/**
+	 * @param mixed $message
+	 * @param string $method
+	 * @param string $file
+	 * @throws Exception
+	 */
+	public function success(mixed $message, string $method = '', string $file = '')
+	{
+		if (!is_string($message)) {
+			$message = print_r($message, true);
+		}
 
-        $message = "\033[41;37m[" . date('Y-m-d H:i:s') . '][ERROR]: ' . $content . "\033[0m";
+		$message = "\033[36m" . $message . "\033[0m";
 
-        if (!empty($file)) {
-            $message .= PHP_EOL . "\033[41;37m[" . date('Y-m-d H:i:s') . '][ERROR]: ' . $file . "\033[0m";
-        }
 
-        $this->logger()->error(Logger::ERROR, [$message, $method, $file]);
-    }
+		$context = [];
+		if (!empty($method)) $context['method'] = $method;
+		if (!empty($file)) $context['file'] = $file;
+
+		$this->logger()->notice($message, $context);
+	}
+
+
+	/**
+	 * @param mixed $message
+	 * @param string $method
+	 * @param string $file
+	 * @throws Exception
+	 */
+	public function warning(mixed $message, string $method = '', string $file = '')
+	{
+		if (!is_string($message)) {
+			$message = print_r($message, true);
+		}
+
+		$message = "\033[33m" . $message . "\033[0m";
+
+
+		$context = [];
+		if (!empty($method)) $context['method'] = $method;
+		if (!empty($file)) $context['file'] = $file;
+
+		$this->logger()->critical($message, $context);
+	}
+
+
+	/**
+	 * @param mixed $message
+	 * @param null $method
+	 * @param null $file
+	 * @throws Exception
+	 */
+	public function error(mixed $message, $method = null, $file = null)
+	{
+		if ($message instanceof \Throwable) {
+			$message = $message->getMessage() . " on line " . $message->getLine() . " at file " . $message->getFile();
+		}
+		$content = (empty($method) ? '' : $method . ': ') . $message;
+
+		$message = "\033[41;37m" . $content . "\033[0m";
+
+		if (!empty($file)) {
+			$message .= PHP_EOL . "\03341;37m[" . $file . "\033[0m";
+		}
+
+		$context = [];
+		if (!empty($method)) $context['method'] = $method;
+		if (!empty($file)) $context['file'] = $file;
+
+		$this->logger()->error($message, $context);
+	}
 
 }

kiri-engine/Application.php

@@ -223,6 +223,7 @@ class Application extends BaseApplication
 	}
 
 
+
 	/**
 	 * @throws NotFindClassException
 	 * @throws ReflectionException
@@ -240,6 +241,8 @@ class Application extends BaseApplication
 	}
 
 
+
+
 	/**
 	 * @param $className
 	 * @param null $abstracts

kiri-engine/Error/LoggerProcess.php

@@ -20,22 +20,16 @@ use Server\Abstracts\BaseProcess;
 class LoggerProcess extends BaseProcess
 {
 
-	/**
-	 * @param Process $process
-	 * @return string
-	 */
-	#[Pure] public function getProcessName(Process $process): string
-	{
-		// TODO: Implement getProcessName() method.
-		return get_called_class();
-	}
+
+	public string $name = 'logger process';
+
 
 
 	/**
 	 * @param Process $process
 	 * @throws ComponentException
 	 */
-	public function onHandler(Process $process): void
+	public function process(Process $process): void
 	{
 		// TODO: Implement onHandler() method.
 		$this->message($process);
@@ -49,10 +43,6 @@ class LoggerProcess extends BaseProcess
 	 */
 	public function message(Process $process)
 	{
-		if ($this->checkProcessIsStop()) {
-			$this->exit();
-			return;
-		}
 		$message = Json::decode($process->read());
 		if (!empty($message)) {
 			Kiri::writeFile($this->getDirName($message), $message[0], FILE_APPEND);

kiri-engine/FileListen/HotReload.php

@@ -2,9 +2,10 @@
 
 namespace Kiri\FileListen;
 
+use Annotation\Inject;
 use Exception;
 use Kiri\Abstracts\Config;
-use Kiri\Abstracts\Logger;
+use Kiri\Error\Logger;
 use Kiri\Exception\ConfigException;
 use Kiri\Kiri;
 use Swoole\Coroutine;
@@ -24,12 +25,24 @@ class HotReload extends Command
 	public bool $isReloading = false;
 	public bool $isReloadingOut = false;
 	public ?array $dirs = [];
+
 	public int $events;
 
 	public int $int = -1;
 
 
+	private ?Process $process = null;
+
+
+	public Inotify|Scaner $driver;
+
+
+	#[Inject(Logger::class)]
+	public Logger $logger;
+
+
 	protected mixed $source = null;
+
 	protected mixed $pipes = [];
 
 	protected ?Coroutine\Channel $channel = null;
@@ -56,40 +69,51 @@ class HotReload extends Command
 	{
 		// TODO: Implement onHandler() method.
 		set_error_handler([$this, 'onErrorHandler']);
-
 		$this->dirs = Config::get('inotify', [APP_PATH . 'app']);
+		swoole_async_set(['enable_coroutine' => false]);
 		if (!extension_loaded('inotify')) {
-			$driver = Kiri::getDi()->get(Scaner::class, [$this->dirs, $this]);
+			$this->driver = Kiri::getDi()->get(Scaner::class, [$this->dirs, $this]);
 		} else {
-			$driver = Kiri::getDi()->get(Inotify::class, [$this->dirs, $this]);
+			$this->driver = Kiri::getDi()->get(Inotify::class, [$this->dirs, $this]);
 		}
-
 		if (Kiri::getPlatform()->isLinux()) {
 			swoole_set_process_name('[' . Config::get('id', 'sw service.') . '].sw:wather');
 		}
 		$this->trigger_reload();
-		Coroutine::create(function () use ($driver) {
-			$driver->start();
-		});
+
+		var_dump(getmypid());
+		Process::signal(SIGTERM, [$this, 'onSignal']);
+		Process::signal(SIGKILL, [$this, 'onSignal']);
+
+		$this->driver->start();
 		return 0;
 	}
 
 
 	/**
+	 * @param $data
 	 * @throws Exception
 	 */
-	private function stop(): void
+	public function onSignal($data)
 	{
-		if (is_resource($this->source)) {
-			proc_terminate($this->source);
-			while (proc_get_status($this->source)['running']) {
-				Coroutine::sleep(1);
-			}
-			proc_close($this->source);
-			$this->source = null;
+		if (!$data) {
+			return;
+		}
+		$this->driver->clear();
+		$pid = file_get_contents(storage('.swoole.pid'));
+		if (!empty($pid) && Process::kill($pid, 0)) {
+			Process::kill($pid, SIGTERM);
+		}
+		if ($this->process && Process::kill($this->process->pid, 0)) {
+			Process::kill($this->process->pid) && Process::wait(true);
+		}
+		while ($ret = Process::wait(true)) {
+			echo "PID={$ret['pid']}\n";
+			sleep(1);
 		}
 	}
 
+
 	/**
 	 * @param $code
 	 * @param $message
@@ -113,12 +137,18 @@ class HotReload extends Command
 	 */
 	public function trigger_reload()
 	{
-		Kiri::getDi()->get(Logger::class)->warning('change reload');
-
-		$this->stop();
-		Coroutine::create(function () {
-			$this->source = proc_open("php " . APP_PATH . "kiri.php", [], $pipes);
+		$this->logger->warning('change reload');
+//		$pid = file_get_contents(storage('.swoole.pid'));
+//		if (!empty($pid) && Process::kill($pid, 0)) {
+//			Process::kill($pid, SIGTERM);
+//		}
+		if ($this->process && Process::kill($this->process->pid, 0)) {
+			Process::kill($this->process->pid) && Process::wait(true);
+		}
+		$this->process = new Process(function (Process $process) {
+			$process->exec(PHP_BINARY, [APP_PATH . "kiri.php", "sw:server", "restart"]);
 		});
+		$this->process->start();
 	}
 
 

kiri-engine/FileListen/Inotify.php

@@ -47,13 +47,17 @@ class Inotify
 	public function clear()
 	{
 		Event::del($this->inotify);
+
+        var_dump('clear event');
+
 		Event::exit();
 	}
 
 
-	/**
-	 * 开始监听
-	 */
+    /**
+     * 开始监听
+     * @throws Exception
+     */
 	public function check()
 	{
 		if (!($events = inotify_read($this->inotify))) {
@@ -75,9 +79,8 @@ class Inotify
 				if ($this->process->int !== -1) {
 					return;
 				}
-				$this->process->int = @swoole_timer_after(2000, [$this, 'reload']);
-
-				$this->process->isReloading = true;
+                usleep(200);
+				$this->reload();
 			}
 		}
 	}
@@ -89,7 +92,6 @@ class Inotify
 	{
 		$this->process->isReloading = true;
 		$this->process->trigger_reload();
-
 		$this->clearWatch();
 		foreach ($this->dirs as $root) {
 			$this->watch($root);
@@ -151,7 +153,6 @@ class Inotify
 			} else if (!str_ends_with($f, '.php')) {
 				continue;
 			}
-
 			//检测文件类型
 			if (strstr($f, '.') == '.php') {
 				$wd = @inotify_add_watch($this->inotify, $path, $this->events);