fix(changelog): don't attribute native preview tools to the web app#953
Merged
Merged
Conversation
The weekly changelog generator in claude.yml builds its post from the raw git log, which includes commits for the Windows Explorer preview/thumbnail handlers and the macOS/iOS Quick Look extensions. Those are separate native companion tools that read PKHeX files from the filesystem and do not integrate with the Blazor WebAssembly web app. A recent Change Log discussion post folded them in as PKMDS features, which is misleading. Add a scope/attribution instruction to Step 4 so the generator keeps "PKMDS" scoped to the web app and frames the native tools as separate companions. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the changelog-generation prompt in .github/workflows/claude.yml to ensure weekly changelog posts don’t incorrectly present changes to native preview/thumbnail companion tools as “PKMDS” (the Blazor WebAssembly web app) features.
Changes:
- Adds explicit scope/attribution guidance defining “PKMDS” as the Blazor WASM web app.
- Instructs the generator to treat Windows Explorer preview/thumbnail handlers and macOS/iOS Quick Look extensions as separate native companion tools (or omit them).
- Adds a “when in doubt, leave it out” rule to avoid misattribution when commit impact is unclear.
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.
Summary
The weekly changelog generator in
.github/workflows/claude.ymlbuilds its discussion post from the rawgit log, which includes commits for the Windows Explorer preview/thumbnail handlers and the macOS/iOS Quick Look extensions. Those are separate native companion tools that read PKHeX files from the filesystem and do not integrate with the Blazor WebAssembly web app.A recent Change Log discussion post folded those native tools in as PKMDS features, which is misleading — "PKMDS" is the web app, and that is the product the changelog is about.
Change
Adds a scope/attribution instruction to Step 4 of the changelog prompt so the generator:
No code or runtime behavior changes — prompt text only, within the existing
prompt: |block scalar.🤖 Generated with Claude Code