rename no-rollback to skip-rollback

Author: mikolalysenko

src/commands/remove.test.ts

@@ -83,7 +83,7 @@ describe('remove command with rollback', () => {
     assert.equal(updatedManifest.patches['pkg:npm/[email protected]'], undefined)
   })
 
-  it('should allow removal without rollback using --no-rollback flag', async () => {
+  it('should allow removal without rollback using --skip-rollback flag', async () => {
     const { manifestPath, packageDirs } = await setupTestEnvironment({
       testDir: path.join(testDir, 'norollback1'),
       patches: [
@@ -103,7 +103,7 @@ describe('remove command with rollback', () => {
 
     const pkgDir = packageDirs.get('pkg:npm/[email protected]')!
 
-    // Simulate --no-rollback: only remove from manifest without rollback
+    // Simulate --skip-rollback: only remove from manifest without rollback
     const manifestContent = await fs.readFile(manifestPath, 'utf-8')
     const manifest = PatchManifestSchema.parse(JSON.parse(manifestContent))
 

src/commands/remove.ts

@@ -16,7 +16,7 @@ interface RemoveArgs {
   identifier: string
   cwd: string
   'manifest-path': string
-  'no-rollback': boolean
+  'skip-rollback': boolean
 }
 
 async function removePatch(
@@ -83,7 +83,7 @@ export const removeCommand: CommandModule<{}, RemoveArgs> = {
         type: 'string',
         default: DEFAULT_PATCH_MANIFEST_PATH,
       })
-      .option('no-rollback', {
+      .option('skip-rollback', {
         describe: 'Skip rolling back files before removing (only update manifest)',
         type: 'boolean',
         default: false,
@@ -97,7 +97,7 @@ export const removeCommand: CommandModule<{}, RemoveArgs> = {
         'Rollback and remove a patch by UUID',
       )
       .example(
-        '$0 remove pkg:npm/[email protected] --no-rollback',
+        '$0 remove pkg:npm/[email protected] --skip-rollback',
         'Remove from manifest without rolling back files',
       )
   },
@@ -116,7 +116,7 @@ export const removeCommand: CommandModule<{}, RemoveArgs> = {
       }
 
       // First, rollback the patch if not skipped
-      if (!argv['no-rollback']) {
+      if (!argv['skip-rollback']) {
         console.log(`Rolling back patch before removal...`)
         const { success: rollbackSuccess, results: rollbackResults } =
           await rollbackPatches(
@@ -129,7 +129,7 @@ export const removeCommand: CommandModule<{}, RemoveArgs> = {
 
         if (!rollbackSuccess) {
           console.error(
-            '\nRollback failed. Use --no-rollback to remove from manifest without restoring files.',
+            '\nRollback failed. Use --skip-rollback to remove from manifest without restoring files.',
           )
           process.exit(1)
         }