Deduplikator autorów: tryb ogólny + UI overhaul

.gitignore

@@ -175,6 +175,8 @@ dump.rdb
 .worktrees/
 .claude/
 
+.grunt-build-stamp
+
 # Local TODO / audit notes (not committed)
 TODO-*.txt
 TODO-*.md

CLAUDE.md

@@ -25,6 +25,22 @@ management system built with Django. Python >=3.10,<3.15.
   - Public frontend (Foundation CSS): monochrome Foundation-Icons
     (`<span class="fi-icon"/>`)
   - Django admin (`templates/admin/`): use emoji (no Foundation Icons)
+- **Django template comments `{# ... #}` są jedno-liniowe — KAZDA LINIA
+  MUSI mieć własne otwarcie `{#` i zamknięcie `#}` na tej samej linii.**
+  Po `\n` w środku komentarza parser przestaje go widzieć i tekst wycieka
+  do wyrenderowanego HTML-u. Powtarzający się błąd. Reguła:
+  - ❌ ZABRONIONE wieloliniowe komentarze typu:
+    ```django
+    {# linia 1
+       linia 2 #}
+    ```
+  - ✅ ZAWSZE każda linia z osobnym `{# ... #}`:
+    ```django
+    {# linia 1 #}
+    {# linia 2 #}
+    ```
+  - Alternatywa dla bloków: `{% comment %}...{% endcomment %}` (też OK,
+    ale per-line `{# #}` jest preferowane przez użytkownika).
 
 ## Python and Django Execution
 

Makefile

@@ -119,13 +119,13 @@ clean-pycache: ## Usuń __pycache__, *.pyc oraz .eggs/.cache
 	rm -rf .eggs .cache
 
 clean: clean-pycache ## Szersze czyszczenie: egg-info, logi, build, dist, staticroot/CACHE, .tox
+	rm -f .grunt-build-stamp
 	find . -type d -name \*egg-info -print0 | xargs -0 rm -rf
 	find . -name \*~ -print0 | xargs -0 rm -f
 	find . -name \*.prof -print0 | xargs -0 rm -f
 	rm -rf prof/
 	find . -name \*\\.log -print0 | xargs -0 rm -f
