mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 11:18:48 +00:00
Карточка и редакторский рантайм держали ≈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
122 lines
7.0 KiB
JavaScript
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;
|
|
}
|