umbot-ydb-adapter 1.0.0 → 1.0.1

package/README.md

@@ -181,7 +181,7 @@ Yandex Lockbox.
 | `SoundTokens` | `soundToken`         | `idx_platform_path` по `(platform, path)` |
 
 Все колонки — `Utf8`, колонки ключа — `NOT NULL`. Индексы синхронные (`GLOBAL SYNC`): токен, сохранённый через
-`UPSERT`, сразу находится поиском по пути. YDB использует вторичный индекс только при явном `VIEW`, поэтому адаптер
+`INSERT`, сразу находится поиском по пути. YDB использует вторичный индекс только при явном `VIEW`, поэтому адаптер
 сам добавляет `VIEW idx_platform_path` в поиск токена по `(platform, path)`.
 
 Если таблица уже есть, адаптер добавляет недостающие колонки и индексы, но не меняет и не удаляет существующие.
@@ -210,6 +210,10 @@ new YdbAdapter({
 Типы колонок: `Utf8`, `Int64`, `Double`, `Bool`. Значения приводятся к типу колонки: число `42` в колонке `Utf8`
 запишется строкой `'42'`.
 
+Таблицы можно описать и общей для всех адаптеров опцией umbot `appConfig.dbTables` (umbot 3.1.6+): адаптер создаст
+их так же, но все колонки получат тип `Utf8` — в `IDbTableSchema` есть только строковые поля. Если таблица с тем же
+именем есть и в `tables`, используется описание из `tables`.
+
 Для таблиц, не описанных в `tables` (созданных миграцией или вручную), тип значения берётся из правил модели
 (`rules()`), а если правила нет — из самого значения:
 
@@ -255,8 +259,10 @@ const stats = await new UsersData(appContext).query(async (sql) => {
 
 ## Поведение и ограничения
 
-- **Вставка — `UPSERT`.** Повторная запись с тем же ключом перезаписывает строку, поэтому SDK может безопасно
-  повторить запрос после временной ошибки сети.
+- **Вставка — `INSERT`.** `add()` с занятым ключом возвращает `false` и не перезаписывает запись, как у SQL-адаптеров;
+  для изменения записи используйте `update()` или `save()`. SDK повторяет вставку после временной ошибки сети: если
+  первая попытка всё же записала строку, повтор получит «ключ занят», и `add()` вернёт `false` (с ошибкой в логе),
+  хотя запись сохранена.
 - **Запись — в Serializable-транзакции**: так YDB требует для таблиц со вторичными индексами.
 - **`UPDATE` не меняет колонки ключа**: YDB этого не позволяет, а модель umbot передаёт `platform` и в данных, и в
   условии.

package/dist/YdbAdapter.d.ts

@@ -119,7 +119,7 @@ export declare class YdbAdapter extends BaseDbAdapter<IYdbDbInfo> {
      * Сначала выполняется один запрос-проверка (`SELECT ... LIMIT 0` по всем таблицам и индексам).
      * Если схема уже готова — на этом всё: в Cloud Functions метод вызывается на каждом холодном
      * старте, и DDL на каждый старт стоил бы заметного времени.
-     * @param tables Описание таблиц (`DB_TABLES_SCHEMA`)
+     * @param tables Описание таблиц от umbot: `DB_TABLES_SCHEMA` и `appConfig.dbTables`
      * @returns true, если схема готова
      */
     ensureSchema(tables: readonly IDbTableSchema[]): Promise<boolean>;
@@ -135,10 +135,11 @@ export declare class YdbAdapter extends BaseDbAdapter<IYdbDbInfo> {
      */
     _select(selectData: IQuery, where: IQueryData | null, isOne: boolean): Promise<IModelRes>;
     /**
-     * Добавляет запись (UPSERT): повторная запись с тем же ключом перезаписывает строку,
-     * поэтому SDK может безопасно повторить запрос после временной ошибки сети.
+     * Добавляет запись (INSERT). Занятый ключ YDB отклоняет — вернётся false, запись не
+     * перезаписывается (как у SQL-адаптеров). SDK повторяет запрос после временной ошибки сети;
+     * если первая попытка всё же записала строку, повтор получит «ключ занят» и вернёт false.
      * @param insertData Запрос с данными в `data`
-     * @returns true при успехе
+     * @returns true при успехе, false — ключ занят или ошибка базы (в логе)
      */
     _insert(insertData: IQuery): Promise<boolean>;
     /**

package/dist/YdbAdapter.js

@@ -157,13 +157,19 @@ class YdbAdapter extends plugins_1.BaseDbAdapter {
      * Сначала выполняется один запрос-проверка (`SELECT ... LIMIT 0` по всем таблицам и индексам).
      * Если схема уже готова — на этом всё: в Cloud Functions метод вызывается на каждом холодном
      * старте, и DDL на каждый старт стоил бы заметного времени.
-     * @param tables Описание таблиц (`DB_TABLES_SCHEMA`)
+     * @param tables Описание таблиц от umbot: `DB_TABLES_SCHEMA` и `appConfig.dbTables`
      * @returns true, если схема готова
      */
     async ensureSchema(tables) {
         let infos;
         try {
-            infos = [...tables.map(yql_1.tableInfoFromSchema), ...this.#customTables()];
+            // Одноимённая таблица из опции tables (с типами колонок) заменяет описание от umbot
+            // (appConfig.dbTables, все колонки Utf8) — иначе таблица создалась бы с колонками Utf8.
+            const byName = new Map();
+            for (const info of [...tables.map(yql_1.tableInfoFromSchema), ...this.#customTables()]) {
+                byName.set(info.tableName, info);
+            }
+            infos = [...byName.values()];
         }
         catch (error) {
             this._saveLog('ensureSchema: некорректное описание таблицы', error);
@@ -291,14 +297,15 @@ class YdbAdapter extends plugins_1.BaseDbAdapter {
         }
     }
     /**
-     * Добавляет запись (UPSERT): повторная запись с тем же ключом перезаписывает строку,
-     * поэтому SDK может безопасно повторить запрос после временной ошибки сети.
+     * Добавляет запись (INSERT). Занятый ключ YDB отклоняет — вернётся false, запись не
+     * перезаписывается (как у SQL-адаптеров). SDK повторяет запрос после временной ошибки сети;
+     * если первая попытка всё же записала строку, повтор получит «ключ занят» и вернёт false.
      * @param insertData Запрос с данными в `data`
-     * @returns true при успехе
+     * @returns true при успехе, false — ключ занят или ошибка базы (в логе)
      */
     async _insert(insertData) {
         try {
-            const statement = (0, yql_1.buildUpsert)(insertData.tableName, insertData.data ?? {}, this.#context(insertData));
+            const statement = (0, yql_1.buildInsert)(insertData.tableName, insertData.data ?? {}, this.#context(insertData));
             await this.#execute(statement, { write: true });
             return true;
         }

package/dist/yql.d.ts

@@ -124,14 +124,14 @@ export declare function chooseIndex(table: IYdbTableInfo | undefined, where: IQu
  */
 export declare function buildSelect(tableName: string, where: IQueryData | null, isOne: boolean, ctx: IYqlContext): IYqlStatement;
 /**
- * Собирает UPSERT одной записи. Поля со значением undefined пропускаются,
- * null записывается как NULL.
+ * Собирает INSERT одной записи: занятый ключ YDB отклоняет, запись не перезаписывается.
+ * Поля со значением undefined пропускаются, null записывается как NULL.
  * @param tableName Имя таблицы
  * @param data Данные записи
  * @param ctx Контекст запроса
  * @returns Запрос
  */
-export declare function buildUpsert(tableName: string, data: IQueryData, ctx: IYqlContext): IYqlStatement;
+export declare function buildInsert(tableName: string, data: IQueryData, ctx: IYqlContext): IYqlStatement;
 /**
  * Собирает UPDATE. Колонки первичного ключа из SET убираются: YDB не меняет ключ
  * записи, а модель umbot передаёт `uniqueKeys` и в данных, и в условии.

package/dist/yql.js

@@ -7,7 +7,7 @@ exports.quoteTable = quoteTable;
 exports.quoteColumn = quoteColumn;
 exports.chooseIndex = chooseIndex;
 exports.buildSelect = buildSelect;
-exports.buildUpsert = buildUpsert;
+exports.buildInsert = buildInsert;
 exports.buildUpdate = buildUpdate;
 exports.buildDelete = buildDelete;
 exports.buildCreateTable = buildCreateTable;
@@ -397,14 +397,14 @@ function buildSelect(tableName, where, isOne, ctx) {
     return { text, params: params.params };
 }
 /**
- * Собирает UPSERT одной записи. Поля со значением undefined пропускаются,
- * null записывается как NULL.
+ * Собирает INSERT одной записи: занятый ключ YDB отклоняет, запись не перезаписывается.
+ * Поля со значением undefined пропускаются, null записывается как NULL.
  * @param tableName Имя таблицы
  * @param data Данные записи
  * @param ctx Контекст запроса
  * @returns Запрос
  */
-function buildUpsert(tableName, data, ctx) {
+function buildInsert(tableName, data, ctx) {
     const params = new YqlParams();
     const columns = [];
     const values = [];
@@ -424,7 +424,7 @@ function buildUpsert(tableName, data, ctx) {
     if (!columns.length) {
         throw new YqlBuildError(`Нет данных для записи в таблицу "${tableName}".`);
     }
-    const text = `UPSERT INTO ${quoteTable(tableName)} (${columns.join(', ')}) VALUES (${values.join(', ')})`;
+    const text = `INSERT INTO ${quoteTable(tableName)} (${columns.join(', ')}) VALUES (${values.join(', ')})`;
     return { text, params: params.params };
 }
 /**
@@ -478,7 +478,7 @@ function buildDelete(tableName, where, ctx) {
 }
 /**
  * Описание вторичного индекса для CREATE TABLE / ALTER TABLE.
- * Индекс синхронный: запись, только что сохранённая через UPSERT, сразу видна в поиске по индексу.
+ * Индекс синхронный: запись, только что сохранённая через INSERT или UPDATE, сразу видна в поиске по индексу.
  */
 function indexDefinition(index) {
     return `INDEX ${quoteColumn(index.name)} GLOBAL SYNC ON (${index.fields.map(quoteColumn).join(', ')})`;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "umbot-ydb-adapter",
-    "version": "1.0.0",
+    "version": "1.0.1",
     "description": "YDB (Yandex Database) adapter for the umbot framework: storage for Alice skills and chat bots in Yandex Cloud Functions without a separate database server.",
     "keywords": [
         "umbot",
@@ -15,7 +15,8 @@
         "database",
         "db-adapter",
         "typescript",
-        "bot-framework"
+        "bot-framework",
+        "smartapp"
     ],
     "author": {
         "name": "Maxim-M",
@@ -79,7 +80,7 @@
         "ts-jest": "^29.4.9",
         "typescript": "^5.9.3",
         "typescript-eslint": "^8.60.0",
-        "umbot": "^3.1.4"
+        "umbot": "^3.1.6"
     },
     "funding": {
         "type": "individual",