Skip to content

Require opt-in for local screenshot capture #69

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
saagpatel merged 1 commit into from
Jun 21, 2026
Merged

Require opt-in for local screenshot capture #69

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/executive-view-redesign/screenshots/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Both captured at 3840×4232 (1920-wide viewport × 2 DPR × full-page scroll).
# one-time, while the dev stack is up on :5173 + :8000
cd frontend
npm install --no-save --legacy-peer-deps puppeteer-core
node scripts/capture-screenshots.mjs
node scripts/capture-screenshots.mjs --allow-local-browser
```

Output lands in `docs/executive-view-redesign/screenshots/`.
Expand Down
23 changes: 21 additions & 2 deletions frontend/scripts/capture-screenshots.mjs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
import puppeteer from "puppeteer-core";
import path from "node:path";
import { fileURLToPath } from "node:url";

Expand All @@ -10,11 +9,31 @@ const OUT_DIR = path.resolve(
"../docs/executive-view-redesign/screenshots",
);

const allowed =
process.argv.includes("--allow-local-browser") ||
process.env.ITSH_ALLOW_SCREENSHOTS === "1";

if (!allowed) {
console.error(
"Refusing local screenshot capture without explicit opt-in. " +
"Pass --allow-local-browser or set ITSH_ALLOW_SCREENSHOTS=1.",
);
process.exit(2);
}

const { default: puppeteer } = await import("puppeteer-core");

const browser = await puppeteer.launch({
executablePath: CHROME,
headless: "new",
defaultViewport: { width: 1920, height: 1080, deviceScaleFactor: 2 },
args: ["--disable-web-security", "--no-sandbox"],
args: [
"--disable-web-security",
"--no-sandbox",
"--disable-gpu",
"--disable-dev-shm-usage",
...(process.platform === "darwin" ? ["--disable-features=Metal"] : []),
],
});

async function captureOperational() {
Expand Down
Loading