orquestra-mcp 1.16.0 → 1.16.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "orquestra-mcp",
-  "version": "1.16.0",
+  "version": "1.16.1",
   "description": "Servidor MCP de Orquestra -- 89 tools para leer y escribir tareas, ideas, changelog, infraestructura, contenido, QA y más desde un agente de IA",
   "type": "module",
   "bin": {

package/src/platformPlaybooks.mjs

@@ -71,6 +71,11 @@ export const PLATFORM_PLAYBOOKS = {
         title: 'Eliminar cuenta desde la app (Guideline 5.1.1(v))',
         body: 'Obligatorio desde 2022: si la app permite crear cuenta, tiene que permitir ELIMINARLA desde DENTRO de la app -- "escríbenos por correo" no cumple el requisito, aunque sea rápido y real. Verifica que exista un flujo de borrado self-service (botón "Eliminar cuenta" en Perfil/Ajustes, con confirmación) ANTES de enviar a revisión -- si no existe, Apple puede rechazar la app durante el review, incluso si el resto de la metadata está perfecta. Orquestra ya tiene este flujo (Zona de riesgo en Perfil, backend server-side que borra cuenta+datos), úsalo de referencia de implementación si tu app lo necesita desde cero.',
       },
+      {
+        key: 'sign_in_apple',
+        title: 'Sign in with Apple si hay login de terceros (Guideline 4.8)',
+        body: 'Si la app ofrece login con Google, Facebook u otro proveedor de terceros, Apple exige que TAMBIÉN ofrezcas "Sign in with Apple" como alternativa equivalente -- Guideline 4.8 ("Login Services"). No es una recomendación: si falta, Apple RECHAZA el build completo en revisión (motivo real, no hipotético) sin importar que el resto de la metadata esté perfecta. Implementación de referencia: botón nativo (`AuthenticationServices`, nonce) + capability "Sign in with Apple" habilitada en el App ID (Apple Developer Portal) + provider "Apple" activado en Firebase Auth si se usa Firebase. Si el login es solo email/password propio, sin ningún proveedor de terceros, esta regla no aplica.',
+      },
     ],
 
     tasks: [
@@ -114,6 +119,11 @@ export const PLATFORM_PLAYBOOKS = {
         description: 'Guideline 5.1.1(v), obligatorio -- "escríbenos por correo" NO cumple el requisito. Tiene que haber un botón real de "Eliminar cuenta" (con confirmación) en Perfil/Ajustes que borre la cuenta y los datos sin salir de la app. Si no existe, constrúyelo ANTES de mandar a revisión, no después de un rechazo.',
         type: 'security', priority: 'high', tags: ['publicacion', 'ios', 'compliance'], offsetDays: 1,
       },
+      {
+        title:       'Verificar que si hay login de terceros (Google/Facebook/etc.), también esté "Sign in with Apple"',
+        description: 'Guideline 4.8, obligatorio -- si la app ofrece login con Google, Facebook u otro proveedor externo, Apple rechaza la app en revisión si no ofrece también "Sign in with Apple" como alternativa. No aplica si el login es solo email/password propio, sin proveedores de terceros. Si falta, agrégalo ANTES de mandar a revisión, no después de un rechazo.',
+        type: 'security', priority: 'high', tags: ['publicacion', 'ios', 'compliance'], offsetDays: 1,
+      },
     ],
   },
 
@@ -190,6 +200,11 @@ export const PLATFORM_PLAYBOOKS = {
         title: 'App access / Sign-in details',
         body: 'Si la app requiere login, "Add sign in details" pide un set de credenciales de prueba REALES (nombre descriptivo del set, usuario/email, contraseña) que el reviewer va a usar de verdad para entrar -- tienen que funcionar y, si es posible, mostrar contenido de ejemplo real (no una cuenta vacía cayendo en un onboarding). Marca el checkbox de "esta cuenta da acceso completo a todas las funciones" si no hay niveles pagos/premium.',
       },
