Repository navigation
Fix four small bugs in issue descriptions - #128
Merged
Merged
Conversation
The schema that reads dependencies.json back dropped the provider fields the templates use, so issues said to edit `undefined`. A Snyk advisory could list the same fix release twice. The rationale sentence joined a lowercase clause after a period. The owner line in the summary had no label. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
srubin
approved these changes
Oct 19, 2026
Collaborator
Author
|
Thanks! Hope you're all having fun over there. |
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.
This fixes four bugs in the text of a Linear or GitHub issue.
Issues said to edit
undefined. Theupdatecommand writes the provider'scatalogFile,patchHintandupdate*strings todependencies.json. The schema that reads the file did not list these fields. Zod removes the fields it does not know. Somake-linear-issuesgot each provider without them, and the catalog advice said:The schema now lists these fields. A test makes sure they come back.
A fix release was listed twice. Snyk can give the same release in two coordinates of one advisory. The issue showed
fixed in 15.5.27, 16.3.8, 16.3.8. The list now has each release one time.A lowercase clause after a period. The "Why this matters" section joins the rationale clauses with a period. The Snyk and OSV sources wrote
fix available in a newer version. The result was4 Snyk advisories. fix available in a newer version.The clause now starts with a capital letter.An owner line with no label. The summary list has
- **Current version:**and- **Target version:**. Between them was- Growth, with no label. It is now- **Owner:** Growth.🤖