fix(lint): PSAvoidAssignmentToAutomaticVariable 15件を修正 (closes #128)

[Kensan196948G]

Summary

• PSScriptAnalyzer の PSAvoidAssignmentToAutomaticVariable 15件（6ファイル）を修正
• $args, $Host, $profile, $error, $Args への代入を安全な名前に変更
• Message Bus 設計書 (.claude/claudeos/system/message-bus-design.md) を追加（Issue [P3] v8 後続フェーズ: Message Bus パターンの設計検討 #127 対応）

変更ファイル

ファイル	変更内容
scripts/lib/IssueSyncManager.psm1	$args → $ghArgs
scripts/lib/SSHHelper.psm1	param $Host → $HostName（_Show-SSHDiagnostics 内部のみ、公開 I/F 互換性維持）
scripts/main/Start-Menu.ps1	$args → $scriptArgs
scripts/setup/setup-windows-terminal.ps1	$profile → $terminalProfile（全箇所、return $profile 漏れを含む回帰修正）
scripts/test/Test-AllTools.ps1	$Args → $CmdArgs、$error → $errItem
tests/Diagnostics.Tests.ps1	テスト内の $profile → $terminalProfile（5ブロック）

Test plan

• PSScriptAnalyzer: 0 findings (Error severity: 0)
• Pester 449/449 tests pass (Tests Passed: 449, Failed: 0)
• Diagnostics.Tests.ps1 35/35 pass（return $profile 回帰を確認・修正済み）

影響範囲

• 自動変数への代入は実行エンジン内部状態を破壊する可能性があるため、修正は安全性向上
• 公開 API（Test-SSHConnection -Host）の引数名は変更なし（外部互換性維持）

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

• Documentation

  • マルチエージェント通信を最適化するための新しい設計ドキュメントを追加しました。

• Refactor

  • スクリプト全体の変数名とパラメータ名を改善し、コードの一貫性と可読性を向上させました。

• Tests

  • テストのコード品質を向上させるため、変数名を統一しました。

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

ClaudeOS v8 マルチエージェント調整のためのメッセージバス設計ドキュメントを追加し、複数のPowerShellスクリプトとテストの自動変数への割り当てを回避するために変数・パラメータ名を体系的にリネームしました。

Changes

Cohort / File(s)	Summary
Documentation .claude/claudeos/system/message-bus-design.md, TASKS.md	メッセージバス設計（トピック定義、schema、実装フェーズ、受け入れ基準）のドキュメント追加、および末尾の空行追加。
PowerShell Script Variable Renames scripts/lib/IssueSyncManager.psm1, scripts/lib/SSHHelper.psm1, scripts/main/Start-Menu.ps1, scripts/setup/setup-windows-terminal.ps1, scripts/test/Test-AllTools.ps1	PowerShellの自動変数（$args、$profile、$error、$Host）への割り当てを回避するため、複数のスクリプトで変数・パラメータをリネーム（例：$args → $ghArgs/$scriptArgs、$profile → $terminalProfile、$Host → $HostName、$Args → $CmdArgs、$error → $errItem）。
Test Variable Renames tests/Diagnostics.Tests.ps1	テスト内でWindows Terminalプロファイルオブジェクト参照を$profileから$terminalProfileにリネーム、すべての関連アサーションを更新。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related issues

• Issue #127: ClaudeOS v8メッセージバス設計の定義と状態管理アーキテクチャの選定に関連。
• Issue #128: 複数スクリプトにおけるPowerShell自動変数への割り当て回避（PSAvoidAssignmentToAutomaticVariable）リファクタリングに関連。

Poem

📬✨ ウサギのメッセージバス、ぴょん！
自動変数もお片付け、ホップ！
競合なく、順序も確か、
ClaudeOSの調整、スムーズにぴょい〜🐰

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PRタイトルは、PSAvoidAssignmentToAutomaticVariable 15件の修正という主要な変更内容を正確に表現しており、変更セットの主要な目的を明確に要約しています。
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/psscriptanalyzer-automatic-variables-issue128

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

PSScriptAnalyzer の PSAvoidAssignmentToAutomaticVariable 指摘（$args/$Host/$profile/$error/$Args への代入）を解消しつつ、ClaudeOS v8 の Message Bus パターン設計書を追加する PR です。

Changes:

• PowerShell スクリプト/テストで自動変数名への代入を安全な変数名へリネーム
• Windows Terminal プロファイル生成/更新処理の変数名修正（return $profile 漏れの回帰も解消）
• Message Bus 設計書（Draft）を .claude/claudeos/system/ に追加

Reviewed changes

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

Show a summary per file

File	Description
scripts/lib/IssueSyncManager.psm1	gh 呼び出し引数配列を $args → $ghArgs に変更
scripts/lib/SSHHelper.psm1	内部診断ヘルパーの param $Host を $HostName に変更し、自動変数衝突を回避
scripts/main/Start-Menu.ps1	メニュー起動引数の配列を $args → $scriptArgs に変更
scripts/setup/setup-windows-terminal.ps1	プロファイルオブジェクト変数を $profile → $terminalProfile に変更し、返却も修正
scripts/test/Test-AllTools.ps1	Get-CommandVersionLine の $Args → $CmdArgs、foreach の $error → $errItem に変更
tests/Diagnostics.Tests.ps1	テスト内の $profile を $terminalProfile に置換
TASKS.md	末尾に空行が追加
.claude/claudeos/system/message-bus-design.md	Message Bus の設計書を新規追加

[Copilot]

末尾に空行が大量に追加されていますが、タスク内容に意味のある変更がなく diff ノイズになります。不要であれば削除して、TASKS.md の更新は実タスク追加・状態変更時のみに絞ってください。

[Copilot]

参照セクションのパス表記が、同ディレクトリ内ドキュメントで多い system/orchestrator.md / system/role-contracts.md 形式と異なるため、読む側が実ファイル位置を追いにくいです。system/... など既存ドキュメントに合わせた表記に統一することを検討してください。

Suggested change

	- `claudeos/system/orchestrator.md` — Orchestrator の状態管理責務
	- `claudeos/system/role-contracts.md` — Agent 間協調パターン
	- `system/orchestrator.md` — Orchestrator の状態管理責務
	- `system/role-contracts.md` — Agent 間協調パターン