@kolibri-day/feedback 0.0.0-stage → 0.1.0

package/dist/cjs/adapters/capture/capture.js

@@ -0,0 +1,398 @@
+"use strict";
+// Client-side screenshot capture of the CURRENT VIEWPORT.
+//
+// TWO RENDERERS, TRIED IN ORDER — not a platform split.
+//   1. modern-screenshot (foreignObject): serializes the DOM into an SVG and lets the
+//      BROWSER'S OWN ENGINE draw it. Measured against Chromium's real screenshot of
+//      the same page it scores a mean absolute error of 0.51 with 0.52% of pixels
+//      materially off, versus html2canvas's 2.48 / 2.05% — roughly 5x closer. It also
+//      gets natively right the things html2canvas fabricates: a closed <details> stays
+//      closed, <summary> keeps its ▶ instead of a bogus "1.", native <select> arrows
+//      and checkboxes render as themselves.
+//   2. html2canvas: the fallback, kept because foreignObject has a history of Safari
+//      bugs that CANNOT be tested from a headless environment, and iPhones are exactly
+//      what people report bugs from. Selection is by OUTCOME, not by sniffing the platform: if the
+//      first renderer throws or returns something unusable, we quietly use the second.
+//      A user-agent test would guess; this observes.
+//
+// WHY NOT getDisplayMedia (re-evaluated after repeated fidelity complaints, and still
+// the answer): it is genuinely pixel-exact, and that is a real advantage. But it is
+// UNSUPPORTED ON iOS SAFARI — undefined, not degraded — so it can never be the only
+// mechanism for this userbase; it prompts on every single report; and it can make the
+// PRIVACY POSTURE WORSE rather than better, because `preferCurrentTab` is only a hint
+// and one mis-click shares an entire desktop (mail, chat) into a shared tracker
+// project. A DOM render is bounded to this app's own markup by construction. If
+// pixel-exactness is still wanted, the next step is an OPT-IN "take a real
+// screenshot" button on desktop, leaving this silent path as the default everywhere.
+//
+// Remaining honest limits of any DOM render: cross-origin images cannot be inlined, so
+// a map view still has holes where third-party tiles are (the surrounding chrome
+// captures fine, which is usually where the bug is).
+//
+// Both renderers are DYNAMICALLY IMPORTED at capture time, so neither enters the main
+// bundle — people on mobile data do not pay for a feature they have not opened. They
+// are optional peer dependencies of this package: install `modern-screenshot` and
+// `html2canvas` in the app that captures.
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.REDACT_ATTRIBUTE = exports.HARD_MAX_BYTES = exports.TARGET_MAX_BYTES = exports.MAX_EDGE_PX = void 0;
+exports.fitWithin = fitWithin;
+exports.isRequestedType = isRequestedType;
+exports.encodeWithinBudget = encodeWithinBudget;
+exports.looksUsable = looksUsable;
+exports.captureViewport = captureViewport;
+/**
+ * Longest edge of an uploaded screenshot, in pixels.
+ *
+ * DO NOT LOWER THIS BACK toward ~1600 by reasoning about photo file sizes — that was
+ * the original mistake and it produced a real, reported defect.
+ *
+ * This started at 1600, sized against "a full-page PNG can be 2-6 MB". That is true of
+ * PHOTOGRAPHIC content and false of a UI screenshot in WebP: measured on a
+ * data-table page, the full-resolution 2560x1440 capture encodes to ~30 KB — against
+ * a 1 MB target. So the old cap bought nothing and cost legibility: a 2560-wide
+ * desktop was downscaled by 0.625, turning 12px badge text into 7.5px and smearing
+ * white-on-near-black labels into unreadable dark blocks.
+ *
+ * That was confirmed to be the RESIZE, not the encoder: the same downscale applied to
+ * a pixel-perfect browser screenshot smears identically, and lossless PNG at 1600
+ * smears just as badly. Only stopping the downscale fixes it.
+ *
+ * 2560 rather than unlimited: it still bounds an ultrawide or scaled-up display,
+ * where a capture genuinely can get large. Everything below it is now 1:1.
+ */
+exports.MAX_EDGE_PX = 2560;
+/** Target ceiling for the encoded blob; the encoder steps quality down to reach it. */
+exports.TARGET_MAX_BYTES = 1_000_000;
+/** Absolute client-side refusal (the server independently enforces its own cap). */
+exports.HARD_MAX_BYTES = 2_000_000;
+/**
+ * THE EXCLUSION CONTRACT: anything carrying this attribute is dropped from the capture
+ * by both renderers — the report dialog and its backdrop, the trigger button, a menu
+ * mid-animation, and any app data that must never appear in a screenshot.
+ */
+exports.REDACT_ATTRIBUTE = 'data-feedback-redact';
+/**
+ * Encoder quality ladder. 0.92 is the working value — fine light-on-dark text is the
+ * first thing to go, and it survives cleanly there. The lower rungs are a BACKSTOP for
+ * an unexpectedly heavy page (an image-rich view), not the normal path; a typical UI
+ * screenshot never leaves the first rung.
+ */
+const QUALITY_STEPS = [0.92, 0.8, 0.65];
+/**
+ * Target dimensions after downscaling. Pure — this is the arithmetic the tests pin
+ * down, separated from any canvas.
+ *
+ * Never upscales: a small phone viewport is already the size it is, and enlarging it
+ * would inflate the payload while adding exactly zero detail.
+ */
+function fitWithin(width, height, maxEdge = exports.MAX_EDGE_PX) {
+    const longest = Math.max(width, height);
+    if (longest <= maxEdge || longest === 0)
+        return { width, height };
+    const factor = maxEdge / longest;
+    // Floor + a floor of 1: rounding could land a hair over maxEdge, and a zero
+    // dimension would make the canvas throw.
+    return { width: Math.max(1, Math.floor(width * factor)), height: Math.max(1, Math.floor(height * factor)) };
+}
+/**
+ * Did the canvas actually encode the type we asked for?
+ *
+ * A canvas asked for a type it cannot encode does not fail — it silently returns PNG
+ * instead. So the requested type is not evidence of anything; `blob.type` is. An
+ * empty type (very old engines) is taken at its word.
+ */
+function isRequestedType(produced, requested) {
+    return produced === '' || produced === requested;
+}
+/** Draw `source` into a downscaled canvas. Separated so capture() stays readable. */
+function downscale(source, maxEdge) {
+    const { width, height } = fitWithin(source.width, source.height, maxEdge);
+    if (width === source.width && height === source.height)
+        return source;
+    const target = document.createElement('canvas');
+    target.width = width;
+    target.height = height;
+    const ctx = target.getContext('2d');
+    if (!ctx)
+        return source;
+    // Smooth downsampling; the default is already 'low' in some engines and text
+    // suffers noticeably at these ratios.
+    ctx.imageSmoothingEnabled = true;
+    ctx.imageSmoothingQuality = 'high';
+    ctx.drawImage(source, 0, 0, width, height);
+    return target;
+}
+function toBlob(canvas, type, quality) {
+    return new Promise((resolve) => canvas.toBlob(resolve, type, quality));
+}
+/**
+ * Encode, stepping quality down until the blob fits `targetBytes`.
+ *
+ * Formats in order of preference: WebP (visibly better than JPEG on UI text at the
+ * same size), then JPEG. A format counts as supported only if the browser actually
+ * produced it (isRequestedType): a browser that cannot encode WebP hands back a PNG
+ * rather than null, and taking that PNG as "WebP done" used to skip JPEG entirely —
+ * leaving such browsers with an uncompressible PNG and a quality ladder that did
+ * nothing. A substituted PNG is kept only as the last resort, when neither lossy
+ * format is available.
+ *
+ * Terminates unconditionally: the ladder is finite, and the smallest blob of the
+ * supported format is returned whether or not it met the target — the caller (and the
+ * server) enforce the hard cap.
+ */
+async function encodeWithinBudget(canvas, targetBytes = exports.TARGET_MAX_BYTES) {
+    let substitute = null;
+    for (const type of ['image/webp', 'image/jpeg']) {
+        let smallest = null;
+        for (const quality of QUALITY_STEPS) {
+            const blob = await toBlob(canvas, type, quality);
+            if (!blob)
+                break; // cannot encode this type at all
+            if (!isRequestedType(blob.type, type)) {
+                substitute ??= blob; // unsupported — the browser fell back to PNG
+                break;
+            }
+            if (blob.size <= targetBytes)
+                return blob;
+            if (!smallest || blob.size < smallest.size)
+                smallest = blob;
+        }
+        if (smallest)
+            return smallest; // supported, just heavy: the best this format can do
+    }
+    return substitute;
+}
+/** Should this element be kept out of the capture entirely? */
+function isRedacted(el, attribute) {
+    return el.nodeType === 1 && el.hasAttribute?.(attribute) === true;
+}
+/**
+ * A render is "usable" if it has real pixels. foreignObject failures are quiet — a
+ * browser that chokes on the serialized SVG hands back a blank or zero-sized canvas
+ * rather than throwing — so a try/catch alone would let a white rectangle through and
+ * call it a screenshot. This is what makes the fallback trigger on the failure mode
+ * that actually happens.
+ */
+function looksUsable(canvas) {
+    if (canvas.width < 2 || canvas.height < 2)
+        return false;
+    try {
+        const ctx = canvas.getContext('2d');
+        if (!ctx)
+            return false;
+        // Sample a coarse grid rather than every pixel: enough to tell "drew something"
+        // from "drew nothing", cheap enough to run on every capture.
+        const { data } = ctx.getImageData(0, 0, canvas.width, canvas.height);
+        const step = Math.max(4, Math.floor((canvas.width * canvas.height) / 5000)) * 4;
+        let first = -1;
+        for (let i = 0; i < data.length; i += step) {
+            const lum = (data[i] ?? 0) + (data[i + 1] ?? 0) + (data[i + 2] ?? 0);
+            if (first === -1)
+                first = lum;
+            else if (Math.abs(lum - first) > 12)
+                return true; // some variation ⇒ real content
+        }
+        return false; // uniform ⇒ blank
+    }
+    catch {
+        // Tainted canvas (a cross-origin image got in). Unreadable here, and unencodable
+        // later — treat as unusable so the other renderer gets its turn.
+        return false;
+    }
+}
+/**
+ * Preferred renderer: modern-screenshot, via foreignObject.
+ *
+ * The browser draws it, so UA behaviour (a closed `<details>`, `<summary>`'s marker,
+ * native form controls) is correct for free — no clone patching required.
+ *
+ * Scroll handling differs from html2canvas, which takes x/y/scroll options: here the
+ * clone is translated by the scroll offset and the output box fixed to the viewport,
+ * which crops to the visible region.
+ */
+async function renderWithModernScreenshot(attribute) {
+    const { domToCanvas } = await Promise.resolve().then(() => __importStar(require('modern-screenshot')));
+    const canvas = await domToCanvas(document.documentElement, {
+        width: window.innerWidth,
+        height: window.innerHeight,
+        backgroundColor: getComputedStyle(document.body).backgroundColor || '#ffffff',
+        scale: 1,
+        style: {
+            transform: `translate(${-window.scrollX}px, ${-window.scrollY}px)`,
+            transformOrigin: 'top left',
+        },
+        // Same redaction contract as the html2canvas path — `filter` returns false to drop
+        // a node. Keeping both honour one attribute is what stops the two renderers from
+        // disagreeing about what is sensitive.
+        filter: (node) => !(node instanceof Element && isRedacted(node, attribute)),
+    });
+    return canvas ?? null;
+}
+/**
+ * html2canvas ships CommonJS with an ES-module build beside it. A bundler picks the
+ * ES build, where the function IS the default export; Node's own ESM-to-CJS interop
+ * wraps `module.exports` once more, so there it sits at `.default.default`. Accept
+ * both rather than depend on which one the consuming app's tooling chose.
+ */
+async function loadHtml2Canvas() {
+    const mod = (await Promise.resolve().then(() => __importStar(require('html2canvas'))));
+    return typeof mod.default === 'function' ? mod.default : mod.default.default;
+}
+/**
+ * Fallback renderer: html2canvas.
+ *
+ * Needs the clone patched (see neutralizeCollapsedDetails) because it re-implements
+ * layout from computed styles. That workaround stays here, and ONLY here — deleting it
+ * along with the switch to modern-screenshot would quietly reintroduce the original
+ * over-painted-filter-panel bug for anyone whose browser lands on this path.
+ */
+async function renderWithHtml2Canvas(attribute) {
+    const html2canvas = await loadHtml2Canvas();
+    return html2canvas(document.body, {
+        // Anything the app marks as sensitive never enters the render — the permanent
+        // hook for excluding e.g. a salary column from every screenshot. The trigger
+        // button and the report dialog carry the same attribute.
+        ignoreElements: (el) => isRedacted(el, attribute),
+        onclone: neutralizeCollapsedDetails,
+        // CSS pixels, not devicePixelRatio. (Sharpening mobile captures by rendering at
+        // DPR is a possible enhancement, deliberately not done here.)
+        scale: 1,
+        useCORS: true,
+        // Cross-origin images that can't be read taint the canvas and make toBlob throw;
+        // dropping them costs a blank rectangle and saves the whole capture.
+        allowTaint: false,
+        logging: false,
+        backgroundColor: getComputedStyle(document.body).backgroundColor || '#ffffff',
+        // The visible window only.
+        x: window.scrollX,
+        y: window.scrollY,
+        width: window.innerWidth,
+        height: window.innerHeight,
+        windowWidth: document.documentElement.clientWidth,
+        windowHeight: document.documentElement.clientHeight,
+        scrollX: 0,
+        scrollY: 0,
+    });
+}
+/**
+ * Make a COLLAPSED `<details>` actually look collapsed in the capture.
+ *
+ * html2canvas walks the DOM and paints anything whose COMPUTED STYLE says it is
+ * visible. But a closed `<details>` is collapsed by the user agent — the content lives
+ * in a slot the browser simply doesn't paint — and that is not expressed in CSS at
+ * all. Measured on a closed panel: `display: flex`, `visibility: visible`,
+ * `content-visibility: visible`, and a full 950x82 bounding rect. So html2canvas has
+ * every reason to draw it, and does — over whatever follows.
+ *
+ * That produced a reported defect: a closed "more filters" panel was painted on top of
+ * the table header, making the screenshot actively misleading (it showed filter fields
+ * the reporter never had open).
+ *
+ * The `<summary>` line is the same root cause wearing a different hat: `summary` is
+ * `display: list-item`, and html2canvas renders that marker as an ORDERED-LIST NUMBER,
+ * so a stray "1." appeared beside the panel's title. Forcing `display: block` drops the
+ * marker box. The cost is losing the ▶ disclosure triangle, which html2canvas cannot
+ * draw anyway — a missing triangle beats a fabricated "1.".
+ *
+ * Mutates the CLONE html2canvas renders from, never the live document.
+ */
+function neutralizeCollapsedDetails(doc) {
+    doc.querySelectorAll('details:not([open])').forEach((details) => {
+        for (const child of Array.from(details.children)) {
+            if (child.tagName !== 'SUMMARY')
+                child.style.display = 'none';
+        }
+    });
+    doc.querySelectorAll('summary').forEach((summary) => {
+        summary.style.display = 'block';
+    });
+}
+/**
+ * Capture the CURRENT VIEWPORT.
+ *
+ * Viewport, not full page, on purpose: what the reporter is looking at is what they
+ * are reporting, a full-page render multiplies the payload, and sticky/fixed elements
+ * render badly when the capture is taller than the window.
+ *
+ * THE EXCLUSION CONTRACT: anything that must not appear in the shot carries
+ * REDACT_ATTRIBUTE — both renderers drop those subtrees. That attribute, NOT
+ * call ordering, is what keeps the report dialog out of the screenshot: the dialog
+ * (panel and overlay both marked) opens in the same tick as the capture starts,
+ * because a full-page render measures in SECONDS (~1.5-4.5s at ~2000 elements) and
+ * holding the dialog for it made the button read as frozen. Two shipped mistakes
+ * inform this design: v1 opened the dialog unredacted before capturing (the form was
+ * in every screenshot); v2 awaited the capture before opening (correct pixels, 5-8s
+ * of dead air). Anything new that must never be captured — a future shake-to-report
+ * toast, a debug panel — gets the attribute, not a sequencing rule.
+ *
+ * Returns null on any failure. A missing screenshot degrades the report; a thrown
+ * error would lose it entirely, and the reporter's text is the part that cannot be
+ * re-derived.
+ */
+async function captureViewport(options = {}) {
+    if (typeof window === 'undefined' || typeof document === 'undefined')
+        return null;
+    const attribute = options.redactAttribute ?? exports.REDACT_ATTRIBUTE;
+    // Preferred first, fallback second — chosen by OUTCOME (threw, or produced something
+    // blank/tainted), never by sniffing the user agent.
+    let rendered = null;
+    for (const render of [renderWithModernScreenshot, renderWithHtml2Canvas]) {
+        try {
+            const candidate = await render(attribute);
+            if (candidate && looksUsable(candidate)) {
+                rendered = candidate;
+                break;
+            }
+        }
+        catch {
+            // Try the next renderer. Both failing is handled below.
+        }
+    }
+    if (!rendered)
+        return null;
+    try {
+        const canvas = downscale(rendered, options.maxEdge ?? exports.MAX_EDGE_PX);
+        const blob = await encodeWithinBudget(canvas, options.targetBytes ?? exports.TARGET_MAX_BYTES);
+        if (!blob || blob.size > (options.hardMaxBytes ?? exports.HARD_MAX_BYTES))
+            return null;
+        return { blob, width: canvas.width, height: canvas.height };
+    }
+    catch {
+        // Encoding failed (out of memory on a huge page). The report still goes, without
+        // a screenshot — the reporter's text is what cannot be re-derived.
+        return null;
+    }
+}

