Skip to content

MIT-618: tidy two GitHub-task cause-word notes (batch 9, no new tasks) - #173

Merged
ZachFara merged 1 commit into
feature/eval-setupfrom
feat/MIT-618-batch9
Nov 11, 2026
Merged

ZachFara merged 1 commit into
feature/eval-setupfrom
feat/MIT-618-batch9

Conversation

@ZachFara

Copy link
Copy Markdown
Owner

What changed

This PR is the cleanup commit in tools/eval/github_tasks.py. It adds 0 new GitHub tasks, so the file still holds 9. Zach wants 20. Batch 9 covered developer tooling and packaging. I hit the turn budget with no candidate that met the bar; see the rejected candidates.

The cleanup

  • gh-pydantic-settings-validate-by-name:
    • Dropped the cause words leaves model_config, model_config unchanged and the bare no longer mutates. Kept no longer mutates model_config.
    • The comment now says "every phrase but the version carries its object". That makes it true: every remaining phrase names model_config or validate_by_name, and the only exception is 2.14.
    • Two comment lines were re-wrapped to stay under the line limit.
  • gh-table-extraction-bool-form: the comment no longer says the laid-over test names "say" "boolean form field". The names use underscores, so grading can't match them. The reason given now is that a fix treating only the symptom says it ("parse the boolean form field and apply its default").

No other line in the file changed.

New tasks

None. Pairs: 0. New version words: none. The expected set in test_the_cause_words_grader_v2_skips_are_exactly_the_versions is unchanged.

Rejected candidates (this batch)

Why

This is the batch-9 slice of MIT-618, the GitHub-task eval. The cleanup makes two task records' notes match the cause-word rules the reviews settled on.

How it was tested

  • server/tests/test_eval_grader.py and server/tests/test_eval_repos.py pass, 109 tests.
  • make check passes after git merge origin/feature/eval-setup, which was already up to date.
  • pip-tools candidate runs: the containerized steps that agent_eval.py check runs, through a .scratch/ helper calling repos.materialize, repos.prepare and repos.run_tests with container.run. Containers ran one at a time, none were left behind, and --on-host was never used.

Learned

  • Developer-tooling releases after the cutoff, for future mining:
    • setuptools 83.0.0 (2026-07-04), 84.0.0 (2026-08-08), 84.1/84.2 (2026-10-19)
    • hatchling 1.31.0 (07-08), 1.32.0 (08-11), 1.32.3 (09-17), 1.33.0 (10-28)
    • packaging 26.3 (08-04)
    • pip 26.2 (07-29), 26.3 (about 11-02)
    • Sphinx 9.1.1 (10-20), 9.2.0 (10-21)
    • griffe 2.2.0 to 2.4.1 (08-16 to 10-22)
    • tomlkit 0.15.1 (07-17), 0.15.2 (10-16)
    • coverage 7.15 to 7.17 (07-02 to 11-04)
    • build 1.6.0 (08-27)
    • twine 7.0.0 (07-28)
    • typeguard 4.6.0 (07-26)
    • pre-commit 4.6.1 to 4.7.0 (07-21 to 10-13)
    • beartype 0.23.0 (10-10, rc0 08-08)
    • typing_extensions 4.16.0 (07-02, rc 06-24)
  • Tooling fixes in this area are often test-only adaptations (tox), compat shims with no test (zope meta), or regression tests whose docstrings state the cause (mxdev, bsb). The yield is low, lower than for runtime libraries.
  • pip-tools' pip_conf, pip_with_index_conf and minimal_wheels_path fixtures download setuptools from PyPI at session start, so any test that uses them fails under --network none. Plain runner compile tests with --no-index run offline.
  • GitHub issue search tokenizes dotted versions: "hatchling 1.32" in:title finds nothing, while "hatchling 1.32.3" in:title,body finds the issues. Search the full patch version, and search bodies too.
  • The worktree guard refuses shell loops over gh with a loop variable. Put repeated gh api calls in a .scratch/ helper run with python3 -I.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y

Drop the validate-by-name task's cause words that name no change to
pydantic's object ("leaves model_config", "model_config unchanged",
bare "no longer mutates"), and say that every phrase but the version
carries its object. In the table-extraction task, the reason to leave
out a bare "boolean form field" is that a symptom-only fix says it;
the test names use underscores, so grading can't match them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Qc1CQTb8r67J32HpkhS43y
@ZachFara
ZachFara merged commit 405f4e4 into feature/eval-setup Nov 11, 2026
2 checks passed
@ZachFara
ZachFara deleted the feat/MIT-618-batch9 branch November 11, 2026 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant