@rex0220/formula-authoring-tools 0.1.0 → 0.2.1

package/README.md

@@ -27,9 +27,10 @@ Projects/
 - `src/connections.ts` / `src/workspace.ts` / `src/permission.ts` / `src/commit-file.ts` … kintone の接続のファイル（kSQL の `ksql.config.json` と同じ形。profile、`tokenMap`、`userApiToken`）、`kintone/<profile>/<番号>-<アプリ名>/` のフォルダーと印 `.formula-app.json`、kintone/ の下を変える許可、ファイルの確定とロック（print-craft-authoring tools 2.0.0 の写し）
 - `src/env.ts` / `src/kintone-rest.ts` / `src/kintone-url.ts` … `.env`（kintone 公式 MCP と同じ `KINTONE_*` + `FORMULA_PLUGIN_ZIP` / `FORMULA_KINTONE_CONFIG` / `FORMULA_USER_API_TOKEN`）、GET 専用・許可 API 固定・送信先固定の REST（kintone REST API と User API）
 - `src/meta.ts` … tools の版、扱う計算式プラグイン（プラグイン ID `FORMULA_PLUGIN_ID`、封筒の `FORMULA_CONFIG_PLUGIN_ID`、版の下限 `MIN_PLUGIN_VERSION`）、API の版（`SUPPORTED_API_VERSIONS`）、API の契約（`REQUIRED_API`）
-- `src/core.ts` … 共通の中核の入口（npm の `@rex0220/formula-authoring-tools/core`。MCP が使う予定）。`scripts/build.mjs` が `dist/core.mjs` と `dist/types/` を作る
+- `src/commands/save.ts` … 保存の約束（formula MCP の `fml_save_settings` / `fml_update_fields` の本体。CLI は使わない）。新しい設定は同じ名前があれば保存しない、既存の設定は `fml_summary` の digest を照合して項目の行（`fieldcode`）を複数まとめて差し替える。normalize は Worker で行い、エラーがあれば確定しない。確定はロック付き（`commit-file.ts`）
+- `src/core.ts` … 共通の中核の入口（npm の `@rex0220/formula-authoring-tools/core`。formula MCP が使う。`CORE_API_VERSION` 2）。`scripts/build.mjs` が `dist/core.mjs` と `dist/types/` を作る。MCP が使う名前は `test/core-exports.test.mjs` で固定する
 - `scripts/vendor.mjs` … moment 2.24.0 を CDN から `vendor/` に取る（`vendor/moment.json` の SHA-256 と照合）。`scripts/build.mjs` … esbuild で `dist/cli.mjs` / `dist/eval-worker.mjs` / `dist/core.mjs`。`scripts/gen-docs.mjs` … `docs/`（設定ファイル仕様、関数一覧、functions.json、schema-manifest、既定のテストデータ）と `samples/users.json` を zip から機械生成
-- `pending/` … print-craft から写してまだ計算式用にしていないもの（`save.ts` = MCP の保存の約束）。npm には入らない
+- `pending/` … print-craft から写してまだ計算式用にしていないもの（`diff.ts` / `normalize.ts` は計算式用の `src/commands/` が別にあるので使っていない）。npm には入らない
 - `test/` … `node --test`（src の .ts を直接読む。`test/helpers.mjs` が fixture の zip を読む。`test/fixtures/formula/` はテスト用アプリ 2305 の項目定義と設定）
 
 ## 使い方（開発）
@@ -58,4 +59,6 @@ npm pack --dry-run  # 公開前に中身を確かめる（dist、vendor、LICENS
 
 ## 履歴
 
+- 0.2.1（2026-10）: formula MCP の Codex レビューへの対応。`updateFields` の `common` を共通の設定のキー（`COMMON_KEYS`）だけに。Worker の中の誤りの種類（`EvalError.kind`）と名前を中核が instanceof で決める（zip のコードの例外の name / constructor.name / 文を応答や標準エラーに出さない。`classifyEvalError` / `builtinErrorName`）。保存の結果の想定外の誤りも種類だけ（`zipCode`）。CLI の想定外の誤りは種類だけ（`FORMULA_DEBUG=1` で文とスタック）。接続の誤りの文の print-craft の名残を直した
+- 0.2.0（2026-10）: formula MCP のための中核の入口（`CORE_API_VERSION` 2。CLI の動きは変えない）。`core` に workspace / connections / permission / commit-file / take / users の参照コード / JSON の誤りの文 / 行のキーを追加。`src/commands/save.ts`（`saveNewSettings` / `updateFields`）。`runEval` の `workerFile`（Worker の入口の絶対パス）と `captureOutput`（Worker の標準出力・標準エラーを親に流さず受け取る）、`loadEngine` の `vendorDir`（moment の置き場所）。評価の結果に `findings` と calc のレコードごとの行
 - 0.1.0（2026-10）: 最初の公開。version / fields / record / records / pull / summary / diff / users / normalize / calc / take / edit / files。扱う計算式プラグイン Ver.101 以降、authoring API 1