+      {
+        key: 'google_signin_sha1',
+        title: 'Google Sign-In: SHA-1 de las TRES firmas en Firebase (debug, upload key y Play App Signing)',
+        body: 'Si la app tiene "Continuar con Google", Google valida la firma del APK contra los SHA-1 registrados en Firebase → Configuración del proyecto → app Android → "Agregar huella digital". Son TRES firmas distintas y cada una necesita su SHA-1: (1) debug (`~/.android/debug.keystore`, builds locales), (2) upload key (el keystore con el que firmas el AAB, `assembleRelease`), y (3) Play App Signing -- la clave con la que Google RE-FIRMA los APK que instalan testers y usuarios desde Play; NO es la upload key, y es la que siempre se olvida. Se ve en Play Console → Test and release → App integrity → Play Store protection → Play app signing → "App signing key certificate" (columna Classical key). Las dos primeras salen de `./gradlew signingReport`. Síntoma si falta alguna: al elegir la cuenta "no pasa nada" -- Play Services lo reporta como cancelación del usuario, no como error de configuración, así que no hay ningún mensaje que te avise. Después de registrarlas, vuelve a descargar google-services.json.',
+      },
       {
         key: 'delete_account',
         title: 'Eliminar cuenta desde la app (política de borrado de cuenta de Google Play)',
@@ -228,6 +243,11 @@ export const PLATFORM_PLAYBOOKS = {
         description: 'Política de borrado de cuenta de Google Play (equivalente a la Guideline 5.1.1(v) de Apple) -- lo ideal es un flujo real dentro de la app, no solo la "Delete account URL" de la ficha. Si no existe, constrúyelo antes de publicar.',
         type: 'security', priority: 'high', tags: ['publicacion', 'android', 'compliance'], offsetDays: 1,
       },
+      {
+        title:       'Registrar en Firebase el SHA-1 de las TRES firmas (debug, upload key y Play App Signing) si la app usa Google Sign-In',
+        description: 'Gotcha real: cada firma distinta del APK necesita su propio SHA-1 en Firebase (Configuración del proyecto → app Android → "Agregar huella digital"), y la que siempre se olvida es la de Play App Signing -- Google re-firma con ella los APK que se instalan desde Play, así que un login con Google que funciona en debug y en el release local falla en el build de los testers sin ningún mensaje (Play Services lo reporta como cancelación). Debug y upload key: `./gradlew signingReport`. Play App Signing: Play Console → App integrity → Play app signing → "App signing key certificate". Al terminar, re-descargar google-services.json.',
+        type: 'tech', priority: 'high', tags: ['publicacion', 'android', 'google-sign-in'], offsetDays: 1,
+      },
       {
         title:       'Preparar capturas de pantalla en los tamaños exactos que pide Play Console',
         description: 'Mínimo 2 capturas de teléfono, más el gráfico destacado (1024x500) y el ícono (512x512) -- Play Console valida las dimensiones al subir, ajusta según lo que rechace.',

package/src/server.mjs

@@ -1589,6 +1589,8 @@ export function createServer() {
     saves: z.number().optional().describe('Guardados -- entrada manual, típico de X/Instagram (solo con channelId suelto)'),
     avgReadMinutes: z.number().optional().describe('Minutos de lectura media -- entrada manual, típico de blog (solo con channelId suelto)'),
     imagePrompt: z.string().optional().describe('Descripción visual en inglés para generar la portada -- no genera la imagen, solo guarda el texto (solo con channelId suelto)'),
+    recommendedMedia: z.enum(['image', 'video']).optional().describe('Qué conviene más de portada para esta pieza -- puramente informativo, ningún flujo cambia solo por esto (solo con channelId suelto)'),
+    videoPrompt: z.string().optional().describe('Análogo a imagePrompt pero para video: descripción de la escena en movimiento, en inglés, para pegar en un generador de video externo -- Orquestra no genera video, solo guarda el texto (solo con channelId suelto)'),
     language: z.string().optional().describe('\'en\' si esta pieza es la traducción de otra. Sin valor = idioma original.'),
     translationOf: z.string().optional().describe('Si esta pieza es una traducción, id del content original'),
   })
@@ -1616,6 +1618,8 @@ export function createServer() {
     saves: z.number().optional(),
     avgReadMinutes: z.number().optional(),
     imagePrompt: z.string().optional(),
+    recommendedMedia: z.enum(['image', 'video']).optional(),
+    videoPrompt: z.string().optional(),
     language: z.string().optional(),
     translationOf: z.string().optional(),
   })
