Files
houseplan-card/test/houseplan-source.mjs
Claude 47469bab22 Монолит: мёртвый код снят по noUnusedLocals, связность измеряется шестью числами и гейтом (#624)
Карточка и редакторский рантайм держали ≈380 неиспользуемых импортов, 56
мёртвых объявлений и дублей типов (warm-boot, LS_*, GLOW_*, debounce,
navigate, lruRead — копии карточки в рантайме) и 112 приватных членов
карточки, которых не читал никто — делегаты `_editorRuntimeOrThrow()._x()`,
оставшиеся от выноса #425, и аксессоры glow-состояния. Всё это снято; в 9
других файлах — по одиночной ошибке. Делегаты и поля, которых касаются
браузерные смоки (`card._x(...)`), оставлены и посчитаны отдельно.

Гейт `npm run lint:unused` (scripts/unused-locals-gate.mjs, в gate:small и
Validate после сборки): `tsc --noUnusedLocals` чист, кроме приватных членов
карточки из порта HouseplanEditorHostPort / `host.` (portPrivates) и членов,
которых зовёт харнесс (harnessPrivates); храповик по шести числам
scripts/monolith-metrics.mjs против scripts/monolith-baseline.json —
delegates 260→159, portMembers 350, hostRefs 4948, portPrivates 96,
harnessPrivates 107, bundleBytes 2 510 141→2 500 387. `npm run inventory`
печатает те же числа. Заморозка 54 тестов, читающих монолит как текст
(test/monolith-text-anchors.test.mjs); PROCESS.md §2.7 — правило.

Логический исходник для контрактных тестов (test/houseplan-source.mjs)
дописывает члены рантайма без делегата в карточке — контракт продукта не
зависит от наличия заглушки. Потолки ядер и initial gzip опущены на выигрыш
(292 000 → 290 400). Бандл пересобран, три копии синхронны.

Issue: #624
User-Visible: no
2026-09-23 21:09:44 +03:00

122 lines
7.0 KiB
JavaScript

import { readFileSync } from 'node:fs';
import ts from 'typescript';
/**
* Source-contract tests historically inspected one monolithic card file.
* Editor implementations are now lazy, so expose the same logical production
* surface with the implementation before its typed host stubs.
*
* #592: разметка четырёх диалогов настроек уехала из редакторского рантайма в
* отдельные модули, и в классе остались однострочные делегаты. Контрактные
* тесты обязаны видеть ту же разметку, что и раньше, поэтому сборка логического
* исходника подставляет тело вынесенной функции вместо делегата. Иначе шесть
* тестов пришлось бы переучивать на новые пути — то есть ослаблять их ровно в
* тот момент, когда они должны доказать, что перенос ничего не изменил.
*/
const DIALOG_MODULES = [
['../src/editors/marker-dialog.ts', 'renderMarkerDialog', '_renderMarkerDialog'],
['../src/editors/space-settings-dialog.ts', 'renderSpaceSettingsDialog', '_renderSpaceDialog'],
['../src/editors/general-settings-dialog.ts', 'renderGeneralSettingsDialog', '_renderSettingsDialog'],
['../src/editors/room-settings-dialog.ts', 'renderRoomSettingsDialog', '_renderRoomDialog'],
];
/**
* #600: форма пространства общая для редактора и онбординга и живёт отдельным
* модулем; точка входа `renderSpaceSettingsDialog` — тонкая обёртка. Чтобы
* контрактные тесты продолжали видеть разметку (счётчики пикеров, kind
* диалога, футер), текст общих модулей дописывается к логическому исходнику.
* Обращения `port.host.` и `host.` приводятся к `this.`, как и `this.host.`.
*/
const SHARED_DIALOG_MODULES = ['../src/editors/space-form.ts'];
export function readHouseplanProductionSource() {
const runtime = readFileSync(
new URL('../src/houseplan-editor-runtime.ts', import.meta.url),
'utf8',
);
const card = readFileSync(new URL('../src/houseplan-card.ts', import.meta.url), 'utf8');
const parse = (name, source) => ts.createSourceFile(
name, source, ts.ScriptTarget.Latest, true, ts.ScriptKind.TS,
);
const runtimeFile = parse('houseplan-editor-runtime.ts', runtime);
const cardFile = parse('houseplan-card.ts', card);
const classOf = (file, name) => file.statements.find(
(statement) => ts.isClassDeclaration(statement) && statement.name?.text === name,
);
const runtimeClass = classOf(runtimeFile, 'HouseplanEditorRuntime');
const cardClass = classOf(cardFile, 'HouseplanCard');
if (!runtimeClass || !cardClass) throw new Error('Houseplan source classes are missing');
const nameOf = (member, file) => member.name?.getText(file);
const implementations = new Map(runtimeClass.members
.map((member) => [nameOf(member, runtimeFile), member])
.filter(([name]) => name));
// Тело вынесенного диалога возвращается на место делегата под именем метода:
// ниже по коду ветка не различает, откуда пришёл текст.
const moved = new Map();
for (const [path, exported, method] of DIALOG_MODULES) {
const source = readFileSync(new URL(path, import.meta.url), 'utf8');
const file = parse(path, source);
const fn = file.statements.find((statement) => ts.isFunctionDeclaration(statement)
&& statement.name?.text === exported);
if (!fn) throw new Error(`Houseplan dialog module is missing ${exported}`);
// #600: тела диалогов держат `const host = this.host` и пишут `host.…`;
// для контрактных регулярок это тот же `this.…`.
const body = source.slice(source.indexOf('{', fn.getStart(file)), fn.end)
.replaceAll(/(?<!this\.)\bhost\./g, 'this.');
moved.set(method, body);
}
const replacements = [];
for (const member of cardClass.members) {
const name = nameOf(member, cardFile);
const implementation = implementations.get(name);
if (!implementation) continue;
const baseText = card.slice(member.getStart(cardFile), member.end);
if (!baseText.includes('_editorRuntime')) continue;
const visibility = /^(private|protected|public)\b/.exec(baseText)?.[1] || 'private';
const runtimeText = moved.has(name)
? `public ${name}(): TemplateResult ${moved.get(name)}`
: runtime.slice(implementation.getStart(runtimeFile), implementation.end);
const implementationText = runtimeText
.replace(/^public\b/, visibility)
.replaceAll('this.host.', 'this.');
replacements.push({
name,
from: member.getStart(cardFile),
to: member.end,
text: implementationText,
});
}
let reconstructed = card;
for (const replacement of replacements.sort((a, b) => b.from - a.from)) {
reconstructed = reconstructed.slice(0, replacement.from)
+ replacement.text
+ reconstructed.slice(replacement.to);
}
// #624: мёртвые делегаты карточки удалены — реализация, у которой в
// карточке больше нет заглушки, живёт только в рантайме. Логическая
// поверхность продукта от этого не меняется, поэтому такие члены
// дописываются к исходнику в том же виде, в каком раньше подставлялись на
// место делегата. Контракт «что делает продукт» не должен зависеть от того,
// есть ли у метода однострочная заглушка в карточке.
const delegated = new Set(replacements.map((replacement) => replacement.name));
const appended = [];
for (const [name, implementation] of implementations) {
if (delegated.has(name) || name === 'host') continue;
const text = moved.has(name)
? `public ${name}(): TemplateResult ${moved.get(name)}`
: runtime.slice(implementation.getStart(runtimeFile), implementation.end);
appended.push(text.replace(/^public\b/, 'private').replaceAll('this.host.', 'this.'));
}
if (appended.length) {
reconstructed += `\n// --- editor runtime members without a card delegate (#624) ---\n${appended.join('\n\n')}\n`;
}
for (const path of SHARED_DIALOG_MODULES) {
const shared = readFileSync(new URL(path, import.meta.url), 'utf8')
.replaceAll('port.host.', 'this.')
.replaceAll('port.help(', 'this._help(')
.replaceAll(/(?<!this\.)\bhost\./g, 'this.');
reconstructed += `\n// --- shared dialog module ${path} ---\n${shared}`;
}
return reconstructed;
}