refactor(zsh): 設定ファイルをディレクトリ構造で再編成 by nyatinte · Pull Request #28 · nyatinte/dotfiles

nyatinte · 2026-04-06T13:15:31Z

概要

フラットに並んでいた zsh 設定ファイルを、責務ごとにディレクトリ分けして再編しました。

変更前

~/.zsh.d/
  path.zsh       # PATH定義
  dev-tools.zsh  # 開発ツール全般（責務が広すぎた）
  fzf.zsh        # fzf設定
  utils.zsh      # ユーティリティ関数

dev-tools.zsh が proto の起動、Docker 初期化、エディタ設定、パッケージマネージャ設定、Vite+ の読み込みなど、無関係な設定をすべて抱えていた。また path.zsh と dev-tools.zsh に PATH 追加が重複していた。

変更後

~/.zsh.d/
  core/
    path.zsh     # PATH 一元管理
    env.zsh      # 環境変数（EDITOR, NI_*, LG_CONFIG_FILE）
  tools/
    proto.zsh    # proto バージョンマネージャの起動
    docker.zsh   # Docker Desktop 初期化（任意）
    fzf.zsh      # fzf キーバインド・プレビュー設定
    vite-plus.zsh # Vite+ 環境変数（任意）
  functions/
    git.zsh      # gwn（git worktree ヘルパー）
    notify.zsh   # notify-after（コマンド完了通知）

機能差分

リファクタリングにあたり全項目を照合した結果、実質的な挙動の変化は 1点のみ です：

項目	変更前	変更後
`vite-plus` の読み込み	`. "$HOME/.vite-plus/env"`（無条件）	`[[ -f ... ]] && source`（存在確認あり）

変更前はファイルが存在しない環境でシェル起動がエラー終了するバグがありました。今回の変更で安全になっています。

その他の変更は機能的に同等です：

dev-tools.zsh にあった export PATH="$PROTO_HOME/tools/node/globals/bin:$PATH" は core/path.zsh の typeset -U path PATH + path 配列に同エントリが既にあったため、削除しても動作は変わりません
環境変数の読み込み順が fpath 設定より前になりましたが、独立した変数のため影響なし

その他の改善点

functions/ はグロブ（*.zsh(N)）で読み込むため、新しい関数ファイルを置くだけで自動認識されます
(N) glob 修飾子により、functions/ が空でもエラーになりません
.local.zsh の存在チェックを [ -f ] から [[ -f ]] に統一しました

…tories - core/path.zsh: PATH management (consolidated from path.zsh + dev-tools.zsh) - core/env.zsh: environment variables (EDITOR, NI_*, LG_CONFIG_FILE) - tools/proto.zsh, docker.zsh, fzf.zsh, vite-plus.zsh: per-tool activation - functions/git.zsh, notify.zsh: utility functions split by domain https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

…acket style - Remove self-documenting comments from env.zsh, docker.zsh, vite-plus.zsh - Use (N) glob qualifier in functions loop to handle empty directory safely - Standardize to [[ ]] throughout (was mixing [ ] for .local.zsh) https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

gemini-code-assist

Code Review

This pull request refactors the Zsh configuration by modularizing settings into dedicated directories for core environment variables, tools, and functions. The .zshrc file has been updated to source these new locations, including a dynamic loop for loading functions. Feedback includes a suggestion to improve the notify-after function by displaying the full command string instead of just the first argument, and a recommendation to use a similar loop for sourcing tool configurations to enhance maintainability.

gemini-code-assist · 2026-04-06T13:17:30Z

+	if [ $exit_code -eq 0 ]; then
+		cmux notify --title "✓ Command Complete" --body "$1"
+	else
+		cmux notify --title "✗ Command Failed" --body "$1 (exit $exit_code)"
+	fi


The notification currently only shows the first argument of the command (e.g., sleep instead of sleep 10). Using "$*" will provide the full command string in the notification body, making it much more informative for the user.

if [ $exit_code -eq 0 ]; then cmux notify --title "✓ Command Complete" --body "$*" else cmux notify --title "✗ Command Failed" --body "$* (exit $exit_code)" fi

ご指摘ありがとうございます。$1 だと sleep 10 の通知が sleep だけになってしまいますね。$* に修正しました。

Generated by Claude Code

