Merge pull request #243 from launchql/feat/cli-recur

Feat/cli recur

Author: pyramation

packages/cli/__tests__/cli-deploy-stage-unique-names.test.ts

@@ -23,7 +23,7 @@ describe('CLIDeployTestFixture Migrate', () => {
 
   it('should emulate terminal commands with database operations', async () => {
     const terminalCommands = `
-      lql deploy --recursive --database ${testDb.name} --yes --no-usePlan --package unique-names
+      lql deploy --database ${testDb.name} --yes --no-usePlan --package unique-names
     `;
 
     const results = await fixture.runTerminalCommands(terminalCommands, {
@@ -35,7 +35,7 @@ describe('CLIDeployTestFixture Migrate', () => {
 
   it('should emulate terminal commands with database operations usePlan', async () => {
     const terminalCommands = `
-      lql deploy --recursive --database ${testDb.name} --yes --usePlan --package unique-names
+      lql deploy --database ${testDb.name} --yes --usePlan --package unique-names
     `;
 
     const results = await fixture.runTerminalCommands(terminalCommands, {

packages/cli/__tests__/deploy.test.ts

@@ -90,7 +90,7 @@ describe('CLI Deploy Command', () => {
     expect(await testDb.exists('table', 'myfirstapp.users')).toBe(true);
     expect(await testDb.exists('table', 'myfirstapp.products')).toBe(true);
 
-    const revertCommands = `lql revert --database $database --yes`;
+    const revertCommands = `lql revert --database $database  --package my-first --yes`;
     await fixture.runTerminalCommands(revertCommands, {
       database: testDb.name
     }, true);
@@ -99,4 +99,22 @@ describe('CLI Deploy Command', () => {
     expect(await testDb.exists('table', 'myfirstapp.users')).toBe(false);
     expect(await testDb.exists('table', 'myfirstapp.products')).toBe(false);
   });
+
+
+  it('should deploy with --no-recursive flag', async () => {
+    const commands = `lql deploy --database ${testDb.name} --package my-first --no-recursive --yes`;
+    
+    await fixture.runTerminalCommands(commands, {
+      database: testDb.name
+    }, true);
+    
+    expect(await testDb.exists('schema', 'myfirstapp')).toBe(true);
+    expect(await testDb.exists('table', 'myfirstapp.users')).toBe(true);
+    expect(await testDb.exists('table', 'myfirstapp.products')).toBe(true);
+    
+    const deployedChanges = await testDb.getDeployedChanges();
+    expect(deployedChanges.some((change: any) => change.package === 'my-first')).toBe(true);
+    expect(deployedChanges.find((change: any) => change.package === 'my-second')).toBeFalsy();
+    expect(deployedChanges.find((change: any) => change.package === 'my-third')).toBeFalsy();
+  });
 });

packages/cli/src/commands/deploy.ts

@@ -78,6 +78,14 @@ export default async (
       message: 'Are you sure you want to proceed?',
       required: true
     },
+    {
+      name: 'recursive',
+      type: 'confirm',
+      message: 'Deploy recursively through dependencies?',
+      useDefault: true,
+      default: true,
+      required: false
+    },
     {
       name: 'tx',
       type: 'confirm',

packages/cli/src/commands/revert.ts

@@ -54,6 +54,14 @@ export default async (
       message: 'Are you sure you want to proceed?',
       required: true
     },
+    {
+      name: 'recursive',
+      type: 'confirm',
+      message: 'Deploy recursively through dependencies?',
+      useDefault: true,
+      default: true,
+      required: false
+    },
     {
       name: 'tx',
       type: 'confirm',
@@ -65,7 +73,7 @@ export default async (
   ];
 
   let { yes, recursive, cwd, tx } = await prompter.prompt(argv, questions);
-
+  
   if (!yes) {
     log.info('Operation cancelled.');
     return;

packages/cli/src/commands/verify.ts

@@ -45,7 +45,16 @@ export default async (
     message: 'Select database'
   });
 
-  const questions: Question[] = [];
+  const questions: Question[] = [
+    {
+      name: 'recursive',
+      type: 'confirm',
+      message: 'Deploy recursively through dependencies?',
+      useDefault: true,
+      default: true,
+      required: false
+    },
+  ];
 
   let { recursive, cwd } = await prompter.prompt(argv, questions);
 

packages/cli/test-utils/CLIDeployTestFixture.ts

@@ -217,7 +217,13 @@ export class CLIDeployTestFixture extends TestFixture {
 
       if (token.startsWith('--')) {
         const key = token.substring(2);
-        if (i + 1 < tokens.length && !tokens[i + 1].startsWith('--')) {
+        
+        // Handle --no-<thing> pattern by setting <thing>: false
+        if (key.startsWith('no-')) {
+          const baseKey = key.substring(3); // Remove 'no-' prefix
+          argv[baseKey] = false;
+          i += 1;
+        } else if (i + 1 < tokens.length && !tokens[i + 1].startsWith('--')) {
           argv[key] = tokens[i + 1];
           i += 2;
         } else {