# 问题 修复 文件 1 前端构建失败(引号错误) size="small type=" → size="small" type=" PosterHistoryPage.vue 2 migrate_014 ORM vs 缺失列 全部改为原始 SQL,不再引用 ORM 模型 migrate_014.py 3 cleanup 字段名错误 output_path → ppt_path cleanup.py 4 文案生成 case 越权 添加 case.user_id != user_id 校验 poster/service.py 5 存储路径未接通持久化卷 全部改用 get_storage_root()(默认 /app/api/storage/insurance) config.py, ppt/routes.py, poster/service.py, poster/tasks.py 高风险问题修复 # 问题 修复 文件 6 migrate_019 rollback 撤销成功字段 每个 ALTER 后立即 commit,失败只回滚当前语句 migrate_019.py 7 迁移锁 Windows 不兼容 + 句柄未持久化 全局变量保存锁句柄,支持 Windows msvcrt api/insurance/db/__init__.py 8 PDF 校验异常时放行 异常返回 False(文件损坏) security.py 9 健康检查始终返回成功 缺少关键资源时返回 503 + missing 列表 poster/routes.py 10 短密钥掩码泄露原值 ≤4 字符返回 **** ppt_admin_service.py 11 设置无键名白名单 添加 _ALLOWED_SETTING_KEYS 白名单 ppt_admin_service.py 12 容器重启任务永久 stuck 添加 recover_stale_tasks() 启动恢复函数 poster/tasks.py, ppt/parse_worker.py
880 lines
35 KiB
JavaScript
880 lines
35 KiB
JavaScript
/**
|
|
* CLI helper: find an element in source and wrap it in a variant container.
|
|
*
|
|
* Usage:
|
|
* node <scripts_path>/live-wrap.mjs --id SESSION_ID --count N --query "hero-combined-left" [--file path]
|
|
*
|
|
* Searches project files for the element matching the query (class name, ID, or
|
|
* text snippet), wraps it with the variant scaffolding, and prints the file path
|
|
* + line range where the agent should insert variant HTML.
|
|
*
|
|
* This replaces 3-4 agent tool calls (grep + read + edit) with a single CLI call.
|
|
*/
|
|
|
|
import fs from 'node:fs';
|
|
import path from 'node:path';
|
|
import { isGeneratedFile } from './lib/is-generated.mjs';
|
|
import { resolveLiveTemplateExtensions } from './lib/template-extensions.mjs';
|
|
import { readBuffer as readManualEditsBuffer } from './live/manual-edits-buffer.mjs';
|
|
import { findSourceFile } from './live/source-search.mjs';
|
|
import {
|
|
buildSvelteComponentCssAuthoring,
|
|
scaffoldSvelteComponentSession,
|
|
shouldUseSvelteComponentInjection,
|
|
} from './live/svelte-component.mjs';
|
|
|
|
export async function wrapCli() {
|
|
const args = process.argv.slice(2);
|
|
|
|
if (args.includes('--help') || args.includes('-h')) {
|
|
console.log(`Usage: impeccable wrap [options]
|
|
|
|
Find an element in source and wrap it in a variant container.
|
|
|
|
Required:
|
|
--id ID Session ID for the variant wrapper
|
|
--count N Number of expected variants (1-8)
|
|
|
|
Element identification (at least one required):
|
|
--element-id ID HTML id attribute of the element
|
|
--classes A,B,C Comma- or space-separated CSS class names
|
|
--tag TAG Tag name (div, section, etc.)
|
|
--query TEXT Fallback: raw text to search for
|
|
|
|
Optional:
|
|
--file PATH Source file to search in (skips auto-detection)
|
|
--text TEXT Picked element's textContent. Used to disambiguate when
|
|
classes/tag match multiple sibling elements (e.g. a list
|
|
of <Card>s with the same className). Pass the first ~80
|
|
chars of event.element.textContent.
|
|
--page-url URL Current page URL. Required when pending manual edits may
|
|
affect the picked source block. Pending edits are filtered
|
|
to this page so an edit on /a doesn't bleed into /b.
|
|
--help Show this help message
|
|
|
|
Output (JSON):
|
|
{ file, startLine, endLine, insertLine, commentSyntax }
|
|
|
|
The agent should insert variant HTML at insertLine.`);
|
|
process.exit(0);
|
|
}
|
|
|
|
const id = argVal(args, '--id');
|
|
const count = parseInt(argVal(args, '--count') || '3');
|
|
const elementId = argVal(args, '--element-id');
|
|
const classes = argVal(args, '--classes');
|
|
const tag = argVal(args, '--tag');
|
|
const query = argVal(args, '--query');
|
|
const filePath = argVal(args, '--file');
|
|
const text = argVal(args, '--text');
|
|
const pageUrl = argVal(args, '--page-url');
|
|
|
|
if (!id) { console.error('Missing --id'); process.exit(1); }
|
|
if (!elementId && !classes && !query) {
|
|
console.error('Need at least one of: --element-id, --classes, --query');
|
|
process.exit(1);
|
|
}
|
|
|
|
// Build search queries in priority order (most specific first)
|
|
const queries = buildSearchQueries(elementId, classes, tag, query);
|
|
|
|
const genOpts = { cwd: process.cwd() };
|
|
|
|
// Find the source file. Generated files are excluded from auto-search so we
|
|
// don't silently write variants into a file the next build will wipe.
|
|
let targetFile = filePath;
|
|
let matchedQuery = null;
|
|
if (!targetFile) {
|
|
for (const q of queries) {
|
|
targetFile = findFileWithQuery(q, process.cwd(), genOpts);
|
|
if (targetFile) { matchedQuery = q; break; }
|
|
}
|
|
if (!targetFile) {
|
|
// Nothing in source. Did the element show up in a generated file? That
|
|
// tells the agent "fall back to the agent-driven flow" vs "element just
|
|
// doesn't exist in this project."
|
|
let generatedHit = null;
|
|
for (const q of queries) {
|
|
generatedHit = findFileWithQuery(q, process.cwd(), { ...genOpts, includeGenerated: true });
|
|
if (generatedHit) break;
|
|
}
|
|
if (generatedHit) {
|
|
console.error(JSON.stringify({
|
|
error: 'element_not_in_source',
|
|
fallback: 'agent-driven',
|
|
generatedMatch: path.relative(process.cwd(), generatedHit),
|
|
hint: 'Element found only in a generated file. See "Handle fallback" in live.md.',
|
|
}));
|
|
} else {
|
|
console.error(JSON.stringify({
|
|
error: 'element_not_found',
|
|
fallback: 'agent-driven',
|
|
hint: 'Element not found in any project file. It may be runtime-injected (JS component, etc.). See "Handle fallback" in live.md.',
|
|
}));
|
|
}
|
|
process.exit(1);
|
|
}
|
|
} else {
|
|
if (isGeneratedFile(targetFile, genOpts)) {
|
|
console.error(JSON.stringify({
|
|
error: 'file_is_generated',
|
|
fallback: 'agent-driven',
|
|
file: path.relative(process.cwd(), path.resolve(process.cwd(), targetFile)),
|
|
hint: 'Explicit --file points at a generated file. Writing here gets wiped by the next build. See "Handle fallback" in live.md.',
|
|
}));
|
|
process.exit(1);
|
|
}
|
|
matchedQuery = queries[0];
|
|
}
|
|
|
|
const content = fs.readFileSync(targetFile, 'utf-8');
|
|
const lines = content.split('\n');
|
|
|
|
// Find the element, trying each query in priority order. When `--text` is
|
|
// supplied, collect every candidate the queries surface and disambiguate
|
|
// by the picked element's textContent. Without `--text`, fall back to the
|
|
// legacy first-match behavior so unmodified callers keep working.
|
|
let match = null;
|
|
if (text) {
|
|
const candidates = [];
|
|
for (const q of queries) {
|
|
const all = findAllElements(lines, q, tag);
|
|
for (const c of all) {
|
|
if (!candidates.some((x) => x.startLine === c.startLine)) {
|
|
candidates.push(c);
|
|
}
|
|
}
|
|
// Once a more-specific query (ID, full className combo) yielded a unique
|
|
// result, stop — falling through to the loose tag+single-class query
|
|
// would readmit the siblings we just disambiguated past.
|
|
if (candidates.length === 1) break;
|
|
}
|
|
if (candidates.length === 0) {
|
|
console.error(JSON.stringify({ error: 'Found file but could not locate element in ' + targetFile + '. Searched for: ' + queries.join(', ') }));
|
|
process.exit(1);
|
|
}
|
|
if (candidates.length === 1) {
|
|
match = candidates[0];
|
|
} else {
|
|
const filtered = filterByText(candidates, lines, text);
|
|
if (filtered.length === 1) {
|
|
match = filtered[0];
|
|
} else if (filtered.length === 0) {
|
|
const normalizedText = String(text).replace(/\s+/g, ' ').trim();
|
|
if (normalizedText.length < 8) {
|
|
// Very short labels cannot disambiguate siblings reliably. Preserve
|
|
// the legacy behavior for these low-information picker events.
|
|
match = candidates[0];
|
|
} else {
|
|
// Rendered text that is absent from every candidate usually means
|
|
// the source uses expressions or component props. Picking the first
|
|
// same-class sibling silently edits the wrong instance (observed on
|
|
// Astro result cards), so stop and surface every candidate instead.
|
|
console.error(JSON.stringify({
|
|
error: 'element_ambiguous',
|
|
fallback: 'agent-driven',
|
|
reason: 'rendered_text_not_in_source',
|
|
file: path.relative(process.cwd(), targetFile),
|
|
candidates: candidates.map((c) => ({
|
|
startLine: c.startLine + 1,
|
|
endLine: c.endLine + 1,
|
|
})),
|
|
hint: 'Rendered text does not occur in any matching source branch. The element may use dynamic props or expressions; inspect the candidates and wrap the intended instance manually.',
|
|
}));
|
|
process.exit(1);
|
|
}
|
|
} else {
|
|
// Multiple candidates ALSO match the text. Truly ambiguous — refuse
|
|
// rather than pick wrong, and hand the agent the candidate locations
|
|
// so it can disambiguate by reading the file.
|
|
console.error(JSON.stringify({
|
|
error: 'element_ambiguous',
|
|
fallback: 'agent-driven',
|
|
file: path.relative(process.cwd(), targetFile),
|
|
candidates: filtered.map((c) => ({
|
|
startLine: c.startLine + 1,
|
|
endLine: c.endLine + 1,
|
|
})),
|
|
hint: 'Multiple source elements match both classes/tag and textContent. Pass --element-id, a more specific --text, or write the wrapper manually. See "Handle fallback" in live.md.',
|
|
}));
|
|
process.exit(1);
|
|
}
|
|
}
|
|
} else {
|
|
for (const q of queries) {
|
|
match = findElement(lines, q, tag);
|
|
if (match) break;
|
|
}
|
|
if (!match) {
|
|
console.error(JSON.stringify({ error: 'Found file but could not locate element in ' + targetFile + '. Searched for: ' + queries.join(', ') }));
|
|
process.exit(1);
|
|
}
|
|
}
|
|
|
|
const { startLine, endLine } = match;
|
|
const commentSyntax = detectCommentSyntax(targetFile);
|
|
const styleMode = detectStyleMode(targetFile);
|
|
const isJsx = commentSyntax.open === '{/*';
|
|
const indent = lines[startLine].match(/^(\s*)/)[1];
|
|
|
|
// Extract the original element. Reindent under the wrapper while preserving
|
|
// the relative depth between lines — `l.trimStart()` would strip ALL leading
|
|
// whitespace and collapse e.g. `<aside>`/` <h1>`/`</aside>` (6/8/6 spaces)
|
|
// to a single uniform indent, so on accept/discard the round-trip restores
|
|
// the inner element at its parent's depth instead of nested inside it.
|
|
// Strip only the COMMON minimum leading whitespace across the picked lines;
|
|
// `deindentContent` on the accept side already mirrors this convention.
|
|
let originalLines = lines.slice(startLine, endLine + 1);
|
|
|
|
// Buffer-aware "original" content: if the user has pending manual edits for
|
|
// this page whose originalText appears in the picked source range, apply
|
|
// them so the wrap block's "original" variant reflects what the user was
|
|
// looking at (their edited DOM), not the raw source. Source itself stays
|
|
// untouched here — only the wrap block's embedded "original" copy is
|
|
// adjusted. The pending edits remain in the buffer until committed.
|
|
//
|
|
// Apply buffered edits only when the browser provided the current page URL.
|
|
// Without it, fail if pending edits plausibly touch this exact source range;
|
|
// otherwise skip buffer awareness so unrelated staged edits on another page
|
|
// do not block normal wrap work.
|
|
let pendingBuffer = { entries: [] };
|
|
try { pendingBuffer = readManualEditsBuffer(process.cwd()); } catch {}
|
|
const pendingEntriesForTarget = pageUrl
|
|
? []
|
|
: pendingEntriesThatMayAffectWrap(pendingBuffer.entries, targetFile, originalLines, startLine, process.cwd());
|
|
if (pendingEntriesForTarget.length > 0) {
|
|
console.error(JSON.stringify({
|
|
error: 'missing_page_url_with_pending_edits',
|
|
pendingEntries: pendingEntriesForTarget.length,
|
|
hint: 'Pending manual edits may affect the selected source block. Pass --page-url=$event.pageUrl so the wrap block reflects the user\'s staged DOM.',
|
|
}));
|
|
process.exit(1);
|
|
}
|
|
if (pageUrl) {
|
|
const failedBufferedOps = [];
|
|
for (const entry of pendingBuffer.entries || []) {
|
|
if (entry.pageUrl !== pageUrl) continue;
|
|
for (const op of entry.ops || []) {
|
|
const mayAffectWrap = manualEditMayAffectWrap(op, targetFile, originalLines, startLine, process.cwd());
|
|
const result = applyBufferedManualEditToLines(originalLines, startLine, op);
|
|
if (result.changed) {
|
|
originalLines = result.lines;
|
|
continue;
|
|
}
|
|
if (!mayAffectWrap) continue;
|
|
failedBufferedOps.push({
|
|
entryId: entry.id,
|
|
ref: op?.ref || null,
|
|
originalText: op?.originalText || null,
|
|
reason: 'ambiguous_or_unmatched_pending_edit',
|
|
});
|
|
}
|
|
}
|
|
if (failedBufferedOps.length > 0) {
|
|
console.error(JSON.stringify({
|
|
error: 'manual_edit_buffer_apply_failed',
|
|
pendingOps: failedBufferedOps,
|
|
hint: 'A staged copy edit appears to affect the selected source block, but could not be applied unambiguously to the wrap original. Apply or discard copy edits first, or write the wrapper manually.',
|
|
}));
|
|
process.exit(1);
|
|
}
|
|
}
|
|
|
|
const originalBaseIndent = minLeadingSpaces(originalLines);
|
|
const reindentOriginal = (extra) => originalLines
|
|
.map((l) => (l.trim() === '' ? '' : indent + extra + l.slice(originalBaseIndent)))
|
|
.join('\n');
|
|
const originalIndented = reindentOriginal(' ');
|
|
const relTargetFile = path.relative(process.cwd(), targetFile).split(path.sep).join('/');
|
|
const useSvelteComponent = shouldUseSvelteComponentInjection(targetFile);
|
|
const useFrameworkComponent = useSvelteComponent;
|
|
|
|
// Wrapper attributes differ by syntax. HTML allows plain string attrs;
|
|
// JSX requires object-literal style and parses string attrs as HTML (which
|
|
// either type-errors or renders a literal CSS string).
|
|
const styleContents = isJsx ? 'style={{ display: "contents" }}' : 'style="display: contents"';
|
|
|
|
// JSX/TSX guard: the picked element occupies a single JSX child slot
|
|
// (inside `return (...)`, an array `.map(...)`, an `asChild` branch, or
|
|
// any other expression position). Replacing it with `comment + <div> +
|
|
// comment` yields three adjacent siblings — invalid JSX. We can't use a
|
|
// Fragment `<></>` either: parents that clone children (Radix `asChild`,
|
|
// Headless UI, etc.) hit "Invalid prop supplied to React.Fragment" when
|
|
// they try to pass an `id` through.
|
|
//
|
|
// Solution: keep the wrapper `<div>` as the single JSX-slot child and
|
|
// tuck both marker comments INSIDE it. accept/discard then expands its
|
|
// replacement range to include the wrapper's `<div>` open / close lines
|
|
// so the entire scaffold gets removed cleanly.
|
|
const wrapperLines = isJsx ? [
|
|
indent + '<div data-impeccable-variants="' + id + '" data-impeccable-variant-count="' + count + '"' + ' ' + styleContents + '>',
|
|
indent + ' ' + commentSyntax.open + ' impeccable-variants-start ' + id + ' ' + commentSyntax.close,
|
|
indent + ' ' + commentSyntax.open + ' Original ' + commentSyntax.close,
|
|
indent + ' <div data-impeccable-variant="original">',
|
|
reindentOriginal(' '),
|
|
indent + ' </div>',
|
|
indent + ' ' + commentSyntax.open + ' Variants: insert below this line ' + commentSyntax.close,
|
|
indent + ' ' + commentSyntax.open + ' impeccable-variants-end ' + id + ' ' + commentSyntax.close,
|
|
indent + '</div>',
|
|
] : [
|
|
indent + commentSyntax.open + ' impeccable-variants-start ' + id + ' ' + commentSyntax.close,
|
|
indent + '<div data-impeccable-variants="' + id + '" data-impeccable-variant-count="' + count + '"' + ' ' + styleContents + '>',
|
|
indent + ' ' + commentSyntax.open + ' Original ' + commentSyntax.close,
|
|
indent + ' <div data-impeccable-variant="original">',
|
|
originalIndented,
|
|
indent + ' </div>',
|
|
indent + ' ' + commentSyntax.open + ' Variants: insert below this line ' + commentSyntax.close,
|
|
indent + '</div>',
|
|
indent + commentSyntax.open + ' impeccable-variants-end ' + id + ' ' + commentSyntax.close,
|
|
];
|
|
|
|
let outputFile = targetFile;
|
|
let outputLines;
|
|
let outputStartLine = startLine + 1;
|
|
let outputEndLine = startLine + wrapperLines.length + (originalLines.length - 1);
|
|
let insertLine;
|
|
let svelteSession = null;
|
|
|
|
if (useSvelteComponent) {
|
|
// Svelte/SvelteKit resets component-local state on markup HMR updates.
|
|
// Keep generation source-neutral: agents write real variant components
|
|
// under the generated componentDir, the browser mounts them into the live
|
|
// DOM, and live-accept.mjs inlines the accepted variant back into the route.
|
|
svelteSession = scaffoldSvelteComponentSession({
|
|
id,
|
|
count,
|
|
sourceFile: relTargetFile,
|
|
sourceStartLine: startLine + 1,
|
|
sourceEndLine: endLine + 1,
|
|
originalLines,
|
|
cwd: process.cwd(),
|
|
});
|
|
outputFile = path.resolve(process.cwd(), svelteSession.manifestFile);
|
|
outputStartLine = 1;
|
|
outputEndLine = 1;
|
|
insertLine = 1;
|
|
} else {
|
|
// Replace the original element with the wrapper
|
|
const newLines = [
|
|
...lines.slice(0, startLine),
|
|
...wrapperLines,
|
|
...lines.slice(endLine + 1),
|
|
];
|
|
fs.writeFileSync(targetFile, newLines.join('\n'), 'utf-8');
|
|
|
|
// Calculate insert line (the "insert below this line" comment).
|
|
// 0-indexed file position. Both HTML and JSX wrappers have 6 lines above
|
|
// the insert marker (HTML: start-comment + outer-div + Original-comment +
|
|
// original-div + content + close-original-div; JSX: outer-div +
|
|
// start-comment + Original-comment + original-div + content +
|
|
// close-original-div). Multi-line originals push the marker by their
|
|
// extra line count.
|
|
insertLine = startLine + 6 + (originalLines.length - 1) + 1;
|
|
}
|
|
|
|
const outputRelFile = path.relative(process.cwd(), outputFile).split(path.sep).join('/');
|
|
|
|
const svelteComponentAuthoring = useSvelteComponent ? buildSvelteComponentCssAuthoring(count) : null;
|
|
const componentSession = svelteSession;
|
|
const componentPreviewMode = useSvelteComponent ? 'svelte-component' : undefined;
|
|
const previewMode = componentPreviewMode;
|
|
|
|
console.log(JSON.stringify({
|
|
file: outputRelFile,
|
|
sourceFile: useFrameworkComponent ? relTargetFile : undefined,
|
|
previewMode,
|
|
componentDir: componentSession?.componentDir,
|
|
propContract: componentSession?.propContract,
|
|
sourceStartLine: useFrameworkComponent ? startLine + 1 : undefined,
|
|
sourceEndLine: useFrameworkComponent ? endLine + 1 : undefined,
|
|
startLine: outputStartLine, // 1-indexed for the agent
|
|
// wrapperLines is an array but one element (the original-content slot)
|
|
// is a `\n`-joined multi-line string, so the actual file-row count is
|
|
// wrapperLines.length + (originalLines.length - 1). Without the offset,
|
|
// endLine pointed inside the wrapper for any picked element that
|
|
// spanned more than one source line.
|
|
endLine: outputEndLine, // 1-indexed
|
|
insertLine, // 1-indexed: where variants go
|
|
commentSyntax: commentSyntax,
|
|
styleMode: componentPreviewMode || styleMode.mode,
|
|
styleTag: useFrameworkComponent ? null : styleMode.styleTag,
|
|
cssSelectorPrefixExamples: useFrameworkComponent ? [] : buildCssSelectorPrefixExamples(styleMode.mode, count),
|
|
cssAuthoring: svelteComponentAuthoring || buildCssAuthoring(styleMode, count),
|
|
originalLineCount: originalLines.length,
|
|
}));
|
|
}
|
|
|
|
// ---------------------------------------------------------------------------
|
|
// Helpers
|
|
// ---------------------------------------------------------------------------
|
|
|
|
function argVal(args, flag) {
|
|
const prefix = flag + '=';
|
|
for (const arg of args) {
|
|
if (arg.startsWith(prefix)) return arg.slice(prefix.length);
|
|
}
|
|
const idx = args.indexOf(flag);
|
|
return idx !== -1 && idx + 1 < args.length ? args[idx + 1] : null;
|
|
}
|
|
|
|
function pendingEntriesThatMayAffectWrap(entries, targetFile, originalLines, selectionStartLine, cwd) {
|
|
const targetAbs = path.resolve(cwd, targetFile);
|
|
return (entries || []).filter((entry) => {
|
|
return (entry.ops || []).some((op) => {
|
|
return manualEditMayAffectWrap(op, targetAbs, originalLines, selectionStartLine, cwd);
|
|
});
|
|
});
|
|
}
|
|
|
|
function manualEditMayAffectWrap(op, targetFile, originalLines, selectionStartLine, cwd) {
|
|
const targetAbs = path.resolve(cwd, targetFile);
|
|
if (manualEditHintFallsInsideSelection(op, targetAbs, originalLines, selectionStartLine, cwd)) return true;
|
|
if (manualEditLocatorMatchesSelection(op, originalLines)) return true;
|
|
if (typeof op?.originalText === 'string' && op.originalText.length > 0) {
|
|
return originalLines.join('\n').includes(op.originalText);
|
|
}
|
|
return false;
|
|
}
|
|
|
|
function manualEditHintFallsInsideSelection(op, targetAbs, originalLines, selectionStartLine, cwd) {
|
|
const hintFile = op?.sourceHint?.file;
|
|
const hintedLine = Number(op?.sourceHint?.line);
|
|
if (!hintFile || !Number.isFinite(hintedLine)) return false;
|
|
const hintAbs = path.isAbsolute(hintFile) ? hintFile : path.resolve(cwd, hintFile);
|
|
if (path.resolve(hintAbs) !== targetAbs) return false;
|
|
const hintedIndex = hintedLine - 1 - selectionStartLine;
|
|
return hintedIndex >= 0
|
|
&& hintedIndex < originalLines.length
|
|
&& typeof op?.originalText === 'string'
|
|
&& originalLines[hintedIndex].includes(op.originalText);
|
|
}
|
|
|
|
function manualEditLocatorMatchesSelection(op, originalLines) {
|
|
if (!op || typeof op.originalText !== 'string' || op.originalText.length === 0) return false;
|
|
return originalLines.some((line) => (
|
|
line.includes(op.originalText) && lineMatchesManualEditLocator(line, op)
|
|
));
|
|
}
|
|
|
|
function applyBufferedManualEditToLines(originalLines, selectionStartLine, op) {
|
|
if (
|
|
!op
|
|
|| typeof op.originalText !== 'string'
|
|
|| op.originalText.length === 0
|
|
|| typeof op.newText !== 'string'
|
|
) {
|
|
return { lines: originalLines, changed: false };
|
|
}
|
|
|
|
const replaceLine = (lineIndex) => ({
|
|
lines: originalLines.map((line, index) => (
|
|
index === lineIndex ? replaceOnce(line, op.originalText, op.newText) : line
|
|
)),
|
|
changed: true,
|
|
});
|
|
|
|
const hintedLine = Number(op.sourceHint?.line);
|
|
if (Number.isFinite(hintedLine)) {
|
|
const hintedIndex = hintedLine - 1 - selectionStartLine;
|
|
if (hintedIndex >= 0 && hintedIndex < originalLines.length && originalLines[hintedIndex].includes(op.originalText)) {
|
|
return replaceLine(hintedIndex);
|
|
}
|
|
}
|
|
|
|
const locatorMatches = [];
|
|
for (let index = 0; index < originalLines.length; index += 1) {
|
|
const line = originalLines[index];
|
|
if (!line.includes(op.originalText)) continue;
|
|
if (!lineMatchesManualEditLocator(line, op)) continue;
|
|
locatorMatches.push(index);
|
|
}
|
|
if (locatorMatches.length === 1) return replaceLine(locatorMatches[0]);
|
|
|
|
const originalBlock = originalLines.join('\n');
|
|
if (countOccurrences(originalBlock, op.originalText) === 1) {
|
|
return {
|
|
lines: replaceOnce(originalBlock, op.originalText, op.newText).split('\n'),
|
|
changed: true,
|
|
};
|
|
}
|
|
|
|
return { lines: originalLines, changed: false };
|
|
}
|
|
|
|
function lineMatchesManualEditLocator(line, op) {
|
|
if (op.tag) {
|
|
const tagRe = new RegExp('<\\s*' + escapeRegExp(op.tag) + '(?=[\\s>/]|$)', 'i');
|
|
if (!tagRe.test(line)) return false;
|
|
}
|
|
|
|
if (op.elementId) {
|
|
const id = escapeRegExp(op.elementId);
|
|
const idRe = new RegExp('\\bid\\s*=\\s*["\']' + id + '["\']');
|
|
if (!idRe.test(line)) return false;
|
|
}
|
|
|
|
const classes = Array.isArray(op.classes) ? op.classes.filter(Boolean) : [];
|
|
for (const className of classes) {
|
|
if (!line.includes(className)) return false;
|
|
}
|
|
|
|
return true;
|
|
}
|
|
|
|
function replaceOnce(value, needle, replacement) {
|
|
const index = value.indexOf(needle);
|
|
if (index === -1) return value;
|
|
return value.slice(0, index) + replacement + value.slice(index + needle.length);
|
|
}
|
|
|
|
function countOccurrences(value, needle) {
|
|
if (!needle) return 0;
|
|
let count = 0;
|
|
let index = 0;
|
|
while (true) {
|
|
index = value.indexOf(needle, index);
|
|
if (index === -1) return count;
|
|
count += 1;
|
|
index += needle.length;
|
|
}
|
|
}
|
|
|
|
function escapeRegExp(value) {
|
|
return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
|
|
}
|
|
|
|
/**
|
|
* Build search query strings in priority order (most specific first).
|
|
* ID is most reliable, then specific class combos, then single classes, then raw query.
|
|
*/
|
|
function buildSearchQueries(elementId, classes, tag, query) {
|
|
const queries = [];
|
|
|
|
// 1. ID is the most specific
|
|
if (elementId) {
|
|
queries.push('id="' + elementId + '"');
|
|
}
|
|
|
|
// 2. Full class attribute match (for elements with distinctive multi-class combos).
|
|
// Emit both class="..." (HTML) and className="..." (React/JSX) so whichever
|
|
// convention the file uses will match.
|
|
if (classes) {
|
|
const classList = splitClassList(classes);
|
|
if (classList.length > 1) {
|
|
const joined = classList.join(' ');
|
|
const sorted = [...classList].sort((a, b) => b.length - a.length);
|
|
queries.push('class="' + joined + '"');
|
|
queries.push('className="' + joined + '"');
|
|
for (const className of sorted) {
|
|
queries.push(className);
|
|
}
|
|
} else if (classList.length === 1) {
|
|
queries.push(classList[0]);
|
|
}
|
|
}
|
|
|
|
// 3. Tag + class combo (e.g., <section class="hero">).
|
|
// Same dual-emit for JSX compatibility.
|
|
if (tag && classes) {
|
|
const firstClass = splitClassList(classes)[0];
|
|
queries.push('<' + tag + ' class="' + firstClass);
|
|
queries.push('<' + tag + ' className="' + firstClass);
|
|
}
|
|
|
|
// 4. Raw fallback query
|
|
if (query) {
|
|
queries.push(query);
|
|
}
|
|
|
|
return queries;
|
|
}
|
|
|
|
function splitClassList(classes) {
|
|
return String(classes).split(/[,\s]+/).map(c => c.trim()).filter(Boolean);
|
|
}
|
|
|
|
function attrEscapeDouble(str) {
|
|
return String(str)
|
|
.replace(/&/g, '&')
|
|
.replace(/"/g, '"')
|
|
.replace(/</g, '<')
|
|
.replace(/>/g, '>');
|
|
}
|
|
|
|
function detectCommentSyntax(filePath) {
|
|
const ext = path.extname(filePath).toLowerCase();
|
|
if (ext === '.jsx' || ext === '.tsx') {
|
|
return { open: '{/*', close: '*/}' };
|
|
}
|
|
// HTML, Vue, Svelte, Astro all use HTML comments
|
|
return { open: '<!--', close: '-->' };
|
|
}
|
|
|
|
function detectStyleMode(filePath) {
|
|
const ext = path.extname(filePath).toLowerCase();
|
|
if (ext === '.astro') {
|
|
return {
|
|
mode: 'astro-global-prefixed',
|
|
styleTag: '<style is:inline data-impeccable-css="SESSION_ID">',
|
|
};
|
|
}
|
|
return {
|
|
mode: 'scoped',
|
|
styleTag: '<style data-impeccable-css="SESSION_ID">',
|
|
};
|
|
}
|
|
|
|
function buildCssSelectorPrefixExamples(styleMode, count) {
|
|
if (styleMode !== 'astro-global-prefixed') return [];
|
|
return Array.from({ length: count }, (_, i) => `[data-impeccable-variant="${i + 1}"]`);
|
|
}
|
|
|
|
function buildCssAuthoring(styleMode, count) {
|
|
const variantNumbers = Array.from({ length: count }, (_, i) => i + 1);
|
|
if (styleMode.mode === 'astro-global-prefixed') {
|
|
return {
|
|
mode: styleMode.mode,
|
|
styleTag: styleMode.styleTag,
|
|
strategy: 'global-prefixed',
|
|
rulePattern: '[data-impeccable-variant="N"] > .variant-class { ... }',
|
|
selectorExamples: variantNumbers.map((n) => `[data-impeccable-variant="${n}"] > .variant-class`),
|
|
requirements: [
|
|
'Use the styleTag exactly; the is:inline attribute is required for this file.',
|
|
'Put raw CSS directly between the styleTag opening and a plain </style> close.',
|
|
'Prefix every preview selector with the matching [data-impeccable-variant="N"] selector.',
|
|
'Keep selectors anchored to the generated variant wrapper; do not rely on component CSS scoping for preview rules.',
|
|
],
|
|
forbidden: [
|
|
'Do not use @scope for this styleMode.',
|
|
'Do not wrap style content in a JSX/TSX template literal ({` ... `}); that syntax is for .tsx/.jsx only.',
|
|
'Do not put { immediately after the style opening tag; Astro parses { as expression syntax.',
|
|
],
|
|
};
|
|
}
|
|
return {
|
|
mode: styleMode.mode,
|
|
styleTag: styleMode.styleTag,
|
|
strategy: 'scope-rule',
|
|
rulePattern: '@scope ([data-impeccable-variant="N"]) { :scope > .variant-class { ... } }',
|
|
selectorExamples: variantNumbers.map((n) => `@scope ([data-impeccable-variant="${n}"]) { :scope > .variant-class { ... } }`),
|
|
requirements: [
|
|
'Use @scope blocks keyed to each [data-impeccable-variant="N"] wrapper.',
|
|
'Inside each @scope block, make :scope rules step into the replacement element with a descendant combinator.',
|
|
'Use the styleTag exactly; do not add framework-specific style attributes unless this object says to.',
|
|
],
|
|
forbidden: [
|
|
'Do not use global [data-impeccable-variant="N"] selector prefixes for this styleMode.',
|
|
'Do not add is:inline to the style tag for this styleMode.',
|
|
],
|
|
};
|
|
}
|
|
|
|
/**
|
|
* Search project files for the query string (class name, ID, etc.)
|
|
* Returns the first matching file path, or null.
|
|
*
|
|
* Only `node_modules`, `.git`, and `.impeccable` are skipped outright.
|
|
* dist/build/out are left to the isGeneratedFile guard so the
|
|
* `includeGenerated` second pass can still find the element there and report
|
|
* `generatedMatch`.
|
|
*/
|
|
function findFileWithQuery(query, cwd, genOpts = {}) {
|
|
return findSourceFile({
|
|
query,
|
|
cwd,
|
|
extensions: resolveLiveTemplateExtensions(cwd),
|
|
fileFilter: (filePath) => genOpts.includeGenerated || !isGeneratedFile(filePath, genOpts),
|
|
});
|
|
}
|
|
|
|
/**
|
|
* Regex that matches a tag opener on a line. Allows the tag name to be
|
|
* followed by whitespace, `>`, `/`, or end-of-line so that multi-line JSX
|
|
* openers (e.g. `<section\n className="..."\n>`) are recognised.
|
|
*/
|
|
const OPENER_RE = /<([A-Za-z][A-Za-z0-9]*)(?=[\s/>]|$)/;
|
|
|
|
/**
|
|
* Find the element's start and end line in the file.
|
|
*
|
|
* `query` is a class name, attribute fragment (`class="..."`, `className="..."`,
|
|
* `id="..."`), or a raw text snippet. Because a query can appear on a
|
|
* continuation line of a multi-line tag (e.g. the `className="..."` row of a
|
|
* `<section\n className="..."\n>` JSX tag), we walk backward from the match
|
|
* line to find the actual tag opener. When `tag` is provided, opener candidates
|
|
* must match that tag name.
|
|
*/
|
|
/**
|
|
* Return the smallest leading-whitespace count across a set of lines,
|
|
* ignoring blank lines (whose indent isn't load-bearing). Used to compute
|
|
* the common base indent of a multi-line picked element so reindenting
|
|
* under the wrapper preserves the relative depth between lines.
|
|
*/
|
|
function minLeadingSpaces(lines) {
|
|
let min = Infinity;
|
|
for (const l of lines) {
|
|
if (l.trim() === '') continue;
|
|
const m = l.match(/^(\s*)/);
|
|
if (m && m[1].length < min) min = m[1].length;
|
|
}
|
|
return min === Infinity ? 0 : min;
|
|
}
|
|
|
|
function findElement(lines, query, tag = null) {
|
|
// Iterate all matches — the first substring hit isn't always the right one.
|
|
for (let i = 0; i < lines.length; i++) {
|
|
if (!lines[i].includes(query)) continue;
|
|
|
|
const stripped = lines[i].trim();
|
|
if (stripped.startsWith('<!--') || stripped.startsWith('{/*') || stripped.startsWith('//')) continue;
|
|
// Skip lines already inside a variant wrapper
|
|
if (lines[i].includes('data-impeccable-variant')) continue;
|
|
|
|
const openerLine = findOpenerLine(lines, i, tag);
|
|
if (openerLine === -1) continue;
|
|
|
|
const endLine = findClosingLine(lines, openerLine);
|
|
return { startLine: openerLine, endLine };
|
|
}
|
|
|
|
return null;
|
|
}
|
|
|
|
/**
|
|
* Like findElement, but returns every match. Used for ambiguity detection
|
|
* when the agent passes --text: when the same className appears on multiple
|
|
* sibling elements (a list of cards, repeated section variants, etc.),
|
|
* first-match silently lands on the wrong branch. Returning all matches lets
|
|
* the caller narrow by textContent or fail with a structured ambiguity error.
|
|
*/
|
|
function findAllElements(lines, query, tag = null) {
|
|
const out = [];
|
|
const seen = new Set();
|
|
for (let i = 0; i < lines.length; i++) {
|
|
if (!lines[i].includes(query)) continue;
|
|
const stripped = lines[i].trim();
|
|
if (stripped.startsWith('<!--') || stripped.startsWith('{/*') || stripped.startsWith('//')) continue;
|
|
if (lines[i].includes('data-impeccable-variant')) continue;
|
|
const openerLine = findOpenerLine(lines, i, tag);
|
|
if (openerLine === -1) continue;
|
|
if (seen.has(openerLine)) continue; // multiple matches inside the same element
|
|
seen.add(openerLine);
|
|
const endLine = findClosingLine(lines, openerLine);
|
|
out.push({ startLine: openerLine, endLine });
|
|
}
|
|
return out;
|
|
}
|
|
|
|
/**
|
|
* Narrow a candidate set to those whose source body matches a meaningful
|
|
* prefix of the picked element's textContent. The compare strips tags and
|
|
* JSX expressions, then checks two whitespace normalizations side-by-side:
|
|
*
|
|
* - single-space ("hero two second card body")
|
|
* - no-whitespace ("herotwosecondcardbody")
|
|
*
|
|
* Both are needed because `el.textContent` concatenates sibling text without
|
|
* inserting whitespace (e.g. `<h1>Hero Two</h1><p>Second…</p>` reads as
|
|
* `"Hero TwoSecond…"`), while the source has whitespace between tags. If
|
|
* EITHER normalization matches, the candidate keeps. A snippet shorter than
|
|
* 8 chars after stripping is too weak to disambiguate — the caller falls
|
|
* back to first-match.
|
|
*/
|
|
function filterByText(candidates, lines, text) {
|
|
const trimmed = text.replace(/\s+/g, ' ').trim().toLowerCase().slice(0, 80);
|
|
// Too short to disambiguate. Return [] so the caller's `filtered.length
|
|
// === 0` branch fires (fall back to first-match) — the previous
|
|
// `candidates.slice()` return forced `filtered.length > 1` and surfaced
|
|
// a spurious `element_ambiguous` error on every short-text picker event
|
|
// with multiple candidates.
|
|
if (trimmed.length < 8) return [];
|
|
const targetSpaced = trimmed;
|
|
const targetCompact = trimmed.replace(/\s+/g, '');
|
|
|
|
return candidates.filter((c) => {
|
|
const body = lines.slice(c.startLine, c.endLine + 1).join(' ');
|
|
const inner = body
|
|
.replace(/<[^>]*>/g, ' ') // strip HTML/JSX tags
|
|
.replace(/\{[^}]*\}/g, ' ') // strip JSX expressions
|
|
.toLowerCase();
|
|
const sourceSpaced = inner.replace(/\s+/g, ' ').trim();
|
|
const sourceCompact = inner.replace(/\s+/g, '');
|
|
return sourceSpaced.includes(targetSpaced) || sourceCompact.includes(targetCompact);
|
|
});
|
|
}
|
|
|
|
/**
|
|
* Resolve a match line to the real tag opener. If the match line itself opens
|
|
* a tag, return it. Otherwise walk up to 10 lines backward looking for the
|
|
* first tag opener. If `tag` is specified, the opener must match that tag
|
|
* name; an opener with a different tag name aborts the backward walk for this
|
|
* match (we don't jump across element boundaries).
|
|
*
|
|
* Returns the line index of the opener, or -1 if none can be resolved.
|
|
*/
|
|
function findOpenerLine(lines, matchLine, tag) {
|
|
const self = lines[matchLine].match(OPENER_RE);
|
|
if (self) {
|
|
if (!tag || self[1] === tag) return matchLine;
|
|
return -1;
|
|
}
|
|
const MAX_BACKWALK = 10;
|
|
for (let i = matchLine - 1; i >= Math.max(0, matchLine - MAX_BACKWALK); i--) {
|
|
const opener = lines[i].match(OPENER_RE);
|
|
if (!opener) continue;
|
|
if (!tag || opener[1] === tag) return i;
|
|
// Different tag name than requested — abort; we're inside a non-target opener.
|
|
return -1;
|
|
}
|
|
return -1;
|
|
}
|
|
|
|
/**
|
|
* Starting from a line with an opening tag, find the line with the matching
|
|
* closing tag by counting tag nesting depth.
|
|
*/
|
|
function findClosingLine(lines, start) {
|
|
const openMatch = lines[start].match(OPENER_RE);
|
|
if (!openMatch) return start; // caller passed a non-opener; nothing to span
|
|
|
|
const tagName = openMatch[1];
|
|
let depth = 0;
|
|
const openRe = new RegExp('<' + tagName + '(?=[\\s/>]|$)', 'g');
|
|
const selfCloseRe = new RegExp('<' + tagName + '[^>]*/>', 'g');
|
|
const closeRe = new RegExp('</' + tagName + '\\s*>', 'g');
|
|
|
|
for (let i = start; i < lines.length; i++) {
|
|
const line = lines[i];
|
|
const opens = (line.match(openRe) || []).length;
|
|
const selfCloses = (line.match(selfCloseRe) || []).length;
|
|
const closes = (line.match(closeRe) || []).length;
|
|
|
|
depth += opens - selfCloses - closes;
|
|
|
|
if (depth <= 0) return i;
|
|
}
|
|
|
|
// If we can't find the close, return a reasonable guess
|
|
return Math.min(start + 50, lines.length - 1);
|
|
}
|
|
|
|
// Auto-execute when run directly (node live-wrap.mjs ...)
|
|
const _running = process.argv[1];
|
|
if (_running?.endsWith('live-wrap.mjs') || _running?.endsWith('live-wrap.mjs/')) {
|
|
wrapCli();
|
|
}
|
|
|
|
// Test exports (used by tests/live-wrap.test.mjs)
|
|
export {
|
|
buildSearchQueries,
|
|
findElement,
|
|
findClosingLine,
|
|
detectCommentSyntax,
|
|
findAllElements,
|
|
filterByText,
|
|
findFileWithQuery,
|
|
detectStyleMode,
|
|
buildCssAuthoring,
|
|
buildCssSelectorPrefixExamples,
|
|
};
|