@eifi1/ui-kit 0.53.0 → 0.55.0

package/src/eslint/index.ts

@@ -1,161 +0,0 @@
-// `@eifi1/ui-kit/eslint` — the kit's role-utility rule for an app's own markup (round 0.35
-// R5, Kurvenschmiede's Q6).
-//
-// The kit holds its markup to "write `text-muted`, not `text-[var(--text-muted)]`" with
-// `npm run check:tokens`, at zero since 0.34. The apps are asked to follow the same rule
-// (README "How to write a colour"), but nothing in their builds objected when a
-// `[var(--role)]` class came back. This is that check as an ESLint plugin, so an app's
-// `eslint.config.js` turns it on with one line:
-//
-//   import uiKit from "@eifi1/ui-kit/eslint";
-//   export default [..., uiKit.configs.recommended];
-//
-// It reads every string literal and template-literal quasi — a JSX `className`, a `cn()` or
-// `clsx()` argument, a class constant — and reports a `[var(--x)]` class where the property
-// has a role utility for that token, naming it. One utility is an autofix that keeps the
-// variants and the opacity: `hover:text-[var(--text-muted)]/70` → `hover:text-muted/70`.
-//
-// Plain Node, like the rest of an ESLint config: no React, no stylesheet, and no import of
-// `eslint` itself, which is the app's (packaging-contract.test.ts refuses an undeclared
-// package in src/), so the few types it needs are written out below.
-import { findRoleClasses } from "./role-utilities";
-import { ROLE_UTILITIES } from "./role-utilities.generated";
-
-interface Position {
-  line: number;
-  column: number;
-}
-
-/** The part of an ESTree string node the rule reads. */
-interface StringNode {
-  type: string;
-  value?: unknown;
-  range?: [number, number];
-  loc?: { start: Position; end: Position } | null;
-}
-
-interface Fix {
-  range: [number, number];
-  text: string;
-}
-
-/**
- * The part of ESLint's rule context the rule uses — `sourceCode.text` and `report`, which
- * every ESLint since flat config has — so that ESLint's own RuleContext, and @eslint/core's
- * that `defineConfig` checks a plugin against, are assignable to it. The source is only an
- * `object` here because a language's source may be binary, with no `text` at all.
- */
-interface RoleUtilitiesContext {
-  sourceCode: object;
-  report(descriptor: {
-    loc: { start: Position; end: Position };
-    messageId: string;
-    data: Record<string, string>;
-    fix: ((fixer: { replaceTextRange(range: [number, number], text: string): Fix }) => Fix) | null;
-  }): void;
-}
-
-interface RoleUtilitiesRule {
-  meta: {
-    type: "suggestion";
-    docs: { description: string; url: string };
-    fixable: "code";
-    schema: [];
-    messages: { useRoleUtility: string };
-  };
-  create(context: RoleUtilitiesContext): {
-    Literal(node: StringNode): void;
-    TemplateElement(node: StringNode): void;
-  };
-}
-
-const roleUtilities: RoleUtilitiesRule = {
-  meta: {
-    type: "suggestion",
-    docs: {
-      description: "Write a colour role's utility (text-muted) rather than its [var(--…)] class",
-      url: "https://github.com/Eifi1/ui-kit#lint-the-role-utilities-in-an-app",
-    },
-    fixable: "code",
-    schema: [],
-    messages: {
-      useRoleUtility:
-        "Write the role utility `{{utilities}}`, not `{{found}}`: variants and an opacity work on it the same (tokens.css, @theme inline).",
-    },
-  },
-  create(context) {
-    const source = context.sourceCode as { text?: unknown };
-    const text = typeof source.text === "string" ? source.text : "";
-    const check = (node: StringNode) => {
-      if (!node.range || !node.loc) return;
-      const [start] = node.range;
-      const { line, column } = node.loc.start;
-      // An offset in the node's source text — quotes included, so it is an offset in the
-      // file — as a position. A template literal may span lines.
-      const at = (offset: number): Position => {
-        const lines = text.slice(start, start + offset).split("\n");
-        const last = lines.length - 1;
-        return { line: line + last, column: last ? lines[last].length : column + offset };
-      };
-      for (const hit of findRoleClasses(text.slice(start, node.range[1]), ROLE_UTILITIES)) {
-        const end = hit.index + hit.found.length;
-        const range: [number, number] = [start + hit.index, start + end];
-        const [only] = hit.utilities;
-        context.report({
-          loc: { start: at(hit.index), end: at(end) },
-          messageId: "useRoleUtility",
-          data: { found: hit.found, utilities: hit.utilities.join("` or `") },
-          // Several names for one token are a choice, not a fix.
-          fix: hit.utilities.length === 1 ? (fixer) => fixer.replaceTextRange(range, only) : null,
-        });
-      }
-    };
-    return {
-      Literal(node) {
-        if (typeof node.value === "string") check(node);
-      },
-      TemplateElement: check,
-    };
-  },
-};
-
-// The kit's version, baked in by tsup's `define` (tsup.config.ts). ESLint's cache keys a
-// file's result on each plugin's name@version, so without it an app's `eslint --cache`
-// would keep a file's old verdict after a kit release that gave a role a utility. Unset
-// when the source runs (the tests), where nothing is cached.
-declare const __UI_KIT_VERSION__: string | undefined;
-const VERSION = typeof __UI_KIT_VERSION__ === "string" ? __UI_KIT_VERSION__ : "0.0.0-source";
-
-/** Spelled out, self-reference included, so a typed config (`defineConfig`, `tseslint.config`) takes it. */
-interface UiKitEslintPlugin {
-  meta: { name: string; version: string; namespace: string };
-  rules: { "role-utilities": RoleUtilitiesRule };
-  configs: {
-    /** The rule as an error, on every file the app's other config objects lint but its tests. */
-    recommended: {
-      name: string;
-      ignores: string[];
-      plugins: { "@eifi1/ui-kit": UiKitEslintPlugin };
-      rules: { "@eifi1/ui-kit/role-utilities": "error" };
-    };
-  };
-}
-
-const recommended: UiKitEslintPlugin["configs"]["recommended"] = {
-  name: "@eifi1/ui-kit/recommended",
-  // Not the tests, as check:tokens skips the kit's: a test names the old class to assert it
-  // is gone (keksdose's cleared-status-toggle.test.tsx does, three times).
-  ignores: ["**/__tests__/**", "**/*.test.*", "**/*.spec.*"],
-  plugins: {} as UiKitEslintPlugin["configs"]["recommended"]["plugins"],
-  rules: { "@eifi1/ui-kit/role-utilities": "error" },
-};
-
-/** The flat-config plugin: add `configs.recommended`, or register it under `@eifi1/ui-kit`. */
-const plugin: UiKitEslintPlugin = {
-  meta: { name: "@eifi1/ui-kit", version: VERSION, namespace: "@eifi1/ui-kit" },
-  rules: { "role-utilities": roleUtilities },
-  configs: { recommended },
-};
-recommended.plugins["@eifi1/ui-kit"] = plugin;
-
-export default plugin;

