[PR #1737] Reduce code duplication between Chrome utilities #2994

New issue

Closed

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

kerem commented

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

Owner

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

State: closed
Merged: Yes

This change consolidates duplicated logic between chrome_utils.js and extension installer hooks, as well as between Python plugin tests:

JavaScript changes:

Add getExtensionsDir() to centralize extension directory path calculation
Add installExtensionWithCache() to handle extension install + cache workflow
Add CLI commands for new utilities
Refactor all 3 extension installers (ublock, istilldontcareaboutcookies, twocaptcha) to use shared utilities, reducing each from ~115 lines to ~60
Update chrome_launch hook to use getExtensionsDir()

Python test changes:

Add chrome_test_helpers.py with shared Chrome session management utilities
Refactor infiniscroll and modalcloser tests to use shared helpers
setup_chrome_session(), cleanup_chrome(), get_test_env() now centralized
Add chrome_session() context manager for automatic cleanup

Net result: ~208 lines of code removed while maintaining same functionality.

Summary

Changes these areas

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

Summary by cubic

Consolidated Chrome extension path/install logic and unified Python test helpers to reduce duplication with no behavior changes. This removes ~208 LOC and makes extension installers and tests simpler to maintain.

New Features
- getExtensionsDir() centralizes extension dir resolution (env override supported).
- installExtensionWithCache() handles install + cache read/write.
- Added CLI commands for both utilities.
Refactors
- uBlock, I Still Don’t Care About Cookies, and 2captcha installers now use shared utilities.
- chrome_launch hook now uses getExtensionsDir().
- Added chrome_test_helpers.py (get_test_env, setup_chrome_session, cleanup_chrome, chrome_session); updated infiniscroll and modalcloser tests to use it.

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

**Original Pull Request:** https://github.com/ArchiveBox/ArchiveBox/pull/1737 **State:** closed **Merged:** Yes --- This change consolidates duplicated logic between chrome_utils.js and extension installer hooks, as well as between Python plugin tests: JavaScript changes: - Add getExtensionsDir() to centralize extension directory path calculation - Add installExtensionWithCache() to handle extension install + cache workflow - Add CLI commands for new utilities - Refactor all 3 extension installers (ublock, istilldontcareaboutcookies, twocaptcha) to use shared utilities, reducing each from ~115 lines to ~60 - Update chrome_launch hook to use getExtensionsDir() Python test changes: - Add chrome_test_helpers.py with shared Chrome session management utilities - Refactor infiniscroll and modalcloser tests to use shared helpers - setup_chrome_session(), cleanup_chrome(), get_test_env() now centralized - Add chrome_session() context manager for automatic cleanup Net result: ~208 lines of code removed while maintaining same functionality.  # Summary  # Related issues  # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk  --- ## Summary by cubic Consolidated Chrome extension path/install logic and unified Python test helpers to reduce duplication with no behavior changes. This removes ~208 LOC and makes extension installers and tests simpler to maintain. - **New Features** - getExtensionsDir() centralizes extension dir resolution (env override supported). - installExtensionWithCache() handles install + cache read/write. - Added CLI commands for both utilities. - **Refactors** - uBlock, I Still Don’t Care About Cookies, and 2captcha installers now use shared utilities. - chrome_launch hook now uses getExtensionsDir(). - Added chrome_test_helpers.py (get_test_env, setup_chrome_session, cleanup_chrome, chrome_session); updated infiniscroll and modalcloser tests to use it. <sup>Written for commit fd9ba86220479c7b2406644e79871334fed9887a. Summary will update on new commits.</sup>

kerem

2026-03-01 18:01:21 +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#2994

No description provided.

Rows
Columns