Prevent duplicate seed data and move widgets nav

- Move Embed Widgets from Insights to Website section in nav
- Add semantic duplicate detection for links (same link_path + placement)
- Improve logging in seed_pack for links and pages (shows skipped count)
- Properties and pages already had proper duplicate protection

This prevents issues like duplicate footer links when seeds are run
multiple times or when YAML slugs change but content is the same.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: etewiah

app/views/layouts/site_admin/_navigation.html.erb

@@ -197,13 +197,6 @@
             </svg>
             <span>Analytics</span>
           <% end %>
-
-          <%= link_to site_admin_widgets_path, id: "tour-widgets", class: "flex items-center px-4 py-2 rounded-lg hover:bg-gray-700 #{request.path.start_with?('/site_admin/widgets') ? 'bg-gray-700' : ''}" do %>
-            <svg class="w-5 h-5 mr-3" fill="currentColor" viewBox="0 0 20 20">
-              <path fill-rule="evenodd" d="M12.316 3.051a1 1 0 01.633 1.265l-4 12a1 1 0 11-1.898-.632l4-12a1 1 0 011.265-.633zM5.707 6.293a1 1 0 010 1.414L3.414 10l2.293 2.293a1 1 0 11-1.414 1.414l-3-3a1 1 0 010-1.414l3-3a1 1 0 011.414 0zm8.586 0a1 1 0 011.414 0l3 3a1 1 0 010 1.414l-3 3a1 1 0 11-1.414-1.414L16.586 10l-2.293-2.293a1 1 0 010-1.414z" clip-rule="evenodd"></path>
-            </svg>
-            <span>Embed Widgets</span>
-          <% end %>
         </div>
       </div>
 
@@ -285,6 +278,13 @@
             <span>Domain</span>
           <% end %>
 
+          <%= link_to site_admin_widgets_path, id: "tour-widgets", class: "flex items-center px-4 py-2 rounded-lg hover:bg-gray-700 #{request.path.start_with?('/site_admin/widgets') ? 'bg-gray-700' : ''}" do %>
+            <svg class="w-5 h-5 mr-3" fill="currentColor" viewBox="0 0 20 20">
+              <path fill-rule="evenodd" d="M12.316 3.051a1 1 0 01.633 1.265l-4 12a1 1 0 11-1.898-.632l4-12a1 1 0 011.265-.633zM5.707 6.293a1 1 0 010 1.414L3.414 10l2.293 2.293a1 1 0 11-1.414 1.414l-3-3a1 1 0 010-1.414l3-3a1 1 0 011.414 0zm8.586 0a1 1 0 011.414 0l3 3a1 1 0 010 1.414l-3 3a1 1 0 11-1.414-1.414L16.586 10l-2.293-2.293a1 1 0 010-1.414z" clip-rule="evenodd"></path>
+            </svg>
+            <span>Embed Widgets</span>
+          <% end %>
+
           <%= link_to site_admin_onboarding_path, id: "tour-onboarding", class: "flex items-center px-4 py-2 rounded-lg hover:bg-gray-700 #{request.path.start_with?('/site_admin/onboarding') ? 'bg-gray-700' : ''}" do %>
             <svg class="w-5 h-5 mr-3" fill="currentColor" viewBox="0 0 20 20">
               <path fill-rule="evenodd" d="M6.267 3.455a3.066 3.066 0 001.745-.723 3.066 3.066 0 013.976 0 3.066 3.066 0 001.745.723 3.066 3.066 0 012.812 2.812c.051.643.304 1.254.723 1.745a3.066 3.066 0 010 3.976 3.066 3.066 0 00-.723 1.745 3.066 3.066 0 01-2.812 2.812 3.066 3.066 0 00-1.745.723 3.066 3.066 0 01-3.976 0 3.066 3.066 0 00-1.745-.723 3.066 3.066 0 01-2.812-2.812 3.066 3.066 0 00-.723-1.745 3.066 3.066 0 010-3.976 3.066 3.066 0 00.723-1.745 3.066 3.066 0 012.812-2.812zm7.44 5.252a1 1 0 00-1.414-1.414L9 10.586 7.707 9.293a1 1 0 00-1.414 1.414l2 2a1 1 0 001.414 0l4-4z" clip-rule="evenodd"></path>

lib/pwb/seed_pack.rb

@@ -378,16 +378,33 @@ def seed_links
 
       links = YAML.safe_load(File.read(links_file), symbolize_names: true) || []
       count = 0
+      skipped = 0
 
       links.each do |link_data|
+        # Check by slug first
         existing = @website.links.find_by(slug: link_data[:slug])
-        unless existing
-          @website.links.create!(link_data)
-          count += 1
+        if existing
+          skipped += 1
+          next
         end
+
+        # Also check for semantic duplicates (same link_path + placement)
+        semantic_duplicate = @website.links.find_by(
+          link_path: link_data[:link_path],
+          link_path_params: link_data[:link_path_params],
+          placement: link_data[:placement]
+        )
+        if semantic_duplicate
+          log "  Skipping '#{link_data[:slug]}' - semantic duplicate of '#{semantic_duplicate.slug}'", :detail
+          skipped += 1
+          next
+        end
+
+        @website.links.create!(link_data)
+        count += 1
       end
 
-      log "  Created #{count} links", :detail
+      log "  Created #{count} links (#{skipped} skipped)", :detail
     end
 
     def seed_pages
@@ -397,18 +414,23 @@ def seed_pages
       log "Seeding pages...", :info
 
       count = 0
+      skipped = 0
       Dir.glob(pages_dir.join('*.yml')).each do |page_file|
         page_data = YAML.safe_load(File.read(page_file), symbolize_names: true)
         next unless page_data
 
+        # Check by slug to prevent duplicates
         existing = @website.pages.find_by(slug: page_data[:slug])
-        unless existing
-          @website.pages.create!(page_data)
-          count += 1
+        if existing
+          skipped += 1
+          next
         end
+
+        @website.pages.create!(page_data)
+        count += 1
       end
 
-      log "  Created #{count} pages", :detail
+      log "  Created #{count} pages (#{skipped} skipped)", :detail
     end
 
     def seed_page_parts

lib/pwb/seeder.rb

@@ -212,10 +212,22 @@ def set_field_key_translations(field_key, translations)
       def seed_links(yml_file)
         links_yml = load_seed_yml yml_file
         links_yml.each do |single_link_yml|
-          # Check if this link already exists for this website
+          # Check if this link already exists for this website by slug
           link_record = current_website.links.find_by(slug: single_link_yml["slug"])
 
+          # Also check for semantic duplicates (same link_path + placement)
+          # This prevents creating duplicate links with different slugs
           unless link_record.present?
+            semantic_duplicate = current_website.links.find_by(
+              link_path: single_link_yml["link_path"],
+              link_path_params: single_link_yml["link_path_params"],
+              placement: single_link_yml["placement"]
+            )
+            if semantic_duplicate
+              Rails.logger.info "Skipping link '#{single_link_yml['slug']}' - semantic duplicate of '#{semantic_duplicate.slug}'"
+              next
+            end
+
             # Filter out unsupported locale attributes before creating
             filtered_attrs = filter_supported_locale_attrs(single_link_yml)
             # Create link with website association