deel-local-cli 1.7.0 → 1.10.0

package/src/safety/keystore.js

@@ -36,12 +36,25 @@ import { join } from 'node:path';
 /** 잠긴 값의 꼴. `dpapi:<base64>` · `keychain:<이름>` */
 const 꼴 = /^(dpapi|keychain):(.*)$/s;
 
-/** 키체인에 넣을 때 쓰는 이름. 사람이 키체인 앱에서 찾아 지울 수 있어야 한다. */
-export const 키체인이름 = 'deel-gateway-key';
+/** 키체인에 넣을 때 쓰는 기본 이름. 사람이 키체인 앱에서 찾아 지울 수 있어야 한다. */
+export const 기본키체인이름 = 'deel-gateway-key';
 
-let 마지막탈 = null;
-/** 마지막으로 잠그거나 풀다 난 탈. 화면·심사서가 읽는다. */
-export function 열쇠탈() { return 마지막탈; }
+/**
+ * 실제로 쓸 키체인 이름.
+ *
+ * 기본값은 **바꾸지 않는다.** 이미 넣어 둔 열쇠를 계속 찾아야 하기 때문이다 —
+ * 이름을 갈면 사용자의 열쇠가 키체인에 남은 채로 안 보이게 된다.
+ *
+ * DEEL_KEYCHAIN_NAME 은 **따로 두고 쓰라고** 있는 자리다. 이름이 고정이라
+ * DEEL_HOME 을 임시 폴더로 돌려놓아도 키체인만은 진짜 자리를 썼고, 맥에서
+ * 이 저장소의 검사를 돌리면 잠그기/풀기가 **사용자의 진짜 게이트웨이 열쇠를
+ * 덮어썼다.** 검사 하나가 사람의 열쇠를 지우는 셈이다. 그래서 검사는 제 이름을
+ * 주고 쓰고, 끝나면 제 것만 지운다. 윈도우(DPAPI)는 잠근 덩이가 설정 파일 안에
+ * 있어서 이 이름과 상관이 없다.
+ */
+export function 키체인이름() {
+  return process.env.DEEL_KEYCHAIN_NAME || 기본키체인이름;
+}
 
 /** 이 값이 우리가 잠근 것인가. 평문 열쇠와 헷갈리면 안 된다. */
 export function 잠긴것인가(값) { return 꼴.test(String(값 ?? '')); }