package/dist/cjs/adapters/capture/client-context.d.ts

@@ -0,0 +1,9 @@
+import type { FeedbackClientContext } from '../../kernel/feedback/types.js';
+/**
+ * Collect the current view's context.
+ *
+ * `win` is injectable so this is testable in node without a DOM — the alternative
+ * (a jsdom environment for one function) would pull a second test environment into a
+ * suite that is deliberately node-only.
+ */
+export declare function collectClientContext(win?: Window & typeof globalThis): FeedbackClientContext;

package/dist/cjs/adapters/capture/client-context.js

@@ -0,0 +1,41 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.collectClientContext = collectClientContext;
+/** Anything that can throw here yields `undefined`, never an exception. */
+function safe(read) {
+    try {
+        return read();
+    }
+    catch {
+        return undefined;
+    }
+}
+/**
+ * Collect the current view's context.
+ *
+ * `win` is injectable so this is testable in node without a DOM — the alternative
+ * (a jsdom environment for one function) would pull a second test environment into a
+ * suite that is deliberately node-only.
+ */
+function collectClientContext(win = globalThis) {
+    const nav = win.navigator;
+    const doc = win.document;
+    return {
+        url: safe(() => win.location?.href) ?? '',
+        title: safe(() => doc?.title) ?? '',
+        viewport: safe(() => `${win.innerWidth}x${win.innerHeight}`) ?? '',
+        screen: safe(() => (win.screen ? `${win.screen.width}x${win.screen.height}` : '')) ?? '',
+        devicePixelRatio: safe(() => win.devicePixelRatio) ?? 1,
+        userAgent: safe(() => nav?.userAgent) ?? '',
+        language: safe(() => nav?.language) ?? '',
+        // Intl is present everywhere this app runs, but a locked-down browser can still
+        // make resolvedOptions throw; the timezone is a nice-to-have, not a reason to fail.
+        timezone: safe(() => Intl.DateTimeFormat().resolvedOptions().timeZone) ?? '',
+        online: safe(() => nav?.onLine) ?? true,
+        // People work on mobile data; "it only breaks on 3g" is a real bug class.
+        connection: safe(() => nav?.connection?.effectiveType) ?? null,
+        // The CLIENT clock, deliberately alongside the server's own timestamp. Comparing
+        // the two is free and explains a whole family of date/timezone bugs.
+        clientTime: safe(() => new Date().toISOString()) ?? '',
+    };
+}

