Skip to content

docs(security): open the SECURITY.md monthly auto-update PR (CI can't create it) - #546

Merged
logbie merged 1 commit into
mainfrom
auto/update-security-doc-1782867320
Jul 2, 2026
Merged

logbie merged 1 commit into
mainfrom
auto/update-security-doc-1782867320

Conversation

@logbie

@logbie logbie commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

What was broken

The scheduled Update SECURITY.md workflow (update-security-doc.yml) has failed on the 1st of every month — most recently run 28485935619 on 2026-07-01, and identically on 06-01, 05-01, 04-01, 03-01, and 02-01. The regeneration script runs fine and the branch pushes successfully; the run dies at the final Create Pull Request step with:

##[error]GitHub Actions is not permitted to create or approve pull requests.

Root cause

This is a repository/organization setting, not a code bug. The org-level "Allow GitHub Actions to create and approve pull requests" toggle is off (can_approve_pull_request_reviews: false at both repo and org), so the built-in GITHUB_TOKEN used by peter-evans/create-pull-request is blocked from opening the PR. The push half of the action succeeds, which is why an orphaned auto/update-security-doc-* branch is left behind each month.

The fix (this PR)

The branch auto/update-security-doc-1782867320 that the 07-01 run pushed already contains the correct, docs-only regeneration. This PR simply opens it for review so the update isn't lost. The diff is exactly what the workflow intended:

  • Supported-versions table advanced from 26.1.x26.6.x (and limited/unsupported rows shifted accordingly).
  • Footer Last Updated: January 2026 / Version 26.1.35July 2026 / Version 26.6.5.

No source or behavior changes; nothing in TestPrograms/ is affected.

Permanent fix (needs a maintainer — cannot be done from CI)

To stop the monthly failure, enable Settings → Actions → General → Workflow permissions → "Allow GitHub Actions to create and approve pull requests" (repo or org level). Alternatively, pass a PAT secret to the create-pull-request step instead of GITHUB_TOKEN. Until one of those is done, this workflow will keep failing on the 1st and I'll keep opening the PR by hand.

Verification

Diff reviewed via the compare API: 1 commit, SECURITY.md (+5/-5) plus a mode-only touch to scripts/update_security_doc.sh. Docs-only; no local build needed.

Posted by the WFL repo warden (automated triage pass).


Open in Devin Review

Summary by CodeRabbit

  • Documentation
    • Updated the security policy’s supported version ranges to reflect the latest release line.
    • Refreshed the “Last Updated” information with the current date and version.

Copilot AI review requested due to automatic review settings July 1, 2026 09:04
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: ec977405-73c4-47b9-892a-b0aeafe385f8

📥 Commits

Reviewing files that changed from the base of the PR and between a44c9d7 and 698322b.

📒 Files selected for processing (2)
  • SECURITY.md
  • scripts/update_security_doc.sh

📝 Walkthrough

Walkthrough

Updated SECURITY.md to reflect new supported version patterns (current, limited, unsupported cutoff) and refreshed the "Last Updated" metadata to a newer date and version number.

Changes

Security policy version update

Layer / File(s) Summary
Update supported versions and metadata
SECURITY.md
Supported version patterns changed (26.6.x current, 26.5.x limited, 26.4.x and older unsupported) and Last Updated metadata changed to July 2026 / version 26.6.5.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes this docs-only SECURITY.md auto-update PR and its CI-related context.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch auto/update-security-doc-1782867320

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Opens the monthly auto-regenerated SECURITY.md update branch for review, keeping WFL’s published supported-version window and “Last Updated” footer in sync with the current release and the automation that generates it.

Changes:

  • Update the supported-versions table to advance the support window to 26.6.x (current), 26.5.x (limited), and 26.4.x and older (unsupported).
  • Update the footer “Last Updated” month/year and the referenced WFL version.
  • Add/modify the scripts/update_security_doc.sh automation used by the scheduled workflow to regenerate SECURITY.md.

Reviewed changes

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

File Description
SECURITY.md Advances supported-version tiers and updates the footer metadata (month/version).
scripts/update_security_doc.sh Script used by automation to regenerate SECURITY.md from Cargo.toml version data.

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

Comment thread SECURITY.md
Comment on lines +13 to +15
| 26.6.x (Current)| ✅ Yes | Active development, security fixes prioritized |
| 26.5.x | ⚠️ Limited | Critical security issues only |
| 26.4.x and older| ❌ No | No security updates provided |
@logbie

logbie commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator Author

CI is fully green on this PR — 14 checks passed (Build/Test/Clippy, both Integration matrices, Database Tests, Run WFL Programs on ubuntu+windows, CodeQL, fmt, config-lint, CodeRabbit, claude-review), 2 expected skips (Bump Version, claude), zero failures. MERGEABLE, no conflicts. Ready for a maintainer to merge.

Reminder: this only lands the July regeneration. To stop update-security-doc.yml failing on the 1st of every month, enable Settings → Actions → General → Workflow permissions → "Allow GitHub Actions to create and approve pull requests" (currently off at the org level), or switch the create-pull-request step to a PAT secret.

Posted by the WFL repo warden (automated triage pass).

@logbie
logbie merged commit 40d1f64 into main Jul 2, 2026
18 checks passed
@logbie
logbie deleted the auto/update-security-doc-1782867320 branch July 2, 2026 06:42
@coderabbitai coderabbitai Bot mentioned this pull request Aug 14, 2026
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