fix(v3.2.97): S1 起動時 ~/パス修正 + statusline.js 自動デプロイ - #255
Merged
Merged
Conversation
New-RemoteTemplateDeployScript で bash double-quote 内 ~ が展開されない バグを修正(~ → $HOME 変換)。cron-launcher.sh が ~/~/.claudeos/ に 誤配置される問題を解消。S1 起動時に statusline.js を ~/.claude/ へ InitializeOnly でデプロイするよう deployParts に追加。 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughS1 SSH起動時のテンプレートデプロイにおいて、bash での Changes
Sequence Diagram(s)sequenceDiagram
participant PowerShell as PowerShell<br/>Start-ClaudeCode
participant SSH as SSH Connection
participant Remote as Remote Host
participant StatusLine as statusline.js
PowerShell->>SSH: 1. SSH接続確立
PowerShell->>SSH: 2. テンプレートscripts/templates/statusline.js<br/>をリモートへ送信
SSH->>Remote: 3. InitializeOnlyで~/.claude/statusline.js<br/>へデプロイ(既存時は上書き不可)
PowerShell->>SSH: 4. デプロイ用bashスクリプト生成<br/>(~をすべて$HOMEに置換)
SSH->>Remote: 5. bash実行環境で$HOME展開<br/>(正しいパスでリソース配置)
Remote->>StatusLine: 6. S1起動時にJSON入力で呼び出し
StatusLine-->>Remote: 7. ステータスライン出力<br/>(model/branch/context/レート制限)
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
S1(SSH モード)起動時のリモートテンプレート配布を安定させるために、bash の double-quote 内で ~ が展開されない問題を回避しつつ、statusline.js を初回のみ自動配布できるようにするPRです。
Changes:
New-RemoteTemplateDeployScriptで~/を$HOME/に正規化し、リモートでの誤配置を防止- S1 起動時に
statusline.jsを~/.claude/statusline.jsへ-InitializeOnlyで自動デプロイ - README / CHANGELOG を v3.2.97 向けに更新
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| scripts/templates/statusline.js | statusline 用の Node.js スクリプトテンプレートを追加 |
| scripts/main/Start-ClaudeCode.ps1 | SSH 配布時の ~/ 正規化($HOME/)と statusline.js の初回配布を追加 |
| README.md | v3.2.97 の変更点に合わせてトップとバージョン表記を更新 |
| CHANGELOG.md | v3.2.97 の変更内容を追記 |
Comment on lines
+34
to
+38
| // Green for filled, dim gray for empty | ||
| let color = C.green; | ||
| if (pct >= 80) color = C.red; | ||
| else if (pct >= 50) color = C.yellow; | ||
| return color + "\u25b0".repeat(filled) + C.cyan + "\u25b1".repeat(empty) + C.r; |
|
|
||
| | バグ | 原因 | 修正 | | ||
| |---|---|---| | ||
| | `~/.claudeos/cron-launcher.sh` が `~/~/.cloudeos/` に誤配置 | bash double-quoteの中では `~` は展開されない | `~` → `$HOME` に変換するロジック追加 | |
| @@ -0,0 +1,145 @@ | |||
| // Claude Code status line script (Windows / Node.js) | |||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
~が展開されないバグを修正 —New-RemoteTemplateDeployScriptで~を\$HOMEに変換cron-launcher.shが/home/kensan/~/.claudeos/ではなく/home/kensan/.claudeos/に正しく配置されるよう修正~/ディレクトリ(/home/kensan/~/)は手動で削除済みstatusline.jsを~/.claude/へ自動デプロイ(-InitializeOnly)scripts/templates/statusline.jsを新規追加Test plan
node scripts/check-doc-versions.jsPASSED (README=v3.2.97)New-RemoteTemplateDeployScriptで~/→\$HOME/に変換されること確認~/.claude/statusline.jsが存在すること確認🤖 Generated with Claude Code
Summary by CodeRabbit
Release Notes v3.2.97
Bug Fixes
Documentation