-	find . -name \*\\.log -print0 | xargs -0 rm -f
-	find . -name \#\* -print0 | xargs -0 rm -f
+	find . -name \#\* -not -path './node_modules/*' -print0 | xargs -0 rm -rf
 	rm -rf build dist/*django_bpp*whl dist/*bpp_iplweb*whl *.log dist
 	rm -rf src/django_bpp/staticroot/CACHE
 	rm -rf .tox
@@ -149,11 +149,13 @@ distclean: clean ## Pełne czyszczenie: + node_modules, staticroot, media, dist,
 grunt-build: ## Uruchom `grunt build` (SCSS → CSS, bundling JS)
 	grunt build
 
-# CSS output files (targets)
-CSS_TARGETS := src/bpp/static/scss/app-blue.css src/bpp/static/scss/app-green.css src/bpp/static/scss/app-orange.css
+# grunt build kompiluje WSZYSTKIE SCSS → CSS za jednym odpaleniem.
+# Pattern rule $(CSS_TARGETS): $(SCSS_SOURCES) odpalałby grunt N razy
+# (raz per out-of-date target). Zamiast tego: jeden stamp file zależy od
+# wszystkich SCSS + node_modules; grunt dotyka stampu po zakończeniu.
 
-# SCSS source files
-SCSS_SOURCES := $(wildcard src/bpp/static/scss/*.scss)
+SCSS_SOURCES := $(wildcard src/bpp/static/scss/*.scss) \
+                $(wildcard src/*/static/*/scss/*.scss)
 
 # Node modules dependency
 NODE_MODULES := node_modules/.installed
@@ -166,14 +168,16 @@ $(NODE_MODULES): package.json yarn.lock
 	export PUPPETEER_SKIP_CHROME_DOWNLOAD=true PUPPETEER_SKIP_CHROME_HEADLESS_SHELL_DOWNLOAD=true && $(YARN_CMD) install  --no-progress --emoji false -s
 	touch $(NODE_MODULES)
 
-$(CSS_TARGETS): $(SCSS_SOURCES) $(NODE_MODULES)
+CSS_STAMP := .grunt-build-stamp
+
+$(CSS_STAMP): $(SCSS_SOURCES) $(NODE_MODULES)
 	grunt build
+	@touch $(CSS_STAMP)
 
 $(MO_FILES): $(PO_FILES)
-	# cd src &&  django-admin compilemessages
 	uv run python src/manage.py compilemessages --locale=pl --ignore=site-packages
 
-assets: $(CSS_TARGETS) $(MO_FILES) ## Zbuduj frontend (CSS + .mo); uruchamia `yarn install` jeśli trzeba
+assets: $(CSS_STAMP) $(MO_FILES) ## Zbuduj frontend (CSS + .mo); uruchamia `yarn install` jeśli trzeba
 
 yarn: $(NODE_MODULES) ## Zainstaluj zależności Node.js (yarn install)
 

src/bpp/newsfragments/+deduplikator-autorow-general.feature.rst

@@ -0,0 +1,6 @@
+Deduplikator autorów: nowy tryb "ogólny" znajdujący duplikaty wśród
+autorów spoza listy pracowników instytucji w PBN. Jeden przycisk
+"Skanuj duplikaty" uruchamia obie fazy (PBN + ogólna) sekwencyjnie.
+Widok pozwala filtrować wyniki radio-button-em (PBN/Ogólny/Oba),
+eksport XLSX zawiera kolumnę "Tryb". Anulowanie fazy ogólnej skutkuje
+statusem "Częściowo zakończone" — wyniki PBN pozostają dostępne.

src/bpp/newsfragments/+deduplikator-autorow-ui-overhaul.feature.rst

@@ -0,0 +1,28 @@
+Deduplikator autorów: gruntowna przebudowa UI. Tytuł i pozycje
+menu uproszczone z "Deduplikator autorów PBN" na "Deduplikator
+autorów" (bez znacznika BETA), wpis dodany dodatkowo do podmenu
+"Operacje". Tryb skanowania (PBN/ogólny) prezentowany jest jako
+kolorowy badge przy "Główny rekord autora", filtr "Pokaż wyniki"
+zmieniony z radio-buttonów na poziomy button-group.
+
+Przyciski na karcie każdego potencjalnego duplikatu pogrupowane
+w trzy logiczne sekcje: Podgląd (otwórz wyd. ciągłe/zwarte,
+redagowanie, stronę główną, PBN), Decyzja ("Nie jest duplikatem
+głównego autora", usuń autora bez publikacji), Scalanie (cztery
+warianty scalania). Przyciski "Scal + ustaw dyscyplinę" oraz
+"Scal + ustaw subdyscyplinę" są ukryte, gdy główny autor nie ma
+żadnej dyscypliny.
+
+Powody podobieństwa renderowane są jako kolorowe chipy z ikonami
+Foundation, z tonami match/info/weak/warn dobranymi do siły
+przesłanki. Procent pewności jest sklampowany do zakresu 0–100%
+(wcześniej widoczne były wartości typu 140% wynikające z surowego
+score).
+
+Naprawione: oznaczenie autora jako nie-duplikat (przycisk
+"Nie jest duplikatem głównego autora") wykonuje się teraz przez
+AJAX z fadeOut karty, zamiast przeładowywać widok i przeskakiwać
+do kolejnego głównego autora. Naprawiono też "Scal wszystkie",
+który dla kandydatów z trybu ogólnego zwracał błąd 400 (JS
+wysyłał ``main_scientist_id`` zamiast ``main_autor_id``); brakujące
+parametry trafiają teraz dodatkowo do Rollbara.

src/bpp/system.py

@@ -78,7 +78,7 @@
 from bpp.models.struktura import Jednostka_Wydzial
 from bpp.models.system import Charakter_PBN
 from bpp.models.wydawca import Poziom_Wydawcy, Wydawca
-from deduplikator_autorow.models import IgnoredAuthor, LogScalania, NotADuplicate
+from deduplikator_autorow.models import IgnoredScientist, LogScalania, NotADuplicate
 from dynamic_columns.models import ModelAdmin, ModelAdminColumn
 from ewaluacja_common.models import Rodzaj_Autora
 from ewaluacja_liczba_n.models import IloscUdzialowDlaAutoraZaRok, LiczbaNDlaUczelni
@@ -189,7 +189,7 @@
         RozbieznosciZrodelView,
         NotADuplicate,
         LogScalania,
-        IgnoredAuthor,
+        IgnoredScientist,
     ],
     "indeks autorów": [Autor, Autor_Jednostka],
     "administracja": [

src/bpp/tests/test_autocomplete/test_autocomplete_authors.py

@@ -20,7 +20,6 @@
 )
 
 
-
 def test_dyscyplina_naukowa_przypisanie_autocomplete(
     app, autor_jan_kowalski, dyscyplina1, dyscyplina2, rok
 ):
@@ -75,7 +74,6 @@ def test_dyscyplina_naukowa_przypisanie_autocomplete(
     assert res.json["results"][0]["text"] == "memetyka stosowana"
 
 
-
 def test_dyscyplina_naukowa_przypisanie_autocomplete_brak_autora(
     app,
 ):
@@ -90,7 +88,6 @@ def test_dyscyplina_naukowa_przypisanie_autocomplete_brak_autora(
     assert res.json["results"][0]["text"] == "Podaj autora"
 
 
-
 def test_dyscyplina_naukowa_przypisanie_autocomplete_brak_drugiej(
     app, autor_jan_kowalski, dyscyplina1, dyscyplina2, rok
 ):
@@ -133,6 +130,30 @@ def autocomplete(s):
     assert Autor.objects.first().imiona == "Baz Quux"
 
 
+@pytest.mark.django_db
+def test_AutorAutocomplete_create_object_creates_log_entry(rf, admin_user, db):
+    from django.contrib.admin.models import ADDITION, LogEntry
+    from django.contrib.contenttypes.models import ContentType
+
+    autor_count_before = Autor.objects.count()
+
+    ac = AutorAutocomplete()
+    ac.request = rf.post("/", data={"text": "Kowalski Jan"})
+    ac.request.user = admin_user
+
+    obj = ac.create_object("Kowalski Jan")
+
+    assert obj.pk != -1
+    assert Autor.objects.count() == autor_count_before + 1
+
+    ct = ContentType.objects.get_for_model(Autor)
+    log = LogEntry.objects.get(
+        content_type=ct, object_id=str(obj.pk), action_flag=ADDITION
+    )
+    assert log.user == admin_user
+    assert "autocomplete" in log.change_message
+
+
 @pytest.mark.django_db
 def test_Status_KorektyAutocomplete(statusy_korekt):
     """Test status korekty autocomplete filtering."""

src/bpp/views/autocomplete/authors.py

@@ -94,10 +94,27 @@ class AutorAutocomplete(GroupRequiredMixin, AutorAutocompleteBase):
 
     def create_object(self, text):
         try:
-            return Autor.objects.create_from_string(text)
+            obj = Autor.objects.create_from_string(text)
         except ValueError:
             return self.err
 
+        from django.contrib.admin.models import ADDITION, LogEntry
+        from django.contrib.contenttypes.models import ContentType
+
+        try:
+            LogEntry.objects.create(
+                user_id=self.request.user.pk,
+                content_type_id=ContentType.objects.get_for_model(Autor).pk,
+                object_id=str(obj.pk),
+                object_repr=str(obj)[:200],
+                action_flag=ADDITION,
+                change_message="Utworzono z formularza autocomplete",
+            )
+        except (AttributeError, TypeError):
+            pass
+
+        return obj
+
 
 class PublicAutorAutocomplete(AutorAutocompleteBase):
     """Public autocomplete for authors (no create, no PBN/MNISW markers)."""

src/deduplikator_autorow/admin.py

@@ -10,6 +10,7 @@
     DuplicateCandidate,
     DuplicateScanRun,
     IgnoredAuthor,
+    IgnoredScientist,
     LogScalania,
     NotADuplicate,
 )
@@ -76,8 +77,8 @@ def get_author_last_name(self, obj):
     get_author_last_name.admin_order_field = "scientist_pk"
 
 
-@admin.register(IgnoredAuthor)
-class IgnoredAuthorAdmin(DynamicAdminFilterMixin, admin.ModelAdmin):
+@admin.register(IgnoredScientist)
+class IgnoredScientistAdmin(DynamicAdminFilterMixin, admin.ModelAdmin):
     list_display = [
         "get_scientist_display",
         "get_autor_display",
@@ -133,6 +134,42 @@ def save_model(self, request, obj, form, change):
         super().save_model(request, obj, form, change)
 
 
+@admin.register(IgnoredAuthor)
+class IgnoredAuthorAdmin(DynamicAdminFilterMixin, admin.ModelAdmin):
+    list_display = [
+        "get_autor_display",
+        "reason",
+        "created_by",
+        "created_on",
+    ]
+
+    list_filter = ["created_on", "created_by"]
+
+    search_fields = [
+        "autor__nazwisko",
+        "autor__imiona",
+        "reason",
+        "created_by__username",
+    ]
+
+    readonly_fields = ["created_on"]
+    date_hierarchy = "created_on"
+    ordering = ["-created_on"]
+
+    def get_autor_display(self, obj):
+        if obj.autor:
+            url = reverse("admin:bpp_autor_change", args=[obj.autor.pk])
+            return mark_safe(f'<a href="{url}">{obj.autor}</a>')
+        return "-"
+
+    get_autor_display.short_description = "Autor (BPP)"
+
+    def save_model(self, request, obj, form, change):
+        if not change:
+            obj.created_by = request.user
+        super().save_model(request, obj, form, change)
+
+
 @admin.register(LogScalania)
 class LogScalaniaAdmin(DynamicAdminFilterMixin, admin.ModelAdmin):
     list_display = [

src/deduplikator_autorow/migrations/0009_rename_ignoredauthor_ignoredscientist.py

@@ -0,0 +1,23 @@
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("deduplikator_autorow", "0008_add_priority_field"),
+    ]
+
+    operations = [
+        migrations.RenameModel(
+            old_name="IgnoredAuthor",
+            new_name="IgnoredScientist",
+        ),
+        migrations.AlterModelOptions(
+            name="ignoredscientist",
+            options={
+                "ordering": ["-created_on"],
+                "verbose_name": "Ignorowany Scientist (PBN)",
+                "verbose_name_plural": "Ignorowani Scientist (PBN)",
+            },
+        ),
+    ]