Skip to content

監査から旧い言葉の表を外し、雛形の手順を新しい言葉にする - #36

Merged
massu2002 merged 1 commit into
mainfrom
feature/35-drop-legacy-words
Sep 30, 2026
Merged

massu2002 merged 1 commit into
mainfrom
feature/35-drop-legacy-words

Conversation

@massu2002

Copy link
Copy Markdown
Contributor

関連Issue

Closes #35

変更目的

Custom Properties を標準の言葉へ寄せ終えた(2026-09-30 適用、監査の所見 0 件)ので、監査に残した旧い言葉の表を外し、顧客案件の雛形の手順を新しい言葉にする。

変更内容

  • org_audit.py から LEGACY と寄せる処理を外した。旧い言葉は「標準に無い値」(Warning)になる
  • owner_team に実在する Team の名前が入っていれば知らせる(API で取った Team 一覧と比べる。表を持たない)
  • templates/customer-project/README.md の手順 3 を 8 Property の値にした(稼働前 / 稼働後で production_impact と governance_profile が変わる)

確認結果

  • Issueの完了条件を満たしている
  • 変更箇所の動作確認を実施した
  • 既存機能への影響を確認した
  • 必要なテストが通っている
確認 結果
test_org_audit.py 35 件 OK
判定を 1 か所ずつ壊す(14 通り) すべてテストが落ちた
実 Organization への読み取り実行 所見 0 件のまま。Exception 6 件は不変
他の CI(basic / 混入 / 照合 / 記録 / 秘密情報 / guard 65 件) すべて OK

壊した 14 通りのうち今回の分: Team 名の判定を無効にする / Team 一覧を渡さない / 標準に無い値を通す(旧い言葉が戻る) / lifecycle に development を戻す。

影響範囲

  • 手で実行する org_audit.py の所見。いまの Organization では 0 件のまま
  • 雛形の README(新しい顧客 Repository を作るときの手順)

顧客差分・Coreへの影響(該当する場合)

Reviewerに確認してほしい点

  • 雛形の手順 3 で、稼働前を none / standard にしている点(標準 §8 では customer-project は standard。現場で稼働したら direct / strict)

🤖 Generated with Claude Code

Custom Properties を標準の言葉へ寄せ終えた(2026-09-30 に人が適用し、
定義と値を API で照合、監査の所見 0 件)。寄せる途中だけ旧い言葉を通す
ための LEGACY を外した。旧い言葉は「標準に無い値」(Warning)になる。
定義からも外したので入れられないが、選択肢を戻されたときに気づける。

owner_team に Team の名前を使わない決定は、表を外すと確かめられなく
なる。API で取った Team 一覧と比べ、Team の名前が入っていれば知らせる
ようにした。Team が増えても表を直さなくてよい。

顧客案件の雛形の手順 3 が旧い言葉(repo_type=customer、
lifecycle=development)のままだったので、8 Property の値にした。稼働前と
稼働後で production_impact と governance_profile が変わることも書いた。

実測: テスト 35 件 OK。判定を壊す 14 通りすべてで落ちた。実 Organization
への読み取り実行で所見 0 件のまま。

Closes #35

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@massu2002
massu2002 merged commit c62c402 into main Sep 30, 2026
1 check passed
@massu2002
massu2002 deleted the feature/35-drop-legacy-words branch September 30, 2026 06:23
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.

[Feature] 監査から旧い言葉の表を外し、雛形の手順を新しい言葉にする

1 participant