Inline homework upsert text fields

Author: alexeygrigorev

api/views/homework_upsert_save.py

@@ -54,7 +54,12 @@ def create_homework_for_upsert(upsert):
 
 
 def apply_homework_data(homework, data):
-    apply_homework_text_fields(homework, data)
+    title = homework_title_from_data(data)
+    if title is not None:
+        homework.title = title
+
+    if "description" in data:
+        homework.description = data["description"]
 
     error = apply_homework_validated_fields(homework, data)
     if error:
@@ -64,15 +69,6 @@ def apply_homework_data(homework, data):
     return None
 
 
-def apply_homework_text_fields(homework, data):
-    title = homework_title_from_data(data)
-    if title is not None:
-        homework.title = title
-
-    if "description" in data:
-        homework.description = data["description"]
-
-
 def apply_homework_validated_fields(homework, data):
     for apply_field in (
         apply_homework_instructions_url,

docs/refactoring-plan.md

@@ -2035,6 +2035,20 @@ Steps:
   `wide_positional_calls=0`, `wide_function_args=0`,
   `nested_wide_for_unpacking=0`, `range_len_loops=0`), and
   `git diff --check`.
+- [x] 2026-07-02: Removed the single-use homework-upsert text-field assignment
+  helper. The title/name fallback and optional description assignment now live
+  directly in `apply_homework_data`, before validated fields and direct update
+  fields. Verification:
+  `uv run ruff check api/views/homework_upsert_save.py docs/refactoring-plan.md`,
+  `python -m py_compile api/views/homework_upsert_save.py`,
+  `uv run python manage.py test api.tests.test_homework_upserts api.tests.test_homework_mutations api.tests.test_homeworks`,
+  removed-helper reference scan, touched-function line-threshold scan,
+  `uvx pyrefly check`, repository AST cleanup scan excluding migrations
+  (`forbidden_comprehensions=0`, `threshold_violations=0`,
+  `append_constructed=0`, `wide_tuple_unpacking=0`,
+  `wide_positional_calls=0`, `wide_function_args=0`,
+  `nested_wide_for_unpacking=0`, `range_len_loops=0`), and
+  `git diff --check`.
 - [x] Replace deadline reminder spec factory functions with named static spec
   values and constants so reminder event planning no longer uses trivial
   pass-through factories or nine-field constructor calls. Verification: