@gravity-ui/querieskit 1.3.0 → 1.3.1

package/plans/queries-navigation-architecture-review.md

@@ -1,217 +0,0 @@
-# Архитектурный анализ `QueriesNavigation` и дочерних компонент
-
-Область анализа: состояние кода после коммита `bcaa6e4c` (анализировалось текущее
-состояние рабочей директории — прямого доступа к git-diff в среде нет).
-
-Затронутые единицы:
-
-- Виджет [`QueriesNavigation`](src/widgets/QueriesNavigation/QueriesNavigation.tsx:51) + хелперы
-  [`createNavigationDetailResolver`](src/widgets/QueriesNavigation/helpers/createNavigationDetailResolver.ts:13),
-  [`createTableDetailConfig`](src/widgets/QueriesNavigation/helpers/createTableDetailConfig.tsx:45),
-  [`createEmptyDetailConfig`](src/widgets/QueriesNavigation/helpers/createEmptyDetailConfig.ts:5).
-- Модули: [`NavigationDetail`](src/modules/NavigationDetail/NavigationDetail.tsx:39),
-  [`NavigationHeader`](src/modules/NavigationHeader/NavigationHeader.tsx:15),
-  [`ClustersList`](src/modules/ClustersList/ClustersList.tsx:23),
-  [`NavigationItemsList`](src/modules/NavigationItemsList/NavigationItemsList.tsx:31),
-  [`NavigationSchema`](src/modules/NavigationSchema/NavigationSchema.tsx:36),
-  [`NavigationPreview`](src/modules/NavigationPreview/NavigationPreview.tsx:34),
-  [`NavigationMeta`](src/modules/NavigationMeta/NavigationMeta.tsx:25),
-  [`NavigationView`](src/modules/NavigationView/NavigationView.tsx:24).
-- Типы: [`src/types/navigation.ts`](src/types/navigation.ts:1).
-
-Общая оценка: архитектура зрелая и последовательная. Уровни `components → modules → widgets`
-выдержаны, generic-типизация сквозная, controlled/uncontrolled паттерн реализован. Ниже —
-конкретные замечания, сгруппированные по 7 пунктам задачи.
-
----
-
-## 1. Соответствие стилю проекта и Gravity UI
-
-Что хорошо:
-
-- Везде используется `bem-cn-lite` с префиксом `qp-*`, компоновка через `Flex`, spacing через
-  `gap`, что совпадает с остальным проектом.
-- Detail-табы построены на `SegmentedRadioGroup`, meta-скелетоны на `Skeleton`, секции на
-  `Disclosure` — всё нативные примитивы Gravity UI, без самописных аналогов.
-
-Замечания:
-
-- **Инлайн-стили вместо SCSS/токенов в продакшн-коде.** В [`buildSchemaColumns`](src/modules/NavigationSchema/helpers/buildSchemaColumns.tsx:22)
-  используется `style={{display:'inline-flex', alignItems:'center', gap:4}}`. По стилю проекта
-  раскладка/отступы задаются через `Flex gap` или `--g-spacing-*` в `.scss`. Инлайн-`gap:4` стоит
-  заменить на `Flex gap` или CSS-класс с токеном. (Инлайн-стили и текстовые заглушки в сторибуках
-  оставляем как есть — для демо это допустимо.)
-- **Пустой `title: ''` в конфиге таба.** В [`createEmptyDetailConfig`](src/widgets/QueriesNavigation/helpers/createEmptyDetailConfig.ts:9)
-  создаётся таб с пустым заголовком — формально валидация i18n не срабатывает (строка не идёт
-  через `t`), но семантически это «фейковый» таб только ради контейнера. См. п.4.
-
-## 2. Размещение новых компонент по уровням
-
-В целом соответствует правилам из [`AGENTS.md`](AGENTS.md:1):
-
-- Списковые/скомпонованные блоки (`ClustersList`, `NavigationItemsList`, `NavigationDetail`,
-  `NavigationSchema/Preview/Meta/View`) корректно лежат в `modules`.
-- Внутренние части (`ClusterRow`, `NavigationItemRow`, `NavigationDetailTabs`,
-  `NavigationViewSectionItem`) — в `internal/`, что верно.
-- Хелперы конфигов (`createTableDetailConfig` и пр.) — на уровне виджета, т.к. содержат
-  widget-specific i18n. Это оправданно.
-
-Замечания:
-
-- **`ClusterRow` и `NavigationItemRow` реэкспортируются как публичные из
-  [`src/modules/index.ts`](src/modules/index.ts:17), но физически лежат в `internal/`.**
-  Противоречие: либо это публичный API (тогда вынести из `internal/` на уровень модуля или в
-  `components`, т.к. они атомарны и переиспользуются в сторибуке `CustomRows`), либо это внутренняя
-  деталь (тогда убрать из barrel). Судя по использованию в кастомном рендере — это публичный
-  контракт, и их правильнее поднять до `components` (у них стабильный props-контракт, отдельное
-  использование есть).
-- **`NavigationDetailTabs` в `internal/` — ок**, но у него нет `index.ts` и он импортируется по
-  прямому пути. По правилу «каждая единица — папка с `index.ts`» это допустимо для internal, но
-  стоит свериться на консистентность с другими internal-частями.
-- **`NavigationDetail` не экспортирует `NavigationDetailConfig`-фабрики** из своего `index.ts`,
-  тогда как связанные типы живут в `types/navigation`. Это нормально, но стоит проверить, что
-  потребитель может собрать конфиг детали без импортов из `widgets`.
-
-## 3. Кандидаты на вынос в хелперы
-
-- **`visibleActions`-рендер дублируется.** Логика «отфильтровать по `hidden` → отрисовать список
-  `Button view=flat size=s` с `title/aria-label/disabled/qa`» повторяется в
-  [`NavigationHeader`](src/modules/NavigationHeader/NavigationHeader.tsx:22) и
-  [`NavigationViewSectionItem`](src/modules/NavigationView/internal/NavigationViewSectionItem.tsx:29).
-  Кандидат на общий компонент `ActionButtons`/`NavigationActions` (в `components`) с generic-типом
-  экшена, чтобы `NavigationHeaderAction` и `NavigationViewSectionAction` покрывались одним рендером.
-- **Единый тип «action».** См. п.6 — `NavigationHeaderAction` и `NavigationViewSectionAction`
-  отличаются только сигнатурой `onClick`. Можно ввести generic `NavigationAction<TCtx>`.
-- **`getInitialTab` / фильтр видимых табов.** В [`NavigationDetail`](src/modules/NavigationDetail/NavigationDetail.tsx:30)
-  `config.tabs.filter(t => !t.hidden)` считается дважды (в `getInitialTab` и в `useMemo`). Можно
-  вынести хелпер `getVisibleTabs`/`resolveInitialTab` в `NavigationDetail/helpers/`.
-- **SKELETON-заглушка.** Блок «N `Skeleton` в колонку» полностью повторяется в
-  [`NavigationMeta`](src/modules/NavigationMeta/NavigationMeta.tsx:47) и
-  [`NavigationView`](src/modules/NavigationView/NavigationView.tsx:40) (одна и та же
-  `SKELETON_ROWS_COUNT = 4`). Кандидат на общий `ListSkeleton`/`SkeletonRows` в `components`.
-
-## 4. Удобство использования, гибкость переопределения и типизация
-
-Сильные стороны:
-
-- Сквозные дженерики `<TItem, TCluster>`, `<TColumn>`, `<TRow>` дают типобезопасное расширение
-  строк/колонок.
-- Паттерн `renderRowItem` + fallback на дефолтный ряд (`ClustersList`, `NavigationItemsList`) —
-  гибкий и предсказуемый.
-- Controlled/uncontrolled в [`NavigationDetail`](src/modules/NavigationDetail/NavigationDetail.tsx:51)
-  реализован аккуратно (search и activeTab независимо).
-- `view`-проп у detail-модулей (`tableColumns`/`extraColumns`/`render`) — хороший механизм
-  переопределения таблиц.
-
-Замечания:
-
-- **Дженерик теряется в detail-резолверах.** В [`createTableDetailConfig`](src/widgets/QueriesNavigation/helpers/createTableDetailConfig.tsx:19)
-  резолверы типизированы как `NavigationSchemaResolver<T>`, но возвращают `NavigationSchemaConfig`
-  без `<TColumn>`. Из-за этого `extraColumns`-типизация кастомных колонок в конфиге таблицы теряется.
-  Стоит пробросить `TColumn`/`TRow` до `NavigationSchema data`.
-- **`createEmptyDetailConfig` через `React.createElement`.** Файл `.ts`, поэтому используется
-  `createElement` вместо JSX. Логичнее переименовать в `.tsx` и вернуть JSX, а «пустое состояние»
-  моделировать не фейковым табом с `title:''`, а отдельной веткой в `NavigationDetail`
-  (например, `config.empty?: ReactNode` или рендер `EmptyContent`, когда `tabs` пуст).
-- **`[key: string]: unknown` в типах.** `NavigationSchemaColumn` и `NavigationMetaItem` имеют
-  индексную сигнатуру `[key: string]: unknown`. Это ослабляет типизацию расширений (лучше решать
-  через дженерик `T extends ...`, который и так есть). Возможная избыточность — см. п.6.
-- **`resolvedDetailActions = detailActions ?? actions`** в
-  [`QueriesNavigation`](src/widgets/QueriesNavigation/QueriesNavigation.tsx:86): fallback header-экшенов
-  на detail неочевиден. Стоит задокументировать поведение в типе `NavigationDetailPanelConfig`.
-
-## 5. Дублирование кода
-
-- **`buildPreviewColumns` и `buildViewColumns` идентичны.**
-  [`buildPreviewColumns`](src/modules/NavigationPreview/helpers/buildPreviewColumns.tsx:5) и
-  [`buildViewColumns`](src/modules/NavigationView/helpers/buildViewColumns.tsx:5) — побайтово
-  одинаковая функция (маппинг `columns → Column` с заглушкой `value_empty`). Разнесены только по
-  разным i18n. Кандидат на общий хелпер (например, `buildStringColumns` в `src/helpers/` или
-  `components`), принимающий функцию перевода/`emptyText`.
-- **Логика «пустого значения» повторяется трижды.** `value === undefined || null || ''` есть в
-  `buildPreviewColumns`, `buildViewColumns`, [`buildMetaGroups`](src/modules/NavigationMeta/helpers/buildMetaGroups.tsx:15)
-  (`isEmptyValue`). Вынести единый `isEmptyValue` в `src/helpers/`.
-- **Скелетоны и action-рендер** — дублирование описано в п.3.
-- **Блок `errorContent → <Text color="danger">`** повторяется в Schema/Preview/Meta/View. Можно
-  ввести общий `DetailError`/использовать существующий паттерн вывода ошибки.
-
-## 6. Оценка новых типов (избыточность/дублирование)
-
-Файл [`src/types/navigation.ts`](src/types/navigation.ts:1) большой и в целом хорошо
-структурирован, но есть избыточность:
-
-- **`NavigationViewRow = NavigationPreviewRow`** ([`строка 109`](src/types/navigation.ts:109)) — псевдоним.
-  А `NavigationViewConfig`/`NavigationPreviewConfig` очень близки (secions vs rows). Стоит оценить,
-  не свести ли preview к частному случаю view, либо явно задокументировать, что это осознанно разные
-  сущности.
-- **`NavigationHeaderAction` vs `NavigationViewSectionAction`** — отличаются только типом аргумента
-  `onClick` (`NavigationLocation` против `NavigationViewSection`). Кандидат на generic
-  `NavigationAction<TArg>` c общими полями `id/title/content/hidden/disabled/qa`.
-- **Четыре почти одинаковых `*Config`** (`Schema/Preview/Meta/View`) с общими полями
-  `loading/loaded/errorContent`. Можно ввести базовый `NavigationAsyncConfig` и расширять его
-  (`& {columns}`, `& {rows}`, `& {groups}`, `& {sections}`).
-- **`ResolveNavigationDetail` vs `NavigationDetailConfigFactory`** — обе `(item) => Config`,
-  различие только в `| undefined`. Возможно, достаточно одного типа с опциональностью на месте
-  использования.
-- **`NavigationPreviewCell = ReactNode` и `NavigationMetaValue = ReactNode`** — псевдонимы одного
-  и того же. Либо один общий `NavigationCellValue`, либо убрать псевдонимы.
-- **Индексные сигнатуры `[key: string]: unknown`** в `NavigationSchemaColumn`/`NavigationMetaItem`
-  дублируют возможности дженериков — стоит выбрать один механизм расширения.
-
-## 7. Оценка сторибуков (избыточность, объединение)
-
-Наблюдения:
-
-- **Повторяющиеся `Loading/Empty/Error`-стори у 4 detail-модулей.** У Schema, Preview, Meta, View
-  практически идентичные state-стори. Их можно оставить (они полезны для autodocs), но mock-данные
-  дублируются между сторибуками и [`QueriesNavigation.stories`](src/widgets/QueriesNavigation/QueriesNavigation.stories.tsx:148)
-  (`TABLE_SCHEMA_COLUMNS`, `TABLE_PREVIEW_ROWS`, `TABLE_META_GROUPS`, `TABLE_VIEW_SECTIONS`
-  повторяют данные модульных сторибуков). Кандидат на общий `story/mockData.ts`
-  (как уже сделано в `DashboardCharts/story/mockData.ts` и `ChartEditor/story/mockData.ts`).
-- **`CustomColumns`-стори у Schema и Preview идентичны** (тот же `lock`-паттерн с `Label`+`LockIcon`).
-  Можно объединить логически или вынести общий пример.
-- **Кандидат на объединение в одно демо.** Отдельные `Loading`/`Empty`/`Error` можно объединить в
-  одну «States»-стори через controls/args-матрицу, оставив `Default` и `Custom*` отдельными. Это
-  сократит число стори без потери покрытия.
-- **Несогласованность с гайдом структуры сторибуков.** Часть модулей (`ChartEditor`,
-  `DashboardCharts`) держат стори в подпапке `story/` с `mockData.ts`, а Navigation-модули — рядом
-  с компонентом. Стоит привести к единому подходу (вероятно `story/` + вынесенные моки).
-- **Мелочь:** опечатки в mock-данных `QueriesNavigation.stories` (`tesdting`, `Prestable`) —
-  косметика, но заметна в autodocs.
-
----
-
-## Приоритизация правок (для последующей реализации в Code mode)
-
-Высокий приоритет (дублирование/типобезопасность):
-
-1. Объединить [`buildPreviewColumns`](src/modules/NavigationPreview/helpers/buildPreviewColumns.tsx:5)
-   и [`buildViewColumns`](src/modules/NavigationView/helpers/buildViewColumns.tsx:5) в один хелпер.
-2. Вынести общий `isEmptyValue` в `src/helpers/`.
-3. Пробросить дженерики `TColumn`/`TRow` в резолверах
-   [`createTableDetailConfig`](src/widgets/QueriesNavigation/helpers/createTableDetailConfig.tsx:45).
-4. Решить статус `ClusterRow`/`NavigationItemRow`: поднять в `components` либо убрать из
-   публичного barrel.
-
-Средний приоритет (переиспользование/типы):
-
-5. Ввести generic `NavigationAction<TArg>` и общий рендер экшенов
-   (`NavigationHeader` + `NavigationViewSectionItem`).
-6. Ввести базовый `NavigationAsyncConfig` (`loading/loaded/errorContent`) и общий `SkeletonRows`.
-7. Убрать псевдонимы-дубликаты типов (`NavigationPreviewCell`/`NavigationMetaValue`,
-   `NavigationViewRow`) и пересмотреть индексные сигнатуры.
-
-Низкий приоритет (стиль/сторибуки):
-
-8. Вынести mock-данные detail-модулей в общий `story/mockData.ts`, объединить state-стори,
-   унифицировать расположение сторибуков.
-9. Заменить инлайн-стиль `gap:4` в продакшн-хелпере [`buildSchemaColumns`](src/modules/NavigationSchema/helpers/buildSchemaColumns.tsx:22)
-   на `Flex gap` / CSS-класс с токеном (сторибуков не касается).
-10. Переосмыслить `createEmptyDetailConfig` (JSX + отдельная ветка пустого состояния вместо
-    таба с `title:''`).
-
----
-
-Примечание по процессу: анализ выполнен по текущему состоянию файлов. Если нужно строго сверить
-именно diff коммита `bcaa6e4c` (какие строки добавлены/удалены), это удобнее сделать в среде с
-доступом к `git`/`arc` — тогда часть замечаний можно будет привязать к конкретным ханкам.

