shilo-cli 0.11.0 → 0.12.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shilo-cli",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "CLI para sincronizar carpetas locales con repositorios de Shilo: push, pull, status, log, diff, sync y propuestas.",
   "keywords": [
     "shilo",

package/src/commands/propuestas.js

@@ -9,6 +9,7 @@ const sync = require('../sync');
 const prompt = require('../prompt');
 const textdiff = require('../textdiff');
 const { resolveContext } = require('../context');
+const { puedeAplicar } = require('../viewer');
 
 /**
  * shilo propuestas [ver|bajar|aplicar|descartar] [id] [opciones]
@@ -147,7 +148,7 @@ async function ver(ctx, { proposal: resumen, viewer }, args) {
       chequeos: d.chequeos,
       usee: d.usee,
       blocked_by: d.blockedBy,
-      puedo_aplicar: !!(viewer && viewer.is_owner),
+      puedo_aplicar: puedeAplicar(viewer),
       changes: d.changes.map((c) => ({
         ...c,
         ...(args.contenido ? { contenido_base: texto(c.base), contenido_propuesta: texto(c.head) } : {}),
@@ -222,7 +223,7 @@ async function ver(ctx, { proposal: resumen, viewer }, args) {
     ui.info('');
     const id8 = p.id.slice(0, 8);
     if (!args.diff) ui.info(ui.c.dim(`Diferencias:  shilo propuestas ver ${id8} --diff`));
-    if (viewer && viewer.is_owner) {
+    if (puedeAplicar(viewer)) {
       ui.info(ui.c.dim(d.conflicts.length > 0
         ? `Resolver:     shilo propuestas aplicar ${id8} --propuesta <ruta> --actual <ruta> (o --todo propuesta)`
         : `Aplicar:      shilo propuestas aplicar ${id8}`));
@@ -362,8 +363,8 @@ async function aplicar(ctx, { proposal: resumen, viewer }, args) {
     ui.error(`Esa propuesta ya está ${resumen.state}.`);
     return 1;
   }
-  if (viewer && !viewer.is_owner) {
-    ui.error('Sólo el dueño del proyecto puede aplicar una propuesta.');
+  if (viewer && !puedeAplicar(viewer)) {
+    ui.error('Para aplicar una propuesta hace falta ser el dueño o tener el permiso de aprobar propuestas.');
     return 1;
   }
 

package/src/commands/push.js

@@ -22,6 +22,7 @@ const {
   MAX_FILE_BYTES,
 } = require('../textguard');
 const { ShiloClient } = require('../ShiloClient');
+const { puedeAplicar } = require('../viewer');
 
 const DEFAULT_BASE_URL = 'https://www.shilo.city';
 const COMMIT_BATCH = 100; // máximo de archivos por commit (límite del backend, camino anterior)
@@ -1412,11 +1413,12 @@ async function push(args) {
 
   config.persistCredsIfChanged(cfg, mgr.current);
 
-  // 9b) Si quien sube es el dueño, lo que quedó abierto lo puede aplicar ahora
-  //     mismo en vez de ir a la web a aprobarse a sí mismo.
+  // 9b) Si quien sube puede aprobar (dueño o permiso `approve_proposals`), lo
+  //     que quedó abierto lo puede aplicar ahora mismo en vez de ir a la web a
+  //     aprobarse a sí mismo.
   const failures = failedWrites.length + plan.failed.length;
   let openShown = false;
-  if (open.length > 0 && failures === 0 && viewer && viewer.is_owner && !args.yes && process.stdin.isTTY) {
+  if (open.length > 0 && failures === 0 && puedeAplicar(viewer) && !args.yes && process.stdin.isTTY) {
     const res = await offerOwnerApply(open);
     if (res.applied.length > 0) {
       committed += res.applied.reduce((n, o) => n + o.paths.length, 0);
@@ -1492,7 +1494,7 @@ async function push(args) {
   const pendingFiles = open.reduce((n, o) => n + o.paths.length, 0);
   const owner = projectInfo && projectInfo.owner;
   const ownerName = owner ? owner.display_name || owner.username : null;
-  const approver = viewer && viewer.is_owner ? 'tu aprobación' : ownerName ? `la aprobación de ${ownerName}` : 'la aprobación del dueño del proyecto';
+  const approver = puedeAplicar(viewer) ? 'tu aprobación' : ownerName ? `la aprobación de ${ownerName}` : 'la aprobación del dueño del proyecto';
   ui.success(
     `Tus ${pendingFiles} cambio(s) están guardados en Shilo y esperan ${approver}.` +
       (committed > 0 ? ` (Otros ${committed} ya son parte de la versión actual.)` : '')
@@ -1516,7 +1518,8 @@ async function push(args) {
     ui.info('');
     for (const o of list) reportOpen(o);
     openShown = true;
-    ui.info(`Sos el dueño del proyecto: podés aplicar estos ${files} cambio(s) ahora, sin pasar por la web.`);
+    const quien = viewer && viewer.is_owner ? 'Sos el dueño del proyecto' : 'Tenés permiso de aprobar propuestas';
+    ui.info(`${quien}: podés aplicar estos ${files} cambio(s) ahora, sin pasar por la web.`);
     if (conflicted > 0) {
       ui.info(
         `En ${conflicted} archivo(s) que también cambiaron en el proyecto se va a quedar ` +

package/src/commands/status.js

@@ -5,6 +5,7 @@ const ui = require('../ui');
 const sync = require('../sync');
 const remote = require('../remote');
 const { resolveContext } = require('../context');
+const { puedeAplicar } = require('../viewer');
 
 // A partir de cuántos días sin sincronizar se recuerda hacerlo.
 const STALE_DAYS = 30;
@@ -126,11 +127,11 @@ async function status(args) {
     };
   };
 
-  // 4b) Propuestas abiertas: las mías (esperan aprobación) y, si soy el dueño,
-  //     las de los demás (esperan mi decisión).
+  // 4b) Propuestas abiertas: las mías (esperan aprobación) y, si puedo aprobar
+  //     (dueño o permiso `approve_proposals`), las de los demás.
   const openProposals = listed ? listed.proposals : [];
   const mineOpen = me ? openProposals.filter((p) => p.author_id === me) : [];
-  const othersOpen = viewer && viewer.is_owner ? openProposals.filter((p) => p.author_id !== me) : [];
+  const othersOpen = puedeAplicar(viewer) ? openProposals.filter((p) => p.author_id !== me) : [];
   const projectInfo = (listed && listed.project) || null;
   const owner = projectInfo && projectInfo.owner;
   const ownerName = owner ? owner.display_name || owner.username : null;
@@ -217,7 +218,7 @@ async function status(args) {
   /** Propuestas abiertas: lo primero que hay que saber, antes que la lista de archivos. */
   const reportProposals = () => {
     if (mineOpen.length > 0) {
-      const who = viewer && viewer.is_owner ? 'tu aprobación' : ownerName ? `la aprobación de ${ownerName}` : 'aprobación';
+      const who = puedeAplicar(viewer) ? 'tu aprobación' : ownerName ? `la aprobación de ${ownerName}` : 'aprobación';
       ui.info(ui.c.bold(`Tenés ${mineOpen.length} propuesta(s) esperando ${who}:`));
       for (const p of mineOpen) {
         const link = linkTo(p);

package/src/viewer.js

@@ -0,0 +1,17 @@
+'use strict';
+
+/**
+ * Si quien pregunta puede aplicar propuestas en el proyecto: el dueño o un
+ * colaborador con el permiso `approve_proposals` (migración 021).
+ *
+ * Lo dice el servidor en `viewer.can_apply`. Un servidor anterior a la 021 no
+ * lo manda y sólo el dueño podía aplicar, así que sin el campo se usa
+ * `is_owner`. Es sólo para mostrar u ofrecer: el servidor decide igual.
+ */
+function puedeAplicar(viewer) {
+  if (!viewer) return false;
+  if (typeof viewer.can_apply === 'boolean') return viewer.can_apply;
+  return !!viewer.is_owner;
+}
+
+module.exports = { puedeAplicar };