6 Commits

This Branch

This Branch

All Branches

Author	SHA1	Message	Date
clawbot	d5d7908ff6	Test the live-photo hash check's error paths (closes #117) ... Three downloadFile tests cover a live photo whose ZIP names an unknown compression method, one whose ZIP has no image entry and one with no video entry. Each checks that nothing is stored and the error names the file ID; the unreadable ZIP is also checked not to be retried. Each test fails when its check in livePhotoHasher is removed. Model: opus-5-5	2026-09-23 04:51:31 +00:00
clawbot	075b1bb921	Expand single-line make variables in the lint-once resolver ... A recipe of `@$(FMT)` with `FMT := script/fmt-check` gave `make check` two prettier passes while the suite stayed green: the resolver read the line as invoking nothing. The shipped Makefile already writes recipes that way (`@$(YARN) tsc --watch`), so this was a gap in the repo's own house style. Variables assigned a literal on one line (`:=`, `=`, `?=`) are collected in a pass of their own and substituted into target and recipe lines. Values needing evaluation -- another reference, a make function, a `define` body -- are left verbatim, and the header's not-followed list now says so.	2026-09-04 11:05:41 +00:00
clawbot	ff4cc63c8b	Name the define/endef hole in the lint-once header ... A `define EXTRA ... endef` body pulled into a recipe as $(EXTRA) gives `make check` a second prettier pass that the walk still scores as one. The parser expands no variables, so this is a name it cannot resolve, not a body it declines to read. The header's exclusion list says so rather than claiming coverage the code does not have.	2026-09-04 10:55:41 +00:00
clawbot	f4ecef8820	Follow make conditionals and semicolon recipes in lint-once ... makeRecipes() reset the current target on every non-tab line, so an ifeq/endif block ended the recipe and every tab-indented line inside it was discarded; and the target line's tail was read entirely as prerequisites, so `check: ; @script/fmt-check` split to tokens that named no target and vanished. Both gave `make check` a second prettier pass with the suite green. Conditional directives no longer end a recipe, and every branch is treated as reachable rather than evaluating the condition. The target line is split on the first `;`: what precedes it is the prerequisite list, what follows is the first recipe line. Both pinned directly, and the header's exclusion list now names what the parser actually skips.	2026-09-04 10:54:24 +00:00
clawbot	9ed92e1231	Format the lint-once resolver changes ... Reflow the heredoc RUN fixture array in the BuildKit heredoc test to satisfy prettier --check, which rejected the single-line form. No behavioural change to the tests or the resolver.	2026-09-04 10:31:40 +00:00
clawbot	8bf5138582	WIP: follow Makefile prerequisites and $(MAKE) in lint-once	2026-09-04 10:05:38 +00:00