mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
Fixes 15 defects found by a max-effort multi-agent review of the last 6 merged PRs (#388, #390, #391, #392, #393, #394), all adversarially verified before fixing. Install queue + dispatcher (the serious cluster): - features.ts: finalize the installer child exactly once. A failed spawn fires both "error" and "close", and the second event released the file lock and active slot that pump() had just handed to the next queued bundle, letting two pip processes write the same venv concurrently. Outcome recording now happens before pump() so the next bundle's first progress frame cannot race the previous install's bookkeeping. - feature-status.ts: keep failed-install errors in a per-bundle map instead of the single progress slot. With the queue auto-starting the next install, the slot was overwritten within seconds and a failed install vanished without ever surfacing to GET /features. - bridge.ts: scope child lifecycle per process (stopped-children set + request generation tags) instead of an instance-wide shuttingDown flag that the next spawn reset. A stale SIGTERMed child's late close event could record a phantom crash (5 of which permanently disable the dispatcher), null out the freshly spawned child, and reject the new child's pending requests. The request-timeout kill path still counts as a real crash. - install_feature.py: the pre-write disk re-check measured ai_dir's filesystem even when budgeting the cross-filesystem copy that lands on the venv's disk; now each budget is checked against the filesystem the bytes actually land on, so ENOSPC cannot strike mid-write and leave site-packages half overwritten. Behavior regressions: - embed-subtitles: preserve pre-existing subtitle tracks (0:s?) and MKV attachments (0:t?) that the -map 0:v:0/0:a? rewrite silently dropped; data streams stay unmapped on purpose (the actual MPEG remux fix). The new subtitle maps first so the language tag hits the right stream. - usage-survey-overlay: fail closed when the settings fetch fails; the fail-open path rendered the blocking survey against an unhealthy API and soft-locked admins, the lock-out class #392 fixed. - features-store: queued bundles poll instead of each holding an SSE connection (Install All could pin 7 EventSources and exhaust the browser's 6-per-origin HTTP/1.1 limit, hanging the whole app); listenToProgress closes any prior stream and stops any poll before subscribing; installAll skips bundles already installing or queued. Contracts, tests, i18n: - openapi.yaml: add "queued" to the features status enum and document downloadBytes/installedBytes (Schemathesis conformance). - feature-lifecycle e2e: queue transcription (~0.5 GB) instead of ocr (~6 GB) and give the test a budget that covers both install drains (the stacked waits exceeded the old 900s timeout). - docker-compose.qa.yml: parameterize the host port (QA_APP_PORT) so QA_PROJECT_NAME concurrent stacks can actually bind. - compare + watermark-image: restore per-input error attribution ("Invalid first/second image", "Invalid watermark image") lost in the shared-handler migration. - ai-features-section: the "{size} on disk" suffix now goes through i18n; key added to all 21 locales. - watermark-image + content-aware-resize: migrate to the shared inputHandlerFor("image") chain like compare/vectorize/compose, fixing drift in the inline copies (no SVG sanitize, no RAW extension hint, no AVIF probe). Verified: typecheck across 9 workspaces, Biome clean on all changed files, 584 targeted unit tests and 249 integration tests green (including real-ffmpeg embed-subtitles runs). One unit test updated to the new poll-while-queued contract with a single-EventSource assertion. Claude-Session: https://claude.ai/code/session_017mR1HiHaf3a1BmUtrHX4j3
100 lines
4.4 KiB
TypeScript
100 lines
4.4 KiB
TypeScript
// Drives the REAL in-app AI bundle install flow (Settings > AI Features > Install
|
|
// All) against a running Docker container, exactly as a user would click through
|
|
// it. This is the primary AI-install path per docs/prompts/engineering/QA_PROMPT.md
|
|
// Phase 2: the curl-based /api/v1/admin/features/<bundle>/install route is a
|
|
// verification/fallback path only, never the primary install.
|
|
//
|
|
// This script only KICKS OFF the install and captures pre/mid-install evidence; the
|
|
// installs continue server-side once triggered. Poll status separately with:
|
|
// curl -s $QA_BASE_URL/api/v1/features -H "Authorization: Bearer $TOKEN" | jq
|
|
// then run verify-ai-install-complete.mts once every bundle reports installed.
|
|
//
|
|
// Usage:
|
|
// QA_BASE_URL=http://localhost:13599 QA_USERNAME=admin QA_PASSWORD=admin \
|
|
// apps/api/node_modules/.bin/tsx tests/qa/install-ai-bundles-ui.mts
|
|
import fs from "node:fs";
|
|
import path from "node:path";
|
|
import { chromium } from "@playwright/test";
|
|
|
|
const BASE = process.env.QA_BASE_URL || "http://localhost:13499";
|
|
const USERNAME = process.env.QA_USERNAME || "admin";
|
|
const PASSWORD = process.env.QA_PASSWORD || "admin";
|
|
const SHOT_DIR = path.join("tests", "e2e", "screenshots", "qa", "ai-install");
|
|
fs.mkdirSync(SHOT_DIR, { recursive: true });
|
|
|
|
async function shot(page: import("@playwright/test").Page, name: string) {
|
|
const file = path.join(SHOT_DIR, `${name}.png`);
|
|
await page.screenshot({ path: file, fullPage: true });
|
|
console.log(` screenshot: ${file}`);
|
|
}
|
|
|
|
async function main() {
|
|
const browser = await chromium.launch({ channel: "chrome" });
|
|
const page = await browser.newPage({ viewport: { width: 1440, height: 900 } });
|
|
|
|
// Don't interpolate the env-derived base URL / username into the log
|
|
// (clear-text logging of environment values is flagged by static analysis).
|
|
console.log("Logging in...");
|
|
await page.goto(BASE);
|
|
await page.waitForLoadState("networkidle").catch(() => {});
|
|
if (page.url().includes("/login")) {
|
|
await page.locator("#username").waitFor({ timeout: 10_000 });
|
|
await page.locator("#username").fill(USERNAME);
|
|
await page.locator("#password").fill(PASSWORD);
|
|
await page.getByRole("button", { name: /^log ?in$/i }).click();
|
|
await page.waitForURL((url) => !url.pathname.startsWith("/login"), { timeout: 15_000 });
|
|
} else {
|
|
console.log(" already authenticated (no login form shown)");
|
|
}
|
|
|
|
// Bail loudly if we land on the forced password-change screen; the caller
|
|
// should set SKIP_MUST_CHANGE_PASSWORD=true for automated runs.
|
|
if (page.url().includes("/change-password")) {
|
|
throw new Error(
|
|
"Landed on /change-password; set SKIP_MUST_CHANGE_PASSWORD=true on the container for automated QA runs",
|
|
);
|
|
}
|
|
|
|
console.log("Opening Settings > AI Features...");
|
|
await page.getByRole("button", { name: USERNAME, exact: true }).click();
|
|
await page.getByText("Settings", { exact: true }).click();
|
|
await page.getByText("AI Features", { exact: true }).click();
|
|
await page.waitForTimeout(1000);
|
|
|
|
await shot(page, "01-pre-install");
|
|
|
|
// Record pre-install state from the DOM text (cheap sanity check the download
|
|
// is provably on-demand, not pre-baked).
|
|
const preText = await page.locator("body").innerText();
|
|
const notInstalledCount = (preText.match(/Not installed/g) || []).length;
|
|
console.log(` bundles showing "Not installed" before click: ${notInstalledCount}`);
|
|
|
|
console.log("Clicking Install All...");
|
|
const installAllBtn = page.getByRole("button", { name: /install all/i });
|
|
await installAllBtn.click();
|
|
await page.waitForTimeout(3000);
|
|
await shot(page, "02-mid-install-immediate");
|
|
|
|
// Give the queue a bit longer to actually start downloading before the second
|
|
// "mid-download" screenshot the prompt asks for.
|
|
await page.waitForTimeout(30_000);
|
|
await shot(page, "03-mid-install-30s");
|
|
|
|
const midText = await page.locator("body").innerText();
|
|
const installingCount = (midText.match(/\d+%/g) || []).length;
|
|
const queuedCount = (midText.match(/Queued/g) || []).length;
|
|
console.log(` bundles showing a % progress at +33s: ${installingCount}`);
|
|
console.log(` bundles showing "Queued" at +33s: ${queuedCount}`);
|
|
|
|
console.log(
|
|
"Install kicked off and left running server-side. Poll /api/v1/features until every bundle is installed, then run verify-ai-install-complete.mts.",
|
|
);
|
|
|
|
await browser.close();
|
|
}
|
|
|
|
main().catch((err) => {
|
|
console.error("FAILED:", err);
|
|
process.exit(1);
|
|
});
|