as567-code opened a new pull request, #64197:
URL: https://github.com/apache/airflow/pull/64197

   ## Summary
   
   - Validate `order_by` parameter against allowed `TaskResponse` fields, 
returning HTTP 400 with a helpful error message listing valid fields for 
invalid field names
   - Handle `None` values in task attribute sorting by placing them at the end, 
preventing `TypeError: '<' not supported between instances of 'NoneType' and 
'NoneType'`
   - Add test for sorting by `start_date` on unscheduled DAGs (reproduces the 
bug)
   
   closes: #63927
   
   ## Test plan
   
   - [x] Added test `test_should_handle_order_by_with_none_values` — verifies 
sorting works when task attributes are `None`
   - [x] Updated `test_should_raise_400_for_invalid_order_by_name` — verifies 
improved error message
   - [ ] Run full test suite: `uv run --project airflow-core pytest 
airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_tasks.py -xvs`
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Opus 4.6
   
   Generated-by: Claude Opus 4.6 following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to