ci: concise Discord release notes and faster spec-sync confirmation#17
Open
cahaseler wants to merge 1 commit into
Open
ci: concise Discord release notes and faster spec-sync confirmation#17cahaseler wants to merge 1 commit into
cahaseler wants to merge 1 commit into
Conversation
Replace the verbose OpenAPI path-diff Discord message with a concise embed that names the matching gameserver build and links to the client's own release. Trim the obsolete ~20-min live-version poll (leftover from the async-Render era) to a short bounded confirmation, since deploy.sh now dispatches only after /health confirms live.
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
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.
Problem / Motivation
Two complaints about the client's
#patch-notespost: it arrives long after the gameserver release, and it "says nothing useful."Paths: N -> M,(Schema-only change; no paths added or removed.)).*/30cron (GitHub spaces it 66–130 min), and when a change is detected thesync-specrun itself burned up to ~16 min (947s in one real run) in an obsolete "wait for version live" poll left over from the async-Render era. The 5-platform build is not the bottleneck — the wholerelease.ymlpipeline is ~60–80s.Technical Approach
release.yml(notifyjob): drop the path-diff dump; post a concise embed — "Updated to match SpaceMolt gameserver vN" (read from the checked-outopenapi.jsoninfo.x-gameserver-version) with a link to the client's own GitHub Release. No build restructuring — the post already runs after the Release exists.sync-spec.yml: trim the "wait for version live" poll from 20×60s to a 5×30s bounded confirmation and fix the stale Render-era comment. Safe because the gameserver's on-box deploy now dispatches only after/healthconfirms the version live, so the first probe essentially always matches. Thechanged==truegate (post only when the spec surface actually changed) is unchanged.Both the trigger reliability (the dispatch now firing at all) and adding spacemolt-lib to the nudge are handled in the dev-harness
deploy.shPR. Requires the existingDISCORD_WEBHOOK_URLrepo secret to point at#patch-notes.Generated by Claude Code