Skip to content

~/.mainstaller の設定サンプルを同梱 (#181) - #191

Open
k-yoshimi wants to merge 3 commits into
developfrom
feat/181-mainstaller-sample
Open

~/.mainstaller の設定サンプルを同梱 (#181)#191
k-yoshimi wants to merge 3 commits into
developfrom
feat/181-mainstaller-sample

Conversation

@k-yoshimi

Copy link
Copy Markdown
Collaborator

概要

Fixes #181

~/.mainstaller の設定サンプル dot.mainstaller.example をリポジトリ直下に追加し、README と how_to_use マニュアル(en/ja)から参照します。

  • dot.mainstaller.example: MA_ROOT / BUILD_DIR / SOURCE_DIR を既定値付きで例示。set_prefix が読む残りの変数(MALIVE_REPOSITORYMAINSTALLER_CONFIG による設定ファイル切替)もコメントで記載。= の前後に空白を置かない・パスに空白を含めない、という制約も明記
  • README / docs から参照し、sh check_prefix.sh で現在の解決結果を確認できることも案内

検証

  • sh -n dot.mainstaller.example(構文チェック)通過
  • サンプルを一時 ~/.mainstaller としてコピー → MAINSTALLER_CONFIG 経由で set_prefix に読ませ、print_prefix が期待どおり MA_ROOT/BUILD_DIR/SOURCE_DIR を解決することを確認

補足(マージ順)

README と how_to_use (en/ja) は #178#186 も同じセクションを編集しているため軽微な競合が出る可能性があります(いずれも自明に解消できます)。

🤖 Generated with Claude Code

Ship a commented template of the ~/.mainstaller configuration file
covering MA_ROOT/BUILD_DIR/SOURCE_DIR (and the rarely needed
MALIVE_REPOSITORY / MAINSTALLER_CONFIG), and reference it from the
README and the how_to_use manual (en/ja) together with the
check_prefix.sh hint.

Fixes #181

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@k-yoshimi
k-yoshimi changed the base branch from master to develop July 7, 2026 23:44
k-yoshimi and others added 2 commits July 8, 2026 08:56
The header said the value after '=' is the default, but the commented
MALIVE_REPOSITORY line showed https while the built-in default is
http. Scope the "default" wording to the three directory variables and
label MALIVE_REPOSITORY explicitly as an optional override.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Fix the long-standing "direcoty" typo in the Quick Usage heading and
state that dot.mainstaller.example lives in the repository root, to
match the how_to_use manual wording.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@k-yoshimi

Copy link
Copy Markdown
Collaborator Author

AI review cycle (Codex + Antigravity) 結果

Phase 1 (Codex): must_fix 0 / should_fix 1 → 対応: MALIVE_REPOSITORY の例が util.sh の既定(http)に対し https で「= の右が既定値」の記述と矛盾する、との指摘 → 「既定値」の記述を 3 つのディレクトリ変数に限定し、MALIVE_REPOSITORY は「任意の上書き」と明記。

Phase 2 (Antigravity): must_fix_docs 0 / should_fix 3 → 対応:

  • README 見出しの既存 typo direcotydirectory を修正
  • dot.mainstaller.example が「リポジトリ直下」にある旨を README にも明記(マニュアルと表現統一)
  • echo vs cp の「不整合」指摘は意図的に併存(README Quick Usage は最小の echo 一行+テンプレ参照、マニュアルは cp)で矛盾ではないと判断
  • workflow_risk「cp すると全変数がコメントアウト」は事実誤認: MA_ROOT/BUILD_DIR/SOURCE_DIR は既定値付きで有効なので cp でそのまま動く(MAINSTALLER_CONFIG 経由で source し set_prefix/print_prefix が解決することを実証)
  • 空白ルールの注記は example 本文+共有スクリプトのクォート修正 + shellcheck の CI 導入 #178(whitespace 注記)でカバー済み

最終 delta(テキストのみ)を Codex 再確認済み・指摘ゼロ。

→ 両フェーズ収束。

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.

[UX] ~/.mainstaller の設定サンプルを同梱する

2 participants