claudecode-omc 5.9.1 → 5.11.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.local/settings/settings.json +8 -0
- package/.omc-curation/governance.json +3 -0
- package/.omc-curation/sources.lock.json +5 -0
- package/README.md +10 -1
- package/bundled/manifest.json +2 -1
- package/bundled/upstream/impeccable/.omc-source/bundle.json +20 -0
- package/bundled/upstream/impeccable/.omc-source/provenance.json +105 -0
- package/bundled/upstream/impeccable/agents/impeccable-manual-edit-applier.md +97 -0
- package/bundled/upstream/impeccable/skills/impeccable/SKILL.md +168 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/adapt.md +311 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/animate.md +201 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/audit.md +133 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/bolder.md +113 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/brand.md +108 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/clarify.md +288 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/codex.md +105 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/colorize.md +257 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/craft.md +123 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/critique.md +767 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/delight.md +302 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/distill.md +111 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/document.md +429 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/extract.md +69 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/harden.md +347 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/hooks.md +88 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/init.md +172 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/interaction-design.md +189 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/layout.md +161 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/live.md +718 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/onboard.md +234 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/optimize.md +258 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/overdrive.md +130 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/polish.md +241 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/product.md +60 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/quieter.md +99 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/shape.md +165 -0
- package/bundled/upstream/impeccable/skills/impeccable/reference/typeset.md +279 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/command-metadata.json +94 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/context-signals.mjs +225 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/context.mjs +280 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/critique-storage.mjs +242 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detect-csp.mjs +198 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detect.mjs +21 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/browser/injected/index.mjs +1735 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/cli/main.mjs +244 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/detect-antipatterns-browser.js +4907 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/detect-antipatterns.mjs +43 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/engines/browser/detect-url.mjs +252 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/engines/regex/detect-text.mjs +552 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/engines/static-html/css-cascade.mjs +1013 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/engines/static-html/detect-html.mjs +208 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/engines/visual/screenshot-contrast.mjs +189 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/findings.mjs +12 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/node/file-system.mjs +198 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/profile/profiler.mjs +166 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/registry/antipatterns.mjs +419 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/rules/checks.mjs +2671 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/shared/color.mjs +124 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/shared/constants.mjs +101 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/detector/shared/page.mjs +7 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/hook-admin.mjs +574 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/hook-before-edit.mjs +473 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/hook-lib.mjs +1286 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/hook.mjs +61 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/lib/design-parser.mjs +835 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/lib/impeccable-paths.mjs +126 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/lib/is-generated.mjs +69 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/browser-script-parts.mjs +49 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/completion.mjs +19 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/event-validation.mjs +137 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/insert-ui.mjs +458 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/manual-apply.mjs +939 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/manual-edit-routes.mjs +357 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/manual-edits-buffer.mjs +152 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/session-store.mjs +289 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/svelte-component.mjs +826 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/sveltekit-adapter.mjs +274 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/ui-core.mjs +180 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live/vocabulary.mjs +36 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-accept.mjs +812 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-browser-dom.js +146 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-browser-session.js +123 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-browser.js +11086 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-commit-manual-edits.mjs +1241 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-complete.mjs +75 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-copy-edit-agent.mjs +683 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-discard-manual-edits.mjs +51 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-inject.mjs +583 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-insert.mjs +272 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-manual-edit-evidence.mjs +363 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-poll.mjs +379 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-resume.mjs +94 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-server.mjs +1134 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-status.mjs +61 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live-wrap.mjs +894 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/live.mjs +246 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/modern-screenshot.umd.js +14 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/palette.mjs +633 -0
- package/bundled/upstream/impeccable/skills/impeccable/scripts/pin.mjs +214 -0
- package/package.json +1 -1
- package/src/cli/source.js +6 -0
- package/src/config/sources.js +15 -0
- package/src/merge/content-patch.js +4 -0
|
@@ -0,0 +1,126 @@
|
|
|
1
|
+
import fs from 'node:fs';
|
|
2
|
+
import path from 'node:path';
|
|
3
|
+
|
|
4
|
+
export const IMPECCABLE_DIR = '.impeccable';
|
|
5
|
+
export const LIVE_DIR = 'live';
|
|
6
|
+
export const CRITIQUE_DIR = 'critique';
|
|
7
|
+
|
|
8
|
+
export function getImpeccableDir(cwd = process.cwd()) {
|
|
9
|
+
return path.join(cwd, IMPECCABLE_DIR);
|
|
10
|
+
}
|
|
11
|
+
|
|
12
|
+
export function getDesignSidecarPath(cwd = process.cwd()) {
|
|
13
|
+
return path.join(getImpeccableDir(cwd), 'design.json');
|
|
14
|
+
}
|
|
15
|
+
|
|
16
|
+
export function getDesignSidecarCandidates(cwd = process.cwd(), contextDir = cwd) {
|
|
17
|
+
const candidates = [
|
|
18
|
+
getDesignSidecarPath(cwd),
|
|
19
|
+
path.join(cwd, 'DESIGN.json'),
|
|
20
|
+
];
|
|
21
|
+
const contextLegacy = path.join(contextDir, 'DESIGN.json');
|
|
22
|
+
if (!candidates.includes(contextLegacy)) candidates.push(contextLegacy);
|
|
23
|
+
return candidates;
|
|
24
|
+
}
|
|
25
|
+
|
|
26
|
+
export function resolveDesignSidecarPath(cwd = process.cwd(), contextDir = cwd) {
|
|
27
|
+
return firstExisting(getDesignSidecarCandidates(cwd, contextDir));
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
export function getLiveDir(cwd = process.cwd()) {
|
|
31
|
+
return path.join(getImpeccableDir(cwd), LIVE_DIR);
|
|
32
|
+
}
|
|
33
|
+
|
|
34
|
+
export function getLiveConfigPath(cwd = process.cwd()) {
|
|
35
|
+
return path.join(getLiveDir(cwd), 'config.json');
|
|
36
|
+
}
|
|
37
|
+
|
|
38
|
+
export function getLegacyLiveConfigPath(scriptsDir) {
|
|
39
|
+
return path.join(scriptsDir, 'config.json');
|
|
40
|
+
}
|
|
41
|
+
|
|
42
|
+
export function resolveLiveConfigPath({ cwd = process.cwd(), scriptsDir, env = process.env } = {}) {
|
|
43
|
+
if (env.IMPECCABLE_LIVE_CONFIG && env.IMPECCABLE_LIVE_CONFIG.trim()) {
|
|
44
|
+
const configured = env.IMPECCABLE_LIVE_CONFIG.trim();
|
|
45
|
+
return path.isAbsolute(configured) ? configured : path.resolve(cwd, configured);
|
|
46
|
+
}
|
|
47
|
+
const primary = getLiveConfigPath(cwd);
|
|
48
|
+
if (fs.existsSync(primary)) return primary;
|
|
49
|
+
if (scriptsDir) {
|
|
50
|
+
const legacy = getLegacyLiveConfigPath(scriptsDir);
|
|
51
|
+
if (fs.existsSync(legacy)) return legacy;
|
|
52
|
+
}
|
|
53
|
+
return primary;
|
|
54
|
+
}
|
|
55
|
+
|
|
56
|
+
export function getLiveServerPath(cwd = process.cwd()) {
|
|
57
|
+
return path.join(getLiveDir(cwd), 'server.json');
|
|
58
|
+
}
|
|
59
|
+
|
|
60
|
+
export function getLegacyLiveServerPath(cwd = process.cwd()) {
|
|
61
|
+
return path.join(cwd, '.impeccable-live.json');
|
|
62
|
+
}
|
|
63
|
+
|
|
64
|
+
export function readLiveServerInfo(cwd = process.cwd()) {
|
|
65
|
+
for (const filePath of [getLiveServerPath(cwd), getLegacyLiveServerPath(cwd)]) {
|
|
66
|
+
try {
|
|
67
|
+
const info = JSON.parse(fs.readFileSync(filePath, 'utf-8'));
|
|
68
|
+
if (info && typeof info.pid === 'number' && !isLiveServerPidReachable(info.pid)) {
|
|
69
|
+
try { fs.unlinkSync(filePath); } catch {}
|
|
70
|
+
continue;
|
|
71
|
+
}
|
|
72
|
+
return { info, path: filePath };
|
|
73
|
+
} catch {
|
|
74
|
+
/* try next */
|
|
75
|
+
}
|
|
76
|
+
}
|
|
77
|
+
return null;
|
|
78
|
+
}
|
|
79
|
+
|
|
80
|
+
export function isLiveServerPidReachable(pid) {
|
|
81
|
+
try {
|
|
82
|
+
process.kill(pid, 0);
|
|
83
|
+
return true;
|
|
84
|
+
} catch (err) {
|
|
85
|
+
// ESRCH means "no such process". EPERM means the process exists but this
|
|
86
|
+
// user cannot signal it, so the live server info is still valid.
|
|
87
|
+
return err?.code !== 'ESRCH';
|
|
88
|
+
}
|
|
89
|
+
}
|
|
90
|
+
|
|
91
|
+
export function writeLiveServerInfo(cwd = process.cwd(), info) {
|
|
92
|
+
const filePath = getLiveServerPath(cwd);
|
|
93
|
+
fs.mkdirSync(path.dirname(filePath), { recursive: true });
|
|
94
|
+
fs.writeFileSync(filePath, JSON.stringify(info));
|
|
95
|
+
return filePath;
|
|
96
|
+
}
|
|
97
|
+
|
|
98
|
+
export function removeLiveServerInfo(cwd = process.cwd()) {
|
|
99
|
+
for (const filePath of [getLiveServerPath(cwd), getLegacyLiveServerPath(cwd)]) {
|
|
100
|
+
try { fs.unlinkSync(filePath); } catch {}
|
|
101
|
+
}
|
|
102
|
+
}
|
|
103
|
+
|
|
104
|
+
export function getLiveSessionsDir(cwd = process.cwd()) {
|
|
105
|
+
return path.join(getLiveDir(cwd), 'sessions');
|
|
106
|
+
}
|
|
107
|
+
|
|
108
|
+
export function getLegacyLiveSessionsDir(cwd = process.cwd()) {
|
|
109
|
+
return path.join(cwd, '.impeccable-live', 'sessions');
|
|
110
|
+
}
|
|
111
|
+
|
|
112
|
+
export function getLiveAnnotationsDir(cwd = process.cwd()) {
|
|
113
|
+
return path.join(getLiveDir(cwd), 'annotations');
|
|
114
|
+
}
|
|
115
|
+
|
|
116
|
+
export function getCritiqueDir(cwd = process.cwd()) {
|
|
117
|
+
return path.join(getImpeccableDir(cwd), CRITIQUE_DIR);
|
|
118
|
+
}
|
|
119
|
+
|
|
120
|
+
export function getLegacyLiveAnnotationsDir(cwd = process.cwd()) {
|
|
121
|
+
return path.join(cwd, '.impeccable-live', 'annotations');
|
|
122
|
+
}
|
|
123
|
+
|
|
124
|
+
function firstExisting(paths) {
|
|
125
|
+
return paths.find((filePath) => fs.existsSync(filePath)) || null;
|
|
126
|
+
}
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Decide whether a given file is "generated" (regenerated by a build step,
|
|
3
|
+
* unsafe to write variants into) or "source" (safe to edit, changes persist).
|
|
4
|
+
*
|
|
5
|
+
* Why this matters: when the user picks an element on a page whose underlying
|
|
6
|
+
* file is regenerated by a build step (e.g. `scripts/build-sub-pages.js`
|
|
7
|
+
* rewriting `public/docs/*.html`), writing variants or accepted changes into
|
|
8
|
+
* that file is silent data loss — the next build wipes them.
|
|
9
|
+
*
|
|
10
|
+
* Signals, in order of reliability:
|
|
11
|
+
* 1. Git check-ignore: gitignored files are assumed generated.
|
|
12
|
+
* 2. File-header markers ("GENERATED", "DO NOT EDIT", "AUTO-GENERATED")
|
|
13
|
+
* within the first ~300 characters — catches non-git projects.
|
|
14
|
+
*/
|
|
15
|
+
|
|
16
|
+
import { execSync } from 'node:child_process';
|
|
17
|
+
import fs from 'node:fs';
|
|
18
|
+
import path from 'node:path';
|
|
19
|
+
|
|
20
|
+
const HEADER_SCAN_BYTES = 300;
|
|
21
|
+
const HEADER_MARKERS = [
|
|
22
|
+
/@generated\b/i,
|
|
23
|
+
/\bGENERATED\s+FILE\b/,
|
|
24
|
+
/\bAUTO-?GENERATED\b/i,
|
|
25
|
+
/\bDO\s+NOT\s+EDIT\b/i,
|
|
26
|
+
];
|
|
27
|
+
|
|
28
|
+
/**
|
|
29
|
+
* @param {string} filePath - absolute or cwd-relative path
|
|
30
|
+
* @param {object} [options]
|
|
31
|
+
* @param {string} [options.cwd] - project root (defaults to process.cwd())
|
|
32
|
+
*/
|
|
33
|
+
export function isGeneratedFile(filePath, options = {}) {
|
|
34
|
+
const cwd = options.cwd || process.cwd();
|
|
35
|
+
const absPath = path.isAbsolute(filePath) ? filePath : path.resolve(cwd, filePath);
|
|
36
|
+
|
|
37
|
+
if (isGitIgnored(absPath, cwd)) return true;
|
|
38
|
+
if (hasGeneratedHeader(absPath)) return true;
|
|
39
|
+
return false;
|
|
40
|
+
}
|
|
41
|
+
|
|
42
|
+
function isGitIgnored(absPath, cwd) {
|
|
43
|
+
try {
|
|
44
|
+
execSync(`git check-ignore --quiet ${JSON.stringify(absPath)}`, {
|
|
45
|
+
cwd,
|
|
46
|
+
stdio: 'ignore',
|
|
47
|
+
});
|
|
48
|
+
return true; // exit 0 = ignored
|
|
49
|
+
} catch (err) {
|
|
50
|
+
// Exit code 1 = not ignored. Exit code 128 = not a git repo or other error.
|
|
51
|
+
// In both cases, treat as "not known to be ignored."
|
|
52
|
+
return false;
|
|
53
|
+
}
|
|
54
|
+
}
|
|
55
|
+
|
|
56
|
+
function hasGeneratedHeader(absPath) {
|
|
57
|
+
let fd;
|
|
58
|
+
try {
|
|
59
|
+
fd = fs.openSync(absPath, 'r');
|
|
60
|
+
const buf = Buffer.alloc(HEADER_SCAN_BYTES);
|
|
61
|
+
const bytesRead = fs.readSync(fd, buf, 0, HEADER_SCAN_BYTES, 0);
|
|
62
|
+
const head = buf.slice(0, bytesRead).toString('utf-8');
|
|
63
|
+
return HEADER_MARKERS.some((re) => re.test(head));
|
|
64
|
+
} catch {
|
|
65
|
+
return false;
|
|
66
|
+
} finally {
|
|
67
|
+
if (fd !== undefined) { try { fs.closeSync(fd); } catch {} }
|
|
68
|
+
}
|
|
69
|
+
}
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
import fs from 'node:fs';
|
|
2
|
+
import path from 'node:path';
|
|
3
|
+
|
|
4
|
+
export const LIVE_BROWSER_SCRIPT_PARTS = Object.freeze([
|
|
5
|
+
Object.freeze({ name: 'session-state', file: 'live-browser-session.js' }),
|
|
6
|
+
Object.freeze({ name: 'dom-helpers', file: 'live-browser-dom.js' }),
|
|
7
|
+
Object.freeze({ name: 'browser-ui', file: 'live-browser.js' }),
|
|
8
|
+
]);
|
|
9
|
+
|
|
10
|
+
export function resolveLiveBrowserScriptParts(scriptsDir, parts = LIVE_BROWSER_SCRIPT_PARTS) {
|
|
11
|
+
if (!scriptsDir) throw new Error('scriptsDir is required');
|
|
12
|
+
return parts.map((part, index) => ({
|
|
13
|
+
...part,
|
|
14
|
+
index,
|
|
15
|
+
path: path.join(scriptsDir, part.file),
|
|
16
|
+
}));
|
|
17
|
+
}
|
|
18
|
+
|
|
19
|
+
export function assertLiveBrowserScriptParts(parts, exists = fs.existsSync) {
|
|
20
|
+
for (const part of parts) {
|
|
21
|
+
if (!exists(part.path)) {
|
|
22
|
+
throw new Error(`Live browser script part missing: ${part.name} (${part.path})`);
|
|
23
|
+
}
|
|
24
|
+
}
|
|
25
|
+
return parts;
|
|
26
|
+
}
|
|
27
|
+
|
|
28
|
+
export function readLiveBrowserScriptParts(parts, readFile = (filePath) => fs.readFileSync(filePath, 'utf-8')) {
|
|
29
|
+
return parts.map((part) => ({
|
|
30
|
+
...part,
|
|
31
|
+
source: readFile(part.path),
|
|
32
|
+
}));
|
|
33
|
+
}
|
|
34
|
+
|
|
35
|
+
export function assembleLiveBrowserScript({ token, port, vocabulary, parts }) {
|
|
36
|
+
const prelude =
|
|
37
|
+
`window.__IMPECCABLE_TOKEN__ = '${token}';\n` +
|
|
38
|
+
`window.__IMPECCABLE_PORT__ = ${port};\n` +
|
|
39
|
+
// Canonical command vocabulary (values + labels + icons). live-browser.js
|
|
40
|
+
// builds its action picker from this instead of an inline copy.
|
|
41
|
+
`window.__IMPECCABLE_VOCAB__ = ${JSON.stringify(vocabulary)};\n`;
|
|
42
|
+
|
|
43
|
+
const body = parts.map((part) => {
|
|
44
|
+
const file = part.file || path.basename(part.path || '');
|
|
45
|
+
return `// --- impeccable live script part: ${part.name} (${file}) ---\n${part.source}`;
|
|
46
|
+
}).join('\n');
|
|
47
|
+
|
|
48
|
+
return prelude + body;
|
|
49
|
+
}
|
|
@@ -0,0 +1,19 @@
|
|
|
1
|
+
export function completionTypeForAcceptResult(eventType, acceptResult) {
|
|
2
|
+
if (eventType === 'discard') return acceptResult?.handled === true ? 'discarded' : 'error';
|
|
3
|
+
if (acceptResult?.handled === true && acceptResult?.carbonize === true) return 'agent_done';
|
|
4
|
+
if (acceptResult?.handled === true) return 'complete';
|
|
5
|
+
if (acceptResult?.mode === 'error') return 'error';
|
|
6
|
+
if (eventType === 'accept' && acceptResult?.previewMode === 'svelte-component') return 'error';
|
|
7
|
+
return 'agent_done';
|
|
8
|
+
}
|
|
9
|
+
|
|
10
|
+
export function completionAckForAcceptResult(eventId, completionType, acceptResult) {
|
|
11
|
+
const ack = { ok: true, type: completionType };
|
|
12
|
+
if (acceptResult?.handled === true && acceptResult?.carbonize === true) {
|
|
13
|
+
ack.final = false;
|
|
14
|
+
ack.requiresComplete = true;
|
|
15
|
+
ack.nextCommand = `live-complete.mjs --id ${eventId}`;
|
|
16
|
+
ack.message = 'Carbonize cleanup must be verified, then the session must be completed explicitly before polling again.';
|
|
17
|
+
}
|
|
18
|
+
return ack;
|
|
19
|
+
}
|
|
@@ -0,0 +1,137 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Shared event validation for the live helper server.
|
|
3
|
+
* Extracted for unit testing (insert mode rules).
|
|
4
|
+
*/
|
|
5
|
+
|
|
6
|
+
import { canCreateInsert } from './insert-ui.mjs';
|
|
7
|
+
|
|
8
|
+
// The accepted visual action values come from the canonical vocabulary so the
|
|
9
|
+
// validator, the picker UI, and the marketing demo never drift. Imported (not
|
|
10
|
+
// just re-exported) so it is also in scope for the validators below.
|
|
11
|
+
import { VISUAL_ACTIONS } from './vocabulary.mjs';
|
|
12
|
+
export { VISUAL_ACTIONS };
|
|
13
|
+
|
|
14
|
+
const ID_PATTERN = /^[0-9a-f]{8}$/;
|
|
15
|
+
const VARIANT_ID_PATTERN = /^[0-9]{1,3}$/;
|
|
16
|
+
const INSERT_POSITIONS = new Set(['before', 'after']);
|
|
17
|
+
const FORBIDDEN_MANUAL_EDIT_TEXT_CHARS = ['<', '{', '}', '`'];
|
|
18
|
+
|
|
19
|
+
function isValidId(v) { return typeof v === 'string' && ID_PATTERN.test(v); }
|
|
20
|
+
function isValidVariantId(v) { return typeof v === 'string' && VARIANT_ID_PATTERN.test(v); }
|
|
21
|
+
|
|
22
|
+
function validateManualEditText(newText) {
|
|
23
|
+
if (typeof newText !== 'string') return null;
|
|
24
|
+
const hits = FORBIDDEN_MANUAL_EDIT_TEXT_CHARS.filter((char) => newText.includes(char));
|
|
25
|
+
return hits.length > 0 ? hits : null;
|
|
26
|
+
}
|
|
27
|
+
|
|
28
|
+
function validateAnnotationFields(msg) {
|
|
29
|
+
if (msg.screenshotPath !== undefined && typeof msg.screenshotPath !== 'string') {
|
|
30
|
+
return 'generate: screenshotPath must be string';
|
|
31
|
+
}
|
|
32
|
+
if (msg.comments !== undefined && !Array.isArray(msg.comments)) {
|
|
33
|
+
return 'generate: comments must be array';
|
|
34
|
+
}
|
|
35
|
+
if (msg.strokes !== undefined && !Array.isArray(msg.strokes)) {
|
|
36
|
+
return 'generate: strokes must be array';
|
|
37
|
+
}
|
|
38
|
+
return null;
|
|
39
|
+
}
|
|
40
|
+
|
|
41
|
+
function validateInsertGenerate(msg) {
|
|
42
|
+
if (!msg.insert || typeof msg.insert !== 'object') return 'generate: insert mode requires insert object';
|
|
43
|
+
if (!INSERT_POSITIONS.has(msg.insert.position)) return 'generate: insert.position must be before or after';
|
|
44
|
+
const anchor = msg.insert.anchor;
|
|
45
|
+
if (!anchor || typeof anchor !== 'object') return 'generate: insert.anchor required';
|
|
46
|
+
if (!anchor.tagName && !anchor.outerHTML && !(Array.isArray(anchor.classes) && anchor.classes.length)) {
|
|
47
|
+
return 'generate: insert.anchor needs tagName, classes, or outerHTML';
|
|
48
|
+
}
|
|
49
|
+
if (!msg.placeholder || typeof msg.placeholder !== 'object') return 'generate: insert mode requires placeholder dimensions';
|
|
50
|
+
if (!Number.isFinite(msg.placeholder.width) || !Number.isFinite(msg.placeholder.height)) {
|
|
51
|
+
return 'generate: placeholder width and height must be numbers';
|
|
52
|
+
}
|
|
53
|
+
if (!canCreateInsert({
|
|
54
|
+
prompt: msg.freeformPrompt,
|
|
55
|
+
comments: msg.comments,
|
|
56
|
+
strokes: msg.strokes,
|
|
57
|
+
})) {
|
|
58
|
+
return 'generate: insert requires freeformPrompt or annotations';
|
|
59
|
+
}
|
|
60
|
+
return validateAnnotationFields(msg);
|
|
61
|
+
}
|
|
62
|
+
|
|
63
|
+
function validateReplaceGenerate(msg) {
|
|
64
|
+
if (!msg.action || !VISUAL_ACTIONS.includes(msg.action)) return 'generate: invalid action';
|
|
65
|
+
if (!msg.element || !msg.element.outerHTML) return 'generate: missing element context';
|
|
66
|
+
return validateAnnotationFields(msg);
|
|
67
|
+
}
|
|
68
|
+
|
|
69
|
+
function validateManualEditEvent(msg, label) {
|
|
70
|
+
if (!isValidId(msg.id)) return label + ': missing or malformed id';
|
|
71
|
+
if (!msg.pageUrl || typeof msg.pageUrl !== 'string') return label + ': missing pageUrl';
|
|
72
|
+
if (!msg.element || typeof msg.element !== 'object') return label + ': missing element';
|
|
73
|
+
if (!Array.isArray(msg.ops) || msg.ops.length === 0) return label + ': ops must be non-empty array';
|
|
74
|
+
if (msg.ops.length > 100) return label + ': too many ops (max 100)';
|
|
75
|
+
for (const op of msg.ops) {
|
|
76
|
+
if (typeof op.ref !== 'string') return label + ': op.ref required';
|
|
77
|
+
if (typeof op.tag !== 'string') return label + ': op.tag required';
|
|
78
|
+
if (typeof op.originalText !== 'string') return label + ': op.originalText required';
|
|
79
|
+
if (op.deleted !== true && typeof op.newText !== 'string') {
|
|
80
|
+
return label + ': text op requires newText';
|
|
81
|
+
}
|
|
82
|
+
if (typeof op.newText === 'string') {
|
|
83
|
+
if (op.deleted !== true && op.newText.trim().length === 0) {
|
|
84
|
+
return label + ': newText cannot be empty';
|
|
85
|
+
}
|
|
86
|
+
const forbidden = validateManualEditText(op.newText);
|
|
87
|
+
if (forbidden) {
|
|
88
|
+
return label + ': newText cannot contain ' + forbidden.join(' ') + ' (plain text only; ask the AI to insert markup)';
|
|
89
|
+
}
|
|
90
|
+
}
|
|
91
|
+
}
|
|
92
|
+
return null;
|
|
93
|
+
}
|
|
94
|
+
|
|
95
|
+
export function validateEvent(msg) {
|
|
96
|
+
if (!msg || typeof msg !== 'object' || !msg.type) return 'Missing or invalid message';
|
|
97
|
+
switch (msg.type) {
|
|
98
|
+
case 'generate':
|
|
99
|
+
if (!isValidId(msg.id)) return 'generate: missing or malformed id';
|
|
100
|
+
if (!Number.isInteger(msg.count) || msg.count < 1 || msg.count > 8) return 'generate: count must be 1-8';
|
|
101
|
+
if (msg.mode === 'insert') return validateInsertGenerate(msg);
|
|
102
|
+
return validateReplaceGenerate(msg);
|
|
103
|
+
case 'accept':
|
|
104
|
+
if (!isValidId(msg.id)) return 'accept: missing or malformed id';
|
|
105
|
+
if (!isValidVariantId(msg.variantId)) return 'accept: missing or malformed variantId';
|
|
106
|
+
if (msg.paramValues !== undefined) {
|
|
107
|
+
if (typeof msg.paramValues !== 'object' || msg.paramValues === null || Array.isArray(msg.paramValues)) {
|
|
108
|
+
return 'accept: paramValues must be an object';
|
|
109
|
+
}
|
|
110
|
+
}
|
|
111
|
+
return null;
|
|
112
|
+
case 'discard':
|
|
113
|
+
return isValidId(msg.id) ? null : 'discard: missing or malformed id';
|
|
114
|
+
case 'checkpoint':
|
|
115
|
+
if (!isValidId(msg.id)) return 'checkpoint: missing or malformed id';
|
|
116
|
+
if (!Number.isInteger(msg.revision) || msg.revision < 0) return 'checkpoint: revision must be a non-negative integer';
|
|
117
|
+
if (msg.paramValues !== undefined && (typeof msg.paramValues !== 'object' || msg.paramValues === null || Array.isArray(msg.paramValues))) {
|
|
118
|
+
return 'checkpoint: paramValues must be an object';
|
|
119
|
+
}
|
|
120
|
+
return null;
|
|
121
|
+
case 'exit':
|
|
122
|
+
return null;
|
|
123
|
+
case 'prefetch':
|
|
124
|
+
if (!msg.pageUrl || typeof msg.pageUrl !== 'string') return 'prefetch: missing pageUrl';
|
|
125
|
+
return null;
|
|
126
|
+
case 'manual_edits':
|
|
127
|
+
return validateManualEditEvent(msg, 'manual_edits');
|
|
128
|
+
case 'steer':
|
|
129
|
+
if (!isValidId(msg.id)) return 'steer: missing or malformed id';
|
|
130
|
+
if (typeof msg.message !== 'string' || !msg.message.trim()) return 'steer: message required';
|
|
131
|
+
if (msg.message.length > 4000) return 'steer: message too long';
|
|
132
|
+
if (msg.pageUrl !== undefined && typeof msg.pageUrl !== 'string') return 'steer: pageUrl must be string';
|
|
133
|
+
return null;
|
|
134
|
+
default:
|
|
135
|
+
return 'Unknown event type: ' + msg.type;
|
|
136
|
+
}
|
|
137
|
+
}
|