Files
gstack/extension/inspector.js
Garry Tan c6e6a21d1a refactor: AI slop reduction with cross-model quality review (v0.16.3.0) (#941)
* refactor: add error-handling utility module with selective catches

safeUnlink (ignores ENOENT), safeKill (ignores ESRCH), isProcessAlive
(extracted from cli.ts with Windows support), and json() Response helper.
All catches check err.code and rethrow unexpected errors instead of
swallowing silently. Unit tests cover happy path + error code paths.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: replace defensive try/catches in server.ts with utilities

Replace ~12 try/catch sites with safeUnlink/safeKill calls in shutdown,
emergencyCleanup, killAgent, and log cleanup. Convert empty catches to
selective catches with error code checks. Remove needless welcome page
try/catches (fs.existsSync doesn't need wrapping). Reduces slop-scan
empty-catch locations from 11 to 8 and error-swallowing from 24 to 18.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: extract isProcessAlive and replace try/catches in cli.ts

Move isProcessAlive to shared error-handling module. Replace ~20
try/catch sites with safeUnlink/safeKill in killServer, connect,
disconnect, and cleanup flows. Convert empty catches to selective
catches. Reduces slop-scan empty-catch from 22 to 2 locations.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: remove unnecessary return await in content-security and read-commands

Remove 6 redundant return-await patterns where there's no enclosing
try block. Eliminates all defensive.async-noise findings from these files.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: add slop-scan config to exclude vendor files

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: replace empty catches with selective error handling in sidebar-agent

Convert 8 empty catch blocks to selective catches that check err.code
(ESRCH for process kills, ENOENT for file ops). Import safeUnlink for
cancel file cleanup. Unexpected errors now propagate instead of being
silently swallowed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: replace empty catches and mark pass-through wrappers in browser-manager

Convert 12 empty catch blocks to selective catches: filesystem ops check
ENOENT/EACCES, browser ops check for closed/Target messages, URL parsing
checks TypeError. Add 'alias for active session' comments above 6
pass-through wrapper methods to document their purpose (and exempt from
slop-scan pass-through-wrappers rule).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: selective catches in gstack-global-discover

Convert 8 defensive catch blocks to selective error handling. Filesystem
ops check ENOENT/EACCES, process ops check exit status. Unexpected errors
now propagate instead of returning silent defaults.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: selective catches in write-commands, cdp-inspector, meta-commands, snapshot

Convert ~27 empty/obscuring catches to selective error handling across 4
browse source files. CDP ops check for closed/Target/detached messages,
DOM ops check TypeError/DOMException, filesystem ops check ENOENT/EACCES,
JSON parsing checks SyntaxError. Remove dead code in cdp-inspector where
try/catch wrapped synchronous no-ops.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* refactor: selective catches in Chrome extension files

Convert empty catches and error-swallowing patterns across inspector.js,
content.js, background.js, and sidepanel.js. DOM catches filter
TypeError/DOMException, chrome API catches filter Extension context
invalidated, network catches filter Failed to fetch. Unexpected errors
now propagate.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: restore isProcessAlive boolean semantics, add safeUnlinkQuiet, remove unused json()

isProcessAlive now catches ALL errors and returns false (pure boolean
probe). Callers use it in if/while conditions without try/catch, so
throwing on EPERM was a behavior change that could crash the CLI.
Windows path gets its safety catch restored.

safeUnlinkQuiet added for best-effort cleanup paths where throwing on
non-ENOENT errors (like EPERM during shutdown) would abort cleanup.

json() removed — dead code, never imported anywhere.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: use safeUnlinkQuiet in shutdown and cleanup paths

Shutdown, emergency cleanup, and disconnect paths should never throw
on file deletion failures. Switched from safeUnlink (throws on EPERM)
to safeUnlinkQuiet (swallows all errors) in these best-effort paths.
Normal operation paths (startup, lock release) keep safeUnlink.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* revert: remove brittle string-matching catches and alias comments in browser-manager

Revert 6 catches that matched error messages via includes('closed'),
includes('Target'), etc. back to empty catches. These fire-and-forget
operations (page.close, bringToFront, dialog dismiss) genuinely don't
care about any error type. String matching on error messages is brittle
and will break on Playwright version bumps.

Remove 6 'alias for active session' comments that existed solely to
game slop-scan's pass-through-wrapper exemption rule.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* revert: remove brittle string-matching catches in extension files

Revert error-swallowing fixes in background.js and sidepanel.js that
matched error messages via includes('Failed to fetch'), includes(
'Extension context invalidated'), etc. In Chrome extensions, uncaught
errors crash the entire extension. The original catch-and-log pattern
is the correct choice for extension code where any error is non-fatal.

content.js and inspector.js changes kept — their TypeError/DOMException
catches are typed, not string-based.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: add slop-scan usage guidelines to CLAUDE.md

Instructions for using slop-scan to improve genuine code quality, not
to game metrics or hide that we're AI-coded. Documents what to fix
(empty catches on file/process ops, typed exception narrows, return
await) and what NOT to fix (string-matching on error messages, linter
gaming comments, tightening extension/cleanup catches). Includes
utility function reference and baseline score tracking.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: add slop-scan as diagnostic in test suite

Runs slop-scan after bun test as a non-blocking diagnostic. Prints
the summary (top files, hotspots) so you see the number without it
gating anything. Available standalone via bun run slop.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: slop-diff shows only NEW findings introduced on this branch

Runs slop-scan on HEAD and the merge-base, diffs results with
line-number-insensitive fingerprinting so shifted code doesn't create
false positives. Uses git worktree for clean base comparison. Shows
net new vs removed findings. Runs automatically after bun test.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: design doc for slop-scan integration in /review and /ship

Deferred plan for surfacing slop-diff findings automatically during
code review and shipping. Documents integration points, auto-fix vs
skip heuristics, and implementation notes.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: bump version and changelog (v0.16.3.0)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-10 17:13:15 -10:00

475 lines
16 KiB
JavaScript

/**
* gstack browse — CSS Inspector content script
*
* Dynamically injected via chrome.scripting.executeScript.
* Provides element picker, selector generation, basic computed style capture,
* and page alteration handlers for agent-pushed CSS changes.
*/
(() => {
// Guard against double-injection
if (window.__gstackInspectorActive) return;
window.__gstackInspectorActive = true;
// ─── State ──────────────────────────────────────────────────────
let pickerActive = false;
let highlightEl = null;
let tooltipEl = null;
let lastPickTime = 0;
const PICK_DEBOUNCE_MS = 200;
// Track original inline styles for resetAll
const originalStyles = new Map(); // element -> Map<property, value>
const injectedStyleIds = new Set();
// ─── Highlight Overlay ──────────────────────────────────────────
function createHighlight() {
if (highlightEl) return;
highlightEl = document.createElement('div');
highlightEl.id = 'gstack-inspector-highlight';
highlightEl.style.cssText = `
position: fixed;
pointer-events: none;
z-index: 2147483647;
background: rgba(59, 130, 246, 0.15);
border: 2px solid rgba(59, 130, 246, 0.6);
border-radius: 2px;
transition: top 50ms, left 50ms, width 50ms, height 50ms;
`;
document.documentElement.appendChild(highlightEl);
tooltipEl = document.createElement('div');
tooltipEl.id = 'gstack-inspector-tooltip';
tooltipEl.style.cssText = `
position: fixed;
pointer-events: none;
z-index: 2147483647;
background: #27272A;
color: #e0e0e0;
font-family: 'JetBrains Mono', 'SF Mono', 'Fira Code', monospace;
font-size: 11px;
padding: 3px 8px;
border-radius: 4px;
white-space: nowrap;
box-shadow: 0 2px 8px rgba(0,0,0,0.4);
display: none;
`;
document.documentElement.appendChild(tooltipEl);
}
function removeHighlight() {
if (highlightEl) { highlightEl.remove(); highlightEl = null; }
if (tooltipEl) { tooltipEl.remove(); tooltipEl = null; }
}
function updateHighlight(el) {
if (!highlightEl || !tooltipEl) return;
const rect = el.getBoundingClientRect();
highlightEl.style.top = rect.top + 'px';
highlightEl.style.left = rect.left + 'px';
highlightEl.style.width = rect.width + 'px';
highlightEl.style.height = rect.height + 'px';
highlightEl.style.display = 'block';
// Build tooltip text: <tag> .classes WxH
const tag = el.tagName.toLowerCase();
const classes = el.className && typeof el.className === 'string'
? '.' + el.className.trim().split(/\s+/).join('.')
: '';
const dims = `${Math.round(rect.width)}x${Math.round(rect.height)}`;
tooltipEl.textContent = `<${tag}> ${classes} ${dims}`.trim();
// Position tooltip above element, or below if no room
const tooltipHeight = 24;
const gap = 6;
let tooltipTop = rect.top - tooltipHeight - gap;
if (tooltipTop < 4) tooltipTop = rect.bottom + gap;
let tooltipLeft = rect.left;
if (tooltipLeft < 4) tooltipLeft = 4;
tooltipEl.style.top = tooltipTop + 'px';
tooltipEl.style.left = tooltipLeft + 'px';
tooltipEl.style.display = 'block';
}
// ─── Selector Generation ────────────────────────────────────────
function buildSelector(el) {
// If element has an id, use it directly
if (el.id) {
const sel = '#' + CSS.escape(el.id);
if (isUnique(sel)) return sel;
}
// Build path from element up to nearest ancestor with id or body
const parts = [];
let current = el;
while (current && current !== document.body && current !== document.documentElement) {
let part = current.tagName.toLowerCase();
// If current has an id, use it and stop
if (current.id) {
part = '#' + CSS.escape(current.id);
parts.unshift(part);
break;
}
// Add classes
if (current.className && typeof current.className === 'string') {
const classes = current.className.trim().split(/\s+/).filter(c => c.length > 0);
if (classes.length > 0) {
part += '.' + classes.map(c => CSS.escape(c)).join('.');
}
}
// Add nth-child if needed to disambiguate
const parent = current.parentElement;
if (parent) {
const siblings = Array.from(parent.children).filter(
s => s.tagName === current.tagName
);
if (siblings.length > 1) {
const idx = siblings.indexOf(current) + 1;
part += `:nth-child(${Array.from(parent.children).indexOf(current) + 1})`;
}
}
parts.unshift(part);
current = current.parentElement;
}
// If we didn't reach an id, prepend body
if (parts.length > 0 && !parts[0].startsWith('#')) {
// Don't prepend body, just use the path as-is
}
const selector = parts.join(' > ');
// Verify uniqueness
if (isUnique(selector)) return selector;
// Fallback: add nth-child at each level until unique
return selector;
}
function isUnique(selector) {
try {
return document.querySelectorAll(selector).length === 1;
} catch (e) {
if (!(e instanceof TypeError) && !(e instanceof DOMException)) throw e;
return false;
}
}
// ─── Basic Mode Data Capture ────────────────────────────────────
const KEY_PROPERTIES = [
'display', 'position', 'top', 'right', 'bottom', 'left',
'width', 'height', 'min-width', 'max-width', 'min-height', 'max-height',
'margin-top', 'margin-right', 'margin-bottom', 'margin-left',
'padding-top', 'padding-right', 'padding-bottom', 'padding-left',
'border-top-width', 'border-right-width', 'border-bottom-width', 'border-left-width',
'border-top-style', 'border-right-style', 'border-bottom-style', 'border-left-style',
'border-top-color', 'border-right-color', 'border-bottom-color', 'border-left-color',
'color', 'background-color', 'background-image',
'font-family', 'font-size', 'font-weight', 'line-height', 'letter-spacing',
'text-align', 'text-decoration', 'text-transform',
'overflow', 'overflow-x', 'overflow-y',
'opacity', 'z-index',
'flex-direction', 'justify-content', 'align-items', 'flex-wrap', 'gap',
'grid-template-columns', 'grid-template-rows',
'box-shadow', 'border-radius',
'transition', 'transform',
];
function captureBasicData(el) {
const computed = getComputedStyle(el);
const rect = el.getBoundingClientRect();
// Capture key computed properties
const computedStyles = {};
for (const prop of KEY_PROPERTIES) {
computedStyles[prop] = computed.getPropertyValue(prop);
}
// Box model from computed
const boxModel = {
content: { width: rect.width, height: rect.height },
padding: {
top: parseFloat(computed.paddingTop) || 0,
right: parseFloat(computed.paddingRight) || 0,
bottom: parseFloat(computed.paddingBottom) || 0,
left: parseFloat(computed.paddingLeft) || 0,
},
border: {
top: parseFloat(computed.borderTopWidth) || 0,
right: parseFloat(computed.borderRightWidth) || 0,
bottom: parseFloat(computed.borderBottomWidth) || 0,
left: parseFloat(computed.borderLeftWidth) || 0,
},
margin: {
top: parseFloat(computed.marginTop) || 0,
right: parseFloat(computed.marginRight) || 0,
bottom: parseFloat(computed.marginBottom) || 0,
left: parseFloat(computed.marginLeft) || 0,
},
};
// Matched CSS rules via CSSOM (same-origin only)
const matchedRules = [];
try {
for (const sheet of document.styleSheets) {
try {
const rules = sheet.cssRules || sheet.rules;
if (!rules) continue;
for (const rule of rules) {
if (rule.type !== CSSRule.STYLE_RULE) continue;
try {
if (el.matches(rule.selectorText)) {
const properties = [];
for (let i = 0; i < rule.style.length; i++) {
const prop = rule.style[i];
properties.push({
name: prop,
value: rule.style.getPropertyValue(prop),
priority: rule.style.getPropertyPriority(prop),
});
}
matchedRules.push({
selector: rule.selectorText,
properties,
source: sheet.href || 'inline',
});
}
} catch (e) { if (!(e instanceof TypeError) && !(e instanceof DOMException)) throw e; }
}
} catch (e) { if (!(e instanceof DOMException)) throw e; }
}
} catch (e) { if (!(e instanceof TypeError) && !(e instanceof DOMException)) throw e; }
return { computedStyles, boxModel, matchedRules };
}
// ─── Picker Event Handlers ──────────────────────────────────────
function onMouseMove(e) {
if (!pickerActive) return;
// Ignore our own overlay elements
const target = e.target;
if (target === highlightEl || target === tooltipEl) return;
if (target.id === 'gstack-inspector-highlight' || target.id === 'gstack-inspector-tooltip') return;
updateHighlight(target);
}
function onClick(e) {
if (!pickerActive) return;
e.preventDefault();
e.stopPropagation();
e.stopImmediatePropagation();
// Debounce
const now = Date.now();
if (now - lastPickTime < PICK_DEBOUNCE_MS) return;
lastPickTime = now;
const target = e.target;
if (target === highlightEl || target === tooltipEl) return;
if (target.id === 'gstack-inspector-highlight' || target.id === 'gstack-inspector-tooltip') return;
const selector = buildSelector(target);
const basicData = captureBasicData(target);
// Frame detection
const frameInfo = {};
if (window !== window.top) {
try {
frameInfo.frameSrc = window.location.href;
frameInfo.frameName = window.name || null;
} catch (e) { if (!(e instanceof DOMException)) throw e; }
}
chrome.runtime.sendMessage({
type: 'elementPicked',
selector,
tagName: target.tagName.toLowerCase(),
classes: target.className && typeof target.className === 'string'
? target.className.trim().split(/\s+/).filter(c => c.length > 0)
: [],
id: target.id || null,
dimensions: {
width: Math.round(target.getBoundingClientRect().width),
height: Math.round(target.getBoundingClientRect().height),
},
basicData,
...frameInfo,
});
// Keep highlight on the picked element
}
function onKeyDown(e) {
if (!pickerActive) return;
if (e.key === 'Escape') {
e.preventDefault();
e.stopPropagation();
stopPicker();
chrome.runtime.sendMessage({ type: 'pickerCancelled' });
}
}
// ─── Picker Start/Stop ──────────────────────────────────────────
function startPicker() {
if (pickerActive) return;
pickerActive = true;
createHighlight();
document.addEventListener('mousemove', onMouseMove, true);
document.addEventListener('click', onClick, true);
document.addEventListener('keydown', onKeyDown, true);
}
function stopPicker() {
if (!pickerActive) return;
pickerActive = false;
removeHighlight();
document.removeEventListener('mousemove', onMouseMove, true);
document.removeEventListener('click', onClick, true);
document.removeEventListener('keydown', onKeyDown, true);
}
// ─── Page Alteration Handlers ───────────────────────────────────
function findElement(selector) {
try {
return document.querySelector(selector);
} catch (e) {
if (!(e instanceof TypeError) && !(e instanceof DOMException)) throw e;
return null;
}
}
function applyStyle(selector, property, value) {
// Validate property name: alphanumeric + hyphens only
if (!/^[a-zA-Z-]+$/.test(property)) return { error: 'Invalid property name' };
// Validate CSS value: block exfiltration vectors (url(), expression(), @import, javascript:, data:)
if (/url\s*\(|expression\s*\(|@import|javascript:|data:/i.test(value)) {
return { error: 'CSS value contains blocked pattern' };
}
const el = findElement(selector);
if (!el) return { error: 'Element not found' };
// Track original value for resetAll
if (!originalStyles.has(el)) {
originalStyles.set(el, new Map());
}
const origMap = originalStyles.get(el);
if (!origMap.has(property)) {
origMap.set(property, el.style.getPropertyValue(property));
}
el.style.setProperty(property, value, 'important');
return { ok: true };
}
function toggleClass(selector, className, action) {
if (!/^[a-zA-Z0-9_-]+$/.test(className)) {
return { error: 'Invalid class name' };
}
const el = findElement(selector);
if (!el) return { error: 'Element not found' };
if (action === 'add') {
el.classList.add(className);
} else if (action === 'remove') {
el.classList.remove(className);
} else {
el.classList.toggle(className);
}
return { ok: true };
}
function injectCSS(id, css) {
if (!/^[a-zA-Z0-9_-]+$/.test(id)) {
return { error: 'Invalid CSS injection id' };
}
if (/url\s*\(|expression\s*\(|@import|javascript:|data:/i.test(css)) {
return { error: 'CSS contains blocked pattern (url, expression, @import)' };
}
const styleId = `gstack-inject-${id}`;
let styleEl = document.getElementById(styleId);
if (!styleEl) {
styleEl = document.createElement('style');
styleEl.id = styleId;
document.head.appendChild(styleEl);
}
styleEl.textContent = css;
injectedStyleIds.add(styleId);
return { ok: true };
}
function resetAll() {
// Restore original inline styles
for (const [el, propMap] of originalStyles) {
for (const [prop, origVal] of propMap) {
if (origVal) {
el.style.setProperty(prop, origVal);
} else {
el.style.removeProperty(prop);
}
}
}
originalStyles.clear();
// Remove injected style elements
for (const id of injectedStyleIds) {
const el = document.getElementById(id);
if (el) el.remove();
}
injectedStyleIds.clear();
return { ok: true };
}
// ─── Message Listener ──────────────────────────────────────────
chrome.runtime.onMessage.addListener((msg, sender, sendResponse) => {
if (msg.type === 'startPicker') {
startPicker();
sendResponse({ ok: true });
return;
}
if (msg.type === 'stopPicker') {
stopPicker();
sendResponse({ ok: true });
return;
}
if (msg.type === 'applyStyle') {
const result = applyStyle(msg.selector, msg.property, msg.value);
sendResponse(result);
return;
}
if (msg.type === 'toggleClass') {
const result = toggleClass(msg.selector, msg.className, msg.action);
sendResponse(result);
return;
}
if (msg.type === 'injectCSS') {
const result = injectCSS(msg.id, msg.css);
sendResponse(result);
return;
}
if (msg.type === 'resetAll') {
const result = resetAll();
sendResponse(result);
return;
}
});
})();