[PR #1752] Remove redundant chrome_validate hook, rename wget_validate to wget_i… #3009

New issue

Closed

opened 2026-03-01 18:01:23 +03:00 by kerem · 0 comments

kerem commented

2026-03-01 18:01:23 +03:00

Owner

Original Pull Request: https://github.com/ArchiveBox/ArchiveBox/pull/1752

State: closed
Merged: Yes

…nstall

Delete chrome/on_Crawl__10_chrome_validate.py (duplicates chrome_install)
Rename wget/on_Crawl__11_wget_validate.py → on_Crawl__06_wget_install.py

All hooks now follow consistent naming: install, launch, or config

Summary

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Snapshot data layout on disk

Summary by cubic

Removed the redundant Chrome validate hook, renamed the Wget validate hook to wget_install, and standardized hook names and priorities to match the install/launch/config lifecycle. This removes duplicate logic and fixes priority conflicts across Crawl, Binary, and Snapshot hooks.

Refactors
- Deleted chrome/on_Crawl__10_chrome_validate.py (dup of chrome_install)
- Renamed wget validate to on_Crawl__06_wget_install.py
- Standardized on_Binary hook priorities: npm 10, pip 11, brew 12, apt 13, custom 14, env 15
- Fixed on_Snapshot order: staticfile 32, readability 56, mercury 57, htmltotext 58

^{Written for commit 09a1ca3134. Summary will update on new commits.}

**Original Pull Request:** https://github.com/ArchiveBox/ArchiveBox/pull/1752 **State:** closed **Merged:** Yes --- …nstall - Delete chrome/on_Crawl__10_chrome_validate.py (duplicates chrome_install) - Rename wget/on_Crawl__11_wget_validate.py → on_Crawl__06_wget_install.py All hooks now follow consistent naming: install, launch, or config  # Summary  # Related issues  # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk  --- ## Summary by cubic Removed the redundant Chrome validate hook, renamed the Wget validate hook to wget_install, and standardized hook names and priorities to match the install/launch/config lifecycle. This removes duplicate logic and fixes priority conflicts across Crawl, Binary, and Snapshot hooks. - **Refactors** - Deleted chrome/on_Crawl__10_chrome_validate.py (dup of chrome_install) - Renamed wget validate to on_Crawl__06_wget_install.py - Standardized on_Binary hook priorities: npm 10, pip 11, brew 12, apt 13, custom 14, env 15 - Fixed on_Snapshot order: staticfile 32, readability 56, mercury 57, htmltotext 58 <sup>Written for commit 09a1ca3134847b47ca71576506cbac9c67a360ae. Summary will update on new commits.</sup>

kerem

2026-03-01 18:01:23 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ArchiveBox#3009

No description provided.

Rows
Columns