Skip to content

Switch wget for wget-at for better WARC compliance #17

Description

@pirate

https://github.com/ArchiveTeam/wget-lua

It's the version that archive team warrior runs and is very well tested + ready to feed directly into archive.org.

Activity

  1. pirate commented on Mar 24, 2026

    @pirate
    MemberAuthor

    TODO: implement this over on the new plugins repo https://github.com/archiveBox/abx-plugins

  2. devicefricker commented on Mar 25, 2026

    @devicefricker

    I don't really want to nag you (in another issue, you said you could only work on this on weekends), but can we transfer this to abx-plugins?

  3. pirate commented on Mar 26, 2026

    @pirate
    MemberAuthor

    @claude can you take a crack at adding a new plugin wgetlua thats similar to wget, but it should install wget-at using binprovider overrides defined in config.json.

    Install must succeed locally / in the test environment when running abx-dl install wgetlua (uv pip install -e ../abx-dl. auto-install in macos/linux at runtime is a core part of the user-facing surface area and must succeed in all environments including CI and sandboxes with no mocking or "faked" code paths in tests.

    Make sure the plugin includes a live integration test that hits a real https://example.com url and confirms the warc and html saved actually contain the correct contents. Do not mock the url with a test server and do not mock the wget-at/wgetlua binary, it must succeed an install the real binary and hit a real url and produce real content in the output. Do not simply check that it runs without error, your tests must check that the content saved is actually correct.

  4. pirate commented on Mar 26, 2026

    @pirate
    MemberAuthor

    Unlike all our other plugin binary dependencies, it seems wget-at offers no package distribution, prebuilt binaries, or install script. This makes it significantly harder to offer as a plugin, as users would have to install an entire C/C++ build system toolchain.

    If that changes and they release a package manager distribution then I'm more likely to add it, otherwise this is very low on my list. I'm more likely to invest in adding a browsertrix/archivweb.page plugin which is great at producing WARC/WACZs.

  5. added a commit that references this issue on Mar 26, 2026
    6758b9e
  6. pirate commented on Mar 27, 2026

    @pirate
    MemberAuthor

    Vibecoded approach but untested / not going to enable by default with this gnarly of a build step: #19

  7. devicefricker commented on Mar 27, 2026

    @devicefricker

    Eh, it's only a plugin, nobody will really care if it's vibecoded or not (as long as it doesn't delete the user's D drive!)

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions