mirror of
https://github.com/Matysh/houseplan-card
synced 2026-10-07 23:19:14 +00:00
Claude
28a4cb5eec
Вопрос владельца: зачем агенты снимают PNG на Windows, если кадры мы всё равно не принимаем, тем более что WSL есть на обеих машинах. Ответ по коду: этому ничто не мешало. Ни один из шести скриптов съёмки и приёмки не знал, на какой он ОС — ни `process.platform`, ни win32, ни WSL не упоминались нигде. Съёмка отрабатывала штатно, а стена появлялась на приёмке, и текст стены говорил «сцен-свидетелей 0 из 10», то есть подсказывал неверный вывод «надо объявить больше сцен» — от которого до `--expect-change` на всю матрицу одна команда. Причина запрета не политика, а физика: Windows растеризует текст через DirectWrite, с другим субпиксельным сглаживанием и DPI, поэтому байтового совпадения с принятым эталоном не даёт никогда, свидетелей среды быть не может, и приёмка откажет всё равно. Флаги детерминизма из #410 убирают разброс внутри среды, а не между ОС. Что сделано: - `golden:capture` отказывается до запуска браузера, в тексте отказа готовая команда для WSL; - `golden:verify` остаётся законным в любой среде: он ничего не принимает, а как грубая проверка полезен; - обе приёмки (golden и документации) отказываются в чужой среде; - к отказу свидетелей приписывается фраза про расхождение среды — та самая, которой не хватало, чтобы отказ не читался как «объяви больше сцен»; - платформа уезжает в манифесты рядом с версией Chromium: у кадров появился провенанс среды; - осознанный обход есть и требует причину: HP_ALLOW_FOREIGN_CAPTURE. Главный урок задачи — про цену правки файла, у которого записан хеш. Первая редакция встроила проверку в `demo/docs/capture.mjs`, и гейт документации сразу покраснел: его sha записан в индексе скриншотов, и `scripts/check-docs.mjs` их сверяет. То есть проверка, которая ничего не рисует, стоила бы пересъёмки всех картинок документации и визуальной приёмки владельца. Поэтому для документации отказ живёт шагом раньше — `npm run docs:capture` вызывает `scripts/assert-capture-env.mjs` — и шагом позже, на приёмке. По той же причине гейт golden стоит в `demo/golden/policy.mjs`, а не в `run.mjs`: последний входит в корпус sourceFingerprint. Тест закрепляет обе границы: гейт обязан быть в policy.mjs и в npm-скрипте и обязан отсутствовать в двух фингерпринтуемых файлах. Платформа в юнитах — параметр, а не `process.platform`: иначе тест был бы зелёным на Linux и красным на машине владельца, то есть тестом про хост, а не про правило. AGENTS.md приведён к состоянию после #401 (принимается любая среда, доказавшая себя байтовым совпадением непринятых кадров) и разводит проверку и съёмку — прежний текст сливал их в «advisory» и утверждал «accepted only on a complete Linux CI artefact». Свидетели, все проверены отрицательным прогоном: снятый гейт съёмки, гейт, отказывающий и на verify, обход без причины, отказ без команды, убранная приписка про среду, отцепленные гейты обеих приёмок, переставшая бросать обёртка, npm-скрипт без проверки и возврат гейта в каждый из двух фингерпринтуемых файлов. Проверка подключения сначала смотрела только на импорт модуля и молча проходила, когда отказ заменяли на `void` — теперь она проверяет вызов бросающей обёртки. Гейты: npm test 1918 tests, 1917 pass, 0 fail; typecheck зелёный; check-docs зелёный (индекс скриншотов не задет); pytest без HA 378 passed, 3 skipped. Issue: #455 User-Visible: no
53 lines
3.1 KiB
JavaScript
53 lines
3.1 KiB
JavaScript
import {
|
|
captureEnvironment, foreignCaptureAllowance, foreignCaptureRefusal,
|
|
} from '../../scripts/capture-environment.mjs';
|
|
|
|
// The name must NOT end with `manifest.json`: the HACS submission check globs
|
|
// `*manifest.json` over the whole clone of the default branch and refuses a
|
|
// repository with more than one match (test/repo-hygiene.test.mjs).
|
|
export const GOLDEN_BASELINE_MANIFEST = 'baselines-index.json';
|
|
|
|
/**
|
|
* Проверка вызова — и заодно проверка среды съёмки (#455).
|
|
*
|
|
* Почему здесь, а не в `run.mjs`: `run.mjs` входит в корпус
|
|
* `sourceFingerprint`, и его правка объявляет устаревшими закоммиченный бандл,
|
|
* манифест скриншотов документации и индекс эталонов. Этот файл из корпуса
|
|
* исключён именно как «предикаты, вызываемые до и после съёмки», и уже
|
|
* вызывается первой строкой `run.mjs` — гейт встаёт без единой правки
|
|
* фингерпринтуемого файла.
|
|
*
|
|
* Платформа — ПАРАМЕТР, а не `process.platform` внутри: иначе юнит стал бы
|
|
* зелёным на Linux и красным на машине владельца, то есть тестом про хост, а
|
|
* не про правило.
|
|
*/
|
|
export const assertGoldenInvocation = (mode, scenarioFilter = '', options = {}) => {
|
|
if (!['capture', 'verify'].includes(mode)) throw new Error(`unknown golden mode: ${mode}`);
|
|
if (mode === 'verify' && scenarioFilter)
|
|
throw new Error('golden verify must run the complete matrix; use capture for a diagnostic --scenario run');
|
|
// Диагностический прогон verify в чужой среде законен: он ничего не принимает.
|
|
if (mode !== 'capture') return;
|
|
const platform = options.platform ?? captureEnvironment().platform;
|
|
const allowance = options.allowance ?? foreignCaptureAllowance();
|
|
const { refusal, allowance: accepted } = foreignCaptureRefusal({
|
|
platform, kind: 'golden', stage: 'capture', allowance,
|
|
});
|
|
if (refusal) throw new Error(refusal);
|
|
if (accepted) console.log(`Чужая среда съёмки разрешена осознанно: ${accepted}`);
|
|
};
|
|
|
|
/** A reviewed golden matrix is exact: neither an orphan PNG nor a stale hash
|
|
* entry may survive after a scenario is removed or renamed. */
|
|
export const goldenScenarioSetsMatch = (expected, indexed, baselineFiles) => {
|
|
const normalized = (values) => [...new Set(values)].sort();
|
|
const wanted = normalized(expected);
|
|
return JSON.stringify(normalized(indexed)) === JSON.stringify(wanted)
|
|
&& JSON.stringify(normalized(baselineFiles)) === JSON.stringify(wanted);
|
|
};
|
|
|
|
export const goldenRunFailed = (mode, manifestValid, results) => {
|
|
if (results.some((result) => result.status === 'error')) return true;
|
|
return mode === 'verify'
|
|
&& (!manifestValid || results.some((result) => result.status !== 'passed'));
|
|
};
|