Skip to content

fix: use real project name in notify footer for orchestrator sessions - #38

Merged
nananek merged 1 commit into
masterfrom
fix/notify-footer-project-name
Aug 11, 2026
Merged

fix: use real project name in notify footer for orchestrator sessions#38
nananek merged 1 commit into
masterfrom
fix/notify-footer-project-name

Conversation

@nananek

@nananek nananek commented Aug 11, 2026

Copy link
Copy Markdown
Owner

概要

Discord webhook 通知のフッタにオーケストレータセッションのハッシュ (プロジェクトパスの sha256 24hex) が表示されていた問題を修正。フッタには実プロジェクト名 (cwd の basename) を表示する。

原因

  • 通知フッタの project ラベルは identity.projectName (sessionManager.js で basename(cwd) として導出) を使う
  • コンボグループのオーケストレータの cwd は実プロジェクトではなく、パスから導出したハッシュディレクトリ (~/.local/share/ccserver-sandbox/orchestrator/<sha256 24hex>, orchestratorDirForCwd) であるため、basename がハッシュになる

変更内容

  • server/ws/sessionManager.js: createSessionprojectName オプションを追加。notifyIdentity を projectName ?? basename(cwd) に変更
  • server/routes/groups.js: オーケストレータ生成 (初回・restart 両方) で実プロジェクト cwd の basename を projectName として渡す (restart は cwd 欠損ガード付き)
  • server/ws/terminal.js: ブラウザからの再 init 経路で getGroup(groupId)?.cwd の basename を渡す
  • テスト追加: sessionManager.test.js (explicit projectName 優先 / フォールバック), groups.test.js (restart opts の projectName / cwd 欠損)

影響範囲

  • mcpConfig.js / sandbox-mcp-wrapper.cjs / notify.js / mcpBroker.js は変更不要 (identity は既に projectName を運ぶ)
  • ワーカー (cwd = 実プロジェクト dir) と単独セッションは従来どおり basename(cwd) のため挙動不変

テスト

  • groups.test.js / sessionManager.test.js ローカルで全パス確認済み (該当5ファイル 66 テスト pass)

@nananek
nananek merged commit ce54cfa into master Aug 11, 2026
3 of 4 checks passed
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.

1 participant