Replace hostlist-compiler references with adblock-compiler#273
Replace hostlist-compiler references with adblock-compiler#273jaypatrick merged 4 commits intomainfrom
Conversation
Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
adblock-compiler | cbb0e47 | Feb 09 2026, 01:51 AM |
Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
Co-authored-by: jaypatrick <1800595+jaypatrick@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR standardizes project naming by replacing non-historical hostlist-compiler references with adblock-compiler across build/config, CLI help/version strings, CI/release workflows, docs, and examples—while keeping intentional historical references to @adguard/hostlist-compiler.
Changes:
- Renamed CLI/build outputs and updated help/version text to
adblock-compiler. - Updated CI/release workflows and Cloudflare Pages project naming to
adblock-compiler-ui/adblock-compiler-*artifacts. - Refreshed documentation/examples and schema metadata to match current project naming.
Reviewed changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| wrangler.toml | Updates Pages deploy command example to adblock-compiler-ui. |
| worker/html.ts | Updates redirect page branding/URL to “Adblock Compiler”. |
| src/utils/headerFilter.test.ts | Updates test fixtures for “Compiled by …” header lines. |
| src/schemas/configuration.schema.json | Updates schema $id, title, and description to new project identity. |
| src/cli/CliApp.deno.ts | Updates CLI help examples and startup log message naming. |
| src/cli/ArgumentParser.ts | Updates help text and --version output naming. |
| examples/whitelist/README.md | Updates install/usage examples to adblock-compiler. |
| examples/sdn/README.md | Updates install/usage examples to adblock-compiler. |
| examples/energized/README.md | Updates install/usage examples to adblock-compiler. |
| examples/china/README.md | Updates install/usage examples to adblock-compiler. |
| examples/cloudflare-worker/test.html | Updates UI labels and worker URL references. |
| examples/cloudflare-worker/src/html.ts | Updates embedded redirect HTML branding/URL. |
| docs/deployment/docker.md | Updates Docker volume mounts and invocation to adblock-compiler. |
| docs/deployment/cloudflare-pages.md | Updates Pages deploy command to adblock-compiler-ui. |
| docs/BLOG_POST_ADBLOCK_COMPILER.md | Updates CLI examples and binary name mention. |
| deno.json | Updates deno task build output binary name. |
| README.md | Updates install commands, CLI examples, and help snippets to adblock-compiler. |
| .gitignore | Updates ignored build artifact names to adblock-compiler*. |
| .github/workflows/release.yml | Updates artifact names/patterns and release notes table to adblock-compiler-*. |
| .github/workflows/ci.yml | Updates Pages deploy project name to adblock-compiler-ui. |
| .github/workflows/README.md | Updates build command example output binary name. |
| <title>Adblock Compiler</title> | ||
| <meta http-equiv="refresh" content="0;url=https://adblock-compiler.pages.dev"> | ||
| </head> | ||
| <body> | ||
| <h1>Redirecting to Hostlist Compiler...</h1> | ||
| <p>If you are not redirected, <a href="https://hostlist-compiler.pages.dev">click here</a>.</p> | ||
| <h1>Redirecting to Adblock Compiler...</h1> | ||
| <p>If you are not redirected, <a href="https://adblock-compiler.pages.dev">click here</a>.</p> |
There was a problem hiding this comment.
The redirect URL points to https://adblock-compiler.pages.dev, but the Pages project name used elsewhere in this repo is adblock-compiler-ui (wrangler/CI/docs). Cloudflare Pages *.pages.dev subdomain is derived from the project name, so this will likely redirect to a non-existent site. Align the URL with the Pages project name (or rename the Pages project consistently).
| <title>Adblock Compiler</title> | ||
| <meta http-equiv="refresh" content="0;url=https://adblock-compiler.pages.dev"> | ||
| </head> | ||
| <body> | ||
| <h1>Redirecting to Hostlist Compiler...</h1> | ||
| <p>If you are not redirected, <a href="https://hostlist-compiler.pages.dev">click here</a>.</p> | ||
| <h1>Redirecting to Adblock Compiler...</h1> | ||
| <p>If you are not redirected, <a href="https://adblock-compiler.pages.dev">click here</a>.</p> |
There was a problem hiding this comment.
Same issue as worker/html.ts: the meta refresh and link target https://adblock-compiler.pages.dev, but the Pages project name referenced in CI/docs is adblock-compiler-ui, which implies the Pages URL should be https://adblock-compiler-ui.pages.dev (unless the project name is changed everywhere else). Please make these consistent to avoid broken redirects.
| '! License: https://github.com/AdguardTeam/AdguardSDNSFilter/blob/master/LICENSE', | ||
| '! Last modified: 2026-01-04T02:52:12.913Z', | ||
| '!', | ||
| '! Compiled by @adguard/hostlist-compiler v1.0.38', | ||
| '! Compiled by @jk-com/adblock-compiler v1.0.38', |
There was a problem hiding this comment.
In stripUpstreamHeaders - should handle complex upstream headers, the fixture line was changed to ! Compiled by @jk-com/adblock-compiler ..., but the test assertions still check for removal of Compiled by @adguard specifically. This makes that assertion ineffective (it will always pass regardless of whether the line was stripped). Either keep the fixture as an upstream-style @adguard/hostlist-compiler header, or update the assertion to verify that any Compiled by line is removed.
Updated all references from the original fork name
hostlist-compilerto the current project nameadblock-compilerthroughout the codebase.Changes
Build & Configuration
deno.json: Build task output →adblock-compiler.gitignore: Build artifacts →adblock-compiler,adblock-compiler.exewrangler.toml: Cloudflare Pages project →adblock-compiler-uisrc/schemas/configuration.schema.json: Schema ID and metadataCLI & Source Code
src/cli/ArgumentParser.ts: Help text and version outputsrc/cli/CliApp.deno.ts: Help text and startup logger messagesrc/utils/headerFilter.test.ts: Test fixture dataGitHub Workflows
.github/workflows/ci.yml: Pages deployment project name.github/workflows/release.yml: Artifact names, download patterns, release notes table.github/workflows/README.md: Build command examplesDocumentation & Examples
README.md: Installation commands, CLI usage examples (7 occurrences)docs/deployment/docker.md: Docker volume mounts and CLI invocationsdocs/deployment/cloudflare-pages.md: Pages deployment commandsdocs/BLOG_POST_ADBLOCK_COMPILER.md: All CLI examples (5 occurrences)examples/*/README.md: Installation and usage (4 files)examples/cloudflare-worker/*: HTML templates and test pagesworker/html.ts: Redirect pagePreserved References
Historical references to
@adguard/hostlist-compilerintentionally retained in:README.md,CHANGELOG.md: Project origin attributiondocs/MIGRATION.md: Migration guide from original packagedocs/RELEASE_0.8.0.md: "Based on" creditOriginal prompt