From 285bd5c454f5acf6848d34b7421cf93c1c644dd1 Mon Sep 17 00:00:00 2001 From: Josep Piguillem Date: Wed, 21 Oct 2026 21:05:44 +0200 Subject: [PATCH] feat: what is new since my last visit, and turning a comment into a ticket Opening a ticket notes the visit and answers the one before (issue_visits), so the page marks others' comments and changes since. A comment can become a ticket of its own, as a subtask or linked; it remembers where it went. Co-Authored-By: Claude Opus 5.5 --- apps/api/openapi.json | 273 ++++++++++++++++++ apps/api/schema.gql | 26 ++ apps/mcp/src/app/api/taskstream.schema.ts | 220 ++++++++++++++ apps/mcp/src/app/tools/issues.resolver.ts | 31 ++ docs/01-vision-general.md | 90 +++--- docs/03-api.md | 34 ++- docs/04-cliente-web.md | 40 +-- docs/09-mcp.md | 1 + libs/api/tracker/application/src/index.ts | 4 + .../src/lib/commands/command-handlers.ts | 2 + .../src/lib/commands/commands.spec.ts | 61 ++++ .../lib/commands/turn-comment-into-issue.ts | 94 ++++++ .../application/src/lib/comments.service.ts | 20 ++ .../comment-added.event-handler.ts | 1 + ...comment-turned-into-issue.event-handler.ts | 23 ++ .../src/lib/event-handlers/event-handlers.ts | 2 + .../src/lib/issue-visits.service.ts | 14 + .../src/lib/ports/issue-visits.repository.ts | 11 + .../src/lib/tracker-application.module.ts | 4 + .../src/lib/visits/issue-visits.ts | 51 ++++ .../tracker/domain/src/lib/comment/comment.ts | 25 ++ .../events/comment-turned-into-issue.event.ts | 17 ++ .../tracker/domain/src/lib/events/index.ts | 1 + .../src/lib/read-models/comment.read-model.ts | 2 + .../src/lib/mongo/find.repositories.ts | 1 + .../src/lib/mongo/issue-visits.repository.ts | 30 ++ .../src/lib/mongo/read-model.schemas.ts | 12 + .../src/lib/tracker-infrastructure.module.ts | 3 + .../src/lib/graphql/comment.resolver.ts | 21 ++ .../src/lib/graphql/issue.resolver.ts | 14 + .../presenters/src/lib/graphql/types.ts | 10 + .../src/lib/http/comments.controller.ts | 23 ++ .../src/lib/http/dto/requests.dto.ts | 21 ++ .../src/lib/http/dto/responses.dto.ts | 19 ++ .../src/lib/http/issues.controller.ts | 24 ++ .../src/lib/events/issue.events.ts | 6 + .../src/lib/facades/issue.facade.ts | 15 + .../application/src/lib/stores/issue.store.ts | 22 ++ libs/web/tracker/domain/src/index.ts | 1 + .../tracker/domain/src/lib/models/models.ts | 2 + .../domain/src/lib/models/news.spec.ts | 86 ++++++ .../web/tracker/domain/src/lib/models/news.ts | 102 +++++++ .../tracker/domain/src/lib/ports/gateways.ts | 9 + .../graphql-collaboration.gateways.ts | 17 ++ .../lib/adapters/graphql-issues.gateway.ts | 7 + .../src/lib/graphql/fragments.graphql | 4 + .../src/lib/graphql/generated.ts | 264 +++++++++++++++++ .../src/lib/graphql/issues.graphql | 20 ++ .../presenters/src/lib/pages/issue.page.scss | 71 +++++ .../presenters/src/lib/pages/issue.page.ts | 181 +++++++++++- 50 files changed, 1953 insertions(+), 79 deletions(-) create mode 100644 libs/api/tracker/application/src/lib/commands/turn-comment-into-issue.ts create mode 100644 libs/api/tracker/application/src/lib/event-handlers/comment-turned-into-issue.event-handler.ts create mode 100644 libs/api/tracker/application/src/lib/issue-visits.service.ts create mode 100644 libs/api/tracker/application/src/lib/ports/issue-visits.repository.ts create mode 100644 libs/api/tracker/application/src/lib/visits/issue-visits.ts create mode 100644 libs/api/tracker/domain/src/lib/events/comment-turned-into-issue.event.ts create mode 100644 libs/api/tracker/infrastructure/src/lib/mongo/issue-visits.repository.ts create mode 100644 libs/web/tracker/domain/src/lib/models/news.spec.ts create mode 100644 libs/web/tracker/domain/src/lib/models/news.ts diff --git a/apps/api/openapi.json b/apps/api/openapi.json index a8571f7..0840d32 100644 --- a/apps/api/openapi.json +++ b/apps/api/openapi.json @@ -3683,6 +3683,108 @@ ] } }, + "/api/issues/{issue}/visit": { + "put": { + "description": "Answers when I had opened it before (null: never), to show what others did since. Private to me.", + "operationId": "IssuesController_visit", + "parameters": [ + { + "name": "issue", + "required": true, + "in": "path", + "description": "Ticket id or key", + "schema": { + "example": "WEB-12", + "type": "string" + } + } + ], + "responses": { + "200": { + "description": "", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/IssueVisitDto" + } + } + } + }, + "400": { + "description": "Invalid input", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "401": { + "description": "Missing or rejected Pireon token", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "403": { + "description": "Not a member of the project (or not allowed)", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "404": { + "description": "Not found", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "409": { + "description": "Someone changed it at the same time: retry", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "422": { + "description": "Business rule (e.g. a transition the workflow forbids)", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + } + }, + "security": [ + { + "pireon-oauth": [] + }, + { + "pireon": [] + } + ], + "summary": "I opened this ticket now", + "tags": [ + "Issues" + ] + } + }, "/api/issues/{issue}/follow-tree": { "put": { "description": "Told of its changes and of every ticket hanging from it, at any depth, also the ones added later. Private to me.", @@ -8436,6 +8538,116 @@ ] } }, + "/api/comments/{commentId}/issue": { + "post": { + "description": "Its text becomes the description, as a subtask of its ticket or linked to it (RELATES_TO); the comment shows where it went. Answers the new ticket. Whoever can change tickets.", + "operationId": "CommentsController_turnIntoIssue", + "parameters": [ + { + "name": "commentId", + "required": true, + "in": "path", + "schema": { + "type": "string" + } + } + ], + "requestBody": { + "required": true, + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/TurnCommentIntoIssueDto" + } + } + } + }, + "responses": { + "200": { + "description": "", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/CommandResultDto" + } + } + } + }, + "400": { + "description": "Invalid input", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "401": { + "description": "Missing or rejected Pireon token", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "403": { + "description": "Not a member of the project (or not allowed)", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "404": { + "description": "Not found", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "409": { + "description": "Someone changed it at the same time: retry", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + }, + "422": { + "description": "Business rule (e.g. a transition the workflow forbids)", + "content": { + "application/json": { + "schema": { + "$ref": "#/components/schemas/ApiErrorDto" + } + } + } + } + }, + "security": [ + { + "pireon-oauth": [] + }, + { + "pireon": [] + } + ], + "summary": "Make a ticket of a comment", + "tags": [ + "Comments" + ] + } + }, "/api/comments/{commentId}/resolution": { "put": { "description": "Whoever can change the ticket. A conclusion is added as a reply marked as a decision. The web folds a settled conversation.", @@ -17708,6 +17920,20 @@ "verdict" ] }, + "IssueVisitDto": { + "type": "object", + "properties": { + "previousVisit": { + "type": "string", + "format": "date-time", + "nullable": true, + "description": "My visit before this one (null: first time)" + } + }, + "required": [ + "previousVisit" + ] + }, "IssuePageDto": { "type": "object", "properties": { @@ -18190,6 +18416,22 @@ "winner" ] }, + "CommentSpawnDto": { + "type": "object", + "properties": { + "id": { + "type": "string" + }, + "key": { + "type": "string", + "example": "WEB-45" + } + }, + "required": [ + "id", + "key" + ] + }, "CommentDto": { "type": "object", "properties": { @@ -18272,6 +18514,13 @@ "nullable": true, "format": "date-time" }, + "spawned": { + "description": "Tickets made from it («Convertir en ticket»), oldest first", + "type": "array", + "items": { + "$ref": "#/components/schemas/CommentSpawnDto" + } + }, "createdAt": { "type": "string", "format": "date-time" @@ -18300,6 +18549,7 @@ "resolved", "resolvedBy", "resolvedAt", + "spawned", "createdAt", "editedAt", "version" @@ -18476,6 +18726,29 @@ } } }, + "TurnCommentIntoIssueDto": { + "type": "object", + "properties": { + "title": { + "type": "string", + "example": "También falla en Android", + "maxLength": 200 + }, + "type": { + "type": "string", + "description": "One of the project's ticket type ids" + }, + "asSubtask": { + "type": "boolean", + "default": false, + "description": "Hang it from the ticket of the comment; otherwise it is linked to it (RELATES_TO)" + } + }, + "required": [ + "title", + "type" + ] + }, "ResolveThreadDto": { "type": "object", "properties": { diff --git a/apps/api/schema.gql b/apps/api/schema.gql index 73102c0..cb5a717 100644 --- a/apps/api/schema.gql +++ b/apps/api/schema.gql @@ -318,6 +318,9 @@ type Comment { """Who settled it""" resolvedBy: UserRef + + """Tickets made from it («Convertir en ticket»), oldest first""" + spawned: [CommentSpawn!]! version: Int! } @@ -354,6 +357,12 @@ type CommentReaction { users: [UserRef!]! } +"""A ticket made from a comment""" +type CommentSpawn { + id: ID! + key: String! +} + type CountByAssignee { count: Int! @@ -1209,6 +1218,18 @@ type Mutation { """ trashIssue(issueId: ID!): CommandResult! + """ + Make a ticket of a comment: its text becomes the description, as a subtask of its ticket or linked to it (RELATES_TO). Answers the new ticket. Whoever can change tickets + """ + turnCommentIntoIssue( + asSubtask: Boolean! = false + commentId: ID! + title: String! + + """One of the project's ticket type ids""" + type: String! + ): CommandResult! + """ I am writing on this ticket (send every few seconds while typing): those with it open see it at once via `typing` """ @@ -1258,6 +1279,11 @@ type Mutation { updateSprint(input: UpdateSprintInput!): CommandResult! updateWebhook(input: UpdateWebhookInput!): Webhook! + """ + I opened this ticket now: answers when I had opened it before (null: never), to show what is new since + """ + visitIssue(issueId: ID!): String + """ Vote for a ticket (on) or take the vote back: anyone signed in on a public project, members on a private one """ diff --git a/apps/mcp/src/app/api/taskstream.schema.ts b/apps/mcp/src/app/api/taskstream.schema.ts index 38a95d2..e8a7ccb 100644 --- a/apps/mcp/src/app/api/taskstream.schema.ts +++ b/apps/mcp/src/app/api/taskstream.schema.ts @@ -632,6 +632,26 @@ export interface paths { patch?: never; trace?: never; }; + "/api/issues/{issue}/visit": { + parameters: { + query?: never; + header?: never; + path?: never; + cookie?: never; + }; + get?: never; + /** + * I opened this ticket now + * @description Answers when I had opened it before (null: never), to show what others did since. Private to me. + */ + put: operations["IssuesController_visit"]; + post?: never; + delete?: never; + options?: never; + head?: never; + patch?: never; + trace?: never; + }; "/api/issues/{issue}/follow-tree": { parameters: { query?: never; @@ -1241,6 +1261,26 @@ export interface paths { patch?: never; trace?: never; }; + "/api/comments/{commentId}/issue": { + parameters: { + query?: never; + header?: never; + path?: never; + cookie?: never; + }; + get?: never; + put?: never; + /** + * Make a ticket of a comment + * @description Its text becomes the description, as a subtask of its ticket or linked to it (RELATES_TO); the comment shows where it went. Answers the new ticket. Whoever can change tickets. + */ + post: operations["CommentsController_turnIntoIssue"]; + delete?: never; + options?: never; + head?: never; + patch?: never; + trace?: never; + }; "/api/comments/{commentId}/resolution": { parameters: { query?: never; @@ -3181,6 +3221,13 @@ export interface components { verdict: components["schemas"]["ReviewVerdict"]; answer?: string; }; + IssueVisitDto: { + /** + * Format: date-time + * @description My visit before this one (null: first time) + */ + previousVisit: string | null; + }; IssuePageDto: { items: components["schemas"]["IssueDto"][]; /** @description Total matches (for pagination) */ @@ -3377,6 +3424,11 @@ export interface components { /** @description Index of the winning option once closed (null on a tie) */ winner: number | null; }; + CommentSpawnDto: { + id: string; + /** @example WEB-45 */ + key: string; + }; CommentDto: { /** Format: uuid */ id: string; @@ -3407,6 +3459,8 @@ export interface components { resolvedBy: components["schemas"]["UserRefDto"] | null; /** Format: date-time */ resolvedAt: string | null; + /** @description Tickets made from it («Convertir en ticket»), oldest first */ + spawned: components["schemas"]["CommentSpawnDto"][]; /** Format: date-time */ createdAt: string; /** Format: date-time */ @@ -3490,6 +3544,17 @@ export interface components { */ decide: boolean; }; + TurnCommentIntoIssueDto: { + /** @example También falla en Android */ + title: string; + /** @description One of the project's ticket type ids */ + type: string; + /** + * @description Hang it from the ticket of the comment; otherwise it is linked to it (RELATES_TO) + * @default false + */ + asSubtask: boolean; + }; ResolveThreadDto: { /** @description What was settled: added as a reply marked as a decision before it closes */ conclusion?: string; @@ -6732,6 +6797,82 @@ export interface operations { }; }; }; + IssuesController_visit: { + parameters: { + query?: never; + header?: never; + path: { + /** @description Ticket id or key */ + issue: string; + }; + cookie?: never; + }; + requestBody?: never; + responses: { + 200: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["IssueVisitDto"]; + }; + }; + /** @description Invalid input */ + 400: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Missing or rejected Pireon token */ + 401: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Not a member of the project (or not allowed) */ + 403: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Not found */ + 404: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Someone changed it at the same time: retry */ + 409: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Business rule (e.g. a transition the workflow forbids) */ + 422: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + }; + }; IssuesController_followTree: { parameters: { query?: never; @@ -9935,6 +10076,85 @@ export interface operations { }; }; }; + CommentsController_turnIntoIssue: { + parameters: { + query?: never; + header?: never; + path: { + commentId: string; + }; + cookie?: never; + }; + requestBody: { + content: { + "application/json": components["schemas"]["TurnCommentIntoIssueDto"]; + }; + }; + responses: { + 200: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["CommandResultDto"]; + }; + }; + /** @description Invalid input */ + 400: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Missing or rejected Pireon token */ + 401: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Not a member of the project (or not allowed) */ + 403: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Not found */ + 404: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Someone changed it at the same time: retry */ + 409: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + /** @description Business rule (e.g. a transition the workflow forbids) */ + 422: { + headers: { + [name: string]: unknown; + }; + content: { + "application/json": components["schemas"]["ApiErrorDto"]; + }; + }; + }; + }; CommentsController_resolveThread: { parameters: { query?: never; diff --git a/apps/mcp/src/app/tools/issues.resolver.ts b/apps/mcp/src/app/tools/issues.resolver.ts index 7994056..7ffdc64 100644 --- a/apps/mcp/src/app/tools/issues.resolver.ts +++ b/apps/mcp/src/app/tools/issues.resolver.ts @@ -599,6 +599,37 @@ export class IssuesResolver { ); } + @Tool({ + name: 'turn_comment_into_issue', + description: + "Make a ticket of a comment that turned out to be work of its own: its text becomes the description, as a subtask of the comment's ticket (asSubtask) or linked to it. Read type ids with get_project. Answers the new ticket's key.", + paramsSchema: z.object({ + commentId: z.string().min(1), + title: z.string().min(1).max(200), + type: z.string().min(1).describe("One of the project's ticket type ids"), + asSubtask: z.boolean().default(false), + }), + annotations: WRITE, + }) + turnCommentIntoIssue( + { + commentId, + title, + type, + asSubtask, + }: { commentId: string; title: string; type: string; asSubtask: boolean }, + ctx: McpContext, + ) { + return toolResult(() => + this.api.data( + this.api.as(ctx).POST('/api/comments/{commentId}/issue', { + params: { path: { commentId } }, + body: { title, type, asSubtask }, + }), + ), + ); + } + @Tool({ name: 'link_issues', description: diff --git a/docs/01-vision-general.md b/docs/01-vision-general.md index 86c7639..18051b3 100644 --- a/docs/01-vision-general.md +++ b/docs/01-vision-general.md @@ -2,51 +2,51 @@ ## Qué resuelve -| Necesidad | Cómo se resuelve | -| --------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | -| 🐛 Bugs | Tickets de tipo `BUG` con prioridad, área y etiquetas | -| 🌍 Públicos | Un proyecto público se puede leer sin iniciar sesión (web, GraphQL y REST): tablero, tickets, comentarios, releases, actividad y estadísticas; «Explorar» los lista | -| 🗑️ Borrar | El propietario borra un proyecto confirmando su clave; tickets, comentarios, releases y enlaces desde otros proyectos se borran en cascada (sagas) y la clave queda libre | -| 🧩 Configuración | Cada proyecto tiene sus estados (con categoría, color y motivo obligatorio), transiciones, tipos de ticket y áreas; lo usado se archiva, no se borra | -| 🎨 Cambios de UI | Tipo `UI_CHANGE` con enlace al frame de Figma (validado contra `figma.com`) | -| ⚙️ Features | Tipo `FEATURE` siguiendo el workflow completo | -| 🧪 QA | Por defecto, `READY_FOR_QA → IN_QA → DONE / QA_FAILED`; fallar QA exige un motivo | -| 🚀 Releases | Versiones a las que se asignan tickets; no se publican con tickets sin terminar (salvo forzando) | -| 📝 Notas de versión | Borrador en Markdown con los tickets terminados por tipo, editable y copiable, también tras publicar; y una página pública por release para los clientes, con enlace secreto | -| 👥 Asignaciones | Solo a miembros del proyecto; panel de carga por persona | -| 💬 Comentarios | Decisiones y explicaciones dentro del ticket, con formato (editor visual, guardado en Markdown), capturas pegadas, respuestas, reacciones (👍 🎉…), decisiones (un comentario marcado sale encima del ticket y se encuentra al buscar), conversaciones que se resuelven con su conclusión, encuestas rápidas que acaban en decisión, «está escribiendo…» en vivo y `@menciones`; el autor edita, el admin modera | -| 🔎 Vistas | El tablero se filtra por texto, persona (o sin asignar), tipo, prioridad, área y release; los filtros se guardan como vistas propias o compartidas con el proyecto | -| 📋 Lista | Todos los tickets en filas: ordenar por cualquier columna, filtrar, cambiar estado, prioridad o persona sin abrirlos, editar en bloque y duplicar | -| ↩️ Deshacer | Mover, reasignar o mandar a la papelera ofrece «Deshacer» unos segundos (también Ctrl/⌘ Z), con la orden contraria | -| ☀ Mi día | Cada mañana eliges los tickets de hoy; se tachan al terminar y lo que quede se pasa al día siguiente; y «Recordármelo» en cualquier ticket | -| 🚦 Salud | Un semáforo por proyecto (en su panel y en la cartera) que dice qué va mal y por qué: atascados, vencidos, bugs urgentes, QA, releases tarde | -| 🚦 WIP y carriles | Límite de trabajo en curso por estado (la columna se marca al pasarse) y tablero agrupable por persona, épica, prioridad o tipo | -| 🎲 Estimación | Puntos por ticket (escala 0-1-2-3-5-8-13-21) en la ficha, las tarjetas, las columnas y el backlog; velocidad por sprint y aviso si un sprint planificado supera la media | -| 🏃 Sprints | Iteraciones con fechas y objetivo, una en curso por proyecto, con burndown (tickets o puntos) y traspaso de lo pendiente al cerrar | -| 📅 Calendario | Los sprints mes a mes como franjas, con las releases y los vencimientos de cada día; se planifica un sprint pulsando su primer día y todo se descarga en .ics | -| 📋 Backlog | Lo pendiente de empezar, en el orden en que se va a coger; se reordena arrastrando y el tablero sigue ese orden | -| 🗑️ Papelera | Borrar un ticket lo manda a la papelera (quien lo abrió o un admin); se recupera durante 30 días y luego se borra del todo | -| ☑️ En bloque | Selecciona varios tickets del backlog y cámbiales a la vez estado, persona, sprint, prioridad o etiqueta | -| ⌨️ Atajos y búsqueda | Ctrl/⌘ K o / busca tickets, decisiones y proyectos de todos tus proyectos (antes de escribir, tus tickets fijados); `g b`, `g k`, `g s`, `g r` navegan, `c` crea, en el tablero las flechas recorren las tarjetas y Shift+←/→ las cambia de columna, `e` edita, `?` los lista | -| 🗺️ Hoja de ruta | Las épicas en el tiempo (fecha de inicio y límite), con los sprints y las releases como referencias y flechas entre las que dependen unas de otras | -| 🗳️ Propuestas y votos | Un proyecto público puede abrir un formulario para que cualquiera, sin cuenta, reporte un error o proponga una idea; quien tenga sesión vota lo que quiere | -| 🌐 Web del colectivo | La página de cada proyecto en pireon.dev enseña su hoja de ruta, versiones, sprint y tickets para recién llegados (`good-first-issue`) | -| 🧩 Desglose | Épicas → tickets → subtareas (tres niveles), con su progreso; y una checklist dentro de cada ticket para los pasos pequeños | -| 👀 Seguir | Cualquier miembro sigue los tickets que le interesan y los ve juntos en «Que sigo»; quien lo abre, lo tiene asignado o comenta, lo sigue solo | -| 🔔 Avisos | Campana con lo que te concierne (asignaciones, cambios de estado, QA fallido, comentarios, respuestas), en vivo y opcionalmente por correo | -| ⏰ Recordatorios | Aviso (y correo, si se quiere) el día antes, el mismo día y al vencer la fecha límite, a quien lo tiene y a quien lo sigue; las tarjetas marcan lo que vence | -| 📎 Adjuntos | Capturas, logs y vídeos en el ticket o bajo un comentario; arrastrar, elegir o pegar con Ctrl+V; miniaturas de las imágenes; grabar la pantalla o la voz desde el comentario y verlo ahí mismo | -| 🔗 Relaciones | `BLOCKS`, `DEPENDS_ON`, `TESTS`, `DUPLICATES`, `RELATES_TO`, también entre proyectos (FE ↔ BE ↔ QA); lo que espera a otro ticket abierto sale marcado como «Bloqueado» | -| 🌿 Git | Webhook para GitHub, Gitea o GitLab: commits, ramas y pull requests que nombran un ticket (WEB-12) aparecen en su ficha | -| 🚨 Alertas → bugs | Sentry, Uptime Kuma, Grafana o cualquier herramienta abren bugs por una URL secreta; el mismo problema repetido va al bug abierto, y «resuelta» se dice en él | -| 🧾 Plantillas | Cada tipo de ticket puede traer una descripción inicial y una checklist que se aplican al crearlo | -| 🤖 Automatizaciones | Reglas «cuando… si… entonces…»: al crear, mover, asignar, comentar o fusionar una PR, mover, asignar, etiquetar, priorizar, comentar o meter en el sprint | -| 📡 Webhooks | Un POST firmado (HMAC-SHA256) a otro sistema cuando pasa algo: tickets, comentarios, releases, sprints; con prueba y último envío | -| 📦 Importar/exportar | Todos los tickets a CSV; importar un CSV propio o una exportación de Jira, con vista previa de lo que se va a crear | -| 🏠 Inicio | «Mi trabajo»: mis tickets en todos los proyectos, por categoría (QA fallidos primero), lo que espera QA, próximas releases e «Ir a…» por clave | -| 📊 Seguimiento | Panel con contadores, flujo por estado, el equipo con sus tickets abiertos (enlaza a su tablero), feed de actividad e historial por ticket | -| 📈 Estadísticas | Creados vs. resueltos, flujo acumulado, burnup de releases, tiempos de ciclo y entrega, tasa de fallo en QA y resueltos por persona | -| ⚡ Tiempo real | Tablero, comentarios, actividad, releases y membresías se actualizan solos (GraphQL subscriptions) | +| Necesidad | Cómo se resuelve | +| --------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | +| 🐛 Bugs | Tickets de tipo `BUG` con prioridad, área y etiquetas | +| 🌍 Públicos | Un proyecto público se puede leer sin iniciar sesión (web, GraphQL y REST): tablero, tickets, comentarios, releases, actividad y estadísticas; «Explorar» los lista | +| 🗑️ Borrar | El propietario borra un proyecto confirmando su clave; tickets, comentarios, releases y enlaces desde otros proyectos se borran en cascada (sagas) y la clave queda libre | +| 🧩 Configuración | Cada proyecto tiene sus estados (con categoría, color y motivo obligatorio), transiciones, tipos de ticket y áreas; lo usado se archiva, no se borra | +| 🎨 Cambios de UI | Tipo `UI_CHANGE` con enlace al frame de Figma (validado contra `figma.com`) | +| ⚙️ Features | Tipo `FEATURE` siguiendo el workflow completo | +| 🧪 QA | Por defecto, `READY_FOR_QA → IN_QA → DONE / QA_FAILED`; fallar QA exige un motivo | +| 🚀 Releases | Versiones a las que se asignan tickets; no se publican con tickets sin terminar (salvo forzando) | +| 📝 Notas de versión | Borrador en Markdown con los tickets terminados por tipo, editable y copiable, también tras publicar; y una página pública por release para los clientes, con enlace secreto | +| 👥 Asignaciones | Solo a miembros del proyecto; panel de carga por persona | +| 💬 Comentarios | Decisiones y explicaciones dentro del ticket, con formato (editor visual, guardado en Markdown), capturas pegadas, respuestas, reacciones (👍 🎉…), decisiones (un comentario marcado sale encima del ticket y se encuentra al buscar), conversaciones que se resuelven con su conclusión, convertir un comentario en ticket, lo nuevo desde tu última visita marcado, encuestas rápidas que acaban en decisión, «está escribiendo…» en vivo y `@menciones`; el autor edita, el admin modera | +| 🔎 Vistas | El tablero se filtra por texto, persona (o sin asignar), tipo, prioridad, área y release; los filtros se guardan como vistas propias o compartidas con el proyecto | +| 📋 Lista | Todos los tickets en filas: ordenar por cualquier columna, filtrar, cambiar estado, prioridad o persona sin abrirlos, editar en bloque y duplicar | +| ↩️ Deshacer | Mover, reasignar o mandar a la papelera ofrece «Deshacer» unos segundos (también Ctrl/⌘ Z), con la orden contraria | +| ☀ Mi día | Cada mañana eliges los tickets de hoy; se tachan al terminar y lo que quede se pasa al día siguiente; y «Recordármelo» en cualquier ticket | +| 🚦 Salud | Un semáforo por proyecto (en su panel y en la cartera) que dice qué va mal y por qué: atascados, vencidos, bugs urgentes, QA, releases tarde | +| 🚦 WIP y carriles | Límite de trabajo en curso por estado (la columna se marca al pasarse) y tablero agrupable por persona, épica, prioridad o tipo | +| 🎲 Estimación | Puntos por ticket (escala 0-1-2-3-5-8-13-21) en la ficha, las tarjetas, las columnas y el backlog; velocidad por sprint y aviso si un sprint planificado supera la media | +| 🏃 Sprints | Iteraciones con fechas y objetivo, una en curso por proyecto, con burndown (tickets o puntos) y traspaso de lo pendiente al cerrar | +| 📅 Calendario | Los sprints mes a mes como franjas, con las releases y los vencimientos de cada día; se planifica un sprint pulsando su primer día y todo se descarga en .ics | +| 📋 Backlog | Lo pendiente de empezar, en el orden en que se va a coger; se reordena arrastrando y el tablero sigue ese orden | +| 🗑️ Papelera | Borrar un ticket lo manda a la papelera (quien lo abrió o un admin); se recupera durante 30 días y luego se borra del todo | +| ☑️ En bloque | Selecciona varios tickets del backlog y cámbiales a la vez estado, persona, sprint, prioridad o etiqueta | +| ⌨️ Atajos y búsqueda | Ctrl/⌘ K o / busca tickets, decisiones y proyectos de todos tus proyectos (antes de escribir, tus tickets fijados); `g b`, `g k`, `g s`, `g r` navegan, `c` crea, en el tablero las flechas recorren las tarjetas y Shift+←/→ las cambia de columna, `e` edita, `?` los lista | +| 🗺️ Hoja de ruta | Las épicas en el tiempo (fecha de inicio y límite), con los sprints y las releases como referencias y flechas entre las que dependen unas de otras | +| 🗳️ Propuestas y votos | Un proyecto público puede abrir un formulario para que cualquiera, sin cuenta, reporte un error o proponga una idea; quien tenga sesión vota lo que quiere | +| 🌐 Web del colectivo | La página de cada proyecto en pireon.dev enseña su hoja de ruta, versiones, sprint y tickets para recién llegados (`good-first-issue`) | +| 🧩 Desglose | Épicas → tickets → subtareas (tres niveles), con su progreso; y una checklist dentro de cada ticket para los pasos pequeños | +| 👀 Seguir | Cualquier miembro sigue los tickets que le interesan y los ve juntos en «Que sigo»; quien lo abre, lo tiene asignado o comenta, lo sigue solo | +| 🔔 Avisos | Campana con lo que te concierne (asignaciones, cambios de estado, QA fallido, comentarios, respuestas), en vivo y opcionalmente por correo | +| ⏰ Recordatorios | Aviso (y correo, si se quiere) el día antes, el mismo día y al vencer la fecha límite, a quien lo tiene y a quien lo sigue; las tarjetas marcan lo que vence | +| 📎 Adjuntos | Capturas, logs y vídeos en el ticket o bajo un comentario; arrastrar, elegir o pegar con Ctrl+V; miniaturas de las imágenes; grabar la pantalla o la voz desde el comentario y verlo ahí mismo | +| 🔗 Relaciones | `BLOCKS`, `DEPENDS_ON`, `TESTS`, `DUPLICATES`, `RELATES_TO`, también entre proyectos (FE ↔ BE ↔ QA); lo que espera a otro ticket abierto sale marcado como «Bloqueado» | +| 🌿 Git | Webhook para GitHub, Gitea o GitLab: commits, ramas y pull requests que nombran un ticket (WEB-12) aparecen en su ficha | +| 🚨 Alertas → bugs | Sentry, Uptime Kuma, Grafana o cualquier herramienta abren bugs por una URL secreta; el mismo problema repetido va al bug abierto, y «resuelta» se dice en él | +| 🧾 Plantillas | Cada tipo de ticket puede traer una descripción inicial y una checklist que se aplican al crearlo | +| 🤖 Automatizaciones | Reglas «cuando… si… entonces…»: al crear, mover, asignar, comentar o fusionar una PR, mover, asignar, etiquetar, priorizar, comentar o meter en el sprint | +| 📡 Webhooks | Un POST firmado (HMAC-SHA256) a otro sistema cuando pasa algo: tickets, comentarios, releases, sprints; con prueba y último envío | +| 📦 Importar/exportar | Todos los tickets a CSV; importar un CSV propio o una exportación de Jira, con vista previa de lo que se va a crear | +| 🏠 Inicio | «Mi trabajo»: mis tickets en todos los proyectos, por categoría (QA fallidos primero), lo que espera QA, próximas releases e «Ir a…» por clave | +| 📊 Seguimiento | Panel con contadores, flujo por estado, el equipo con sus tickets abiertos (enlaza a su tablero), feed de actividad e historial por ticket | +| 📈 Estadísticas | Creados vs. resueltos, flujo acumulado, burnup de releases, tiempos de ciclo y entrega, tasa de fallo en QA y resueltos por persona | +| ⚡ Tiempo real | Tablero, comentarios, actividad, releases y membresías se actualizan solos (GraphQL subscriptions) | Los **usuarios no se gestionan aquí**: vienen del servidor de identidad **Pireon** ([Swagger](https://oauth.pireon.dev/api/documentation)). TaskStream gestiona proyectos, membresías y roles por diff --git a/docs/03-api.md b/docs/03-api.md index b6aff6f..dfdf3e7 100644 --- a/docs/03-api.md +++ b/docs/03-api.md @@ -245,6 +245,11 @@ stateDiagram-v2 admin, la cierra (`closePoll(commentId, decide)`, `POST …/poll/close`; `CommentPollClosedEvent`): gana la más votada (con empate, ninguna) y, con `decide`, se añade una respuesta «Decidido: …» marcada como decisión. El comentario da `poll { options { text voters } closed winner }`. +- **Convertir en ticket**: un comentario que resulta ser trabajo propio se convierte en ticket + (`turnCommentIntoIssue(commentId, title, type, asSubtask)`, `POST /api/comments/{id}/issue`), como subtarea del ticket + del comentario o enlazado a él (`RELATES_TO`), por quien puede cambiar tickets. El texto del comentario es la + descripción (sin la plantilla del tipo), con una línea de dónde sale; el comentario lo recuerda + (`CommentTurnedIntoIssueEvent`) y da `spawned { id key }`. Devuelve el ticket nuevo. - **Conversaciones**: un comentario raíz y sus respuestas se resuelven (`resolveThread(commentId, resolved, conclusion)`, `PUT/DELETE /api/comments/{id}/resolution`; evento `CommentThreadResolvedEvent`) por quien puede cambiar el ticket; una respuesta no se resuelve sola. Si trae `conclusion`, antes de cerrarla se añade como respuesta marcada como @@ -454,7 +459,10 @@ historial (`my_days`). **Recordármelo**: cada persona pone en un ticket un reco /api/issues/{issue}/reminder`, `GET /api/me/reminders`): uno por ticket, en el futuro y como mucho a un año. Cada minuto `IssueReminderScheduler` avisa de los que tocan (`REMINDER`, por la campana, el navegador y el correo; una vez, por su id) y olvida los de tickets en la papelera o fuera de alcance. Privados y sin historial (`issue_reminders`); -`REMINDERS_ENABLED=false` los apaga junto con los de fecha límite. **Seguir todo**: seguir un ticket con todo lo que cuelga de él (una épica, un ticket con subtareas): +`REMINDERS_ENABLED=false` los apaga junto con los de fecha límite. **Lo nuevo desde tu última visita**: al abrir un +ticket, `visitIssue(issueId)` (`PUT /api/issues/{issue}/visit`) apunta la visita y devuelve la anterior (null la +primera vez); con ella el cliente marca lo que otros han hecho desde entonces. Personal y sin historial (colección +`issue_visits`). **Seguir todo**: seguir un ticket con todo lo que cuelga de él (una épica, un ticket con subtareas): `followTree`/`unfollowTree`, `Issue.followingTree`; `PUT/DELETE /api/issues/{issue}/follow-tree`. Quien lo hace oye lo mismo que un seguidor de cada ticket de debajo, a cualquier profundidad, también de los que se añadan, y además cuándo uno entra (`IssueParentChangedEvent`). Personal y sin historial (colección `tree_follows`); los invitados de un proyecto @@ -573,17 +581,17 @@ que funcionan también sin token. Todas devuelven `CommandResult { id, key, version }`. -| Ámbito | Mutations | -| ----------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ | -| Proyectos | `createProject`, `updateProject`, `updateProjectSettings`, `deleteProject(confirmKey)`, `addProjectMember`, `changeProjectMemberRole`, `removeProjectMember` | -| Tickets | `createIssue`, `updateIssue`, `changeIssueStatus`, `assignIssue`, `linkIssues`, `unlinkIssues`, `setIssueRelease`, `setIssueParent`, `rankIssue`, `addChecklistItem`, `updateChecklistItem`, `removeChecklistItem`, `watchIssue`, `unwatchIssue`, `requestReview`, `answerReview`, `cancelReview`, `pinIssue`, `unpinIssue`, `followTree`, `unfollowTree`, `clearReminder`, `addToMyDay`, `removeFromMyDay` (devuelven `true`), `setReminder`, `removeAttachment` (subir: por REST), `duplicateIssue`, `voteIssue` | -| Comentarios | `addComment(replyTo, poll)`, `votePoll`, `closePoll`, `editComment`, `deleteComment`, `reactToComment`, `markDecision(commentId, on)`, `resolveThread(commentId, resolved, conclusion)`, `typing(issueId, threadId)`; consulta `decisions(projectId, search)` | -| Webhooks | `createWebhook` (devuelve el secreto), `updateWebhook`, `deleteWebhook`, `testWebhook` | -| Git | `addGitRepository` (devuelve su secreto), `rotateGitRepositorySecret`, `removeGitRepository` | -| Sprints | `createSprint`, `updateSprint`, `startSprint`, `closeSprint(carryOverToSprintId)`, `deleteSprint`, `setIssueSprint` | -| Vistas | `saveFilter(input)` (devuelve el id), `deleteSavedFilter(id)`, `GET/POST /api/me/views`, `DELETE /api/me/views/{id}` (mis vistas de «Mi trabajo») | -| Avisos | `markNotificationsRead(ids)`, `updateNotificationSettings(email)` | -| Releases | `createRelease`, `updateRelease`, `publishRelease(force)`, `writeReleaseNotes`, `archiveRelease` | +| Ámbito | Mutations | +| ----------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | +| Proyectos | `createProject`, `updateProject`, `updateProjectSettings`, `deleteProject(confirmKey)`, `addProjectMember`, `changeProjectMemberRole`, `removeProjectMember` | +| Tickets | `createIssue`, `updateIssue`, `changeIssueStatus`, `assignIssue`, `linkIssues`, `unlinkIssues`, `setIssueRelease`, `setIssueParent`, `rankIssue`, `addChecklistItem`, `updateChecklistItem`, `removeChecklistItem`, `watchIssue`, `unwatchIssue`, `requestReview`, `answerReview`, `cancelReview`, `pinIssue`, `unpinIssue`, `followTree`, `unfollowTree`, `visitIssue`, `clearReminder`, `addToMyDay`, `removeFromMyDay` (devuelven `true`), `setReminder`, `removeAttachment` (subir: por REST), `duplicateIssue`, `voteIssue` | +| Comentarios | `addComment(replyTo, poll)`, `votePoll`, `closePoll`, `editComment`, `deleteComment`, `reactToComment`, `markDecision(commentId, on)`, `resolveThread(commentId, resolved, conclusion)`, `turnCommentIntoIssue(commentId, title, type, asSubtask)`, `typing(issueId, threadId)`; consulta `decisions(projectId, search)` | +| Webhooks | `createWebhook` (devuelve el secreto), `updateWebhook`, `deleteWebhook`, `testWebhook` | +| Git | `addGitRepository` (devuelve su secreto), `rotateGitRepositorySecret`, `removeGitRepository` | +| Sprints | `createSprint`, `updateSprint`, `startSprint`, `closeSprint(carryOverToSprintId)`, `deleteSprint`, `setIssueSprint` | +| Vistas | `saveFilter(input)` (devuelve el id), `deleteSavedFilter(id)`, `GET/POST /api/me/views`, `DELETE /api/me/views/{id}` (mis vistas de «Mi trabajo») | +| Avisos | `markNotificationsRead(ids)`, `updateNotificationSettings(email)` | +| Releases | `createRelease`, `updateRelease`, `publishRelease(force)`, `writeReleaseNotes`, `archiveRelease` | ### Subscriptions @@ -786,7 +794,7 @@ Proyectos y tickets se pueden indicar por **id o por clave** (`WEB`, `WEB-12`). | Usuarios | `GET /api/users/me`, `GET /api/users?search=`, `GET /api/users/{id}` | | Proyectos | `GET/POST /api/projects`, `GET/PATCH/DELETE /api/projects/{project}` (`DELETE …?confirm=CLAVE`), `POST …/archive`, `POST …/unarchive`, `PUT …/settings`, `…/board`, `…/epics`, `…/stats`, `…/analytics?days=`, `…/insights`, `…/activity`, `…/members` (POST, PATCH, DELETE) | | Tickets | `GET /api/me/home` (portada), `GET /api/me/portfolio` (cartera), `GET /api/me/issues` (mi trabajo; filtros `project`, `priority`, `bugsOnly`, `overdue`), `GET/POST /api/projects/{project}/issues`, `GET/PATCH /api/issues/{issue}`, `PUT …/status`, `PUT …/assignee`, `PUT …/release`, `PUT/DELETE …/watch`, `POST …/reviews`, `POST …/reviews/{id}/answer`, `DELETE …/reviews/{id}`, `PUT/DELETE …/follow-tree`, `PUT/DELETE …/reminder`, `GET /api/me/reminders`, `GET /api/me/day`, `PUT/DELETE /api/me/day/{issue}`, `GET /api/me/pins` y `PUT/DELETE …/pin` (fijados), `PUT/DELETE …/presence`, `PUT …/parent`, `PUT …/rank`, `POST …/checklist`, `PATCH/DELETE …/checklist/{item}`, `POST/DELETE …/links`, `GET/POST …/comments`, `GET …/history`, `POST /api/issues/{issue}/duplicate`, `PUT/DELETE …/vote` | -| Comentarios | `PATCH/DELETE /api/comments/{commentId}`, `POST …/reactions`, `PUT/DELETE …/decision`, `PUT/DELETE …/resolution`, `POST …/poll/vote`, `POST …/poll/close`; `GET /api/issues/{issue}/decisions`, `GET /api/projects/{project}/decisions`, `GET /api/me/decisions` | +| Comentarios | `PATCH/DELETE /api/comments/{commentId}`, `POST …/reactions`, `PUT/DELETE …/decision`, `PUT/DELETE …/resolution`, `POST …/poll/vote`, `POST …/poll/close`, `POST …/issue` (convertir en ticket); `GET /api/issues/{issue}/decisions`, `GET /api/projects/{project}/decisions`, `GET /api/me/decisions` | | Sprints | `GET/POST /api/projects/{project}/sprints`, `GET/PATCH/DELETE /api/sprints/{sprint}`, `POST …/start`, `POST …/close`, `GET …/progress`, `…/burndown`, `…/issues`, `PUT /api/issues/{issue}/sprint` | | Vistas | `GET/POST /api/projects/{project}/filters`, `DELETE /api/filters/{id}` | | Avisos | `GET /api/me/notifications`, `POST /api/me/notifications/read`, `GET/PUT /api/me/notification-settings` | diff --git a/docs/04-cliente-web.md b/docs/04-cliente-web.md index ad3431d..192a402 100644 --- a/docs/04-cliente-web.md +++ b/docs/04-cliente-web.md @@ -300,26 +300,26 @@ visualización de la plataforma: ## 4.6 Rutas -| Ruta | Página | -| -------------------------- | --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | -| `/login` | Inicio de sesión (solo sin sesión) | -| `/auth/callback` | Retorno de Pireon (PKCE) | -| `/` | Inicio: saludo según la hora con el resumen del día, «Ir a…» (`WEB-12`), primeros pasos, mis proyectos con su actividad (lo primero, para llegar antes), «Mi día» (`ts-my-day`: los tickets que elijo para hoy, tachados al terminar, mis abiertos para añadir y lo que quedó del último día), mis fijados (`ts-pinned`, una franja que se quita con ×; solo si hay), cuatro cifras (`ts-home-kpis`: abiertos míos, esperando QA, terminados por mí y por el equipo esta semana), la semana (`ts-week-agenda`), mis vistas guardadas y dos columnas: mis tickets y lo último en mis proyectos; sprints en curso, esperando QA, que sigo, próximas releases, quién está ahora y «Retomar» | -| `/portfolio` | Cartera (con sesión): una tarjeta por proyecto activo con lo abierto y en riesgo, el sprint en curso (¿llega?), las próximas releases con su previsión y las épicas abiertas; en directo: se vuelve a leer sola cuando cambia un ticket, sprint o release de esos proyectos. Cada proyecto lleva su semáforo de salud (`portfolioHealth`), con las causas al pasar por encima | -| `/explore` | Proyectos públicos (sin sesión) | -| `/p/:key/board` | Tablero kanban (`?release=` y `?assignee=` preseleccionan los filtros). Quien puede contribuir cambia prioridad, estimación y persona desde la propia tarjeta (selectores nativos sobre sus distintivos; el título es el enlace). Mover una tarjeta o cambiar su persona ofrece «Deshacer» unos segundos (o Ctrl/⌘ Z): la orden contraria, que el flujo puede rechazar. Con el teclado: Tab entra en las tarjetas, las flechas se mueven entre ellas, Shift+←/→ pasa la tarjeta de columna (mismas reglas que arrastrar), `p`/`e`/`a` abren su prioridad, estimación o persona y Enter la abre | -| `/p/:key/issues/:issueKey` | Ticket. Encima del título, quién más lo está viendo o editando ahora (`PresenceFacade`: late cada 10 s; editar es tener abierto el formulario del ticket o de un comentario, o estar escribiendo uno). «Fijar» lo deja a mano en el inicio y en la paleta de búsqueda. Arriba, sus decisiones (comentarios marcados con la bandera por quien puede cambiar el ticket); `?comment=` lleva a un comentario y lo señala. Cambiar el estado o la persona y mandarlo a la papelera (sin confirmación) ofrecen «Deshacer». Cada conversación con respuestas se resuelve («✓ Resolver», con una conclusión opcional que queda como decisión) y se pliega en una línea; la cabecera cuenta las abiertas y terminar el ticket con alguna pregunta antes. Se ve quién está escribiendo en cada conversación («Ana está escribiendo…», `PresenceFacade.typists`), y en las respuestas Enter envía (Shift+Enter, otra línea). Si tiene subtareas, «Seguir todo» avisa de ella y de todo lo que cuelga, también lo que se añada (también desde Épicas). «Recordármelo» pone un aviso solo para mí (dentro de 1 hora, esta tarde, mañana o el lunes a las 9, en una semana o cuando diga), con nota opcional. «☀ Mi día» lo añade a los de hoy. «Pedir revisión» a alguien del equipo: lo ve en «Mi trabajo» («Te piden revisión») y responde «Todo bien» o «Pedir cambios» desde el recuadro Revisiones. Un comentario puede llevar una encuesta («📊 Añadir una encuesta»): se vota pulsando una opción y, al cerrarla, la ganadora queda como decisión. «🎥 Grabar pantalla» (con la voz, si se da permiso al micrófono) y «🎙 Grabar audio» graban en el navegador hasta 3 minutos y lo adjuntan al comentario, que puede ir sin texto; los vídeos y audios adjuntos se reproducen ahí mismo | -| `/p/:key/releases` | Releases. «Página pública» (quien gestiona): un enlace secreto con las notas para clientes, que se copia, renueva o retira | -| `/p/:key/report/:kind/:id` | Informe para imprimir de un sprint (`sprint`) o una release (`release`): datos, totales, notas y tickets por etapa. «Imprimir o guardar en PDF» abre el diálogo del navegador; al imprimir se quitan la cabecera, las pestañas y los fondos (`styles.scss`) | -| `/p/:key/roadmap` | Hoja de ruta: épicas en el tiempo, con sprints y releases; flechas «bloquea» / «depende de» entre épicas y aviso si una empieza antes de que acabe la que la bloquea. Los bloqueos con tickets de otros proyectos (en los dos sentidos, `externalDependencies`) salen en «De otros proyectos» con barra discontinua, sus flechas y el mismo aviso si las fechas no encajan; los que no tienen fechas o no se pueden leer se nombran debajo | -| `/p/:key/dashboard` | Arriba, la salud del proyecto (`ts-project-health`): un semáforo (bien, atención, en riesgo) con cada causa y sus tickets, calculado en la web (`projectHealth`) con los mismos datos: atascados, vencidos, bloqueados, bugs críticos o altos, lo que devuelve QA, releases que no llegan y columnas sobre su WIP. Seguimiento: cifras, flujo por estado, lo que ha cambiado desde tu última visita (guardada en este navegador), sprint en curso, tickets en riesgo, previsión de las próximas releases, antigüedad de lo que está en curso, cuellos de botella, salud del backlog (cada número abre la lista filtrada: `?missing=estimate`, `area`, `assignee` o `?staleDays=90`), calidad (bugs por prioridad y por semana, QA por área, bugs tras cada release), abiertos por un campo de lista a elegir, el recorrido de un ticket (dónde se va el tiempo de creado a hecho y qué parte es trabajo), la actividad de 12 semanas en un mapa de calor (pulsar un día abre lo que pasó, ticket a ticket), el embudo de propuestas del público (si las hay), equipo con sus puntos abiertos y tipos. Se recarga solo con cada cambio. «Modo pantalla» lo pone a pantalla completa para una tele del equipo: un bloque cada vez, ampliado, que cambia cada 15 s (← → para moverse, espacio para pausar, Esc para salir), con la hora y en directo | -| `/p/:key/analytics` | Estadísticas: creados vs. resueltos, flujo acumulado, burnup de una release, tiempos de ciclo y entrega, QA y resueltos por persona (30 días a 1 año) | -| `/p/:key/settings` | Configuración (propietario y admins; pestaña oculta para el resto), con un menú de secciones (`?section=`): General (visibilidad, propuestas, panel para clientes), Flujo de trabajo (estados, transiciones, definición de «Hecho»), Tickets (tipos, áreas, campos), Equipos, Automatizaciones, Integraciones (Git, alertas → bugs con su URL secreta y cómo conectar Sentry, Uptime Kuma o Grafana, webhooks), Importar y exportar y Zona peligrosa; archivar en lugar de borrar, y un punto en el menú donde hay cambios sin guardar | -| `/p/:key/activity` | Actividad en vivo | -| `/p/:key/propose` | Formulario público para reportar un error o proponer una idea sin cuenta (si el proyecto lo tiene abierto); llegan como tickets con la etiqueta «propuesta» | -| `/share/:token` | Panel para clientes, sin sesión: los bloques que eligió el proyecto (flujo, sprint, releases, épicas, actividad), sin enlaces hacia dentro de la aplicación; se vuelve a leer cada minuto. Si el enlace se renovó o retiró, «Este enlace ya no funciona» | -| `/notes/:token` | Notas de una release para sus clientes, sin cuenta: las escritas o las de sus tickets terminados | -| `/p/:key/members` | Miembros, con su rol (propietario, admin, miembro, lector o invitado). Un invitado de un proyecto privado solo tiene Tablero, Lista y Miembros, ve solo sus tickets y abre tickets sin asignarlos ni planificarlos | +| Ruta | Página | +| -------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | +| `/login` | Inicio de sesión (solo sin sesión) | +| `/auth/callback` | Retorno de Pireon (PKCE) | +| `/` | Inicio: saludo según la hora con el resumen del día, «Ir a…» (`WEB-12`), primeros pasos, mis proyectos con su actividad (lo primero, para llegar antes), «Mi día» (`ts-my-day`: los tickets que elijo para hoy, tachados al terminar, mis abiertos para añadir y lo que quedó del último día), mis fijados (`ts-pinned`, una franja que se quita con ×; solo si hay), cuatro cifras (`ts-home-kpis`: abiertos míos, esperando QA, terminados por mí y por el equipo esta semana), la semana (`ts-week-agenda`), mis vistas guardadas y dos columnas: mis tickets y lo último en mis proyectos; sprints en curso, esperando QA, que sigo, próximas releases, quién está ahora y «Retomar» | +| `/portfolio` | Cartera (con sesión): una tarjeta por proyecto activo con lo abierto y en riesgo, el sprint en curso (¿llega?), las próximas releases con su previsión y las épicas abiertas; en directo: se vuelve a leer sola cuando cambia un ticket, sprint o release de esos proyectos. Cada proyecto lleva su semáforo de salud (`portfolioHealth`), con las causas al pasar por encima | +| `/explore` | Proyectos públicos (sin sesión) | +| `/p/:key/board` | Tablero kanban (`?release=` y `?assignee=` preseleccionan los filtros). Quien puede contribuir cambia prioridad, estimación y persona desde la propia tarjeta (selectores nativos sobre sus distintivos; el título es el enlace). Mover una tarjeta o cambiar su persona ofrece «Deshacer» unos segundos (o Ctrl/⌘ Z): la orden contraria, que el flujo puede rechazar. Con el teclado: Tab entra en las tarjetas, las flechas se mueven entre ellas, Shift+←/→ pasa la tarjeta de columna (mismas reglas que arrastrar), `p`/`e`/`a` abren su prioridad, estimación o persona y Enter la abre | +| `/p/:key/issues/:issueKey` | Ticket. Encima del título, quién más lo está viendo o editando ahora (`PresenceFacade`: late cada 10 s; editar es tener abierto el formulario del ticket o de un comentario, o estar escribiendo uno). «Fijar» lo deja a mano en el inicio y en la paleta de búsqueda. Arriba, sus decisiones (comentarios marcados con la bandera por quien puede cambiar el ticket); `?comment=` lleva a un comentario y lo señala. Cambiar el estado o la persona y mandarlo a la papelera (sin confirmación) ofrecen «Deshacer». Cada conversación con respuestas se resuelve («✓ Resolver», con una conclusión opcional que queda como decisión) y se pliega en una línea; la cabecera cuenta las abiertas y terminar el ticket con alguna pregunta antes. Se ve quién está escribiendo en cada conversación («Ana está escribiendo…», `PresenceFacade.typists`), y en las respuestas Enter envía (Shift+Enter, otra línea). Si tiene subtareas, «Seguir todo» avisa de ella y de todo lo que cuelga, también lo que se añada (también desde Épicas). «Recordármelo» pone un aviso solo para mí (dentro de 1 hora, esta tarde, mañana o el lunes a las 9, en una semana o cuando diga), con nota opcional. «☀ Mi día» lo añade a los de hoy. «Pedir revisión» a alguien del equipo: lo ve en «Mi trabajo» («Te piden revisión») y responde «Todo bien» o «Pedir cambios» desde el recuadro Revisiones. Un comentario puede llevar una encuesta («📊 Añadir una encuesta»): se vota pulsando una opción y, al cerrarla, la ganadora queda como decisión. «🎥 Grabar pantalla» (con la voz, si se da permiso al micrófono) y «🎙 Grabar audio» graban en el navegador hasta 3 minutos y lo adjuntan al comentario, que puede ir sin texto; los vídeos y audios adjuntos se reproducen ahí mismo. Al abrirlo, lo que otros han hecho desde tu última visita: un aviso arriba («Desde tu última visita, hace 2 días: 3 comentarios y 1 cambio de Ana y Bob», con «Ver lo nuevo»), los comentarios nuevos marcados («Nuevo», también en una conversación plegada) y los cambios del historial resaltados. Cada comentario se puede «Convertir en ticket» (subtarea de este o enlazado, con el título sacado de su primera línea): el comentario enlaza al ticket nuevo («→ Convertido en WEB-45») | +| `/p/:key/releases` | Releases. «Página pública» (quien gestiona): un enlace secreto con las notas para clientes, que se copia, renueva o retira | +| `/p/:key/report/:kind/:id` | Informe para imprimir de un sprint (`sprint`) o una release (`release`): datos, totales, notas y tickets por etapa. «Imprimir o guardar en PDF» abre el diálogo del navegador; al imprimir se quitan la cabecera, las pestañas y los fondos (`styles.scss`) | +| `/p/:key/roadmap` | Hoja de ruta: épicas en el tiempo, con sprints y releases; flechas «bloquea» / «depende de» entre épicas y aviso si una empieza antes de que acabe la que la bloquea. Los bloqueos con tickets de otros proyectos (en los dos sentidos, `externalDependencies`) salen en «De otros proyectos» con barra discontinua, sus flechas y el mismo aviso si las fechas no encajan; los que no tienen fechas o no se pueden leer se nombran debajo | +| `/p/:key/dashboard` | Arriba, la salud del proyecto (`ts-project-health`): un semáforo (bien, atención, en riesgo) con cada causa y sus tickets, calculado en la web (`projectHealth`) con los mismos datos: atascados, vencidos, bloqueados, bugs críticos o altos, lo que devuelve QA, releases que no llegan y columnas sobre su WIP. Seguimiento: cifras, flujo por estado, lo que ha cambiado desde tu última visita (guardada en este navegador), sprint en curso, tickets en riesgo, previsión de las próximas releases, antigüedad de lo que está en curso, cuellos de botella, salud del backlog (cada número abre la lista filtrada: `?missing=estimate`, `area`, `assignee` o `?staleDays=90`), calidad (bugs por prioridad y por semana, QA por área, bugs tras cada release), abiertos por un campo de lista a elegir, el recorrido de un ticket (dónde se va el tiempo de creado a hecho y qué parte es trabajo), la actividad de 12 semanas en un mapa de calor (pulsar un día abre lo que pasó, ticket a ticket), el embudo de propuestas del público (si las hay), equipo con sus puntos abiertos y tipos. Se recarga solo con cada cambio. «Modo pantalla» lo pone a pantalla completa para una tele del equipo: un bloque cada vez, ampliado, que cambia cada 15 s (← → para moverse, espacio para pausar, Esc para salir), con la hora y en directo | +| `/p/:key/analytics` | Estadísticas: creados vs. resueltos, flujo acumulado, burnup de una release, tiempos de ciclo y entrega, QA y resueltos por persona (30 días a 1 año) | +| `/p/:key/settings` | Configuración (propietario y admins; pestaña oculta para el resto), con un menú de secciones (`?section=`): General (visibilidad, propuestas, panel para clientes), Flujo de trabajo (estados, transiciones, definición de «Hecho»), Tickets (tipos, áreas, campos), Equipos, Automatizaciones, Integraciones (Git, alertas → bugs con su URL secreta y cómo conectar Sentry, Uptime Kuma o Grafana, webhooks), Importar y exportar y Zona peligrosa; archivar en lugar de borrar, y un punto en el menú donde hay cambios sin guardar | +| `/p/:key/activity` | Actividad en vivo | +| `/p/:key/propose` | Formulario público para reportar un error o proponer una idea sin cuenta (si el proyecto lo tiene abierto); llegan como tickets con la etiqueta «propuesta» | +| `/share/:token` | Panel para clientes, sin sesión: los bloques que eligió el proyecto (flujo, sprint, releases, épicas, actividad), sin enlaces hacia dentro de la aplicación; se vuelve a leer cada minuto. Si el enlace se renovó o retiró, «Este enlace ya no funciona» | +| `/notes/:token` | Notas de una release para sus clientes, sin cuenta: las escritas o las de sus tickets terminados | +| `/p/:key/members` | Miembros, con su rol (propietario, admin, miembro, lector o invitado). Un invitado de un proyecto privado solo tiene Tablero, Lista y Miembros, ve solo sus tickets y abre tickets sin asignarlos ni planificarlos | Solo `/` está protegida por `authGuard` (redirige a `/login?returnUrl=…`); `/login` y `/auth/callback` son para quien no tiene sesión, y el resto se puede abrir sin ella si el proyecto es público. Los parámetros de ruta llegan como inputs de diff --git a/docs/09-mcp.md b/docs/09-mcp.md index 5f47566..2cec4b4 100644 --- a/docs/09-mcp.md +++ b/docs/09-mcp.md @@ -74,6 +74,7 @@ flowchart LR | | `answer_review` | Responder una revisión: todo bien o cambios | | | `vote_poll` | Votar en la encuesta de un comentario (o retirar el voto) | | | `close_poll` | Cerrar una encuesta; la ganadora queda como decisión | +| | `turn_comment_into_issue` | Convertir un comentario en ticket: subtarea del suyo o enlazado; su texto es la descripción | | | `search_decisions` | Qué se decidió: decisiones por su texto, de un proyecto o de todos los míos | | | `add_checklist_item` / `update_checklist_item` / `remove_checklist_item` | La lista de comprobación: añadir, marcar o reescribir, quitar | | Organización | `list_epics` | Épicas con el progreso de lo que cuelga de cada una | diff --git a/libs/api/tracker/application/src/index.ts b/libs/api/tracker/application/src/index.ts index 1734ab4..7747855 100644 --- a/libs/api/tracker/application/src/index.ts +++ b/libs/api/tracker/application/src/index.ts @@ -43,6 +43,9 @@ export * from './lib/alerts.service'; export * from './lib/follows/tree-follows'; export * from './lib/ports/tree-follows.repository'; export * from './lib/tree-follows.service'; +export * from './lib/visits/issue-visits'; +export * from './lib/ports/issue-visits.repository'; +export * from './lib/issue-visits.service'; export * from './lib/reminders/issue-reminders'; export * from './lib/ports/issue-reminders.repository'; export * from './lib/issue-reminders.service'; @@ -51,6 +54,7 @@ export * from './lib/ports/my-days.repository'; export * from './lib/my-day.service'; export * from './lib/commands/reviews'; export * from './lib/commands/polls'; +export * from './lib/commands/turn-comment-into-issue'; export * from './lib/sharing/release-notes-shares'; export * from './lib/ports/release-note-shares.repository'; export * from './lib/tracker-application.module'; diff --git a/libs/api/tracker/application/src/lib/commands/command-handlers.ts b/libs/api/tracker/application/src/lib/commands/command-handlers.ts index 9042430..12b2c66 100644 --- a/libs/api/tracker/application/src/lib/commands/command-handlers.ts +++ b/libs/api/tracker/application/src/lib/commands/command-handlers.ts @@ -3,6 +3,7 @@ import { MarkDecisionCommandHandler } from './mark-decision.command-handler'; import { ResolveThreadCommandHandler } from './resolve-thread.command-handler'; import { REVIEW_HANDLERS } from './reviews'; import { POLL_HANDLERS } from './polls'; +import { TURN_INTO_ISSUE_HANDLERS } from './turn-comment-into-issue'; import { VoteIssueCommandHandler } from './vote-issue.command-handler'; import { CreateProjectCommandHandler } from './create-project.command-handler'; import { UpdateProjectCommandHandler } from './update-project.command-handler'; @@ -54,6 +55,7 @@ export const COMMAND_HANDLERS = [ ResolveThreadCommandHandler, ...REVIEW_HANDLERS, ...POLL_HANDLERS, + ...TURN_INTO_ISSUE_HANDLERS, CreateProjectCommandHandler, UpdateProjectCommandHandler, UpdateProjectSettingsCommandHandler, diff --git a/libs/api/tracker/application/src/lib/commands/commands.spec.ts b/libs/api/tracker/application/src/lib/commands/commands.spec.ts index 4939343..59f5f44 100644 --- a/libs/api/tracker/application/src/lib/commands/commands.spec.ts +++ b/libs/api/tracker/application/src/lib/commands/commands.spec.ts @@ -15,6 +15,7 @@ import { import { Comment, Issue, + IssueLinkType, IssueStatus, Project, IssuePriority, @@ -81,6 +82,7 @@ import { ReactToCommentCommand } from './react-to-comment.command'; import { MarkDecisionCommand } from './mark-decision.command'; import { ResolveThreadCommand } from './resolve-thread.command'; import { ClosePollCommand, VotePollCommand } from './polls'; +import { TurnCommentIntoIssueCommand } from './turn-comment-into-issue'; import { AnswerReviewCommand, CancelReviewCommand, @@ -951,6 +953,65 @@ describe('tracker commands (event sourced)', () => { ); }); + it('makes a ticket of a comment, as a subtask or linked, and marks the comment', async () => { + const { run, rehydrator } = await setup(); + const projectId = await projectWithTeam(run); + const issue = await run( + new CreateIssueCommand('dev', projectId, IssueType.BUG, 'Pago', {}), + ); + const comment = await run( + new AddCommentCommand('qa', issue.id, 'También falla en **Android**'), + ); + // Viewers comment and react; making tickets is for the team. + await expect( + run( + new TurnCommentIntoIssueCommand( + 'pm', + comment.id, + 'Android', + IssueType.BUG, + false, + ), + ), + ).rejects.toThrow(ForbiddenError); + + const linked = await run( + new TurnCommentIntoIssueCommand( + 'dev', + comment.id, + 'Falla en Android', + IssueType.BUG, + false, + ), + ); + const sub = await run( + new TurnCommentIntoIssueCommand( + 'dev', + comment.id, + 'Probar en Android', + IssueType.TASK, + true, + ), + ); + + const fromLink = await rehydrator.rehydrate(linked.id, Issue); + expect(fromLink.title).toBe('Falla en Android'); + expect(fromLink.description).toBe( + 'También falla en **Android**\n\n---\n_Sale de un comentario de user qa en TS-1._', + ); + expect(fromLink.links).toEqual([ + expect.objectContaining({ + type: IssueLinkType.RELATES_TO, + targetIssueId: issue.id, + }), + ]); + expect((await rehydrator.rehydrate(sub.id, Issue)).parentId).toBe(issue.id); + expect((await rehydrator.rehydrate(comment.id, Comment)).spawned).toEqual([ + { id: linked.id, key: 'TS-2' }, + { id: sub.id, key: 'TS-3' }, + ]); + }); + it('takes votes in a comment poll and keeps the winner as a decision', async () => { const { run, rehydrator, store } = await setup(); const projectId = await projectWithTeam(run); diff --git a/libs/api/tracker/application/src/lib/commands/turn-comment-into-issue.ts b/libs/api/tracker/application/src/lib/commands/turn-comment-into-issue.ts new file mode 100644 index 0000000..bd3776b --- /dev/null +++ b/libs/api/tracker/application/src/lib/commands/turn-comment-into-issue.ts @@ -0,0 +1,94 @@ +import { + Command, + CommandBus, + CommandHandler, + ICommandHandler, +} from '@nestjs/cqrs'; +import { BusinessRuleError } from '@task-stream/api/shared/domain'; +import { IssueLinkType } from '@task-stream/api/tracker/domain'; +import { UserDirectory } from '../ports/user-directory'; +import { CommandResult } from './command-result'; +import { CreateIssueCommand } from './create-issue.command'; +import { LinkIssuesCommand } from './link-issues.command'; +import { WriteAccess } from './write-access'; + +/* + * «Convertir en ticket»: a comment that turns out to be work of its own + * («también falla en Android») becomes a ticket, as a subtask of the one it + * was said on or linked to it. Its text is the new description, and the + * comment shows where it went («→ WEB-45»). + */ + +export class TurnCommentIntoIssueCommand extends Command { + constructor( + public readonly actorId: string, + public readonly commentId: string, + public readonly title: string, + /** Ticket type id of the project's settings. */ + public readonly type: string, + /** Hang it from the ticket of the comment; otherwise, link them. */ + public readonly asSubtask: boolean, + ) { + super(); + } +} + +/** Answers the new ticket (id, key, version). */ +@CommandHandler(TurnCommentIntoIssueCommand) +export class TurnCommentIntoIssueCommandHandler + implements ICommandHandler +{ + constructor( + private readonly access: WriteAccess, + private readonly commandBus: CommandBus, + private readonly users: UserDirectory, + ) {} + + async execute(command: TurnCommentIntoIssueCommand): Promise { + const comment = await this.access.comment(command.commentId); + const source = await this.access.issue(comment.issueId); + const project = await this.access.project(comment.projectId); + this.access.assertSees(project, source, command.actorId); + // Creating and linking tickets is for the team: those commands check it. + project.assertCanContribute(command.actorId); + if (comment.deleted) throw new BusinessRuleError('Comment was deleted'); + + const author = (await this.users.refs([comment.authorId])).of( + comment.authorId, + ); + const created = await this.commandBus.execute( + new CreateIssueCommand( + command.actorId, + project.id, + command.type, + command.title, + { + description: `${comment.body}\n\n---\n_Sale de un comentario de ${author.name} en ${source.key}._`, + }, + null, + null, + command.asSubtask ? source.id : null, + null, + // The comment is the content: no template on top of it. + false, + ), + ); + if (!command.asSubtask) + await this.commandBus.execute( + new LinkIssuesCommand( + command.actorId, + created.id, + IssueLinkType.RELATES_TO, + source.id, + ), + ); + comment.turnedInto(command.actorId, { + id: created.id, + key: created.key ?? '', + }); + await comment.commitAndWait(); + return created; + } +} + +export const TURN_INTO_ISSUE_HANDLERS = [TurnCommentIntoIssueCommandHandler]; diff --git a/libs/api/tracker/application/src/lib/comments.service.ts b/libs/api/tracker/application/src/lib/comments.service.ts index 9bdca55..704d795 100644 --- a/libs/api/tracker/application/src/lib/comments.service.ts +++ b/libs/api/tracker/application/src/lib/comments.service.ts @@ -11,6 +11,7 @@ import { DeleteCommentCommand } from './commands/delete-comment.command'; import { EditCommentCommand } from './commands/edit-comment.command'; import { ReactToCommentCommand } from './commands/react-to-comment.command'; import { MarkDecisionCommand } from './commands/mark-decision.command'; +import { TurnCommentIntoIssueCommand } from './commands/turn-comment-into-issue'; import { ResolveThreadCommand } from './commands/resolve-thread.command'; import { ClosePollCommand, VotePollCommand } from './commands/polls'; import { Page } from './ports/find-issues.repository'; @@ -109,6 +110,25 @@ export class CommentsService { ); } + /** Makes a ticket of a comment: a subtask of its ticket, or linked to it. */ + turnIntoIssue( + actorId: string, + commentId: string, + title: string, + type: string, + asSubtask: boolean, + ): Promise { + return this.commands.execute( + new TurnCommentIntoIssueCommand( + actorId, + commentId, + title, + type, + asSubtask, + ), + ); + } + /** A ticket's decisions, oldest first. */ decisions( viewerId: string | null, diff --git a/libs/api/tracker/application/src/lib/event-handlers/comment-added.event-handler.ts b/libs/api/tracker/application/src/lib/event-handlers/comment-added.event-handler.ts index a1c08ba..4b30ec7 100644 --- a/libs/api/tracker/application/src/lib/event-handlers/comment-added.event-handler.ts +++ b/libs/api/tracker/application/src/lib/event-handlers/comment-added.event-handler.ts @@ -43,6 +43,7 @@ export class CommentAddedEventHandler resolved: false, resolvedBy: null, resolvedAt: null, + spawned: [], createdAt: comment.createdAt, editedAt: null, version: versionOf(event), diff --git a/libs/api/tracker/application/src/lib/event-handlers/comment-turned-into-issue.event-handler.ts b/libs/api/tracker/application/src/lib/event-handlers/comment-turned-into-issue.event-handler.ts new file mode 100644 index 0000000..c66a9ee --- /dev/null +++ b/libs/api/tracker/application/src/lib/event-handlers/comment-turned-into-issue.event-handler.ts @@ -0,0 +1,23 @@ +import { EventsHandler, IEventHandler } from '@nestjs/cqrs'; +import { CommentTurnedIntoIssueEvent } from '@task-stream/api/tracker/domain'; +import { UpsertMaterializedCommentRepository } from '../ports/upsert-materialized-comment.repository'; +import { ReadModelNotifier, versionOf } from './read-model-notifier'; + +@EventsHandler(CommentTurnedIntoIssueEvent) +export class CommentTurnedIntoIssueEventHandler + implements IEventHandler +{ + constructor( + private readonly upsertMaterializedCommentRepository: UpsertMaterializedCommentRepository, + private readonly notifier: ReadModelNotifier, + ) {} + + async handle(event: CommentTurnedIntoIssueEvent) { + await this.upsertMaterializedCommentRepository.upsert({ + id: event.aggregateId, + spawned: event.spawned, + version: versionOf(event), + }); + await this.notifier.comment(event.aggregateId, event.issueId, event); + } +} diff --git a/libs/api/tracker/application/src/lib/event-handlers/event-handlers.ts b/libs/api/tracker/application/src/lib/event-handlers/event-handlers.ts index e4e6341..a2b5587 100644 --- a/libs/api/tracker/application/src/lib/event-handlers/event-handlers.ts +++ b/libs/api/tracker/application/src/lib/event-handlers/event-handlers.ts @@ -5,6 +5,7 @@ import { CommentEditedEventHandler } from './comment-edited.event-handler'; import { CommentReactedEventHandler } from './comment-reacted.event-handler'; import { CommentDecisionMarkedEventHandler } from './comment-decision-marked.event-handler'; import { CommentThreadResolvedEventHandler } from './comment-thread-resolved.event-handler'; +import { CommentTurnedIntoIssueEventHandler } from './comment-turned-into-issue.event-handler'; import { IssueReviewsEventHandler } from './issue-reviews.event-handler'; import { CommentPollEventHandler } from './comment-poll.event-handler'; import { IssueVotedEventHandler } from './issue-voted.event-handler'; @@ -56,6 +57,7 @@ export const EVENT_HANDLERS = [ CommentReactedEventHandler, CommentDecisionMarkedEventHandler, CommentThreadResolvedEventHandler, + CommentTurnedIntoIssueEventHandler, IssueReviewsEventHandler, CommentPollEventHandler, IssueVotedEventHandler, diff --git a/libs/api/tracker/application/src/lib/issue-visits.service.ts b/libs/api/tracker/application/src/lib/issue-visits.service.ts new file mode 100644 index 0000000..36d5712 --- /dev/null +++ b/libs/api/tracker/application/src/lib/issue-visits.service.ts @@ -0,0 +1,14 @@ +import { CommandRunner } from '@task-stream/api/shared/application'; +import { Injectable } from '@nestjs/common'; +import { VisitIssueCommand } from './visits/issue-visits'; + +/** When I last opened a ticket, to tell what is new since. */ +@Injectable() +export class IssueVisitsService { + constructor(private readonly commands: CommandRunner) {} + + /** Notes that I opened it now; returns my visit before (null: never). */ + visit(userId: string, issueId: string): Promise { + return this.commands.execute(new VisitIssueCommand(userId, issueId)); + } +} diff --git a/libs/api/tracker/application/src/lib/ports/issue-visits.repository.ts b/libs/api/tracker/application/src/lib/ports/issue-visits.repository.ts new file mode 100644 index 0000000..d471902 --- /dev/null +++ b/libs/api/tracker/application/src/lib/ports/issue-visits.repository.ts @@ -0,0 +1,11 @@ +/** When someone last opened a ticket: personal, no history. */ +export interface IssueVisit { + userId: string; + issueId: string; + seenAt: string; +} + +export abstract class IssueVisitsRepository { + /** Notes this visit; returns when the one before it was (null: never). */ + abstract visit(visit: IssueVisit): Promise; +} diff --git a/libs/api/tracker/application/src/lib/tracker-application.module.ts b/libs/api/tracker/application/src/lib/tracker-application.module.ts index f99b3c1..953a809 100644 --- a/libs/api/tracker/application/src/lib/tracker-application.module.ts +++ b/libs/api/tracker/application/src/lib/tracker-application.module.ts @@ -38,6 +38,8 @@ import { MY_VIEWS } from './saved-filters/my-views'; import { PINS } from './saved-filters/pins'; import { ALERT_HANDLERS } from './alerts/alerts'; import { TREE_FOLLOWS } from './follows/tree-follows'; +import { ISSUE_VISITS } from './visits/issue-visits'; +import { IssueVisitsService } from './issue-visits.service'; import { ISSUE_REMINDERS } from './reminders/issue-reminders'; import { MY_DAY } from './my-day/my-day'; import { RELEASE_NOTES_SHARES } from './sharing/release-notes-shares'; @@ -92,6 +94,7 @@ const SERVICES = [ PinsService, AlertsService, TreeFollowsService, + IssueVisitsService, IssueRemindersService, MyDayService, SprintsService, @@ -127,6 +130,7 @@ const SERVICES = [ ...PINS, ...ALERT_HANDLERS, ...TREE_FOLLOWS, + ...ISSUE_VISITS, ...ISSUE_REMINDERS, ...MY_DAY, ...RELEASE_NOTES_SHARES, diff --git a/libs/api/tracker/application/src/lib/visits/issue-visits.ts b/libs/api/tracker/application/src/lib/visits/issue-visits.ts new file mode 100644 index 0000000..3e6d8a6 --- /dev/null +++ b/libs/api/tracker/application/src/lib/visits/issue-visits.ts @@ -0,0 +1,51 @@ +import { + Command, + CommandHandler, + ICommandHandler, + QueryBus, +} from '@nestjs/cqrs'; +import { NotFoundError } from '@task-stream/api/shared/domain'; +import { IssueReadModel } from '@task-stream/api/tracker/domain'; +import { IssueVisitsRepository } from '../ports/issue-visits.repository'; +import { GetIssuesByIdsQuery } from '../queries/get-issues-by-ids.query'; + +/* + * «Lo nuevo desde tu última visita»: opening a ticket notes the moment, and + * says when the visit before was, so the client can tell what others did + * meanwhile. Personal, no history (like pinned tickets). + */ + +export class VisitIssueCommand extends Command { + constructor( + public readonly userId: string, + public readonly issueId: string, + ) { + super(); + } +} + +@CommandHandler(VisitIssueCommand) +export class VisitIssueCommandHandler + implements ICommandHandler +{ + constructor( + private readonly visits: IssueVisitsRepository, + private readonly queryBus: QueryBus, + ) {} + + async execute(command: VisitIssueCommand): Promise { + const { userId, issueId } = command; + const [issue]: IssueReadModel[] = await this.queryBus.execute( + new GetIssuesByIdsQuery(userId, [issueId]), + ); + // A ticket I cannot read does not exist, as far as I know. + if (!issue) throw new NotFoundError('Issue', issueId); + return this.visits.visit({ + userId, + issueId, + seenAt: new Date().toISOString(), + }); + } +} + +export const ISSUE_VISITS = [VisitIssueCommandHandler]; diff --git a/libs/api/tracker/domain/src/lib/comment/comment.ts b/libs/api/tracker/domain/src/lib/comment/comment.ts index 0cfee7d..fa3dc71 100644 --- a/libs/api/tracker/domain/src/lib/comment/comment.ts +++ b/libs/api/tracker/domain/src/lib/comment/comment.ts @@ -7,6 +7,7 @@ import { import { CommentAddedEvent } from '../events/comment-added.event'; import { CommentDecisionMarkedEvent } from '../events/comment-decision-marked.event'; import { CommentThreadResolvedEvent } from '../events/comment-thread-resolved.event'; +import { CommentTurnedIntoIssueEvent } from '../events/comment-turned-into-issue.event'; import { CommentPollClosedEvent, CommentPollVotedEvent, @@ -82,6 +83,8 @@ export class Comment extends VersionedAggregateRoot implements CommentProps { public pollVotes: Record = {}; public pollClosed = false; public pollWinner: number | null = null; + /** Tickets made from it, oldest first. */ + public spawned: { id: string; key: string }[] = []; public deleted = false; public createdAt!: string; public editedAt: string | null = null; @@ -277,10 +280,32 @@ export class Comment extends VersionedAggregateRoot implements CommentProps { this.resolved = event.resolved; } + onCommentTurnedIntoIssueEvent(event: CommentTurnedIntoIssueEvent): void { + this.spawned = event.spawned; + } + onCommentDeletedEvent(): void { this.deleted = true; } + /** + * Notes a ticket made from it (its text became the ticket's description). + * Who may create it is the caller's check. + */ + turnedInto(actorId: string, issue: { id: string; key: string }): void { + this.assertAlive(); + if (this.spawned.some((s) => s.id === issue.id)) return; + this.apply( + new CommentTurnedIntoIssueEvent( + actorId, + this.id, + this.projectId, + this.issueId, + [...this.spawned, issue], + ), + ); + } + private assertAlive(): void { if (this.deleted) throw new BusinessRuleError('Comment was deleted'); } diff --git a/libs/api/tracker/domain/src/lib/events/comment-turned-into-issue.event.ts b/libs/api/tracker/domain/src/lib/events/comment-turned-into-issue.event.ts new file mode 100644 index 0000000..79bf804 --- /dev/null +++ b/libs/api/tracker/domain/src/lib/events/comment-turned-into-issue.event.ts @@ -0,0 +1,17 @@ +import { AutowiredEvent } from '@task-stream/api/shared/domain'; +import { TrackerEvent } from './tracker.event'; + +/** A ticket of its own was made from a comment («→ WEB-45» on it). */ +@AutowiredEvent +export class CommentTurnedIntoIssueEvent extends TrackerEvent { + constructor( + actorId: string, + commentId: string, + projectId: string, + readonly issueId: string, + /** Every ticket made from it so far, this one last. */ + readonly spawned: { id: string; key: string }[], + ) { + super(actorId, commentId, projectId); + } +} diff --git a/libs/api/tracker/domain/src/lib/events/index.ts b/libs/api/tracker/domain/src/lib/events/index.ts index 3dd5702..12ca9fc 100644 --- a/libs/api/tracker/domain/src/lib/events/index.ts +++ b/libs/api/tracker/domain/src/lib/events/index.ts @@ -17,6 +17,7 @@ export * from './comment-edited.event'; export * from './comment-reacted.event'; export * from './comment-decision-marked.event'; export * from './comment-thread-resolved.event'; +export * from './comment-turned-into-issue.event'; export * from './issue-review.events'; export * from './comment-poll.events'; export * from './comment-deleted.event'; diff --git a/libs/api/tracker/domain/src/lib/read-models/comment.read-model.ts b/libs/api/tracker/domain/src/lib/read-models/comment.read-model.ts index 66adb6d..a620b16 100644 --- a/libs/api/tracker/domain/src/lib/read-models/comment.read-model.ts +++ b/libs/api/tracker/domain/src/lib/read-models/comment.read-model.ts @@ -41,6 +41,8 @@ export class CommentReadModel { /** Who settled it and when (null while open). */ resolvedBy!: UserRefReadModel | null; resolvedAt!: string | null; + /** Tickets made from it («Convertir en ticket»), oldest first. */ + spawned!: { id: string; key: string }[]; createdAt!: string; editedAt!: string | null; version!: number; diff --git a/libs/api/tracker/infrastructure/src/lib/mongo/find.repositories.ts b/libs/api/tracker/infrastructure/src/lib/mongo/find.repositories.ts index deb7618..b3243f7 100644 --- a/libs/api/tracker/infrastructure/src/lib/mongo/find.repositories.ts +++ b/libs/api/tracker/infrastructure/src/lib/mongo/find.repositories.ts @@ -58,6 +58,7 @@ const commentFromDoc = (doc: CommentDoc): CommentReadModel => ({ resolved: doc.resolved ?? false, resolvedBy: doc.resolvedBy ?? null, resolvedAt: doc.resolvedAt ?? null, + spawned: doc.spawned ?? [], }); const escapeRegex = (text: string) => diff --git a/libs/api/tracker/infrastructure/src/lib/mongo/issue-visits.repository.ts b/libs/api/tracker/infrastructure/src/lib/mongo/issue-visits.repository.ts new file mode 100644 index 0000000..6669822 --- /dev/null +++ b/libs/api/tracker/infrastructure/src/lib/mongo/issue-visits.repository.ts @@ -0,0 +1,30 @@ +import { Injectable } from '@nestjs/common'; +import { InjectModel } from '@nestjs/mongoose'; +import { + IssueVisit, + IssueVisitsRepository, +} from '@task-stream/api/tracker/application'; +import { Model } from 'mongoose'; +import { ReadModels } from './read-model.schemas'; + +@Injectable() +export class MongoIssueVisitsRepository extends IssueVisitsRepository { + constructor( + @InjectModel(ReadModels.IssueVisit) + private readonly visits: Model, + ) { + super(); + } + + async visit(visit: IssueVisit): Promise { + // One round trip: the document as it was before this visit. + const before = await this.visits + .findOneAndUpdate( + { userId: visit.userId, issueId: visit.issueId }, + { $set: { seenAt: visit.seenAt } }, + { upsert: true, new: false }, + ) + .lean(); + return before?.seenAt ?? null; + } +} diff --git a/libs/api/tracker/infrastructure/src/lib/mongo/read-model.schemas.ts b/libs/api/tracker/infrastructure/src/lib/mongo/read-model.schemas.ts index 9da2e6e..d90ac0f 100644 --- a/libs/api/tracker/infrastructure/src/lib/mongo/read-model.schemas.ts +++ b/libs/api/tracker/infrastructure/src/lib/mongo/read-model.schemas.ts @@ -110,6 +110,7 @@ export const ReadModels = { AlertIntegration: 'AlertIntegration', AlertTrace: 'AlertTrace', TreeFollow: 'TreeFollow', + IssueVisit: 'IssueVisit', IssueReminder: 'IssueReminder', MyDay: 'MyDay', ReleaseNotesShare: 'ReleaseNotesShare', @@ -373,6 +374,16 @@ export const TreeFollowSchema = SchemaFactory.createForClass(TreeFollowDocument); TreeFollowSchema.index({ userId: 1, issueId: 1 }, { unique: true }); +@Schema({ ...LOOSE, collection: 'issue_visits' }) +class IssueVisitDocument { + @Prop({ type: String }) userId!: string; + @Prop({ type: String }) issueId!: string; + @Prop({ type: String }) seenAt!: string; +} +export const IssueVisitSchema = + SchemaFactory.createForClass(IssueVisitDocument); +IssueVisitSchema.index({ userId: 1, issueId: 1 }, { unique: true }); + @Schema({ ...LOOSE, collection: 'issue_reminders' }) class IssueReminderDocument { @Prop({ type: String, required: true, unique: true }) id!: string; @@ -511,6 +522,7 @@ export const READ_MODEL_SCHEMAS = [ { name: ReadModels.AlertIntegration, schema: AlertIntegrationSchema }, { name: ReadModels.AlertTrace, schema: AlertTraceSchema }, { name: ReadModels.TreeFollow, schema: TreeFollowSchema }, + { name: ReadModels.IssueVisit, schema: IssueVisitSchema }, { name: ReadModels.IssueReminder, schema: IssueReminderSchema }, { name: ReadModels.MyDay, schema: MyDaySchema }, { name: ReadModels.ReleaseNotesShare, schema: ReleaseNotesShareSchema }, diff --git a/libs/api/tracker/infrastructure/src/lib/tracker-infrastructure.module.ts b/libs/api/tracker/infrastructure/src/lib/tracker-infrastructure.module.ts index b67fe0e..9340199 100644 --- a/libs/api/tracker/infrastructure/src/lib/tracker-infrastructure.module.ts +++ b/libs/api/tracker/infrastructure/src/lib/tracker-infrastructure.module.ts @@ -16,6 +16,7 @@ import { PinsRepository, AlertIntegrationsRepository, TreeFollowsRepository, + IssueVisitsRepository, IssueRemindersRepository, MyDaysRepository, ReleaseNoteSharesRepository, @@ -87,6 +88,7 @@ import { MongoMyViewsRepository } from './mongo/my-views.repository'; import { MongoPinsRepository } from './mongo/pins.repository'; import { MongoAlertIntegrationsRepository } from './mongo/alert-integrations.repository'; import { MongoTreeFollowsRepository } from './mongo/tree-follows.repository'; +import { MongoIssueVisitsRepository } from './mongo/issue-visits.repository'; import { MongoIssueRemindersRepository } from './mongo/issue-reminders.repository'; import { MongoMyDaysRepository } from './mongo/my-days.repository'; import { MongoReleaseNoteSharesRepository } from './mongo/release-note-shares.repository'; @@ -172,6 +174,7 @@ const PORTS = [ useClass: MongoAlertIntegrationsRepository, }, { provide: TreeFollowsRepository, useClass: MongoTreeFollowsRepository }, + { provide: IssueVisitsRepository, useClass: MongoIssueVisitsRepository }, { provide: IssueRemindersRepository, useClass: MongoIssueRemindersRepository, diff --git a/libs/api/tracker/presenters/src/lib/graphql/comment.resolver.ts b/libs/api/tracker/presenters/src/lib/graphql/comment.resolver.ts index c7cf5ed..c025dd1 100644 --- a/libs/api/tracker/presenters/src/lib/graphql/comment.resolver.ts +++ b/libs/api/tracker/presenters/src/lib/graphql/comment.resolver.ts @@ -113,6 +113,27 @@ export class CommentResolver { ); } + @Mutation(() => CommandResultGql, { + description: + 'Make a ticket of a comment: its text becomes the description, as a subtask of its ticket or linked to it (RELATES_TO). Answers the new ticket. Whoever can change tickets', + }) + turnCommentIntoIssue( + @CurrentUser() me: User, + @Args('commentId', { type: () => ID }) commentId: string, + @Args('title') title: string, + @Args('type', { description: "One of the project's ticket type ids" }) + type: string, + @Args('asSubtask', { defaultValue: false }) asSubtask: boolean, + ) { + return this.comments.turnIntoIssue( + me.id, + commentId, + title, + type, + asSubtask, + ); + } + @OptionalAuth() @Query(() => [DecisionGql], { description: diff --git a/libs/api/tracker/presenters/src/lib/graphql/issue.resolver.ts b/libs/api/tracker/presenters/src/lib/graphql/issue.resolver.ts index 9f5fb12..b3d7742 100644 --- a/libs/api/tracker/presenters/src/lib/graphql/issue.resolver.ts +++ b/libs/api/tracker/presenters/src/lib/graphql/issue.resolver.ts @@ -20,6 +20,7 @@ import { PinsService, TreeFollowsService, IssueRemindersService, + IssueVisitsService, MyDayService, PresenceService, WorkflowsService, @@ -73,6 +74,7 @@ export class IssueResolver { private readonly treeFollows: TreeFollowsService, private readonly reminders: IssueRemindersService, private readonly myDayService: MyDayService, + private readonly visits: IssueVisitsService, ) {} @Mutation(() => [PresenceGql], { @@ -452,6 +454,18 @@ export class IssueResolver { return true; } + @Mutation(() => String, { + nullable: true, + description: + 'I opened this ticket now: answers when I had opened it before (null: never), to show what is new since', + }) + visitIssue( + @CurrentUser() me: User, + @Args('issueId', { type: () => ID }) issueId: string, + ): Promise { + return this.visits.visit(me.id, issueId); + } + @Query(() => MyDayGql, { description: '«Mi día»: the tickets I picked for today, and what was left on my last day', diff --git a/libs/api/tracker/presenters/src/lib/graphql/types.ts b/libs/api/tracker/presenters/src/lib/graphql/types.ts index 64ff453..17ff1d7 100644 --- a/libs/api/tracker/presenters/src/lib/graphql/types.ts +++ b/libs/api/tracker/presenters/src/lib/graphql/types.ts @@ -572,6 +572,12 @@ export class CommentPollGql { winner!: number | null; } +@ObjectType('CommentSpawn', { description: 'A ticket made from a comment' }) +export class CommentSpawnGql { + @Field(() => ID) id!: string; + @Field() key!: string; +} + @ObjectType('Comment') export class CommentGql { @Field(() => ID) id!: string; @@ -615,6 +621,10 @@ export class CommentGql { }) resolvedBy!: UserRefGql | null; @Field(() => String, { nullable: true }) resolvedAt!: string | null; + @Field(() => [CommentSpawnGql], { + description: 'Tickets made from it («Convertir en ticket»), oldest first', + }) + spawned!: CommentSpawnGql[]; @Field() createdAt!: string; @Field(() => String, { nullable: true }) editedAt!: string | null; @Field(() => Int) version!: number; diff --git a/libs/api/tracker/presenters/src/lib/http/comments.controller.ts b/libs/api/tracker/presenters/src/lib/http/comments.controller.ts index 14101fa..72b2e98 100644 --- a/libs/api/tracker/presenters/src/lib/http/comments.controller.ts +++ b/libs/api/tracker/presenters/src/lib/http/comments.controller.ts @@ -22,6 +22,7 @@ import { ClosePollDto, ReactToCommentDto, ResolveThreadDto, + TurnCommentIntoIssueDto, VotePollDto, } from './dto/requests.dto'; import { CommandResultDto } from './dto/responses.dto'; @@ -117,6 +118,28 @@ export class CommentsController { return this.comments.closePoll(me.id, commentId, body.decide ?? true); } + @Post(':commentId/issue') + @ApiOperation({ + summary: 'Make a ticket of a comment', + description: + 'Its text becomes the description, as a subtask of its ticket or linked to it (RELATES_TO); the comment shows where it went. Answers the new ticket. Whoever can change tickets.', + }) + @ApiOkResponse({ type: CommandResultDto }) + @ApiCommandErrors() + turnIntoIssue( + @CurrentUser() me: User, + @Param('commentId') commentId: string, + @Body() body: TurnCommentIntoIssueDto, + ) { + return this.comments.turnIntoIssue( + me.id, + commentId, + body.title, + body.type, + body.asSubtask ?? false, + ); + } + @Put(':commentId/resolution') @ApiOperation({ summary: 'Settle a conversation (a top-level comment and its replies)', diff --git a/libs/api/tracker/presenters/src/lib/http/dto/requests.dto.ts b/libs/api/tracker/presenters/src/lib/http/dto/requests.dto.ts index 824f6b0..ccc97b4 100644 --- a/libs/api/tracker/presenters/src/lib/http/dto/requests.dto.ts +++ b/libs/api/tracker/presenters/src/lib/http/dto/requests.dto.ts @@ -1640,6 +1640,27 @@ export class SetReminderDto { note?: string; } +export class TurnCommentIntoIssueDto { + @ApiProperty({ example: 'También falla en Android', maxLength: 200 }) + @IsString() + @MinLength(1) + @MaxLength(200) + title!: string; + + @ApiProperty({ description: "One of the project's ticket type ids" }) + @IsString() + type!: string; + + @ApiPropertyOptional({ + default: false, + description: + 'Hang it from the ticket of the comment; otherwise it is linked to it (RELATES_TO)', + }) + @IsOptional() + @IsBoolean() + asSubtask?: boolean; +} + export class ResolveThreadDto { @ApiPropertyOptional({ description: diff --git a/libs/api/tracker/presenters/src/lib/http/dto/responses.dto.ts b/libs/api/tracker/presenters/src/lib/http/dto/responses.dto.ts index 45f940f..8fdc936 100644 --- a/libs/api/tracker/presenters/src/lib/http/dto/responses.dto.ts +++ b/libs/api/tracker/presenters/src/lib/http/dto/responses.dto.ts @@ -628,6 +628,10 @@ export class CommentPollDto { winner!: number | null; } +export class CommentSpawnDto { + @ApiProperty() id!: string; + @ApiProperty({ example: 'WEB-45' }) key!: string; +} export class CommentDto { @ApiProperty({ format: 'uuid' }) id!: string; @ApiProperty({ format: 'uuid' }) issueId!: string; @@ -679,6 +683,11 @@ export class CommentDto { resolvedBy!: UserRefDto | null; @ApiProperty({ type: String, nullable: true, format: 'date-time' }) resolvedAt!: string | null; + @ApiProperty({ + type: () => [CommentSpawnDto], + description: 'Tickets made from it («Convertir en ticket»), oldest first', + }) + spawned!: CommentSpawnDto[]; @ApiProperty({ format: 'date-time' }) createdAt!: string; @ApiProperty({ type: String, nullable: true, format: 'date-time' }) editedAt!: string | null; @@ -1447,6 +1456,16 @@ export class IssueReminderDto { @ApiProperty({ type: String, nullable: true }) note!: string | null; } +export class IssueVisitDto { + @ApiProperty({ + type: String, + format: 'date-time', + nullable: true, + description: 'My visit before this one (null: first time)', + }) + previousVisit!: string | null; +} + export class ReminderOnIssueDto { @ApiProperty({ type: IssueReminderDto }) reminder!: IssueReminderDto; @ApiProperty({ type: () => IssueDto }) issue!: IssueDto; diff --git a/libs/api/tracker/presenters/src/lib/http/issues.controller.ts b/libs/api/tracker/presenters/src/lib/http/issues.controller.ts index 82a94b9..d2ea8be 100644 --- a/libs/api/tracker/presenters/src/lib/http/issues.controller.ts +++ b/libs/api/tracker/presenters/src/lib/http/issues.controller.ts @@ -35,6 +35,7 @@ import { PinsService, TreeFollowsService, IssueRemindersService, + IssueVisitsService, MyDayService, PresenceService, ProjectsService, @@ -73,6 +74,7 @@ import { CommentPageDto, DecisionDto, IssueReminderDto, + IssueVisitDto, MyDayDto, ReminderOnIssueDto, EpicDto, @@ -112,6 +114,7 @@ export class IssuesController { private readonly treeFollows: TreeFollowsService, private readonly reminders: IssueRemindersService, private readonly myDayService: MyDayService, + private readonly visits: IssueVisitsService, ) {} @Get('me/home') @@ -382,6 +385,27 @@ export class IssuesController { ); } + @Put('issues/:issue/visit') + @ISSUE + @ApiOperation({ + summary: 'I opened this ticket now', + description: + 'Answers when I had opened it before (null: never), to show what others did since. Private to me.', + }) + @ApiOkResponse({ type: IssueVisitDto }) + @ApiCommandErrors() + async visit( + @CurrentUser() me: User, + @Param('issue') issue: string, + ): Promise { + return { + previousVisit: await this.visits.visit( + me.id, + await this.refs.issueId(me.id, issue), + ), + }; + } + @Put('issues/:issue/follow-tree') @HttpCode(204) @ISSUE diff --git a/libs/web/tracker/application/src/lib/events/issue.events.ts b/libs/web/tracker/application/src/lib/events/issue.events.ts index 59909a2..a1c18b7 100644 --- a/libs/web/tracker/application/src/lib/events/issue.events.ts +++ b/libs/web/tracker/application/src/lib/events/issue.events.ts @@ -110,6 +110,12 @@ export const issuePageEvents = eventGroup({ resolved: boolean; conclusion: string | null; }>(), + commentToIssueRequested: type<{ + commentId: string; + title: string; + type: string; + asSubtask: boolean; + }>(), voteRequested: type<{ issueId: string; on: boolean }>(), /** An image pasted into a text was uploaded (it joins the ticket's files). */ imagePasted: type<{ issueId: string }>(), diff --git a/libs/web/tracker/application/src/lib/facades/issue.facade.ts b/libs/web/tracker/application/src/lib/facades/issue.facade.ts index d38eff6..0041329 100644 --- a/libs/web/tracker/application/src/lib/facades/issue.facade.ts +++ b/libs/web/tracker/application/src/lib/facades/issue.facade.ts @@ -316,6 +316,21 @@ export class IssueFacade { }); } + /** Makes a ticket of a comment: a subtask of this one, or linked to it. */ + turnCommentIntoIssue( + commentId: string, + title: string, + type: string, + asSubtask: boolean, + ): void { + this.dispatch.commentToIssueRequested({ + commentId, + title: title.trim(), + type, + asSubtask, + }); + } + /** Marks a comment as a decision of the ticket (`on`) or unmarks it. */ markDecision(commentId: string, on: boolean): void { this.dispatch.decisionRequested({ commentId, on }); diff --git a/libs/web/tracker/application/src/lib/stores/issue.store.ts b/libs/web/tracker/application/src/lib/stores/issue.store.ts index 1d12e30..9ccaedb 100644 --- a/libs/web/tracker/application/src/lib/stores/issue.store.ts +++ b/libs/web/tracker/application/src/lib/stores/issue.store.ts @@ -608,6 +608,28 @@ export const IssueStore = signalStore( ), ), ), + events.on(issuePageEvents.commentToIssueRequested).pipe( + mergeMap(({ payload }) => + comments + .turnIntoIssue( + payload.commentId, + payload.title, + payload.type, + payload.asSubtask, + ) + .pipe( + mapResponse({ + next: (result) => + done( + result.key + ? `Creado ${result.key}` + : 'Ticket creado', + ), + error: (error) => failed(describeError(error)), + }), + ), + ), + ), events.on(issuePageEvents.reactRequested).pipe( mergeMap(({ payload }) => comments.react(payload.commentId, payload.emoji, payload.on).pipe( diff --git a/libs/web/tracker/domain/src/index.ts b/libs/web/tracker/domain/src/index.ts index 03dc35f..e5a16b0 100644 --- a/libs/web/tracker/domain/src/index.ts +++ b/libs/web/tracker/domain/src/index.ts @@ -4,6 +4,7 @@ export * from './lib/models/bulk'; export * from './lib/models/reminders'; export * from './lib/models/my-day'; export * from './lib/models/health'; +export * from './lib/models/news'; export * from './lib/models/inputs'; export * from './lib/models/workflow'; export * from './lib/ports/gateways'; diff --git a/libs/web/tracker/domain/src/lib/models/models.ts b/libs/web/tracker/domain/src/lib/models/models.ts index fe8abf2..64b963d 100644 --- a/libs/web/tracker/domain/src/lib/models/models.ts +++ b/libs/web/tracker/domain/src/lib/models/models.ts @@ -385,6 +385,8 @@ export interface Comment { resolved: boolean; /** Who settled it (null while open). */ resolvedBy: UserRef | null; + /** Tickets made from it («Convertir en ticket»), oldest first. */ + spawned: { id: string; key: string }[]; createdAt: string; editedAt: string | null; } diff --git a/libs/web/tracker/domain/src/lib/models/news.spec.ts b/libs/web/tracker/domain/src/lib/models/news.spec.ts new file mode 100644 index 0000000..a2a715e --- /dev/null +++ b/libs/web/tracker/domain/src/lib/models/news.spec.ts @@ -0,0 +1,86 @@ +import { Activity, Comment } from './models'; +import { newsSince, titleFrom } from './news'; + +const user = (id: string, name: string) => ({ id, name }); +const comment = ( + id: string, + by: string, + at: string, + deleted = false, +): Comment => + ({ + id, + author: user(by, by === 'me' ? 'Yo' : by === 'ana' ? 'Ana' : 'Bob'), + createdAt: at, + deleted, + }) as Comment; +const change = (id: string, by: string, at: string, type: string): Activity => + ({ + id, + type, + actor: user(by, by === 'ana' ? 'Ana' : 'Bob'), + occurredAt: at, + }) as Activity; + +describe('news since my last visit', () => { + const since = '2026-10-20T10:00:00.000Z'; + + it('counts what others did since, not mine nor what came before', () => { + const news = newsSince( + [ + comment('old', 'ana', '2026-10-19T09:00:00.000Z'), + comment('mine', 'me', '2026-10-21T09:00:00.000Z'), + comment('c1', 'bob', '2026-10-21T09:30:00.000Z'), + comment('gone', 'ana', '2026-10-21T09:40:00.000Z', true), + comment('c2', 'ana', '2026-10-21T10:00:00.000Z'), + ], + [ + change( + 'h1', + 'ana', + '2026-10-21T08:00:00.000Z', + 'IssueStatusChangedEvent', + ), + // The comment itself is already counted as a comment. + change('h2', 'bob', '2026-10-21T09:30:00.000Z', 'CommentAddedEvent'), + change('h3', 'bob', '2026-10-19T08:00:00.000Z', 'IssueAssignedEvent'), + ], + since, + 'me', + ); + expect(news).toEqual({ + since, + what: '2 comentarios y 1 cambio', + who: 'Bob y Ana', + firstComment: 'c1', + }); + }); + + it('says nothing the first time, or when nothing happened', () => { + const comments = [comment('c1', 'ana', '2026-10-21T09:00:00.000Z')]; + expect(newsSince(comments, [], null, 'me')).toBeNull(); + expect( + newsSince(comments, [], '2026-10-22T00:00:00.000Z', 'me'), + ).toBeNull(); + }); +}); + +describe('titleFrom', () => { + it('takes the first line without Markdown', () => { + expect( + titleFrom( + '\n## También falla en **Android** ([log](https://x.dev))\nMás', + ), + ).toBe('También falla en Android (log)'); + expect(titleFrom('- `npm ci` rompe en CI')).toBe('npm ci rompe en CI'); + expect(titleFrom('También falla en Android.\nAdjunto el log.')).toBe( + 'También falla en Android', + ); + }); + + it('cuts long lines at a word', () => { + const title = titleFrom('palabra '.repeat(30)); + expect(title.length).toBeLessThanOrEqual(121); + expect(title.endsWith('palabra…')).toBe(true); + }); +}); diff --git a/libs/web/tracker/domain/src/lib/models/news.ts b/libs/web/tracker/domain/src/lib/models/news.ts new file mode 100644 index 0000000..fefce94 --- /dev/null +++ b/libs/web/tracker/domain/src/lib/models/news.ts @@ -0,0 +1,102 @@ +import { Activity, Comment } from './models'; + +/* + * «Lo nuevo desde tu última visita»: what others did on a ticket since I + * last opened it. My own comments and changes are not news to me. + */ + +/** A comment someone else wrote since (deleted ones say nothing). */ +export function isNewComment( + comment: Comment, + since: string | null, + me: string | null, +): boolean { + return ( + !!since && + !comment.deleted && + comment.createdAt > since && + comment.author.id !== me + ); +} + +/** A change someone else made since; comments count on their own. */ +export function isNewChange( + entry: Activity, + since: string | null, + me: string | null, +): boolean { + return ( + !!since && + entry.occurredAt > since && + entry.actor.id !== me && + !entry.type.startsWith('Comment') + ); +} + +export interface IssueNews { + since: string; + /** «2 comentarios y 1 cambio». */ + what: string; + /** «Ana y Bob». */ + who: string; + /** The first new comment, to go to it. */ + firstComment: string | null; +} + +/** What is new since, or null when nothing is (or it is my first time). */ +export function newsSince( + comments: Comment[], + history: Activity[], + since: string | null, + me: string | null, +): IssueNews | null { + if (!since) return null; + const fresh = comments.filter((c) => isNewComment(c, since, me)); + const changes = history.filter((e) => isNewChange(e, since, me)); + if (!fresh.length && !changes.length) return null; + const what = [ + count(fresh.length, 'comentario', 'comentarios'), + count(changes.length, 'cambio', 'cambios'), + ].filter(Boolean); + const who = [ + ...new Set([ + ...fresh.map((c) => c.author.name), + ...changes.map((e) => e.actor.name), + ]), + ]; + return { + since, + what: what.join(' y '), + who: + who.length > 1 + ? `${who.slice(0, -1).join(', ')} y ${who[who.length - 1]}` + : who[0], + firstComment: fresh[0]?.id ?? null, + }; +} + +function count(n: number, one: string, many: string): string { + return n === 1 ? `1 ${one}` : n ? `${n} ${many}` : ''; +} + +/** A ticket title out of a comment: its first line, without Markdown. */ +export function titleFrom(body: string): string { + const line = + body + .split('\n') + .map((l) => + l + .replace(/!\[[^\]]*\]\([^)]*\)/g, '') + .replace(/\[([^\]]*)\]\([^)]*\)/g, '$1') + .replace(/^\s*(#+|>|[-*+]|\d+\.)\s*/, '') + .replace(/[*_`~]/g, '') + .trim() + // A title ends without the sentence's stop. + .replace(/[.:;,]+$/, ''), + ) + .find(Boolean) ?? ''; + if (line.length <= 120) return line; + const cut = line.slice(0, 120); + const space = cut.lastIndexOf(' '); + return `${space > 60 ? cut.slice(0, space) : cut}…`; +} diff --git a/libs/web/tracker/domain/src/lib/ports/gateways.ts b/libs/web/tracker/domain/src/lib/ports/gateways.ts index 8ef3e52..78b8e03 100644 --- a/libs/web/tracker/domain/src/lib/ports/gateways.ts +++ b/libs/web/tracker/domain/src/lib/ports/gateways.ts @@ -170,6 +170,8 @@ export abstract class IssuesGateway { abstract pin(issueId: string, pinned: boolean): Observable; /** Follows the ticket with everything under it (or stops). */ abstract followTree(issueId: string, on: boolean): Observable; + /** I opened it now: answers my visit before (null: never). */ + abstract visit(issueId: string): Observable; /** Reminds me of the ticket then (replaces the waiting one). */ abstract setReminder( issueId: string, @@ -368,6 +370,13 @@ export abstract class CommentsGateway { resolved: boolean, conclusion: string | null, ): Observable; + /** Makes a ticket of it: a subtask of its ticket, or linked. Answers the new one. */ + abstract turnIntoIssue( + commentId: string, + title: string, + type: string, + asSubtask: boolean, + ): Observable; /** Marks it as a decision of its ticket (`on`) or unmarks it. */ abstract markDecision( commentId: string, diff --git a/libs/web/tracker/infrastructure/src/lib/adapters/graphql-collaboration.gateways.ts b/libs/web/tracker/infrastructure/src/lib/adapters/graphql-collaboration.gateways.ts index 965de3a..938e12d 100644 --- a/libs/web/tracker/infrastructure/src/lib/adapters/graphql-collaboration.gateways.ts +++ b/libs/web/tracker/infrastructure/src/lib/adapters/graphql-collaboration.gateways.ts @@ -25,6 +25,7 @@ import { ReactToCommentDocument, MarkDecisionDocument, ResolveThreadDocument, + TurnCommentIntoIssueDocument, VotePollDocument, ClosePollDocument, PublishReleaseDocument, @@ -111,6 +112,22 @@ export class GraphqlCommentsGateway extends CommentsGateway { .pipe(map((d) => toCommandResult(d.resolveThread))); } + turnIntoIssue( + commentId: string, + title: string, + type: string, + asSubtask: boolean, + ): Observable { + return this.gql + .mutate(TurnCommentIntoIssueDocument, { + commentId, + title, + type, + asSubtask, + }) + .pipe(map((d) => toCommandResult(d.turnCommentIntoIssue))); + } + markDecision(commentId: string, on: boolean): Observable { return this.gql .mutate(MarkDecisionDocument, { commentId, on }) diff --git a/libs/web/tracker/infrastructure/src/lib/adapters/graphql-issues.gateway.ts b/libs/web/tracker/infrastructure/src/lib/adapters/graphql-issues.gateway.ts index 6b31113..63ed78f 100644 --- a/libs/web/tracker/infrastructure/src/lib/adapters/graphql-issues.gateway.ts +++ b/libs/web/tracker/infrastructure/src/lib/adapters/graphql-issues.gateway.ts @@ -57,6 +57,7 @@ import { CancelReviewDocument, MyPinsDocument, FollowTreeDocument, + VisitIssueDocument, SetReminderDocument, MyDayDocument, AddToMyDayDocument, @@ -272,6 +273,12 @@ export class GraphqlIssuesGateway extends IssuesGateway { .pipe(map(() => undefined)); } + visit(issueId: string): Observable { + return this.gql + .mutate(VisitIssueDocument, { issueId }) + .pipe(map((d) => d.visitIssue ?? null)); + } + followTree(issueId: string, on: boolean): Observable { const done$: Observable = on ? this.gql.mutate(FollowTreeDocument, { issueId }) diff --git a/libs/web/tracker/infrastructure/src/lib/graphql/fragments.graphql b/libs/web/tracker/infrastructure/src/lib/graphql/fragments.graphql index cb8157f..d6a797a 100644 --- a/libs/web/tracker/infrastructure/src/lib/graphql/fragments.graphql +++ b/libs/web/tracker/infrastructure/src/lib/graphql/fragments.graphql @@ -218,6 +218,10 @@ fragment CommentFields on Comment { resolvedBy { ...UserRefFields } + spawned { + id + key + } createdAt editedAt author { diff --git a/libs/web/tracker/infrastructure/src/lib/graphql/generated.ts b/libs/web/tracker/infrastructure/src/lib/graphql/generated.ts index 8b20dea..8823a5e 100644 --- a/libs/web/tracker/infrastructure/src/lib/graphql/generated.ts +++ b/libs/web/tracker/infrastructure/src/lib/graphql/generated.ts @@ -754,6 +754,7 @@ export type IssueFieldsFragment = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; }>; reporter: { id: string; name: string }; @@ -817,6 +818,7 @@ export type CommentFieldsFragment = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; }; @@ -1027,6 +1029,7 @@ export type IssueQuery = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; }>; reporter: { id: string; name: string }; @@ -1398,6 +1401,7 @@ export type IssueCommentsQuery = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; }>; }; @@ -1830,6 +1834,23 @@ export type ResolveThreadMutation = { resolveThread: { id: string; key: string | null; version: number }; }; +export type TurnCommentIntoIssueMutationVariables = Exact<{ + commentId: string; + title: string; + type: string; + asSubtask: boolean; +}>; + +export type TurnCommentIntoIssueMutation = { + turnCommentIntoIssue: { id: string; key: string | null; version: number }; +}; + +export type VisitIssueMutationVariables = Exact<{ + issueId: string; +}>; + +export type VisitIssueMutation = { visitIssue: string | null }; + export type MarkDecisionMutationVariables = Exact<{ commentId: string; on: boolean; @@ -1870,6 +1891,7 @@ export type SearchDecisionsQuery = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; }; issue: { id: string; key: string; title: string }; @@ -3254,6 +3276,7 @@ export type CommentChangedSubscription = { }>; } | null; resolvedBy: { id: string; name: string } | null; + spawned: Array<{ id: string; key: string }>; author: { id: string; name: string }; } | null; }; @@ -4091,6 +4114,17 @@ export const CommentFieldsFragmentDoc = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { @@ -4536,6 +4570,17 @@ export const IssueFieldsFragmentDoc = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { @@ -6853,6 +6898,17 @@ export const IssueDocument = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { @@ -9204,6 +9260,17 @@ export const IssueCommentsDocument = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { @@ -12014,6 +12081,181 @@ export const ResolveThreadDocument = { ResolveThreadMutation, ResolveThreadMutationVariables >; +export const TurnCommentIntoIssueDocument = { + kind: 'Document', + definitions: [ + { + kind: 'OperationDefinition', + operation: 'mutation', + name: { kind: 'Name', value: 'TurnCommentIntoIssue' }, + variableDefinitions: [ + { + kind: 'VariableDefinition', + variable: { + kind: 'Variable', + name: { kind: 'Name', value: 'commentId' }, + }, + type: { + kind: 'NonNullType', + type: { kind: 'NamedType', name: { kind: 'Name', value: 'ID' } }, + }, + }, + { + kind: 'VariableDefinition', + variable: { + kind: 'Variable', + name: { kind: 'Name', value: 'title' }, + }, + type: { + kind: 'NonNullType', + type: { + kind: 'NamedType', + name: { kind: 'Name', value: 'String' }, + }, + }, + }, + { + kind: 'VariableDefinition', + variable: { kind: 'Variable', name: { kind: 'Name', value: 'type' } }, + type: { + kind: 'NonNullType', + type: { + kind: 'NamedType', + name: { kind: 'Name', value: 'String' }, + }, + }, + }, + { + kind: 'VariableDefinition', + variable: { + kind: 'Variable', + name: { kind: 'Name', value: 'asSubtask' }, + }, + type: { + kind: 'NonNullType', + type: { + kind: 'NamedType', + name: { kind: 'Name', value: 'Boolean' }, + }, + }, + }, + ], + selectionSet: { + kind: 'SelectionSet', + selections: [ + { + kind: 'Field', + name: { kind: 'Name', value: 'turnCommentIntoIssue' }, + arguments: [ + { + kind: 'Argument', + name: { kind: 'Name', value: 'commentId' }, + value: { + kind: 'Variable', + name: { kind: 'Name', value: 'commentId' }, + }, + }, + { + kind: 'Argument', + name: { kind: 'Name', value: 'title' }, + value: { + kind: 'Variable', + name: { kind: 'Name', value: 'title' }, + }, + }, + { + kind: 'Argument', + name: { kind: 'Name', value: 'type' }, + value: { + kind: 'Variable', + name: { kind: 'Name', value: 'type' }, + }, + }, + { + kind: 'Argument', + name: { kind: 'Name', value: 'asSubtask' }, + value: { + kind: 'Variable', + name: { kind: 'Name', value: 'asSubtask' }, + }, + }, + ], + selectionSet: { + kind: 'SelectionSet', + selections: [ + { + kind: 'FragmentSpread', + name: { kind: 'Name', value: 'CommandResultFields' }, + }, + ], + }, + }, + ], + }, + }, + { + kind: 'FragmentDefinition', + name: { kind: 'Name', value: 'CommandResultFields' }, + typeCondition: { + kind: 'NamedType', + name: { kind: 'Name', value: 'CommandResult' }, + }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + { kind: 'Field', name: { kind: 'Name', value: 'version' } }, + ], + }, + }, + ], +} as unknown as DocumentNode< + TurnCommentIntoIssueMutation, + TurnCommentIntoIssueMutationVariables +>; +export const VisitIssueDocument = { + kind: 'Document', + definitions: [ + { + kind: 'OperationDefinition', + operation: 'mutation', + name: { kind: 'Name', value: 'VisitIssue' }, + variableDefinitions: [ + { + kind: 'VariableDefinition', + variable: { + kind: 'Variable', + name: { kind: 'Name', value: 'issueId' }, + }, + type: { + kind: 'NonNullType', + type: { kind: 'NamedType', name: { kind: 'Name', value: 'ID' } }, + }, + }, + ], + selectionSet: { + kind: 'SelectionSet', + selections: [ + { + kind: 'Field', + name: { kind: 'Name', value: 'visitIssue' }, + arguments: [ + { + kind: 'Argument', + name: { kind: 'Name', value: 'issueId' }, + value: { + kind: 'Variable', + name: { kind: 'Name', value: 'issueId' }, + }, + }, + ], + }, + ], + }, + }, + ], +} as unknown as DocumentNode; export const MarkDecisionDocument = { kind: 'Document', definitions: [ @@ -12304,6 +12546,17 @@ export const SearchDecisionsDocument = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { @@ -19548,6 +19801,17 @@ export const CommentChangedDocument = { ], }, }, + { + kind: 'Field', + name: { kind: 'Name', value: 'spawned' }, + selectionSet: { + kind: 'SelectionSet', + selections: [ + { kind: 'Field', name: { kind: 'Name', value: 'id' } }, + { kind: 'Field', name: { kind: 'Name', value: 'key' } }, + ], + }, + }, { kind: 'Field', name: { kind: 'Name', value: 'createdAt' } }, { kind: 'Field', name: { kind: 'Name', value: 'editedAt' } }, { diff --git a/libs/web/tracker/infrastructure/src/lib/graphql/issues.graphql b/libs/web/tracker/infrastructure/src/lib/graphql/issues.graphql index 542fbcb..7d326ca 100644 --- a/libs/web/tracker/infrastructure/src/lib/graphql/issues.graphql +++ b/libs/web/tracker/infrastructure/src/lib/graphql/issues.graphql @@ -396,6 +396,26 @@ mutation ResolveThread( } } +mutation TurnCommentIntoIssue( + $commentId: ID! + $title: String! + $type: String! + $asSubtask: Boolean! +) { + turnCommentIntoIssue( + commentId: $commentId + title: $title + type: $type + asSubtask: $asSubtask + ) { + ...CommandResultFields + } +} + +mutation VisitIssue($issueId: ID!) { + visitIssue(issueId: $issueId) +} + mutation MarkDecision($commentId: ID!, $on: Boolean!) { markDecision(commentId: $commentId, on: $on) { ...CommandResultFields diff --git a/libs/web/tracker/presenters/src/lib/pages/issue.page.scss b/libs/web/tracker/presenters/src/lib/pages/issue.page.scss index f083b14..0ae00ac 100644 --- a/libs/web/tracker/presenters/src/lib/pages/issue.page.scss +++ b/libs/web/tracker/presenters/src/lib/pages/issue.page.scss @@ -339,6 +339,60 @@ a.ts-key { background: color-mix(in srgb, var(--ts-c-green) 6%, transparent); } +/* Written by someone else since my last visit. */ +.ts-comment.is-new:not(.is-decision) { + margin-inline: -0.75rem; + padding-inline: 0.75rem; + border-left: 3px solid var(--ts-brand); + background: color-mix(in srgb, var(--ts-brand) 5%, transparent); +} + +.ts-comment__new { + padding: 0.0625rem 0.5rem; + border-radius: 999px; + background: color-mix(in srgb, var(--ts-brand) 14%, transparent); + color: var(--ts-brand); + font-size: 0.6875rem; + font-weight: 700; + letter-spacing: 0.04em; + text-transform: uppercase; +} + +/* «→ Convertido en WEB-45». */ +.ts-comment__spawned { + display: flex; + flex-wrap: wrap; + gap: 0.25rem 0.5rem; + margin: 0.375rem 0 0; + color: var(--ts-muted); + font-size: 0.8125rem; + + a { + font-family: var(--ts-font-mono); + font-weight: 600; + } +} + +/* What others did since my last visit, above the ticket. */ +.ts-news { + display: flex; + flex-wrap: wrap; + align-items: center; + gap: 0.25rem 0.625rem; + margin: 0; + padding: 0.5rem 0.875rem; + border-radius: 0.625rem; + background: color-mix(in srgb, var(--ts-brand) 9%, transparent); + font-size: 0.875rem; +} + +.ts-news__dot { + width: 0.5rem; + height: 0.5rem; + border-radius: 50%; + background: var(--ts-brand); +} + .ts-comment.is-pointed { animation: ts-pointed 2.4s ease-out; } @@ -469,6 +523,23 @@ a.ts-key { &:last-child { border-bottom: 0; } + + /* Changed by someone else since my last visit. */ + &.is-new .ts-log__what { + color: var(--ts-ink); + font-weight: 600; + } + + &.is-new .ts-log__when::before { + content: ''; + display: inline-block; + width: 0.375rem; + height: 0.375rem; + margin-right: 0.375rem; + border-radius: 50%; + background: var(--ts-brand); + vertical-align: middle; + } } } diff --git a/libs/web/tracker/presenters/src/lib/pages/issue.page.ts b/libs/web/tracker/presenters/src/lib/pages/issue.page.ts index 5c8b1b7..303ed6c 100644 --- a/libs/web/tracker/presenters/src/lib/pages/issue.page.ts +++ b/libs/web/tracker/presenters/src/lib/pages/issue.page.ts @@ -25,6 +25,7 @@ import { PiIconComponent, PiLabelDirective, PiNoteComponent, + PiSegmentedControlComponent, PiProgressBarComponent, PiSearchableSelectFieldComponent, PiSelectFieldComponent, @@ -44,7 +45,12 @@ import { } from '@task-stream/web/tracker/application'; import { Attachment, + Activity, Comment, + isNewChange, + isNewComment, + newsSince, + titleFrom, CommentPoll, DevLink, Issue, @@ -113,6 +119,7 @@ const MAX_FILE_BYTES = 25 * 1024 * 1024; PiIconButtonComponent, PiLabelDirective, PiNoteComponent, + PiSegmentedControlComponent, PiProgressBarComponent, PiSearchableSelectFieldComponent, PiSelectFieldComponent, @@ -337,6 +344,27 @@ const MAX_FILE_BYTES = 25 * 1024 * 1024;

} + @if (news(); as news) { +

+ + Desde tu última visita, {{ ago(news.since) }}: + {{ news.what }} de {{ news.who }}. + @if (news.firstComment; as first) { + + + + } +

+ } + @if (project.canContribute() && issue.allowedTransitions.length) {
Mover a @@ -852,6 +880,7 @@ const MAX_FILE_BYTES = 25 * 1024 * 1024; class="ts-comment" [id]="'comment-' + comment.id" [class.is-decision]="comment.decision && !comment.deleted" + [class.is-new]="isNew(comment)" >