package/plans/styles-rules.md

@@ -1,137 +0,0 @@
-# Правила владения SCSS-стилями
-
-Подробные правила для агентов, работающих со стилями в `@gravity-ui/querieskit`. Краткая обязательная выжимка находится в разделе [`## Styles`](../AGENTS.md) файла `AGENTS.md`.
-
-## Структура
-
-Каждый компонент, модуль или виджет, которому нужны стили, хранит одноимённый SCSS-файл рядом с файлом реализации и самостоятельно импортирует его:
-
-```text
-ComponentName/
-  ComponentName.tsx
-  ComponentName.scss
-  index.ts
-```
-
-```tsx
-// Child.tsx
-import cn from 'bem-cn-lite';
-
-import './Child.scss';
-
-const block = cn('qp-child');
-
-export type ChildProps = {
-    className?: string;
-};
-
-export const Child = ({className}: ChildProps) => (
-    <div className={block(null, className)}>...</div>
-);
-```
-
-```scss
-// Child.scss
-.qp-child {
-  // Внутренние стили Child.
-}
-```
-
-Если компоненту не нужны собственные стили, создавать пустой SCSS-файл не требуется.
-
-## Flex-раскладка
-
-Для flexbox-раскладки в приоритете компонент `Flex` из `@gravity-ui/uikit`. Если нужное поведение выражается его пропсами, следует задать направление, расстояние между детьми и выравнивание прямо в JSX, не дублируя их через `display: flex` в SCSS:
-
-```tsx
-import {Flex} from '@gravity-ui/uikit';
-
-export const Parent = () => (
-    <Flex direction="column" gap={2} alignItems="center">
-        <Child />
-        <AnotherChild />
-    </Flex>
-);
-```
-
-Не следует без необходимости заменять такую раскладку отдельным классом:
-
-```scss
-// Parent.scss — не нужно, если достаточно пропсов Flex
-.qp-parent {
-  display: flex;
-  flex-direction: column;
-  align-items: center;
-  gap: var(--g-spacing-2);
-}
-```
-
-SCSS остаётся допустимым, когда API `Flex` не может выразить требуемое поведение: например, для псевдоклассов, вложенных селекторов, BEM-модификаторов состояния или отдельных правил внешней геометрии дочернего компонента. В таких случаях продолжают действовать правила владения стилями и передачи `className`.
-
-## Владение стилями
-
-Компонент владеет своим внутренним оформлением: размерами и расположением собственных элементов, цветами, состояниями, внутренними отступами и своим BEM-блоком. Его SCSS импортируется из его же файла реализации, поэтому компонент корректно выглядит независимо от того, через какого родителя он был отрендерен.
-
-Родитель владеет компоновкой своих детей: расположением относительно контейнера и соседних элементов, участием в grid/flex-раскладке и внешней геометрией, необходимой конкретному сценарию. Сначала следует использовать `Flex` и возможности его API, например `gap` и `alignItems`; для grid-раскладки — подходящий контейнер. Если стиль нужно применить непосредственно к корневому элементу ребёнка, родитель передаёт собственный BEM-элемент через `className`:
-
-```tsx
-// Parent.tsx
-import cn from 'bem-cn-lite';
-
-import {Child} from '../Child';
-import './Parent.scss';
-
-const block = cn('qp-parent');
-
-export const Parent = () => <Child className={block('child')} />;
-```
-
-```scss
-// Parent.scss
-.qp-parent {
-  &__child {
-    flex-shrink: 0;
-  }
-}
-```
-
-`Child` обязан объединить переданный `className` со своим корневым классом, а не заменить один класс другим. Текущий пример такого взаимодействия — [`HistorySearchRow`](../src/modules/HistorySearchRow/HistorySearchRow.tsx), который передаёт `className={block('icon')}` в [`QueryStatusIcon`](../src/components/QueryStatusIcon/QueryStatusIcon.tsx), а правило внешней компоновки хранит в [`HistorySearchRow.scss`](../src/modules/HistorySearchRow/HistorySearchRow.scss).
-
-## Запрещённый паттерн
-
-Нельзя складывать собственные стили дочерних компонентов в один SCSS-файл родителя и рассчитывать, что они загрузятся только потому, что родитель импортирует этот файл:
-
-```scss
-// Parent.scss — запрещено
-.qp-child {
-  width: 100%;
-
-  &__title {
-    min-width: 0;
-  }
-}
-
-.qp-another-child {
-  padding: var(--g-spacing-2);
-}
-```
-
-Такой подход связывает внешний вид ребёнка с конкретным родителем: при самостоятельном использовании или переносе ребёнка его стили не загрузятся. Родительский SCSS должен описывать BEM-блок родителя и его элементы, но не BEM-блоки самостоятельных дочерних компонентов.
-
-В текущем коде есть исторические нарушения этого правила:
-
-- [`ClustersList.scss`](../src/modules/ClustersList/ClustersList.scss) объявляет блок `.qp-cluster-row`, принадлежащий [`ClusterRow`](../src/components/ClusterRow/ClusterRow.tsx).
-- [`NavigationItemsList.scss`](../src/modules/NavigationItemsList/NavigationItemsList.scss) объявляет блок `.qp-navigation-item-row`, принадлежащий [`NavigationItemRow`](../src/components/NavigationItemRow/NavigationItemRow.tsx).
-
-Эти места зафиксированы как анти-примеры и не должны использоваться как образец для нового кода. Их исправление выполняется отдельно.
-
-## Чек-лист
-
-1. У каждого компонента со своими стилями есть одноимённый SCSS-файл рядом с `.tsx`.
-2. Компонент самостоятельно импортирует свой SCSS-файл.
-3. SCSS компонента объявляет его собственный BEM-блок, а не блоки самостоятельных дочерних компонентов.
-4. Внутреннее оформление ребёнка остаётся у ребёнка; родитель описывает только компоновку своего сценария.
-5. Flexbox-раскладка сначала выражается через `Flex` из `@gravity-ui/uikit`; `display: flex` в SCSS используется только при недостаточности его API.
-6. Для расстояний между детьми сначала используется `gap` компонента `Flex` или другое свойство подходящего контейнера.
-7. Если родителю нужен класс на корневом элементе ребёнка, ребёнок принимает `className` и объединяет его со своим BEM-классом.
-8. Переданный родителем класс является элементом BEM-блока родителя, например `qp-parent__child`.

