Repository navigation
Organization 監査が標準の Custom Properties の言葉で判定する - #34
Merged
Merged
Conversation
Custom Properties を標準（v1.2 §6）の言葉へ寄せることが決まった。監査は 旧い言葉の組（LIFECYCLE_BY_TYPE）しか知らず、足す 5 つの Property も、 その組も確かめられなかった。寄せる前に監査側を直す。 標準の 8 Property と取りうる値を STANDARD の 1 箇所に置いた。旧い言葉は INFO「寄せる途中」として数え、組の判定は寄せた先の意味で行う。寄せる途中 でも strict の判定などが止まらない。Organization に定義の無い Property は 1 度だけ数え、定義が取れないときは「無い」と言わない。 組の判定を足した。product-core / platform / production_impact=direct は strict（§8）。Public な Repository は data_classification=public で、顧客の 分類が付いていれば Critical。core_version に値があれば知らせる（版の正本は versions.yaml）。 archived の判定は、archived を先に飛ばしていたため一度も動いていなかった。 先に判定するようにした。 実測: テスト 38 件 OK。決めた値の表で所見 0 件、いまの実測値で「寄せる 途中」18 件と「定義が無い」5 件だけ。判定を壊す 13 通りすべてで落ちた。 実 Organization への読み取り実行も同じ 23 件（Critical 0、Exception 6 件は 不変）。 Closes #33 Co-Authored-By: Claude Opus 5.5 <[email protected]>
4 tasks done
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.
関連Issue
Closes #33
変更目的
Custom Properties を標準（v1.2 §6）の言葉へ寄せる前に、Organization 監査が新旧どちらの言葉も正しく判定できるようにする。寄せたかどうかを、目で表と見比べずに「所見 0 件」で確かめられるようにする。
変更内容
STANDARDの 1 箇所に置いた（unclassifiedを残し、data_classificationにpublicを足す。決定どおり）coredevelopmentdevelopersなど）は INFO「寄せる途中」。組の判定は寄せた先の意味で行うproduct-core/platform/production_impact=directは strict（§8）、Public はpublic（顧客の分類なら Critical）core_versionに値があれば知らせる（版の正本はversions.yaml）LIFECYCLE_BY_TYPEは外した。development / production の区別はproduction_impactが持つ確認結果
test_org_audit.py壊した 13 通り: 寄せた値を組に渡さない / 旧い言葉の後も判定を続ける / archived の判定を元に戻す / Public の顧客分類を WARNING に下げる / 定義の無い Property を Repository ごとに数える / 取れない定義を「無い」と扱う / 定義が無いことを数えない / owner_team の選択肢を閉じる / direct で strict を求めない / 版の Property を見ない / Private にも public を求める / unclassified を見ない / 寄せ先を誤る。
影響範囲
org_audit.pyの所見。適用前は 23 件出る（寄せる途中の一覧として読む）test_org_audit.pyを実行するだけで、Organization は叩かない。設定は変えない（GET のみ）顧客差分・Coreへの影響（該当する場合）
Reviewerに確認してほしい点
internalが付いている場合を WARNING、顧客の分類を Critical にした線引き🤖 Generated with Claude Code