diff --git a/AGENTS.md b/AGENTS.md index a24de5eae2b..9b8c091f433 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -158,7 +158,7 @@ reported as such are described in "What is NOT considered a security vulnerabili - Use `@pytest.mark.db_test` for tests that require database access. - Test fixtures: `devel-common/src/tests_common/pytest_plugin.py`. - Test location mirrors source: `airflow/cli/cli_parser.py` → `tests/cli/test_cli_parser.py`. -- Do not use `caplog` in tests, prefer checking logic and not log output. +- Do not assert on raw log text (`caplog.text` for example), these are legacy string matching APIs planned for removal. Structured assertions via `caplog` (which resolves to `cap_structlog` under structlog) are fine and preferred: `"event name" in caplog` or `{"event": ..., "field": ...} in caplog`. ## Output conventions diff --git a/dev/skill-evals/last-eval-hash.txt b/dev/skill-evals/last-eval-hash.txt index d0863d63657..56953a74fd4 100644 --- a/dev/skill-evals/last-eval-hash.txt +++ b/dev/skill-evals/last-eval-hash.txt @@ -1,3 +1,3 @@ # Generated by dev/skill-evals/eval.py — do not edit or resolve conflicts by hand. # Run `prek run run-skill-eval --hook-stage manual --all-files` to regenerate. -f4a71898d35b99e6c42ce756a71fa24ba243ada0611ed55966fce25711ac8c02 +9901af13862556bf571a3db5950f1bd32b1cbb79e8e18d3eb41e4fc3827788ce