@@ -102,7 +115,7 @@ function 키체인넣기(글) {
   const 계정 = userInfo().username;
   // 값 자체는 base64 로 넣는다. 키체인 도구가 줄바꿈·따옴표를 만나면 거기서 끊긴다.
   const 값 = Buffer.from(글, 'utf8').toString('base64');
-  const 명령 = `add-generic-password -a ${계정} -s ${키체인이름} -w ${값} -U\n`;
+  const 명령 = `add-generic-password -a ${계정} -s ${키체인이름()} -w ${값} -U\n`;
   마지막인자 = ['security', '-i'];
   const r = spawnSync('security', ['-i'], { input: 명령, encoding: 'utf8', timeout: 20000 });
   if (r.error) return { ok: false, err: r.error.message };
@@ -111,8 +124,8 @@ function 키체인넣기(글) {
 
 function 키체인읽기() {
   const 계정 = userInfo().username;
-  마지막인자 = ['security', 'find-generic-password', '-a', 계정, '-s', 키체인이름, '-w'];
-  const r = spawnSync('security', ['find-generic-password', '-a', 계정, '-s', 키체인이름, '-w'], {
+  마지막인자 = ['security', 'find-generic-password', '-a', 계정, '-s', 키체인이름(), '-w'];
+  const r = spawnSync('security', ['find-generic-password', '-a', 계정, '-s', 키체인이름(), '-w'], {
     encoding: 'utf8', timeout: 20000,
   });
   if (r.error) return { ok: false, text: '', err: r.error.message };
@@ -133,19 +146,19 @@ export function 쓸수있나() {
   // 꺼 뒀으면 켜져 있는 척하지 않는다. 이 값을 화면 문구도 그대로 쓰기 때문에,
   // 여기서 거짓으로 답하면 "다음 저장 때 잠급니다" 라고 해 놓고 안 잠근다.
   if (process.env.DEEL_KEYSTORE === 'off') {
-    return { 되나: false, 방식: '파일', 왜: 'DEEL_KEYSTORE=off 로 꺼 두었습니다' };
+    return { 되나: false, 방식: '파일', 왜: 'DEEL_KEYSTORE=off 로 꺼 두었습니다', 왜영어: 'turned off with DEEL_KEYSTORE=off' };
   }
   if (process.platform === 'win32') {
     const r = 파워셸실행('[Console]::Out.Write("ok")', '');
     if (r.ok && r.out === 'ok') return { 되나: true, 방식: 'dpapi', 왜: '' };
-    return { 되나: false, 방식: '파일', 왜: r.err || '파워셸을 못 돌렸습니다 (정책으로 막혔을 수 있습니다)' };
+    return { 되나: false, 방식: '파일', 왜: r.err || '파워셸을 못 돌렸습니다 (정책으로 막혔을 수 있습니다)', 왜영어: r.err || 'PowerShell could not be run (execution policy may block it)' };
   }
   if (process.platform === 'darwin') {
     const r = spawnSync('security', ['-h'], { encoding: 'utf8', timeout: 10000 });
     if (!r.error) return { 되나: true, 방식: 'keychain', 왜: '' };
-    return { 되나: false, 방식: '파일', 왜: 'security 명령을 못 찾았습니다' };
+    return { 되나: false, 방식: '파일', 왜: 'security 명령을 못 찾았습니다', 왜영어: 'the security command was not found' };
   }
-  return { 되나: false, 방식: '파일', 왜: '이 운영체제에서는 파일 권한(0600)으로만 둡니다' };
+  return { 되나: false, 방식: '파일', 왜: '이 운영체제에서는 파일 권한(0600)으로만 둡니다', 왜영어: 'this OS has no keystore, so file permissions (0600) are all there is' };
 }
 
 /**
@@ -157,19 +170,16 @@ export function 쓸수있나() {
 export function 잠그기(글) {
   const 값 = String(글 ?? '');
   if (!값) return null;
-  마지막탈 = null;
 
   if (process.platform === 'win32') {
     const r = 파워셸실행(잠그는스크립트, Buffer.from(값, 'utf8').toString('base64'));
     if (r.ok && r.out) return `dpapi:${r.out}`;
-    마지막탈 = `열쇠를 못 잠갔습니다 — ${r.err || '파워셸이 답을 안 줬습니다'}`;
-    return null;
+    return null;   // 사유는 어디로도 안 간다 — 잠긴 것인지 아닌지는 보관방식() 이 따로 말한다
   }
   if (process.platform === 'darwin') {
     const r = 키체인넣기(값);
-    if (r.ok) return `keychain:${키체인이름}`;
-    마지막탈 = `열쇠를 키체인에 못 넣었습니다 — ${r.err || 'security 가 실패했습니다'}`;
-    return null;
+    if (r.ok) return `keychain:${키체인이름()}`;
+    return null;   // 사유는 어디로도 안 간다 — 잠긴 것인지 아닌지는 보관방식() 이 따로 말한다
   }
   return null;
 }
@@ -187,51 +197,117 @@ export function 풀기(태그) {
   const m = 꼴.exec(String(태그 ?? ''));
   if (!m) return { ok: false, text: '', why: '잠긴 열쇠가 아닙니다' };
   const [, 갈래, 값] = m;
-  마지막탈 = null;
 
   if (갈래 === 'dpapi') {
     if (process.platform !== 'win32') {
-      마지막탈 = '이 열쇠는 윈도우에서 잠근 것이라 여기서는 못 풉니다 — deel setup 으로 다시 넣으세요.';
-      return { ok: false, text: '', why: 마지막탈 };
+      return { ok: false, text: '', why: '이 열쇠는 윈도우에서 잠근 것이라 여기서는 못 풉니다 — deel setup 으로 다시 넣으세요.' };
     }
     const r = 파워셸실행(푸는스크립트, 값);
     if (r.ok && r.out) {
       try { return { ok: true, text: Buffer.from(r.out, 'base64').toString('utf8'), why: '' }; }
       catch { /* 아래로 */ }
     }
-    마지막탈 = '잠근 열쇠를 못 풉니다 — 이 PC 의 이 계정에서 잠근 것만 풀립니다. deel setup 으로 다시 넣으세요.';
-    return { ok: false, text: '', why: 마지막탈 };
+    return { ok: false, text: '', why: '잠근 열쇠를 못 풉니다 — 이 PC 의 이 계정에서 잠근 것만 풀립니다. deel setup 으로 다시 넣으세요.' };
   }
 
   if (갈래 === 'keychain') {
     if (process.platform !== 'darwin') {
-      마지막탈 = '이 열쇠는 맥 키체인에 있습니다 — 여기서는 못 읽습니다. deel setup 으로 다시 넣으세요.';
-      return { ok: false, text: '', why: 마지막탈 };
+      return { ok: false, text: '', why: '이 열쇠는 맥 키체인에 있습니다 — 여기서는 못 읽습니다. deel setup 으로 다시 넣으세요.' };
     }
     const r = 키체인읽기();
     if (r.ok) return { ok: true, text: r.text, why: '' };
-    마지막탈 = `키체인에서 열쇠를 못 읽었습니다 — ${r.err}`;
-    return { ok: false, text: '', why: 마지막탈 };
+    return { ok: false, text: '', why: `키체인에서 열쇠를 못 읽었습니다 — ${r.err}` };
   }
 
   return { ok: false, text: '', why: '모르는 잠금 방식입니다' };
 }
 
-/** 화면·심사서에 그대로 쓰는 한 줄. 되는 척도 안 되는 척도 안 한다. */
-export function 보관방식(값 = null) {
+/**
+ * 화면·심사서에 그대로 쓰는 한 줄. 되는 척도 안 되는 척도 안 한다.
+ *
+ * 영어 심사서(pack/sheet.en.js)도 **이 함수**를 쓴다. 사실을 가려내는 규칙은
+ * 하나여야 한다 — 영어판에 같은 판단을 다시 적어 두면, 잠금장치가 달라졌을 때
+ * 한쪽만 고쳐져서 두 서류가 서로 다른 말을 하게 된다.
+ */
+export function 보관방식(값 = null, { lang = 'ko' } = {}) {
+  const 한 = lang === 'ko';
   if (값 && 잠긴것인가(값)) {
-    return 값.startsWith('dpapi:')
-      ? 'DPAPI — 이 PC 의 이 계정만 풉니다'
-      : '맥 키체인 — 이 계정의 로그인 키체인에 있습니다';
+    if (값.startsWith('dpapi:')) {
+      return 한 ? 'DPAPI — 이 PC 의 이 계정만 풉니다'
+        : 'DPAPI - only this account on this machine can unseal it';
+    }
+    return 한 ? '맥 키체인 — 이 계정의 로그인 키체인에 있습니다'
+      : 'macOS keychain - stored in this account\'s login keychain';
   }
   const 쓸수 = 쓸수있나();
   if (값) {
     // 평문인데 잠글 수는 있는 상태. 다음 저장에서 잠긴다.
-    return 쓸수.되나
-      ? `파일에 평문 — 다음 저장 때 ${쓸수.방식 === 'dpapi' ? 'DPAPI' : '키체인'} 로 잠급니다`
-      : `파일에 평문 + 권한 0600 — ${쓸수.왜}`;
+    const 장치 = 쓸수.방식 === 'dpapi' ? 'DPAPI' : (한 ? '키체인' : 'the keychain');
+    if (쓸수.되나) {
+      return 한 ? `파일에 평문 — 다음 저장 때 ${장치} 로 잠급니다`
+        : `Plain text in the file - it will be sealed with ${장치} on the next save`;
+    }
+    return 한 ? `파일에 평문 + 권한 0600 — ${쓸수.왜}`
+      : `Plain text in the file, mode 0600 - ${쓸수.왜영어 ?? 'no OS keystore is available here'}`;
+  }
+  if (process.platform === 'win32') {
+    return 한 ? 'DPAPI (윈도우) · 저장된 열쇠 없음' : 'DPAPI (Windows) - no key stored';
+  }
+  if (process.platform === 'darwin') {
+    return 한 ? '맥 키체인 · 저장된 열쇠 없음' : 'macOS keychain - no key stored';
+  }
+  return 한 ? '파일 권한 0600 · 저장된 열쇠 없음' : 'File mode 0600 - no key stored';
+}
+
+/**
+ * 잠금장치에서 열쇠를 지운다 (`deel reset model` 이 부른다).
+ *
+ * ── 왜 따로 있어야 하나 ────────────────────────────────────────────────
+ *
+ * 설정 파일만 지우면 **잠금장치에는 열쇠가 그대로 남는다.** 화면에는
+ * 「초기화했습니다」 가 뜨는데 열쇠는 살아 있는 상태다. 「초기화」 라고
+ * 말하려면 그것까지 지워야 하고, 못 지웠으면 못 지웠다고 말해야 한다.
+ *
+ * 두 방식이 다르게 생겼다:
+ *
+ *   윈도우(DPAPI)  잠근 덩이가 설정 파일 **안에** 있다. 파일을 지우면 같이
+ *                  사라진다 — 잠금장치에 따로 남는 것이 없다. 그래서 여기서
+ *                  할 일이 없고, 없다고 정직하게 답한다. 「지웠다」 고 하면
+ *                  안 한 일을 했다고 말하는 것이다.
+ *   맥(키체인)     설정에는 이름표만 있고 실물은 로그인 키체인에 남는다.
+ *                  이건 우리가 지워야 한다. 안 지우면 설정을 지운 뒤에도
+ *                  키체인 앱에 `deel-gateway-key` 가 그대로 보인다.
+ *
+ * 열쇠는 여기서도 명령줄에 안 올린다 — 지울 때 넘기는 것은 이름뿐이다.
+ *
+ * @param {string|null} 값 설정에 적혀 있던 잠긴 값. 없으면 이 PC 방식으로 판단한다.
+ * @returns {{지움: boolean, 방식: 'dpapi'|'keychain'|'없음', 왜: string}}
+ */
+export function 잠금지우기(값 = null) {
+  const m = 꼴.exec(String(값 ?? ''));
+  const 갈래 = m ? m[1] : (process.platform === 'darwin' ? 'keychain' : null);
+
+  if (갈래 === 'dpapi' || (!갈래 && process.platform === 'win32')) {
+    return { 지움: false, 방식: 'dpapi', 왜: '설정 파일 안에 있어서 파일과 같이 지워집니다' };
+  }
+  if (갈래 !== 'keychain') {
+    return { 지움: false, 방식: '없음', 왜: '잠금장치에 따로 둔 것이 없습니다' };
+  }
+  if (process.platform !== 'darwin') {
+    return { 지움: false, 방식: 'keychain', 왜: '맥에서 넣은 열쇠라 여기서는 못 지웁니다 — 그 맥에서 지우세요' };
+  }
+
+  const 계정 = userInfo().username;
+  const 인자 = ['delete-generic-password', '-a', 계정, '-s', 키체인이름()];
+  마지막인자 = ['security', ...인자];
+  const r = spawnSync('security', 인자, { encoding: 'utf8', timeout: 20000 });
+  if (r.error) return { 지움: false, 방식: 'keychain', 왜: r.error.message };
+  // 없는 것을 지우라고 해도 실패로 온다. 그건 탈이 아니라 이미 없는 것이다.
+  if (r.status !== 0) {
+    const 말 = (r.stderr ?? '').trim();
+    return /could not be found|SecKeychainSearchCopyNext/i.test(말)
+      ? { 지움: false, 방식: 'keychain', 왜: '키체인에 이미 없습니다' }
+      : { 지움: false, 방식: 'keychain', 왜: 말 || 'security 가 실패했습니다' };
   }
-  if (process.platform === 'win32') return 'DPAPI (윈도우) · 저장된 열쇠 없음';
-  if (process.platform === 'darwin') return '맥 키체인 · 저장된 열쇠 없음';
-  return '파일 권한 0600 · 저장된 열쇠 없음';
+  return { 지움: true, 방식: 'keychain', 왜: '' };
 }

package/src/safety/undo.js

@@ -3,6 +3,7 @@
 import { join, dirname } from 'node:path';
 import { readFileSync, writeFileSync, existsSync, mkdirSync, rmSync, appendFileSync, statSync } from 'node:fs';
 import { looksBinary } from '../tools/encoding.js';
+import { 말 } from '../i18n/index.js';
 
 // 되돌리기 이력은 파일 내용을 통째로 담는다. 이만큼 커지면 오래된 턴을 버린다.
 const MAX_BYTES = 32 * 1024 * 1024;
@@ -167,7 +168,7 @@ export class History {
             continue;
           }
           rmSync(path, { force: true });
-          restored.push({ path, how: '삭제됨(원래 없던 파일)' });
+          restored.push({ path, how: 말('undo.wayDeleted'), ok: true });
         } else {
           /*
            * 담고 있던 폴더가 없어졌을 수 있다 — Move 로 폴더째 옮긴 경우다.
@@ -178,16 +179,34 @@ export class History {
           mkdirSync(dirname(path), { recursive: true });
           // enc 가 붙어 있으면 UTF-8 로 담을 수 없던 파일이다 — 바이트를 그대로 되돌린다.
           writeFileSync(path, rec.enc === 'b64' ? Buffer.from(rec.before, 'base64') : Buffer.from(rec.before, 'utf8'));
-          restored.push({ path, how: '되돌림' });
+          restored.push({ path, how: 말('undo.wayRestored'), ok: true });
         }
       } catch (err) {
-        restored.push({ path, how: `실패: ${err.message}` });
+        restored.push({ path, how: `실패: ${err.message}`, ok: false, 못했나: true });
       }
     }
-    // 되돌린 기록은 잘라낸다.
-    const keep = recs.filter((r) => !turns.includes(r.turn));
+    /*
+     * ── 못 되돌린 것의 기록은 **남긴다** ──────────────────────────────
+     *
+     * 전에는 성패를 안 가리고 그 턴의 기록을 통째로 잘라냈다. 그러면 한 번
+     * 실패한 파일은 스냅샷이 사라져서 **다시 시도할 길이 없다.** 화면은
+     * 「파일 N개를 되돌렸습니다」 라고 하고(실패한 것까지 셌다), 감사기록에도
+     * 그 부풀린 수가 남고, 부르는 쪽은 /diff 목록에서까지 그 파일을 지운다.
+     * 파일만 고쳐진 채로 남고 아무도 그걸 모른다.
+     *
+     * 되돌린 것만 잘라낸다. 못한 것은 그대로 두어 한 번 더 시도할 수 있게 한다.
+     */
+    const 못한것 = new Set(restored.filter((x) => x.ok === false).map((x) => x.path));
+    const keep = recs.filter((r) => !turns.includes(r.turn) || 못한것.has(r.path));
     writeFileSync(this.file, keep.map((r) => JSON.stringify(r)).join('\n') + (keep.length ? '\n' : ''), 'utf8');
-    return { restored, turns: turns.length, turnIds: turns.slice() };
+    return {
+      restored,
+      // 화면·감사기록이 쓰는 수. **진짜로 되돌아간 것만** 센다.
+      되돌린수: restored.filter((x) => x.ok === true).length,
+      못한것: restored.filter((x) => x.ok === false),
+      turns: turns.length,
+      turnIds: turns.slice(),
+    };
   }
 }
 

