Compare commits

..

34 commits
v1.1.0 ... main

Author SHA1 Message Date
4b0d17e2ae Fix flaky boundary test with freezeTime
All checks were successful
CI / ci (push) Successful in 5m36s
CI / ci (pull_request) Successful in 16m47s
Build and Push Docker Image / build (push) Successful in 4m15s
2026-03-18 21:54:20 +01:00
ff855facda Fix flaky test from duplicate Article factory URLs
Some checks failed
CI / ci (push) Successful in 5m14s
CI / ci (pull_request) Failing after 5m13s
2026-03-18 21:39:22 +01:00
a7317e376f 103 - Move notification bell to page header component
Some checks failed
CI / ci (push) Failing after 6m32s
CI / ci (pull_request) Successful in 5m35s
2026-03-18 21:27:14 +01:00
3d58baf156 102 - Fix route modals hidden behind overlay 2026-03-18 20:58:35 +01:00
268bbfd05a 101 - Fix cramped number inputs on Settings page 2026-03-18 20:52:49 +01:00
d2919758f5 Fix Pint 1.29.0 lint issues and update CI workflow
All checks were successful
CI / ci (push) Successful in 5m52s
CI / ci (pull_request) Successful in 5m46s
Build and Push Docker Image / build (push) Successful in 4m6s
2026-03-18 20:01:25 +01:00
dfb87f4152 Debug CI Pint environment
Some checks failed
CI / ci (push) Failing after 4m4s
CI / ci (pull_request) Failing after 4m4s
2026-03-18 19:48:48 +01:00
273fca76e3 Fix CI lint by using ZTS PHP to match dev container
Some checks failed
CI / ci (push) Failing after 4m37s
CI / ci (pull_request) Failing after 4m37s
2026-03-18 18:42:41 +01:00
cc94ba8e55 89 - Add article cleanup job with 30-day retention policy
Some checks failed
CI / ci (push) Failing after 4m35s
CI / ci (pull_request) Failing after 4m23s
2026-03-18 18:09:54 +01:00
bab2557e85 97 - Add auto_approve toggle to Route edit modal 2026-03-18 17:51:35 +01:00
9430158051 96 - Rework Articles page into route_article triage UI with tabs and actions 2026-03-18 17:39:38 +01:00
e7acbb6882 99 - Move publish_status from Article to RouteArticle with PublishStatusEnum 2026-03-18 17:31:47 +01:00
9fb373d139 98 - Add RouteArticle API endpoints for approve, reject, restore, and clear 2026-03-18 17:20:15 +01:00
2b74f24356 85 - Replace ArticleApproved with RouteArticleApproved event and update publishing listener 2026-03-18 17:00:56 +01:00
5e571babda 85 - Fix keyword matching to include title and description, add PHPStan type annotations 2026-03-18 16:35:12 +01:00
f3406b1713 85 - Remove approval_status from Article, migrate to route_articles 2026-03-18 16:23:46 +01:00
d0985fc57d 85 - Simplify Articles page to read-only feed log 2026-03-18 16:09:48 +01:00
0c35af4403 85 - Update publishing pipeline to use route_articles for per-route publishing 2026-03-18 16:05:31 +01:00
f449548123 85 - Update ValidateArticleListener for per-route validation flow 2026-03-18 15:51:33 +01:00
e3ea02ae1c 85 - Refactor ValidationService to per-route keyword evaluation with ApprovalStatusEnum 2026-03-18 15:46:15 +01:00
2a5a8c788b 85 - Add auto_approve column to routes table 2026-03-18 15:28:01 +01:00
b832d6d850 85 - Add route_articles table, model, and factory for per-route approval 2026-03-18 15:24:03 +01:00
d21c054250 94 - Add publishing failure notifications to job and listener
Some checks failed
CI / ci (push) Failing after 5m7s
2026-03-09 21:20:41 +01:00
062b00d01c 93 - Add feed staleness detection with configurable threshold and alerts 2026-03-09 17:32:55 +01:00
4feab96765 92 - Add in-app notification infrastructure with model, service, and bell component 2026-03-09 02:21:24 +01:00
ec09711a6f 21 - Add dynamic channel labels based on platform type
All checks were successful
CI / ci (push) Successful in 5m21s
CI / ci (pull_request) Successful in 5m39s
Build and Push Docker Image / build (push) Successful in 4m36s
2026-03-08 18:10:47 +01:00
bf96489362 50 - Add ActionPerformed event and LogActionListener for centralized DB logging 2026-03-08 17:53:43 +01:00
0bb10729de 78 - Convert Belga from website scraping to RSS feed parsing 2026-03-08 17:19:11 +01:00
19cbea9273 25 - Disable Vite in tests to fix CI manifest errors
All checks were successful
CI / ci (push) Successful in 5m24s
2026-03-08 14:58:00 +01:00
a448c54e73 25 - Fix PHPStan baseline discrepancy between local and CI
Some checks failed
CI / ci (push) Failing after 5m37s
2026-03-08 14:45:24 +01:00
675802e8e5 25 - Move PHPStan ignores to baseline, fix HttpFetcher type
Some checks failed
CI / ci (push) Failing after 4m42s
2026-03-08 14:33:54 +01:00
6784af2ff6 25 - Fix all PHPStan errors and add mockery extension
Some checks failed
CI / ci (push) Failing after 4m31s
2026-03-08 14:18:28 +01:00
56db303b15 25 - Fix code style issues across codebase with Pint 2026-03-08 14:17:55 +01:00
0b2fc5004b 25 - Add code coverage PR comment to CI workflow
Some checks failed
CI / ci (push) Failing after 4m11s
2026-03-08 12:58:01 +01:00
228 changed files with 5788 additions and 3281 deletions

113
.forgejo/workflows/ci.yml Normal file
View file

@ -0,0 +1,113 @@
name: CI
on:
push:
branches: ['release/*']
pull_request:
branches: [main]
jobs:
ci:
runs-on: docker
container:
image: catthehacker/ubuntu:act-latest
steps:
- uses: https://data.forgejo.org/actions/checkout@v4
- name: Set up PHP
uses: https://github.com/shivammathur/setup-php@v2
with:
php-version: '8.3'
extensions: pdo_sqlite, mbstring, xml, dom
coverage: pcov
- name: Cache Composer dependencies
uses: https://data.forgejo.org/actions/cache@v4
with:
path: ~/.composer/cache
key: composer-${{ hashFiles('composer.lock') }}
restore-keys: composer-
- name: Install dependencies
run: composer install --no-interaction --prefer-dist
- name: Prepare environment
run: cp .env.testing .env
- name: Lint
run: vendor/bin/pint --test
- name: Static analysis
run: vendor/bin/phpstan analyse
- name: Tests
run: php artisan test --coverage-clover coverage.xml --coverage-text
- name: Parse coverage
if: github.event_name == 'pull_request'
id: coverage
run: |
COVERAGE=$(php -r '
$xml = simplexml_load_file("coverage.xml");
if ($xml === false || !isset($xml->project->metrics)) {
echo "0";
exit;
}
$metrics = $xml->project->metrics;
$statements = (int) $metrics["statements"];
$covered = (int) $metrics["coveredstatements"];
echo $statements > 0 ? round(($covered / $statements) * 100, 2) : 0;
')
echo "percentage=$COVERAGE" >> "$GITHUB_OUTPUT"
- name: Comment coverage on PR
if: github.event_name == 'pull_request'
env:
FORGEJO_TOKEN: ${{ secrets.FORGEJO_TOKEN }}
PR_NUMBER: ${{ github.event.pull_request.number }}
COVERAGE: ${{ steps.coverage.outputs.percentage }}
REPO: ${{ github.repository }}
SERVER_URL: ${{ github.server_url }}
COMMIT_SHA: ${{ github.sha }}
run: |
API_URL="${SERVER_URL}/api/v1/repos/${REPO}/issues/${PR_NUMBER}/comments"
MARKER="<!-- ffr-ci-coverage-report -->"
BODY="${MARKER}
## Code Coverage Report
| Metric | Value |
|--------|-------|
| **Line Coverage** | ${COVERAGE}% |
_Updated by CI — commit ${COMMIT_SHA}_"
# Find existing coverage comment
EXISTING=$(curl -sf -H "Authorization: token ${FORGEJO_TOKEN}" \
"${API_URL}?limit=50" | \
php -r '
$comments = json_decode(file_get_contents("php://stdin"), true);
if (!is_array($comments)) exit;
foreach ($comments as $c) {
if (str_contains($c["body"], "<!-- ffr-ci-coverage-report -->")) {
echo $c["id"];
exit;
}
}
' || true)
if [ -n "$EXISTING" ]; then
# Update existing comment
curl -sf -X PATCH \
-H "Authorization: token ${FORGEJO_TOKEN}" \
-H "Content-Type: application/json" \
-d "$(php -r 'echo json_encode(["body" => $argv[1]]);' "$BODY")" \
"${SERVER_URL}/api/v1/repos/${REPO}/issues/comments/${EXISTING}" > /dev/null
else
# Create new comment
curl -sf -X POST \
-H "Authorization: token ${FORGEJO_TOKEN}" \
-H "Content-Type: application/json" \
-d "$(php -r 'echo json_encode(["body" => $argv[1]]);' "$BODY")" \
"${API_URL}" > /dev/null
fi

View file

@ -15,7 +15,7 @@ public function execute(string $name, string $provider, int $languageId, ?string
$url = config("feed.providers.{$provider}.languages.{$langCode}.url"); $url = config("feed.providers.{$provider}.languages.{$langCode}.url");
if (!$url) { if (! $url) {
throw new InvalidArgumentException("Invalid provider and language combination: {$provider}/{$langCode}"); throw new InvalidArgumentException("Invalid provider and language combination: {$provider}/{$langCode}");
} }

View file

@ -19,7 +19,7 @@ public function __construct(
*/ */
public function execute(string $instanceDomain, string $username, string $password, string $platform = 'lemmy'): PlatformAccount public function execute(string $instanceDomain, string $username, string $password, string $platform = 'lemmy'): PlatformAccount
{ {
$fullInstanceUrl = 'https://' . $instanceDomain; $fullInstanceUrl = 'https://'.$instanceDomain;
// Authenticate first — if this fails, no records are created // Authenticate first — if this fails, no records are created
$authResponse = $this->lemmyAuthService->authenticate($fullInstanceUrl, $username, $password); $authResponse = $this->lemmyAuthService->authenticate($fullInstanceUrl, $username, $password);

View file

@ -15,13 +15,13 @@ class FetchNewArticlesCommand extends Command
public function handle(): int public function handle(): int
{ {
if (!Setting::isArticleProcessingEnabled()) { if (! Setting::isArticleProcessingEnabled()) {
$this->info('Article processing is disabled. Article discovery skipped.'); $this->info('Article processing is disabled. Article discovery skipped.');
return self::SUCCESS; return self::SUCCESS;
} }
if (!Feed::where('is_active', true)->exists()) { if (! Feed::where('is_active', true)->exists()) {
$this->info('No active feeds found. Article discovery skipped.'); $this->info('No active feeds found. Article discovery skipped.');
return self::SUCCESS; return self::SUCCESS;

View file

@ -11,6 +11,7 @@ public function canParse(string $url): bool;
/** /**
* Extract article data from HTML * Extract article data from HTML
*
* @return array<string, mixed> * @return array<string, mixed>
*/ */
public function extractData(string $html): array; public function extractData(string $html): array;
@ -19,4 +20,4 @@ public function extractData(string $html): array;
* Get the source name for this parser * Get the source name for this parser
*/ */
public function getSourceName(): string; public function getSourceName(): string;
} }

View file

@ -11,6 +11,7 @@ public function canParse(string $url): bool;
/** /**
* Extract article URLs from homepage HTML * Extract article URLs from homepage HTML
*
* @return array<int, string> * @return array<int, string>
*/ */
public function extractArticleUrls(string $html): array; public function extractArticleUrls(string $html): array;
@ -24,4 +25,4 @@ public function getHomepageUrl(): string;
* Get the source name for this parser * Get the source name for this parser
*/ */
public function getSourceName(): string; public function getSourceName(): string;
} }

View file

@ -0,0 +1,10 @@
<?php
namespace App\Enums;
enum ApprovalStatusEnum: string
{
case PENDING = 'pending';
case APPROVED = 'approved';
case REJECTED = 'rejected';
}

View file

@ -0,0 +1,10 @@
<?php
namespace App\Enums;
enum NotificationSeverityEnum: string
{
case INFO = 'info';
case WARNING = 'warning';
case ERROR = 'error';
}

View file

@ -0,0 +1,21 @@
<?php
namespace App\Enums;
enum NotificationTypeEnum: string
{
case GENERAL = 'general';
case FEED_STALE = 'feed_stale';
case PUBLISH_FAILED = 'publish_failed';
case CREDENTIAL_EXPIRED = 'credential_expired';
public function label(): string
{
return match ($this) {
self::GENERAL => 'General',
self::FEED_STALE => 'Feed Stale',
self::PUBLISH_FAILED => 'Publish Failed',
self::CREDENTIAL_EXPIRED => 'Credential Expired',
};
}
}

View file

@ -5,4 +5,18 @@
enum PlatformEnum: string enum PlatformEnum: string
{ {
case LEMMY = 'lemmy'; case LEMMY = 'lemmy';
}
public function channelLabel(): string
{
return match ($this) {
self::LEMMY => 'Community',
};
}
public function channelLabelPlural(): string
{
return match ($this) {
self::LEMMY => 'Communities',
};
}
}

View file

@ -0,0 +1,11 @@
<?php
namespace App\Enums;
enum PublishStatusEnum: string
{
case UNPUBLISHED = 'unpublished';
case PUBLISHING = 'publishing';
case PUBLISHED = 'published';
case ERROR = 'error';
}

View file

@ -0,0 +1,18 @@
<?php
namespace App\Events;
use App\Enums\LogLevelEnum;
use Illuminate\Foundation\Events\Dispatchable;
class ActionPerformed
{
use Dispatchable;
public function __construct(
public string $message,
public LogLevelEnum $level = LogLevelEnum::INFO,
/** @var array<string, mixed> */
public array $context = [],
) {}
}

View file

@ -12,6 +12,5 @@ class ExceptionLogged
public function __construct( public function __construct(
public Log $log public Log $log
) { ) {}
}
} }

View file

@ -17,6 +17,5 @@ public function __construct(
public string $message, public string $message,
/** @var array<string, mixed> */ /** @var array<string, mixed> */
public array $context = [] public array $context = []
) { ) {}
}
} }

View file

@ -11,7 +11,5 @@ class NewArticleFetched
{ {
use Dispatchable, InteractsWithSockets, SerializesModels; use Dispatchable, InteractsWithSockets, SerializesModels;
public function __construct(public Article $article) public function __construct(public Article $article) {}
{
}
} }

View file

@ -2,16 +2,13 @@
namespace App\Events; namespace App\Events;
use App\Models\Article; use App\Models\RouteArticle;
use Illuminate\Foundation\Events\Dispatchable; use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Queue\SerializesModels; use Illuminate\Queue\SerializesModels;
class ArticleApproved class RouteArticleApproved
{ {
use Dispatchable, SerializesModels; use Dispatchable, SerializesModels;
public function __construct(public Article $article) public function __construct(public RouteArticle $routeArticle) {}
{
//
}
} }

View file

@ -4,6 +4,4 @@
use Exception; use Exception;
class ChannelException extends Exception class ChannelException extends Exception {}
{
}

View file

@ -11,7 +11,7 @@ class PublishException extends Exception
{ {
public function __construct( public function __construct(
private readonly Article $article, private readonly Article $article,
private readonly PlatformEnum|null $platform, private readonly ?PlatformEnum $platform,
?Throwable $previous = null ?Throwable $previous = null
) { ) {
$message = "Failed to publish article #$article->id"; $message = "Failed to publish article #$article->id";

View file

@ -4,6 +4,4 @@
use Exception; use Exception;
class RoutingException extends Exception class RoutingException extends Exception {}
{
}

View file

@ -10,4 +10,4 @@ protected static function getFacadeAccessor()
{ {
return \App\Services\Log\LogSaver::class; return \App\Services\Log\LogSaver::class;
} }
} }

View file

@ -3,13 +3,12 @@
namespace App\Http\Controllers\Api\V1; namespace App\Http\Controllers\Api\V1;
use App\Http\Resources\ArticleResource; use App\Http\Resources\ArticleResource;
use App\Jobs\ArticleDiscoveryJob;
use App\Models\Article; use App\Models\Article;
use App\Models\Setting; use App\Models\Setting;
use App\Jobs\ArticleDiscoveryJob;
use Exception; use Exception;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Support\Facades\Artisan;
class ArticlesController extends BaseController class ArticlesController extends BaseController
{ {
@ -41,40 +40,6 @@ public function index(Request $request): JsonResponse
]); ]);
} }
/**
* Approve an article
*/
public function approve(Article $article): JsonResponse
{
try {
$article->approve('manual');
return $this->sendResponse(
new ArticleResource($article->fresh(['feed', 'articlePublication'])),
'Article approved and queued for publishing.'
);
} catch (Exception $e) {
return $this->sendError('Failed to approve article: ' . $e->getMessage(), [], 500);
}
}
/**
* Reject an article
*/
public function reject(Article $article): JsonResponse
{
try {
$article->reject('manual');
return $this->sendResponse(
new ArticleResource($article->fresh(['feed', 'articlePublication'])),
'Article rejected.'
);
} catch (Exception $e) {
return $this->sendError('Failed to reject article: ' . $e->getMessage(), [], 500);
}
}
/** /**
* Manually refresh articles from all active feeds * Manually refresh articles from all active feeds
*/ */
@ -88,7 +53,7 @@ public function refresh(): JsonResponse
'Article refresh started. New articles will appear shortly.' 'Article refresh started. New articles will appear shortly.'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to start article refresh: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to start article refresh: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -5,7 +5,6 @@
use App\Models\User; use App\Models\User;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Validation\ValidationException; use Illuminate\Validation\ValidationException;
@ -24,7 +23,7 @@ public function login(Request $request): JsonResponse
$user = User::where('email', $request->email)->first(); $user = User::where('email', $request->email)->first();
if (!$user || !Hash::check($request->password, $user->password)) { if (! $user || ! Hash::check($request->password, $user->password)) {
return $this->sendError('Invalid credentials', [], 401); return $this->sendError('Invalid credentials', [], 401);
} }
@ -42,7 +41,7 @@ public function login(Request $request): JsonResponse
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Login failed: ' . $e->getMessage(), [], 500); return $this->sendError('Login failed: '.$e->getMessage(), [], 500);
} }
} }
@ -78,7 +77,7 @@ public function register(Request $request): JsonResponse
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Registration failed: ' . $e->getMessage(), [], 500); return $this->sendError('Registration failed: '.$e->getMessage(), [], 500);
} }
} }
@ -92,7 +91,7 @@ public function logout(Request $request): JsonResponse
return $this->sendResponse(null, 'Logged out successfully'); return $this->sendResponse(null, 'Logged out successfully');
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Logout failed: ' . $e->getMessage(), [], 500); return $this->sendError('Logout failed: '.$e->getMessage(), [], 500);
} }
} }
@ -109,4 +108,4 @@ public function me(Request $request): JsonResponse
], ],
], 'User retrieved successfully'); ], 'User retrieved successfully');
} }
} }

