From b9ecb4cbc4b68e53def81d29e97cac46ad596d3a Mon Sep 17 00:00:00 2001 From: Seasawher Date: Tue, 8 Oct 2024 03:25:49 +0900 Subject: [PATCH] =?UTF-8?q?`linter.flexible`=20=E3=82=92=E7=B4=B9=E4=BB=8B?= =?UTF-8?q?=E3=81=99=E3=82=8B=20Fixes=20#941?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .github/workflows/ci.yml | 3 +- LeanByExample/Reference/Option/Flexible.lean | 78 ++++++++++++++++++-- booksrc/SUMMARY.md | 1 + 3 files changed, 75 insertions(+), 7 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 6f2e9feb..cbbefee5 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -25,8 +25,7 @@ jobs: - name: lean action uses: leanprover/lean-action@v1 with: - # 警告がないことを確かめる - build-args: "--fail-level=warning" + build-args: "--log-level=error" # Lean の更新により Windows 環境でだけビルドが壊れる可能性もあるので、 # 念のために Windows 環境でもビルドを行う diff --git a/LeanByExample/Reference/Option/Flexible.lean b/LeanByExample/Reference/Option/Flexible.lean index 7d0101e9..69a5980e 100644 --- a/LeanByExample/Reference/Option/Flexible.lean +++ b/LeanByExample/Reference/Option/Flexible.lean @@ -1,8 +1,76 @@ -import Mathlib.Tactic +/- # linter.flexible +`linter.flexible` オプションは、ライブラリの変更に対して頑強な証明を作るためのリンターの有効/無効を切り替えます。 + +## 背景 + +証明を書く際のベストプラクティスとして、[`simp`](#{root}/Reference/Tactic/Simp.md) タクティクのような柔軟性のあるタクティクは、証明の末尾以外で使わないほうがよいということが知られています。[^non-terminal-simp] `simp` タクティクはどういう補題が `[simp]` 属性で登録されているかに応じて挙動が変わるため、Mathlib に新たな`simp` 補題が追加されたり、あるいは削除されたりするたびに挙動が変わってしまうことがその理由です。 + +たとえば、以下のような証明を考えてみます。 +-/ +import Mathlib.Tactic --# + +section --# +set_option linter.flexible false --# + +opaque foo : Nat +opaque bar : Nat + +/-- `foo` と `bar` が等しいという定理 -/ +axiom foo_eq_bar : foo = bar + +example {P : Nat → Prop} (hbar : P bar) : True ∧ (P foo) := by + simp + rw [foo_eq_bar] + assumption + +/- ここで、定理 `foo_eq_bar` は今のところ `[simp]` 属性が与えられていませんが、与えられると `rw` タクティクが失敗するようになります。-/ + +attribute [simp] foo_eq_bar + +example {P : Nat → Prop} (hbar : P bar) : True ∧ (P foo) := by + -- 元々はゴールを `P foo` に変えていた + simp + + -- ゴールが変わってしまった + guard_target =ₛ P bar + + -- `rw` タクティクが失敗するようになった + fail_if_success rw [foo_eq_bar] + + assumption + +end --# +/- ここで問題は、`[simp]` 属性が与えられる前の証明において、`simp` タクティクがゴールをどのように変えていたのかわからないということです。言い換えると、元々の証明の意図がわからないということです。それがわかっていれば、ライブラリの変更に合わせて証明を修正するのが少しは容易になります。 + +これは `simp` が柔軟(flexible)なタクティクであることの弊害で、対策として次のような方法が知られています。 + +* 証明末でしか `simp` タクティクを使わない。証明末であれば、「壊れる前の証明において `simp` が何を行っていたか」はつねに「残りのゴールを閉じる」であり明確で、ライブラリの変更があってもどう修正すればいいかがわかりやすいからです。 +* [`have`](#{root}/Reference/Tactic/Have.md) タクティクや [`suffices`](#{root}/Reference/Tactic/Suffices.md) タクティクを使って証明末そのものを増やす。 +* `simp only` 構文を使って、ライブラリ側に変更があっても `simp` タクティクの挙動が変わらないようにする。 +* `simpa` タクティクのような、ゴールを閉じなければならないという制約を持つタクティクで書き換える。 +-/ +/- ## このリンタについて +このリンタを有効にすると、上記のようなライブラリの変更に対して脆弱な証明を自動で検出して、警告を出してくれます。 +-/ +section --# + +-- flexible tactic リンタを有効にする set_option linter.flexible true -example {n m : Nat} (h : n + 0 = m) : True ∧ (n = m) := by - constructor - all_goals - simp_all +/-- +warning: 'simp' is a flexible tactic modifying '⊢'… +note: this linter can be disabled with `set_option linter.flexible false` +-/ +#guard_msgs (warning) in + example {n m : Nat} (h : n = m) : True ∧ (n = m) := by + simp + exact h + +example {n m : Nat} (h : n = m) : True ∧ (n = m) := by + -- 書き換えると警告が消える + simpa + +/- [^non-terminal-simp]: を参照のこと。-/ + +end --# diff --git a/booksrc/SUMMARY.md b/booksrc/SUMMARY.md index 47b3721f..fe16c824 100644 --- a/booksrc/SUMMARY.md +++ b/booksrc/SUMMARY.md @@ -74,6 +74,7 @@ - [オプション](./Reference/Option/README.md) - [autoImplicit: 自動束縛暗黙引数](./Reference/Option/AutoImplicit.md) + - [flexible: 脆弱な証明を指摘するリンタ](./Reference/Option/Flexible.md) - [hygiene: マクロ衛生](./Reference/Option/Hygiene.md) - [型クラス](./Reference/TypeClass/README.md)