@@ -1812,7 +1816,7 @@ export function createServer() {
   server.registerTool(
     'apply_platform_playbook',
     {
-      description: 'Aplica el Platform Playbook de publicación (iOS App Store / Google Play) sobre una distribución real: crea el módulo + tareas de la plantilla (igual que apply_stage_playbook) y, si mandás publishForm, también completa el formulario de la Guía de publicación de esa distribución (mismo dato que llena DistributionGuide.jsx en la web). Solo completá campos para los que tengas una base real del proyecto (código, get_project_context, servicios/entornos configurados) -- nunca inventes pricing, availability, package/bundle ID, cuentas demo para el reviewer, o el nombre exacto de la app; dejalos afuera de publishForm para que el usuario los complete a mano. Si la distribución YA tiene valores guardados en alguna de las claves que mandás, la tool NO los sobreescribe en silencio -- devuelve publishFormConflict con el valor actual vs. el propuesto por cada clave en conflicto, sin tocar publishForm. Mostrale ese conflicto al usuario y volvé a llamar con confirmReplace:true SOLO si confirma explícitamente el reemplazo.',
+      description: 'Aplica el Platform Playbook de publicación (iOS App Store / Google Play) sobre una distribución real: crea el módulo + tareas de la plantilla (igual que apply_stage_playbook) y, si mandás publishForm, también completa el formulario de la Guía de publicación de esa distribución (mismo dato que llena DistributionGuide.jsx en la web). Solo completá campos para los que tengas una base real del proyecto (código, get_project_context, servicios/entornos configurados) -- nunca inventes pricing, availability, package/bundle ID, cuentas demo para el reviewer, o el nombre exacto de la app; dejalos afuera de publishForm para que el usuario los complete a mano. Si la distribución YA tiene valores guardados en alguna de las claves que mandás, la tool NO los sobreescribe en silencio -- devuelve publishFormConflict con el valor actual vs. el propuesto por cada clave en conflicto, sin tocar publishForm. Mostrale ese conflicto al usuario y volvé a llamar con confirmReplace:true SOLO si confirma explícitamente el reemplazo. Si este playbook YA se aplicó antes sobre esta misma distribución (módulo+tareas ya creados), la tool lo detecta sola (mismo mecanismo que el badge "Ya aplicado" de DistributionGuide.jsx) y NO vuelve a crear módulo/tareas duplicados -- devuelve alreadyApplied:true. Volvé a llamar con force:true SOLO si el usuario pide explícitamente un checklist nuevo encima del que ya existe.',
       inputSchema: {
         projectId: z.string().optional().describe(PROJECT_ID_DESC),
         platform: z.enum(['ios', 'android']).describe('ios (App Store Connect) | android (Google Play Console) -- ver get_platform_playbook para el contenido'),
@@ -1820,40 +1824,58 @@ export function createServer() {
         moduleName: z.string().optional().describe('Nombre del módulo -- default: el de la plantilla'),
         publishForm: z.record(z.string()).optional().describe('Respuestas a completar, clave = key de referenceGuide (ver get_platform_playbook). Omitido o vacío = solo crea módulo + tareas, sin tocar el formulario.'),
         confirmReplace: z.boolean().optional().describe('Default false. Mandá true solo cuando el usuario ya confirmó explícitamente reemplazar valores existentes que te devolvió publishFormConflict en un llamado anterior.'),
+        force: z.boolean().optional().describe('Default false. Si este playbook ya se aplicó antes sobre esta distribución, por default la tool NO crea un segundo módulo/checklist -- mandá true solo cuando el usuario pidió explícitamente crear uno nuevo encima del que ya existe.'),
       },
     },
-    async ({ projectId, platform, distributionId, moduleName, publishForm, confirmReplace }) => {
+    async ({ projectId, platform, distributionId, moduleName, publishForm, confirmReplace, force }) => {
       try {
         const pid = resolveProjectId(projectId)
         const playbook = PLATFORM_PLAYBOOKS[platform]
-        const moduleData = await apiPost(config, '/v1/modules', {
-          projectId: pid,
-          name: moduleName || playbook.moduleName,
-          description: playbook.moduleDescription,
-          tags: playbook.moduleTags,
-        })
+        const appliedKey = `${distributionId}_publish`
 
-        const taskIds = []
-        for (const t of playbook.tasks) {
-          const dueDate = new Date(Date.now() + t.offsetDays * 86400000).toISOString().slice(0, 10)
-          const created = await apiPost(config, '/v1/tasks', {
+        const project = await apiGet(config, `/v1/projects/${pid}`, {})
+        const alreadyApplied = !!(project.appliedPlaybooks || {})[appliedKey]
+
+        const result = {}
+
+        if (alreadyApplied && !force) {
+          result.alreadyApplied = true
+          result.message = 'Este playbook ya se había aplicado antes sobre esta distribución (el módulo y sus tareas ya existen en el proyecto) -- no se creó nada de nuevo para no duplicarlo. Si el usuario pide explícitamente un checklist nuevo, volvé a llamar con force:true.'
+        } else {
+          const moduleData = await apiPost(config, '/v1/modules', {
             projectId: pid,
-            title: t.title,
-            description: t.description,
-            type: t.type,
-            priority: t.priority,
-            tags: t.tags,
-            moduleId: moduleData.id,
-            dueDate,
+            name: moduleName || playbook.moduleName,
+            description: playbook.moduleDescription,
+            tags: playbook.moduleTags,
           })
-          taskIds.push(created.id)
-        }
 
-        const result = { module: moduleData, tasksCreated: taskIds.length, taskIds }
+          const taskIds = []
+          for (const t of playbook.tasks) {
+            const dueDate = new Date(Date.now() + t.offsetDays * 86400000).toISOString().slice(0, 10)
+            const created = await apiPost(config, '/v1/tasks', {
+              projectId: pid,
+              title: t.title,
+              description: t.description,
+              type: t.type,
+              priority: t.priority,
+              tags: t.tags,
+              moduleId: moduleData.id,
+              dueDate,
+            })
+            taskIds.push(created.id)
+          }
+
+          result.module = moduleData
+          result.tasksCreated = taskIds.length
+          result.taskIds = taskIds
+
+          await apiPatch(config, '/v1/projects', { id: pid, appliedPlaybooks: { [appliedKey]: true } })
+          result.markedApplied = true
+        }
 
         if (publishForm && Object.keys(publishForm).length > 0) {
           const distributions = await apiGet(config, '/v1/distributions', { projectId: pid })
-          const dist = distributions.find(d => d.id === distributionId)
+          const dist = distributions.items.find(d => d.id === distributionId)
           if (!dist) {
             throw new Error(`No se encontró la distribución "${distributionId}" en este proyecto -- revisá list_distributions.`)
           }