From 680653753120daa371386648f6bc3fd55b467db4 Mon Sep 17 00:00:00 2001
From: am0 <am0@tutamail.com>
Date: Wed, 18 Feb 2026 13:18:44 +0100
Subject: [PATCH] refactor: poprawki code review w build_time_abstraction
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

## Zmiany

### sidebar.tsx
- Przywrócono niełamiące spacje (U+00A0) przed krótkimi przyimkami (https://unicode-explorer.com/c/00A0)

### esbuild_config.js - zasada DRY
- Zastąpiono powielone bloki kopiowania 'components/' i 'assets/'
  pojedynczą pętlą 'for' po tablicy ['components', 'assets']
- Zastąpiono powielone bloki kopiowania zależności z 'node_modules'
  ('react/umd', 'react-dom/umd', 'survey-react') pętlą iterującą
  po tablicy obiektów 'nodeDeps'; zależności z selektywnym kopiowaniem
  plików obsługiwane są przez opcjonalne pole 'files'

## Bez zmian funkcjonalnych
Wszystkie powyższe zmiany są wyłącznie refaktoryzacją - zachowanie buildu dla obu targetów (Firefox i Chrome) pozostaje identyczne.
---
 components/sidebar/sidebar.tsx |  6 +--
 esbuild.config.js              | 67 +++++++++++++---------------------
 2 files changed, 29 insertions(+), 44 deletions(-)

diff --git a/components/sidebar/sidebar.tsx b/components/sidebar/sidebar.tsx
index d4709ff..1526cd9 100644
--- a/components/sidebar/sidebar.tsx
+++ b/components/sidebar/sidebar.tsx
@@ -159,7 +159,7 @@ const Sidebar = () => {
                             onClick={() => {
                                 if (
                                     window.confirm(
-                                        'Czy chcesz wczytać wszystkie domeny w celu „splamienia" twojej przeglądarki? Uwaga przeglądarka może zablokować otwieranie nowych kart. (Ten krok jest opcjonalny)'
+                                        'Czy chcesz wczytać wszystkie domeny w celu „splamienia" twojej przeglądarki? Uwaga przeglądarka może zablokować otwieranie nowych kart. (Ten krok jest opcjonalny)'
                                     )
                                 ) {
                                     let deep_copy = JSON.parse(
@@ -211,8 +211,8 @@ const Sidebar = () => {
                             <section className="dialog-container dialog-container--warning">
                                 <span>
                                     <strong>Uwaga!</strong> Niekoniecznie każda przesłana poniżej
-                                    informacja jest daną osobową. Niektóre z podanych domen mogą
-                                    należeć do właściciela strony i nie reprezentować podmiotów
+                                    informacja jest daną osobową. Niektóre z podanych domen mogą
+                                    należeć do właściciela strony i nie reprezentować podmiotów
                                     trzecich.
                                 </span>
                                 <button
diff --git a/esbuild.config.js b/esbuild.config.js
index 6967c4d..984756c 100644
--- a/esbuild.config.js
+++ b/esbuild.config.js
@@ -56,53 +56,38 @@ const copyStaticFiles = {
             copyFileSync(manifestSrc, manifestDest);
             console.log(`   ✓ Skopiowano ${manifestSrc} → ${manifestDest}`);
             
-            // Kopiowanie katalogu components
-            if (existsSync('./components')) {
-                copyDir('./components', join(DIST_DIR, 'components'));
-                console.log('   ✓ Skopiowano components/');
-            }
-            
-            // Kopiowanie katalogu assets
-            if (existsSync('./assets')) {
-                copyDir('./assets', join(DIST_DIR, 'assets'));
-                console.log('   ✓ Skopiowano assets/');
+            // Kopiowanie katalogów statycznych
+            for (const dir of ['components', 'assets']) {
+                if (existsSync(`./${dir}`)) {
+                    copyDir(`./${dir}`, join(DIST_DIR, dir));
+                    console.log(`   ✓ Skopiowano ${dir}/`);
+                }
             }
             
             // Kopiowanie wymaganych bibliotek z node_modules (potrzebne dla plików HTML z UMD React)
             const nodeModulesDest = join(DIST_DIR, 'node_modules');
-            
-            // React
-            const reactUmdSrc = './node_modules/react/umd';
-            const reactUmdDest = join(nodeModulesDest, 'react/umd');
-            if (existsSync(reactUmdSrc)) {
-                copyDir(reactUmdSrc, reactUmdDest);
-                console.log('   ✓ Skopiowano node_modules/react/umd/');
-            }
-            
-            // React-DOM
-            const reactDomUmdSrc = './node_modules/react-dom/umd';
-            const reactDomUmdDest = join(nodeModulesDest, 'react-dom/umd');
-            if (existsSync(reactDomUmdSrc)) {
-                copyDir(reactDomUmdSrc, reactDomUmdDest);
-                console.log('   ✓ Skopiowano node_modules/react-dom/umd/');
-            }
-            
-            // Survey-React
-            const surveyReactSrc = './node_modules/survey-react';
-            const surveyReactDest = join(nodeModulesDest, 'survey-react');
-            if (existsSync(surveyReactSrc)) {
-                // Kopiowanie tylko niezbędnych plików
-                mkdirSync(surveyReactDest, { recursive: true });
-                const surveyFiles = ['survey.react.min.js', 'survey.react.min.css'];
-                surveyFiles.forEach(file => {
-                    const src = join(surveyReactSrc, file);
-                    if (existsSync(src)) {
-                        copyFileSync(src, join(surveyReactDest, file));
+            const nodeDeps = [
+                { src: 'react/umd',    dest: 'react/umd' },
+                { src: 'react-dom/umd', dest: 'react-dom/umd' },
+                { src: 'survey-react', dest: 'survey-react', files: ['survey.react.min.js', 'survey.react.min.css'] },
+            ];
+            for (const dep of nodeDeps) {
+                const srcPath = `./node_modules/${dep.src}`;
+                const destPath = join(nodeModulesDest, dep.dest);
+                if (existsSync(srcPath)) {
+                    if (dep.files) {
+                        mkdirSync(destPath, { recursive: true });
+                        dep.files.forEach(file => {
+                            const fileSrc = join(srcPath, file);
+                            if (existsSync(fileSrc)) copyFileSync(fileSrc, join(destPath, file));
+                        });
+                    } else {
+                        copyDir(srcPath, destPath);
                     }
-                });
-                console.log('   ✓ Skopiowano node_modules/survey-react/');
+                    console.log(`   ✓ Skopiowano node_modules/${dep.src}/`);
+                }
             }
-            
+
             console.log(`✅ Budowanie dla ${TARGET.toUpperCase()} zakończone!`);
         });
     },