@helpfeel/cosense-cli 1.12.0 → 1.12.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@helpfeel/cosense-cli",
-  "version": "1.12.0",
+  "version": "1.12.1",
   "description": "Cosense (旧Scrapbox) のページを読み・調べ・編集するAgent Skill用のCLI",
   "homepage": "https://github.com/helpfeel/cosense-cli",
   "license": "MIT",
@@ -24,13 +24,13 @@
     "format": "oxfmt"
   },
   "dependencies": {
-    "tsx": "4.23.10"
+    "tsx": "4.23.12"
   },
   "devDependencies": {
     "@types/node": "24.13.3",
     "npm-run-all": "4.1.5",
-    "oxfmt": "0.61.0",
-    "oxlint": "1.76.0",
+    "oxfmt": "0.62.0",
+    "oxlint": "1.77.0",
     "typescript": "7.0.2"
   },
   "engines": {

package/src/cli.ts

@@ -242,7 +242,9 @@ if (command === '--version') {
   process.exit(0);
 }
 
-const spec = command ? commands[command] : undefined;
+// cosense constructorやcosense __proto__はObject.prototypeのpropertyを引いてしまう
+const spec =
+  command && Object.hasOwn(commands, command) ? commands[command] : undefined;
 if (!spec) {
   process.stderr.write(
     `invalid command${command ? `: ${command}` : ''}\n` +

package/src/commands/uploadFile.ts

@@ -20,13 +20,13 @@ Usage:
   <filePath>    アップロードするローカルファイルのパス
 
 オプション:
-  --content-type <type>  ファイルのMIME type。省略時は拡張子から推定し、不明な拡張子はapplication/octet-streamになる
+  --content-type <type>  ファイルのMIME type
 
 出力（JSON）:
-  embedUrl      string  ページ本文への埋め込みに使うファイルURL
-  originalname  string  アップロードしたファイル名
-  contentType   string  embedUrlが配信するファイルのMIME type
-  size          number  ファイルのbyte数
+  embedUrl      string   ページ本文への埋め込みに使うファイルURL
+  originalname  string   アップロードしたファイル名
+  contentType   string?  ファイルのMIME type
+  size          number   ファイルのbyte数
 
 例:
   cosense uploadFile 'https://scrapbox.io/example' ./photo.png
@@ -55,7 +55,7 @@ const parseArgs = (args: string[]): ParsedArgs => {
         throw new Error(`--content-type specified multiple times\n${usage}`);
       }
       const value = args[i + 1];
-      if (value === undefined || value.startsWith('--')) {
+      if (!value || value.startsWith('--')) {
         throw new Error(`--content-type requires a value\n${usage}`);
       }
       contentType = value;
@@ -81,11 +81,11 @@ const parseArgs = (args: string[]): ParsedArgs => {
 const uploadToSignedUrl = async (
   url: string,
   body: Uint8Array,
-  contentType: string
+  contentType: string | undefined
 ): Promise<void> => {
   const res = await fetch(url, {
     method: 'PUT',
-    headers: { 'Content-Type': contentType },
+    headers: contentType ? { 'Content-Type': contentType } : {},
     body
   });
   if (res.ok) {
@@ -149,10 +149,10 @@ export const uploadFile = async (args: string[]): Promise<void> => {
   let resultContentType = contentType;
   if (uploadRequest.embedUrl) {
     // 同一ファイルがアップロード済みの場合、serverはupload-requestで即embedUrlを返す。
-    // embedUrlが配信するのは保存済みファイルなので、contentTypeはローカル推定よりserver値を優先する
+    // embedUrlが配信するのは保存済みファイルなので、ローカル推定の型は当てにならない
     embedUrl = uploadRequest.embedUrl;
     originalname = uploadRequest.originalname ?? name;
-    resultContentType = uploadRequest.contentType ?? contentType;
+    resultContentType = uploadRequest.contentType?.trim() || undefined;
   } else {
     const { signedUrl, fileId } = uploadRequest;
     if (!signedUrl || !fileId) {

package/src/lib/mimeTypes.ts

@@ -1,7 +1,8 @@
 import { extname } from 'node:path';
 
-// アップロード時に申告するContent-Typeを拡張子から推定する最小マップ。
-// 網羅せず、外れる型は--content-typeで上書きしてもらう
+// アップロード時に申告するContent-Typeを拡張子から推定するマップ。網羅はしないが、
+// browsePageが<cosense:file type="...">でファイル種別をAIに伝えられるよう、判明した型は足していく。
+// ここに無い型は--content-typeで上書きしてもらう
 const MIME_TYPES: Record<string, string> = {
   png: 'image/png',
   jpg: 'image/jpeg',
@@ -20,6 +21,13 @@ const MIME_TYPES: Record<string, string> = {
   weba: 'audio/webm',
   aac: 'audio/aac',
   pdf: 'application/pdf',
+  docx: 'application/vnd.openxmlformats-officedocument.wordprocessingml.document',
+  xlsx: 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet',
+  xlsm: 'application/vnd.ms-excel.sheet.macroEnabled.12',
+  pptx: 'application/vnd.openxmlformats-officedocument.presentationml.presentation',
+  doc: 'application/msword',
+  xls: 'application/vnd.ms-excel',
+  ppt: 'application/vnd.ms-powerpoint',
   txt: 'text/plain',
   md: 'text/markdown',
   csv: 'text/csv',
@@ -29,7 +37,11 @@ const MIME_TYPES: Record<string, string> = {
   zip: 'application/zip'
 };
 
-export const contentTypeForFile = (filePath: string): string => {
+// 推定できない時にapplication/octet-streamを申告しない。octet-streamで申告するとserverが
+// embedUrlから拡張子を落とすため、URLからファイル種別が読み取れなくなり、ページ本文での
+// 埋め込み表示の判定もできなくなる
+export const contentTypeForFile = (filePath: string): string | undefined => {
   const ext = extname(filePath).slice(1).toLowerCase();
-  return MIME_TYPES[ext] ?? 'application/octet-stream';
+  // a.constructorやa.__proto__はObject.prototypeのpropertyを引いてしまう
+  return Object.hasOwn(MIME_TYPES, ext) ? MIME_TYPES[ext] : undefined;
 };