Skip to content

Fix stalled daily release Dependabot alert collection#53

Open
thinksyncs wants to merge 4 commits intomainfrom
codex/fix-daily-release-dependabot-alerts
Open

Fix stalled daily release Dependabot alert collection#53
thinksyncs wants to merge 4 commits intomainfrom
codex/fix-daily-release-dependabot-alerts

Conversation

@thinksyncs
Copy link
Copy Markdown
Contributor

@thinksyncs thinksyncs commented Apr 26, 2026

Summary

  • prevent Daily Release from failing when GITHUB_TOKEN cannot read Dependabot alerts
  • support optional DEPENDABOT_ALERTS_TOKEN for full Dependabot alert reporting
  • include a status asset and release-note status when alert collection is unavailable
  • reduce scheduled CI cost by moving daily scheduled workflows and Dependabot version updates to weekly cadence
  • keep PR/push quality gates, while limiting scheduled Build to ubuntu-latest instead of the full Ubuntu/macOS/Windows matrix
  • detect .env-style secret-bearing files by filename during workspace safety assessment
  • separate .env template files such as .env.example and .env.sample from secret-bearing files
  • surface env scan truncation so users can tell when detection was incomplete
  • confirm broad git operations and block publish operations when secret-bearing files or incomplete env scans are present
  • add fixture-backed coverage for real .env file layouts, ignored directories, templates, and scan limits

Verification

  • actionlint .github/workflows/build.yml .github/workflows/codeql.yml .github/workflows/daily-release.yml .github/workflows/dependency-maintenance.yml .github/workflows/security.yml
  • ruby -e "require 'yaml'; Dir['.github/workflows/*.yml'].each { |f| YAML.load_file(f) }; YAML.load_file('.github/dependabot.yml'); puts 'yaml ok'"
  • npm test
  • npm run build

Copilot AI review requested due to automatic review settings April 26, 2026 04:55
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the Daily Release GitHub Actions workflow to avoid failing when Dependabot alerts can’t be fetched with the default token, while still supporting full reporting when an explicit Dependabot token is provided.

Changes:

  • Make Dependabot alert collection optional via DEPENDABOT_ALERTS_TOKEN, and gracefully fall back to empty alerts + warning/status output when unavailable.
  • Add a dependabot-alerts-status.txt artifact and include status messaging in generated daily release notes.
  • Upload the Dependabot status file as part of the daily release assets.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/daily-release.yml Adds optional Dependabot token handling, records availability status, and includes status in notes/assets to prevent workflow failure.
.beads/issues.jsonl Records the task closure entry for fixing the stalled daily release automation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 186 to 189
DEP_TOTAL=$(jq 'length' dependabot-alerts.json)
DEP_HIGH=$(jq '[.[] | select(.security_advisory.severity == "high" or .security_advisory.severity == "critical")] | length' dependabot-alerts.json)
DEP_MEDIUM=$(jq '[.[] | select(.security_advisory.severity == "medium")] | length' dependabot-alerts.json)
DEP_LOW=$(jq '[.[] | select(.security_advisory.severity == "low")] | length' dependabot-alerts.json)
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.

2 participants