View file

@ -23,6 +23,8 @@ public function sendResponse(mixed $result, string $message = 'Success', int $co
/** /**
* Error response method * Error response method
*
* @param array<string, mixed> $errorMessages
*/ */
public function sendError(string $error, array $errorMessages = [], int $code = 400): JsonResponse public function sendError(string $error, array $errorMessages = [], int $code = 400): JsonResponse
{ {
@ -31,7 +33,7 @@ public function sendError(string $error, array $errorMessages = [], int $code =
'message' => $error, 'message' => $error,
]; ];
if (!empty($errorMessages)) { if (! empty($errorMessages)) {
$response['errors'] = $errorMessages; $response['errors'] = $errorMessages;
} }
@ -40,6 +42,8 @@ public function sendError(string $error, array $errorMessages = [], int $code =
/** /**
* Validation error response method * Validation error response method
*
* @param array<string, mixed> $errors
*/ */
public function sendValidationError(array $errors): JsonResponse public function sendValidationError(array $errors): JsonResponse
{ {
@ -61,4 +65,4 @@ public function sendUnauthorized(string $message = 'Unauthorized'): JsonResponse
{ {
return $this->sendError($message, [], 401); return $this->sendError($message, [], 401);
} }
} }

View file

@ -3,9 +3,6 @@
namespace App\Http\Controllers\Api\V1; namespace App\Http\Controllers\Api\V1;
use App\Models\Article; use App\Models\Article;
use App\Models\Feed;
use App\Models\PlatformAccount;
use App\Models\PlatformChannel;
use App\Services\DashboardStatsService; use App\Services\DashboardStatsService;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
@ -40,8 +37,7 @@ public function stats(Request $request): JsonResponse
'current_period' => $period, 'current_period' => $period,
]); ]);
} catch (\Exception $e) { } catch (\Exception $e) {
throw $e; return $this->sendError('Failed to fetch dashboard stats: '.$e->getMessage(), [], 500);
return $this->sendError('Failed to fetch dashboard stats: ' . $e->getMessage(), [], 500);
} }
} }
} }

View file

@ -10,8 +10,8 @@
use Exception; use Exception;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use InvalidArgumentException;
use Illuminate\Validation\ValidationException; use Illuminate\Validation\ValidationException;
use InvalidArgumentException;
class FeedsController extends BaseController class FeedsController extends BaseController
{ {
@ -21,7 +21,7 @@ class FeedsController extends BaseController
public function index(Request $request): JsonResponse public function index(Request $request): JsonResponse
{ {
$perPage = min($request->get('per_page', 15), 100); $perPage = min($request->get('per_page', 15), 100);
$feeds = Feed::with(['language']) $feeds = Feed::with(['language'])
->withCount('articles') ->withCount('articles')
->orderBy('is_active', 'desc') ->orderBy('is_active', 'desc')
@ -37,7 +37,7 @@ public function index(Request $request): JsonResponse
'total' => $feeds->total(), 'total' => $feeds->total(),
'from' => $feeds->firstItem(), 'from' => $feeds->firstItem(),
'to' => $feeds->lastItem(), 'to' => $feeds->lastItem(),
] ],
], 'Feeds retrieved successfully.'); ], 'Feeds retrieved successfully.');
} }
@ -64,7 +64,7 @@ public function store(StoreFeedRequest $request, CreateFeedAction $createFeedAct
} catch (InvalidArgumentException $e) { } catch (InvalidArgumentException $e) {
return $this->sendError($e->getMessage(), [], 422); return $this->sendError($e->getMessage(), [], 422);
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to create feed: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to create feed: '.$e->getMessage(), [], 500);
} }
} }
@ -97,7 +97,7 @@ public function update(UpdateFeedRequest $request, Feed $feed): JsonResponse
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to update feed: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update feed: '.$e->getMessage(), [], 500);
} }
} }
@ -114,7 +114,7 @@ public function destroy(Feed $feed): JsonResponse
'Feed deleted successfully!' 'Feed deleted successfully!'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to delete feed: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to delete feed: '.$e->getMessage(), [], 500);
} }
} }
@ -124,7 +124,7 @@ public function destroy(Feed $feed): JsonResponse
public function toggle(Feed $feed): JsonResponse public function toggle(Feed $feed): JsonResponse
{ {
try { try {
$newStatus = !$feed->is_active; $newStatus = ! $feed->is_active;
$feed->update(['is_active' => $newStatus]); $feed->update(['is_active' => $newStatus]);
$status = $newStatus ? 'activated' : 'deactivated'; $status = $newStatus ? 'activated' : 'deactivated';
@ -134,7 +134,7 @@ public function toggle(Feed $feed): JsonResponse
"Feed {$status} successfully!" "Feed {$status} successfully!"
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to toggle feed status: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to toggle feed status: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -62,7 +62,7 @@ public function store(Request $request, Feed $feed, PlatformChannel $channel): J
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to create keyword: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to create keyword: '.$e->getMessage(), [], 500);
} }
} }
@ -90,7 +90,7 @@ public function update(Request $request, Feed $feed, PlatformChannel $channel, K
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to update keyword: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update keyword: '.$e->getMessage(), [], 500);
} }
} }
@ -112,7 +112,7 @@ public function destroy(Feed $feed, PlatformChannel $channel, Keyword $keyword):
'Keyword deleted successfully!' 'Keyword deleted successfully!'
); );
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to delete keyword: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to delete keyword: '.$e->getMessage(), [], 500);
} }
} }
@ -127,7 +127,7 @@ public function toggle(Feed $feed, PlatformChannel $channel, Keyword $keyword):
return $this->sendNotFound('Keyword not found for this route.'); return $this->sendNotFound('Keyword not found for this route.');
} }
$newStatus = !$keyword->is_active; $newStatus = ! $keyword->is_active;
$keyword->update(['is_active' => $newStatus]); $keyword->update(['is_active' => $newStatus]);
$status = $newStatus ? 'activated' : 'deactivated'; $status = $newStatus ? 'activated' : 'deactivated';
@ -137,7 +137,7 @@ public function toggle(Feed $feed, PlatformChannel $channel, Keyword $keyword):
"Keyword {$status} successfully!" "Keyword {$status} successfully!"
); );
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to toggle keyword status: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to toggle keyword status: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -49,7 +49,7 @@ public function index(Request $request): JsonResponse
], ],
], 'Logs retrieved successfully.'); ], 'Logs retrieved successfully.');
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to retrieve logs: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to retrieve logs: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -31,18 +31,18 @@ public function status(): JsonResponse
// 1. They haven't completed or skipped onboarding AND // 1. They haven't completed or skipped onboarding AND
// 2. They don't have all required components // 2. They don't have all required components
$hasAllComponents = $hasPlatformAccount && $hasFeed && $hasChannel && $hasRoute; $hasAllComponents = $hasPlatformAccount && $hasFeed && $hasChannel && $hasRoute;
$needsOnboarding = !$onboardingCompleted && !$onboardingSkipped && !$hasAllComponents; $needsOnboarding = ! $onboardingCompleted && ! $onboardingSkipped && ! $hasAllComponents;
// Determine current step // Determine current step
$currentStep = null; $currentStep = null;
if ($needsOnboarding) { if ($needsOnboarding) {
if (!$hasPlatformAccount) { if (! $hasPlatformAccount) {
$currentStep = 'platform'; $currentStep = 'platform';
} elseif (!$hasFeed) { } elseif (! $hasFeed) {
$currentStep = 'feed'; $currentStep = 'feed';
} elseif (!$hasChannel) { } elseif (! $hasChannel) {
$currentStep = 'channel'; $currentStep = 'channel';
} elseif (!$hasRoute) { } elseif (! $hasRoute) {
$currentStep = 'route'; $currentStep = 'route';
} }
} }
@ -56,7 +56,7 @@ public function status(): JsonResponse
'has_route' => $hasRoute, 'has_route' => $hasRoute,
'onboarding_skipped' => $onboardingSkipped, 'onboarding_skipped' => $onboardingSkipped,
'onboarding_completed' => $onboardingCompleted, 'onboarding_completed' => $onboardingCompleted,
'missing_components' => !$hasAllComponents && $onboardingCompleted, 'missing_components' => ! $hasAllComponents && $onboardingCompleted,
], 'Onboarding status retrieved successfully.'); ], 'Onboarding status retrieved successfully.');
} }
@ -84,8 +84,10 @@ public function options(): JsonResponse
->get(['id', 'platform_instance_id', 'name', 'display_name', 'description']); ->get(['id', 'platform_instance_id', 'name', 'display_name', 'description']);
// Get feed providers from config // Get feed providers from config
$feedProviders = collect(config('feed.providers', [])) /** @var array<string, array<string, mixed>> $providers */
->filter(fn($provider) => $provider['is_active']) $providers = config('feed.providers', []);
$feedProviders = collect($providers)
->filter(fn (array $provider) => $provider['is_active'])
->values(); ->values();
return $this->sendResponse([ return $this->sendResponse([

View file

@ -53,6 +53,7 @@ public function store(StorePlatformAccountRequest $request, CreatePlatformAccoun
if (str_contains($e->getMessage(), 'Rate limited by')) { if (str_contains($e->getMessage(), 'Rate limited by')) {
return $this->sendError($e->getMessage(), [], 429); return $this->sendError($e->getMessage(), [], 429);
} }
return $this->sendError('Invalid username or password. Please check your credentials and try again.', [], 422); return $this->sendError('Invalid username or password. Please check your credentials and try again.', [], 422);
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Unable to connect to the Lemmy instance. Please check the URL and try again.', [], 422); return $this->sendError('Unable to connect to the Lemmy instance. Please check the URL and try again.', [], 422);
@ -97,7 +98,7 @@ public function update(Request $request, PlatformAccount $platformAccount): Json
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to update platform account: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update platform account: '.$e->getMessage(), [], 500);
} }
} }
@ -114,7 +115,7 @@ public function destroy(PlatformAccount $platformAccount): JsonResponse
'Platform account deleted successfully!' 'Platform account deleted successfully!'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to delete platform account: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to delete platform account: '.$e->getMessage(), [], 500);
} }
} }
@ -131,7 +132,7 @@ public function setActive(PlatformAccount $platformAccount): JsonResponse
"Set {$platformAccount->username}@{$platformAccount->instance_url} as active for {$platformAccount->platform->value}!" "Set {$platformAccount->username}@{$platformAccount->instance_url} as active for {$platformAccount->platform->value}!"
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to set platform account as active: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to set platform account as active: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -5,8 +5,8 @@
use App\Actions\CreateChannelAction; use App\Actions\CreateChannelAction;
use App\Http\Requests\StorePlatformChannelRequest; use App\Http\Requests\StorePlatformChannelRequest;
use App\Http\Resources\PlatformChannelResource; use App\Http\Resources\PlatformChannelResource;
use App\Models\PlatformChannel;
use App\Models\PlatformAccount; use App\Models\PlatformAccount;
use App\Models\PlatformChannel;
use Exception; use Exception;
use Illuminate\Http\JsonResponse; use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
@ -54,7 +54,7 @@ public function store(StorePlatformChannelRequest $request, CreateChannelAction
} catch (RuntimeException $e) { } catch (RuntimeException $e) {
return $this->sendError($e->getMessage(), [], 422); return $this->sendError($e->getMessage(), [], 422);
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to create platform channel: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to create platform channel: '.$e->getMessage(), [], 500);
} }
} }
@ -91,7 +91,7 @@ public function update(Request $request, PlatformChannel $platformChannel): Json
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to update platform channel: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update platform channel: '.$e->getMessage(), [], 500);
} }
} }
@ -108,7 +108,7 @@ public function destroy(PlatformChannel $platformChannel): JsonResponse
'Platform channel deleted successfully!' 'Platform channel deleted successfully!'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to delete platform channel: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to delete platform channel: '.$e->getMessage(), [], 500);
} }
} }
@ -118,7 +118,7 @@ public function destroy(PlatformChannel $platformChannel): JsonResponse
public function toggle(PlatformChannel $channel): JsonResponse public function toggle(PlatformChannel $channel): JsonResponse
{ {
try { try {
$newStatus = !$channel->is_active; $newStatus = ! $channel->is_active;
$channel->update(['is_active' => $newStatus]); $channel->update(['is_active' => $newStatus]);
$status = $newStatus ? 'activated' : 'deactivated'; $status = $newStatus ? 'activated' : 'deactivated';
@ -128,7 +128,7 @@ public function toggle(PlatformChannel $channel): JsonResponse
"Platform channel {$status} successfully!" "Platform channel {$status} successfully!"
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to toggle platform channel status: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to toggle platform channel status: '.$e->getMessage(), [], 500);
} }
} }
@ -144,6 +144,7 @@ public function attachAccount(PlatformChannel $channel, Request $request): JsonR
'priority' => 'nullable|integer|min:1|max:100', 'priority' => 'nullable|integer|min:1|max:100',
]); ]);
/** @var PlatformAccount $platformAccount */
$platformAccount = PlatformAccount::findOrFail($validated['platform_account_id']); $platformAccount = PlatformAccount::findOrFail($validated['platform_account_id']);
// Check if account is already attached // Check if account is already attached
@ -165,7 +166,7 @@ public function attachAccount(PlatformChannel $channel, Request $request): JsonR
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to attach platform account: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to attach platform account: '.$e->getMessage(), [], 500);
} }
} }
@ -175,7 +176,7 @@ public function attachAccount(PlatformChannel $channel, Request $request): JsonR
public function detachAccount(PlatformChannel $channel, PlatformAccount $account): JsonResponse public function detachAccount(PlatformChannel $channel, PlatformAccount $account): JsonResponse
{ {
try { try {
if (!$channel->platformAccounts()->where('platform_account_id', $account->id)->exists()) { if (! $channel->platformAccounts()->where('platform_account_id', $account->id)->exists()) {
return $this->sendError('Platform account is not attached to this channel.', [], 422); return $this->sendError('Platform account is not attached to this channel.', [], 422);
} }
@ -186,7 +187,7 @@ public function detachAccount(PlatformChannel $channel, PlatformAccount $account
'Platform account detached from channel successfully!' 'Platform account detached from channel successfully!'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to detach platform account: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to detach platform account: '.$e->getMessage(), [], 500);
} }
} }
@ -201,7 +202,7 @@ public function updateAccountRelation(PlatformChannel $channel, PlatformAccount
'priority' => 'nullable|integer|min:1|max:100', 'priority' => 'nullable|integer|min:1|max:100',
]); ]);
if (!$channel->platformAccounts()->where('platform_account_id', $account->id)->exists()) { if (! $channel->platformAccounts()->where('platform_account_id', $account->id)->exists()) {
return $this->sendError('Platform account is not attached to this channel.', [], 422); return $this->sendError('Platform account is not attached to this channel.', [], 422);
} }
@ -218,7 +219,7 @@ public function updateAccountRelation(PlatformChannel $channel, PlatformAccount
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to update platform account relationship: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update platform account relationship: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -0,0 +1,101 @@
<?php
namespace App\Http\Controllers\Api\V1;
use App\Enums\ApprovalStatusEnum;
use App\Http\Resources\RouteArticleResource;
use App\Models\RouteArticle;
use Exception;
use Illuminate\Http\JsonResponse;
use Illuminate\Http\Request;
class RouteArticlesController extends BaseController
{
public function index(Request $request): JsonResponse
{
$perPage = min($request->get('per_page', 15), 100);
$query = RouteArticle::with(['article.feed', 'feed', 'platformChannel'])
->orderBy('created_at', 'desc');
if ($request->has('status')) {
$status = ApprovalStatusEnum::tryFrom($request->get('status'));
if ($status) {
$query->where('approval_status', $status);
}
}
$routeArticles = $query->paginate($perPage);
return $this->sendResponse([
'route_articles' => RouteArticleResource::collection($routeArticles->items()),
'pagination' => [
'current_page' => $routeArticles->currentPage(),
'last_page' => $routeArticles->lastPage(),
'per_page' => $routeArticles->perPage(),
'total' => $routeArticles->total(),
'from' => $routeArticles->firstItem(),
'to' => $routeArticles->lastItem(),
],
]);
}
public function approve(RouteArticle $routeArticle): JsonResponse
{
try {
$routeArticle->approve();
return $this->sendResponse(
new RouteArticleResource($routeArticle->fresh(['article.feed', 'feed', 'platformChannel'])),
'Route article approved and queued for publishing.'
);
} catch (Exception $e) {
return $this->sendError('Failed to approve route article: '.$e->getMessage(), [], 500);
}
}
public function reject(RouteArticle $routeArticle): JsonResponse
{
try {
$routeArticle->reject();
return $this->sendResponse(
new RouteArticleResource($routeArticle->fresh(['article.feed', 'feed', 'platformChannel'])),
'Route article rejected.'
);
} catch (Exception $e) {
return $this->sendError('Failed to reject route article: '.$e->getMessage(), [], 500);
}
}
public function restore(RouteArticle $routeArticle): JsonResponse
{
try {
$routeArticle->update(['approval_status' => ApprovalStatusEnum::PENDING]);
return $this->sendResponse(
new RouteArticleResource($routeArticle->fresh(['article.feed', 'feed', 'platformChannel'])),
'Route article restored to pending.'
);
} catch (Exception $e) {
return $this->sendError('Failed to restore route article: '.$e->getMessage(), [], 500);
}
}
public function clear(): JsonResponse
{
try {
$count = RouteArticle::where('approval_status', ApprovalStatusEnum::PENDING)->count();
RouteArticle::where('approval_status', ApprovalStatusEnum::PENDING)
->update(['approval_status' => ApprovalStatusEnum::REJECTED]);
return $this->sendResponse(
['rejected_count' => $count],
"Rejected {$count} pending route articles."
);
} catch (Exception $e) {
return $this->sendError('Failed to clear pending route articles: '.$e->getMessage(), [], 500);
}
}
}

View file

@ -52,7 +52,7 @@ public function store(StoreRouteRequest $request, CreateRouteAction $createRoute
201 201
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to create routing configuration: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to create routing configuration: '.$e->getMessage(), [], 500);
} }
} }
@ -62,11 +62,11 @@ public function store(StoreRouteRequest $request, CreateRouteAction $createRoute
public function show(Feed $feed, PlatformChannel $channel): JsonResponse public function show(Feed $feed, PlatformChannel $channel): JsonResponse
{ {
$route = $this->findRoute($feed, $channel); $route = $this->findRoute($feed, $channel);
if (!$route) { if (! $route) {
return $this->sendNotFound('Routing configuration not found.'); return $this->sendNotFound('Routing configuration not found.');
} }
$route->load(['feed', 'platformChannel', 'keywords']); $route->load(['feed', 'platformChannel', 'keywords']);
return $this->sendResponse( return $this->sendResponse(
@ -83,7 +83,7 @@ public function update(Request $request, Feed $feed, PlatformChannel $channel):
try { try {
$route = $this->findRoute($feed, $channel); $route = $this->findRoute($feed, $channel);
if (!$route) { if (! $route) {
return $this->sendNotFound('Routing configuration not found.'); return $this->sendNotFound('Routing configuration not found.');
} }
@ -103,7 +103,7 @@ public function update(Request $request, Feed $feed, PlatformChannel $channel):
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to update routing configuration: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update routing configuration: '.$e->getMessage(), [], 500);
} }
} }
@ -115,7 +115,7 @@ public function destroy(Feed $feed, PlatformChannel $channel): JsonResponse
try { try {
$route = $this->findRoute($feed, $channel); $route = $this->findRoute($feed, $channel);
if (!$route) { if (! $route) {
return $this->sendNotFound('Routing configuration not found.'); return $this->sendNotFound('Routing configuration not found.');
} }
@ -128,7 +128,7 @@ public function destroy(Feed $feed, PlatformChannel $channel): JsonResponse
'Routing configuration deleted successfully!' 'Routing configuration deleted successfully!'
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to delete routing configuration: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to delete routing configuration: '.$e->getMessage(), [], 500);
} }
} }
@ -140,11 +140,11 @@ public function toggle(Feed $feed, PlatformChannel $channel): JsonResponse
try { try {
$route = $this->findRoute($feed, $channel); $route = $this->findRoute($feed, $channel);
if (!$route) { if (! $route) {
return $this->sendNotFound('Routing configuration not found.'); return $this->sendNotFound('Routing configuration not found.');
} }
$newStatus = !$route->is_active; $newStatus = ! $route->is_active;
Route::where('feed_id', $feed->id) Route::where('feed_id', $feed->id)
->where('platform_channel_id', $channel->id) ->where('platform_channel_id', $channel->id)
->update(['is_active' => $newStatus]); ->update(['is_active' => $newStatus]);
@ -156,7 +156,7 @@ public function toggle(Feed $feed, PlatformChannel $channel): JsonResponse
"Routing configuration {$status} successfully!" "Routing configuration {$status} successfully!"
); );
} catch (Exception $e) { } catch (Exception $e) {
return $this->sendError('Failed to toggle routing configuration status: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to toggle routing configuration status: '.$e->getMessage(), [], 500);
} }
} }
@ -169,4 +169,4 @@ private function findRoute(Feed $feed, PlatformChannel $channel): ?Route
->where('platform_channel_id', $channel->id) ->where('platform_channel_id', $channel->id)
->first(); ->first();
} }
} }

