thehansogroup/eltrix_web

Fork 0

fix(deps): remove unused lazy_html that broke cold CI (and record the pitfall) #17

Merged

jlxq0 merged 3 commits from docs-warm-cache-pitfall into main

2026-09-07 05:26:48 +00:00

jlxq0 commented

2026-09-07 05:14:59 +00:00

Owner

Records the pitfall clark surfaced (oddie-apps/infrastructure#98): CI green rests on a warm branch-scoped cache because lazy_html cannot be built cold in ci-elixir, so a green on a warm cache is not a green about the code and any cold path hits it. Also documents the two workflow consequences already landed (check skipped on tags, docker refusing a tag not on main) so the mental model is complete.

Records the pitfall clark surfaced (oddie-apps/infrastructure#98): CI green rests on a warm branch-scoped cache because `lazy_html` cannot be built cold in ci-elixir, so a green on a warm cache is not a green about the code and any cold path hits it. Also documents the two workflow consequences already landed (check skipped on tags, docker refusing a tag not on main) so the mental model is complete.

jlxq0 added 1 commit

2026-09-07 05:14:59 +00:00

docs(agents): a green build depends on a warm cache, and why tags are handled specially

CI / Quality gate (pull_request) Failing after 4m45s

Details

CI / Build and push image (pull_request) Has been skipped

Details

9a38783ebd

Records the lazy_html cold-build fragility (oddie-apps/infrastructure#98) that
cost v0.1.0 its first build, and the two workflow consequences: check skipped on
tags, docker refusing a tag not on main.

jlxq0 added 1 commit

2026-09-07 05:23:15 +00:00

fix(deps): remove the unused only:test lazy_html, which broke cold CI

CI / Quality gate (pull_request) Failing after 13s

Details

CI / Build and push image (pull_request) Has been skipped

Details

a2d7bfb6d1

lazy_html is a NIF whose precompiled download and lexbor source clone both go to
github.com, which the CI network blocks, so it can only build off a warm
branch-scoped cache. A cache eviction on 2026-09-07 made every eltrix_web PR
fail at once, and it had already cost v0.1.0 its first build.

Nothing used it: no code or test references LazyHTML, the suite asserts via
html_response and render_to_string, and phoenix_live_view declares lazy_html
optional. Removed; 17 tests pass, the lock has no :make deps left, so a cold
build no longer reaches github.com. The general egress fragility is
oddie-apps/infrastructure#98. Revises the AGENTS.md pitfall to match.

jlxq0 changed title from ~~docs(agents): a green build depends on a warm cache~~ to fix(deps): remove unused lazy_html that broke cold CI (and record the pitfall)

2026-09-07 05:23:28 +00:00

jlxq0 added 1 commit

2026-09-07 05:25:13 +00:00

fix(deps): drop lazy_html's now-orphaned build deps (cc_precompiler, elixir_make, fine)

CI / Quality gate (pull_request) Successful in 15s

Details

CI / Build and push image (pull_request) Has been skipped

Details

cf82a6c3bf

deps.unlock --check-unused in mix ci flagged them once lazy_html was gone; they
were pulled in only to build its NIF.

jlxq0 merged commit 14e428310f into main

2026-09-07 05:26:48 +00:00

jlxq0 deleted branch docs-warm-cache-pitfall

2026-09-07 05:26:49 +00:00

jlxq0 referenced this pull request from a commit

2026-09-07 05:26:50 +00:00

Merge pull request 'fix(deps): remove unused lazy_html that broke cold CI (and record the pitfall)' (#17) from docs-warm-cache-pitfall into main

No reviewers

No labels

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

thehansogroup/eltrix_web!17

No description provided.

Rows
Columns