In `@app/controllers/general.php`:
- Around line 706-712: The call to $bus->dispatch(new ExecutionCompleted(...))
in the finally block can throw (e.g. PersistExecutionLog::handle) and mask an
earlier exception; wrap the dispatch call in its own try-catch that catches
Throwable, logs the dispatch error (using the controller's logger or error_log)
and does not rethrow so the original exception from the catch block is
preserved; specifically, around $bus->dispatch(...) for the ($type ===
'function' || $type === 'site') case, add try { $bus->dispatch(new
ExecutionCompleted(...)); } catch (\Throwable $e) { /* log error about
ExecutionCompleted dispatch failure including $e->getMessage()/trace */ }.

In `@src/Appwrite/Bus/Event.php`:
- Line 5: The interface declaration `interface Event {}` violates PSR-12 brace
placement; change it so the opening brace is on its own line for the `Event`
interface (i.e., declare `interface Event` then put `{` on the next line) to
satisfy the linter and Pint's `class_definition`/`braces_position` rule.

In `@src/Appwrite/Bus/ExecutionCompleted.php`:
- Around line 7-10: The constructor in class ExecutionCompleted uses inline
empty braces which triggers the PSR-12 braces_position lint; update the
__construct declaration in class ExecutionCompleted so the opening brace is on
the next line and the closing brace is on its own line (i.e., place the method
body braces on separate lines even if empty) to satisfy Pint/PSR-12.

In `@src/Appwrite/Bus/Listener.php`:
- Line 9: The Listener::$callback property is untyped and can be null while
getCallback() declares a non-nullable callable return; add a runtime guard and a
stronger type: declare the property as protected $callback (typed as ?callable
if your PHP version allows) and update getCallback() in class Listener to check
if $this->callback is callable and throw a clear LogicException (or
InvalidArgumentException) if not, so callers like Bus::dispatch never receive
null; alternatively, add a Listener constructor or initialize method that
validates/sets the callback (e.g., require a callable in Listener::__construct
or call setCallback) and ensure subclasses (like PersistExecutionLog) call it.

In `@src/Appwrite/Bus/Bus.php`:
- Around line 25-31: The dispatch method in Bus::dispatch silently uses a null
resolver and will pass raw injection names when setResolver() hasn't been
called, and an exception in one listener aborts remaining listeners; fix by
adding an explicit guard at the start of dispatch that throws a clear
RuntimeException if $this->resolver is null (or unset) so callers must call
setResolver() first, and wrap the per-listener invocation (the
($listener->getCallback())($event, ...$deps) call) in a try/catch to log or
rethrow per policy so one listener's exception doesn't silently stop others (or
alternatively add a docblock above dispatch stating fail-fast behavior if you
opt to keep current behavior); reference Bus::dispatch, setResolver,
$this->resolver, $listener->getInjections(), and $listener->getCallback().