package/skills-lock.json

@@ -1,11 +0,0 @@
-{
-  "version": 1,
-  "skills": {
-    "gravity-ui": {
-      "source": "gravity-ui/skills",
-      "sourceType": "github",
-      "skillPath": "skills/gravity-ui/SKILL.md",
-      "computedHash": "e2ecf858568e39c22fd2bb35b7b34721e79a61ac47b45134d11db2ce937d8115"
-    }
-  }
-}

package/src/components/AddChartButton/AddChartButton.scss

@@ -1,12 +0,0 @@
-.qp-add-chart-button {
-    flex-shrink: 0;
-
-    &__menu {
-        inline-size: 360px;
-        max-inline-size: calc(100vw - var(--g-spacing-8));
-    }
-
-    &__option-icon {
-        color: var(--g-color-text-brand);
-    }
-}

package/src/components/AddChartButton/AddChartButton.stories.tsx

@@ -1,34 +0,0 @@
-import type {Meta, StoryObj} from '@storybook/react';
-import {action} from 'storybook/actions';
-
-import {AddChartButton} from './AddChartButton';
-
-const meta: Meta<typeof AddChartButton> = {
-    title: 'Components/AddChartButton',
-    component: AddChartButton,
-    tags: ['autodocs'],
-    parameters: {
-        layout: 'padded',
-    },
-    args: {
-        onSelect: action('onSelect'),
-        onOpenToggle: action('onOpenToggle'),
-    },
-};
-
-export default meta;
-type Story = StoryObj<typeof AddChartButton>;
-
-export const Default: Story = {};
-
-export const Opened: Story = {
-    args: {
-        open: true,
-    },
-};
-
-export const CustomText: Story = {
-    args: {
-        text: 'Add visualization',
-    },
-};

