Advertise the Yutils feed in knownFeeds#33
Open
line0 wants to merge 1 commit into
Open
Conversation
Yutils is packaged for DependencyControl in TypesettingTools/Yutils#2 as `tstools.Yutils`, which declares the bare `Yutils` name in `provides`. Advertising its feed here folds it into the officially trusted set, so ASSFoundation's existing feed-less `Yutils` requirement resolves to it without ASSFoundation having to reference a 0.4.0 feed that v0.6.x clients can't parse. The retired `master` feed is deliberately left untouched, so only v0.7.0+ clients ever discover the feed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
Test results — macos-latest ✅1060 · ubuntu-latest ✅1060 · windows-latest ✅1057Summary
🎉 No failed tests in this run. Github Test Reporter by CTRF 💚 |
CoffeeFlux
approved these changes
Jul 24, 2026
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
2 participants
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.
Advertises the Yutils feed added in TypesettingTools/Yutils#2, so DependencyControl officially trusts it.
Yutils has been distributed by bundling it with the Aegisub installer, because no feed offered it. TypesettingTools/Yutils#2 adds one, packaging the library as
tstools.Yutilsand declaring the bareYutilsname inprovides.Listing that feed in
knownFeedsfolds it into the officially trusted set, which is all it takes for ASSFoundation's existing requirement —{"moduleName": "Yutils", "optional": true}, bare and feed-less — to resolve to it through theprovidesalias. ASSFoundation needs no change.The indirection is deliberate. Putting the feed URL directly into ASSFoundation's feed would expose a 0.4.0 feed to v0.6.x clients, which can't parse it — the same format-compatibility trap we just dug out of. Advertising it here means the URL only ever appears in feeds that v0.7.0+ clients read; the retired
masterfeed is left untouched, so v0.6.x never learns it exists.Merge order: land TypesettingTools/Yutils#2 and push its
v1.0.0tag before this change is published to thepublishbranch. Otherwise DependencyControl advertises a feed URL that 404s — not fatal, since feed discovery tolerates unloadable feeds and the ASSFoundation requirement is optional, but it causes pointless failed fetches.🤖 Generated with Claude Code