Conversation
* Avoid duplicating a terminal backslash while splitting names. * Escape comma-adjacent name sections without changing the final section. * Enable the strict and non-strict inverse regressions.
|
Hi @claell — same note across all of your July 16 issues and PRs (#567–#597), apologies for the repetition. Closing this. Passing CI isn't evidence the underlying claim is real — it just means generated tests match generated code. The batch's timing, structure, and forward-referenced numbers point to this being AI-generated rather than manually found and checked, and reviewing it properly would take more effort than a targeted pass over the parser myself. If this fixes something you actually hit: open a small, human-verified PR against a reopened issue with a real repro, and I'll review it in good faith. Any nontrivial design or API choice needs to be discussed and agreed first, not decided inside a PR. Please disclose and verify AI-assisted work before submitting going forward. |
Summary
of input.
skipping names with odd trailing backslashes.
Fixes #567.
Validation
deprecation warnings in
tests/test_entrypoint.py. The two removed skips arethe now-passing strict and non-strict inverse cases.
git diff --check: passed.environment; GitHub CI should independently run the configured hooks.
Review note
This change was developed and validated in a concentrated session rather than
exercised over a long period in production. The broad existing name fixture
matrix now passes without the targeted skip, but careful human review of TeX
escaping semantics is requested.
AI assistance
This pull request was prepared with ChatGPT Codex using GPT-5.6 Sol with high
reasoning effort. Codex assisted with analysis, implementation, branch
isolation, and test execution. Automated validation is not a substitute for
maintainer review.