package/dist/cli.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-/*! @rex0220/formula-authoring-tools 0.1.0 (c) rex0220. MIT License. 計算式エンジンと計算式プラグインのコードは利用者の計算式プラグインプラグインの zip から読む（このファイルには含まれない）。commit 69de71e. */
+/*! @rex0220/formula-authoring-tools 0.2.1 (c) rex0220. MIT License. 計算式エンジンと計算式プラグインのコードは利用者の計算式プラグインプラグインの zip から読む（このファイルには含まれない）。commit f2e46ef. */
 
 // src/cli.ts
 import { chmodSync, existsSync as existsSync9, mkdirSync as mkdirSync2, realpathSync as realpathSync5, writeFileSync as writeFileSync3 } from "node:fs";
@@ -355,7 +355,7 @@ function devMeta() {
   return { toolsVersion: pkg.version, pluginId: FORMULA_PLUGIN_ID, minPluginVersion: MIN_PLUGIN_VERSION, supportedApiVersions: [...SUPPORTED_API_VERSIONS], commit: gitCommit(AUTHORING_ROOT), builtAt: "", mode: "dev" };
 }
 function toolsMeta() {
-  if (true) return JSON.parse('{"toolsVersion":"0.1.0","pluginId":"dhgihghfgfmibkdldkeibboabfnmdpke","minPluginVersion":101,"supportedApiVersions":[1],"commit":"69de71e","builtAt":"2026-10-18T00:25:51.113Z","mode":"build"}');
+  if (true) return JSON.parse('{"toolsVersion":"0.2.1","pluginId":"dhgihghfgfmibkdldkeibboabfnmdpke","minPluginVersion":101,"supportedApiVersions":[1],"commit":"f2e46ef","builtAt":"2026-10-18T03:55:45.474Z","mode":"build"}');
   return devMeta();
 }
 
@@ -659,6 +659,9 @@ function summarizeUsers(src, origin, user) {
 }
 
 // src/engine.ts
+function momentFileOf(opt) {
+  return opt.vendorDir ? path6.join(opt.vendorDir, path6.basename(MOMENT_FILE)) : MOMENT_FILE;
+}
 var DEFAULT_CONTEXT_BASE_URL = "https://example.cybozu.com";
 var SENTINEL_USER = { id: "0", code: "__sentinel__", name: "", email: "", timezone: "Asia/Tokyo", language: "ja", isGuest: false };
 var loaded = null;
@@ -758,15 +761,16 @@ async function loadOnce(opt) {
   }
   if (!src.api) throw new PluginZipError(`計算式プラグインの zip に ${API_ENTRY} が無い（Ver.${MIN_PLUGIN_VERSION} 以降の zip が要る）: ${src.from}`, "no-api", known);
   if (!src.decimal) warnings.push(`zip に ${DECIMAL_ENTRY} が無い（べき乗の小数の指数がエラーになる）`);
-  if (!existsSync3(MOMENT_FILE)) throw new Error(`moment が無い: ${MOMENT_FILE}（tools の npm install 時に取る。手で取るなら npm run vendor）`);
+  const momentFile = momentFileOf(opt);
+  if (!existsSync3(momentFile)) throw new Error(`moment が無い: ${momentFile}（tools の npm install 時に取る。手で取るなら npm run vendor）`);
   try {
-    return await startEngine({ ...src, api: src.api }, known, warnings, stamp);
+    return await startEngine({ ...src, api: src.api }, known, warnings, stamp, momentFile);
   } catch (e) {
     poisoned = { code: e instanceof PluginZipError ? e.code : "unexpected", info: e instanceof PluginZipError ? e.info : known };
     throw e;
   }
 }
