Skip to content

docs(cicd): standardize self-references on @main + scanner posture#89

Open
JacobPEvans-personal wants to merge 4 commits into
mainfrom
docs/reusable-workflow-main-pin
Open

docs(cicd): standardize self-references on @main + scanner posture#89
JacobPEvans-personal wants to merge 4 commits into
mainfrom
docs/reusable-workflow-main-pin

Conversation

@JacobPEvans-personal

Copy link
Copy Markdown
Member

What

Documents the convention that dryvist/* reusable workflows (including
ai-workflows) are pinned at @main, and explains why that is safe — the
org-wide scanner overrides shipped in
dryvist/.github#52.

Changes

  • infrastructure/cicd/policy.mdx
    • Dependency-versioning table: tightened the self-reference row from
      "@main or a major version tag" to @main only, with a pointer to the
      new posture section.
    • New Scanner posture for self-references subsection: Renovate
      pinDigests: false for dryvist/**, zizmor dryvist/*: ref-pin, a
      pre-staged CodeQL actions/unpinned-tag exclude, and OSV (N/A) — linking to
      dryvist/.githubSECURITY.md.
  • automation/cloud-pipelines/ai-workflows.mdx — Versioning section now
    states consumers pin @main (never a SHA or version pin), with the
    scanner-posture link.

Verification

  • Internal anchor #scanner-posture-for-self-references matches Mintlify's slug
    for the new heading. Mintlify build (CI) validates links/anchors.

🤖 Generated with Claude Code

- Dependency-versioning table: tighten the self-reference row to `@main`
  (drop "or a major version tag") and point to the new scanner posture.
- New "Scanner posture for self-references" subsection: how each scanner is
  configured to allow `dryvist/*@main` — Renovate `pinDigests: false`,
  zizmor `dryvist/*: ref-pin`, a pre-staged CodeQL `actions/unpinned-tag`
  exclude, and OSV (N/A) — linking to dryvist/.github SECURITY.md.
- ai-workflows Versioning section: state that consumers pin `@main` (never a
  SHA or version pin) and link to the scanner posture.

Documents the org-wide overrides shipped in dryvist/.github#52.

Assisted-by: Claude:claude-opus-4-8
Claude-Session: https://claude.ai/code/session_019mTayMHweSbSdF4ug83FhB

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the CI/CD policy and AI workflows documentation to mandate pinning self-referenced reusable workflows (such as dryvist/* and JacobPEvans/) to @main instead of a SHA or version tag, and details the scanner posture configurations that support this. Feedback was provided to address an inconsistency where the scanner posture details only mention dryvist/ overrides, suggesting they should also include JacobPEvans/* to align with the policy table.

Comment thread infrastructure/cicd/policy.mdx
Match dryvist/.github#52: CodeQL is managed via the org Security
Configuration (public repos, free) — no committed codeql-config.yml. Same-org
@main false positives are dismissed natively in the code scanning UI.

Assisted-by: Claude:claude-opus-4-8
Claude-Session: https://claude.ai/code/session_019mTayMHweSbSdF4ug83FhB
@mintlify

mintlify Bot commented Jun 29, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
jacobpevans 🟢 Ready View Preview Jun 29, 2026, 12:42 AM

💡 Tip: Enable Workflows to automatically generate PRs for you.

…onfiguration

Align with dryvist/.github#52 / dryvist/tofu-github#18: code scanning default
setup is managed per-repo as IaC in tofu-github (pending the provider resource),
not via an org-level Security Configuration.

Assisted-by: Claude:claude-opus-4-8
Claude-Session: https://claude.ai/code/session_019mTayMHweSbSdF4ug83FhB
Review feedback (gemini-code-assist on #89): the self-reference table listed
both dryvist/* and JacobPEvans/*, but the Renovate/zizmor overrides are
dryvist/*-only. Clarify that the scanner overrides target dryvist/* action
sources; JacobPEvans/* follows the same @main convention (overrides not
expanded — out of scope for this dryvist-focused change).

Assisted-by: Claude:claude-opus-4-8
Claude-Session: https://claude.ai/code/session_019mTayMHweSbSdF4ug83FhB
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