Skip to content

ci: bypass L1/Issue/NOW gates for trusted bots (Closes #1059) - #1060

Merged
gHashTag merged 2 commits into
masterfrom
fix/dependabot-gate-bypass
Jun 8, 2026
Merged

ci: bypass L1/Issue/NOW gates for trusted bots (Closes #1059)#1060
gHashTag merged 2 commits into
masterfrom
fix/dependabot-gate-bypass

Conversation

@gHashTag

@gHashTag gHashTag commented Jun 8, 2026

Copy link
Copy Markdown
Owner

What

Adds if: github.actor != 'dependabot[bot]' && github.actor != 'github-actions[bot]' to three job-level guards:

  • .github/workflows/l1-traceability.yml
  • .github/workflows/issue-gate.yml
  • .github/workflows/now-sync-gate.yml

Why

Dependabot PRs #1057 (rusqlite) and #1058 (chrono) failed all three gates because bots cannot reference issues or update docs/NOW.md for routine dep bumps.

Failing runs:

Effect

  • Trusted bot PRs: gates SKIPPED (not failed)
  • Human PRs: gates remain fully enforced
  • L1 summary job now prints Skipped (trusted bot actor) for clarity

Acceptance

After merge, rerun #1057 and #1058 -- all three gates should show 'skipped' instead of 'failed'.

Closes #1059

Closes #1059

Dependabot PRs (#1057 rusqlite, #1058 chrono) fail three gates by design:
- L1 TRACEABILITY (commits need Closes #N)
- Issue Gate (PR body needs Closes|Fixes|Resolves #N)
- NOW Sync Gate (docs/NOW.md must change)

Bots maintain deps -- not features -- and cannot reference issues.

Adds job-level guard:
  if: github.actor != 'dependabot[bot]' && github.actor != 'github-actions[bot]'

Effect: gates are SKIPPED (not FAILED) for trusted bot actors.
Constitutional Law L1 still enforced for all human PRs.

The L1 summary job now reports 'Skipped (trusted bot actor)' for clarity.
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-06-08 10:07:57 UTC

Summary

Status Count
Total Open PRs 10
PRs with Failing Checks 8
PRs with All Checks Green 2
READY 1
FAILING 8
PENDING 0

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-06-08 10:14:58 UTC

Summary

Status Count
Total Open PRs 10
PRs with Failing Checks 8
PRs with All Checks Green 2
READY 1
FAILING 8
PENDING 0

Closes #1059

NOW Sync Gate requires docs/NOW.md to be updated in every PR to master.
This entry documents the bot-bypass change introduced by the first commit
on this branch (2eecb39 ci: bypass L1/Issue/NOW gates for trusted bots).
@gHashTag
gHashTag force-pushed the fix/dependabot-gate-bypass branch from c5ee9b1 to 3f4af70 Compare June 8, 2026 10:32
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

📓 NotebookLM Notebook linked to this PR

This notebook contains session context, decisions, and artifacts for this work.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

PR Dashboard

Generated at: 2026-06-08 10:32:20 UTC

Summary

Status Count
Total Open PRs 10
PRs with Failing Checks 8
PRs with All Checks Green 2
READY 1
FAILING 8
PENDING 0

@gHashTag
gHashTag merged commit 06671c2 into master Jun 8, 2026
21 checks passed
@gHashTag
gHashTag deleted the fix/dependabot-gate-bypass branch June 8, 2026 10:34
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.

CI: bypass L1 TRACEABILITY / Issue Gate / NOW Sync Gate for Dependabot PRs

1 participant