package/dist/cjs/adapters/capture/reporter.d.ts

@@ -0,0 +1,42 @@
+import { type CaptureOptions, type CaptureResult } from './capture.js';
+export type CaptureState = {
+    status: 'idle';
+} | {
+    status: 'capturing';
+} | {
+    status: 'failed';
+} | ({
+    status: 'ready';
+} & CaptureResult);
+export interface FeedbackReporterSnapshot {
+    open: boolean;
+    capture: CaptureState;
+}
+export interface FeedbackReporterOptions {
+    /** Replace the capture (tests, or an app with its own renderer). Default: captureViewport(captureOptions). */
+    capture?: () => Promise<CaptureResult | null>;
+    captureOptions?: CaptureOptions;
+}
+export declare class FeedbackReporterController {
+    private snapshot;
+    private readonly listeners;
+    /** The in-flight capture, for a submit that arrives before it resolves. */
+    private pending;
+    /** Generation stamp, so a capture orphaned by close() can never write stale state. */
+    private generation;
+    private readonly capture;
+    constructor(options?: FeedbackReporterOptions);
+    /** The current state. The same object until something changes — safe for useSyncExternalStore. */
+    getSnapshot: () => FeedbackReporterSnapshot;
+    subscribe: (listener: () => void) => (() => void);
+    private set;
+    /** Start the capture and open, in the same tick. A no-op while already open. */
+    open: () => void;
+    /**
+     * Close and forget. The render itself cannot be aborted, but its result is
+     * discarded — no reason to hold an image for a report that was cancelled.
+     */
+    close: () => void;
+    /** The settled capture — immediately when done, or after an in-flight one finishes. */
+    waitForCapture: () => Promise<CaptureResult | null>;
+}

