Repository navigation
Install link moderation permissions, automatic production deploys, verifier readiness wait - #54
Merged
Conversation
…link The homepage install link asked only for View Channels, Send Messages and Read Message History (68608). The bot refuses delete, timeout, kick and ban unless it holds Manage Messages, Moderate Members, Kick Members or Ban Members, and warnings in threads need Send Messages in Threads. New servers therefore installed TypeCheck without the permissions its configured actions require. Request 1374389611526 instead; Discord still lets the installer untick any permission, and Administrator is not requested. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
5e66945 limited Deploy production to manual dispatch, so merged changes stopped reaching GoDaddy unless someone dispatched them. Restore the workflow_run trigger: a successful Validate run for a push to main in this repository deploys that exact commit. Pull request runs, other branches and failed or cancelled runs still never deploy, and the existing guard still ends out-of-order runs as superseded. Manual dispatch remains for redeploys and rollbacks. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ication Deploy run 39061306019 failed although the release was serving: right after the PREVIEW restart handed control to PUBLISH, /api/ready returned 503 not_ready because the new leader's Gateway had not connected yet, and the verifier read readiness once. Poll readiness for up to 90 seconds (within the overall deadline) before recording the result. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
68608(View Channels, Send Messages, Read Message History). The bot refuses delete, timeout, kick and ban unless it holds Manage Messages, Moderate Members, Kick Members or Ban Members, and warnings in threads need Send Messages in Threads. The link now requests1374389611526. Administrator is still not requested, and Discord lets the installer untick any permission.5e66945restricted Deploy production to manual dispatch. This restores theworkflow_runtrigger: a successfulValidaterun for a push tomainin this repository deploys that exact commit. PR runs, other branches and failed/cancelled runs never deploy; the existing guard still ends out-of-order runs assuperseded; manual dispatch remains for redeploys and rollbacks./api/readyreturned503 not_readywhile the new leader's Gateway connected, and the verifier read readiness only once. It now polls for up to 90 s. (The other recent failure, run 39063785246, was the PREVIEW restart 404 already fixed by9530953, which landed after that run.)Validation
npm run checkclean;npm test379/379 pass.SHA256SUMSrefreshed and verified against the staged tree.actionlintandzizmorclean ondeploy.yml.scripts/verify-production.mjsrun read-only against live production: all checks pass.Note
Merging this PR triggers the first automatic production deploy, of
mainincluding PRs #48–#53.🤖 Generated with Claude Code