View file

@ -23,7 +23,7 @@ public function index(): JsonResponse
return $this->sendResponse($settings, 'Settings retrieved successfully.'); return $this->sendResponse($settings, 'Settings retrieved successfully.');
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to retrieve settings: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to retrieve settings: '.$e->getMessage(), [], 500);
} }
} }
@ -64,7 +64,7 @@ public function update(Request $request): JsonResponse
} catch (ValidationException $e) { } catch (ValidationException $e) {
return $this->sendValidationError($e->errors()); return $this->sendValidationError($e->errors());
} catch (\Exception $e) { } catch (\Exception $e) {
return $this->sendError('Failed to update settings: ' . $e->getMessage(), [], 500); return $this->sendError('Failed to update settings: '.$e->getMessage(), [], 500);
} }
} }
} }

View file

@ -11,6 +11,7 @@
use Illuminate\Support\Facades\Password; use Illuminate\Support\Facades\Password;
use Illuminate\Support\Str; use Illuminate\Support\Str;
use Illuminate\Validation\Rules; use Illuminate\Validation\Rules;
use Illuminate\Validation\ValidationException;
use Illuminate\View\View; use Illuminate\View\View;
class NewPasswordController extends Controller class NewPasswordController extends Controller
@ -26,7 +27,7 @@ public function create(Request $request): View
/** /**
* Handle an incoming new password request. * Handle an incoming new password request.
* *
* @throws \Illuminate\Validation\ValidationException * @throws ValidationException
*/ */
public function store(Request $request): RedirectResponse public function store(Request $request): RedirectResponse
{ {

View file

@ -6,6 +6,7 @@
use Illuminate\Http\RedirectResponse; use Illuminate\Http\RedirectResponse;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Support\Facades\Password; use Illuminate\Support\Facades\Password;
use Illuminate\Validation\ValidationException;
use Illuminate\View\View; use Illuminate\View\View;
class PasswordResetLinkController extends Controller class PasswordResetLinkController extends Controller
@ -21,7 +22,7 @@ public function create(): View
/** /**
* Handle an incoming password reset link request. * Handle an incoming password reset link request.
* *
* @throws \Illuminate\Validation\ValidationException * @throws ValidationException
*/ */
public function store(Request $request): RedirectResponse public function store(Request $request): RedirectResponse
{ {

View file

@ -10,6 +10,7 @@
use Illuminate\Support\Facades\Auth; use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\Hash; use Illuminate\Support\Facades\Hash;
use Illuminate\Validation\Rules; use Illuminate\Validation\Rules;
use Illuminate\Validation\ValidationException;
use Illuminate\View\View; use Illuminate\View\View;
class RegisteredUserController extends Controller class RegisteredUserController extends Controller
@ -25,7 +26,7 @@ public function create(): View
/** /**
* Handle an incoming registration request. * Handle an incoming registration request.
* *
* @throws \Illuminate\Validation\ValidationException * @throws ValidationException
*/ */
public function store(Request $request): RedirectResponse public function store(Request $request): RedirectResponse
{ {

View file

@ -4,6 +4,7 @@
use App\Http\Controllers\Controller; use App\Http\Controllers\Controller;
use Illuminate\Auth\Events\Verified; use Illuminate\Auth\Events\Verified;
use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\EmailVerificationRequest; use Illuminate\Foundation\Auth\EmailVerificationRequest;
use Illuminate\Http\RedirectResponse; use Illuminate\Http\RedirectResponse;
@ -19,7 +20,9 @@ public function __invoke(EmailVerificationRequest $request): RedirectResponse
} }
if ($request->user()->markEmailAsVerified()) { if ($request->user()->markEmailAsVerified()) {
event(new Verified($request->user())); /** @var MustVerifyEmail $user */
$user = $request->user();
event(new Verified($user));
} }
return redirect()->intended(route('dashboard', absolute: false).'?verified=1'); return redirect()->intended(route('dashboard', absolute: false).'?verified=1');

View file

@ -26,4 +26,4 @@ public function handle(Request $request, Closure $next): Response
return $next($request); return $next($request);
} }
} }

View file

@ -12,7 +12,7 @@ class HandleAppearance
/** /**
* Handle an incoming request. * Handle an incoming request.
* *
* @param \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response) $next * @param Closure(Request): (Response) $next
*/ */
public function handle(Request $request, Closure $next): Response public function handle(Request $request, Closure $next): Response
{ {

View file

@ -39,4 +39,4 @@ public function share(Request $request): array
// //
]); ]);
} }
} }

View file

@ -20,10 +20,10 @@ public function __construct(
*/ */
public function handle(Request $request, Closure $next): Response public function handle(Request $request, Closure $next): Response
{ {
if (!$this->onboardingService->needsOnboarding()) { if (! $this->onboardingService->needsOnboarding()) {
return redirect()->route('dashboard'); return redirect()->route('dashboard');
} }
return $next($request); return $next($request);
} }
} }

View file

