@bifos/dooray-cli 0.16.0 → 0.17.0

package/README.md

@@ -110,7 +110,8 @@ dooray post comment file upload <project> <number> <comment-id> <path>
 ```
 
 이미지 확장자는 이미지 마크다운으로, 그 외 파일은 일반 링크로 댓글 본문에 추가한다.
-`comment file list`는 웹 UI에서 직접 첨부한 파일을 놓칠 수 있으며, 이 경우 `post file list`로 확인한다.
+`comment file list`는 웹 UI 첨부와 CLI 업로드 파일을 함께 보여주며 `출처` 열로 구분한다.
+CLI로 올린 파일은 댓글의 첨부 카드가 아니라 본문 링크로 표시된다.
 
 ### 삭제 명령의 확인
 

package/dist/index.js

@@ -385,7 +385,7 @@ var import_node_path3 = __toESM(require("path"));
 var import_node_os3 = require("os");
 
 // src/version.ts
-var CLI_VERSION = true ? "0.16.0" : "0.0.0-dev";
+var CLI_VERSION = true ? "0.17.0" : "0.0.0-dev";
 
 // src/skill/context.ts
 function resolveSkillDataRoot(homeDir, xdgDataHome = process.env.XDG_DATA_HOME) {
@@ -3151,14 +3151,21 @@ async function readStdin() {
 
 // src/utils/attachment-check.ts
 var import_node_readline = __toESM(require("readline"));
-function extractAttachmentFileIds(body) {
-  const ids = /* @__PURE__ */ new Set();
-  const re = /!?\[[^\]]*\]\(\/files\/([^\s)?#]+)/g;
+function extractAttachmentReferences(body) {
+  const references = [];
+  const seen = /* @__PURE__ */ new Set();
+  const re = /!?\[([^\]]*)\]\(\/files\/([^\s)?#]+)/g;
   let m;
   while ((m = re.exec(body)) !== null) {
-    ids.add(m[1]);
+    const id = m[2];
+    if (seen.has(id)) continue;
+    seen.add(id);
+    references.push({ id, label: m[1] });
   }
-  return ids;
+  return references;
+}
+function extractAttachmentFileIds(body) {
+  return new Set(extractAttachmentReferences(body).map((reference) => reference.id));
 }
 function findDroppedAttachments(oldBody, newBody, attachments) {
   const oldIds = extractAttachmentFileIds(oldBody);
@@ -4117,6 +4124,14 @@ var commentDeleteCommand = new import_commander23.Command("delete").description(
 // src/commands/post/comment/get.ts
 var import_commander24 = require("commander");
 
+// src/utils/format-size.ts
+function formatSize(bytes) {
+  if (bytes == null) return "-";
+  if (bytes < 1024) return `${bytes}B`;
+  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
+  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
+}
+
 // src/formatters/comment.ts
 function formatCommentDetail(comment, opts) {
   if (opts.json) {
@@ -4140,7 +4155,7 @@ function formatCommentDetail(comment, opts) {
     process.stdout.write("\n[Attachments]\n");
     printTable(
       ["Name", "Size", "ID"],
-      comment.files.map((f) => [f.name, String(f.size), f.id])
+      comment.files.map((f) => [f.name ?? "-", formatSize(f.size), f.id])
     );
   }
 }
@@ -4296,13 +4311,54 @@ async function resolveCommentFileInput(client, args) {
   return { projectId, postId, commentId, secondary };
 }
 
+// src/utils/comment-file-merge.ts
+function labelOrNull(label) {
+  if (label === void 0 || label.length === 0) return null;
+  return label;
+}
+function mergeCommentFiles(input2) {
+  const postFilesById = new Map(input2.postFiles.map((file) => [file.id, file]));
+  const bodyRefsById = /* @__PURE__ */ new Map();
+  for (const reference of input2.bodyRefs) {
+    if (!bodyRefsById.has(reference.id)) bodyRefsById.set(reference.id, reference);
+  }
+  const merged = [];
+  const seen = /* @__PURE__ */ new Set();
+  for (const commentFile of input2.commentFiles) {
+    if (seen.has(commentFile.id)) continue;
+    seen.add(commentFile.id);
+    const postFile = postFilesById.get(commentFile.id);
+    const bodyRef = bodyRefsById.get(commentFile.id);
+    merged.push({
+      id: commentFile.id,
+      name: postFile?.name ?? commentFile.name ?? labelOrNull(bodyRef?.label),
+      size: postFile?.size ?? null,
+      mimeType: postFile?.mimeType ?? null,
+      source: bodyRef ? "both" : "attachment"
+    });
+  }
+  for (const bodyRef of input2.bodyRefs) {
+    if (seen.has(bodyRef.id)) continue;
+    seen.add(bodyRef.id);
+    const postFile = postFilesById.get(bodyRef.id);
+    merged.push({
+      id: bodyRef.id,
+      name: postFile?.name ?? labelOrNull(bodyRef.label),
+      size: postFile?.size ?? null,
+      mimeType: postFile?.mimeType ?? null,
+      source: "body-link"
+    });
+  }
+  return merged;
+}
+
 // src/commands/post/comment/file/list.ts
-function formatSize(bytes) {
-  if (bytes < 1024) return `${bytes}B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
-  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
+function formatSource(source) {
+  if (source === "attachment") return "\uCCA8\uBD80";
+  if (source === "body-link") return "\uBCF8\uBB38 \uB9C1\uD06C";
+  return "\uB458 \uB2E4";
 }
