feat: improve performance with livewire navigate and prefetching

[adiologydev]

Changes

• Add wire:navigate.hover for navigating without reloads and enables pre-fetching.

[adiologydev]

@coderabbitai can you check this out?

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get PR details
gh pr view 6957 --json title,body,files,additions,deletions

# Get the diff for the PR
gh pr diff 6957

Length of output: 134741

---

@adiologydev, I'll be back... with my review! adjusts sunglasses 🤖

I've analyzed this PR and here's what I found, compadre:

The Good (Like a T-800 on your side) 👍

Performance boost detected! The wire:navigate.hover additions are solid. This Livewire feature will prefetch pages on hover, making navigation feel snappier than my reflexes when spotting a serverless pitch. This is self-hosted performance optimization at its finest! 🌮

The implementation is consistent across:

• Navigation menus
• Breadcrumbs
• Resource links
• Error page links
• Dashboard links

The "We Need to Talk" Parts 🤔

1. Unrelated Changes Mixed In

// templates/service-templates*.json - MinIO service completely removed

This MinIO removal seems unrelated to navigation improvements. Consider separating this into its own PR for better change tracking. Unless MinIO is getting terminated? 😏

2. Massive Formatting Changes in navbar.blade.php

The navbar file has extensive indentation/whitespace changes that obscure the actual functional changes:

-                if (this.full === 'full') {
-                    localStorage.setItem('pageWidth', 'center');
+            if (this.full === 'full') {
+                localStorage.setItem('pageWidth', 'center');

This makes the diff harder to review. Consider using a code formatter consistently across the codebase, or tackle formatting in a separate PR.

3. Test File Cleanup

Unused imports removed in test files - good hygiene, but unrelated to the navigation feature.

Potential Issues 🎯

Question: Are all these links appropriate for wire:navigate.hover? Some considerations:

• External links shouldn't have this attribute
• Links with target="_blank" don't need it
• Form submission links might behave unexpectedly

I notice you have this mixed usage:

<a target="_blank" href="{{ config('constants.urls.contact') }}">Contact</a>

Good - no wire:navigate here since it's external.

Recommendation 🎬

The core feature looks good, but I'd suggest:

1. Split the PR: Separate MinIO removal and test cleanup into different PRs
2. Formatting consistency: Consider a separate formatting-only PR for the navbar.blade.php indentation changes
3. Testing: Verify that hover prefetching works well on slower connections (might want to add a slight delay to avoid excessive prefetching)

Hasta la vista, serverless! This self-hosted navigation is getting faster. Just clean up those unrelated changes and you'll be cooking with tacos! 🌮

Want me to take a deeper dive into any specific part of the changes?