package/src/tools/desc.en.js

@@ -25,8 +25,12 @@
  *   Bash  “끝나지 않는 것은 background”   · 시간 초과로 죽는 것을 막는 자리
  *   Verify“확인 못 한 것은 못 했다고”     · 이 프로그램이 거짓말을 안 하게 하는 자리
  *
- * 도구 이름과 인자 이름은 **안 옮긴다.** 그건 식별자다. Task 의 목적·할일처럼
- * 한글로 된 인자 이름도 그대로 둔다 — 이름을 바꾸면 그 도구가 아예 안 불린다.
+ * 도구 이름과 인자 이름은 **안 옮긴다.** 그건 식별자다 — 이름을 바꾸면 그 도구가
+ * 아예 안 불린다. 여기 적힌 이름은 tools/*.js 의 설명서와 **글자까지 같아야**
+ * 한다. 다르면 그 인자만 영어 설명이 조용히 빠진다.
+ *
+ * 인자 이름은 전부 영문이다. 우리 취향이 아니라 서버가 검사한다 — Bedrock 은
+ * `^[a-zA-Z0-9_.-]{1,64}$` 가 아니면 첫 한마디를 통째로 400 으로 돌려보낸다.
  */
 export const 도구설명EN = {
   Read: {
@@ -134,14 +138,14 @@ export const 도구설명EN = {
     },
   },
   Ask: {
-    desc: 'Ask the person one question at a genuine fork. **First fill in `이해` with what you'
+    desc: 'Ask the person one question at a genuine fork. **First fill in `understanding` with what you'
       + ' understood this request to be**, and only ask if something is genuinely left to decide.'
       + ' Never write "let me know" and stop; asking in prose ends the turn and throws away'
       + ' everything you have looked at. Give 2-4 options and they answer with a single number.'
       + ' **Never ask back something the user already said.** "Tidy up the files" is the answer —'
       + ' go and do it. Deciding how is your job, not theirs.',
     params: {
-      이해: 'one line: what you understood this request to be. The person reads this line to judge'
+      understanding: 'one line: what you understood this request to be. The person reads this line to judge'
         + ' whether you actually read them. "I understand your request" is not an understanding —'
         + ' name the actual work',
       question: 'one sentence. Make it clear what has to be decided',
@@ -197,14 +201,14 @@ export const 도구설명EN = {
       + ' not pile up in your window. That is why work that creates or edits several files has to be'
       + ' divided this way to get to the end. One chunk must be finishable on its own (e.g. "create'
       + ' index.html and style.css"). The subtask cannot see your conversation — put everything it'
-      + ' needs into 할일. Do not use this for one short job. Doing it yourself is faster.',
+      + ' needs into `task`. Do not use this for one short job. Doing it yourself is faster.',
     params: {
-      목적: 'this chunk in one line (e.g. "build the dashboard page skeleton")',
-      할일: 'everything the subtask has to do. It cannot see this conversation, so put the background,'
+      purpose: 'this chunk in one line (e.g. "build the dashboard page skeleton")',
+      task: 'everything the subtask has to do. It cannot see this conversation, so put the background,'
         + ' the decisions, and the file paths here. Say what counts as done, too.',
-      모드: 'how the subtask works: code (builds and edits) · debug (finds causes) · ask (reads and'
+      mode: 'how the subtask works: code (builds and edits) · debug (finds causes) · ask (reads and'
         + ' answers only). Defaults to code.',
-      모델: 'hand this chunk to a **different model**. Only profile names the user has configured'
+      model: 'hand this chunk to a **different model**. Only profile names the user has configured'
         + ' work (do not invent an address — it will not be accepted). Left out, it stays on the model'
         + ' you are using. Handing routine work (formatting, repetitive edits, short summaries) to a'
         + ' small model keeps your window from filling. Do the work that needs judgement yourself.',
@@ -236,12 +240,13 @@ export const 도구설명EN = {
     },
   },
   Jobs: {
-    desc: 'List, read, and end background commands (Bash with background). Called with no number,'
-      + ' you get the list. Given a number, you get whatever output arrived since last time.'
-      + ' If you started a server, you must end it when the job is done.',
+    desc: 'List, read, and end background commands (Bash with background). Called with no `job`,'
+      + ' you get the list. Given a `job`, you get whatever output arrived since last time.'
+      + ' If you started a server, you must end it with `stop` when the job is done.',
     params: {
-      번호: 'job number to look at. Left out, the list',
-      끝내기: 'true to end that job (stop)',
+      job: 'job number to look at. Left out, the list',
+      stop: 'true to end that job',
+      from_start: 'true to read from the beginning again',
     },
   },
 };