gemini-code-assist · 2026-04-06T13:17:30Z

+source "${ZSH_CONFIG_DIR}/tools/proto.zsh"
+source "${ZSH_CONFIG_DIR}/tools/docker.zsh"
+source "${ZSH_CONFIG_DIR}/tools/fzf.zsh"
+source "${ZSH_CONFIG_DIR}/tools/vite-plus.zsh"


To improve maintainability and consistency with how functions are loaded (lines 24-26), consider using a loop to source all files in the tools/ directory. This allows for adding or removing tool configurations without needing to manually update .zshrc each time.

for f in "${ZSH_CONFIG_DIR}/tools/"*.zsh(N); do source "$f" done

tools/ はロード順に意味があります（core/env.zsh で設定した変数を proto.zsh が参照するなど）。グロブによるアルファベット順読み込みでは順序が保証できないため、今回は明示的 source を維持します。

Generated by Claude Code

Ensures chezmoi apply cleans up old files from ~/.zsh.d/: - remove_path.zsh → deletes ~/.zsh.d/path.zsh - remove_dev-tools.zsh → deletes ~/.zsh.d/dev-tools.zsh - remove_fzf.zsh → deletes ~/.zsh.d/fzf.zsh - remove_utils.zsh → deletes ~/.zsh.d/utils.zsh https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

Copilot

Pull request overview

zsh 設定ファイルを責務ごとにディレクトリ分割し、.zshrc から新構成を読み込むように整理するPRです（Vite+ の env 読み込みは存在チェック付きに変更）。

Changes:

~/.zsh.d を core/, tools/, functions/ に再編し、.zshrc の読み込み順・対象を更新
PATH/環境変数を core/path.zsh, core/env.zsh に分離し、ツール設定を tools/ 配下へ移動
notify-after を functions/notify.zsh に分離し、関数ファイルをグロブで自動読み込み

Reviewed changes

Copilot reviewed 8 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
home/dot_zshrc	新ディレクトリ構成に合わせて core/tools/functions を読み込むよう変更
home/dot_zsh.d/core/path.zsh	PATH 管理を配列 + `typeset -U` で一元化
home/dot_zsh.d/core/env.zsh	エディタ/ni/lazygit などの環境変数を分離
home/dot_zsh.d/tools/proto.zsh	proto 有効化設定を分離
home/dot_zsh.d/tools/docker.zsh	Docker init の任意読み込みを分離
home/dot_zsh.d/tools/vite-plus.zsh	Vite+ env を存在チェック付きで読み込み
home/dot_zsh.d/tools/fzf.zsh	fzf のキーバインド/ウィジェット設定を独立ファイル化
home/dot_zsh.d/functions/git.zsh	`notify-after` を分離したため削除
home/dot_zsh.d/functions/notify.zsh	`notify-after` を新規ファイルとして追加
home/dot_zsh.d/dev-tools.zsh	旧まとめファイルを削除

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-04-06T13:22:58Z

+for f in "${ZSH_CONFIG_DIR}/functions/"*.zsh(N); do
+	source "$f"
+done


