chore: add a few refactors

Author: owenvoke

Diff for: .env.example

@@ -0,0 +1 @@
+TRAKT_API_TOKEN=

Diff for: .gitignore

@@ -1,6 +1,7 @@
 /build
 /torrents
 /vendor
+/.env
 /.phpunit.result.cache
 /.php-cs-fixer.cache
 /composer.lock

Diff for: app/Commands/DownloadCommand.php

@@ -15,7 +15,7 @@
 class DownloadCommand extends Command
 {
     /** {@inheritdoc} */
-    protected $signature = 'download { trakt-user= : Trakt username for the list }
+    protected $signature = 'download { trakt-user? : Trakt username for the list }
                                      { --l|list=watchlist : A custom list id or stub }
                                      { --o|output=./torrents : The directory to output data to }
                                      { --quality=1080p : The quality to download (720p, 1080p or 3D) }
@@ -33,12 +33,11 @@ class DownloadCommand extends Command
 
     private YTSClient $yts;
 
-    public function handle(TraktClient $traktClient): void
+    public function handle(TraktClient $traktClient, YTSClient $ytsClient): void
     {
         $this->trakt = $traktClient;
         $this->yts = $ytsClient;
 
-        $
         $this->quality = Quality::tryFrom($this->option('quality')) ?? Quality::Q_1080P;
 
         try {

Diff for: app/Exceptions/Handler.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Exceptions;
+
+use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
+use function Termwind\render;
+use Throwable;
+
+class Handler extends ExceptionHandler
+{
+    public function renderForConsole($output, Throwable $e): void
+    {
+        if ($e instanceof UnauthorizedException) {
+            render(view('unauthorised')->render());
+
+            return;
+        }
+
+        if ($e instanceof TraktToYTSException) {
+            render(view('error', ['message' =>  $e->getMessage()])->render());
+
+            return;
+        }
+
+        parent::renderForConsole($output, $e);
+    }
+}

Diff for: app/Exceptions/NoMovieDataFound.php renamed to app/Exceptions/NoMovieDataFoundException.php

@@ -4,7 +4,7 @@
 
 use RuntimeException;
 
-class NoMovieDataFound extends RuntimeException
+class NoMovieDataFoundException extends RuntimeException implements TraktToYTSException
 {
     public static function forImdbIdOnYts(string $imdbId): self
     {

Diff for: app/Exceptions/TraktToYTSException.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Exceptions;
+
+interface TraktToYTSException
+{
+}

Diff for: app/Exceptions/ApiKeyNotSpecified.php renamed to app/Exceptions/UnauthorizedException.php

@@ -4,7 +4,7 @@
 
 use RuntimeException;
 
-class ApiKeyNotSpecified extends RuntimeException
+class UnauthorizedException extends RuntimeException implements TraktToYTSException
 {
     public static function forServiceWithId(string $serviceId): self
     {

Diff for: app/Providers/AppServiceProvider.php

@@ -2,7 +2,7 @@
 
 namespace App\Providers;
 
-use App\Exceptions\ApiKeyNotSpecified;
+use App\Exceptions\UnauthorizedException;
 use App\Services\Trakt\Client;
 use Illuminate\Config\Repository;
 use Illuminate\Support\ServiceProvider;
@@ -13,7 +13,7 @@ public function register(): void
     {
         $this->app->singleton(Client::class, function (): Client {
             if (! $traktApiKey = $this->app->get(Repository::class)->get('services.trakt.token')) {
-                throw ApiKeyNotSpecified::forServiceWithId(Client::SERVICE_ID);
+                throw UnauthorizedException::forServiceWithId(Client::SERVICE_ID);
             }
 
             return new Client($traktApiKey);

Diff for: app/Services/YTS/Client.php

@@ -2,7 +2,7 @@
 
 namespace App\Services\YTS;
 
-use App\Exceptions\NoMovieDataFound;
+use App\Exceptions\NoMovieDataFoundException;
 use App\Services\YTS\Enums\Quality;
 use App\Services\YTS\ValueObjects\Movie;
 use App\Services\YTS\ValueObjects\Torrent;
@@ -11,15 +11,8 @@
 
 class Client
 {
-    public const SERVICE_ID = 'YTS';
-
     public const BASE_URI = 'https://yts.mx';
 
-    /**
-     * @param  string  $imdbId
-     * @param  Quality|null  $quality
-     * @return Movie
-     */
     public function getMovieByImdbId(string|null $imdbId, Quality|null $quality = null): Movie
     {
         $response = Http::baseUrl(self::BASE_URI)->get('/api/v2/list_movies.json', [
@@ -28,7 +21,7 @@ public function getMovieByImdbId(string|null $imdbId, Quality|null $quality = nu
         ])->json();
 
         if (! Arr::has($response, 'data.movies.0')) {
-            throw NoMovieDataFound::forImdbIdOnYts($imdbId);
+            throw NoMovieDataFoundException::forImdbIdOnYts($imdbId);
         }
 
         return Movie::fromResponse(

Diff for: bootstrap/app.php

@@ -33,7 +33,7 @@
 
 $app->singleton(
     Illuminate\Contracts\Debug\ExceptionHandler::class,
-    Illuminate\Foundation\Exceptions\Handler::class
+    App\Exceptions\Handler::class
 );
 
 /*

Diff for: box.json

@@ -3,11 +3,13 @@
         "app",
         "bootstrap",
         "config",
+        "resources",
         "vendor"
     ],
     "files": [
         "composer.json"
     ],
+    "exclude-dev-files": false,
     "exclude-composer-files": false,
     "compression": "GZ",
     "compactors": [

Diff for: builds/trakt-to-yts

@@ -0,0 +1,11 @@
+<?php
+
+declare(strict_types=1);
+
+return [
+    'paths' => [
+        resource_path('views'),
+    ],
+
+    'compiled' => sys_get_temp_dir(),
+];

Diff for: config/view.php

@@ -0,0 +1,3 @@
+<div {{ $attributes->merge(['class' => 'ml-2 my-1']) }}>
+    {{ $slot }}
+</div>

Diff for: resources/views/components/layouts/app.blade.php

@@ -0,0 +1,3 @@
+<x-layouts.app>
+    <span class="text-red">{{ $notice }}</span>
+</x-layouts.app>

Diff for: resources/views/error.blade.php

@@ -0,0 +1,3 @@
+<x-layouts.app>
+    <span>{{ $notice }}</span>
+</x-layouts.app>

Diff for: resources/views/notice.blade.php

@@ -0,0 +1,9 @@
+<x-layouts.app>
+    <div class="text-red-500">
+        <span>Unauthorised</span>
+    </div>
+
+    <p class="mt-1">
+        Please set the <span class="font-bold">TRAKT_API_TOKEN</span> environment variable to your <a href="https://trakt.tv/oauth/applications">Trakt API token</a>.
+    </p>
+</x-layouts.app>