chore: fixed edit function in cli and Delete numerous miscellaneous files from the node_modules directory.

Author: snip-cli

STRATEGY.md

@@ -1,18 +1,31 @@
+const fs = require('fs');
+const os = require('os');
+const path = require('path');
 const storage = require('../lib/storage');
 const cfg = require('../lib/config');
 const editCmd = require('../lib/commands/edit');
 const rmCmd = require('../lib/commands/rm');
 
 describe('edit and rm', () => {
-  test('edit updates updatedAt and rm removes snippet', () => {
+  test('edit detects no-change with noop editor, and rm removes snippet', () => {
     const s = storage.addSnippet({ name: 'editrm', content: 'before', language: 'txt', tags: [] });
-    const before = s.updatedAt;
-    cfg.saveConfig({ editor: 'true' }); // use shell `true` as noop editor
+    cfg.saveConfig({ editor: 'true' }); // noop editor — file unchanged
     editCmd(s.id);
-    const updated = storage.getSnippetByIdOrName(s.id);
-    expect(updated.updatedAt).not.toBe(before);
+    // Content unchanged → updatedAt should NOT change
+    const unchanged = storage.getSnippetByIdOrName(s.id);
+    expect(unchanged.updatedAt).toBe(s.updatedAt);
     rmCmd(s.id);
     const after = storage.getSnippetByIdOrName(s.id);
     expect(after).toBeNull();
   });
+
+  test('edit updates content when file changes', () => {
+    const s = storage.addSnippet({ name: 'editchange', content: 'original', language: 'txt', tags: [] });
+    // Directly update the content via storage to simulate an edit
+    storage.updateSnippetContent(s.id, 'modified');
+    const updated = storage.getSnippetByIdOrName(s.id);
+    expect(updated.updatedAt).not.toBe(s.updatedAt);
+    // Cleanup
+    rmCmd(s.id);
+  });
 });

__tests__/rm_edit.test.js

@@ -9,14 +9,18 @@ function edit(idOrName) {
   const s = storage.getSnippetByIdOrName(idOrName);
   if (!s) return console.error('Snippet not found');
   const content = storage.readSnippetContent(s);
-  const editor = (config.loadConfig().editor || 'vi').split(' ');
+  const editor = (config.loadConfig().editor || process.env.EDITOR || 'vi').split(' ');
   const fileToEdit = s.path || path.join(os.tmpdir(), `snip-edit-${s.id}.tmp`);
   if (!s.path) fs.writeFileSync(fileToEdit, content, 'utf8');
   spawnSync(editor[0], editor.slice(1).concat([fileToEdit]), { stdio: 'inherit' });
   const newContent = fs.readFileSync(fileToEdit, 'utf8');
-  if (!s.path) fs.unlinkSync(fileToEdit);
-  storage.updateSnippetContent(s.id, newContent);
-  console.log('Updated', s.id);
+  if (!s.path) try { fs.unlinkSync(fileToEdit); } catch (e) { }
+  if (newContent !== content) {
+    storage.updateSnippetContent(s.id, newContent);
+    console.log(`Updated "${s.name}"`);
+  } else {
+    console.log('No changes.');
+  }
 }
 
 module.exports = edit;