Skip to content

Conversation

@o-l-a-v
Copy link
Contributor

@o-l-a-v o-l-a-v commented Oct 19, 2025

Changes:

  • Remove checkver and autoupdate
  • Use Wyback Machine / Inernet Archive for downloads

Because:

  • Downloads and checkver is blocked by Cloudflare.
  • No new version in years.
  • Could not easily find contact info on homepage about whitelisting Scoop from Cloudflare.
  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

Summary by CodeRabbit

  • Chores
    • Updated download sources to use Internet Archive archives for improved availability and reliability.
    • Removed automatic update and version checking features.

@coderabbitai
Copy link

coderabbitai bot commented Oct 19, 2025

Walkthrough

The PR updates a Scoop package manifest to replace direct download URLs with Internet Archive links, removes automatic version checking, and adds a note explaining Cloudflare blocking issues.

Changes

Cohort / File(s) Summary
Package manifest updates
bucket/dreamset.json
Adds top-level note about Cloudflare blocks; removes checkver and autoupdate sections; updates 32bit and 64bit architecture URLs to point to archived versions via web.archive.org while preserving existing hashes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 The archives preserve what Cloudflare won't share,
No more update checks floating through the air,
With Internet Memory standing by our side,
The downloads now safely inside we'll confide,
A little JSON tweak keeps the dream alive! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title "[email protected]: Moved downloads to Wayback Machine" clearly and specifically describes the primary change in the pull request. It follows the conventional format specified in the repository's template (<manifest-name[@version]>: <general summary>), is concise and readable, and directly conveys the main modification—switching downloads to archived Wayback Machine URLs. A teammate reviewing the repository history would immediately understand the purpose of this change.
Description Check ✅ Passed The pull request description includes a clear "Changes" section explaining what was modified (removal of checkver/autoupdate, use of Wayback Machine) and a "Because" section providing strong rationale (Cloudflare blocking, no recent updates, contact unavailable). Both required checklist items confirming conventional PR title format and reading the Contributing Guide are marked as complete. However, the description is missing the issue reference section (Closes #XXXX or Relates to #XXXX) specified in the template, which would link this PR to a related discussion issue.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 37348dc and 6090121.

📒 Files selected for processing (1)
  • bucket/dreamset.json (1 hunks)
⏰ 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 (2)
bucket/dreamset.json (2)

2-2: Metadata note follows Scoop conventions and clearly documents the rationale.

The "##" key correctly adds machine-readable documentation explaining the Cloudflare blocking issue and Wayback Machine fallback. This is appropriate and improves transparency for end-users and maintainers.


8-15: Verify hashes and timestamp discrepancy for 32bit URL before merging.

The Internet Archive URLs are functional and return valid ZIP files. However, verification reveals critical concerns:

  1. 32bit URL timestamp mismatch: The URL with web/20211021072934if_/ redirects to web/20211021073001if_/ (different timestamp by 27 seconds). Confirm whether this redirect is expected or indicates accessing a different binary version.

  2. Hash verification incomplete: Both files download successfully (4.4 MB for 32bit, 5.1 MB for 64bit), but hash validation could not be completed in the sandbox environment. You must manually verify the SHA-256 hashes match the provided values by downloading the files and computing their checksums.

  3. 64bit URL: Verified working correctly with if_ mode.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

Your changes did not pass all checks.

Please address the issues in the manifest and comment starting with /verify to rerun the checks.

dreamset

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

Removed checkver and autoupdate sections from dreamset.json.
@o-l-a-v
Copy link
Contributor Author

o-l-a-v commented Oct 19, 2025

Related: #16379.

@o-l-a-v
Copy link
Contributor Author

o-l-a-v commented Oct 19, 2025

/verify

@github-actions
Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

dreamset

  • Lint
  • Description
  • License
  • Hashes

Check the full log for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant