Skip to content

fix: make tactic .. at * save info contexts#12607

Merged
kmill merged 1 commit intoleanprover:masterfrom
kmill:kmill_fix_10898
Feb 25, 2026
Merged

fix: make tactic .. at * save info contexts#12607
kmill merged 1 commit intoleanprover:masterfrom
kmill:kmill_fix_10898

Conversation

@kmill
Copy link
Copy Markdown
Collaborator

@kmill kmill commented Feb 20, 2026

This PR fixes an issue where withLocation wasn't saving the info context, which meant that tactics that use at * location syntax and do term elaboration would save infotrees but revert the metacontext, leading to Infoview messages like "Error updating: Error fetching goals: Rpc error: InternalError: unknown metavariable" if the tactic failed at some locations but succeeded at others.

Closes #10898

This PR fixes an issue where `withLocation` wasn't saving the info context, which meant that tactics using `at *` that did term elaboration but failed would save infotrees but revert the metacontext, leading to Infoview messages like "Error updating: Error fetching goals: Rpc error: InternalError: unknown metavariable '?_uniq.187'."

Closes leanprover#10898
@kmill kmill requested a review from kim-em as a code owner February 20, 2026 11:06
@kmill kmill added the changelog-tactics User facing tactics label Feb 20, 2026
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Feb 20, 2026
@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2026-02-19 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2026-02-20 12:09:41)

mathlib-nightly-testing bot pushed a commit to leanprover-community/batteries that referenced this pull request Feb 20, 2026
@github-actions github-actions bot added the mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN label Feb 20, 2026
mathlib-nightly-testing bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Feb 20, 2026
@mathlib-lean-pr-testing mathlib-lean-pr-testing bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Feb 20, 2026
@mathlib-lean-pr-testing
Copy link
Copy Markdown

Mathlib CI status (docs):

@kmill kmill enabled auto-merge February 25, 2026 01:59
@kmill kmill added this pull request to the merge queue Feb 25, 2026
Merged via the queue into leanprover:master with commit c032af2 Feb 25, 2026
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-tactics User facing tactics mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unknown metavariable error in rw [] at *

2 participants