-var listCommentFileCommand = new import_commander25.Command("list").description("\uB313\uAE00 \uCCA8\uBD80 \uD30C\uC77C \uBAA9\uB85D \uC870\uD68C").argument("[arg1]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC, Dooray URL, \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uB313\uAE00 ID").argument("[arg2]", "\uC5C5\uBB34 \uBC88\uD638 (positional \uBAA8\uB4DC)").argument("[arg3]", "\uB313\uAE00 ID (positional 3\uAC1C \uBAA8\uB4DC)").option("--id <postId>", "Dooray post ID (project/post-number \uB300\uC2E0)").option("--url <url>", "Dooray \uC5C5\uBB34 URL (project/post-number \uB300\uC2E0)").option("--comment-id <logId>", "\uB313\uAE00 ID (positional \uB300\uCCB4)").action(async (arg1, arg2, arg3, opts) => {
+var listCommentFileCommand = new import_commander25.Command("list").description("\uB313\uAE00 \uCCA8\uBD80 \uD30C\uC77C\uACFC \uBCF8\uBB38 \uD30C\uC77C \uB9C1\uD06C \uBAA9\uB85D \uD1B5\uD569 \uC870\uD68C").argument("[arg1]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC, Dooray URL, \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uB313\uAE00 ID").argument("[arg2]", "\uC5C5\uBB34 \uBC88\uD638 (positional \uBAA8\uB4DC)").argument("[arg3]", "\uB313\uAE00 ID (positional 3\uAC1C \uBAA8\uB4DC)").option("--id <postId>", "Dooray post ID (project/post-number \uB300\uC2E0)").option("--url <url>", "Dooray \uC5C5\uBB34 URL (project/post-number \uB300\uC2E0)").option("--comment-id <logId>", "\uB313\uAE00 ID (positional \uB300\uCCB4)").action(async (arg1, arg2, arg3, opts) => {
   const globalOpts = listCommentFileCommand.optsWithGlobals();
   const config = await getConfigOrThrow();
   const client = new DoorayApiClient(config.apiKey, config.baseUrl);
@@ -4316,23 +4372,50 @@ var listCommentFileCommand = new import_commander25.Command("list").description(
     requireSecondary: false
   });
   startSpinner("\uB313\uAE00 \uCCA8\uBD80 \uD30C\uC77C \uBAA9\uB85D \uC870\uD68C \uC911...");
-  const res = await client.getPostComment(projectId, postId, commentId);
-  const files = res.result.files ?? [];
-  stopSpinner(true, `\uCCA8\uBD80 \uD30C\uC77C ${files.length}\uAC1C`);
-  if (files.length === 0) {
-    if (globalOpts.json) {
-      process.stdout.write("[]\n");
-    } else if (!globalOpts.quiet) {
-      process.stdout.write("\uCCA8\uBD80 \uC5C6\uC74C\n");
+  try {
+    const res = await client.getPostComment(projectId, postId, commentId);
+    const commentFiles = res.result.files ?? [];
+    const bodyRefs = extractAttachmentReferences(res.result.body.content);
+    if (commentFiles.length === 0 && bodyRefs.length === 0) {
+      stopSpinner(true, "\uCCA8\uBD80 \uD30C\uC77C 0\uAC1C");
+      if (globalOpts.json) {
+        process.stdout.write("[]\n");
+      } else if (!globalOpts.quiet) {
+        process.stdout.write("\uCCA8\uBD80 \uC5C6\uC74C\n");
+      }
+      return;
     }
-    return;
+    let postFiles = [];
+    let metadataLookupError = null;
+    try {
+      const postFilesRes = await client.getPostFiles(projectId, postId);
+      postFiles = postFilesRes.result;
+    } catch (error) {
+      metadataLookupError = error instanceof Error ? error.message : String(error);
+    }
+    const merged = mergeCommentFiles({ commentFiles, bodyRefs, postFiles });
+    stopSpinner(true, `\uCCA8\uBD80 \uD30C\uC77C ${merged.length}\uAC1C`);
+    if (metadataLookupError !== null) {
+      process.stderr.write(
+        `\u26A0  \uC5C5\uBB34 \uCCA8\uBD80 \uC774\uB984\xB7\uD06C\uAE30 \uBCF4\uAC15 \uC2E4\uD328 (${metadataLookupError}) \u2014 \uC77C\uBD80 \uD56D\uBAA9\uC758 \uD30C\uC77C\uBA85\xB7\uD06C\uAE30\uAC00 \uBE44\uC5B4 \uC788\uC2B5\uB2C8\uB2E4.
+`
+      );
+    }
+    output(globalOpts, {
+      headers: ["\uD30C\uC77C\uBA85", "\uD06C\uAE30", "\uCD9C\uCC98", "ID"],
+      rows: merged.map((file) => [
+        file.name == null ? "-" : sanitizeFileName(file.name),
+        formatSize(file.size),
+        formatSource(file.source),
+        file.id
+      ]),
+      raw: merged,
+      ids: merged.map((file) => file.id)
+    });
+  } catch (error) {
+    stopSpinner(false);
+    throw error;
   }
-  output(globalOpts, {
-    headers: ["\uD30C\uC77C\uBA85", "\uD06C\uAE30", "ID"],
-    rows: files.map((f) => [f.name, formatSize(f.size), f.id]),
-    raw: files,
-    ids: files.map((f) => f.id)
-  });
 });
 
 // src/commands/post/comment/file/upload.ts
@@ -4359,7 +4442,7 @@ function removeFileReference(body, fileId) {
 }
 
 // src/commands/post/comment/file/upload.ts
-var uploadCommentFileCommand = new import_commander26.Command("upload").description("\uB313\uAE00\uC5D0 \uD30C\uC77C \uC5C5\uB85C\uB4DC (\uBCF8\uBB38\uC5D0 reference \uC790\uB3D9 \uCD94\uAC00)").argument("[arg1]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC, Dooray URL, \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uB313\uAE00 ID").argument("[arg2]", "\uC5C5\uBB34 \uBC88\uD638 \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uD30C\uC77C \uACBD\uB85C").argument("[arg3]", "\uB313\uAE00 ID (positional \uBAA8\uB4DC)").argument("[arg4]", "\uD30C\uC77C \uACBD\uB85C (positional \uBAA8\uB4DC)").option("--id <postId>", "Dooray post ID (project/post-number \uB300\uC2E0)").option("--url <url>", "Dooray \uC5C5\uBB34 URL (project/post-number \uB300\uC2E0)").option("--comment-id <logId>", "\uB313\uAE00 ID (positional \uB300\uCCB4)").option("--file <path>", "\uC5C5\uB85C\uB4DC\uD560 \uD30C\uC77C \uACBD\uB85C (positional \uB300\uCCB4)").action(async (arg1, arg2, arg3, arg4, opts) => {
+var uploadCommentFileCommand = new import_commander26.Command("upload").description("\uB313\uAE00\uC5D0 \uD30C\uC77C \uC5C5\uB85C\uB4DC (\uCCA8\uBD80 \uCE74\uB4DC\uAC00 \uC544\uB2CC \uBCF8\uBB38 \uB9C1\uD06C\uB85C \uD45C\uC2DC)").argument("[arg1]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC, Dooray URL, \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uB313\uAE00 ID").argument("[arg2]", "\uC5C5\uBB34 \uBC88\uD638 \uB610\uB294 (`--id`/`--url` \uBAA8\uB4DC\uC77C \uB54C) \uD30C\uC77C \uACBD\uB85C").argument("[arg3]", "\uB313\uAE00 ID (positional \uBAA8\uB4DC)").argument("[arg4]", "\uD30C\uC77C \uACBD\uB85C (positional \uBAA8\uB4DC)").option("--id <postId>", "Dooray post ID (project/post-number \uB300\uC2E0)").option("--url <url>", "Dooray \uC5C5\uBB34 URL (project/post-number \uB300\uC2E0)").option("--comment-id <logId>", "\uB313\uAE00 ID (positional \uB300\uCCB4)").option("--file <path>", "\uC5C5\uB85C\uB4DC\uD560 \uD30C\uC77C \uACBD\uB85C (positional \uB300\uCCB4)").action(async (arg1, arg2, arg3, arg4, opts) => {
   const globalOpts = uploadCommentFileCommand.optsWithGlobals();
   const config = await getConfigOrThrow();
   const client = new DoorayApiClient(config.apiKey, config.baseUrl);
@@ -4502,11 +4585,6 @@ var commentFileCommand = new import_commander29.Command("file").description("\uB
 
 // src/commands/post/file/list.ts
 var import_commander30 = require("commander");
-function formatSize2(bytes) {
-  if (bytes < 1024) return `${bytes}B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
-  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
-}
 var fileListCommand = new import_commander30.Command("list").description("\uC5C5\uBB34 \uCCA8\uBD80\uD30C\uC77C \uBAA9\uB85D \uC870\uD68C").argument("[project]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC (\uB610\uB294 \uCCAB \uC778\uC790\uC5D0 Dooray URL)").argument("[post-number]", "\uC5C5\uBB34 \uBC88\uD638 (project\uC640 \uD568\uAED8 \uC0AC\uC6A9)").option("--id <postId>", "Dooray post ID (project/post-number \uB300\uC2E0)").option("--url <url>", "Dooray \uC5C5\uBB34 URL (project/post-number \uB300\uC2E0)").action(async (project, postNumberStr, opts) => {
   const globalOpts = fileListCommand.optsWithGlobals();
   const config = await getConfigOrThrow();
@@ -4525,7 +4603,7 @@ var fileListCommand = new import_commander30.Command("list").description("\uC5C5
     rows: res.result.map((f) => [
       f.id,
       f.name,
-      formatSize2(f.size),
+      formatSize(f.size),
       f.mimeType,
       f.createdAt
     ]),
@@ -5190,11 +5268,6 @@ var import_commander47 = require("commander");
 
 // src/commands/wiki/page-file/list.ts
 var import_commander42 = require("commander");
-function formatSize3(bytes) {
-  if (bytes < 1024) return `${bytes}B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
-  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
-}
 var wikiPageFileListCommand = new import_commander42.Command("list").description("\uC704\uD0A4 \uD398\uC774\uC9C0 \uCCA8\uBD80\uD30C\uC77C \uBAA9\uB85D \uC870\uD68C (general + inline image)").argument("[project]", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC (\uB610\uB294 \uCCAB \uC778\uC790\uC5D0 Dooray Wiki URL)").argument("[page-id]", "\uC704\uD0A4 \uD398\uC774\uC9C0 ID (project\uC640 \uD568\uAED8 \uC0AC\uC6A9)").option("--id <pageId>", "\uC704\uD0A4 \uD398\uC774\uC9C0 ID (--project \uB3D9\uBC18 \uD544\uC694)").option("--url <url>", "Dooray Wiki URL").option("--project <code>", "\uD504\uB85C\uC81D\uD2B8 \uCF54\uB4DC (--id \uBAA8\uB4DC\uC5D0\uC11C wikiId \uD574\uC11D\uC6A9)").action(async (project, pageIdArg, opts) => {
   const globalOpts = wikiPageFileListCommand.optsWithGlobals();
   const config = await getConfigOrThrow();
@@ -5215,7 +5288,7 @@ var wikiPageFileListCommand = new import_commander42.Command("list").description
     stopSpinner(true, `\uCCA8\uBD80\uD30C\uC77C ${merged.length}\uAC1C (general ${files.length} + inline ${images.length})`);
     output(globalOpts, {
       headers: ["ID", "Type", "\uD30C\uC77C\uBA85", "\uD06C\uAE30"],
-      rows: merged.map((f) => [f.id, f.type, f.name, formatSize3(f.size)]),
+      rows: merged.map((f) => [f.id, f.type, f.name, formatSize(f.size)]),
       raw: merged,
       ids: merged.map((f) => f.id)
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bifos/dooray-cli",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "CLI tool for Dooray project management — AI agent & terminal friendly",
   "keywords": [
     "dooray",

package/skills/dooray-cli/SKILL.md

@@ -159,7 +159,13 @@ NHN Dooray REST API 를 래핑한 CLI 다. 이 파일은 라우터이므로, 작
 | 댓글 첨부 삭제 | `dooray post comment file delete <project> <number> <comment-id> <file-id>` — 확인 있음, `-y`/`--yes`로 생략 |
 
 - 댓글 파일 업로드는 이미지 확장자면 이미지 마크다운을, 그 외에는 일반 링크를 만든다.
-- `comment file list`가 비어도 웹 UI 첨부가 없다고 단정하지 말고 `post file list`로 확인한다.
+- `comment file list`의 `출처`는 다음과 같다.
+  - `attachment` (`첨부`): 웹 UI 첨부
+  - `body-link` (`본문 링크`): CLI 업로드 링크
+  - `both` (`둘 다`): 양쪽에 있는 파일
+- `--json` 항목은 `{ id, name, size, mimeType, source }` 형식이다.
+  `source`는 위 값 중 하나다.
+- 메타데이터를 채우지 못하면 `name`, `size`, `mimeType`은 `null`이다.
 
 ## 위키
 

package/skills/dooray-cli/references/post.md

@@ -104,8 +104,14 @@ dooray post comment get <project> <number> <comment-id> --json | jq -r '.body.co
 
 첨부를 정말 떼려는 것이면 `--no-confirm` 으로 진행한다.
 
-`comment file list`는 댓글 조회 API가 노출한 첨부만 보여주므로 웹 UI에서 직접 첨부한 파일을 놓칠 수 있다.
-목록이 비어 있으면 `post file list`로 업무 전체 첨부를 확인한다.
+`comment file list`는 웹 UI 첨부와 CLI 업로드 링크를 함께 보여주고 `출처` 열로 구분한다.
+
+- `attachment`: 웹 UI 첨부
+- `body-link`: CLI 업로드 링크
+- `both`: 양쪽에 있는 파일
+
+`--json` 항목은 `{ id, name, size, mimeType, source }` 형식이다.
+메타데이터를 채우지 못하면 `name`, `size`, `mimeType`은 `null`이다.
 
 ## 이름이 겹칠 때