chore: pr comments

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: designcode

.github/workflows/pr.yaml .github/workflows/ci.yaml.github/workflows/pr.yaml renamed to .github/workflows/ci.yaml

@@ -1,10 +1,13 @@
-name: Pull Request
+name: CI
 
 on:
   pull_request:
     branches:
       - main
       - next
+  push:
+    branches:
+      - main
 
 jobs:
   build:
@@ -25,8 +28,7 @@ jobs:
 
   integration:
     runs-on: ubuntu-latest
-    if: github.event.pull_request.base.ref == 'main'
-    environment: integration
+    if: github.event_name == 'push'
     steps:
       - uses: actions/checkout@v4
         with:

package.json

@@ -32,7 +32,6 @@
     "test": "vitest run test/utils test/cli-core.test.ts test/specs-completeness.test.ts",
     "test:watch": "vitest",
     "test:all": "vitest run",
-    "test:unit": "vitest run test/utils test/cli-core.test.ts test/specs-completeness.test.ts",
     "test:integration": "vitest run test/cli.test.ts",
     "publint": "publint",
     "updatedocs": "tsx scripts/update-docs.ts",

src/lib/buckets/delete.ts

@@ -39,20 +39,25 @@ export default async function deleteBucket(options: Record<string, unknown>) {
     }
   }
 
-  const results: string[] = [];
+  const deleted: string[] = [];
+  const errors: { name: string; error: string }[] = [];
   for (const name of bucketNames) {
     const { error } = await removeBucket(name, { config });
 
     if (error) {
       printFailure(context, error.message, { name });
-      process.exit(1);
+      errors.push({ name, error: error.message });
+    } else {
+      deleted.push(name);
+      printSuccess(context, { name });
     }
-
-    results.push(name);
-    printSuccess(context, { name });
   }
 
   if (format === 'json') {
-    console.log(JSON.stringify({ action: 'deleted', names: results }));
+    console.log(JSON.stringify({ action: 'deleted', names: deleted, errors }));
+  }
+
+  if (errors.length > 0) {
+    process.exit(1);
   }
 }

src/lib/cp.ts

@@ -435,7 +435,7 @@ async function copyLocalToRemote(
       destParsed.bucket,
       destKey,
       config,
-      true
+      !_jsonMode
     );
     if (result.error) {
       console.error(result.error);
@@ -602,7 +602,7 @@ async function copyRemoteToLocal(
       srcParsed.path,
       localFilePath,
       config,
-      true
+      !_jsonMode
     );
     if (result.error) {
       console.error(result.error);
@@ -831,7 +831,7 @@ async function copyRemoteToRemote(
       srcParsed.path,
       destParsed.bucket,
       destKey,
-      true
+      !_jsonMode
     );
 
     if (result.error) {
@@ -870,7 +870,7 @@ export default async function cp(options: Record<string, unknown>) {
   const jsonFlag = getOption<boolean>(options, ['json']);
   const format = jsonFlag
     ? 'json'
-    : getOption<string>(options, ['format', 'f', 'F'], 'table');
+    : getOption<string>(options, ['format'], 'table');
   _jsonMode = format === 'json';
 
   const direction = detectDirection(src, dest);

src/lib/mv.ts

@@ -23,7 +23,7 @@ export default async function mv(options: Record<string, unknown>) {
   const jsonFlag = getOption<boolean>(options, ['json']);
   const format = jsonFlag
     ? 'json'
-    : getOption<string>(options, ['format', 'f', 'F'], 'table');
+    : getOption<string>(options, ['format'], 'table');
   _jsonMode = format === 'json';
 
   if (!src || !dest) {
@@ -289,7 +289,7 @@ export default async function mv(options: Record<string, unknown>) {
       srcPath.path,
       destPath.bucket,
       destKey,
-      true // show progress for single file
+      !_jsonMode // show progress for single file (not in JSON mode)
     );
 
     if (result.error) {

src/lib/objects/delete.ts

@@ -48,6 +48,7 @@ export default async function deleteObject(options: Record<string, unknown>) {
   }
 
   const deleted: string[] = [];
+  const errors: { key: string; error: string }[] = [];
   for (const key of keyList) {
     const { error } = await remove(key, {
       config: {
@@ -58,14 +59,20 @@ export default async function deleteObject(options: Record<string, unknown>) {
 
     if (error) {
       printFailure(context, error.message, { key });
-      process.exit(1);
+      errors.push({ key, error: error.message });
+    } else {
+      deleted.push(key);
+      printSuccess(context, { key });
     }
-
-    deleted.push(key);
-    printSuccess(context, { key });
   }
 
   if (format === 'json') {
-    console.log(JSON.stringify({ action: 'deleted', bucket, keys: deleted }));
+    console.log(
+      JSON.stringify({ action: 'deleted', bucket, keys: deleted, errors })
+    );
+  }
+
+  if (errors.length > 0) {
+    process.exit(1);
   }
 }

src/lib/rm.ts

@@ -20,7 +20,7 @@ export default async function rm(options: Record<string, unknown>) {
   const jsonFlag = getOption<boolean>(options, ['json']);
   const format = jsonFlag
     ? 'json'
-    : getOption<string>(options, ['format', 'f', 'F'], 'table');
+    : getOption<string>(options, ['format'], 'table');
   _jsonMode = format === 'json';
 
   if (!pathString) {