fix: only treat an explicitly configured url as a card link

getUrl() falls back to the livewire component's getDefaultActionUrl(), which
Filament's resource pages implement for modal-less Create/Edit/View actions.
BoardResourcePage extends that class, so honouring the fallback would have turned
existing boards into links without the developer asking for it. Gate on hasUrl()
so only an explicit ->url() opts a card into link rendering.

Author: ManukMinasyan

resources/views/livewire/card.blade.php

@@ -9,10 +9,17 @@
 
     // A card action configured with ->url() must navigate like a native link instead
     // of mounting a modal, so middle-click, cmd-click and "copy link" keep working.
-    // getUrl() returns null when the action has a modal, and POST urls need a form
+    //
+    // Only an explicitly configured url opts a card into link rendering. getUrl()
+    // also falls back to the livewire component's getDefaultActionUrl(), which
+    // Filament's resource pages implement for modal-less Create/Edit/View actions;
+    // honouring that here would silently turn existing boards into links. getUrl()
+    // still returns null when the action has a modal, and POST urls need a form
     // rather than an anchor, so both fall through to the Livewire click handler.
     $cardActionInstance = $this->getBoard()->resolveCardAction($processedRecordActions);
-    $cardActionUrl = $cardActionInstance?->shouldPostToUrl() ? null : $cardActionInstance?->getUrl();
+    $cardActionUrl = ($cardActionInstance?->hasUrl() && ! $cardActionInstance->shouldPostToUrl())
+        ? $cardActionInstance->getUrl()
+        : null;
     $hasCardActionUrl = filled($cardActionUrl);
     $cardActionHref = $hasCardActionUrl
         ? \Filament\Support\generate_href_html(

tests/Feature/CardActionUrlTest.php

@@ -6,6 +6,7 @@
 use Relaticle\Flowforge\Tests\Fixtures\Task;
 use Relaticle\Flowforge\Tests\Fixtures\TestBoard;
 use Relaticle\Flowforge\Tests\Fixtures\TestCardActionBoard;
+use Relaticle\Flowforge\Tests\Fixtures\TestCardActionDefaultUrlBoard;
 use Relaticle\Flowforge\Tests\Fixtures\TestCardActionModalBoard;
 use Relaticle\Flowforge\Tests\Fixtures\TestCardActionNewTabBoard;
 
@@ -60,6 +61,13 @@ function cardTitleHtml(string $html): string
         ->and($title)->not->toContain('href=');
 });
 
+test('card action inheriting only a default action url still mounts the action', function () {
+    $title = cardTitleHtml(Livewire::test(TestCardActionDefaultUrlBoard::class)->html());
+
+    expect($title)->toContain("mountAction('run'")
+        ->and($title)->not->toContain('href=');
+});
+
 test('resolveCardAction finds the configured action among record actions', function () {
     $board = Livewire::test(TestCardActionBoard::class)->instance()->getBoard();
 

tests/Fixtures/TestCardActionBoard.php

@@ -38,6 +38,8 @@ public function board(Board $board): Board
                         TextInput::make('title'),
                     ])
                     ->action(function (): void {}),
+                Action::make('run')
+                    ->action(function (): void {}),
             ])
             ->cardAction($this->cardActionName());
     }

tests/Fixtures/TestCardActionDefaultUrlBoard.php

@@ -0,0 +1,26 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Relaticle\Flowforge\Tests\Fixtures;
+
+use Filament\Actions\Action;
+
+/**
+ * A card action with no ->url() of its own, on a page whose getDefaultActionUrl()
+ * returns a url. Filament's resource pages do exactly this for modal-less
+ * Create/Edit/View actions, so the card must keep mounting the action rather than
+ * silently becoming a link.
+ */
+class TestCardActionDefaultUrlBoard extends TestCardActionBoard
+{
+    public function getDefaultActionUrl(Action $action): ?string
+    {
+        return 'https://example.test/default-action-url';
+    }
+
+    protected function cardActionName(): string
+    {
+        return 'run';
+    }
+}

tests/Fixtures/TestPanelProvider.php

@@ -29,6 +29,7 @@ public function panel(Panel $panel): Panel
             ->pages([
                 TestBoard::class,
                 TestCardActionBoard::class,
+                TestCardActionDefaultUrlBoard::class,
                 TestCardActionModalBoard::class,
                 TestCardActionNewTabBoard::class,
             ])