@yanqirenshi/ihub.srv 0.2.2 → 0.2.4

package/CHANGELOG.md

@@ -4,6 +4,26 @@
 版は [semver](https://semver.org/lang/ja/) で付けます。0.x の間は、API・データのフォーマット・CLI を変えるときに、minor を上げます。
 (リポジトリ全体の更新履歴は、[ルートの CHANGELOG.md](https://github.com/yanqirenshi/iHub/blob/main/CHANGELOG.md) にあります)
 
+## 0.2.4 (2026-11)
+
+- コードの変更は無い。(`@yanqirenshi/ihub.core` 0.2.4 と、版をそろえるだけ)
+
+## 0.2.3 (2026-11)
+
+### 修正
+
+- `ihub-srv migrate-er` が、カラムのインスタンスの名前を、落とさないようにした。(yanqirenshi/iHub#100)
+  - これまでは、インスタンス (`column_instances.json`) の `physical_name`・`description` を捨て、マスタの名前を、COLUMN の `{ logical: name, physical: code }` にしていた。(例: `id` が `USER-ID` になる。同じ COLUMN を違う名前で 2 回使うテーブルは、どちらも同じ名前になる)
+  - いまは、インスタンスの名前 (`logical_name`・`physical_name`・`description`) と `data_type` が COLUMN と違うときは、名前の違うインスタンスごとに、別のマスタ (identifiers / attributes) を作って指す。
+    - COLUMN と同じ名前 (または名前が無い) インスタンスは、これまでどおり、COLUMN の 1 つのマスタを指す。
+    - 同じ COLUMN で同じ名前のインスタンスは、1 つのマスタを指す。
+    - 最初に現れた名前 (インスタンスの `_id` の順) のマスタが、COLUMN の `_id` を引き継ぎ、2 つ目からのマスタには、ER のどの `_id` とも重ならない値を振る。(何度変換しても、同じ結果になる)
+    - インスタンスの、写し先の決まっていない項目 (アプリが足した項目) は、`entities[].identifiers[]` / `attributes[]` の行に、そのまま写す。
+  - 変換のあとに、マスタの数 (COLUMN から作った数・名前の違いで増えた数) を表示する。
+  - 描画 (d3.er・model.er) は、変わらない。
+  - **0.2.2 以前の `migrate-er` で変換したアプリ**は、元のフラット形式のデータ (git の履歴) から、0.2.3 以降の `migrate-er` で変換し直すと、名前が戻る。
+- 確かめた: iwo.Studio (インスタンス 132 件) と UkiyoSnow (202 件) の、変換前のフラット形式のデータで、すべてのインスタンスの名前が、変換のあとのマスタの名前と一致した。(`ihub-srv check` も通る)
+
 ## 0.2.2 (2026-11)
 
 - コードの変更は無い。(`@yanqirenshi/ihub.core` 0.2.2 と、版をそろえるだけ)

package/README.md

@@ -114,6 +114,9 @@ ihub-srv migrate-er data
 
 フラット形式の項目のうち、階層形式に写し先が決まっていないもの (TABLE の `color` など、アプリが足した項目) は、そのまま entity (COLUMN は identifiers / attributes) に写します。
 
+カラムのインスタンスの名前 (`logical_name`・`physical_name`・`description`) と `data_type` は、COLUMN と違うときは、名前の違うインスタンスごとに、別のマスタ (identifiers / attributes) を作って指します。(階層形式では、名前はマスタだけが持つため。同じ COLUMN で同じ名前のインスタンスは、1 つのマスタを指す。最初の名前のマスタが COLUMN の `_id` を引き継ぎ、2 つ目からは、ER のどの `_id` とも重ならない値を振る) 変換のあとに、マスタの数 (COLUMN から作った数・名前の違いで増えた数) を表示します。
+0.2.2 以前の `migrate-er` は、インスタンスの名前を、COLUMN の `name`・`code` に替えていました。0.2.3 以降で、元のフラット形式のデータから変換し直すと、名前が戻ります。
+
 ## createApp
 
 ```js

package/bin/ihub-srv.mjs

@@ -13,7 +13,7 @@ import { parseArgs } from 'node:util';
 import { createApp, serve } from '../src/index.mjs';
 import { createValidation } from '../src/validation/index.mjs';
 import { convertLegacy } from '../src/tools/convert-legacy-data.mjs';
-import { migrateDir } from '../src/tools/migrate-er-hierarchical.mjs';
+import { migrateDir, summary } from '../src/tools/migrate-er-hierarchical.mjs';
 
 const USAGE = `使い方:
   ihub-srv [--data <dir>] [--static <dir>] [--port <n>] [--app-name <name>]
@@ -317,11 +317,9 @@ function migrateEr (args) {
     }
 
     try {
-        const data = migrateDir(resolve(args[0]));
+        const { data, stats } = migrateDir(resolve(args[0]));
 
-        console.log(`${args[0]}/er を階層形式にしました。` +
-                    ` (identifiers ${data.identifiers.length}, attributes ${data.attributes.length},` +
-                    ` entities ${data.entities.length}, relationships ${data.relationships.length}。check: 問題なし)`);
+        console.log(`${args[0]}/er を階層形式にしました。 (${summary(data, stats)}。check: 問題なし)`);
 
         return 0;
     } catch (e) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yanqirenshi/ihub.srv",
-  "version": "0.2.2",
+  "version": "0.2.4",
   "description": "iHub の API サーバー。data/ の JSON を、スキーマとルールで検証しながら読み書きする。",
   "keywords": [
     "ihub",

package/src/tools/migrate-er-hierarchical.mjs

@@ -16,7 +16,15 @@
 // 変換の規則:
 //   COLUMN            → identifiers / attributes
 //                       (column_type が IDENTIFIER のインスタンスが参照している COLUMN を identifiers に、それ以外を attributes に)
-//                       name は { logical: name, physical: code }
+//                       name は { logical: name, physical: code }。
+//                       ただし、インスタンスが持つ名前 (logical_name・physical_name・description) と data_type が COLUMN と違うときは、
+//                       その名前の組ごとに、別のマスタを作る。(階層形式では、名前はマスタだけが持ち、インスタンスはマスタを指すだけのため)
+//                       - 同じ COLUMN の、同じ名前のインスタンスは、1 つのマスタを指す
+//                       - COLUMN で最初に現れた名前 (インスタンスの _id の順) のマスタが、COLUMN の _id を引き継ぐ
+//                         (どのインスタンスも COLUMN と同じ名前なら、いままでどおり COLUMN の _id・name・code のマスタ 1 つ)
+//                       - 2 つ目からの名前のマスタには、ER のデータの中のどの _id とも重ならない id を、_id の順に振る (何度変換しても同じ)
+//                       - 識別子 / 属性の振り分けは COLUMN ごと。(同じ COLUMN のマスタは、同じ側に入る)
+//                       - インスタンスの、写し先の決まっていない項目 (アプリが足した項目) は、entities[].identifiers[] / attributes[] の行に写す
 //   TABLE             → entities (position / size はそのまま)
 //                       identifiers[] / attributes[] は、TABLE→COLUMN-INSTANCE の行と、COLUMN→COLUMN-INSTANCE (INSTANCE-OF) の行から組む
 //                       INSTANCE-OF の行が無いインスタンスは、code の同じ COLUMN を参照する
@@ -36,6 +44,11 @@ const HIERARCHICAL_FILES = ['schemas', 'identifiers', 'attributes', 'entities',
 
 // フラット形式のコレクション ({ entities, columns, column_instances, ports, relationships }) を、階層形式にする。
 export function toHierarchical (flat) {
+    return convert(flat).data;
+}
+
+// 階層形式にして、マスタの数 ({ columns: COLUMN の数, masters: 作ったマスタの数 }) も返す。
+export function convert (flat) {
     const { entities: tables, columns, column_instances: instances, ports, relationships: edges } = flat;
 
     const byId = (list)=> new Map(list.map(d=> [d._id, d]));
@@ -79,22 +92,62 @@ export function toHierarchical (flat) {
     const TABLE_KNOWN = ['_id', '_class', 'code', 'name', 'description', 'position', 'size'];
     const rest = (d, known)=> Object.fromEntries(Object.entries(d).filter(([k])=> !known.includes(k)));
 
-    const master = (column)=> {
-        const out = { id: column._id, name: { logical: column.name, physical: column.code } };
+    // インスタンスの項目のうち、マスタか、行の id・参照に写し先が決まっているもの。これ以外の項目 (アプリが足した項目) は、行にそのまま写す。
+    const INSTANCE_KNOWN = ['_id', '_class', 'code', 'physical_name', 'logical_name', 'description', 'column_type', 'data_type'];
+
+    const sorted = (list)=> [...list].sort((a, b)=> a._id - b._id);
+
+    // マスタの名前の組。(インスタンスの値が無い・空のときは、COLUMN の値)
+    const names_of = (instance, column)=> ({
+        logical: instance.logical_name || column.name,
+        physical: instance.physical_name || column.code,
+        description: instance.description || column.description || '',
+        data_type: instance.data_type || column.data_type || '',
+    });
+
+    // COLUMN ごとの、名前の組 → マスタの id。(インスタンスの _id の順に、最初に現れた名前が、COLUMN の _id を引き継ぐ)
+    const masters_of = new Map(columns.map(c=> [c._id, new Map()]));
+    const master_of = new Map();    // インスタンス → マスタの id
+    const column_by_id = byId(columns);
+
+    // 2 つ目からの名前のマスタの id。ER のデータの中のどの _id とも重ならない値から、順に振る。
+    let next_id = Math.max(0, ...[tables, columns, instances, ports, edges].flat().map(d=> d._id).filter(Number.isFinite)) + 1;
+
+    for (const instance of sorted(instances)) {
+        const column = column_by_id.get(column_of.get(instance._id));
+        const names = names_of(instance, column);
+        const key = JSON.stringify([names.logical, names.physical, names.description, names.data_type]);
+        const known = masters_of.get(column._id);
+
+        if (!known.has(key))
+            known.set(key, { id: known.size===0 ? column._id : next_id++, names });
 
-        if (column.description)
-            out.description = column.description;
+        master_of.set(instance._id, known.get(key).id);
+    }
+
+    const master = (column, id, names)=> {
+        const out = { id, name: { logical: names.logical, physical: names.physical } };
+
+        if (names.description)
+            out.description = names.description;
 
-        if (column.data_type)
-            out.data_type = column.data_type;
+        if (names.data_type)
+            out.data_type = names.data_type;
 
         return { ...out, ...rest(column, COLUMN_KNOWN) };
     };
 
-    const sorted = (list)=> [...list].sort((a, b)=> a._id - b._id);
+    // COLUMN ごとのマスタ。(インスタンスが 1 つも無い COLUMN は、COLUMN の名前のマスタ 1 つ)
+    const masters_for = (column)=> {
+        const known = [...masters_of.get(column._id).values()];
+
+        return known.length > 0
+            ? known.map(m=> master(column, m.id, m.names))
+            : [master(column, column._id, { logical: column.name, physical: column.code, description: column.description || '', data_type: column.data_type || '' })];
+    };
 
-    const identifiers = sorted(columns.filter(c=> identifier_columns.has(c._id))).map(master);
-    const attributes = sorted(columns.filter(c=> !identifier_columns.has(c._id))).map(master);
+    const identifiers = sorted(columns.filter(c=> identifier_columns.has(c._id))).flatMap(masters_for);
+    const attributes = sorted(columns.filter(c=> !identifier_columns.has(c._id))).flatMap(masters_for);
 
     const entities = tables.map(table=> {
         const mine = sorted(instances.filter(d=> table_of.get(d._id)===table._id));
@@ -108,9 +161,9 @@ export function toHierarchical (flat) {
             size: table.size,
             ...rest(table, TABLE_KNOWN),
             identifiers: mine.filter(d=> identifier_columns.has(column_of.get(d._id)))
-                .map(d=> ({ id: d._id, identifier: column_of.get(d._id) })),
+                .map(d=> ({ id: d._id, identifier: master_of.get(d._id), ...rest(d, INSTANCE_KNOWN) })),
             attributes: mine.filter(d=> !identifier_columns.has(column_of.get(d._id)))
-                .map(d=> ({ id: d._id, attribute: column_of.get(d._id) })),
+                .map(d=> ({ id: d._id, attribute: master_of.get(d._id), ...rest(d, INSTANCE_KNOWN) })),
         };
     });
 
@@ -159,7 +212,19 @@ export function toHierarchical (flat) {
               return out;
           });
 
-    return { schemas: [], identifiers, attributes, entities, relationships };
+    const data = { schemas: [], identifiers, attributes, entities, relationships };
+
+    return { data, stats: { columns: columns.length, masters: identifiers.length + attributes.length } };
+}
+
+// 変換した結果の説明。(マスタの数と、名前の違いで増えた数)
+export function summary (data, stats) {
+    const added = stats.masters - stats.columns;
+
+    return `identifiers ${data.identifiers.length}, attributes ${data.attributes.length},` +
+        ` entities ${data.entities.length}, relationships ${data.relationships.length}。` +
+        `マスタは、COLUMN から ${stats.columns} 件、名前の違いで増えたもの ${added} 件` +
+        ` (COLUMN と名前の違うインスタンスごとに、別のマスタを作ります)`;
 }
 
 // API サーバーと同じ書き方 (4 つの空白で字下げし、最後に改行) で書く。
@@ -174,7 +239,7 @@ export function migrateDir (dir) {
         throw new Error(`${er}: フラット形式ではありません。(columns.json がありません。変換済みです)`);
 
     const flat = Object.fromEntries(FLAT_FILES.map(name=> [name, JSON.parse(readFileSync(join(er, `${name}.json`), 'utf8'))]));
-    const data = toHierarchical(flat);
+    const { data, stats } = convert(flat);
 
     const errors = check(data, 'er/');
 
@@ -188,7 +253,7 @@ export function migrateDir (dir) {
         if (!HIERARCHICAL_FILES.includes(name))
             unlinkSync(join(er, `${name}.json`));
 
-    return data;
+    return { data, stats };
 }
 
 if (process.argv[1] && resolve(process.argv[1])===fileURLToPath(import.meta.url)) {
@@ -200,11 +265,9 @@ if (process.argv[1] && resolve(process.argv[1])===fileURLToPath(import.meta.url)
     }
 
     try {
-        const data = migrateDir(resolve(dir));
+        const { data, stats } = migrateDir(resolve(dir));
 
-        console.log(`${dir}/er を階層形式にしました。` +
-                    ` (identifiers ${data.identifiers.length}, attributes ${data.attributes.length},` +
-                    ` entities ${data.entities.length}, relationships ${data.relationships.length}。check: 問題なし)`);
+        console.log(`${dir}/er を階層形式にしました。 (${summary(data, stats)}。check: 問題なし)`);
     } catch (e) {
         console.error(e.message);
         process.exit(1);