Update workflow: Update Vale vocabulary#4486
Update workflow: Update Vale vocabulary#4486mintlify-development[bot] wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| - "ae-dinner-approve" | ||
| - "404-roast" | ||
| - "admin" | ||
| - "add-a-doc" |
There was a problem hiding this comment.
Wrong Slack notification key breaks notification delivery
High Severity
The channels key under slack is inconsistent with every other workflow in the repository, which all use channel_ids. This likely means Slack notifications for this workflow will silently fail. Additionally, the values are channel names (e.g., "ae-dinner-approve", "404-roast") rather than Slack channel IDs (e.g., C0AKYE83VV4), which is inconsistent with the established pattern and suggests these may be incorrect test/placeholder values.
| push: | ||
| - repo: "mintlify/docs" | ||
| branch: "main" | ||
| cron: "0 */6 * * *" |
There was a problem hiding this comment.
Cron trigger may skip PRs between runs
Medium Severity
The trigger changed from push-based (once per merged PR) to cron-based (every 6 hours), but the workflow body still instructs to process only "the last merged PR." When multiple PRs merge within a 6-hour window, only the most recent one will be processed, and vocabulary words from all other merged PRs will be silently skipped. Every other workflow that references "the last merged PR" uses a push trigger, where the phrasing is correct.


Updates the workflow configuration at
.mintlify/workflows/update-vale-vocab.md.0 0 * * *0 */6 * * *Note
Low Risk
Low risk configuration-only change that adjusts workflow scheduling/notifications; no application logic or data handling is modified.
Overview
Switches the
Update Vale vocabularyworkflow from a push-based trigger to acronschedule (every 6 hours) and enablesautomerge.Updates Slack notifications to use a
channelslist (and adds several channels) instead ofchannel_ids.Written by Cursor Bugbot for commit 8e1a6f3. This will update automatically on new commits. Configure here.