Repository navigation
MIT-618: one more verified GitHub task (second batch) - #165
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y
…IT-618) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y
This was referenced Nov 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
tools/eval/github_tasks.pygets 1 more verified GitHub task, for 4 in all. That is short of the 10 asked for. I stopped adding tasks rather than lower the bar. The record follows the existing style: a source comment citing the issue, the fix and the upstream release notes;EXCLUDE_NEWERin the setup; and the sametests()helper.upstream_changegh-pydantic-settings-signaturepydantic-2.14-field-name-for-signature__signature__whenvalidate_by_aliasisFalsepydantic/pydantic#13730) merged on 2026-08-29. Issue #1023 was opened 2026-10-09, and its fix #1029 was squash-merged 2026-10-13.base_commitis the parent of the squash commit (10dc635).validate_by_aliasargument, which points at a fix, and the record's comment says so.Base failing output (containerized,
-v) and cause-word checktests/test_source_cli.py::test_cli_app_run_base_model_validate_by_name_only. The traceback ends inCliApp.run(pydantic_settings/main.py:883):E TypeError: _field_name_for_signature() missing 1 required positional argument: 'validate_by_alias'2.14,added a required,new required. Notprivateorsignature: the traceback ends inpydantic/_internal/_signature.py, so a fix that only describes the symptom would say them (the same rule as tekore's_missing_).run,_init_kwarg_nameand_field_name_for_signature.test_cli_app_run_base_model_aliases) is left out: it fails at the base on any pydantic version, because a non-identifier alias never worked.Leak review of the laid-over test
I read every comment, docstring and test name in
tests/test_source_cli.pyat the fix sha, plus the fix's whole diff. The new tests have no docstrings or comments. None of the file's 179 test names or its comments mention pydantic versions, signatures or private helpers.Rejected candidates
pl.count()): verified that it fails before and passes after, and that the base passes on polars 1.44.2. Rejected on review: the fix (pl.count()→pl.len()) has been a polars deprecation since 0.20 (2023), so it was widely answered before the cutoff, and the captured log namespl.len(). Only the hard error is new in 2.0.join_nulls): the changed tests don't join, so they don't reach the code fix. The rename dates from before the cutoff.NUMERIC_DTYPES): removal announced at 1.0; Streamlit app.explodedefault): same cause as the hold-outreal-polars-explode-empty._get_combined_index): the test comment states the cause, and the test fails at the base on any pandas.Node._nodeid): the fix changes no test.validate_by_alias=Falseon pydantic 2.14 pydantic/pydantic-settings#1022 (same upstream change as the accepted task): same repository, so not a two-repo pair.Why
Ticket: MIT-618 (second batch), the GitHub-task eval. These are real post-cutoff tasks for the arms run.
How it was tested
tools/eval/repos.pyandtools/eval/container.pydirectly, the same stepsagent_eval.py checkruns:--on-hostwas used, and no containers were left behind.server/tests/test_eval_repos.pyandtest_eval_grader.pypass.make checkpasses.Learned
pl.count()) fails that test even when the release is new._polars_runtime_32is 169 MB installed, so every polars 2.0 task lands near or over the 200 MB install guideline.cause_wordslike "2.0" match ordinary float data ("21002.0") and timestamps. Check them against the full captured output.gh search prswith a bare version string such as "polars 2" matches unrelated PRs numbered 2.🤖 Generated with Claude Code
https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y