Repository navigation
Conversation
neriousy
added this pull request to stack #56077
October 16, 2026 13:25
This was referenced Oct 16, 2026
neriousy
force-pushed
the
drop-dead-triggers
branch
from
October 16, 2026 16:08
6948708 to
531d516
Compare
neriousy
force-pushed
the
drop-dead-triggers
branch
from
October 16, 2026 16:21
531d516 to
0a68058
Compare
This branch has not been deployed
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.
Stacked on #56075 (
pack-libraries), the top of the publish speed-up stack (bottom: #55987).Summary
publish.ymlnow runs on pushes todevandbetaonly. It drops the push triggers forci,fix/npm-native-binary-installandsnapshot-*, which can no longer publish anything. This removes 3 lines. No job or step condition changes: none of them names these branches, andworkflow_dispatchstill works on any branch.Why each trigger was dead
snapshot-*: before this stack, these runs failed at "Build legacy CLI", which buildspackages/opencode/script/build.ts.packages/opencodeno longer exists on dev. The last run (snapshot-repository, 29934522977, July 22) failed there.ci: same "Build legacy CLI" path. It last ran in January 2026.fix/npm-native-binary-install: a branch trigger for testing Fix npm CLI binary installation #27801, which merged on 2026-05-15. Its one run (25944766634) failed at Azure login insign-cli-windowsand the Windowsbuild-electronlegs.After #56070 (
cli-target-jobs), any of these pushes would run all 12build-clilegs plus Developer ID signing on macOS. It would then stop at publish's "Check CLI packages" guard, becausesign-cli-windowsonly runs ondev. Every push would cost runner time and produce nothing.Dependencies checked
publish.ymlis the only workflow with a branch filter for them.packages/scriptchannel logic: nothing keyed on them. The fallback that turns a branch name into a channel is generic. It still covers local runs and dispatches, and dev setsOPENCODE_CHANNEL=devitself.services/update:channelsForRefstill mapsrefs/heads/ci,refs/heads/fix/npm-native-binary-installandrefs/heads/snapshot-*to update channels for publish OIDC tokens. Those entries are now unused, but removing them changes a deployed service, so it is left for a separate PR.Safety rule
The stack's handover rule "never push branches named
ci,dev,beta,fix/npm-native-binary-installorsnapshot-*" shrinks todevandbeta. GitHub readson.pushfrom the workflow file in the pushed commit, so the rule only shrinks for commits that contain this change. Pushing an oldsnapshot-*orcitip, or one based on dev before this merges, still triggers the old workflow.Verification
devandbetapaths are byte for byte unchanged.