feat: implement proper edit form restrictions and move functionality

- Remove type and parent_id from edit form completely
- Type is now completely immutable after creation
- Add dedicated 'Move' action for changing parent folder
- Move action available in both table actions and edit page header
- Move action prevents moving items into themselves or descendants
- Edit form now only allows changing the name field
- Clean separation of concerns: edit for name, move for location

Author: scott grayson

src/Resources/LibraryItemResource.php

@@ -11,6 +11,8 @@
 use Filament\Actions\ViewAction;
 use Filament\Actions\BulkActionGroup;
 use Filament\Actions\DeleteBulkAction;
+use Filament\Actions\Action;
+use Filament\Forms\Components\Select;
 use Tapp\FilamentLibrary\Models\LibraryItem;
 
 class LibraryItemResource extends Resource
@@ -126,11 +128,40 @@ public static function table(Table $table): Table
             ])
             ->actions([
                 ViewAction::make()
-                    ->url(fn (LibraryItem $record): string =>
-                        $record->type === 'folder'
+                    ->url(fn (LibraryItem $record): string => 
+                        $record->type === 'folder' 
                             ? static::getUrl('index', ['parent' => $record->id])
                             : static::getUrl('view', ['record' => $record])
                     ),
+                Action::make('move')
+                    ->label('Move')
+                    ->icon('heroicon-o-arrow-right-circle')
+                    ->color('warning')
+                    ->form([
+                        Select::make('parent_id')
+                            ->label('Move to folder')
+                            ->options(function (LibraryItem $record) {
+                                $currentId = $record->id;
+                                
+                                return LibraryItem::where('type', 'folder')
+                                    ->where('id', '!=', $currentId)
+                                    ->where(function ($query) use ($currentId) {
+                                        // Prevent moving into self or descendants
+                                        $query->whereNull('parent_id')
+                                              ->orWhere('parent_id', '!=', $currentId);
+                                    })
+                                    ->pluck('name', 'id')
+                                    ->prepend('Root (No parent)', null);
+                            })
+                            ->searchable()
+                            ->preload()
+                            ->default(fn (LibraryItem $record) => $record->parent_id),
+                    ])
+                    ->action(function (LibraryItem $record, array $data): void {
+                        $record->update([
+                            'parent_id' => $data['parent_id'],
+                        ]);
+                    }),
                 EditAction::make(),
                 DeleteAction::make(),
             ])

src/Resources/Pages/EditLibraryItem.php

@@ -5,6 +5,9 @@
 use Filament\Resources\Pages\EditRecord;
 use Tapp\FilamentLibrary\Resources\LibraryItemResource;
 use Filament\Actions\DeleteAction;
+use Filament\Actions\Action;
+use Filament\Forms\Components\Select;
+use Tapp\FilamentLibrary\Models\LibraryItem;
 
 class EditLibraryItem extends EditRecord
 {
@@ -13,13 +16,45 @@ class EditLibraryItem extends EditRecord
     protected function getHeaderActions(): array
     {
         return [
+            Action::make('move')
+                ->label('Move')
+                ->icon('heroicon-o-arrow-right-circle')
+                ->color('warning')
+                ->form([
+                    Select::make('parent_id')
+                        ->label('Move to folder')
+                        ->options(function () {
+                            $currentId = $this->getRecord()->id;
+                            
+                            return LibraryItem::where('type', 'folder')
+                                ->where('id', '!=', $currentId)
+                                ->where(function ($query) use ($currentId) {
+                                    // Prevent moving into self or descendants
+                                    $query->whereNull('parent_id')
+                                          ->orWhere('parent_id', '!=', $currentId);
+                                })
+                                ->pluck('name', 'id')
+                                ->prepend('Root (No parent)', null);
+                        })
+                        ->searchable()
+                        ->preload()
+                        ->default($this->getRecord()->parent_id),
+                ])
+                ->action(function (array $data): void {
+                    $this->getRecord()->update([
+                        'parent_id' => $data['parent_id'],
+                    ]);
+                    
+                    $this->redirect(static::getResource()::getUrl('index', $data['parent_id'] ? ['parent' => $data['parent_id']] : []));
+                }),
             DeleteAction::make(),
         ];
     }
 
     protected function mutateFormDataBeforeFill(array $data): array
     {
-        // Remove fields that shouldn't be editable, but preserve type
+        // Remove fields that shouldn't be editable
+        unset($data['type']);
         unset($data['parent_id']);
         unset($data['created_by']);
 
@@ -38,11 +73,4 @@ protected function getForms(): array
         ];
     }
 
-    protected function mutateFormDataBeforeSave(array $data): array
-    {
-        // Preserve the original type - don't let it be changed
-        $data['type'] = $this->getRecord()->type;
-        
-        return $data;
-    }
 }