docs: add CHANGELOG.md and package manifest URLs - #210
Open
bobbyg603 wants to merge 1 commit into
Open
Conversation
5.0.0 is a breaking release with no changelog, and the Package Manager UI surfaced nothing for changelog, documentation, or licenses because the manifest omitted all three URL fields. The 5.0.0 entry is derived from the merged PR #123 and verified against the code rather than transcribed. Two claims in the audit's summary did not survive that check: PostAllCrashes/PostCrash/PostMostRecentCrash were removed outright, not marked [Obsolete], and the Unity 6000.0 minimum landed in 4.0.0 (v3.2.2 declared 2021.3), so it is recorded there instead. Entries for 4.0.0 through 4.1.0 come from the GitHub release notes and tag history. Nothing earlier is included — there is no sourced record of it. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds end-user facing package metadata so Unity Package Manager can surface docs/license/changelog links, and introduces an in-repo Keep a Changelog-style CHANGELOG.md to document notable releases (including 5.0.0’s breaking changes).
Changes:
- Added
documentationUrl,changelogUrl, andlicensesUrlto the Unity package manifest (package.json). - Added a Keep a Changelog-format
CHANGELOG.mdat the package root. - Added
CHANGELOG.md.metaso the asset ships with a stable Unity GUID.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Adds Unity Package Manager “custom location” URL fields for docs/changelog/license. |
| CHANGELOG.md | Introduces package-level release notes covering 5.0.0 and recent 4.x releases. |
| CHANGELOG.md.meta | Ensures Unity tracks the changelog as a stable asset via GUID. |
Files not reviewed (1)
- CHANGELOG.md.meta: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
daveplunkett
approved these changes
Aug 12, 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
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.
Closes #156
Adds a keepachangelog-format
CHANGELOG.mdat the package root and fills in the three manifest URL fields the Package Manager UI reads.Manifest fields
documentationUrl,changelogUrl, andlicensesUrlare documented optional properties of the Unity package manifest — confirmed against Package manifest, which describes each as a "custom location for this package's ... specified as a URL". The Package Manager falls back to the in-packageCHANGELOG.md,Documentation~, andLICENSE.mdwhen the URL is unreachable, so shipping the file and the URL is the intended combination, not a duplication.documentationUrl→ https://docs.bugsplat.com/integrations/game-development/unity (verified live; the olderintroduction/getting-started/integrations/...path 404s)changelogUrl→ the newCHANGELOG.mdonmainlicensesUrl→ the existingLICENSE.mdonmainpackage.jsonparses cleanly under bothpython -m json.toolandnode -e "require('./package.json')". ACHANGELOG.md.metais included so the package ships a stable GUID for the asset, matching the existingREADME.md.meta/LICENSE.md.meta.Why "Unreleased" rather than a date
The 5.0.0 section is headed
## [5.0.0] - Unreleased. Keep a Changelog's guidance is anUnreleasedsection at the top, but the version here is not unknown —package.jsonalready says5.0.0and the tag simply hasn't been cut. Naming the version keeps the link reference and the compare URL usable now, while- Unreleasedin the date slot stops it reading as shipped. At release, that one word becomes the date; nothing else moves. Per the task, no version bump is included.Sourcing
The 5.0.0 entry is derived from #123 and then verified against the code, not transcribed. Two items in the issue's summary did not survive that check:
PostAllCrashes/PostCrash/PostMostRecentCrashwere removed, not marked[Obsolete]. There is no[Obsolete]attribute anywhere in the tree, the methods are gone fromRuntime/BugSplat.cs(they exist atv4.1.0:Runtime/BugSplat.cs:395,410,425), andREADME.md:393already says "have been removed". Listed under Removed.v3.2.2declared2021.3andv4.0.0declared6000.0, and the v4.0.0 release notes call it out as that release's breaking change. Recorded under 4.0.0 instead.Everything else was confirmed:
WindowsReporter.cs/INativeCrashReporter.csdeleted,Attributeschanged fromDictionary<string, string>toList<BugSplatAttribute>with the new[Serializable]type,SymbolUploadClientId/SymbolUploadClientSecretdeleted from the options asset,SYMBOL_UPLOAD_CLIENT_ID/SYMBOL_UPLOAD_CLIENT_SECRETas the env var names inBugSplatSymbolUploadCredentials.cs, the~/.bugsplat/credentials/<database>.shpath shape, the three sample scripts and theButton_ForceCrashprefab deleted,Post(FileInfo)freed from its#if UNITY_STANDALONE_WIN || UNITY_WSAguard,WindowsWerEnabled, the WER and symbol-upload menu items, and.github/workflows/tests.yml.One correction from the code:
WindowsShowCrashDialogdefaults totrue(BugSplatOptions.cs), so the changelog says the dialog shows by default and you disable it to upload silently — not the other way round.Earlier entries cover 4.1.0, 4.0.1, and 4.0.0, sourced from the GitHub release notes and tag dates. Anything before 4.0.0 is deliberately absent — those releases carry no notes worth transcribing and reconstructing them from commit subjects alone would be invention. The file points at the releases page for those instead.
Not included