ios-webkit-debug-proxy: Fix checkver, update license#16870
Conversation
WalkthroughThis pull request updates the Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
All changes look good. Wait for review from human collaborators. ios-webkit-debug-proxy
|
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
bucket/ios-webkit-debug-proxy.json
🧰 Additional context used
🧠 Learnings (2)
📚 Learning: 2025-10-16T13:42:43.224Z
Learnt from: o-l-a-v
Repo: ScoopInstaller/Extras PR: 16349
File: bucket/debugviewpp.json:14-21
Timestamp: 2025-10-16T13:42:43.224Z
Learning: In the DebugViewPP project (CobaltFusion/DebugViewPP), the executable name changed from "DebugView++.exe" (in v1.8.0.103 and earlier) to "Debugviewpp.exe" (in v1.9.0.28 and later). The manifest bucket/debugviewpp.json correctly uses "Debugviewpp.exe" for current versions.
Applied to files:
bucket/ios-webkit-debug-proxy.json
📚 Learning: 2025-11-29T22:02:34.868Z
Learnt from: Ra2-IFV
Repo: ScoopInstaller/Extras PR: 16672
File: bucket/chromium-clang.json:12-12
Timestamp: 2025-11-29T22:02:34.868Z
Learning: In the chromium-clang.json manifest (bucket/chromium-clang.json), the extract_dir value "chrome-win32" is a fixed name determined by the upstream Chromium_Clang archive structure and should not be changed, even though the manifest targets 64-bit architecture.
Applied to files:
bucket/ios-webkit-debug-proxy.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: WindowsPowerShell
🔇 Additional comments (1)
bucket/ios-webkit-debug-proxy.json (1)
21-21: LGTM! Proper use of $matchTag for release URLs.The change from
v$versionto$matchTagcorrectly handles release tags that include the "v" prefix. Since the regex captures the full tag (including optional "v") in the named group "tag", using$matchTagensures the download URL correctly references the GitHub release tag as it appears in the repository.
Summary
Improves the
ios-webkit-debug-proxymanifest by correcting license metadata and making version detection more reliable through explicit GitHub Releases API querying and asset-based filtering.Related issues or pull requests
Changes
licensefield to reflect the complete set of licenses included in the distributioncheckver: githubwith an explicit GitHub Releases API endpointNotes
The license information in the GitHub project is outdated; the currently applicable licenses can be found in the
$dirdirectory.Testing
The test results are as follows:
<manifest-name[@version]|chore>: <general summary of the pull request>Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.