refactor: ignore markdown on metadata

app/Models/Article.php

@@ -13,9 +13,11 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsToMany;
 use Illuminate\Database\Eloquent\SoftDeletes;
+use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Str;
 use Spatie\Image\Manipulations;
+use Spatie\LaravelMarkdown\MarkdownRenderer;
 use Spatie\MediaLibrary\HasMedia;
 use Spatie\MediaLibrary\InteractsWithMedia;
 use Spatie\Sluggable\HasSlug;
@@ -70,6 +72,11 @@ public function registerMediaCollections(): void
             });
     }
 
+    public function renderedMarkdown(): string
+    {
+        return app(MarkdownRenderer::class)->toHtml($this->content);
+    }
+
     /**
      * @return BelongsToMany<Collection>
      */
@@ -148,7 +155,15 @@ public function scopeWithFeaturedCollections(Builder $query): Builder
 
     public function metaDescription(): string
     {
-        return $this->meta_description ?? Str::limit(strip_tags($this->content), 157);
+        return Cache::rememberForever("article:{$this->id}:meta_description", function () {
+            if (boolval($this->meta_description)) {
+                return $this->meta_description;
+            }
+
+            $rawContent = $this->renderedMarkdown();
+
+            return Str::limit(trim(preg_replace("/\r?\n/", ' ', html_entity_decode(strip_tags($rawContent)))), 157);
+        });
     }
 
     public function isNotPublished(): bool

app/Observers/ArticleObserver.php

@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Observers;
+
+use App\Models\Article;
+use Illuminate\Support\Facades\Cache;
+
+class ArticleObserver
+{
+    public function created(Article $article): void
+    {
+        $this->forgeMetaDescriptionCache($article);
+    }
+
+    public function updated(Article $article): void
+    {
+        $this->forgeMetaDescriptionCache($article);
+    }
+
+    private function forgeMetaDescriptionCache(Article $article): void
+    {
+        if ($article->isNotPublished()) {
+            return;
+        }
+
+        if (! ($article->wasRecentlyCreated || $article->wasChanged(['meta_description', 'content', 'is_published']))) {
+            return;
+        }
+
+        Cache::forget("article:{$article->id}:meta_description");
+    }
+}

app/Providers/EventServiceProvider.php

@@ -6,9 +6,11 @@
 
 use App\Events\WalletBecameActive;
 use App\Listeners\WalletBecameActiveListener;
+use App\Models\Article;
 use App\Models\CoingeckoToken;
 use App\Models\SpamToken;
 use App\Models\Token;
+use App\Observers\ArticleObserver;
 use App\Observers\CoingeckoTokenObserver;
 use App\Observers\SpamTokenObserver;
 use App\Observers\TokenObserver;
@@ -39,6 +41,7 @@ public function boot()
         Token::observe(TokenObserver::class);
         CoingeckoToken::observe(CoingeckoTokenObserver::class);
         SpamToken::observe(SpamTokenObserver::class);
+        Article::observe(ArticleObserver::class);
     }
 
     /**

composer.json

@@ -30,6 +30,7 @@
         "simplito/elliptic-php": "^1.0",
         "spatie/browsershot": "^3.59",
         "spatie/laravel-data": "^3.9",
+        "spatie/laravel-markdown": "^2.4",
         "spatie/laravel-medialibrary": "^10.13",
         "spatie/laravel-permission": "^5.11",
         "spatie/laravel-schemaless-attributes": "^2.4",

tests/App/Models/ArticleTest.php

@@ -4,6 +4,7 @@
 
 use App\Models\Article;
 use App\Models\Collection;
+use Illuminate\Support\Facades\Cache;
 use Illuminate\Support\Facades\DB;
 
 it('should create an article', function () {
@@ -49,6 +50,25 @@
     expect($article->metaDescription())->toBe('This is the content');
 });
 
+it('removes markdown in meta description', function () {
+    $article = Article::factory()->create([
+        'meta_description' => null,
+        'content' => "### This is the content\n\nwith some *markdown*\n",
+    ]);
+
+    expect($article->metaDescription())->toBe('This is the content with some markdown');
+});
+
+it('gets meta description from cache', function () {
+    $article = Article::factory()->create();
+
+    Cache::shouldReceive('rememberForever')
+        ->with('article:'.$article->id.':meta_description', Closure::class)
+        ->andReturn('This is the content');
+
+    expect($article->metaDescription())->toBe('This is the content');
+});
+
 it('truncates the content if used as meta description', function () {
     $article = Article::factory()->create([
         'meta_description' => null,