Docs: make the README's tool list match what ships - #33
Merged
Conversation
Bumps VERSION to 0.16.0 and closes the changelog section for the release: File Shelf as a new tool, support-file removal in Uninstall Apps, next-display moves and an arrange undo in Window Manager, per-directory listing in Clean Drive, acting on scan results in Disk Usage Analyzer, a global OCR shortcut in Grab Text, sparklines and a top-process list in System Monitor, saved device profiles in Audio Switcher, and the store-file and threading hardening that followed. Moves the File Shelf entry out of the 0.15.0 section, where 5f2b0b6 filed it. File Shelf was merged after v0.15.0 was tagged, so the entry claimed a tool that is not in the build users already have, and it would have gone missing from the release that actually carries it — the one place someone would look for it. The 0.15.0 section is now byte-identical to the tag again, and File Shelf leads 0.16.0's Added list where the other new tools sit in their own releases. Sparkle notes for the release are the usual short curated file rather than the changelog section, which is 60 lines here — well past the point where the update window grows taller than the screen and puts the Update button out of reach. Eight tools in one release is more to cover than 0.15.0 had, so the notes lead with File Shelf and reduce the rest to one line apiece. Nothing else in the tree carries the version: the Info.plist strings are stamped from VERSION by package_app.sh at build time, and a sweep for a hardcoded 0.15.0 outside the changelog found none.
The nine merges that landed today left the README describing a smaller, older tool box than the one in ToolboxCatalog. The headline count was one short: File Shelf brought the catalog to 27 tools while the opening line still said 26. The table itself listed all 27 but carried a 28th row — a stale duplicate of Dev Tools without JWT, reintroduced by the merge for PR #10 after the JWT commit had already corrected the original. Every catalog tool now appears exactly once, and no row names a tool the catalog does not have. Five descriptions had been overtaken by the features merged today and were understating the tool to the point of being wrong about what it can do: System Monitor's sparklines and top-process list, Clean Drive's per-directory developer junk, Disk Analyzer acting on a scanned row, Window Manager's next-display move and undo, and Audio Switcher's device profiles. Uninstall Apps and Grab Text needed nothing — the first already claimed the leftover sweep that only now exists, and the second had its shortcut added with the feature. Packaging was already consistent and is untouched: all 27 helpers in package_app.sh resolve to a Packaging/*Info.plist that exists, every plist but the main app's Info.plist is referenced, and each helper's CFBundleIdentifier matches its catalog entry.
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
1 participant
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.
Today's nine merges left the README describing a smaller, older tool box than the one the code actually builds. This is a docs-only pass over every count, list and claim in
README.mdagainstSources/DMonteCore/ToolboxCatalog.swift,Scripts/package_app.shandPackaging/.Drift found and fixed
README.md:5README.md:48README.md:25README.md:27README.md:28README.md:37⌃⌥ + arrows⌃⌥⌘→) and undo (⌃⌥⌫)README.md:38The table now lists all 27 catalog tools exactly once, with no row for a tool the catalog does not have — checked in both directions.
Checked and deliberately left alone
⌃⌥Gshortcut was added to the row in the feature commit itself.package_app.shname aPackaging/*Info.plistthat exists; the only unreferenced plist is the main app'sInfo.plist, whichpackage_app.shcopies directly; every helper'sCFBundleIdentifiermatches its catalogbundleID; nothing references a removed tool.macOS 14+matchesPackage.swift's.macOS(.v14);Swift 6.1matches the tools version; Volume Mixer'smacOS 14.2+matches the availability guards; Video Splitter's15/30/60/90matchesclipLengthChoices.CHANGELOG.mduntouched — a release is being prepared from it.Verification
swift build—Build complete!swift test—Executed 1182 tests, with 0 failures (0 unexpected) in 6.244 (6.309) secondsToolRegistrationConsistencyTestspasses; note it cross-checks catalog / packaging script / manifest, not the README, which is why this drift was invisible to CI.