Perform the legacy-filename re-standardization on prod (#1390)

Remove --dry-run from entrypoint step 4.10b so restandardize_artifact_filenames
actually renames the ~213 historical talk/poster/publication files to the
standardized Author_TitleInTitleCase_VenueYear scheme. This follows two
diagnostic dry-runs (2.25.1/2.25.2) and the forum-name cleanup that removed the
doubled-year artifacts; the reviewed prod dry-run was clean (0 errors, 0
doubled years, no real collisions, longest path 240<255).

Provenance was captured by #1391, so original upload names are preserved and
stale publication links still resolve via the serve_pdf original_pdf_filename
fallback.

Also elevate the per-rename log line from DEBUG to INFO so the one-time backfill
leaves a per-file audit trail in prod's (INFO-level) debug.log. The step is
idempotent, so once rows are standardized this goes silent on later deploys.

Bump to 2.25.3.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: jonfroehlich

docker-entrypoint.sh

@@ -166,10 +166,7 @@ python manage.py setup_admin_groups
 echo "****************** STEP 4.10b/5: docker-entrypoint.sh ************************"
 echo "4.10b Running 'python manage.py restandardize_artifact_filenames' to rename legacy talk/poster/pub files to the standardized scheme (#1401)"
 echo "******************************************"
-# TEMPORARY (#1390): --dry-run only logs what WOULD be renamed, touching nothing
-# on disk or in the DB, so we can review the exact prod rename set in debug.log
-# before doing it for real. REMOVE --dry-run and redeploy to perform the rename.
-python manage.py restandardize_artifact_filenames --dry-run
+python manage.py restandardize_artifact_filenames
 
 # echo "****************** STEP 4.3/5: docker-entrypoint.sh ************************"
 # echo "4.3 Running 'python manage.py rename_person_images' to rename person images"

makeabilitylab/settings.py

@@ -86,8 +86,8 @@
     SECURE_PROXY_SSL_HEADER = ('HTTP_X_FORWARDED_PROTO', 'https')
 
 # Makeability Lab Global Variables, including Makeability Lab version
-ML_WEBSITE_VERSION = "2.25.2" # Keep this updated with each release and also change the short description below
-ML_WEBSITE_VERSION_DESCRIPTION = "Prep release for the legacy-filename re-standardization (#1390). Normalizes Talk and Poster forum names by stripping an embedded trailing year (e.g. 'ASSETS 2016' becomes 'ASSETS'), which previously only ran on publications; this is why some standardized filenames doubled the year (e.g. '...ASSETS20162016'). The cleanup writes only the forum_name field (never renaming files), so the file re-standardization stays gated and continues to run in --dry-run mode in this release. No files are renamed in this release; the venue label shown on affected talk pages now omits the year (the date is shown separately, matching publications)."
+ML_WEBSITE_VERSION = "2.25.3" # Keep this updated with each release and also change the short description below
+ML_WEBSITE_VERSION_DESCRIPTION = "Performs the legacy-filename re-standardization (#1390). After two diagnostic dry-runs and a forum-name cleanup (2.25.1/2.25.2), this release renames the ~213 historical talk/poster/publication files that were never converted to the standardized Author_TitleInTitleCase_VenueYear scheme (bulk-imported rows that never went through an authored save). The original uploaded names were already captured (#1391), so provenance is preserved and stale publication links still resolve via the serve_pdf fallback. Each rename is logged to debug.log; the step is idempotent, so subsequent deploys are no-ops."
 DATE_MAKEABILITYLAB_FORMED = datetime.date(2012, 1, 1)  # Date Makeability Lab was formed
 MAX_BANNERS = 7 # Maximum number of banners on a page
 

website/management/commands/restandardize_artifact_filenames.py

@@ -136,7 +136,11 @@ def _restandardize_row(self, artifact, dry_run):
         # untouched (it only captures on a new upload, not a no-update_fields
         # save), so the provenance recorded by #1391 is preserved.
         artifact.save()
-        _logger.debug(
+        # INFO (not DEBUG) so the real rename is captured on prod, giving a
+        # per-file audit trail of the one-time backfill. Idempotent: once a row
+        # is standardized it stops matching, so this stays silent on later
+        # deploys rather than logging every artifact forever.
+        _logger.info(
             f"Re-standardized {model_name} id={artifact.pk} to "
             f"'{Artifact.generate_filename(artifact)}'."
         )