The functions autoload loop can behave unexpectedly when the glob matches nothing: in many shells (including zsh), for f in; do ... falls back to iterating over positional parameters. If functions/*.zsh(N) expands to empty (e.g., directory missing/empty), this could end up attempting to source arbitrary shell args. Consider guarding with an explicit existence check (e.g., build an array of matches and only loop when non-empty) or add a [[ -r $f ]] || continue inside the loop.

Suggested change

for f in "${ZSH_CONFIG_DIR}/functions/"*.zsh(N); do

source "$f"

done

function_files=("${ZSH_CONFIG_DIR}/functions/"*.zsh(N))

if (( ${#function_files[@]} )); then

for f in "${function_files[@]}"; do

source "$f"

done

fi

zsh の (N) glob 修飾子（Null glob）は、マッチするファイルが0件のとき展開結果を空にしてループをスキップします。for f in *.zsh(N) でマッチなしの場合にループ本体は実行されないため、配列化によるガードは不要と判断し現状を維持します。

Generated by Claude Code

Copilot · 2026-04-06T13:22:58Z

+notify-after() {
+	"$@"
+	local exit_code=$?
+	if [ $exit_code -eq 0 ]; then


Use of single-bracket test with an unquoted variable is brittle in zsh ([ $exit_code -eq 0 ] can error if the variable is ever empty/non-numeric). Prefer zsh-native arithmetic ((( exit_code == 0 ))) or [[ $exit_code -eq 0 ]] for a more robust numeric check.

Suggested change

if [ $exit_code -eq 0 ]; then

if (( exit_code == 0 )); then

ご指摘ありがとうございます。(( exit_code == 0 )) に修正しました。

Generated by Claude Code

https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

- (( exit_code == 0 )) is more robust than [ \$exit_code -eq 0 ] in zsh - \$* shows full command (e.g. "sleep 10") instead of just first arg https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

nyatinte · 2026-04-06T13:29:49Z

@copilot 機能のデグレない？

Copilot · 2026-04-06T13:31:22Z

@copilot 機能のデグレない？

機能のデグレはありません。全項目を照合した結果：

core/path.zsh — 元の path.zsh と同一（typeset -U の重複排除で dev-tools.zsh 側の $PROTO_HOME/tools/node/globals/bin の再追加も吸収済み）
tools/proto.zsh — eval "$(proto activate zsh --no-shim)" + PROTO_AUTO_INSTALL=true と同一
tools/docker.zsh — Docker init スクリプトの条件ロードと同一
tools/fzf.zsh — fzf 設定・キーバインドはすべて同一
core/env.zsh — VISUAL, EDITOR, NI_*, LG_CONFIG_FILE すべて同一
functions/git.zsh — gwn 関数と同一

意図的な変更（改善）は2点：

vite-plus — . "$HOME/.vite-plus/env"（無条件）→ [[ -f ... ]] && source（存在確認付き）。元のコードはファイル不在時にシェル起動がエラー終了するバグがありました
notify-after — 通知本文が $1（コマンド名のみ）→ $*（コマンド全体）に修正。こちらも元がバグでした

claude added 2 commits April 6, 2026 13:06

nyatinte changed the title ~~refactor(zsh): reorganize config into core/, tools/, functions/ directories~~ refactor(zsh): 設定ファイルをディレクトリ構造で再編成 Apr 6, 2026

gemini-code-assist Bot reviewed Apr 6, 2026

View reviewed changes

nyatinte requested a review from Copilot April 6, 2026 13:18

Copilot started reviewing on behalf of nyatinte April 6, 2026 13:19 View session

Copilot AI reviewed Apr 6, 2026

View reviewed changes

claude added 2 commits April 6, 2026 13:23

chore(zsh): replace remove_ files with .chezmoiremove

5d47534

https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

fix(notify): use (( )) for numeric check and \$* for full command string

f8ec604

- (( exit_code == 0 )) is more robust than [ \$exit_code -eq 0 ] in zsh - \$* shows full command (e.g. "sleep 10") instead of just first arg https://claude.ai/code/session_01GkwneqBX3TWvXCVmAsAU33

Copilot started work on behalf of nyatinte April 6, 2026 13:29 View session

Copilot finished work on behalf of nyatinte April 6, 2026 13:32

nyatinte merged commit 93d3e2e into main Apr 6, 2026
2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor(zsh): 設定ファイルをディレクトリ構造で再編成#28

refactor(zsh): 設定ファイルをディレクトリ構造で再編成#28
nyatinte merged 5 commits intomainfrom
claude/review-zsh-config-7wOHq

nyatinte commented Apr 6, 2026 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

gemini-code-assist Bot Apr 6, 2026

Uh oh!

nyatinte Apr 6, 2026

Uh oh!

gemini-code-assist Bot Apr 6, 2026

Uh oh!

nyatinte Apr 6, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Apr 6, 2026

Uh oh!

nyatinte Apr 6, 2026

Uh oh!

Copilot AI Apr 6, 2026

Uh oh!

nyatinte Apr 6, 2026

Uh oh!

nyatinte commented Apr 6, 2026

Uh oh!

Copilot AI commented Apr 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

nyatinte commented Apr 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

変更前

変更後

機能差分

その他の改善点

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

nyatinte Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

nyatinte Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

nyatinte Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

nyatinte Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

nyatinte commented Apr 6, 2026

Uh oh!

Copilot AI commented Apr 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

nyatinte commented Apr 6, 2026 •

edited

Loading