fix: restore --force flag and remove -f alias from --format (#54)

* fix: restore --force flag on destructive commands and remove -f alias from --format

The previous refactor (auto-inject global arguments) silently changed the
meaning of -f from --force to --format on rm/mv commands. This restores
--force/-f on rm and mv (supporting the -rf convention), adds --force
(without -f alias) back to buckets delete, objects delete, access-keys
delete, iam policies delete, iam users remove, and iam users
revoke-invitation for backwards compatibility, and removes -f as an
alias for --format globally.

Also excludes auto-generated command-registry.ts from eslint.

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

* ci: enable workflow_dispatch for integration tests

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

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: designcode

.github/workflows/ci.yaml

@@ -8,6 +8,7 @@ on:
   push:
     branches:
       - main
+  workflow_dispatch:
 
 jobs:
   build:
@@ -28,7 +29,7 @@ jobs:
 
   integration:
     runs-on: ubuntu-latest
-    if: github.event_name == 'push'
+    if: github.event_name == 'push' || github.event_name == 'workflow_dispatch'
     steps:
       - uses: actions/checkout@v4
         with:

eslint.config.js

@@ -24,6 +24,6 @@ export default tseslint.config(
     },
   },
   {
-    ignores: ["dist/", "node_modules/", "**/*.cjs"],
+    ignores: ["dist/", "node_modules/", "**/*.cjs", "src/command-registry.ts"],
   },
 );

src/lib/access-keys/delete.ts

@@ -13,7 +13,7 @@ export default async function remove(options: Record<string, unknown>) {
   const format = getFormat(options);
 
   const id = getOption<string>(options, ['id']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   if (!id) {
     failWithError(context, 'Access key ID is required');

src/lib/buckets/delete.ts

@@ -23,7 +23,7 @@ export default async function deleteBucket(options: Record<string, unknown>) {
   const format = getFormat(options);
 
   const names = getOption<string | string[]>(options, ['name']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   if (!names) {
     failWithError(context, 'Bucket name is required');

src/lib/iam/policies/delete.ts

@@ -15,7 +15,7 @@ export default async function del(options: Record<string, unknown>) {
   const format = getFormat(options);
 
   let resource = getOption<string>(options, ['resource']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   const iamConfig = await getOAuthIAMConfig(context);
 

src/lib/iam/users/remove.ts

@@ -15,7 +15,7 @@ export default async function removeUser(options: Record<string, unknown>) {
   const format = getFormat(options);
 
   const resourceOption = getOption<string | string[]>(options, ['resource']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   if (isFlyOrganization()) return;
 

src/lib/iam/users/revoke-invitation.ts

@@ -17,7 +17,7 @@ export default async function revokeInvitation(
   const format = getFormat(options);
 
   const resourceOption = getOption<string | string[]>(options, ['resource']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   if (isFlyOrganization()) return;
 

src/lib/mv.ts

@@ -19,7 +19,7 @@ let _jsonMode = false;
 export default async function mv(options: Record<string, unknown>) {
   const src = getOption<string>(options, ['src']);
   const dest = getOption<string>(options, ['dest']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force', 'f']);
   const recursive = !!getOption<boolean>(options, ['recursive', 'r']);
   const format = getFormat(options);
   _jsonMode = format === 'json';

src/lib/objects/delete.ts

@@ -24,7 +24,7 @@ export default async function deleteObject(options: Record<string, unknown>) {
 
   const bucket = getOption<string>(options, ['bucket']);
   const keys = getOption<string | string[]>(options, ['key']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force']);
 
   if (!bucket) {
     failWithError(context, 'Bucket name is required');

src/lib/rm.ts

@@ -16,7 +16,7 @@ let _jsonMode = false;
 
 export default async function rm(options: Record<string, unknown>) {
   const pathString = getOption<string>(options, ['path']);
-  const force = getOption<boolean>(options, ['yes', 'y']);
+  const force = getOption<boolean>(options, ['yes', 'y', 'force', 'f']);
   const recursive = !!getOption<boolean>(options, ['recursive', 'r']);
   const format = getFormat(options);
   _jsonMode = format === 'json';