@myvillage/cli 1.73.0 → 1.74.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myvillage/cli",
-  "version": "1.73.0",
+  "version": "1.74.0",
   "description": "MyVillageOS CLI for community developers",
   "type": "module",
   "bin": {

package/src/commands/dataset.js

@@ -19,6 +19,8 @@ import {
   ingestAsset,
   readRows,
   freezeVersion,
+  deleteRow,
+  updateRow,
 } from '../utils/dataset-workspace.js';
 
 // ── create ─────────────────────────────────────────────
@@ -176,6 +178,55 @@ export async function datasetFreezeCommand(slug, options) {
   }
 }
 
+// ── delete a row ───────────────────────────────────────
+
+export async function datasetRowDeleteCommand(options) {
+  try {
+    const slug = options.dataset;
+    if (!slug) throw new Error('--dataset <slug> is required.');
+    if (!options.id) throw new Error('--id <rowId> is required.');
+    const result = deleteRow(slug, options.id);
+    if (options.json) {
+      console.log(JSON.stringify(result, null, 2));
+      return;
+    }
+    success(`Deleted row ${options.id} from ${slug}`);
+    info(`  total: ${result.stats.rowCount} (train ${result.stats.bySplit.train} / val ${result.stats.bySplit.val})`);
+  } catch (e) {
+    error(e.message);
+    process.exitCode = 1;
+  }
+}
+
+// ── update a row ───────────────────────────────────────
+
+export async function datasetRowUpdateCommand(options) {
+  try {
+    const slug = options.dataset;
+    if (!slug) throw new Error('--dataset <slug> is required.');
+    if (!options.id) throw new Error('--id <rowId> is required.');
+    const updates = {};
+    if (options.features !== undefined) updates.features = parseFeatures(options.features);
+    else if (options.text !== undefined) updates.features = options.text;
+    if (options.label !== undefined) updates.label = options.label;
+    if (options.target !== undefined) updates.target = Number(options.target);
+    if (options.split !== undefined) updates.split = options.split;
+    if (Object.keys(updates).length === 0) {
+      throw new Error('Nothing to update. Pass --text/--features, --label, --target, or --split.');
+    }
+    const result = updateRow(slug, options.id, updates);
+    if (options.json) {
+      console.log(JSON.stringify(result, null, 2));
+      return;
+    }
+    success(`Updated row ${options.id} in ${slug}`);
+    info(`  total: ${result.stats.rowCount} (train ${result.stats.bySplit.train} / val ${result.stats.bySplit.val})`);
+  } catch (e) {
+    error(e.message);
+    process.exitCode = 1;
+  }
+}
+
 // ── helpers ────────────────────────────────────────────
 
 function readRowsFromFile(path) {

package/src/index.js

@@ -124,6 +124,8 @@ import {
   datasetAppendCommand,
   datasetShowCommand,
   datasetFreezeCommand,
+  datasetRowDeleteCommand,
+  datasetRowUpdateCommand,
 } from './commands/dataset.js';
 import {
   modelTrainCommand,
@@ -900,6 +902,27 @@ export function run() {
     .option('--json', 'Output raw JSON')
     .action(datasetFreezeCommand);
 
+  datasetCmd
+    .command('rm-row')
+    .description('Delete a row from the working set (by id)')
+    .requiredOption('--dataset <slug>', 'Target dataset slug')
+    .requiredOption('--id <rowId>', 'Row id to delete')
+    .option('--json', 'Output raw JSON')
+    .action(datasetRowDeleteCommand);
+
+  datasetCmd
+    .command('update-row')
+    .description('Update a working-set row (by id)')
+    .requiredOption('--dataset <slug>', 'Target dataset slug')
+    .requiredOption('--id <rowId>', 'Row id to update')
+    .option('--text <text>', 'New text features')
+    .option('--features <value>', 'New features (string, or numeric vector for tabular)')
+    .option('--label <label>', 'New class label (classification)')
+    .option('--target <number>', 'New numeric target (regression)')
+    .option('--split <split>', 'train | val')
+    .option('--json', 'Output raw JSON')
+    .action(datasetRowUpdateCommand);
+
   // ── Studio Lab: model training (local, TF.js CPU engine) ───────────────
   const modelCmd = program
     .command('model')

package/src/utils/dataset-workspace.js

@@ -308,6 +308,93 @@ export function readRows(slug, { version, split, limit } = {}) {
   return out;
 }
 
+// ── Edit / delete working-set rows ─────────────────────
+//
+// Only the working set (current.jsonl) is mutable; frozen versions (v*.jsonl)
+// are immutable snapshots and are never touched here. Both ops rewrite the
+// whole working-set file and recompute manifest stats from the result, so the
+// manifest can never drift from the rows on disk.
+
+/** Recompute manifest stats (rowCount / bySplit / byLabel) from a row array. */
+function recomputeStats(rows) {
+  const stats = emptyStats();
+  for (const row of rows) {
+    stats.rowCount += 1;
+    if (stats.bySplit[row.split] != null) stats.bySplit[row.split] += 1;
+    if (row.label != null) {
+      stats.byLabel[row.label] = (stats.byLabel[row.label] || 0) + 1;
+    }
+  }
+  return stats;
+}
+
+/** Overwrite the working set with `rows` and refresh the manifest's stats. */
+function writeWorkingSet(slug, manifest, rows) {
+  const body = rows.map((r) => JSON.stringify(r)).join('\n');
+  writeFileSync(getCurrentDataFile(slug), rows.length ? body + '\n' : '');
+  manifest.stats = recomputeStats(rows);
+  return writeManifest(slug, manifest);
+}
+
+/**
+ * Delete one working-set row by id. Throws if the id isn't in current.jsonl
+ * (e.g. it only exists in a frozen version, which can't be edited).
+ * @returns {{datasetId:string, deleted:string, stats:object}}
+ */
+export function deleteRow(slug, rowId) {
+  const manifest = readManifest(slug);
+  if (!manifest) throw new Error(`Dataset "${slug}" not found.`);
+  if (!rowId) throw new Error('A row id is required.');
+  const rows = readRows(slug);
+  const idx = rows.findIndex((r) => r.id === rowId);
+  if (idx === -1) {
+    throw new Error(`Row "${rowId}" not found in the working set of "${slug}".`);
+  }
+  const [removed] = rows.splice(idx, 1);
+  writeWorkingSet(slug, manifest, rows);
+  return { datasetId: slug, deleted: removed.id, stats: manifest.stats };
+}
+
+/**
+ * Update one working-set row by id. `updates` is a partial
+ * {features, label, target, split}; the merged row is re-validated against the
+ * dataset's modality/task (same rules as append). The row id is preserved and
+ * provenance.editedAt is stamped. Image/audio asset refs can't be edited here —
+ * delete the row and re-add the asset instead.
+ * @returns {{datasetId:string, updated:string, stats:object}}
+ */
+export function updateRow(slug, rowId, updates = {}) {
+  const manifest = readManifest(slug);
+  if (!manifest) throw new Error(`Dataset "${slug}" not found.`);
+  if (!rowId) throw new Error('A row id is required.');
+  const rows = readRows(slug);
+  const idx = rows.findIndex((r) => r.id === rowId);
+  if (idx === -1) {
+    throw new Error(`Row "${rowId}" not found in the working set of "${slug}".`);
+  }
+  const existing = rows[idx];
+  if ((manifest.modality === 'image' || manifest.modality === 'audio') && updates.features !== undefined) {
+    throw new Error(`Can't edit the asset of an ${manifest.modality} row — delete it and re-add.`);
+  }
+  const merged = normalizeRow(
+    {
+      features: updates.features !== undefined ? updates.features : existing.features,
+      split: updates.split !== undefined ? updates.split : existing.split,
+      label: updates.label !== undefined ? updates.label : existing.label,
+      target: updates.target !== undefined ? updates.target : existing.target,
+    },
+    manifest,
+  );
+  merged.id = existing.id;
+  merged.provenance = { ...(existing.provenance || {}), editedAt: nowISO() };
+  if (merged.label != null && !manifest.labels.includes(merged.label)) {
+    manifest.labels.push(merged.label);
+  }
+  rows[idx] = merged;
+  writeWorkingSet(slug, manifest, rows);
+  return { datasetId: slug, updated: merged.id, stats: manifest.stats };
+}
+
 // ── Freeze a version ───────────────────────────────────
 
 /**