package/src/components/AddChartButton/AddChartButton.tsx

@@ -1,139 +0,0 @@
-import React, {type ReactNode, useMemo, useState} from 'react';
-import {
-    ChartAreaStacked,
-    ChartBar,
-    ChartColumn,
-    ChartLine,
-    ChartPie,
-    Circles5Random,
-    Plus,
-    SquareBars,
-} from '@gravity-ui/icons';
-import {
-    Button,
-    type ButtonButtonProps,
-    DropdownMenu,
-    type DropdownMenuProps,
-    Icon,
-    type IconData,
-} from '@gravity-ui/uikit';
-import cn from 'bem-cn-lite';
-
-import './AddChartButton.scss';
-
-const block = cn('qp-add-chart-button');
-
-export type AddChartButtonOption<TValue extends string = string> = {
-    value: TValue;
-    text: ReactNode;
-    icon?: IconData;
-    disabled?: boolean;
-    hidden?: boolean;
-};
-
-export type DefaultAddChartButtonValue =
-    'Area' | 'BarX' | 'BarY' | 'Line' | 'Pie' | 'Scatter' | 'Waterfall';
-
-export const DEFAULT_ADD_CHART_BUTTON_OPTIONS: readonly AddChartButtonOption<DefaultAddChartButtonValue>[] =
-    [
-        {value: 'Area', text: 'Area', icon: ChartAreaStacked},
-        {value: 'BarX', text: 'BarX', icon: ChartBar},
-        {value: 'BarY', text: 'BarY', icon: ChartColumn},
-        {value: 'Line', text: 'Line', icon: ChartLine},
-        {value: 'Pie', text: 'Pie', icon: ChartPie},
-        {value: 'Scatter', text: 'Scatter', icon: Circles5Random},
-        {value: 'Waterfall', text: 'Waterfall', icon: SquareBars},
-    ];
-
-type SwitcherButtonProps = Omit<
-    ButtonButtonProps,
-    'children' | 'className' | 'disabled' | 'onClick' | 'onKeyDown' | 'selected' | 'size' | 'view'
->;
-
-type MenuProps = NonNullable<DropdownMenuProps<unknown>['menuProps']>;
-type PopupProps = NonNullable<DropdownMenuProps<unknown>['popupProps']>;
-
-export type AddChartButtonProps<TValue extends string = DefaultAddChartButtonValue> = {
-    text?: ReactNode;
-    options?: readonly AddChartButtonOption<TValue>[];
-    onSelect: (value: TValue) => void;
-    open?: boolean;
-    onOpenToggle?: (open: boolean) => void;
-    disabled?: boolean;
-    className?: string;
-    buttonProps?: SwitcherButtonProps;
-    menuProps?: MenuProps;
-    popupProps?: PopupProps;
-};
-
-export const AddChartButton = <TValue extends string = DefaultAddChartButtonValue>({
-    text = 'Add chart',
-    options = DEFAULT_ADD_CHART_BUTTON_OPTIONS as readonly AddChartButtonOption<TValue>[],
-    onSelect,
-    open,
-    onOpenToggle,
-    disabled,
-    className,
-    buttonProps,
-    menuProps,
-    popupProps,
-}: AddChartButtonProps<TValue>) => {
-    const [innerOpen, setInnerOpen] = useState(false);
-    const isOpen = open ?? innerOpen;
-
-    const items = useMemo(
-        () =>
-            options.map(({value, text: optionText, icon, disabled: optionDisabled, hidden}) => ({
-                text: optionText,
-                iconStart: icon && <Icon data={icon} size={16} className={block('option-icon')} />,
-                disabled: optionDisabled,
-                hidden,
-                action: () => onSelect(value),
-            })),
-        [onSelect, options],
-    );
-
-    const handleOpenToggle: React.Dispatch<React.SetStateAction<boolean>> = (nextOpen) => {
-        const nextValue = typeof nextOpen === 'function' ? nextOpen(isOpen) : nextOpen;
-
-        if (open === undefined) {
-            setInnerOpen(nextValue);
-        }
-
-        onOpenToggle?.(nextValue);
-    };
-
-    return (
-        <DropdownMenu
-            items={items}
-            open={isOpen}
-            onOpenToggle={handleOpenToggle}
-            disabled={disabled}
-            size="s"
-            switcherWrapperClassName={block(null, className)}
-            renderSwitcher={(switcherProps) => (
-                <Button
-                    {...buttonProps}
-                    {...switcherProps}
-                    type={buttonProps?.type ?? 'button'}
-                    view="flat-action"
-                    size="m"
-                    selected={isOpen}
-                    disabled={disabled}
-                    aria-haspopup="menu"
-                    aria-expanded={isOpen}
-                    className={block('button')}
-                >
-                    <Icon data={Plus} size={16} />
-                    {text}
-                </Button>
-            )}
-            menuProps={{
-                ...menuProps,
-                size: menuProps?.size ?? 's',
-                className: block('menu', menuProps?.className),
-            }}
-            popupProps={{placement: 'bottom-start', ...popupProps}}
-        />
-    );
-};

package/src/components/AddChartButton/index.ts

@@ -1,6 +0,0 @@
-export {AddChartButton, DEFAULT_ADD_CHART_BUTTON_OPTIONS} from './AddChartButton';
-export type {
-    AddChartButtonOption,
-    AddChartButtonProps,
-    DefaultAddChartButtonValue,
-} from './AddChartButton';

package/src/components/AttachmentList/AttachmentList.scss

@@ -1,25 +0,0 @@
-.qp-attachment-list {
-    display: flex;
-    flex-direction: column;
-    min-width: 0;
-    margin: 0;
-    padding: 0;
-    list-style: none;
-
-    &__item {
-        min-width: 0;
-    }
-
-    &__button {
-        --g-button-padding: var(--g-spacing-2);
-
-        min-width: 0;
-        justify-content: flex-start;
-        text-align: start;
-
-        &_active {
-            --g-button-background-color: var(--g-color-base-selection);
-            --g-button-background-color-hover: var(--g-color-base-selection-hover);
-        }
-    }
-}