fix: address PR review feedback for bundle command

- Extract readStdin to shared utility in src/utils/options.ts
- Check for commas before existsSync to reduce key/file ambiguity
- Document file-detection behavior in --keys description

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: designcode

src/lib/bundle.ts

@@ -1,22 +1,14 @@
 import { getStorageConfig } from '@auth/provider.js';
 import { bundle } from '@tigrisdata/storage';
 import { exitWithError } from '@utils/exit.js';
-import { getFormat, getOption } from '@utils/options.js';
+import { getFormat, getOption, readStdin } from '@utils/options.js';
 import { parseAnyPath } from '@utils/path.js';
 import { createWriteStream, existsSync, readFileSync } from 'fs';
 import { Readable } from 'stream';
 import { pipeline } from 'stream/promises';
 
 const MAX_KEYS = 5000;
 
-async function readStdin(): Promise<string> {
-  const chunks: Buffer[] = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return Buffer.concat(chunks).toString('utf-8');
-}
-
 function parseKeys(content: string): string[] {
   return content
     .split('\n')
@@ -65,13 +57,18 @@ export default async function bundleCommand(options: Record<string, unknown>) {
   let keys: string[];
 
   if (keysArg) {
-    if (existsSync(keysArg)) {
-      keys = parseKeys(readFileSync(keysArg, 'utf-8'));
-    } else {
+    if (keysArg.includes(',')) {
+      // Commas present → always treat as inline comma-separated keys
       keys = keysArg
         .split(',')
         .map((k) => k.trim())
         .filter((k) => k.length > 0);
+    } else if (existsSync(keysArg)) {
+      // No commas and local file exists → read as keys file
+      keys = parseKeys(readFileSync(keysArg, 'utf-8'));
+    } else {
+      // Single key
+      keys = [keysArg.trim()];
     }
   } else if (!process.stdin.isTTY) {
     const input = await readStdin();

src/lib/iam/policies/utils.ts

@@ -1,12 +1,6 @@
 import type { PolicyDocument } from '@tigrisdata/iam';
 
-export async function readStdin(): Promise<string> {
-  const chunks: Buffer[] = [];
-  for await (const chunk of process.stdin) {
-    chunks.push(chunk);
-  }
-  return Buffer.concat(chunks).toString('utf-8');
-}
+export { readStdin } from '@utils/options.js';
 
 export function parseDocument(jsonString: string): PolicyDocument {
   const raw = JSON.parse(jsonString);

src/specs.yaml

@@ -567,7 +567,7 @@ commands:
           - my-bucket
           - t3://my-bucket
       - name: keys
-        description: "Comma-separated object keys, or path to a file with one key per line. If omitted, reads keys from stdin"
+        description: "Comma-separated object keys, or path to a file with one key per line. If a local file matching the value exists, it is read as a keys file. If omitted, reads keys from stdin"
         alias: k
       - name: output
         description: Output file path. Defaults to stdout (for piping)

src/utils/options.ts

@@ -73,3 +73,15 @@ export function parseBoolean(
   if (typeof value === 'boolean') return value;
   return value === 'true';
 }
+
+/**
+ * Read all of stdin as a UTF-8 string.
+ * Use when stdin is piped (i.e. `!process.stdin.isTTY`).
+ */
+export async function readStdin(): Promise<string> {
+  const chunks: Buffer[] = [];
+  for await (const chunk of process.stdin) {
+    chunks.push(chunk);
+  }
+  return Buffer.concat(chunks).toString('utf-8');
+}