Skip to content

Fix apply-to proof argument round-trip printing#976

Merged
jsiek merged 1 commit into
mainfrom
codex/issue-931-lib-roundtrip-slice-2
Jun 13, 2026
Merged

Fix apply-to proof argument round-trip printing#976
jsiek merged 1 commit into
mainfrom
codex/issue-931-lib-roundtrip-slice-2

Conversation

@jsiek

@jsiek jsiek commented Jun 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • Wrap complex proof arguments in braces when pretty-printing apply ... to ..., so proof statements such as arbitrary ... assume ... are not flattened into an unparsable token stream.
  • Add a focused should-validate fixture to the parser round-trip corpus for apply ... to arbitrary ... assume ....

Progresses #931.

Validation

  • python3.13 deduce.py test/should-validate/apply_to_intro_roundtrip.pf
  • python3.13 test-deduce.py --equiv --serial
  • make tests

Codex session

codex://threads/019ebf3e-88c6-7b42-a2d9-afee9278e467

open 'codex://threads/019ebf3e-88c6-7b42-a2d9-afee9278e467'

@jsiek jsiek closed this Jun 13, 2026
@jsiek jsiek deleted the codex/issue-931-lib-roundtrip-slice-2 branch June 13, 2026 04:59
@jsiek jsiek restored the codex/issue-931-lib-roundtrip-slice-2 branch June 13, 2026 09:32
@jsiek jsiek reopened this Jun 13, 2026
@jsiek jsiek force-pushed the codex/issue-931-lib-roundtrip-slice-2 branch from 16ab44b to 4f221f2 Compare June 13, 2026 16:34
@jsiek jsiek marked this pull request as ready for review June 13, 2026 16:44
@jsiek jsiek merged commit c091a8a into main Jun 13, 2026
9 checks passed
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