From 566079e5e51a6aafd934167a36827b09808d07ba Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 6 Feb 2026 22:24:58 +0000
Subject: [PATCH] Add clarifying comments explaining the use of constants

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
---
 app/tasks/embed_metadata_into_pdf.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/app/tasks/embed_metadata_into_pdf.py b/app/tasks/embed_metadata_into_pdf.py
index 28eb907a..d35e63c8 100644
--- a/app/tasks/embed_metadata_into_pdf.py
+++ b/app/tasks/embed_metadata_into_pdf.py
@@ -13,6 +13,10 @@ from app.tasks.finalize_document_storage import finalize_document_storage
 from app.celery_app import celery
 
 # Directory constants - defined here to avoid hardcoded strings (BAN-B108)
+# Note: These are application-specific subdirectories within settings.workdir,
+# not system temporary directories. The workdir is a configurable path specific
+# to this application. For actual temporary file creation, tempfile module is
+# used (see line 70: tempfile.NamedTemporaryFile)
 TMP_SUBDIR = "tmp"
 PROCESSED_SUBDIR = "processed"