Use trap method instead of pcntl_signal for better signal handling

[Thavarshan]

Description

This PR replaces the direct use of pcntl_signal with the trap method provided by the InteractsWithSignals trait in the TopCommand class. The trap method allows for multiple handlers for the same signal, preventing potential issues in environments like Laravel Octane where multiple handlers might be necessary.

Changes

• Replaced pcntl_signal(SIGINT, function () use ($guiBuilder) { ... }); with $this->trap(SIGINT, function () use ($guiBuilder) { ... });

Benefits

• Supports multiple handlers for signals, ensuring that no handlers are overwritten.
• Provides better integration with Laravel's command lifecycle and signal management.
• Enhances the robustness and reliability of signal handling in the application.

Testing

• Tested the changes locally to ensure that the signal handling works as expected.
• Verified that the application exits alternate screen mode and shows the cursor upon receiving SIGINT.

[marcioal1991]

This closes the #4 issue.

Cheers!

[leventcz]

Thank you for your contribution.

I like this approach and want to merge it, but it's not working for me right now.

• Verified that the application exits alternate screen mode and shows the cursor upon receiving SIGINT.

I'm using Docker with PHP 8.2 and have PCNTL enabled. Somehow, it is not triggering the callback.

I didn't have this issue with the previous version.

I'll investigate further.