-async function startEngine(src, known, warnings, stamp) {
+async function startEngine(src, known, warnings, stamp, momentFile) {
   const g = globalThis;
   const { Window } = await import("happy-dom");
   const ctx = {
@@ -842,7 +846,7 @@ async function startEngine(src, known, warnings, stamp) {
       throw new PluginZipError(`計算式プラグインの zip のコード（${entry}）を実行できない: ${e instanceof Error ? e.message : String(e)}`, "engine-unreadable", known);
     }
   };
-  runScript(readFileSync4(MOMENT_FILE, "utf8"), MOMENT_FILE);
+  runScript(readFileSync4(momentFile, "utf8"), momentFile);
   runScript(src.momentTimezone, `${src.from}!${MOMENT_TZ_ENTRY}`, MOMENT_TZ_ENTRY);
   runScript(src.bignumber, `${src.from}!${BIGNUMBER_ENTRY}`, BIGNUMBER_ENTRY);
   if (src.decimal) runScript(src.decimal, `${src.from}!${DECIMAL_ENTRY}`, DECIMAL_ENTRY);
@@ -1397,7 +1401,7 @@ function authFor(set, profile, appId) {
   if (value === void 0) throw new ConnectionError(`profile「${profile.profile}」の tokenMap に APP${appId} が無い（そのアプリの API トークンを足す）`, "no-token");
   if (value.startsWith("env:")) {
     const token = nonblank(held.env[value.slice(4)]);
-    if (!token) throw new ConnectionError(`profile「${profile.profile}」の APP${appId} のトークンの環境変数が無い（設定した後、print-craft を再接続する。Desktop は起動し直す）`, "env-missing");
+    if (!token) throw new ConnectionError(`profile「${profile.profile}」の APP${appId} のトークンの環境変数が無い（環境変数を置いてから実行し直す。MCP なら再接続する: Claude Code は /mcp、Desktop は起動し直す）`, "env-missing");
     return { baseUrl: profile.baseUrl, token };
   }
   return { baseUrl: profile.baseUrl, token: value };
@@ -3075,8 +3079,24 @@ function calcSettings(input) {
 }
 
 // src/worker/eval-run.ts
+function classifyEvalError(e) {
+  const message = String(e?.message ?? e);
+  if (e instanceof InputError) return { kind: "input", name: "InputError", message };
+  if (e instanceof PluginZipError) return { kind: "zip", name: "PluginZipError", message, code: e.code };
+  if (e instanceof UsersDataError) return { kind: "users", name: "UsersDataError", message };
+  if (e instanceof PolicyError) return { kind: "policy", name: "PolicyError", message };
+  return { kind: "other", name: builtinErrorName(e), message };
+}
+function builtinErrorName(e) {
+  if (e instanceof TypeError) return "TypeError";
+  if (e instanceof RangeError) return "RangeError";
+  if (e instanceof ReferenceError) return "ReferenceError";
+  if (e instanceof SyntaxError) return "SyntaxError";
+  if (e instanceof URIError) return "URIError";
+  return "Error";
+}
 async function runEvalJob(job) {
-  const engine = await loadEngine({ pluginZip: job.pluginZip, devPluginDir: job.devPluginDir, mode: job.mode });
+  const engine = await loadEngine({ pluginZip: job.pluginZip, devPluginDir: job.devPluginDir, mode: job.mode, vendorDir: job.vendorDir });
   if (job.baseUrl) engine.setContext({ baseUrl: job.baseUrl });
   const usersInfo = buildUsersInfo(job.usersSource);
   const usersLabel = job.usersOrigin.kind === "file" ? job.usersOrigin.file : "既定のテストデータ（架空）";
@@ -3093,7 +3113,16 @@ async function runEvalJob(job) {
     usersOrigin: job.usersOrigin
   });
   const outcome = {
-    normalize: { summary: res.summary, hasErrors: res.findings.hasErrors, warnings: res.findings.count("warning"), output: res.output, checkDiffs: res.checkDiffs },
+    normalize: {
+      summary: res.summary,
+      hasErrors: res.findings.hasErrors,
+      warnings: res.findings.count("warning"),
+      findings: [...res.findings.items],
+      output: res.output,
+      checkDiffs: res.checkDiffs,
+      appId: res.appId,
+      size: res.size
+    },
     engineWarnings: [...engine.warnings]
   };
   if (job.kind === "calc" && job.calc && !res.findings.hasErrors) {
@@ -3111,7 +3140,7 @@ async function runEvalJob(job) {
       usersInfo,
       usersOrigin: job.usersOrigin
     });
-    outcome.calc = { text: r.text, failed: r.records.some((x) => x.errors.length > 0 || x.failed) };
+    outcome.calc = { text: r.text, failed: r.records.some((x) => x.errors.length > 0 || x.failed), records: r.records, contextLine: r.contextLine, notice: r.notice, summary: r.summary };
   }
   return outcome;
 }
@@ -3124,10 +3153,12 @@ var EvalTimeoutError = class extends Error {
 var EvalError = class extends Error {
   innerName;
   code;
-  constructor(message, innerName, code) {
+  kind;
+  constructor(message, innerName, code, kind = "other") {
     super(message);
     this.innerName = innerName;
     this.code = code;
+    this.kind = kind;
   }
 };
 function workerEntry() {
@@ -3141,10 +3172,26 @@ function workerEntry() {
 }
 async function runEval(job, opt = {}) {
   const timeoutSec = Math.min(Math.max(opt.timeoutSec ?? DEFAULT_TIMEOUT_SEC, 1), MAX_TIMEOUT_SEC);
-  if (opt.inline) return runEvalJob(job);
-  const entry = workerEntry();
+  if (opt.inline) {
+    try {
+      return await runEvalJob(job);
+    } catch (e) {
+      const c = classifyEvalError(e);
+      throw new EvalError(c.message, c.name, c.code, c.kind);
+    }
+  }
+  let entry;
+  if (opt.workerFile !== void 0) {
+    if (!path13.isAbsolute(opt.workerFile) || !existsSync8(opt.workerFile)) throw new Error(`Worker の入口が無い（絶対パスで指定する）: ${opt.workerFile}`);
+    entry = opt.workerFile;
+  } else entry = workerEntry();
+  const capture = opt.captureOutput;
   return new Promise((resolve, reject) => {
-    const worker = new Worker(entry, { workerData: job, stdout: false, stderr: false });
+    const worker = new Worker(entry, { workerData: job, stdout: capture !== void 0, stderr: capture !== void 0 });
+    if (capture) {
+      worker.stdout.on("data", (d) => capture(String(d), "stdout"));
+      worker.stderr.on("data", (d) => capture(String(d), "stderr"));
+    }
     let settled = false;
     const finish = (fn) => {
       if (settled) return;
@@ -3161,10 +3208,10 @@ async function runEval(job, opt = {}) {
     worker.on("message", (m) => {
       finish(() => {
         if (m.ok) resolve(m.outcome);
-        else reject(new EvalError(m.error.message, m.error.name, m.error.code));
+        else reject(new EvalError(m.error.message, m.error.name, m.error.code, m.error.kind ?? "other"));
       });
     });
-    worker.on("error", (e) => finish(() => reject(e)));
+    worker.on("error", (e) => finish(() => reject(new EvalError(e?.message ?? String(e), builtinErrorName(e), void 0, "other"))));
     worker.on("exit", (code) => finish(() => reject(new Error(`評価の Worker が結果を返さずに終わった（exit ${code}）`))));
   });
 }
@@ -3770,13 +3817,18 @@ ${USAGE}`);
       return 1;
     }
     if (e instanceof EvalError) {
-      if (["PluginZipError", "InputError", "UsersDataError", "PolicyError", "AuthError"].includes(e.innerName)) {
+      if (e.kind !== "other") {
         console.error(e.message);
         return 1;
       }
-      throw new Error(`${e.innerName}: ${e.message}`);
+      console.error(`評価の途中で想定外の誤り（${e.innerName}）。誤りの文は値を含みうるので表示しない（FORMULA_DEBUG=1 で表示）。normalize で設定を、version で zip を確かめる`);
+      if (W.env.FORMULA_DEBUG === "1") console.error(e.message);
+      return 1;
     }
-    throw e;
+    const name = builtinErrorName(e);
+    console.error(`想定外の誤り（${name}）。誤りの文は値を含みうるので表示しない（FORMULA_DEBUG=1 で表示）`);
+    if (W.env.FORMULA_DEBUG === "1") console.error(e instanceof Error ? e.stack ?? e.message : String(e));
+    return 1;
   }
 }
 var invokedRaw = process.argv[1] ? path14.resolve(process.argv[1]) : "";
@@ -3795,6 +3847,9 @@ try {
 if (invoked && (invoked === self || /[\\/]dist[\\/]cli\.mjs$/.test(invoked))) {
   main(process.argv.slice(2)).then((code) => {
     process.exitCode = code;
+  }, (e) => {
+    console.error(`想定外の誤り（${builtinErrorName(e)}）`);
+    process.exitCode = 1;
   });
 }
 export {