package/src/eslint/role-utilities.generated.ts

@@ -1,146 +0,0 @@
-// GENERATED by scripts/gen-role-utilities.mjs from tokens.css's `@theme inline` block — do
-// not edit. `npm run check:tokens` fails while it is stale; re-run the script.
-//
-// Per property, each token that has a role utility → the utility's name after the
-// property's prefix (`border`: `--border` → `subtle`, i.e. `border-subtle`). Read by
-// `@eifi1/ui-kit/eslint` and scripts/check-token-discipline.mjs (round 0.35 R5).
-import type { RoleUtilityTable } from "./role-utilities";
-
-export const ROLE_UTILITIES: RoleUtilityTable = {
-  "text": {
-    "--text-primary": ["primary"],
-    "--text-secondary": ["secondary"],
-    "--text-muted": ["muted"],
-    "--text-placeholder": ["placeholder"],
-    "--text-inverse": ["inverse"],
-    "--brand": ["brand"],
-    "--brand-muted": ["brand-muted"],
-    "--danger": ["danger"],
-    "--warning": ["warning"],
-    "--success": ["success"],
-    "--info": ["info"],
-    "--money-income": ["money-income"],
-    "--money-expense": ["money-expense"],
-    "--money-net": ["money-net"],
-    "--money-neutral": ["money-neutral"],
-    "--hue-blue": ["hue-blue"],
-    "--hue-indigo": ["hue-indigo"],
-    "--hue-purple": ["hue-purple"],
-    "--hue-teal": ["hue-teal"],
-    "--hue-orange": ["hue-orange"],
-    "--brand-contrast": ["brand-contrast"],
-    "--danger-contrast": ["danger-contrast"],
-    "--warning-contrast": ["warning-contrast"],
-    "--success-contrast": ["success-contrast"],
-    "--info-contrast": ["info-contrast"],
-    "--neutral-contrast": ["neutral-contrast"],
-    "--money-income-contrast": ["money-income-contrast"],
-    "--money-expense-contrast": ["money-expense-contrast"],
-    "--money-net-contrast": ["money-net-contrast"],
-    "--money-neutral-contrast": ["money-neutral-contrast"],
-    "--hue-blue-contrast": ["hue-blue-contrast"],
-    "--hue-indigo-contrast": ["hue-indigo-contrast"],
-    "--hue-purple-contrast": ["hue-purple-contrast"],
-    "--hue-teal-contrast": ["hue-teal-contrast"],
-    "--hue-orange-contrast": ["hue-orange-contrast"],
-    "--media-ink": ["media-ink"],
-  },
-  "bg": {
-    "--bg-page": ["page"],
-    "--bg-surface": ["surface"],
-    "--bg-surface-2": ["surface-2"],
-    "--bg-hover": ["hover"],
-    "--bg-active": ["active"],
-    "--bg-inverse": ["inverse"],
-    "--brand": ["brand"],
-    "--brand-hover": ["brand-hover"],
-    "--brand-bg": ["brand-soft"],
-    "--brand-bg-hover": ["brand-soft-hover"],
-    "--danger": ["danger"],
-    "--danger-hover": ["danger-hover"],
-    "--warning": ["warning"],
-    "--success": ["success"],
-    "--info": ["info"],
-    "--neutral": ["neutral"],
-    "--money-income": ["money-income"],
-    "--money-expense": ["money-expense"],
-    "--money-net": ["money-net"],
-    "--money-neutral": ["money-neutral"],
-    "--danger-bg": ["danger-soft"],
-    "--warning-bg": ["warning-soft"],
-    "--success-bg": ["success-soft"],
-    "--info-bg": ["info-soft"],
-    "--hue-blue": ["hue-blue"],
-    "--hue-indigo": ["hue-indigo"],
-    "--hue-purple": ["hue-purple"],
-    "--hue-teal": ["hue-teal"],
-    "--hue-orange": ["hue-orange"],
-    "--hue-blue-bg": ["hue-blue-soft"],
-    "--hue-indigo-bg": ["hue-indigo-soft"],
-    "--hue-purple-bg": ["hue-purple-soft"],
-    "--hue-teal-bg": ["hue-teal-soft"],
-    "--hue-orange-bg": ["hue-orange-soft"],
-    "--media-scrim": ["media-scrim"],
-    "--media-scrim-hover": ["media-scrim-hover"],
-  },
-  "border": {
-    "--border": ["subtle"],
-    "--border-strong": ["strong"],
-    "--brand": ["brand"],
-    "--danger-border": ["danger"],
-    "--warning-border": ["warning"],
-    "--success-border": ["success"],
-    "--info-border": ["info"],
-    "--danger-border-strong": ["danger-strong"],
-    "--warning-border-strong": ["warning-strong"],
-    "--hue-blue-border": ["hue-blue"],
-    "--hue-indigo-border": ["hue-indigo"],
-    "--hue-purple-border": ["hue-purple"],
-    "--hue-teal-border": ["hue-teal"],
-    "--hue-orange-border": ["hue-orange"],
-    "--media-ink": ["media-ink"],
-  },
-  "ring": {
-    "--border": ["subtle"],
-    "--border-strong": ["strong"],
-    "--brand": ["brand"],
-    "--brand-contrast": ["brand-contrast"],
-    "--danger-border-strong": ["danger-strong"],
-    "--warning-border-strong": ["warning-strong"],
-    "--media-ink": ["media-ink"],
-  },
-  "ring-offset": {
-    "--bg-page": ["page"],
-    "--bg-surface": ["surface"],
-    "--media-scrim-hover": ["media-scrim-hover"],
-  },
-  "outline": {
-    "--border": ["subtle"],
-    "--border-strong": ["strong"],
-    "--brand": ["brand"],
-    "--danger-border-strong": ["danger-strong"],
-    "--warning-border-strong": ["warning-strong"],
-  },
-  "fill": {
-    "--text-primary": ["primary"],
-    "--text-secondary": ["secondary"],
-    "--text-muted": ["muted"],
-    "--bg-surface": ["surface"],
-    "--border": ["subtle"],
-    "--border-strong": ["strong"],
-  },
-  "stroke": {
-    "--text-primary": ["primary"],
-    "--text-secondary": ["secondary"],
-    "--text-muted": ["muted"],
-    "--bg-surface": ["surface"],
-    "--border": ["subtle"],
-    "--border-strong": ["strong"],
-  },
-  "ring-width": {
-    "--focus-ring-width": ["focus"],
-  },
-  "outline-width": {
-    "--focus-ring-width": ["focus"],
-  },
-};

