@codifycli/plugin-core 1.2.1-beta.1 → 1.2.2-beta.1

package/dist/messages/sender.d.ts

@@ -4,7 +4,7 @@
 declare class CodifyCliSenderImpl {
     private readonly validateIpcMessageV2;
     requestPressKeyToContinuePrompt(message?: string): Promise<void>;
-    sendApplyNote(message: string, resourceType?: string): Promise<void>;
+    sendApplyNote(message: string, resourceType?: string): void;
     getCodifyCliCredentials(): Promise<string>;
     private sendAndWaitForResponse;
 }

package/dist/messages/sender.js

@@ -17,16 +17,17 @@ class CodifyCliSenderImpl {
             }
         });
     }
-    async sendApplyNote(message, resourceType) {
+    sendApplyNote(message, resourceType) {
         if (!process.send || !process.connected) {
             return;
         }
-        await this.sendAndWaitForResponse({
+        process.send({
             cmd: MessageCmd.APPLY_NOTE_REQUEST,
             data: {
                 message,
                 ...(resourceType && { resourceType }),
-            }
+            },
+            requestId: nanoid(8),
         });
     }
     async getCodifyCliCredentials() {

package/dist/utils/file-utils.js

@@ -24,7 +24,7 @@ export class FileUtils {
         await FileUtils.createShellRcIfNotExists();
         const lineToInsert = addLeadingSpacer(addTrailingSpacer(line));
         await fs.appendFile(Utils.getPrimaryShellRc(), lineToInsert);
-        await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+        CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
         function addLeadingSpacer(line) {
             return line.startsWith('\n')
                 ? line
@@ -43,7 +43,7 @@ export class FileUtils {
         console.log(`Adding to ${path.basename(shellRc)}:
 ${lines.join('\n')}`);
         await fs.appendFile(shellRc, formattedLines);
-        await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+        CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
     }
     /**
      * This method adds a directory path to the shell rc file if it doesn't already exist.
@@ -64,7 +64,7 @@ ${lines.join('\n')}`);
         else {
             await fs.appendFile(shellRc, `\nexport PATH=${value}:$PATH;`, { encoding: 'utf8' });
         }
-        await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+        CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
     }
     static async removeFromFile(filePath, search) {
         const contents = await fs.readFile(filePath, 'utf8');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codifycli/plugin-core",
-  "version": "1.2.1-beta.1",
+  "version": "1.2.2-beta.1",
   "description": "TypeScript library for building Codify plugins to manage system resources (applications, CLI tools, settings) through infrastructure-as-code",
   "main": "dist/index.js",
   "typings": "dist/index.d.ts",

package/src/messages/sender.ts

@@ -27,17 +27,18 @@ class CodifyCliSenderImpl {
     })
   }
 
-  async sendApplyNote(message: string, resourceType?: string): Promise<void> {
+  sendApplyNote(message: string, resourceType?: string): void {
     if (!process.send || !process.connected) {
       return;
     }
 
-    await this.sendAndWaitForResponse(<IpcMessageV2>{
+    process.send(<IpcMessageV2>{
       cmd: MessageCmd.APPLY_NOTE_REQUEST,
       data: <ApplyNoteRequestData>{
         message,
         ...(resourceType && { resourceType }),
-      }
+      },
+      requestId: nanoid(8),
     });
   }
 

package/src/plan/change-set.test.ts

@@ -304,6 +304,33 @@ describe('Change set tests', () => {
     expect(cs.operation).to.eq(ResourceOperation.DESTROY);
   })
 
+  it('treats empty objects {} as no-op', () => {
+    const desired = {
+      keyboard: {},
+      dock: {},
+    }
+
+    const current = {
+      keyboard: {},
+      dock: {},
+    }
+
+    const cs = ChangeSet.calculateModification(desired, current);
+    expect(cs.parameterChanges.length).to.eq(0);
+    expect(cs.operation).to.eq(ResourceOperation.NOOP);
+  })
+
+  it('treats empty object {} as absent when current has no value', () => {
+    const desired = {
+      keyboard: {},
+      dock: {},
+    }
+
+    const cs = ChangeSet.calculateModification(desired, {});
+    expect(cs.parameterChanges.length).to.eq(0);
+    expect(cs.operation).to.eq(ResourceOperation.NOOP);
+  })
+
   it('correctly determines array equality 5', () => {
     const arrA = [{ key1: 'b' }, { key1: 'a' }, { key1: 'a' }];
     const arrB = [{ key1: 'a' }, { key1: 'a' }, { key1: 'b' }];

package/src/utils/file-utils.ts

@@ -35,7 +35,7 @@ export class FileUtils {
     );
 
     await fs.appendFile(Utils.getPrimaryShellRc(), lineToInsert)
-    await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+    CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
 
     function addLeadingSpacer(line: string): string {
       return line.startsWith('\n')
@@ -60,7 +60,7 @@ export class FileUtils {
 ${lines.join('\n')}`)
 
     await fs.appendFile(shellRc, formattedLines)
-    await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+    CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
   }
 
   /**
@@ -85,7 +85,7 @@ ${lines.join('\n')}`)
       await fs.appendFile(shellRc, `\nexport PATH=${value}:$PATH;`, { encoding: 'utf8' });
     }
 
-    await CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
+    CodifyCliSender.sendApplyNote(ApplyNotes.sourceShellRc());
   }
 
   static async removeFromFile(filePath: string, search: string): Promise<void> {