@ -3,6 +3,7 @@
namespace App\Http\Requests\Auth; namespace App\Http\Requests\Auth;
use Illuminate\Auth\Events\Lockout; use Illuminate\Auth\Events\Lockout;
use Illuminate\Contracts\Validation\ValidationRule;
use Illuminate\Foundation\Http\FormRequest; use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Support\Facades\Auth; use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\RateLimiter; use Illuminate\Support\Facades\RateLimiter;
@ -22,7 +23,7 @@ public function authorize(): bool
/** /**
* Get the validation rules that apply to the request. * Get the validation rules that apply to the request.
* *
* @return array<string, \Illuminate\Contracts\Validation\ValidationRule|array<mixed>|string> * @return array<string, ValidationRule|array<mixed>|string>
*/ */
public function rules(): array public function rules(): array
{ {
@ -35,7 +36,7 @@ public function rules(): array
/** /**
* Attempt to authenticate the request's credentials. * Attempt to authenticate the request's credentials.
* *
* @throws \Illuminate\Validation\ValidationException * @throws ValidationException
*/ */
public function authenticate(): void public function authenticate(): void
{ {
@ -55,7 +56,7 @@ public function authenticate(): void
/** /**
* Ensure the login request is not rate limited. * Ensure the login request is not rate limited.
* *
* @throws \Illuminate\Validation\ValidationException * @throws ValidationException
*/ */
public function ensureIsNotRateLimited(): void public function ensureIsNotRateLimited(): void
{ {

View file

@ -3,6 +3,7 @@
namespace App\Http\Requests; namespace App\Http\Requests;
use App\Models\User; use App\Models\User;
use Illuminate\Contracts\Validation\ValidationRule;
use Illuminate\Foundation\Http\FormRequest; use Illuminate\Foundation\Http\FormRequest;
use Illuminate\Validation\Rule; use Illuminate\Validation\Rule;
@ -11,7 +12,7 @@ class ProfileUpdateRequest extends FormRequest
/** /**
* Get the validation rules that apply to the request. * Get the validation rules that apply to the request.
* *
* @return array<string, \Illuminate\Contracts\Validation\ValidationRule|array<mixed>|string> * @return array<string, ValidationRule|array<mixed>|string>
*/ */
public function rules(): array public function rules(): array
{ {

View file

@ -23,7 +23,7 @@ public function rules(): array
'provider' => "required|in:{$providers}", 'provider' => "required|in:{$providers}",
'language_id' => 'required|exists:languages,id', 'language_id' => 'required|exists:languages,id',
'description' => 'nullable|string', 'description' => 'nullable|string',
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
} }
} }

View file

@ -19,11 +19,11 @@ public function rules(): array
{ {
return [ return [
'name' => 'required|string|max:255', 'name' => 'required|string|max:255',
'url' => 'required|url|unique:feeds,url,' . ($this->route('feed') instanceof Feed ? (string)$this->route('feed')->id : (string)$this->route('feed')), 'url' => 'required|url|unique:feeds,url,'.($this->route('feed') instanceof Feed ? (string) $this->route('feed')->id : (string) $this->route('feed')),
'type' => 'required|in:website,rss', 'type' => 'required|in:website,rss',
'language_id' => 'required|exists:languages,id', 'language_id' => 'required|exists:languages,id',
'description' => 'nullable|string', 'description' => 'nullable|string',
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
} }
} }

View file

@ -2,9 +2,13 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\ArticlePublication;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin ArticlePublication
*/
class ArticlePublicationResource extends JsonResource class ArticlePublicationResource extends JsonResource
{ {
/** /**
@ -17,10 +21,10 @@ public function toArray(Request $request): array
return [ return [
'id' => $this->id, 'id' => $this->id,
'article_id' => $this->article_id, 'article_id' => $this->article_id,
'status' => $this->status, 'platform' => $this->platform,
'published_at' => $this->published_at?->toISOString(), 'published_at' => $this->published_at->toISOString(),
'created_at' => $this->created_at->toISOString(), 'created_at' => $this->created_at->toISOString(),
'updated_at' => $this->updated_at->toISOString(), 'updated_at' => $this->updated_at->toISOString(),
]; ];
} }
} }

View file

@ -2,14 +2,18 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\Article;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/** /**
* @property int $id * @mixin Article
*/ */
class ArticleResource extends JsonResource class ArticleResource extends JsonResource
{ {
/**
* @return array<string, mixed>
*/
public function toArray(Request $request): array public function toArray(Request $request): array
{ {
return [ return [
@ -18,13 +22,6 @@ public function toArray(Request $request): array
'url' => $this->url, 'url' => $this->url,
'title' => $this->title, 'title' => $this->title,
'description' => $this->description, 'description' => $this->description,
'is_valid' => $this->is_valid,
'is_duplicate' => $this->is_duplicate,
'approval_status' => $this->approval_status,
'publish_status' => $this->publish_status,
'approved_at' => $this->approved_at?->toISOString(),
'approved_by' => $this->approved_by,
'fetched_at' => $this->fetched_at?->toISOString(),
'validated_at' => $this->validated_at?->toISOString(), 'validated_at' => $this->validated_at?->toISOString(),
'is_published' => $this->relationLoaded('articlePublication') && $this->articlePublication !== null, 'is_published' => $this->relationLoaded('articlePublication') && $this->articlePublication !== null,
'created_at' => $this->created_at->toISOString(), 'created_at' => $this->created_at->toISOString(),

View file

@ -2,9 +2,13 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\Feed;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin Feed
*/
class FeedResource extends JsonResource class FeedResource extends JsonResource
{ {
/** /**
@ -26,9 +30,9 @@ public function toArray(Request $request): array
'created_at' => $this->created_at->toISOString(), 'created_at' => $this->created_at->toISOString(),
'updated_at' => $this->updated_at->toISOString(), 'updated_at' => $this->updated_at->toISOString(),
'articles_count' => $this->when( 'articles_count' => $this->when(
$request->routeIs('api.feeds.*') && isset($this->articles_count), $request->routeIs('api.feeds.*') && isset($this->articles_count),
$this->articles_count $this->articles_count
), ),
]; ];
} }
} }

View file

@ -2,9 +2,16 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\PlatformAccount;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin PlatformAccount
*/
/**
* @mixin PlatformAccount
*/
class PlatformAccountResource extends JsonResource class PlatformAccountResource extends JsonResource
{ {
/** /**
@ -28,4 +35,4 @@ public function toArray(Request $request): array
'channels' => PlatformChannelResource::collection($this->whenLoaded('channels')), 'channels' => PlatformChannelResource::collection($this->whenLoaded('channels')),
]; ];
} }
} }

View file

@ -2,9 +2,13 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\PlatformChannel;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin PlatformChannel
*/
class PlatformChannelResource extends JsonResource class PlatformChannelResource extends JsonResource
{ {
/** /**
@ -30,4 +34,4 @@ public function toArray(Request $request): array
'routes' => RouteResource::collection($this->whenLoaded('routes')), 'routes' => RouteResource::collection($this->whenLoaded('routes')),
]; ];
} }
} }

View file

@ -2,9 +2,13 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\PlatformInstance;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin PlatformInstance
*/
class PlatformInstanceResource extends JsonResource class PlatformInstanceResource extends JsonResource
{ {
/** /**
@ -24,4 +28,4 @@ public function toArray(Request $request): array
'updated_at' => $this->updated_at->toISOString(), 'updated_at' => $this->updated_at->toISOString(),
]; ];
} }
} }

View file

@ -0,0 +1,39 @@
<?php
namespace App\Http\Resources;
use App\Models\RouteArticle;
use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin RouteArticle
*/
class RouteArticleResource extends JsonResource
{
/**
* @return array<string, mixed>
*/
public function toArray(Request $request): array
{
return [
'id' => $this->id,
'feed_id' => $this->feed_id,
'platform_channel_id' => $this->platform_channel_id,
'article_id' => $this->article_id,
'approval_status' => $this->approval_status->value,
'publish_status' => $this->publish_status->value,
'validated_at' => $this->validated_at?->toISOString(),
'created_at' => $this->created_at->toISOString(),
'updated_at' => $this->updated_at->toISOString(),
'article' => [
'id' => $this->article->id,
'title' => $this->article->title,
'url' => $this->article->url,
'description' => $this->article->description,
'feed_name' => $this->article->feed->name,
],
'route_name' => $this->feed->name.' → '.$this->platformChannel->name,
];
}
}

View file

@ -2,9 +2,13 @@
namespace App\Http\Resources; namespace App\Http\Resources;
use App\Models\Route;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Http\Resources\Json\JsonResource; use Illuminate\Http\Resources\Json\JsonResource;
/**
* @mixin Route
*/
class RouteResource extends JsonResource class RouteResource extends JsonResource
{ {
/** /**
@ -15,7 +19,6 @@ class RouteResource extends JsonResource
public function toArray(Request $request): array public function toArray(Request $request): array
{ {
return [ return [
'id' => $this->id,
'feed_id' => $this->feed_id, 'feed_id' => $this->feed_id,
'platform_channel_id' => $this->platform_channel_id, 'platform_channel_id' => $this->platform_channel_id,
'is_active' => $this->is_active, 'is_active' => $this->is_active,
@ -35,4 +38,4 @@ public function toArray(Request $request): array
}), }),
]; ];
} }
} }

View file

@ -25,7 +25,7 @@ public function handle(LogSaver $logSaver, ArticleFetcher $articleFetcher): void
$logSaver->info('Starting feed article fetch', null, [ $logSaver->info('Starting feed article fetch', null, [
'feed_id' => $this->feed->id, 'feed_id' => $this->feed->id,
'feed_name' => $this->feed->name, 'feed_name' => $this->feed->name,
'feed_url' => $this->feed->url 'feed_url' => $this->feed->url,
]); ]);
$articles = $articleFetcher->getArticlesFromFeed($this->feed); $articles = $articleFetcher->getArticlesFromFeed($this->feed);
@ -33,7 +33,7 @@ public function handle(LogSaver $logSaver, ArticleFetcher $articleFetcher): void
$logSaver->info('Feed article fetch completed', null, [ $logSaver->info('Feed article fetch completed', null, [
'feed_id' => $this->feed->id, 'feed_id' => $this->feed->id,
'feed_name' => $this->feed->name, 'feed_name' => $this->feed->name,
'articles_count' => $articles->count() 'articles_count' => $articles->count(),
]); ]);
$this->feed->update(['last_fetched_at' => now()]); $this->feed->update(['last_fetched_at' => now()]);
@ -42,7 +42,7 @@ public function handle(LogSaver $logSaver, ArticleFetcher $articleFetcher): void
public static function dispatchForAllActiveFeeds(): void public static function dispatchForAllActiveFeeds(): void
{ {
$logSaver = app(LogSaver::class); $logSaver = app(LogSaver::class);
Feed::where('is_active', true) Feed::where('is_active', true)
->get() ->get()
->each(function (Feed $feed, $index) use ($logSaver) { ->each(function (Feed $feed, $index) use ($logSaver) {
@ -56,7 +56,7 @@ public static function dispatchForAllActiveFeeds(): void
$logSaver->info('Dispatched feed discovery job', null, [ $logSaver->info('Dispatched feed discovery job', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_name' => $feed->name, 'feed_name' => $feed->name,
'delay_minutes' => $delayMinutes 'delay_minutes' => $delayMinutes,
]); ]);
}); });
} }

View file

@ -18,7 +18,7 @@ public function __construct()
public function handle(LogSaver $logSaver): void public function handle(LogSaver $logSaver): void
{ {
if (!Setting::isArticleProcessingEnabled()) { if (! Setting::isArticleProcessingEnabled()) {
$logSaver->info('Article processing is disabled. Article discovery skipped.'); $logSaver->info('Article processing is disabled. Article discovery skipped.');
return; return;

View file

@ -0,0 +1,51 @@
<?php
namespace App\Jobs;
use App\Enums\NotificationSeverityEnum;
use App\Enums\NotificationTypeEnum;
use App\Models\Feed;
use App\Models\Notification;
use App\Models\Setting;
use App\Services\Notification\NotificationService;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Queue\Queueable;
class CheckFeedStalenessJob implements ShouldQueue
{
use Queueable;
public function handle(NotificationService $notificationService): void
{
$thresholdHours = Setting::getFeedStalenessThreshold();
if ($thresholdHours === 0) {
return;
}
$staleFeeds = Feed::stale($thresholdHours)->get();
foreach ($staleFeeds as $feed) {
$alreadyNotified = Notification::query()
->where('type', NotificationTypeEnum::FEED_STALE)
->where('notifiable_type', $feed->getMorphClass())
->where('notifiable_id', $feed->getKey())
->unread()
->exists();
if ($alreadyNotified) {
continue;
}
$notificationService->send(
type: NotificationTypeEnum::FEED_STALE,
severity: NotificationSeverityEnum::WARNING,
title: "Feed \"{$feed->name}\" is stale",
message: $feed->last_fetched_at
? "Last fetched {$feed->last_fetched_at->diffForHumans()}. Threshold is {$thresholdHours} hours."
: "This feed has never been fetched. Threshold is {$thresholdHours} hours.",
notifiable: $feed,
);
}
}
}

View file

@ -0,0 +1,25 @@
<?php
namespace App\Jobs;
use App\Enums\ApprovalStatusEnum;
use App\Models\Article;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Queue\Queueable;
class CleanupArticlesJob implements ShouldQueue
{
use Queueable;
private const RETENTION_DAYS = 30;
public function handle(): void
{
Article::where('created_at', '<', now()->subDays(self::RETENTION_DAYS))
->whereDoesntHave('routeArticles', fn ($q) => $q->whereIn('approval_status', [
ApprovalStatusEnum::PENDING,
ApprovalStatusEnum::APPROVED,
]))
->delete();
}
}

View file

@ -2,17 +2,24 @@
namespace App\Jobs; namespace App\Jobs;
use App\Enums\ApprovalStatusEnum;
use App\Enums\LogLevelEnum;
use App\Enums\NotificationSeverityEnum;
use App\Enums\NotificationTypeEnum;
use App\Enums\PublishStatusEnum;
use App\Events\ActionPerformed;
use App\Exceptions\PublishException; use App\Exceptions\PublishException;
use App\Models\Article;
use App\Models\ArticlePublication; use App\Models\ArticlePublication;
use App\Models\RouteArticle;
use App\Models\Setting; use App\Models\Setting;
use App\Services\Article\ArticleFetcher; use App\Services\Article\ArticleFetcher;
use App\Services\Notification\NotificationService;
use App\Services\Publishing\ArticlePublishingService; use App\Services\Publishing\ArticlePublishingService;
use Illuminate\Contracts\Queue\ShouldBeUnique; use Illuminate\Contracts\Queue\ShouldBeUnique;
use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Queue\Queueable; use Illuminate\Foundation\Queue\Queueable;
class PublishNextArticleJob implements ShouldQueue, ShouldBeUnique class PublishNextArticleJob implements ShouldBeUnique, ShouldQueue
{ {
use Queueable; use Queueable;
@ -28,9 +35,10 @@ public function __construct()
/** /**
* Execute the job. * Execute the job.
*
* @throws PublishException * @throws PublishException
*/ */
public function handle(ArticleFetcher $articleFetcher, ArticlePublishingService $publishingService): void public function handle(ArticleFetcher $articleFetcher, ArticlePublishingService $publishingService, NotificationService $notificationService): void
{ {
$interval = Setting::getArticlePublishingInterval(); $interval = Setting::getArticlePublishingInterval();
@ -42,40 +50,74 @@ public function handle(ArticleFetcher $articleFetcher, ArticlePublishingService
} }
} }
// Get the oldest approved article that hasn't been published yet // Get the oldest approved route_article that hasn't been published to its channel yet
$article = Article::where('approval_status', 'approved') $routeArticle = RouteArticle::where('approval_status', ApprovalStatusEnum::APPROVED)
->whereDoesntHave('articlePublication') ->whereDoesntHave('article.articlePublications', function ($query) {
->oldest('created_at') $query->whereColumn('article_publications.platform_channel_id', 'route_articles.platform_channel_id');
})
->oldest('route_articles.created_at')
->with(['article', 'platformChannel.platformInstance', 'platformChannel.activePlatformAccounts'])
->first(); ->first();
if (! $article) { if (! $routeArticle) {
return; return;
} }
logger()->info('Publishing next article from scheduled job', [ $article = $routeArticle->article;
ActionPerformed::dispatch('Publishing next article from scheduled job', LogLevelEnum::INFO, [
'article_id' => $article->id, 'article_id' => $article->id,
'title' => $article->title, 'title' => $article->title,
'url' => $article->url, 'url' => $article->url,
'created_at' => $article->created_at 'route' => $routeArticle->feed_id.'-'.$routeArticle->platform_channel_id,
]); ]);
// Fetch article data $routeArticle->update(['publish_status' => PublishStatusEnum::PUBLISHING]);
$extractedData = $articleFetcher->fetchArticleData($article);
try { try {
$publishingService->publishToRoutedChannels($article, $extractedData); $extractedData = $articleFetcher->fetchArticleData($article);
$publication = $publishingService->publishRouteArticle($routeArticle, $extractedData);
logger()->info('Successfully published article', [
'article_id' => $article->id, if ($publication) {
'title' => $article->title $routeArticle->update(['publish_status' => PublishStatusEnum::PUBLISHED]);
]);
ActionPerformed::dispatch('Successfully published article', LogLevelEnum::INFO, [
'article_id' => $article->id,
'title' => $article->title,
]);
} else {
$routeArticle->update(['publish_status' => PublishStatusEnum::ERROR]);
ActionPerformed::dispatch('No publication created for article', LogLevelEnum::WARNING, [
'article_id' => $article->id,
'title' => $article->title,
]);
$notificationService->send(
NotificationTypeEnum::PUBLISH_FAILED,
NotificationSeverityEnum::WARNING,
"Publish failed: {$article->title}",
'No publication was created for this article. Check channel routing configuration.',
$article,
);
}
} catch (PublishException $e) { } catch (PublishException $e) {
logger()->error('Failed to publish article', [ $routeArticle->update(['publish_status' => PublishStatusEnum::ERROR]);
ActionPerformed::dispatch('Failed to publish article', LogLevelEnum::ERROR, [
'article_id' => $article->id, 'article_id' => $article->id,
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
$notificationService->send(
NotificationTypeEnum::PUBLISH_FAILED,
NotificationSeverityEnum::ERROR,
"Publish failed: {$article->title}",
$e->getMessage(),
$article,
);
throw $e; throw $e;
} }
} }
} }

View file

@ -15,7 +15,7 @@
use Illuminate\Foundation\Queue\Queueable; use Illuminate\Foundation\Queue\Queueable;
use Illuminate\Support\Facades\Cache; use Illuminate\Support\Facades\Cache;
class SyncChannelPostsJob implements ShouldQueue, ShouldBeUnique class SyncChannelPostsJob implements ShouldBeUnique, ShouldQueue
{ {
use Queueable; use Queueable;
@ -28,7 +28,7 @@ public function __construct(
public static function dispatchForAllActiveChannels(): void public static function dispatchForAllActiveChannels(): void
{ {
$logSaver = app(LogSaver::class); $logSaver = app(LogSaver::class);
PlatformChannel::with(['platformInstance', 'platformAccounts']) PlatformChannel::with(['platformInstance', 'platformAccounts'])
->whereHas('platformInstance', fn ($query) => $query->where('platform', PlatformEnum::LEMMY)) ->whereHas('platformInstance', fn ($query) => $query->where('platform', PlatformEnum::LEMMY))
->whereHas('platformAccounts', fn ($query) => $query->where('platform_accounts.is_active', true)) ->whereHas('platformAccounts', fn ($query) => $query->where('platform_accounts.is_active', true))
@ -78,7 +78,7 @@ private function syncLemmyChannelPosts(LogSaver $logSaver): void
} catch (Exception $e) { } catch (Exception $e) {
$logSaver->error('Failed to sync channel posts', $this->channel, [ $logSaver->error('Failed to sync channel posts', $this->channel, [
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
throw $e; throw $e;
@ -97,13 +97,13 @@ private function getAuthToken(LemmyApiService $api, PlatformAccount $account): s
return $cachedToken; return $cachedToken;
} }
if (!$account->username || !$account->password) { if (! $account->username || ! $account->password) {
throw new PlatformAuthException(PlatformEnum::LEMMY, 'Missing credentials for account'); throw new PlatformAuthException(PlatformEnum::LEMMY, 'Missing credentials for account');
} }
$token = $api->login($account->username, $account->password); $token = $api->login($account->username, $account->password);
if (!$token) { if (! $token) {
throw new PlatformAuthException(PlatformEnum::LEMMY, 'Login failed for account'); throw new PlatformAuthException(PlatformEnum::LEMMY, 'Login failed for account');
} }

View file

@ -0,0 +1,21 @@
<?php
namespace App\Listeners;
use App\Events\ActionPerformed;
use App\Services\Log\LogSaver;
use Exception;
class LogActionListener
{
public function __construct(private LogSaver $logSaver) {}
public function handle(ActionPerformed $event): void
{
try {
$this->logSaver->log($event->level, $event->message, context: $event->context);
} catch (Exception $e) {
error_log('Failed to log action to database: '.$e->getMessage());
}
}
}

View file

@ -5,14 +5,14 @@
use App\Events\ExceptionLogged; use App\Events\ExceptionLogged;
use App\Events\ExceptionOccurred; use App\Events\ExceptionOccurred;
use App\Models\Log; use App\Models\Log;
class LogExceptionToDatabase class LogExceptionToDatabase
{ {
public function handle(ExceptionOccurred $event): void public function handle(ExceptionOccurred $event): void
{ {
// Truncate the message to prevent database errors // Truncate the message to prevent database errors
$message = strlen($event->message) > 255 $message = strlen($event->message) > 255
? substr($event->message, 0, 252) . '...' ? substr($event->message, 0, 252).'...'
: $event->message; : $event->message;
try { try {
@ -24,15 +24,15 @@ public function handle(ExceptionOccurred $event): void
'file' => $event->exception->getFile(), 'file' => $event->exception->getFile(),
'line' => $event->exception->getLine(), 'line' => $event->exception->getLine(),
'trace' => $event->exception->getTraceAsString(), 'trace' => $event->exception->getTraceAsString(),
...$event->context ...$event->context,
] ],
]); ]);
ExceptionLogged::dispatch($log); ExceptionLogged::dispatch($log);
} catch (\Exception $e) { } catch (\Exception $e) {
// Prevent infinite recursion by not logging this exception // Prevent infinite recursion by not logging this exception
// Optionally log to file or other non-database destination // Optionally log to file or other non-database destination
error_log("Failed to log exception to database: " . $e->getMessage()); error_log('Failed to log exception to database: '.$e->getMessage());
} }
} }
} }

View file

@ -2,8 +2,14 @@
namespace App\Listeners; namespace App\Listeners;
use App\Events\ArticleApproved; use App\Enums\LogLevelEnum;
use App\Enums\NotificationSeverityEnum;
use App\Enums\NotificationTypeEnum;
use App\Enums\PublishStatusEnum;
use App\Events\ActionPerformed;
use App\Events\RouteArticleApproved;
use App\Services\Article\ArticleFetcher; use App\Services\Article\ArticleFetcher;
use App\Services\Notification\NotificationService;
use App\Services\Publishing\ArticlePublishingService; use App\Services\Publishing\ArticlePublishingService;
use Exception; use Exception;
use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Contracts\Queue\ShouldQueue;
@ -14,51 +20,67 @@ class PublishApprovedArticleListener implements ShouldQueue
public function __construct( public function __construct(
private ArticleFetcher $articleFetcher, private ArticleFetcher $articleFetcher,
private ArticlePublishingService $publishingService private ArticlePublishingService $publishingService,
private NotificationService $notificationService,
) {} ) {}
public function handle(ArticleApproved $event): void public function handle(RouteArticleApproved $event): void
{ {
$article = $event->article->fresh(); $routeArticle = $event->routeArticle;
$article = $routeArticle->article;
// Skip if already published // Skip if already published to this channel
if ($article->articlePublication()->exists()) { if ($article->articlePublications()
->where('platform_channel_id', $routeArticle->platform_channel_id)
->exists()
) {
return; return;
} }
// Skip if not approved (safety check) $routeArticle->update(['publish_status' => PublishStatusEnum::PUBLISHING]);
if (! $article->isApproved()) {
return;
}
$article->update(['publish_status' => 'publishing']);
try { try {
$extractedData = $this->articleFetcher->fetchArticleData($article); $extractedData = $this->articleFetcher->fetchArticleData($article);
$publications = $this->publishingService->publishToRoutedChannels($article, $extractedData); $publication = $this->publishingService->publishRouteArticle($routeArticle, $extractedData);
if ($publications->isNotEmpty()) { if ($publication) {
$article->update(['publish_status' => 'published']); $routeArticle->update(['publish_status' => PublishStatusEnum::PUBLISHED]);
logger()->info('Published approved article', [ ActionPerformed::dispatch('Published approved article', LogLevelEnum::INFO, [
'article_id' => $article->id, 'article_id' => $article->id,
'title' => $article->title, 'title' => $article->title,
]); ]);
} else { } else {
$article->update(['publish_status' => 'error']); $routeArticle->update(['publish_status' => PublishStatusEnum::ERROR]);
logger()->warning('No publications created for approved article', [ ActionPerformed::dispatch('No publication created for approved article', LogLevelEnum::WARNING, [
'article_id' => $article->id, 'article_id' => $article->id,
'title' => $article->title, 'title' => $article->title,
]); ]);
$this->notificationService->send(
NotificationTypeEnum::PUBLISH_FAILED,
NotificationSeverityEnum::WARNING,
"Publish failed: {$article->title}",
'No publication was created for this article. Check channel routing configuration.',
$article,
);
} }
} catch (Exception $e) { } catch (Exception $e) {
$article->update(['publish_status' => 'error']); $routeArticle->update(['publish_status' => PublishStatusEnum::ERROR]);
logger()->error('Failed to publish approved article', [ ActionPerformed::dispatch('Failed to publish approved article', LogLevelEnum::ERROR, [
'article_id' => $article->id, 'article_id' => $article->id,
'error' => $e->getMessage(), 'error' => $e->getMessage(),
]); ]);
$this->notificationService->send(
NotificationTypeEnum::PUBLISH_FAILED,
NotificationSeverityEnum::ERROR,
"Publish failed: {$article->title}",
$e->getMessage(),
$article,
);
} }
} }
} }

View file

@ -2,8 +2,9 @@
namespace App\Listeners; namespace App\Listeners;
use App\Enums\LogLevelEnum;
use App\Events\ActionPerformed;
use App\Events\NewArticleFetched; use App\Events\NewArticleFetched;
use App\Models\Setting;
use App\Services\Article\ValidationService; use App\Services\Article\ValidationService;
use Exception; use Exception;
use Illuminate\Contracts\Queue\ShouldQueue; use Illuminate\Contracts\Queue\ShouldQueue;
@ -24,37 +25,18 @@ public function handle(NewArticleFetched $event): void
return; return;
} }
// Only validate articles that are still pending
if (! $article->isPending()) {
return;
}
// Skip if already has publication (prevents duplicate processing) // Skip if already has publication (prevents duplicate processing)
if ($article->articlePublication()->exists()) { if ($article->articlePublication()->exists()) {
return; return;
} }
try { try {
$article = $this->validationService->validate($article); $this->validationService->validate($article);
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Article validation failed', [ ActionPerformed::dispatch('Article validation failed', LogLevelEnum::ERROR, [
'article_id' => $article->id, 'article_id' => $article->id,
'error' => $e->getMessage(), 'error' => $e->getMessage(),
]); ]);
return;
}
if ($article->isValid()) {
// Double-check publication doesn't exist (race condition protection)
if ($article->articlePublication()->exists()) {
return;
}
// If approvals are enabled, article waits for manual approval.
// If approvals are disabled, auto-approve and publish.
if (! Setting::isPublishingApprovalsEnabled()) {
$article->approve();
}
} }
} }
} }

View file

@ -2,9 +2,10 @@
namespace App\Livewire; namespace App\Livewire;
use App\Models\Article; use App\Enums\ApprovalStatusEnum;
use App\Models\Setting;
use App\Jobs\ArticleDiscoveryJob; use App\Jobs\ArticleDiscoveryJob;
use App\Models\RouteArticle;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
use Livewire\WithPagination; use Livewire\WithPagination;
@ -12,22 +13,44 @@ class Articles extends Component
{ {
use WithPagination; use WithPagination;
public string $tab = 'pending';
public string $search = '';
public bool $isRefreshing = false; public bool $isRefreshing = false;
public function approve(int $articleId): void public function setTab(string $tab): void
{ {
$article = Article::findOrFail($articleId); $this->tab = $tab;
$article->approve(); $this->search = '';
$this->resetPage();
$this->dispatch('article-updated');
} }
public function reject(int $articleId): void public function updatedSearch(): void
{ {
$article = Article::findOrFail($articleId); $this->resetPage();
$article->reject(); }
$this->dispatch('article-updated'); public function approve(int $routeArticleId): void
{
RouteArticle::findOrFail($routeArticleId)->approve();
}
public function reject(int $routeArticleId): void
{
RouteArticle::findOrFail($routeArticleId)->reject();
}
public function restore(int $routeArticleId): void
{
$routeArticle = RouteArticle::findOrFail($routeArticleId);
$routeArticle->update(['approval_status' => ApprovalStatusEnum::PENDING]);
}
public function clear(): void
{
RouteArticle::where('approval_status', ApprovalStatusEnum::PENDING)
->update(['approval_status' => ApprovalStatusEnum::REJECTED]);
} }
public function refresh(): void public function refresh(): void
@ -39,17 +62,28 @@ public function refresh(): void
$this->dispatch('refresh-started'); $this->dispatch('refresh-started');
} }
public function render() public function render(): View
{ {
$articles = Article::with(['feed', 'articlePublication']) $query = RouteArticle::with(['article.feed', 'feed', 'platformChannel'])
->orderBy('created_at', 'desc') ->orderBy('created_at', 'desc');
->paginate(15);
$approvalsEnabled = Setting::isPublishingApprovalsEnabled(); if ($this->tab === 'pending') {
$query->where('approval_status', ApprovalStatusEnum::PENDING);
} elseif ($this->search !== '') {
$search = $this->search;
$query->whereHas('article', function ($q) use ($search) {
$q->where('title', 'like', "%{$search}%")
->orWhere('description', 'like', "%{$search}%");
});
}
$routeArticles = $query->paginate(15);
$pendingCount = RouteArticle::where('approval_status', ApprovalStatusEnum::PENDING)->count();
return view('livewire.articles', [ return view('livewire.articles', [
'articles' => $articles, 'routeArticles' => $routeArticles,
'approvalsEnabled' => $approvalsEnabled, 'pendingCount' => $pendingCount,
])->layout('layouts.app'); ])->layout('layouts.app');
} }
} }

View file

@ -4,6 +4,7 @@
use App\Models\PlatformAccount; use App\Models\PlatformAccount;
use App\Models\PlatformChannel; use App\Models\PlatformChannel;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
class Channels extends Component class Channels extends Component
@ -13,7 +14,7 @@ class Channels extends Component
public function toggle(int $channelId): void public function toggle(int $channelId): void
{ {
$channel = PlatformChannel::findOrFail($channelId); $channel = PlatformChannel::findOrFail($channelId);
$channel->is_active = !$channel->is_active; $channel->is_active = ! $channel->is_active;
$channel->save(); $channel->save();
} }
@ -29,13 +30,13 @@ public function closeAccountModal(): void
public function attachAccount(int $accountId): void public function attachAccount(int $accountId): void
{ {
if (!$this->managingChannelId) { if (! $this->managingChannelId) {
return; return;
} }
$channel = PlatformChannel::findOrFail($this->managingChannelId); $channel = PlatformChannel::findOrFail($this->managingChannelId);
if (!$channel->platformAccounts()->where('platform_account_id', $accountId)->exists()) { if (! $channel->platformAccounts()->where('platform_account_id', $accountId)->exists()) {
$channel->platformAccounts()->attach($accountId, [ $channel->platformAccounts()->attach($accountId, [
'is_active' => true, 'is_active' => true,
'priority' => 1, 'priority' => 1,
@ -51,7 +52,7 @@ public function detachAccount(int $channelId, int $accountId): void
$channel->platformAccounts()->detach($accountId); $channel->platformAccounts()->detach($accountId);
} }
public function render() public function render(): View
{ {
$channels = PlatformChannel::with(['platformInstance', 'platformAccounts'])->orderBy('name')->get(); $channels = PlatformChannel::with(['platformInstance', 'platformAccounts'])->orderBy('name')->get();
$allAccounts = PlatformAccount::where('is_active', true)->get(); $allAccounts = PlatformAccount::where('is_active', true)->get();
@ -61,7 +62,7 @@ public function render()
: null; : null;
$availableAccounts = $managingChannel $availableAccounts = $managingChannel
? $allAccounts->filter(fn($account) => !$managingChannel->platformAccounts->contains('id', $account->id)) ? $allAccounts->filter(fn ($account) => ! $managingChannel->platformAccounts->contains('id', $account->id))
: collect(); : collect();
return view('livewire.channels', [ return view('livewire.channels', [

View file

@ -3,6 +3,7 @@
namespace App\Livewire; namespace App\Livewire;
use App\Services\DashboardStatsService; use App\Services\DashboardStatsService;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
class Dashboard extends Component class Dashboard extends Component
@ -19,7 +20,7 @@ public function setPeriod(string $period): void
$this->period = $period; $this->period = $period;
} }
public function render() public function render(): View
{ {
$service = app(DashboardStatsService::class); $service = app(DashboardStatsService::class);

View file

@ -3,6 +3,7 @@
namespace App\Livewire; namespace App\Livewire;
use App\Models\Feed; use App\Models\Feed;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
class Feeds extends Component class Feeds extends Component
@ -10,11 +11,11 @@ class Feeds extends Component
public function toggle(int $feedId): void public function toggle(int $feedId): void
{ {
$feed = Feed::findOrFail($feedId); $feed = Feed::findOrFail($feedId);
$feed->is_active = !$feed->is_active; $feed->is_active = ! $feed->is_active;
$feed->save(); $feed->save();
} }
public function render() public function render(): View
{ {
$feeds = Feed::orderBy('name')->get(); $feeds = Feed::orderBy('name')->get();

View file

@ -0,0 +1,42 @@
<?php
namespace App\Livewire;
use App\Models\Notification;
use Illuminate\Contracts\View\View;
use Illuminate\Database\Eloquent\Collection;
use Livewire\Attributes\Computed;
use Livewire\Component;
class NotificationBell extends Component
{
public function markAsRead(int $id): void
{
Notification::findOrFail($id)->markAsRead();
}
public function markAllAsRead(): void
{
Notification::markAllAsRead();
}
#[Computed]
public function unreadCount(): int
{
return Notification::unread()->count();
}
/**
* @return Collection<int, Notification>
*/
#[Computed]
public function notifications(): Collection
{
return Notification::recent()->get();
}
public function render(): View
{
return view('livewire.notification-bell');
}
}

View file

@ -18,7 +18,9 @@
use App\Models\Setting; use App\Models\Setting;
use App\Services\OnboardingService; use App\Services\OnboardingService;
use Exception; use Exception;
use Illuminate\Contracts\View\View;
use InvalidArgumentException; use InvalidArgumentException;
use Livewire\Attributes\Locked;
use Livewire\Component; use Livewire\Component;
use RuntimeException; use RuntimeException;
@ -29,36 +31,54 @@ class Onboarding extends Component
// Platform form // Platform form
public string $instanceUrl = ''; public string $instanceUrl = '';
public string $username = ''; public string $username = '';
public string $password = ''; public string $password = '';
/** @var array<string, mixed>|null */
public ?array $existingAccount = null; public ?array $existingAccount = null;
// Feed form // Feed form
public string $feedName = ''; public string $feedName = '';
public string $feedProvider = 'vrt'; public string $feedProvider = 'vrt';
public ?int $feedLanguageId = null; public ?int $feedLanguageId = null;
public string $feedDescription = ''; public string $feedDescription = '';
// Channel form // Channel form
public string $channelName = ''; public string $channelName = '';
public ?int $platformInstanceId = null; public ?int $platformInstanceId = null;
public ?int $channelLanguageId = null; public ?int $channelLanguageId = null;
public string $channelDescription = ''; public string $channelDescription = '';
// Route form // Route form
public ?int $routeFeedId = null; public ?int $routeFeedId = null;
public ?int $routeChannelId = null; public ?int $routeChannelId = null;
public int $routePriority = 50; public int $routePriority = 50;
// State // State
/** @var array<string, string> */
public array $formErrors = []; public array $formErrors = [];
public bool $isLoading = false; public bool $isLoading = false;
#[\Livewire\Attributes\Locked]
#[Locked]
public ?int $previousChannelLanguageId = null; public ?int $previousChannelLanguageId = null;
protected CreatePlatformAccountAction $createPlatformAccountAction; protected CreatePlatformAccountAction $createPlatformAccountAction;
protected CreateFeedAction $createFeedAction; protected CreateFeedAction $createFeedAction;
protected CreateChannelAction $createChannelAction; protected CreateChannelAction $createChannelAction;
protected CreateRouteAction $createRouteAction; protected CreateRouteAction $createRouteAction;
public function boot( public function boot(
@ -188,7 +208,7 @@ public function createPlatformAccount(): void
} }
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Lemmy platform account creation failed', [ logger()->error('Lemmy platform account creation failed', [
'instance_url' => 'https://' . $this->instanceUrl, 'instance_url' => 'https://'.$this->instanceUrl,
'username' => $this->username, 'username' => $this->username,
'error' => $e->getMessage(), 'error' => $e->getMessage(),
'class' => get_class($e), 'class' => get_class($e),
@ -319,17 +339,20 @@ public function completeOnboarding(): void
/** /**
* Get language codes that have at least one active provider. * Get language codes that have at least one active provider.
*/ */
/**
* @return list<string>
*/
public function getAvailableLanguageCodes(): array public function getAvailableLanguageCodes(): array
{ {
$providers = config('feed.providers', []); $providers = config('feed.providers', []);
$languageCodes = []; $languageCodes = [];
foreach ($providers as $provider) { foreach ($providers as $provider) {
if (!($provider['is_active'] ?? false)) { if (! ($provider['is_active'] ?? false)) {
continue; continue;
} }
foreach (array_keys($provider['languages'] ?? []) as $code) { foreach (array_keys($provider['languages'] ?? []) as $code) {
$languageCodes[$code] = true; $languageCodes[(string) $code] = true;
} }
} }
@ -339,14 +362,17 @@ public function getAvailableLanguageCodes(): array
/** /**
* Get providers available for the current channel language. * Get providers available for the current channel language.
*/ */
/**
* @return array<int, array<string, string>>
*/
public function getProvidersForLanguage(): array public function getProvidersForLanguage(): array
{ {
if (!$this->channelLanguageId) { if (! $this->channelLanguageId) {
return []; return [];
} }
$language = Language::find($this->channelLanguageId); $language = Language::find($this->channelLanguageId);
if (!$language) { if (! $language) {
return []; return [];
} }
@ -355,7 +381,7 @@ public function getProvidersForLanguage(): array
$available = []; $available = [];
foreach ($providers as $key => $provider) { foreach ($providers as $key => $provider) {
if (!($provider['is_active'] ?? false)) { if (! ($provider['is_active'] ?? false)) {
continue; continue;
} }
if (isset($provider['languages'][$langCode])) { if (isset($provider['languages'][$langCode])) {
@ -375,13 +401,14 @@ public function getProvidersForLanguage(): array
*/ */
public function getChannelLanguage(): ?Language public function getChannelLanguage(): ?Language
{ {
if (!$this->channelLanguageId) { if (! $this->channelLanguageId) {
return null; return null;
} }
return Language::find($this->channelLanguageId); return Language::find($this->channelLanguageId);
} }
public function render() public function render(): View
{ {
// For channel step: only show languages that have providers // For channel step: only show languages that have providers
$availableCodes = $this->getAvailableLanguageCodes(); $availableCodes = $this->getAvailableLanguageCodes();

View file

@ -6,24 +6,32 @@
use App\Models\Keyword; use App\Models\Keyword;
use App\Models\PlatformChannel; use App\Models\PlatformChannel;
use App\Models\Route; use App\Models\Route;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
class Routes extends Component class Routes extends Component
{ {
public bool $showCreateModal = false; public bool $showCreateModal = false;
public ?int $editingFeedId = null; public ?int $editingFeedId = null;
public ?int $editingChannelId = null; public ?int $editingChannelId = null;
// Create form // Create form
public ?int $newFeedId = null; public ?int $newFeedId = null;
public ?int $newChannelId = null; public ?int $newChannelId = null;
public int $newPriority = 50; public int $newPriority = 50;
// Edit form // Edit form
public int $editPriority = 50; public int $editPriority = 50;
public string $editAutoApprove = '';
// Keyword management // Keyword management
public string $newKeyword = ''; public string $newKeyword = '';
public bool $showKeywordInput = false; public bool $showKeywordInput = false;
public function openCreateModal(): void public function openCreateModal(): void
@ -53,6 +61,7 @@ public function createRoute(): void
if ($exists) { if ($exists) {
$this->addError('newFeedId', 'This route already exists.'); $this->addError('newFeedId', 'This route already exists.');
return; return;
} }
@ -75,6 +84,7 @@ public function openEditModal(int $feedId, int $channelId): void
$this->editingFeedId = $feedId; $this->editingFeedId = $feedId;
$this->editingChannelId = $channelId; $this->editingChannelId = $channelId;
$this->editPriority = $route->priority; $this->editPriority = $route->priority;
$this->editAutoApprove = $route->auto_approve === null ? '' : ($route->auto_approve ? '1' : '0');
$this->newKeyword = ''; $this->newKeyword = '';
$this->showKeywordInput = false; $this->showKeywordInput = false;
} }
@ -87,7 +97,7 @@ public function closeEditModal(): void
public function updateRoute(): void public function updateRoute(): void
{ {
if (!$this->editingFeedId || !$this->editingChannelId) { if (! $this->editingFeedId || ! $this->editingChannelId) {
return; return;
} }
@ -95,9 +105,18 @@ public function updateRoute(): void
'editPriority' => 'required|integer|min:0', 'editPriority' => 'required|integer|min:0',
]); ]);
$autoApprove = match ($this->editAutoApprove) {
'1' => true,
'0' => false,
default => null,
};
Route::where('feed_id', $this->editingFeedId) Route::where('feed_id', $this->editingFeedId)
->where('platform_channel_id', $this->editingChannelId) ->where('platform_channel_id', $this->editingChannelId)
->update(['priority' => $this->editPriority]); ->update([
'priority' => $this->editPriority,
'auto_approve' => $autoApprove,
]);
$this->closeEditModal(); $this->closeEditModal();
} }
@ -108,7 +127,7 @@ public function toggle(int $feedId, int $channelId): void
->where('platform_channel_id', $channelId) ->where('platform_channel_id', $channelId)
->firstOrFail(); ->firstOrFail();
$route->is_active = !$route->is_active; $route->is_active = ! $route->is_active;
$route->save(); $route->save();
} }
@ -126,7 +145,7 @@ public function delete(int $feedId, int $channelId): void
public function addKeyword(): void public function addKeyword(): void
{ {
if (!$this->editingFeedId || !$this->editingChannelId || empty(trim($this->newKeyword))) { if (! $this->editingFeedId || ! $this->editingChannelId || empty(trim($this->newKeyword))) {
return; return;
} }
@ -144,7 +163,7 @@ public function addKeyword(): void
public function toggleKeyword(int $keywordId): void public function toggleKeyword(int $keywordId): void
{ {
$keyword = Keyword::findOrFail($keywordId); $keyword = Keyword::findOrFail($keywordId);
$keyword->is_active = !$keyword->is_active; $keyword->is_active = ! $keyword->is_active;
$keyword->save(); $keyword->save();
} }
@ -153,22 +172,23 @@ public function deleteKeyword(int $keywordId): void
Keyword::destroy($keywordId); Keyword::destroy($keywordId);
} }
public function render() public function render(): View
{ {
$routes = Route::with(['feed', 'platformChannel']) $routes = Route::with(['feed', 'platformChannel.platformInstance'])
->orderBy('priority', 'desc') ->orderBy('priority', 'desc')
->get(); ->get();
// Batch load keywords for all routes to avoid N+1 queries // Batch load keywords for all routes to avoid N+1 queries
$routeKeys = $routes->map(fn($r) => $r->feed_id . '-' . $r->platform_channel_id); $routeKeys = $routes->map(fn ($r) => $r->feed_id.'-'.$r->platform_channel_id);
$allKeywords = Keyword::whereIn('feed_id', $routes->pluck('feed_id')) $allKeywords = Keyword::whereIn('feed_id', $routes->pluck('feed_id'))
->whereIn('platform_channel_id', $routes->pluck('platform_channel_id')) ->whereIn('platform_channel_id', $routes->pluck('platform_channel_id'))
->get() ->get()
->groupBy(fn($k) => $k->feed_id . '-' . $k->platform_channel_id); ->groupBy(fn ($k) => $k->feed_id.'-'.$k->platform_channel_id);
$routes = $routes->map(function ($route) use ($allKeywords) { $routes = $routes->map(function ($route) use ($allKeywords) {
$key = $route->feed_id . '-' . $route->platform_channel_id; $key = $route->feed_id.'-'.$route->platform_channel_id;
$route->keywords = $allKeywords->get($key, collect()); $route->setRelation('keywords', $allKeywords->get($key, collect()));
return $route; return $route;
}); });
@ -179,7 +199,7 @@ public function render()
$editingKeywords = collect(); $editingKeywords = collect();
if ($this->editingFeedId && $this->editingChannelId) { if ($this->editingFeedId && $this->editingChannelId) {
$editingRoute = Route::with(['feed', 'platformChannel']) $editingRoute = Route::with(['feed', 'platformChannel.platformInstance'])
->where('feed_id', $this->editingFeedId) ->where('feed_id', $this->editingFeedId)
->where('platform_channel_id', $this->editingChannelId) ->where('platform_channel_id', $this->editingChannelId)
->first(); ->first();

View file

@ -3,15 +3,21 @@
namespace App\Livewire; namespace App\Livewire;
use App\Models\Setting; use App\Models\Setting;
use Illuminate\Contracts\View\View;
use Livewire\Component; use Livewire\Component;
class Settings extends Component class Settings extends Component
{ {
public bool $articleProcessingEnabled = true; public bool $articleProcessingEnabled = true;
public bool $publishingApprovalsEnabled = false; public bool $publishingApprovalsEnabled = false;
public int $articlePublishingInterval = 5; public int $articlePublishingInterval = 5;
public int $feedStalenessThreshold = 48;
public ?string $successMessage = null; public ?string $successMessage = null;
public ?string $errorMessage = null; public ?string $errorMessage = null;
public function mount(): void public function mount(): void
@ -19,18 +25,19 @@ public function mount(): void
$this->articleProcessingEnabled = Setting::isArticleProcessingEnabled(); $this->articleProcessingEnabled = Setting::isArticleProcessingEnabled();
$this->publishingApprovalsEnabled = Setting::isPublishingApprovalsEnabled(); $this->publishingApprovalsEnabled = Setting::isPublishingApprovalsEnabled();
$this->articlePublishingInterval = Setting::getArticlePublishingInterval(); $this->articlePublishingInterval = Setting::getArticlePublishingInterval();
$this->feedStalenessThreshold = Setting::getFeedStalenessThreshold();
} }
public function toggleArticleProcessing(): void public function toggleArticleProcessing(): void
{ {
$this->articleProcessingEnabled = !$this->articleProcessingEnabled; $this->articleProcessingEnabled = ! $this->articleProcessingEnabled;
Setting::setArticleProcessingEnabled($this->articleProcessingEnabled); Setting::setArticleProcessingEnabled($this->articleProcessingEnabled);
$this->showSuccess(); $this->showSuccess();
} }
public function togglePublishingApprovals(): void public function togglePublishingApprovals(): void
{ {
$this->publishingApprovalsEnabled = !$this->publishingApprovalsEnabled; $this->publishingApprovalsEnabled = ! $this->publishingApprovalsEnabled;
Setting::setPublishingApprovalsEnabled($this->publishingApprovalsEnabled); Setting::setPublishingApprovalsEnabled($this->publishingApprovalsEnabled);
$this->showSuccess(); $this->showSuccess();
} }
@ -45,6 +52,16 @@ public function updateArticlePublishingInterval(): void
$this->showSuccess(); $this->showSuccess();
} }
public function updateFeedStalenessThreshold(): void
{
$this->validate([
'feedStalenessThreshold' => 'required|integer|min:0',
]);
Setting::setFeedStalenessThreshold($this->feedStalenessThreshold);
$this->showSuccess();
}
protected function showSuccess(): void protected function showSuccess(): void
{ {
$this->successMessage = 'Settings updated successfully!'; $this->successMessage = 'Settings updated successfully!';
@ -60,7 +77,7 @@ public function clearMessages(): void
$this->errorMessage = null; $this->errorMessage = null;
} }
public function render() public function render(): View
{ {
return view('livewire.settings')->layout('layouts.app'); return view('livewire.settings')->layout('layouts.app');
} }

View file

@ -2,12 +2,12 @@
namespace App\Models; namespace App\Models;
use App\Events\ArticleApproved;
use App\Events\NewArticleFetched; use App\Events\NewArticleFetched;
use Database\Factories\ArticleFactory; use Database\Factories\ArticleFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Database\Eloquent\Relations\HasOne; use Illuminate\Database\Eloquent\Relations\HasOne;
use Illuminate\Support\Carbon; use Illuminate\Support\Carbon;
@ -15,15 +15,17 @@
* @method static firstOrCreate(array<string, mixed> $array) * @method static firstOrCreate(array<string, mixed> $array)
* @method static where(string $string, string $url) * @method static where(string $string, string $url)
* @method static create(array<string, mixed> $array) * @method static create(array<string, mixed> $array)
* @property integer $id *
* @property int $id
* @property int $feed_id * @property int $feed_id
* @property Feed $feed * @property Feed $feed
* @property string $url * @property string $url
* @property bool|null $is_valid * @property string $title
* @property string|null $description
* @property Carbon|null $validated_at * @property Carbon|null $validated_at
* @property Carbon $created_at * @property Carbon $created_at
* @property Carbon $updated_at * @property Carbon $updated_at
* @property ArticlePublication $articlePublication * @property ArticlePublication|null $articlePublication
*/ */
class Article extends Model class Article extends Model
{ {
@ -39,9 +41,7 @@ class Article extends Model
'image_url', 'image_url',
'published_at', 'published_at',
'author', 'author',
'approval_status',
'validated_at', 'validated_at',
'publish_status',
]; ];
/** /**
@ -50,8 +50,6 @@ class Article extends Model
public function casts(): array public function casts(): array
{ {
return [ return [
'approval_status' => 'string',
'publish_status' => 'string',
'published_at' => 'datetime', 'published_at' => 'datetime',
'validated_at' => 'datetime', 'validated_at' => 'datetime',
'created_at' => 'datetime', 'created_at' => 'datetime',
@ -59,58 +57,6 @@ public function casts(): array
]; ];
} }
public function isValid(): bool
{
return $this->validated_at !== null && ! $this->isRejected();
}
public function isApproved(): bool
{
return $this->approval_status === 'approved';
}
public function isPending(): bool
{
return $this->approval_status === 'pending';
}
public function isRejected(): bool
{
return $this->approval_status === 'rejected';
}
public function approve(string $approvedBy = null): void
{
$this->update([
'approval_status' => 'approved',
]);
// Fire event to trigger publishing
event(new ArticleApproved($this));
}
public function reject(string $rejectedBy = null): void
{
$this->update([
'approval_status' => 'rejected',
]);
}
public function canBePublished(): bool
{
if (!$this->isValid()) {
return false;
}
// If approval system is disabled, auto-approve valid articles
if (!\App\Models\Setting::isPublishingApprovalsEnabled()) {
return true;
}
// If approval system is enabled, only approved articles can be published
return $this->isApproved();
}
public function getIsPublishedAttribute(): bool public function getIsPublishedAttribute(): bool
{ {
return $this->articlePublication()->exists(); return $this->articlePublication()->exists();
@ -124,6 +70,14 @@ public function articlePublication(): HasOne
return $this->hasOne(ArticlePublication::class); return $this->hasOne(ArticlePublication::class);
} }
/**
* @return HasMany<ArticlePublication, $this>
*/
public function articlePublications(): HasMany
{
return $this->hasMany(ArticlePublication::class);
}
/** /**
* @return BelongsTo<Feed, $this> * @return BelongsTo<Feed, $this>
*/ */
@ -132,6 +86,14 @@ public function feed(): BelongsTo
return $this->belongsTo(Feed::class); return $this->belongsTo(Feed::class);
} }
/**
* @return HasMany<RouteArticle, $this>
*/
public function routeArticles(): HasMany
{
return $this->hasMany(RouteArticle::class);
}
public function dispatchFetchedEvent(): void public function dispatchFetchedEvent(): void
{ {
event(new NewArticleFetched($this)); event(new NewArticleFetched($this));

View file

@ -6,11 +6,19 @@
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Support\Carbon;
/** /**
* @property integer $article_id * @property int $id
* @property integer $platform_channel_id * @property int $article_id
* @property integer $post_id * @property int $platform_channel_id
* @property string $post_id
* @property string $platform
* @property string $published_by
* @property array<string, mixed>|null $publication_data
* @property Carbon $published_at
* @property Carbon $created_at
* @property Carbon $updated_at
* *
* @method static create(array<string, mixed> $array) * @method static create(array<string, mixed> $array)
*/ */
@ -18,7 +26,7 @@ class ArticlePublication extends Model
{ {
/** @use HasFactory<ArticlePublicationFactory> */ /** @use HasFactory<ArticlePublicationFactory> */
use HasFactory; use HasFactory;
protected $fillable = [ protected $fillable = [
'article_id', 'article_id',
'platform_channel_id', 'platform_channel_id',

View file

@ -3,6 +3,7 @@
namespace App\Models; namespace App\Models;
use Database\Factories\FeedFactory; use Database\Factories\FeedFactory;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;
@ -16,7 +17,7 @@
* @property string $url * @property string $url
* @property string $type * @property string $type
* @property string $provider * @property string $provider
* @property int $language_id * @property int|null $language_id
* @property Language|null $language * @property Language|null $language
* @property string $description * @property string $description
* @property array<string, mixed> $settings * @property array<string, mixed> $settings
@ -24,6 +25,7 @@
* @property Carbon|null $last_fetched_at * @property Carbon|null $last_fetched_at
* @property Carbon $created_at * @property Carbon $created_at
* @property Carbon $updated_at * @property Carbon $updated_at
*
* @method static orderBy(string $string, string $string1) * @method static orderBy(string $string, string $string1)
* @method static where(string $string, true $true) * @method static where(string $string, true $true)
* @method static findOrFail(mixed $feed_id) * @method static findOrFail(mixed $feed_id)
@ -32,7 +34,9 @@ class Feed extends Model
{ {
/** @use HasFactory<FeedFactory> */ /** @use HasFactory<FeedFactory> */
use HasFactory; use HasFactory;
private const RECENT_FETCH_THRESHOLD_HOURS = 2; private const RECENT_FETCH_THRESHOLD_HOURS = 2;
private const DAILY_FETCH_THRESHOLD_HOURS = 24; private const DAILY_FETCH_THRESHOLD_HOURS = 24;
protected $fillable = [ protected $fillable = [
@ -44,13 +48,13 @@ class Feed extends Model
'description', 'description',
'settings', 'settings',
'is_active', 'is_active',
'last_fetched_at' 'last_fetched_at',
]; ];
protected $casts = [ protected $casts = [
'settings' => 'array', 'settings' => 'array',
'is_active' => 'boolean', 'is_active' => 'boolean',
'last_fetched_at' => 'datetime' 'last_fetched_at' => 'datetime',
]; ];
public function getTypeDisplayAttribute(): string public function getTypeDisplayAttribute(): string
@ -64,11 +68,11 @@ public function getTypeDisplayAttribute(): string
public function getStatusAttribute(): string public function getStatusAttribute(): string
{ {
if (!$this->is_active) { if (! $this->is_active) {
return 'Inactive'; return 'Inactive';
} }
if (!$this->last_fetched_at) { if (! $this->last_fetched_at) {
return 'Never fetched'; return 'Never fetched';
} }
@ -79,12 +83,25 @@ public function getStatusAttribute(): string
} elseif ($hoursAgo < self::DAILY_FETCH_THRESHOLD_HOURS) { } elseif ($hoursAgo < self::DAILY_FETCH_THRESHOLD_HOURS) {
return "Fetched {$hoursAgo}h ago"; return "Fetched {$hoursAgo}h ago";
} else { } else {
return "Fetched " . $this->last_fetched_at->diffForHumans(); return 'Fetched '.$this->last_fetched_at->diffForHumans();
} }
} }
/** /**
* @return BelongsToMany<PlatformChannel, $this, Route> * @param Builder<Feed> $query
* @return Builder<Feed>
*/
public function scopeStale(Builder $query, int $thresholdHours): Builder
{
return $query->where('is_active', true)
->where(function (Builder $query) use ($thresholdHours) {
$query->whereNull('last_fetched_at')
->orWhere('last_fetched_at', '<', now()->subHours($thresholdHours));
});
}
/**
* @return BelongsToMany<PlatformChannel, $this>
*/ */
public function channels(): BelongsToMany public function channels(): BelongsToMany
{ {
@ -94,7 +111,7 @@ public function channels(): BelongsToMany
} }
/** /**
* @return BelongsToMany<PlatformChannel, $this, Route> * @return BelongsToMany<PlatformChannel, $this>
*/ */
public function activeChannels(): BelongsToMany public function activeChannels(): BelongsToMany
{ {

View file

@ -2,6 +2,7 @@
namespace App\Models; namespace App\Models;
use Database\Factories\KeywordFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;
@ -20,17 +21,18 @@
*/ */
class Keyword extends Model class Keyword extends Model
{ {
/** @use HasFactory<KeywordFactory> */
use HasFactory; use HasFactory;
protected $fillable = [ protected $fillable = [
'feed_id', 'feed_id',
'platform_channel_id', 'platform_channel_id',
'keyword', 'keyword',
'is_active' 'is_active',
]; ];
protected $casts = [ protected $casts = [
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
/** /**
@ -48,5 +50,4 @@ public function platformChannel(): BelongsTo
{ {
return $this->belongsTo(PlatformChannel::class); return $this->belongsTo(PlatformChannel::class);
} }
} }

View file

@ -15,13 +15,13 @@ class Language extends Model
protected $fillable = [ protected $fillable = [
'short_code', 'short_code',
'name', 'name',
'native_name', 'native_name',
'is_active' 'is_active',
]; ];
protected $casts = [ protected $casts = [
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
/** /**

View file

@ -3,12 +3,14 @@
namespace App\Models; namespace App\Models;
use App\Enums\LogLevelEnum; use App\Enums\LogLevelEnum;
use Database\Factories\LogFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Carbon; use Illuminate\Support\Carbon;
/** /**
* @method static create(array $array) * @method static create(array<string, mixed> $array)
*
* @property LogLevelEnum $level * @property LogLevelEnum $level
* @property string $message * @property string $message
* @property array<string, mixed> $context * @property array<string, mixed> $context
@ -17,6 +19,7 @@
*/ */
class Log extends Model class Log extends Model
{ {
/** @use HasFactory<LogFactory> */
use HasFactory; use HasFactory;
protected $table = 'logs'; protected $table = 'logs';

View file

@ -0,0 +1,92 @@
<?php
namespace App\Models;
use App\Enums\NotificationSeverityEnum;
use App\Enums\NotificationTypeEnum;
use Database\Factories\NotificationFactory;
use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\MorphTo;
use Illuminate\Support\Carbon;
/**
* @property int $id
* @property NotificationTypeEnum $type
* @property NotificationSeverityEnum $severity
* @property string $title
* @property string $message
* @property array<string, mixed>|null $data
* @property string|null $notifiable_type
* @property int|null $notifiable_id
* @property Carbon|null $read_at
* @property Carbon $created_at
* @property Carbon $updated_at
*/
class Notification extends Model
{
/** @use HasFactory<NotificationFactory> */
use HasFactory;
protected $fillable = [
'type',
'severity',
'title',
'message',
'data',
'notifiable_type',
'notifiable_id',
'read_at',
];
protected $casts = [
'type' => NotificationTypeEnum::class,
'severity' => NotificationSeverityEnum::class,
'data' => 'array',
'read_at' => 'datetime',
'created_at' => 'datetime',
'updated_at' => 'datetime',
];
/**
* @return MorphTo<Model, $this>
*/
public function notifiable(): MorphTo
{
return $this->morphTo();
}
public function isRead(): bool
{
return $this->read_at !== null;
}
public function markAsRead(): void
{
$this->update(['read_at' => now()]);
}
/**
* @param Builder<Notification> $query
* @return Builder<Notification>
*/
public function scopeUnread(Builder $query): Builder
{
return $query->whereNull('read_at');
}
/**
* @param Builder<Notification> $query
* @return Builder<Notification>
*/
public function scopeRecent(Builder $query): Builder
{
return $query->latest()->limit(50);
}
public static function markAllAsRead(): void
{
static::unread()->update(['read_at' => now()]);
}
}

View file

@ -2,15 +2,15 @@
namespace App\Models; namespace App\Models;
use App\Enums\PlatformEnum;
use Database\Factories\PlatformAccountFactory; use Database\Factories\PlatformAccountFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Casts\Attribute; use Illuminate\Database\Eloquent\Casts\Attribute;
use Illuminate\Database\Eloquent\Collection; use Illuminate\Database\Eloquent\Collection;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsToMany; use Illuminate\Database\Eloquent\Relations\BelongsToMany;
use Illuminate\Support\Carbon; use Illuminate\Support\Carbon;
use Illuminate\Support\Facades\Crypt; use Illuminate\Support\Facades\Crypt;
use App\Enums\PlatformEnum;
/** /**
* @property int $id * @property int $id
@ -18,13 +18,14 @@
* @property string $instance_url * @property string $instance_url
* @property string $username * @property string $username
* @property string $password * @property string $password
* @property string $settings * @property array<string, mixed> $settings
* @property bool $is_active * @property bool $is_active
* @property Carbon $last_tested_at * @property Carbon|null $last_tested_at
* @property string $status * @property string $status
* @property Carbon $created_at * @property Carbon $created_at
* @property Carbon $updated_at * @property Carbon $updated_at
* @property Collection<int, PlatformChannel> $activeChannels * @property Collection<int, PlatformChannel> $activeChannels
*
* @method static where(string $string, PlatformEnum $platform) * @method static where(string $string, PlatformEnum $platform)
* @method static orderBy(string $string) * @method static orderBy(string $string)
* @method static create(array<string, mixed> $validated) * @method static create(array<string, mixed> $validated)
@ -42,14 +43,14 @@ class PlatformAccount extends Model
'settings', 'settings',
'is_active', 'is_active',
'last_tested_at', 'last_tested_at',
'status' 'status',
]; ];
protected $casts = [ protected $casts = [
'platform' => PlatformEnum::class, 'platform' => PlatformEnum::class,
'settings' => 'array', 'settings' => 'array',
'is_active' => 'boolean', 'is_active' => 'boolean',
'last_tested_at' => 'datetime' 'last_tested_at' => 'datetime',
]; ];
// Encrypt password when storing // Encrypt password when storing
@ -64,12 +65,12 @@ protected function password(): Attribute
if (is_null($value)) { if (is_null($value)) {
return null; return null;
} }
// Return empty string if value is empty // Return empty string if value is empty
if (empty($value)) { if (empty($value)) {
return ''; return '';
} }
try { try {
return Crypt::decryptString($value); return Crypt::decryptString($value);
} catch (\Exception $e) { } catch (\Exception $e) {
@ -82,18 +83,17 @@ protected function password(): Attribute
if (is_null($value)) { if (is_null($value)) {
return null; return null;
} }
// Store empty string as null // Store empty string as null
if (empty($value)) { if (empty($value)) {
return null; return null;
} }
return Crypt::encryptString($value); return Crypt::encryptString($value);
}, },
)->withoutObjectCaching(); )->withoutObjectCaching();
} }
// Get the active accounts for a platform (returns collection) // Get the active accounts for a platform (returns collection)
/** /**
* @return Collection<int, PlatformAccount> * @return Collection<int, PlatformAccount>

View file

@ -10,15 +10,16 @@
/** /**
* @method static findMany(mixed $channel_ids) * @method static findMany(mixed $channel_ids)
* @method static create(array $array) * @method static create(array<string, mixed> $array)
* @property integer $id *
* @property integer $platform_instance_id * @property int $id
* @property int $platform_instance_id
* @property PlatformInstance $platformInstance * @property PlatformInstance $platformInstance
* @property integer $channel_id * @property int $channel_id
* @property string $name * @property string $name
* @property int $language_id * @property int $language_id
* @property Language|null $language * @property Language|null $language
* @property boolean $is_active * @property bool $is_active
*/ */
class PlatformChannel extends Model class PlatformChannel extends Model
{ {
@ -34,11 +35,11 @@ class PlatformChannel extends Model
'channel_id', 'channel_id',
'description', 'description',
'language_id', 'language_id',
'is_active' 'is_active',
]; ];
protected $casts = [ protected $casts = [
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
/** /**
@ -70,11 +71,11 @@ public function activePlatformAccounts(): BelongsToMany
public function getFullNameAttribute(): string public function getFullNameAttribute(): string
{ {
// For Lemmy, use /c/ prefix // For Lemmy, use /c/ prefix
return $this->platformInstance->url . '/c/' . $this->name; return $this->platformInstance->url.'/c/'.$this->name;
} }
/** /**
* @return BelongsToMany<Feed, $this, Route> * @return BelongsToMany<Feed, $this>
*/ */
public function feeds(): BelongsToMany public function feeds(): BelongsToMany
{ {
@ -84,7 +85,7 @@ public function feeds(): BelongsToMany
} }
/** /**
* @return BelongsToMany<Feed, $this, Route> * @return BelongsToMany<Feed, $this>
*/ */
public function activeFeeds(): BelongsToMany public function activeFeeds(): BelongsToMany
{ {

View file

@ -3,6 +3,7 @@
namespace App\Models; namespace App\Models;
use App\Enums\PlatformEnum; use App\Enums\PlatformEnum;
use Illuminate\Database\Eloquent\Factories\Factory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
@ -12,7 +13,9 @@
*/ */
class PlatformChannelPost extends Model class PlatformChannelPost extends Model
{ {
/** @use HasFactory<Factory<PlatformChannelPost>> */
use HasFactory; use HasFactory;
protected $fillable = [ protected $fillable = [
'platform', 'platform',
'channel_id', 'channel_id',
@ -44,7 +47,7 @@ public static function urlExists(PlatformEnum $platform, string $channelId, stri
public static function duplicateExists(PlatformEnum $platform, string $channelId, ?string $url, ?string $title): bool public static function duplicateExists(PlatformEnum $platform, string $channelId, ?string $url, ?string $title): bool
{ {
if (!$url && !$title) { if (! $url && ! $title) {
return false; return false;
} }

View file

@ -12,28 +12,29 @@
/** /**
* @method static updateOrCreate(array<string, mixed> $array, $instanceData) * @method static updateOrCreate(array<string, mixed> $array, $instanceData)
* @method static where(string $string, mixed $operator) * @method static where(string $string, mixed $operator)
*
* @property PlatformEnum $platform * @property PlatformEnum $platform
* @property string $url * @property string $url
* @property string $name * @property string $name
* @property string $description * @property string $description
* @property boolean $is_active * @property bool $is_active
*/ */
class PlatformInstance extends Model class PlatformInstance extends Model
{ {
/** @use HasFactory<PlatformInstanceFactory> */ /** @use HasFactory<PlatformInstanceFactory> */
use HasFactory; use HasFactory;
protected $fillable = [ protected $fillable = [
'platform', 'platform',
'url', 'url',
'name', 'name',
'description', 'description',
'is_active' 'is_active',
]; ];
protected $casts = [ protected $casts = [
'platform' => PlatformEnum::class, 'platform' => PlatformEnum::class,
'is_active' => 'boolean' 'is_active' => 'boolean',
]; ];
/** /**

View file

@ -4,9 +4,9 @@
use Database\Factories\RouteFactory; use Database\Factories\RouteFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo; use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Database\Eloquent\Relations\HasMany; use Illuminate\Database\Eloquent\Relations\HasMany;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Carbon; use Illuminate\Support\Carbon;
/** /**
@ -14,6 +14,7 @@
* @property int $platform_channel_id * @property int $platform_channel_id
* @property bool $is_active * @property bool $is_active
* @property int $priority * @property int $priority
* @property bool|null $auto_approve
* @property Carbon $created_at * @property Carbon $created_at
* @property Carbon $updated_at * @property Carbon $updated_at
*/ */
@ -21,22 +22,25 @@ class Route extends Model
{ {
/** @use HasFactory<RouteFactory> */ /** @use HasFactory<RouteFactory> */
use HasFactory; use HasFactory;
protected $table = 'routes'; protected $table = 'routes';
// Laravel doesn't handle composite primary keys well, so we'll use regular queries // Laravel doesn't handle composite primary keys well, so we'll use regular queries
protected $primaryKey = null; protected $primaryKey = null;
public $incrementing = false; public $incrementing = false;
protected $fillable = [ protected $fillable = [
'feed_id', 'feed_id',
'platform_channel_id', 'platform_channel_id',
'is_active', 'is_active',
'priority' 'priority',
'auto_approve',
]; ];
protected $casts = [ protected $casts = [
'is_active' => 'boolean' 'is_active' => 'boolean',
'auto_approve' => 'boolean',
]; ];
/** /**
@ -63,4 +67,13 @@ public function keywords(): HasMany
return $this->hasMany(Keyword::class, 'feed_id', 'feed_id') return $this->hasMany(Keyword::class, 'feed_id', 'feed_id')
->where('platform_channel_id', $this->platform_channel_id); ->where('platform_channel_id', $this->platform_channel_id);
} }
/**
* @return HasMany<RouteArticle, $this>
*/
public function routeArticles(): HasMany
{
return $this->hasMany(RouteArticle::class, 'feed_id', 'feed_id')
->where('platform_channel_id', $this->platform_channel_id);
}
} }

104
app/Models/RouteArticle.php Normal file
View file

@ -0,0 +1,104 @@
<?php
namespace App\Models;
use App\Enums\ApprovalStatusEnum;
use App\Enums\PublishStatusEnum;
use App\Events\RouteArticleApproved;
use Database\Factories\RouteArticleFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Relations\BelongsTo;
use Illuminate\Support\Carbon;
/**
* @property int $id
* @property int $feed_id
* @property int $platform_channel_id
* @property int $article_id
* @property ApprovalStatusEnum $approval_status
* @property PublishStatusEnum $publish_status
* @property Carbon|null $validated_at
* @property Carbon $created_at
* @property Carbon $updated_at
*/
class RouteArticle extends Model
{
/** @use HasFactory<RouteArticleFactory> */
use HasFactory;
protected $fillable = [
'feed_id',
'platform_channel_id',
'article_id',
'approval_status',
'publish_status',
'validated_at',
];
protected $casts = [
'approval_status' => ApprovalStatusEnum::class,
'publish_status' => PublishStatusEnum::class,
'validated_at' => 'datetime',
];
/**
* @return BelongsTo<Route, $this>
*/
public function route(): BelongsTo
{
return $this->belongsTo(Route::class, 'feed_id', 'feed_id')
->where('platform_channel_id', $this->platform_channel_id);
}
/**
* @return BelongsTo<Article, $this>
*/
public function article(): BelongsTo
{
return $this->belongsTo(Article::class);
}
/**
* @return BelongsTo<Feed, $this>
*/
public function feed(): BelongsTo
{
return $this->belongsTo(Feed::class);
}
/**
* @return BelongsTo<PlatformChannel, $this>
*/
public function platformChannel(): BelongsTo
{
return $this->belongsTo(PlatformChannel::class);
}
public function isPending(): bool
{
return $this->approval_status === ApprovalStatusEnum::PENDING;
}
public function isApproved(): bool
{
return $this->approval_status === ApprovalStatusEnum::APPROVED;
}
public function isRejected(): bool
{
return $this->approval_status === ApprovalStatusEnum::REJECTED;
}
public function approve(): void
{
$this->update(['approval_status' => ApprovalStatusEnum::APPROVED]);
event(new RouteArticleApproved($this));
}
public function reject(): void
{
$this->update(['approval_status' => ApprovalStatusEnum::REJECTED]);
}
}

View file

@ -2,16 +2,18 @@
namespace App\Models; namespace App\Models;
use Database\Factories\SettingFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model; use Illuminate\Database\Eloquent\Model;
/** /**
* @method static updateOrCreate(string[] $array, array $array1) * @method static updateOrCreate(array<string, string> $array, array<string, mixed> $array1)
* @method static create(string[] $array) * @method static create(array<string, string> $array)
* @method static where(string $string, string $key) * @method static where(string $string, string $key)
*/ */
class Setting extends Model class Setting extends Model
{ {
/** @use HasFactory<SettingFactory> */
use HasFactory; use HasFactory;
protected $fillable = ['key', 'value']; protected $fillable = ['key', 'value'];
@ -69,4 +71,14 @@ public static function setArticlePublishingInterval(int $minutes): void
{ {
static::set('article_publishing_interval', (string) $minutes); static::set('article_publishing_interval', (string) $minutes);
} }
public static function getFeedStalenessThreshold(): int
{
return (int) static::get('feed_staleness_threshold', 48);
}
public static function setFeedStalenessThreshold(int $hours): void
{
static::set('feed_staleness_threshold', (string) $hours);
}
} }

View file

@ -3,6 +3,7 @@
namespace App\Models; namespace App\Models;
// use Illuminate\Contracts\Auth\MustVerifyEmail; // use Illuminate\Contracts\Auth\MustVerifyEmail;
use Database\Factories\UserFactory;
use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Foundation\Auth\User as Authenticatable; use Illuminate\Foundation\Auth\User as Authenticatable;
use Illuminate\Notifications\Notifiable; use Illuminate\Notifications\Notifiable;
@ -10,8 +11,8 @@
class User extends Authenticatable class User extends Authenticatable
{ {
/** @use HasFactory<\Database\Factories\UserFactory> */ /** @use HasFactory<UserFactory> */
use HasFactory, Notifiable, HasApiTokens; use HasApiTokens, HasFactory, Notifiable;
/** /**
* The attributes that are mass assignable. * The attributes that are mass assignable.

View file

@ -2,13 +2,15 @@
namespace App\Modules\Lemmy; namespace App\Modules\Lemmy;
use Illuminate\Support\Facades\Http;
use Illuminate\Http\Client\Response; use Illuminate\Http\Client\Response;
use Illuminate\Support\Facades\Http;
class LemmyRequest class LemmyRequest
{ {
private string $instance; private string $instance;
private ?string $token; private ?string $token;
private string $scheme = 'https'; private string $scheme = 'https';
public function __construct(string $instance, ?string $token = null) public function __construct(string $instance, ?string $token = null)
@ -45,11 +47,12 @@ public function withScheme(string $scheme): self
if (in_array($scheme, ['http', 'https'], true)) { if (in_array($scheme, ['http', 'https'], true)) {
$this->scheme = $scheme; $this->scheme = $scheme;
} }
return $this; return $this;
} }
/** /**
* @param array<string, mixed> $params * @param array<string, mixed> $params
*/ */
public function get(string $endpoint, array $params = []): Response public function get(string $endpoint, array $params = []): Response
{ {
@ -65,7 +68,7 @@ public function get(string $endpoint, array $params = []): Response
} }
/** /**
* @param array<string, mixed> $data * @param array<string, mixed> $data
*/ */
public function post(string $endpoint, array $data = []): Response public function post(string $endpoint, array $data = []): Response
{ {
@ -83,6 +86,7 @@ public function post(string $endpoint, array $data = []): Response
public function withToken(string $token): self public function withToken(string $token): self
{ {
$this->token = $token; $this->token = $token;
return $this; return $this;
} }
} }

View file

@ -39,7 +39,7 @@ public function login(string $username, string $password): ?string
'password' => $password, 'password' => $password,
]); ]);
if (!$response->successful()) { if (! $response->successful()) {
$responseBody = $response->body(); $responseBody = $response->body();
logger()->error('Lemmy login failed', [ logger()->error('Lemmy login failed', [
'status' => $response->status(), 'status' => $response->status(),
@ -61,6 +61,7 @@ public function login(string $username, string $password): ?string
} }
$data = $response->json(); $data = $response->json();
return $data['jwt'] ?? null; return $data['jwt'] ?? null;
} catch (Exception $e) { } catch (Exception $e) {
// Re-throw rate limit exceptions immediately // Re-throw rate limit exceptions immediately
@ -74,7 +75,7 @@ public function login(string $username, string $password): ?string
continue; continue;
} }
// Connection failed - throw exception to distinguish from auth failure // Connection failed - throw exception to distinguish from auth failure
throw new Exception('Connection failed: ' . $e->getMessage()); throw new Exception('Connection failed: '.$e->getMessage());
} }
} }
@ -88,11 +89,12 @@ public function getCommunityId(string $communityName, string $token): int
$request = new LemmyRequest($this->instance, $token); $request = new LemmyRequest($this->instance, $token);
$response = $request->get('community', ['name' => $communityName]); $response = $request->get('community', ['name' => $communityName]);
if (!$response->successful()) { if (! $response->successful()) {
throw new Exception('Failed to fetch community: ' . $response->status()); throw new Exception('Failed to fetch community: '.$response->status());
} }
$data = $response->json(); $data = $response->json();
return $data['community_view']['community']['id'] ?? throw new Exception('Community not found'); return $data['community_view']['community']['id'] ?? throw new Exception('Community not found');
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Community lookup failed', ['error' => $e->getMessage()]); logger()->error('Community lookup failed', ['error' => $e->getMessage()]);
@ -107,14 +109,15 @@ public function syncChannelPosts(string $token, int $platformChannelId, string $
$response = $request->get('post/list', [ $response = $request->get('post/list', [
'community_id' => $platformChannelId, 'community_id' => $platformChannelId,
'limit' => 50, 'limit' => 50,
'sort' => 'New' 'sort' => 'New',
]); ]);
if (!$response->successful()) { if (! $response->successful()) {
logger()->warning('Failed to sync channel posts', [ logger()->warning('Failed to sync channel posts', [
'status' => $response->status(), 'status' => $response->status(),
'platform_channel_id' => $platformChannelId 'platform_channel_id' => $platformChannelId,
]); ]);
return; return;
} }
@ -137,13 +140,13 @@ public function syncChannelPosts(string $token, int $platformChannelId, string $
logger()->info('Synced channel posts', [ logger()->info('Synced channel posts', [
'platform_channel_id' => $platformChannelId, 'platform_channel_id' => $platformChannelId,
'posts_count' => count($posts) 'posts_count' => count($posts),
]); ]);
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Exception while syncing channel posts', [ logger()->error('Exception while syncing channel posts', [
'error' => $e->getMessage(), 'error' => $e->getMessage(),
'platform_channel_id' => $platformChannelId 'platform_channel_id' => $platformChannelId,
]); ]);
} }
} }
@ -176,8 +179,8 @@ public function createPost(string $token, string $title, string $body, int $plat
$response = $request->post('post', $postData); $response = $request->post('post', $postData);
if (!$response->successful()) { if (! $response->successful()) {
throw new Exception('Failed to create post: ' . $response->status() . ' - ' . $response->body()); throw new Exception('Failed to create post: '.$response->status().' - '.$response->body());
} }
return $response->json(); return $response->json();
@ -196,19 +199,22 @@ public function getLanguages(): array
$request = new LemmyRequest($this->instance); $request = new LemmyRequest($this->instance);
$response = $request->get('site'); $response = $request->get('site');
if (!$response->successful()) { if (! $response->successful()) {
logger()->warning('Failed to fetch site languages', [ logger()->warning('Failed to fetch site languages', [
'status' => $response->status() 'status' => $response->status(),
]); ]);
return []; return [];
} }
$data = $response->json(); $data = $response->json();
return $data['all_languages'] ?? []; return $data['all_languages'] ?? [];
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Exception while fetching languages', [ logger()->error('Exception while fetching languages', [
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
return []; return [];
} }
} }

View file

@ -12,6 +12,7 @@
class LemmyPublisher class LemmyPublisher
{ {
private LemmyApiService $api; private LemmyApiService $api;
private PlatformAccount $account; private PlatformAccount $account;
public function __construct(PlatformAccount $account) public function __construct(PlatformAccount $account)
@ -21,8 +22,9 @@ public function __construct(PlatformAccount $account)
} }
/** /**
* @param array<string, mixed> $extractedData * @param array<string, mixed> $extractedData
* @return array<string, mixed> * @return array<string, mixed>
*
* @throws PlatformAuthException * @throws PlatformAuthException
* @throws Exception * @throws Exception
*/ */
@ -37,6 +39,7 @@ public function publishToChannel(Article $article, array $extractedData, Platfor
// If the cached token was stale, refresh and retry once // If the cached token was stale, refresh and retry once
if (str_contains($e->getMessage(), 'not_logged_in') || str_contains($e->getMessage(), 'Unauthorized')) { if (str_contains($e->getMessage(), 'not_logged_in') || str_contains($e->getMessage(), 'Unauthorized')) {
$token = $authService->refreshToken($this->account); $token = $authService->refreshToken($this->account);
return $this->createPost($token, $extractedData, $channel, $article); return $this->createPost($token, $extractedData, $channel, $article);
} }
throw $e; throw $e;
@ -44,7 +47,7 @@ public function publishToChannel(Article $article, array $extractedData, Platfor
} }
/** /**
* @param array<string, mixed> $extractedData * @param array<string, mixed> $extractedData
* @return array<string, mixed> * @return array<string, mixed>
*/ */
private function createPost(string $token, array $extractedData, PlatformChannel $channel, Article $article): array private function createPost(string $token, array $extractedData, PlatformChannel $channel, Article $article): array
@ -65,5 +68,4 @@ private function createPost(string $token, array $extractedData, PlatformChannel
$languageId $languageId
); );
} }
} }

View file

@ -3,8 +3,14 @@
namespace App\Providers; namespace App\Providers;
use App\Enums\LogLevelEnum; use App\Enums\LogLevelEnum;
use App\Events\ActionPerformed;
use App\Events\ExceptionOccurred; use App\Events\ExceptionOccurred;
use App\Events\NewArticleFetched;
use App\Events\RouteArticleApproved;
use App\Listeners\LogActionListener;
use App\Listeners\LogExceptionToDatabase; use App\Listeners\LogExceptionToDatabase;
use App\Listeners\PublishApprovedArticleListener;
use App\Listeners\ValidateArticleListener;
use Error; use Error;
use Illuminate\Contracts\Debug\ExceptionHandler; use Illuminate\Contracts\Debug\ExceptionHandler;
use Illuminate\Support\Facades\Event; use Illuminate\Support\Facades\Event;
@ -14,25 +20,28 @@
class AppServiceProvider extends ServiceProvider class AppServiceProvider extends ServiceProvider
{ {
public function register(): void public function register(): void {}
{
}
public function boot(): void public function boot(): void
{ {
Event::listen(
ActionPerformed::class,
LogActionListener::class,
);
Event::listen( Event::listen(
ExceptionOccurred::class, ExceptionOccurred::class,
LogExceptionToDatabase::class, LogExceptionToDatabase::class,
); );
Event::listen( Event::listen(
\App\Events\NewArticleFetched::class, NewArticleFetched::class,
\App\Listeners\ValidateArticleListener::class, ValidateArticleListener::class,
); );
Event::listen( Event::listen(
\App\Events\ArticleApproved::class, RouteArticleApproved::class,
\App\Listeners\PublishApprovedArticleListener::class, PublishApprovedArticleListener::class,
); );
app()->make(ExceptionHandler::class) app()->make(ExceptionHandler::class)

View file

@ -4,9 +4,9 @@
use App\Models\Article; use App\Models\Article;
use App\Models\Feed; use App\Models\Feed;
use App\Services\Http\HttpFetcher;
use App\Services\Factories\ArticleParserFactory; use App\Services\Factories\ArticleParserFactory;
use App\Services\Factories\HomepageParserFactory; use App\Services\Factories\HomepageParserFactory;
use App\Services\Http\HttpFetcher;
use App\Services\Log\LogSaver; use App\Services\Log\LogSaver;
use Exception; use Exception;
use Illuminate\Support\Collection; use Illuminate\Support\Collection;
@ -28,9 +28,9 @@ public function getArticlesFromFeed(Feed $feed): Collection
return $this->getArticlesFromWebsiteFeed($feed); return $this->getArticlesFromWebsiteFeed($feed);
} }
$this->logSaver->warning("Unsupported feed type", null, [ $this->logSaver->warning('Unsupported feed type', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_type' => $feed->type 'feed_type' => $feed->type,
]); ]);
return collect(); return collect();
@ -53,8 +53,8 @@ private function getArticlesFromRssFeed(Feed $feed): Collection
libxml_use_internal_errors($previousUseErrors); libxml_use_internal_errors($previousUseErrors);
} }
if ($rss === false || !isset($rss->channel->item)) { if ($rss === false || ! isset($rss->channel->item)) {
$this->logSaver->warning("Failed to parse RSS feed XML", null, [ $this->logSaver->warning('Failed to parse RSS feed XML', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_url' => $feed->url, 'feed_url' => $feed->url,
]); ]);
@ -72,7 +72,7 @@ private function getArticlesFromRssFeed(Feed $feed): Collection
return $articles; return $articles;
} catch (Exception $e) { } catch (Exception $e) {
$this->logSaver->error("Failed to fetch articles from RSS feed", null, [ $this->logSaver->error('Failed to fetch articles from RSS feed', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_url' => $feed->url, 'feed_url' => $feed->url,
'error' => $e->getMessage(), 'error' => $e->getMessage(),
@ -92,9 +92,9 @@ private function getArticlesFromWebsiteFeed(Feed $feed): Collection
$parser = HomepageParserFactory::getParserForFeed($feed); $parser = HomepageParserFactory::getParserForFeed($feed);
if (! $parser) { if (! $parser) {
$this->logSaver->warning("No parser available for feed URL", null, [ $this->logSaver->warning('No parser available for feed URL', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_url' => $feed->url 'feed_url' => $feed->url,
]); ]);
return collect(); return collect();
@ -107,10 +107,10 @@ private function getArticlesFromWebsiteFeed(Feed $feed): Collection
->map(fn (string $url) => $this->saveArticle($url, $feed->id)); ->map(fn (string $url) => $this->saveArticle($url, $feed->id));
} catch (Exception $e) { } catch (Exception $e) {
$this->logSaver->error("Failed to fetch articles from website feed", null, [ $this->logSaver->error('Failed to fetch articles from website feed', null, [
'feed_id' => $feed->id, 'feed_id' => $feed->id,
'feed_url' => $feed->url, 'feed_url' => $feed->url,
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
return collect(); return collect();
@ -130,7 +130,7 @@ public function fetchArticleData(Article $article): array
} catch (Exception $e) { } catch (Exception $e) {
$this->logSaver->error('Exception while fetching article data', null, [ $this->logSaver->error('Exception while fetching article data', null, [
'url' => $article->url, 'url' => $article->url,
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
return []; return [];
@ -155,8 +155,8 @@ private function saveArticle(string $url, ?int $feedId = null): Article
} }
return $article; return $article;
} catch (\Exception $e) { } catch (Exception $e) {
$this->logSaver->error("Failed to create article", null, [ $this->logSaver->error('Failed to create article', null, [
'url' => $url, 'url' => $url,
'feed_id' => $feedId, 'feed_id' => $feedId,
'error' => $e->getMessage(), 'error' => $e->getMessage(),

View file

@ -2,7 +2,13 @@
namespace App\Services\Article; namespace App\Services\Article;
use App\Enums\ApprovalStatusEnum;
use App\Models\Article; use App\Models\Article;
use App\Models\Keyword;
use App\Models\Route;
use App\Models\RouteArticle;
use App\Models\Setting;
use Illuminate\Support\Collection;
class ValidationService class ValidationService
{ {
@ -12,70 +18,99 @@ public function __construct(
public function validate(Article $article): Article public function validate(Article $article): Article
{ {
logger('Checking keywords for article: ' . $article->id); logger('Validating article for routes: '.$article->id);
$articleData = $this->articleFetcher->fetchArticleData($article); $articleData = $this->articleFetcher->fetchArticleData($article);
// Update article with fetched metadata (title, description)
$updateData = []; $updateData = [];
if (!empty($articleData)) { if (! empty($articleData)) {
$updateData['title'] = $articleData['title'] ?? $article->title; $updateData['title'] = $articleData['title'] ?? $article->title;
$updateData['description'] = $articleData['description'] ?? $article->description; $updateData['description'] = $articleData['description'] ?? $article->description;
$updateData['content'] = $articleData['full_article'] ?? null; $updateData['content'] = $articleData['full_article'] ?? null;
} }
if (!isset($articleData['full_article']) || empty($articleData['full_article'])) { if (! isset($articleData['full_article']) || empty($articleData['full_article'])) {
logger()->warning('Article data missing full_article content', [ logger()->warning('Article data missing full_article content', [
'article_id' => $article->id, 'article_id' => $article->id,
'url' => $article->url 'url' => $article->url,
]); ]);
$updateData['approval_status'] = 'rejected'; $updateData['validated_at'] = now();
$article->update($updateData); $article->update($updateData);
return $article->refresh(); return $article->refresh();
} }
// Validate content against keywords. If validation fails, reject.
// If validation passes, leave approval_status as-is (pending) —
// the listener decides whether to auto-approve based on settings.
$validationResult = $this->validateByKeywords($articleData['full_article']);
if (! $validationResult) {
$updateData['approval_status'] = 'rejected';
}
$updateData['validated_at'] = now(); $updateData['validated_at'] = now();
$article->update($updateData); $article->update($updateData);
$this->createRouteArticles($article, $articleData['full_article']);
return $article->refresh(); return $article->refresh();
} }
private function validateByKeywords(string $full_article): bool private function createRouteArticles(Article $article, string $content): void
{ {
// Belgian news content keywords - broader set for Belgian news relevance $activeRoutes = Route::where('feed_id', $article->feed_id)
$keywords = [ ->where('is_active', true)
// Political parties and leaders ->get();
'N-VA', 'Bart De Wever', 'Frank Vandenbroucke', 'Alexander De Croo',
'Vooruit', 'Open Vld', 'CD&V', 'Vlaams Belang', 'PTB', 'PVDA',
// Belgian locations and institutions // Batch-load all active keywords for this feed, grouped by channel
'Belgium', 'Belgian', 'Flanders', 'Flemish', 'Wallonia', 'Brussels', $keywordsByChannel = Keyword::where('feed_id', $article->feed_id)
'Antwerp', 'Ghent', 'Bruges', 'Leuven', 'Mechelen', 'Namur', 'Liège', 'Charleroi', ->where('is_active', true)
'parliament', 'government', 'minister', 'policy', 'law', 'legislation', ->get()
->groupBy('platform_channel_id');
// Common Belgian news topics // Match keywords against full article content, title, and description
'economy', 'economic', 'education', 'healthcare', 'transport', 'climate', 'energy', $searchableContent = $content.' '.$article->title.' '.$article->description;
'European', 'EU', 'migration', 'security', 'justice', 'culture', 'police'
]; foreach ($activeRoutes as $route) {
$routeKeywords = $keywordsByChannel->get($route->platform_channel_id, collect());
$status = $this->evaluateKeywords($routeKeywords, $searchableContent);
if ($status === ApprovalStatusEnum::PENDING && $this->shouldAutoApprove($route)) {
$status = ApprovalStatusEnum::APPROVED;
}
RouteArticle::firstOrCreate(
[
'feed_id' => $route->feed_id,
'platform_channel_id' => $route->platform_channel_id,
'article_id' => $article->id,
],
[
'approval_status' => $status,
'validated_at' => now(),
]
);
}
}
/**
* @param Collection<int, Keyword> $keywords
*/
private function evaluateKeywords(Collection $keywords, string $content): ApprovalStatusEnum
{
if ($keywords->isEmpty()) {
return ApprovalStatusEnum::PENDING;
}
foreach ($keywords as $keyword) { foreach ($keywords as $keyword) {
if (stripos($full_article, $keyword) !== false) { if (stripos($content, $keyword->keyword) !== false) {
return true; return ApprovalStatusEnum::PENDING;
} }
} }
return false; return ApprovalStatusEnum::REJECTED;
}
private function shouldAutoApprove(Route $route): bool
{
if ($route->auto_approve !== null) {
return $route->auto_approve;
}
return ! Setting::isPublishingApprovalsEnabled();
} }
} }

View file

@ -32,14 +32,14 @@ public function getToken(PlatformAccount $account): string
public function refreshToken(PlatformAccount $account): string public function refreshToken(PlatformAccount $account): string
{ {
if (! $account->username || ! $account->password || ! $account->instance_url) { if (! $account->username || ! $account->password || ! $account->instance_url) {
throw new PlatformAuthException(PlatformEnum::LEMMY, 'Missing credentials for account: ' . $account->username); throw new PlatformAuthException(PlatformEnum::LEMMY, 'Missing credentials for account: '.$account->username);
} }
$api = new LemmyApiService($account->instance_url); $api = new LemmyApiService($account->instance_url);
$token = $api->login($account->username, $account->password); $token = $api->login($account->username, $account->password);
if (!$token) { if (! $token) {
throw new PlatformAuthException(PlatformEnum::LEMMY, 'Login failed for account: ' . $account->username); throw new PlatformAuthException(PlatformEnum::LEMMY, 'Login failed for account: '.$account->username);
} }
// Cache the token for future use // Cache the token for future use
@ -52,15 +52,19 @@ public function refreshToken(PlatformAccount $account): string
/** /**
* Authenticate with Lemmy API and return user data with JWT * Authenticate with Lemmy API and return user data with JWT
*
* @throws PlatformAuthException * @throws PlatformAuthException
*/ */
/**
* @return array<string, mixed>
*/
public function authenticate(string $instanceUrl, string $username, string $password): array public function authenticate(string $instanceUrl, string $username, string $password): array
{ {
try { try {
$api = new LemmyApiService($instanceUrl); $api = new LemmyApiService($instanceUrl);
$token = $api->login($username, $password); $token = $api->login($username, $password);
if (!$token) { if (! $token) {
// Throw a clean exception that will be caught and handled by the controller // Throw a clean exception that will be caught and handled by the controller
throw new PlatformAuthException(PlatformEnum::LEMMY, 'Invalid credentials'); throw new PlatformAuthException(PlatformEnum::LEMMY, 'Invalid credentials');
} }
@ -75,8 +79,8 @@ public function authenticate(string $instanceUrl, string $username, string $pass
'id' => 0, // Would need API call to get actual user info 'id' => 0, // Would need API call to get actual user info
'display_name' => null, 'display_name' => null,
'bio' => null, 'bio' => null,
] ],
] ],
]; ];
} catch (PlatformAuthException $e) { } catch (PlatformAuthException $e) {
// Re-throw PlatformAuthExceptions as-is to avoid nesting // Re-throw PlatformAuthExceptions as-is to avoid nesting

View file

@ -9,10 +9,12 @@
use App\Models\PlatformChannel; use App\Models\PlatformChannel;
use App\Models\Route; use App\Models\Route;
use Carbon\Carbon; use Carbon\Carbon;
use Illuminate\Support\Facades\DB;
class DashboardStatsService class DashboardStatsService
{ {
/**
* @return array<string, mixed>
*/
public function getStats(string $period = 'today'): array public function getStats(string $period = 'today'): array
{ {
$dateRange = $this->getDateRange($period); $dateRange = $this->getDateRange($period);
@ -73,6 +75,9 @@ private function getDateRange(string $period): ?array
}; };
} }
/**
* @return array<string, int>
*/
public function getSystemStats(): array public function getSystemStats(): array
{ {
$totalFeeds = Feed::query()->count(); $totalFeeds = Feed::query()->count();

View file

@ -4,9 +4,9 @@
use App\Contracts\ArticleParserInterface; use App\Contracts\ArticleParserInterface;
use App\Models\Feed; use App\Models\Feed;
use App\Services\Parsers\VrtArticleParser;
use App\Services\Parsers\BelgaArticleParser; use App\Services\Parsers\BelgaArticleParser;
use App\Services\Parsers\GuardianArticleParser; use App\Services\Parsers\GuardianArticleParser;
use App\Services\Parsers\VrtArticleParser;
use Exception; use Exception;
class ArticleParserFactory class ArticleParserFactory
@ -26,7 +26,7 @@ class ArticleParserFactory
public static function getParser(string $url): ArticleParserInterface public static function getParser(string $url): ArticleParserInterface
{ {
foreach (self::$parsers as $parserClass) { foreach (self::$parsers as $parserClass) {
$parser = new $parserClass(); $parser = new $parserClass;
if ($parser->canParse($url)) { if ($parser->canParse($url)) {
return $parser; return $parser;
@ -38,21 +38,22 @@ public static function getParser(string $url): ArticleParserInterface
public static function getParserForFeed(Feed $feed, string $parserType = 'article'): ?ArticleParserInterface public static function getParserForFeed(Feed $feed, string $parserType = 'article'): ?ArticleParserInterface
{ {
if (!$feed->provider) { if (! $feed->provider) {
return null; return null;
} }
$providerConfig = config("feed.providers.{$feed->provider}"); $providerConfig = config("feed.providers.{$feed->provider}");
if (!$providerConfig || !isset($providerConfig['parsers'][$parserType])) { if (! $providerConfig || ! isset($providerConfig['parsers'][$parserType])) {
return null; return null;
} }
/** @var class-string<ArticleParserInterface> $parserClass */
$parserClass = $providerConfig['parsers'][$parserType]; $parserClass = $providerConfig['parsers'][$parserType];
if (!class_exists($parserClass)) { if (! class_exists($parserClass)) {
return null; return null;
} }
return new $parserClass(); return new $parserClass;
} }
/** /**
@ -60,18 +61,19 @@ public static function getParserForFeed(Feed $feed, string $parserType = 'articl
*/ */
public static function getSupportedSources(): array public static function getSupportedSources(): array
{ {
return array_map(function($parserClass) { return array_map(function ($parserClass) {
$parser = new $parserClass(); $parser = new $parserClass;
return $parser->getSourceName(); return $parser->getSourceName();
}, self::$parsers); }, self::$parsers);
} }
/** /**
* @param class-string<ArticleParserInterface> $parserClass * @param class-string<ArticleParserInterface> $parserClass
*/ */
public static function registerParser(string $parserClass): void public static function registerParser(string $parserClass): void
{ {
if (!in_array($parserClass, self::$parsers)) { if (! in_array($parserClass, self::$parsers)) {
self::$parsers[] = $parserClass; self::$parsers[] = $parserClass;
} }
} }

View file

@ -9,21 +9,22 @@ class HomepageParserFactory
{ {
public static function getParserForFeed(Feed $feed): ?HomepageParserInterface public static function getParserForFeed(Feed $feed): ?HomepageParserInterface
{ {
if (!$feed->provider) { if (! $feed->provider) {
return null; return null;
} }
$providerConfig = config("feed.providers.{$feed->provider}"); $providerConfig = config("feed.providers.{$feed->provider}");
if (!$providerConfig || !isset($providerConfig['parsers']['homepage'])) { if (! $providerConfig || ! isset($providerConfig['parsers']['homepage'])) {
return null; return null;
} }
/** @var class-string<HomepageParserInterface> $parserClass */
$parserClass = $providerConfig['parsers']['homepage']; $parserClass = $providerConfig['parsers']['homepage'];
if (!class_exists($parserClass)) { if (! class_exists($parserClass)) {
return null; return null;
} }
$language = $feed->language?->short_code ?? 'en'; $language = $feed->language->short_code ?? 'en';
return new $parserClass($language); return new $parserClass($language);
} }

View file

@ -2,8 +2,9 @@
namespace App\Services\Http; namespace App\Services\Http;
use Illuminate\Support\Facades\Http;
use Exception; use Exception;
use Illuminate\Http\Client\Response;
use Illuminate\Support\Facades\Http;
class HttpFetcher class HttpFetcher
{ {
@ -15,7 +16,7 @@ public static function fetchHtml(string $url): string
try { try {
$response = Http::get($url); $response = Http::get($url);
if (!$response->successful()) { if (! $response->successful()) {
throw new Exception("Failed to fetch URL: {$url} - Status: {$response->status()}"); throw new Exception("Failed to fetch URL: {$url} - Status: {$response->status()}");
} }
@ -23,7 +24,7 @@ public static function fetchHtml(string $url): string
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('HTTP fetch failed', [ logger()->error('HTTP fetch failed', [
'url' => $url, 'url' => $url,
'error' => $e->getMessage() 'error' => $e->getMessage(),
]); ]);
throw $e; throw $e;
@ -31,7 +32,7 @@ public static function fetchHtml(string $url): string
} }
/** /**
* @param array<int, string> $urls * @param array<int, string> $urls
* @return array<int, array<string, mixed>> * @return array<int, array<string, mixed>>
*/ */
public static function fetchMultipleUrls(array $urls): array public static function fetchMultipleUrls(array $urls): array
@ -44,24 +45,24 @@ public static function fetchMultipleUrls(array $urls): array
}); });
return collect($responses) return collect($responses)
->filter(fn($response, $index) => isset($urls[$index])) ->filter(fn ($response, $index) => isset($urls[$index]))
->reject(fn($response, $index) => $response instanceof Exception) ->reject(fn ($response, $index) => $response instanceof Exception)
->map(function ($response, $index) use ($urls) { ->map(function ($response, $index) use ($urls) {
$url = $urls[$index]; $url = $urls[$index];
/** @var Response $response */
try { try {
if ($response->successful()) { if ($response->successful()) {
return [ return [
'url' => $url, 'url' => $url,
'html' => $response->body(), 'html' => $response->body(),
'success' => true 'success' => true,
]; ];
} else { } else {
return [ return [
'url' => $url, 'url' => $url,
'html' => null, 'html' => null,
'success' => false, 'success' => false,
'status' => $response->status() 'status' => $response->status(),
]; ];
} }
} catch (Exception) { } catch (Exception) {
@ -69,11 +70,10 @@ public static function fetchMultipleUrls(array $urls): array
'url' => $url, 'url' => $url,
'html' => null, 'html' => null,
'success' => false, 'success' => false,
'error' => 'Exception occurred' 'error' => 'Exception occurred',
]; ];
} }
}) })
->filter(fn($result) => $result !== null)
->toArray(); ->toArray();
} catch (Exception $e) { } catch (Exception $e) {
logger()->error('Multiple URL fetch failed', ['error' => $e->getMessage()]); logger()->error('Multiple URL fetch failed', ['error' => $e->getMessage()]);

View file

@ -9,7 +9,7 @@
class LogSaver class LogSaver
{ {
/** /**
* @param array<string, mixed> $context * @param array<string, mixed> $context
*/ */
public function info(string $message, ?PlatformChannel $channel = null, array $context = []): void public function info(string $message, ?PlatformChannel $channel = null, array $context = []): void
{ {
@ -17,7 +17,7 @@ public function info(string $message, ?PlatformChannel $channel = null, array $c
} }
/** /**
* @param array<string, mixed> $context * @param array<string, mixed> $context
*/ */
public function error(string $message, ?PlatformChannel $channel = null, array $context = []): void public function error(string $message, ?PlatformChannel $channel = null, array $context = []): void
{ {
@ -25,7 +25,7 @@ public function error(string $message, ?PlatformChannel $channel = null, array $
} }
/** /**
* @param array<string, mixed> $context * @param array<string, mixed> $context
*/ */
public function warning(string $message, ?PlatformChannel $channel = null, array $context = []): void public function warning(string $message, ?PlatformChannel $channel = null, array $context = []): void
{ {
@ -33,7 +33,7 @@ public function warning(string $message, ?PlatformChannel $channel = null, array
} }
/** /**
* @param array<string, mixed> $context * @param array<string, mixed> $context
*/ */
public function debug(string $message, ?PlatformChannel $channel = null, array $context = []): void public function debug(string $message, ?PlatformChannel $channel = null, array $context = []): void
{ {
@ -41,9 +41,9 @@ public function debug(string $message, ?PlatformChannel $channel = null, array $
} }
/** /**
* @param array<string, mixed> $context * @param array<string, mixed> $context
*/ */
private function log(LogLevelEnum $level, string $message, ?PlatformChannel $channel = null, array $context = []): void public function log(LogLevelEnum $level, string $message, ?PlatformChannel $channel = null, array $context = []): void
{ {
$logContext = $context; $logContext = $context;

View file

@ -0,0 +1,69 @@
<?php
namespace App\Services\Notification;
use App\Enums\NotificationSeverityEnum;
use App\Enums\NotificationTypeEnum;
use App\Models\Notification;
use Illuminate\Database\Eloquent\Model;
class NotificationService
{
/**
* @param array<string, mixed> $data
*/
public function send(
NotificationTypeEnum $type,
NotificationSeverityEnum $severity,
string $title,
string $message,
?Model $notifiable = null,
array $data = [],
): Notification {
return Notification::create([
'type' => $type,
'severity' => $severity,
'title' => $title,
'message' => $message,
'data' => $data ?: null,
'notifiable_type' => $notifiable?->getMorphClass(),
'notifiable_id' => $notifiable?->getKey(),
]);
}
/**
* @param array<string, mixed> $data
*/
public function info(
string $title,
string $message,
?Model $notifiable = null,
array $data = [],
): Notification {
return $this->send(NotificationTypeEnum::GENERAL, NotificationSeverityEnum::INFO, $title, $message, $notifiable, $data);
}
/**
* @param array<string, mixed> $data
*/
public function warning(
string $title,
string $message,
?Model $notifiable = null,
array $data = [],
): Notification {
return $this->send(NotificationTypeEnum::GENERAL, NotificationSeverityEnum::WARNING, $title, $message, $notifiable, $data);
}
/**
* @param array<string, mixed> $data
*/
public function error(
string $title,
string $message,
?Model $notifiable = null,
array $data = [],
): Notification {
return $this->send(NotificationTypeEnum::GENERAL, NotificationSeverityEnum::ERROR, $title, $message, $notifiable, $data);
}
}

View file

@ -41,6 +41,6 @@ private function checkOnboardingStatus(): bool
$hasAllComponents = $hasPlatformAccount && $hasFeed && $hasChannel && $hasRoute; $hasAllComponents = $hasPlatformAccount && $hasFeed && $hasChannel && $hasRoute;
return !$hasAllComponents; return ! $hasAllComponents;
} }
} }

Some files were not shown because too many files have changed in this diff Show more