Skip to content

feat: security-checklist に permissions: {} (deny by default) パターンを追加 #24

@yk-lab

Description

@yk-lab

背景

PR #21 で全ワークフローに permissions: {} (deny by default) + ジョブレベル permissions 宣言パターンを適用した。
これは zizmor の excessive-permissions 指摘を解消する最も安全なパターンで、audit-actions スキルでも推奨すべき。

現状

  • security-checklist のチェック CI に LLM ベースのスキル品質チェックを追加 #1 は「Top-level permissions」の存在確認のみ
  • permissions: {} + ジョブレベル宣言という deny by default パターン への言及がない
  • コメントによる理由明記(zizmor の undocumented-permissions 対策)の推奨もない

対応

参考

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions