Skip to content

skills: git の状態が不明のとき「変更あり」ではなく「不明」と表示する #3445

Description

@Kewton

事象

#3435(PR #3440)で、git status を読めなかったときに getGitStatus が clean を返さないよう statusUnknown を足した。skills の画面の写し(src/lib/skills/preview-diff.ts の readSkillGitTargetState)は、不明を dirty 側(警告を出す側)に倒しただけで、「不明」を区別して表示できない。

workingTreeDirty は boolean のまま次へ渡っている:

  • src/lib/skills/install-plan.ts / update-plan.ts / uninstall-plan.ts
  • src/cli/commands/skill-format.ts(CLI の表示)

そのため、負荷で git status が読めなかったときも「未コミットの変更がある」と表示され、利用者は実際には変更が無いのに確かめに行くことになる。#3435 のワーカーが範囲外として報告した。

受入基準

  • skills の計画(install / update / uninstall)と CLI の表示が、「変更あり」と「不明」を区別して出す(文言は ja / en の両方)
  • 不明のときも警告の側に倒す動き(安全側)は保つ。clean と表示しない
  • 型を変えるなら写し(src/cli/types/、API の応答型)も合わせ、既存の読み手を壊さない
  • 成功時(clean / dirty)の表示は従来どおり。既存のテストは変えずに通る
  • npm run lint・npx tsc --noEmit・対のテストが通る

Activity

  1. Kewton commented on Oct 8, 2026

    @Kewton
    OwnerAuthor

    PR #3456(develop マージ済み・本番サーバーに反映済み)で対応しました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions