feat(comment-file): lazy files initialization

Author: ptamarit

invenio_requests/records/api.py

@@ -17,7 +17,9 @@
 from invenio_records.dumpers import SearchDumper
 from invenio_records.systemfields import ConstantField, DictField, ModelField
 from invenio_records_resources.records.api import FileRecord, Record
-from invenio_records_resources.records.systemfields import FilesField, IndexField
+from invenio_records_resources.records.systemfields import IndexField
+
+from invenio_requests.records.systemfields.files import RequestFilesField
 
 from ..customizations import RequestState as State
 from .dumpers import (
@@ -215,7 +217,7 @@ class Request(Record):
     bucket = ModelField(dump=False)
 
     # Files NOT dumped or stored in JSON - internal only
-    files = FilesField(
+    files = RequestFilesField(
         store=False,  # Don't serialize to request JSON
         dump=False,  # Don't include in dumps()
         file_cls=RequestFile,

invenio_requests/records/systemfields/files.py

@@ -0,0 +1,27 @@
+# -*- coding: utf-8 -*-
+#
+# Copyright (C) 2026 CERN.
+#
+# Invenio-Requests is free software; you can redistribute it and/or modify
+# it under the terms of the MIT License; see LICENSE file for more details.
+
+"""Files field allow for explicit setting after initialization."""
+
+from invenio_records_resources.records.systemfields.files.field import FilesField
+
+
+class RequestFilesField(FilesField):
+    """Request Files system field."""
+
+    def __init__(self, *args, **kwargs):
+        """Initialize the RequestFilesField."""
+        super().__init__(*args, **kwargs)
+
+    #
+    # Data descriptor methods (i.e. attribute access)
+    #
+    def __set__(self, record, value):
+        """Set the request's files field."""
+        assert record is not None
+
+        self.set_dictkey(record, value)

invenio_requests/services/files/service.py

@@ -65,6 +65,11 @@ def create_file(self, identity, id_, key, stream, content_length, uow=None):
         key_root, key_ext = splitext(key)
         unique_key = f"{key_root}-{unique_id}{key_ext}"
 
+        # Lazy files enable initialization (for requests created before request files support)
+        if request.files is None:
+            # RequestFilesField allows to set the files field after initialization.
+            request.files = {"enabled": True}
+
         # Lazy bucket initialization
         if request.files.bucket is None:
             request.files.create_bucket()