package/dist/cjs/adapters/capture/reporter.js

@@ -0,0 +1,83 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.FeedbackReporterController = void 0;
+// The reporter's state machine — open/close and the screenshot that runs alongside —
+// without any UI framework. `@kolibri-day/feedback/react` binds it to React; a Vue or
+// vanilla app subscribes to it directly.
+//
+// THE DIALOG OPENS IMMEDIATELY; THE CAPTURE RESOLVES INTO ITS PREVIEW. This sequencing
+// is the third iteration, and the constraint it finally gets right is about PIXELS,
+// not order. v1 opened the dialog before capturing — the form was in every screenshot.
+// v2 held the dialog until the capture finished — correct pixels, but on a real page a
+// DOM render takes seconds (measured ~4.5s for modern-screenshot and ~1.7s for
+// html2canvas at ~2000 elements), and a button that does nothing for seconds reads as
+// broken. No renderer choice fixes that; the render cost has to come OFF the
+// click-to-dialog path.
+//
+// So: the capture starts at open(), the dialog opens in the same tick, and the dialog
+// stays out of the shot because its panel AND its backdrop carry REDACT_ATTRIBUTE.
+// The preview shows "capturing…" until the frame lands; a submit that outruns it
+// awaits waitForCapture().
+//
+// Known minor artifact, accepted: a modal that locks page scroll removes the
+// scrollbar, so on classic-scrollbar systems the capture can sit ~15px off from what
+// was on screen. A shifted-by-a-scrollbar screenshot is a fair trade for an instant
+// dialog.
+const capture_ts_1 = require("./capture.js");
+const IDLE = Object.freeze({
+    open: false,
+    capture: Object.freeze({ status: 'idle' }),
+});
+class FeedbackReporterController {
+    snapshot = IDLE;
+    listeners = new Set();
+    /** The in-flight capture, for a submit that arrives before it resolves. */
+    pending = null;
+    /** Generation stamp, so a capture orphaned by close() can never write stale state. */
+    generation = 0;
+    capture;
+    constructor(options = {}) {
+        this.capture = options.capture ?? (() => (0, capture_ts_1.captureViewport)(options.captureOptions));
+    }
+    /** The current state. The same object until something changes — safe for useSyncExternalStore. */
+    getSnapshot = () => this.snapshot;
+    subscribe = (listener) => {
+        this.listeners.add(listener);
+        return () => this.listeners.delete(listener);
+    };
+    set(next) {
+        this.snapshot = next;
+        for (const listener of this.listeners)
+            listener();
+    }
+    /** Start the capture and open, in the same tick. A no-op while already open. */
+    open = () => {
+        if (this.snapshot.open)
+            return;
+        const generation = ++this.generation;
+        // A capture that throws (it should not — captureViewport returns null on failure)
+        // still settles as "no screenshot" rather than an unhandled rejection.
+        const pending = this.capture().catch(() => null);
+        this.pending = pending;
+        this.set({ open: true, capture: { status: 'capturing' } });
+        void pending.then((result) => {
+            if (this.generation !== generation)
+                return; // closed meanwhile — stale frame
+            this.set({ open: true, capture: result ? { status: 'ready', ...result } : { status: 'failed' } });
+        });
+    };
+    /**
+     * Close and forget. The render itself cannot be aborted, but its result is
+     * discarded — no reason to hold an image for a report that was cancelled.
+     */
+    close = () => {
+        this.generation++;
+        this.pending = null;
+        this.set(IDLE);
+    };
+    /** The settled capture — immediately when done, or after an in-flight one finishes. */
+    waitForCapture = async () => {
+        return this.pending ? await this.pending : null;
+    };
+}
+exports.FeedbackReporterController = FeedbackReporterController;

package/dist/cjs/adapters/capture/submit.d.ts

@@ -0,0 +1,30 @@
+import type { FeedbackClientContext } from '../../kernel/feedback/types.js';
+export interface FeedbackSubmitInput {
+    kind: string;
+    message: string;
+    clientContext?: FeedbackClientContext;
+    /** The capture's blob, or null/undefined for a text-only report. */
+    screenshot?: Blob | null;
+}
+/** Build the multipart body. Identity is deliberately not a field — the server takes it from the session. */
+export declare function buildFeedbackForm(input: FeedbackSubmitInput): FormData;
+export type FeedbackSubmitResult = {
+    ok: true;
+    id: string | null;
+} | {
+    ok: false;
+    status: number;
+    /** The server's message for the reporter, when it sent one (`{ "error": "…" }`). */
+    error: string | null;
+};
+/**
+ * POST the report. Resolves with the server's verdict; REJECTS only when the request
+ * never got an answer (offline, DNS, CORS) — the caller words that case differently
+ * ("network error, try again") from a refusal.
+ *
+ * Success means the server ACCEPTED the report — with the durable pipeline, that it is
+ * stored, not that it reached the tracker. Word the confirmation accordingly.
+ */
+export declare function submitFeedback(endpoint: string, input: FeedbackSubmitInput, options?: {
+    fetch?: typeof fetch;
+}): Promise<FeedbackSubmitResult>;

package/dist/cjs/adapters/capture/submit.js

@@ -0,0 +1,32 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.buildFeedbackForm = buildFeedbackForm;
+exports.submitFeedback = submitFeedback;
+const files_ts_1 = require("../../kernel/feedback/files.js");
+/** Build the multipart body. Identity is deliberately not a field — the server takes it from the session. */
+function buildFeedbackForm(input) {
+    const form = new FormData();
+    form.set('kind', input.kind);
+    form.set('message', input.message);
+    form.set('clientContext', JSON.stringify(input.clientContext ?? {}));
+    if (input.screenshot)
+        form.set('screenshot', input.screenshot, (0, files_ts_1.screenshotFilename)(input.screenshot.type));
+    return form;
+}
+/**
+ * POST the report. Resolves with the server's verdict; REJECTS only when the request
+ * never got an answer (offline, DNS, CORS) — the caller words that case differently
+ * ("network error, try again") from a refusal.
+ *
+ * Success means the server ACCEPTED the report — with the durable pipeline, that it is
+ * stored, not that it reached the tracker. Word the confirmation accordingly.
+ */
+async function submitFeedback(endpoint, input, options = {}) {
+    const doFetch = options.fetch ?? fetch;
+    const res = await doFetch(endpoint, { method: 'POST', body: buildFeedbackForm(input) });
+    const body = (await res.json().catch(() => null));
+    if (!res.ok) {
+        return { ok: false, status: res.status, error: typeof body?.error === 'string' ? body.error : null };
+    }
+    return { ok: true, id: typeof body?.id === 'string' ? body.id : null };
+}

package/dist/cjs/adapters/kolibri/env.d.ts

@@ -0,0 +1,16 @@
+import { type KolibriConfig } from './target.js';
+import { type FeedbackMessages } from '../../kernel/feedback/messages.js';
+export type Env = Readonly<Record<string, string | undefined>>;
+export declare const KOLIBRI_ENV_VARS: readonly ["KOLIBRI_API_URL", "KOLIBRI_TOKEN", "KOLIBRI_WORKSPACE_ID", "KOLIBRI_PROJECT_ID"];
+export type KolibriEnvResult = {
+    ok: true;
+    config: Pick<KolibriConfig, 'apiUrl' | 'token' | 'workspaceId' | 'projectId'>;
+} | {
+    ok: false;
+    reason: string;
+};
+/**
+ * Read and validate the four variables. Never throws: a problem comes back as a
+ * reason worded for an operator.
+ */
+export declare function kolibriConfigFromEnv(env: Env, messages?: FeedbackMessages): KolibriEnvResult;