package/src/eslint/role-utilities.ts

@@ -1,233 +0,0 @@
-// Which `[var(--token)]` classes have a role utility, and what it is called — the one
-// definition `@eifi1/ui-kit/eslint`'s `role-utilities` rule and `npm run check:tokens`
-// (scripts/check-token-discipline.mjs) both read (round 0.35 R5, Kurvenschmiede's Q6).
-//
-// The table itself is role-utilities.generated.ts, re-rendered from tokens.css's
-// `@theme inline` block by scripts/gen-role-utilities.mjs with the parser below; the
-// check fails while it is stale. So tokens.css stays the source: a role that gains a
-// utility there is flagged in the kit and in the apps from the next release on. The
-// README's role-utility table (scripts/gen-styling-inventory.mjs) reads the block with
-// the same parser since 0.37, so the README, the check and the plugin cannot disagree.
-//
-// This module imports NOTHING, and its TypeScript is erasable only (no enum, no
-// namespace, no parameter property): the check imports it from source under Node's type
-// stripping, before any build, and the plugin ships it to an app's ESLint, which is
-// plain Node with no React and no stylesheet.
-
-/**
- * The properties a role utility exists for, each with the Tailwind theme namespace it is
- * declared in. `ring-width` and `outline-width` are a width typed `length:` —
- * `ring-[length:var(--focus-ring-width)]` is `ring-focus`. `divide-*` and the side borders
- * (`border-t-*`, `border-x-*` …) read `border`'s, as Tailwind does.
- */
-export const ROLE_NAMESPACES = {
-  text: "text-color",
-  bg: "background-color",
-  border: "border-color",
-  ring: "ring-color",
-  "ring-offset": "ring-offset-color",
-  outline: "outline-color",
-  fill: "fill",
-  stroke: "stroke",
-  "ring-width": "ring-width",
-  "outline-width": "outline-width",
-} as const;
-
-export type RoleProperty = keyof typeof ROLE_NAMESPACES;
-
-/**
- * Per property, each token that has a role utility → the utility's name after the
- * property's prefix: `border: { "--border": ["subtle"] }` is `border-subtle`. A list,
- * because nothing stops two names from reading one token; one name is a fix, several a
- * choice.
- */
-export type RoleUtilityTable = {
-  readonly [P in RoleProperty]: Readonly<Record<string, readonly string[]>>;
-};
-
-/** One `--<namespace>-<name>: <value>;` of the `@theme inline` block. */
-export interface ThemeInlineDeclaration {
-  /** The theme variable without its dashes: `text-color-muted`, `ring-width-focus`. */
-  variable: string;
-  /** The property whose namespace it is in; `null` for a namespace ROLE_NAMESPACES lacks. */
-  property: RoleProperty | null;
-  /** The name after the namespace, `muted`; `null` with `property`. */
-  name: string | null;
-  /** The utility it makes: `text-muted`, `ring-focus` (a width is `ring-*` like a colour). */
-  utility: string | null;
-  /** The token the value reads, `--text-muted`; `null` unless the value is a bare `var()`. */
-  token: string | null;
-  /** The value as written, trimmed. */
-  value: string;
-}
-
-/** The `@theme inline` block: where it stands, and every declaration in it in block order. */
-export interface ThemeInlineBlock {
-  /** Its first and one-past-last offset in `stripCssComments(css)`, not in `css`. With
-   *  several `@theme inline` blocks, the first one's; `ranges` holds each. */
-  start: number;
-  end: number;
-  ranges: { start: number; end: number }[];
-  /** Every block's, in stylesheet order. */
-  declarations: ThemeInlineDeclaration[];
-}
-
-/** `css` without its comments: a declaration commented out is not one. */
-export function stripCssComments(css: string): string {
-  return css.replace(/\/\*[\s\S]*?\*\//g, "");
-}
-
-/** One `@theme … { … }` block: its first and one-past-last offset, and what its braces hold. */
-export interface ThemeBlock {
-  start: number;
-  end: number;
-  body: string;
-}
-
-/**
- * Every `@theme` block of `css` (comments already stripped) whose prelude names `inline`
- * (`@theme inline`, `@theme inline reference`) — or, with `inline: false`, every other
- * one (`@theme {`, Tailwind's `@theme default {`). Counted brace by brace, so a nested
- * block does not end it early.
- *
- * The one block finder (refactor audit 2026-10-30, DUP-13): this parser read only the
- * first `@theme inline {…}` up to its first `}`, the testing guard's
- * `declaredCustomProperties` every inline block brace-aware, and the README's inventory a
- * plain `@theme {` by a regex of its own — three readers that disagreed on what one block
- * is. It lives here because this module imports nothing, so the scripts load it from
- * source under Node.
- */
-export function themeBlocks(css: string, { inline }: { inline: boolean }): ThemeBlock[] {
-  const blocks: ThemeBlock[] = [];
-  for (const m of css.matchAll(/@theme\b([^{;]*)\{/g)) {
-    if (/\binline\b/.test(m[1]) !== inline) continue;
-    const open = m.index + m[0].length - 1;
-    let depth = 0;
-    let close = open;
-    for (; close < css.length; close++) {
-      if (css[close] === "{") depth++;
-      else if (css[close] === "}" && --depth === 0) break;
-    }
-    blocks.push({ start: m.index, end: close + 1, body: css.slice(open + 1, close) });
-  }
-  return blocks;
-}
-
-const PROPERTIES = Object.keys(ROLE_NAMESPACES) as RoleProperty[];
-
-/**
- * tokens.css's `@theme inline` block (every one, should there be several), read once for
- * every reader: the table below, and the README's (scripts/gen-styling-inventory.mjs),
- * which fails on a `property: null` where the table leaves it out. `null` when there is
- * no such block — the role utilities are gone, or this parser is. No namespace is
- * another's prefix, so the first that fits is the one.
- */
-export function parseThemeInline(css: string): ThemeInlineBlock | null {
-  const blocks = themeBlocks(stripCssComments(css), { inline: true });
-  if (blocks.length === 0) return null;
-  const declarations = [
-    ...blocks
-      .map((b) => b.body)
-      .join("\n")
-      .matchAll(/--([a-z0-9-]+)\s*:\s*([^;]+);/g),
-  ].map(([, variable, raw]) => {
-    const value = raw.trim();
-    const property = PROPERTIES.find((p) => variable.startsWith(`${ROLE_NAMESPACES[p]}-`)) ?? null;
-    const name = property && variable.slice(ROLE_NAMESPACES[property].length + 1);
-    return {
-      variable,
-      property,
-      name,
-      utility: property && `${property.replace(/-width$/, "")}-${name}`,
-      token: /^var\((--[a-z0-9-]+)\)$/.exec(value)?.[1] ?? null,
-      value,
-    };
-  });
-  const ranges = blocks.map(({ start, end }) => ({ start, end }));
-  return { start: ranges[0].start, end: ranges[0].end, ranges, declarations };
-}
-
-/**
- * The table `tokens.css` declares: every `--<namespace>-<name>: var(--<token>)` in its
- * `@theme inline` block, in block order, each property present (in ROLE_NAMESPACES' order)
- * even when it has none. `null` when there is no such block.
- */
-export function roleUtilitiesFromCss(css: string): RoleUtilityTable | null {
-  const block = parseThemeInline(css);
-  if (!block) return null;
-  const table = Object.fromEntries(PROPERTIES.map((p) => [p, {}])) as Record<RoleProperty, Record<string, string[]>>;
-  for (const { property, name, token } of block.declarations) {
-    if (property && name && token) (table[property][token] ??= []).push(name);
-  }
-  return table;
-}
-
-/**
- * `text-[var(--text-muted)]`, `hover:bg-[var(--bg-hover)]`, `border-t-[var(--border)]`,
- * `focus-visible:ring-[var(--brand)]/40`, `ring-offset-[var(--bg-surface)]`,
- * `ring-[length:var(--focus-ring-width)]` — and, since 0.52, Tailwind v4's own spellings
- * of the same: the shorthand `text-(--text-muted)`, a type hint `text-[color:var(--x)]` /
- * `text-(color:--x)` / `ring-(length:--x)`, and the arbitrary property
- * `[color:var(--text-muted)]`, which is `text-muted` (refactor audit 2026-10-30, DOC-1:
- * all of them passed the check and every app's lint). Preceded by a start, a space, a
- * quote or a variant's colon, which the match does not include — nor the variants before
- * it or an opacity or `!` after it — so replacing the match keeps all of them.
- *
- * Groups: 1 the property; then 2 the hint and 3 the token of `-[…var()]`, or 4 and 5 of
- * `-(…)`; or 6 the CSS property and 7 the token of `[property:var()]`.
- */
-const TOKEN = "(--[a-zA-Z0-9-]+)";
-const ROLE_CLASS = new RegExp(
-  "(?<=^|[\\s\"'`:])(?:" +
-    "(text|bg|border(?:-[xytrblse])?|divide|ring-offset|ring|outline|fill|stroke)-" +
-    `(?:\\[(?:(length|color):)?var\\(${TOKEN}\\)\\]|\\((?:(length|color):)?${TOKEN}\\))` +
-    `|\\[(color|background-color|border-color|outline-color|fill|stroke):var\\(${TOKEN}\\)\\]` +
-    ")",
-  "g",
-);
-
-/** The utility prefix an arbitrary property `[property:var(--x)]` is written as. */
-const ARBITRARY_PROPERTY: Record<string, string> = {
-  color: "text",
-  "background-color": "bg",
-  "border-color": "border",
-  "outline-color": "outline",
-  fill: "fill",
-  stroke: "stroke",
-};
-
-export interface RoleClassHit {
-  /** Where the class starts in the scanned text, after its variants. */
-  index: number;
-  /** The class as written, without variants or opacity: `border-t-[var(--border)]`. */
-  found: string;
-  /** The role utilities that paint the same: `["border-t-subtle"]`. */
-  utilities: string[];
-}
-
-/**
- * Every `[var(--token)]` class in `text` (or one of v4's spellings of it, above) whose
- * token has a role utility on that property.
- * A token with no utility there (`text-[var(--chart-1)]`, `--status-*`,
- * `bg-[var(--border)]` as a divider's fill) is not a hit: `[var(--…)]` is right for it.
- */
-export function findRoleClasses(text: string, table: RoleUtilityTable): RoleClassHit[] {
-  // Every form names a token: `var(--x)`, or `(--x)` in v4's shorthand.
-  if (!text.includes("--")) return [];
-  const hits: RoleClassHit[] = [];
-  for (const m of text.matchAll(ROLE_CLASS)) {
-    const [found, utility, bracketHint, bracketToken, parenHint, parenToken, cssProperty, propertyToken] = m;
-    const prop = utility ?? ARBITRARY_PROPERTY[cssProperty];
-    const token = bracketToken ?? parenToken ?? propertyToken;
-    // `border-t` and `divide` read `border`'s names; a `length:` hint reads the widths.
-    const key =
-      (bracketHint ?? parenHint) === "length"
-        ? `${prop}-width`
-        : prop.startsWith("border-") || prop === "divide"
-          ? "border"
-          : prop;
-    const names = (table as Partial<Record<string, Readonly<Record<string, readonly string[]>>>>)[key]?.[token];
-    if (names?.length) hits.push({ index: m.index, found, utilities: names.map((name) => `${prop}-${name}`) });
-  }
-  return hits;
-}

package/src/feedback/feedback-attachment-download.tsx

@@ -1,77 +0,0 @@
-import { useState } from "react";
-import type { MouseEvent } from "react";
-import { Download, FileText } from "lucide-react";
-import { Button } from "../components/button";
-import { toast } from "../components/toast";
-import type { AuthedFetcher } from "../hooks/use-authed-src";
-import { useKitLabels } from "../i18n/kit-labels";
-import { filenameFromDisposition, saveBlob } from "../lib/download";
-import { attachmentName } from "./feedback-record";
-import { DEFAULT_FEEDBACK_DETAIL_LABELS } from "./feedback-row-detail-labels";
-
-// Internal (not in the `./feedback` entry): the row detail's download button and its
-// plain fetch, shared since 0.47 with ./feedback-entry-file's inline entry files.
-
-/** The bytes behind a fetcher's answer — a `Blob` as it is, a `Response` checked. */
-async function toBlob(answer: Blob | Response): Promise<Blob> {
-  if (typeof Response !== "undefined" && answer instanceof Response) {
-    if (!answer.ok) throw new Error(`HTTP ${answer.status}`);
-    return answer.blob();
-  }
-  return answer as Blob;
-}
-
-/** No fetcher: the browser's own `fetch`, with the cookies a same-origin app uses. */
-export const plainFeedbackFetch: AuthedFetcher = (url, { signal }) =>
-  fetch(url, { signal, credentials: "same-origin" });
-
-/**
- * A PDF or a text log (keksdose `FeedbackFileDownload`, dev #578): the bytes sit behind
- * an authenticated route that serves them as `attachment`, never inline (§3.5), so a
- * plain link would be a 401 the moment it was opened. Fetched through the app's
- * fetcher and saved with {@link saveBlob}; a failure says so in a toast.
- *
- * Saved under the name the server gives it when the fetcher answers with a `Response`
- * (server-kit's `content_disposition` puts the uploader's own name in `filename*`), and
- * under the URL's last segment — the name on the button — when it answers with a bare
- * `Blob`, which has no headers to ask. Words: `feedbackDetail.download` /
- * `downloadFailed`.
- */
-export function FeedbackAttachmentDownload({ url, fetcher }: { url: string; fetcher: AuthedFetcher }) {
-  const labels = useKitLabels("feedbackDetail", DEFAULT_FEEDBACK_DETAIL_LABELS);
-  const [busy, setBusy] = useState(false);
-  const name = attachmentName(url);
-  const download = async (event: MouseEvent) => {
-    // The row around the detail may toggle on a click.
-    event.stopPropagation();
-    setBusy(true);
-    try {
-      const answer = await fetcher(url, { signal: new AbortController().signal });
-      const blob = await toBlob(answer);
-      const saveAs =
-        typeof Response !== "undefined" && answer instanceof Response
-          ? filenameFromDisposition(answer.headers.get("Content-Disposition"), name)
-          : name;
-      saveBlob(blob, saveAs);
-    } catch {
-      toast.error(labels.downloadFailed);
-    } finally {
-      setBusy(false);
-    }
-  };
-  return (
-    <Button
-      type="button"
-      variant="secondary"
-      size="sm"
-      pending={busy}
-      onClick={(event) => void download(event)}
-      aria-label={labels.download(name)}
-      className="max-w-full"
-    >
-      <FileText className="size-3.5 shrink-0" aria-hidden />
-      <span className="min-w-0 max-w-[14rem] truncate-until-large">{name}</span>
-      <Download className="size-3.5 shrink-0" aria-hidden />
-    </Button>
-  );
-}

package/src/feedback/feedback-attachment-labels.ts

@@ -1,37 +0,0 @@
-// The English words of ./feedback-attachment, in a module with no component code: the
-// English catalogue (src/i18n/defaults.ts) imports them without the component, so `en`
-// loads as one chunk like the other languages (0.34, KS U8). ./feedback-attachment
-// re-exports them.
-
-import type { FeedbackAttachmentFieldLabels } from "./feedback-attachment";
-
-/**
- * English, in the feedback contract's wording since 0.27.0 (docs/feedback-harmonization.md
- * §4.2: keksdose's `feedback.*` keys are the canon, and the kit owns them now). Three
- * strings moved with it:
- *
- * - `attachmentAdd` "Attach image" → **"Add attachment"**: the field takes a PDF or a text
- *   log as well wherever the app passes `FEEDBACK_ATTACHMENT_ACCEPT` (keksdose feedback
- *   #87), and a button that says "image" over a picker that offers a PDF is wrong in one
- *   of the two places.
- * - `attachmentPaste` → keksdose's longer line, which says WHY one would paste — a region
- *   snip shows one corner rather than the whole page (Steering Design feedback #39).
- * - `attachmentLimit` for one: "remove **it**", as keksdose's `attachments_full_one`;
- *   "remove one" of a single file read oddly.
- */
-export const DEFAULT_FEEDBACK_ATTACHMENT_LABELS: FeedbackAttachmentFieldLabels = {
-  attachmentAdd: "Add attachment",
-  attachmentCapture: "Capture screenshot",
-  attachmentPaste:
-    "…or paste a screenshot straight from the clipboard, so you can show one corner rather than the whole page.",
-  attachmentRemove: "Remove attachment",
-  attachmentList: "Attachments",
-  attachmentScreenshot: "Screenshot",
-  attachmentRemoveFile: (name) => `Remove ${name}`,
-  attachmentLimit: (max) =>
-    max === 1
-      ? "Up to 1 attachment — remove it to add another."
-      : `Up to ${max} attachments — remove one to add another.`,
-  attachmentUploading: "Uploading…",
-  attachmentUploadFailed: (name) => `“${name}” could not be uploaded`,
-};