@kaydash9999/banker-plugins 0.24.0 → 0.25.0

package/skills/tune-tmux/scripts/tmux-config.mjs

@@ -6,9 +6,9 @@
  * Every function that runs a program takes `exec(cmd, args)` returning { status, stdout, stderr, error }, so the
  * tests can stand in for tmux on every OS. No network is used.
  */
-import { accessSync, constants, lstatSync, readFileSync, rmSync, statSync } from "node:fs";
+import { accessSync, constants, lstatSync, readFileSync, realpathSync, rmSync, statSync } from "node:fs";
 import { devNull, homedir } from "node:os";
-import { basename, delimiter, join, resolve } from "node:path";
+import { basename, delimiter, dirname, join, resolve } from "node:path";
 
 /** The tool and version in `tmux -V` output. psmux says "tmux 3.3.8" on line 1 and "psmux" on line 2. */
 export function parseVersion(text) {
@@ -76,13 +76,45 @@ export function pickClipboard({ platform, env, has }) {
 
 const exists = (p) => lstatSync(p, { throwIfNoEntry: false }) !== undefined;
 
+// The path as the file system spells it (native first: the long name and the case on disk), a missing tail joined to
+// its nearest existing folder so spelled.
+function spelled(p) {
+  try {
+    return realpathSync.native(p);
+  } catch {
+    try {
+      return realpathSync(p);
+    } catch {
+      const up = dirname(p);
+      return up === p ? p : join(spelled(up), basename(p));
+    }
+  }
+}
+
+/**
+ * Whether a and b name one file: device and inode (bigint) when both are there, else the spelled paths, case folded on
+ * Windows and macOS. A string compare took a --conf in another case, through a linked XDG_CONFIG_HOME or an 8.3 name
+ * for another file and warned wrongly.
+ */
+export function sameFile(a, b) {
+  try {
+    const x = statSync(a, { bigint: true });
+    const y = statSync(b, { bigint: true });
+    if (x.ino !== 0n && y.ino !== 0n) return x.dev === y.dev && x.ino === y.ino;
+  } catch {
+    // one is not there (yet): the spellings decide
+  }
+  const fold = (p) => (process.platform === "win32" || process.platform === "darwin" ? p.toLowerCase() : p);
+  return fold(spelled(resolve(a))) === fold(spelled(resolve(b)));
+}
+
 /** The config files tmux reads by itself, in order. Measured: 3.6a reads every one there; 2.7 only ~/.tmux.conf. */
 export function tmuxReads(v, env, home) {
   const files = [join(home, ".tmux.conf")];
   if (v && !atLeast(v, 3, 1)) return files;
   if (env.XDG_CONFIG_HOME) files.push(join(env.XDG_CONFIG_HOME, "tmux", "tmux.conf"));
   files.push(join(home, ".config", "tmux", "tmux.conf"));
-  return [...new Set(files.map((f) => resolve(f)))];
+  return files.map((f) => resolve(f)).filter((f, i, all) => !all.slice(0, i).some((g) => sameFile(g, f)));
 }
 
 /** The tmux file for the block: --conf, else the first of tmuxReads that is there, else ~/.tmux.conf. */

package/skills/tune-tmux/scripts/tune-tmux.mjs

@@ -19,15 +19,19 @@
  * Exit status: 0 done (nothing to change included), 1 refused, write or reload failed, 2 usage error,
  * 3 environment (on or off with neither tmux nor psmux). No network is used.
  */
-import { mkdtempSync, readFileSync, realpathSync, rmSync, writeFileSync } from "node:fs";
+import { existsSync, mkdtempSync, readFileSync, realpathSync, rmSync, writeFileSync } from "node:fs";
 import { spawnSync } from "node:child_process";
-import { tmpdir } from "node:os";
+import { homedir, tmpdir } from "node:os";
 import { join, resolve } from "node:path";
 import { pathToFileURL } from "node:url";
 import { HASH, commitBlock, markers, planBlock } from "./marker-block.mjs";
 import * as tc from "./tmux-config.mjs";
 import { appRows } from "./app-notes.mjs";
 
+// The folder programs start in: the home folder (Windows looks for a program in the working folder first, so a
+// program planted in the folder worked on never runs), or the temp folder when there is no home folder.
+const runDir = () => (existsSync(homedir()) ? homedir() : tmpdir());
+
 const USAGE = [
   "tune-tmux: tmux, psmux 설정에 표지 블록을 넣고 빼서 마우스 드래그 복사와 PageUp/PageDown 스크롤을 맞춤",
   "사용법: node tune-tmux.mjs [status|on|off] [--target auto|tmux|psmux] [--conf <경로>] [--follow-symlink] [--force-drag] [--reload] [--socket <이름>] [--dry-run] [--json]",
@@ -68,7 +72,8 @@ const loadCaps = () => JSON.parse(readFileSync(new URL("./psmux-caps.json", impo
 
 function defaults(deps) {
   const env = deps.env ?? process.env;
-  const exec = (cmd, args) => spawnSync(cmd, args, { encoding: "utf8", env, timeout: 15000, windowsHide: true });
+  // from the home folder: Windows looks for tmux or psmux in the working folder first
+  const exec = (cmd, args) => spawnSync(cmd, args, { cwd: runDir(), encoding: "utf8", env, timeout: 15000, windowsHide: true });
   return { env, platform: process.platform, exec, caps: null, ...deps };
 }
 
@@ -109,8 +114,8 @@ function context(o, d) {
   }
   const conf = tc.chooseTmuxConf(t.tool, d.env, home, o.conf);
   const reads = tc.tmuxReads(t.tool, d.env, home);
-  if (o.conf && !reads.includes(conf)) ctx.warnings.push(`경고: tmux 가 스스로 읽지 않는 파일: ${conf}. tmux -f 나 source-file 로 불러야 적용됨`);
-  ctx.warnings.push(...tc.conflictsIn(reads.filter((f) => f !== conf)));
+  if (o.conf && !reads.some((f) => tc.sameFile(f, conf))) ctx.warnings.push(`경고: tmux 가 스스로 읽지 않는 파일: ${conf}. tmux -f 나 source-file 로 불러야 적용됨`);
+  ctx.warnings.push(...tc.conflictsIn(reads.filter((f) => !tc.sameFile(f, conf))));
   const body = t.tool ? tmuxWanted(t.tool, o, d) : { lines: [], omitted: [], clipboard: null };
   return { ...ctx, conf, ...body };
 }
@@ -155,7 +160,9 @@ function change(ctx, o, d) {
   const plan = planOf(ctx, o.cmd === "off" || none ? null : ctx.lines.join("\n"), o);
   if (!plan.ok) return fail(1, `${plan.message}. 아무것도 바꾸지 않음`);
   if (none && !plan.changed) return { code: 0, report: { ...reportOf(ctx, o.cmd, o), changed: false, ...noLines(ctx) } };
-  return reloaded(ctx, plan, o, d, none ? removedOld(written(ctx, o, plan), ctx) : written(ctx, o, plan));
+  const done = o.dryRun ? plan : commitBlock(plan, SKILL);
+  if (!done.ok) return fail(1, `${done.message}. 아무것도 바꾸지 않음`);
+  return reloaded(ctx, plan, o, d, none ? removedOld(written(ctx, o, plan, done), ctx) : written(ctx, o, plan, done));
 }
 
 // The written report, applied to a running server when --reload asks (not on a dry run).
@@ -166,9 +173,8 @@ function reloaded(ctx, plan, o, d, out) {
   return { code: 0, report: { ...out, reload } };
 }
 
-// Writes the plan unless dry-run and says what happened.
-function written(ctx, o, plan) {
-  const done = o.dryRun ? plan : commitBlock(plan, SKILL);
+// What the write (or the dry-run plan) did, as the report says it.
+function written(ctx, o, plan, done) {
   const empty = (o.cmd === "off" || ctx.lines.length === 0) && plan.changed && plan.after?.length === 0;
   const note = empty ? "블록만 있던 파일이라 빈 파일로 남김" : null;
   return { ...reportOf(ctx, o.cmd, o), changed: plan.changed, created: done.created === true, backup: done.backup ?? null, note };

package/skills/ultra-chain/references/chain-ledger.mjs

@@ -18,8 +18,13 @@ import { spawnSync } from "node:child_process";
 import { createHash, randomBytes } from "node:crypto";
 import { appendFileSync, existsSync, mkdirSync, readFileSync, realpathSync, writeFileSync } from "node:fs";
 import { dirname, isAbsolute, join, relative, resolve } from "node:path";
+import { homedir, tmpdir } from "node:os";
 import { fileURLToPath } from "node:url";
 
+// The folder programs start in: the home folder (Windows looks for a program in the working folder first, so a
+// program planted in the folder worked on never runs), or the temp folder when there is no home folder.
+const startDir = () => (existsSync(homedir()) ? homedir() : tmpdir());
+
 export const STAGES = [
   { id: "preflight", label: "사전 점검" },
   { id: "research", label: "자료 조사" },
@@ -219,8 +224,10 @@ function checkQuorum(event, meta) {
 
 // ── run folder ───────────────────────────────────────────────────────────────
 
+// Started from the home folder with git -C: Windows looks for a program in the working folder first, so a git.exe
+// inside the folder worked on never runs.
 function git(cwd, args) {
-  const r = spawnSync("git", args, { cwd, encoding: "utf8" });
+  const r = spawnSync("git", ["-C", resolve(cwd), ...args], { cwd: startDir(), encoding: "utf8" });
   return r.status === 0 ? r.stdout.trim() : null;
 }
 

package/skills/vertical-pptx/verify_a4p.mjs

@@ -41,6 +41,7 @@
  * Usage: node verify_a4p.mjs <file.pptx> [--json] [--strict] [--font-file <path>] [--deck <deck.json>]
  */
 import { spawnSync } from 'node:child_process';
+import { randomBytes } from 'node:crypto';
 import fs from 'node:fs';
 import { createRequire } from 'node:module';
 import os from 'node:os';
@@ -2101,10 +2102,16 @@ export function main(argv) {
         console.error(`verify_a4p: 재압축 산출물이 자기 무결성 검사를 통과하지 못해 쓰지 않았다:\n  ${integrity.problems.join('\n  ')}`);
         return EXIT.FAIL;
       }
-      // Stage then rename, so an interrupted --in-place cannot leave a half-written deck.
-      const tmp = `${target}.a4p-tmp`;
-      fs.writeFileSync(tmp, rep.out);
-      fs.renameSync(tmp, target);
+      // Stage then rename, so an interrupted --in-place cannot leave a half-written deck. The stage name is random
+      // and made exclusively, so a link someone left at a fixed name is never written through.
+      const tmp = `${target}.${randomBytes(6).toString('hex')}.a4p-tmp`;
+      try {
+        fs.writeFileSync(tmp, rep.out, { flag: 'wx' });
+        fs.renameSync(tmp, target);
+      } catch (e) {
+        fs.rmSync(tmp, { force: true });
+        throw e;
+      }
 
       report.write = {
         mode: opts.convert ? 'convert-a4p' : 'repair',