Skip to content

refactor: modの不要な失敗ガードと重複検証を整理する - #100

Merged
milkeclair merged 5 commits into
mainfrom
refactor/mod-guards
Nov 15, 2026
Merged

milkeclair merged 5 commits into
mainfrom
refactor/mod-guards

Conversation

@milkeclair

@milkeclair milkeclair commented Nov 15, 2026 •

Copy link
Copy Markdown
Owner

やったこと

  • Refs 防御的すぎる|| return 1をやめたい #96
  • 検証済みのjobパス計算、生成済みport keyの再解析、常に成功する設定値マージの重複ガードを整理します。
  • psの取得結果が空なら直後の判定でfalseになるため、重複する失敗ガードを削除します。process終了との競合と、補完job取消の常に成功する契約を直接テストします。
  • jobのnameとcommandの検証を公開入口の_validateにまとめ、準備処理で繰り返していた検証を削除します。
  • 補完候補の空判定を結果側へまとめます。private除外のzstyle設定が失敗しても補完登録とcache jobの起動を続け、対応する直接テストを更新します。

やらないこと

  • named && localを含む既存の宣言書式・getterの代入書式を維持します。
  • lib側の整理は #99 です。両PRは独立してmainへ適用できます。

動作確認

  • WSLのworktreeでZ_ROOTを設定し、testディレクトリからz.t -c mod/job mod/completion mod/wtproxyを実行。310件成功。追加整理後のproxy判定・停止・補完cacheの直接テスト15件も成功。
  • 変更したZshファイルのzsh -nとgit diff --checkを確認。
  • 書式を復元した最終コミットに対して、CIのZsh全1,101件とrelease・lockfile検査が成功。
  • 画面を持たないmodの変更のため、実行画面の画像は添付していません。

その他

  • private除外設定の失敗はbest effortとして扱います。compdef未初期化、job起動失敗、Gitコンテキスト取得失敗、状態ファイルの失敗、Docker削除やTCP処理の停止判定は維持しています。

@milkeclair milkeclair changed the title refactor: modの過剰な成功判定と重複検証を整理する refactor: modの不要な失敗ガードと重複検証を整理する Nov 15, 2026
@milkeclair
milkeclair marked this pull request as ready for review November 15, 2026 04:50
@milkeclair
milkeclair merged commit a37a79b into main Nov 15, 2026
3 checks passed
@milkeclair
milkeclair deleted the refactor/mod-guards branch November 15, 2026 04:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant