@managemint-solutions/sdk 0.42.0 → 0.44.0

package/dist/supporting-files/index.js

@@ -15,9 +15,11 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SupportingFilesResource = void 0;
-const enum_1 = require("@managemint-solutions/entities/audit-trail/enum");
+const enum_1 = require("@managemint-solutions/entities/supporting-files/enum");
+const enum_2 = require("@managemint-solutions/entities/audit-trail/enum");
 const audit_1 = require("../audit");
 const errors_1 = require("../errors");
+const won_1 = require("../leads/won");
 const paths_1 = require("./paths");
 __exportStar(require("./dto"), exports);
 __exportStar(require("./paths"), exports);
@@ -45,7 +47,7 @@ class SupportingFilesResource {
         this.auth = auth;
         this.table = (0, audit_1.auditedTable)(supabase, auth, audit, {
             table: 'supporting_files',
-            entityType: enum_1.AuditTrailEntityType.SUPPORTING_FILES,
+            entityType: enum_2.AuditTrailEntityType.SUPPORTING_FILES,
         });
     }
     async getPresignedUrl(supportingFileId) {
@@ -77,6 +79,9 @@ class SupportingFilesResource {
         return data;
     }
     async upload(entity, entityId, input) {
+        if (entity === enum_1.SupportingFilesEntityTypes.LEAD) {
+            await (0, won_1.refuseWonLead)(this.supabase, this.auth.organization_id, entityId);
+        }
         const { data: existing, error: existingError } = await this.supabase
             .from('supporting_files')
             .select('name, extension')

package/dist/tasks/dto.d.ts

@@ -1,7 +1,16 @@
-import type { CreateTaskDto as CreateTaskDtoType, GetTasksDto as GetTasksDtoType, TaskIdDto as TaskIdDtoType, UpdateTaskDto as UpdateTaskDtoType } from '@managemint-solutions/entities/tasks/dto';
+import type { AddTaskLinkDto as AddTaskLinkDtoType, CreateTaskDto as CreateTaskDtoType, GetTasksDto as GetTasksDtoType, TaskIdDto as TaskIdDtoType, TaskLinkIdDto as TaskLinkIdDtoType, UpdateTaskDto as UpdateTaskDtoType } from '@managemint-solutions/entities/tasks/dto';
+import { TaskLinkType } from '@managemint-solutions/entities/tasks/enum';
 export declare class TaskIdDto implements TaskIdDtoType {
     readonly taskId: string;
 }
+export declare class TaskLinkIdDto implements TaskLinkIdDtoType {
+    readonly taskId: string;
+    readonly linkedTaskId: string;
+}
+export declare class AddTaskLinkDto implements AddTaskLinkDtoType {
+    readonly linked_task_id: string;
+    readonly link_type: TaskLinkType;
+}
 export declare class GetTasksDto implements Omit<GetTasksDtoType, 'status_id_in' | 'status_id_not_in' | 'label_id_in' | 'priority_in'> {
     readonly search?: string | null;
     readonly page: number;

package/dist/tasks/dto.js

@@ -9,9 +9,10 @@ var __metadata = (this && this.__metadata) || function (k, v) {
     if (typeof Reflect === "object" && typeof Reflect.metadata === "function") return Reflect.metadata(k, v);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.UpdateTaskDto = exports.CreateTaskDto = exports.GetTasksDto = exports.TaskIdDto = void 0;
+exports.UpdateTaskDto = exports.CreateTaskDto = exports.GetTasksDto = exports.AddTaskLinkDto = exports.TaskLinkIdDto = exports.TaskIdDto = void 0;
 const class_validator_1 = require("class-validator");
 const class_transformer_1 = require("class-transformer");
+const enum_1 = require("@managemint-solutions/entities/tasks/enum");
 const transforms_1 = require("../transforms");
 const validators_1 = require("../validators");
 // class-validator versions of the shared DTO types from @managemint-solutions/entities, so the
@@ -25,6 +26,33 @@ __decorate([
     ,
     __metadata("design:type", String)
 ], TaskIdDto.prototype, "taskId", void 0);
+class TaskLinkIdDto {
+    taskId;
+    linkedTaskId;
+}
+exports.TaskLinkIdDto = TaskLinkIdDto;
+__decorate([
+    (0, class_validator_1.IsUUID)('7', { message: 'Invalid task ID format' }),
+    __metadata("design:type", String)
+], TaskLinkIdDto.prototype, "taskId", void 0);
+__decorate([
+    (0, class_validator_1.IsUUID)('7', { message: 'Invalid linked task ID format' }),
+    __metadata("design:type", String)
+], TaskLinkIdDto.prototype, "linkedTaskId", void 0);
+class AddTaskLinkDto {
+    linked_task_id;
+    // The only check on the value: `task_links.link_type` is plain text.
+    link_type;
+}
+exports.AddTaskLinkDto = AddTaskLinkDto;
+__decorate([
+    (0, class_validator_1.IsUUID)('7', { message: 'Invalid linked task ID format' }),
+    __metadata("design:type", String)
+], AddTaskLinkDto.prototype, "linked_task_id", void 0);
+__decorate([
+    (0, class_validator_1.IsEnum)(enum_1.TaskLinkType, { message: 'Link type must be child or related' }),
+    __metadata("design:type", String)
+], AddTaskLinkDto.prototype, "link_type", void 0);
 // `status_id_in`, `status_id_not_in`, `label_id_in` and `priority_in` are omitted from the
 // implemented type on purpose: the entities type carries the CSV wire values, while this class
 // carries the `@Transform`-parsed arrays the query builder feeds to `.in('status_id', ...)` /

package/dist/tasks/index.d.ts

@@ -3,7 +3,7 @@ import type { TaskEntity, TaskSummary } from '@managemint-solutions/entities/tas
 import type { AuthContext } from '../auth';
 import { type AuditResource } from '../audit';
 import type { TypedSupabaseClient } from '../client';
-import type { CreateTaskDto, GetTasksDto, UpdateTaskDto } from './dto';
+import type { AddTaskLinkDto, CreateTaskDto, GetTasksDto, UpdateTaskDto } from './dto';
 export * from './dto';
 export declare class TasksResource {
     private readonly supabase;
@@ -20,6 +20,20 @@ export declare class TasksResource {
     restore(taskId: string): Promise<TaskEntity>;
     /** Soft delete - the row stays, flagged `deleted`, and `restore()` brings it back. */
     remove(taskId: string): Promise<void>;
+    /**
+     * Links `input.linked_task_id` to the task as a subtask or as related. The rules live here,
+     * checked before the insert: not the task itself, both tasks this organization's and not
+     * deleted, and no loop of subtasks. The two a concurrent write could still break - one link
+     * per pair, one parent per task - are unique indexes, and their violations are turned into
+     * sentences. The link has no audit of its own: it is one UPDATE entry on the task it was
+     * made from.
+     */
+    linkTask(taskId: string, input: AddTaskLinkDto): Promise<TaskEntity>;
+    /**
+     * Removes the link between the two tasks, whichever of them it was made from - a pair carries
+     * one link at most. Unlinked from the subtask's side, the entry names the other task as parent.
+     */
+    unlinkTask(taskId: string, linkedTaskId: string): Promise<TaskEntity>;
     /**
      * Refuses, before anything is written, a label id that is not one of this organization's task
      * labels - one deleted while the form was open, or another organization's. The `task_labels`

package/dist/tasks/index.js

@@ -15,8 +15,9 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TasksResource = void 0;
-const enum_1 = require("@managemint-solutions/entities/labels/enum");
-const enum_2 = require("@managemint-solutions/entities/audit-trail/enum");
+const enum_1 = require("@managemint-solutions/entities/tasks/enum");
+const enum_2 = require("@managemint-solutions/entities/labels/enum");
+const enum_3 = require("@managemint-solutions/entities/audit-trail/enum");
 const audit_1 = require("../audit");
 const errors_1 = require("../errors");
 const transforms_1 = require("../transforms");
@@ -36,7 +37,8 @@ const TASK_LIST_SELECT = `
           label,
           is_complete,
           is_fail,
-          is_triage
+          is_triage,
+          is_won
         ),
         priority,
         labels:labels!task_labels (
@@ -49,6 +51,15 @@ const TASK_LIST_SELECT = `
         project:project_profiles!project_id (*),
         client:client_profiles!client_id (*)
       `;
+// A linked task as the detail read shows it. `deleted` is read only so a soft-deleted task can be
+// left out of the links; it is not part of LinkedTask.
+const LINKED_TASK_FIELDS = `
+            mms_id,
+            task_number:task_id,
+            name,
+            priority,
+            deleted,
+            status:statuses!status_id ( mms_id, colour, label, is_complete, is_fail, is_triage, is_won )`;
 const TASK_DETAIL_SELECT = `
         mms_id,
         created_at,
@@ -66,7 +77,8 @@ const TASK_DETAIL_SELECT = `
           label,
           is_complete,
           is_fail,
-          is_triage
+          is_triage,
+          is_won
         ),
         priority,
         labels:labels!task_labels (
@@ -87,7 +99,15 @@ const TASK_DETAIL_SELECT = `
         deleted_at,
         deleted_by:user_profiles!deleted_by (*),
         restored_at,
-        restored_by:user_profiles!restored_by (*)
+        restored_by:user_profiles!restored_by (*),
+        links_out:task_links!task_id (
+          link_type,
+          task:tasks!linked_task_id ( ${LINKED_TASK_FIELDS} )
+        ),
+        links_in:task_links!linked_task_id (
+          link_type,
+          task:tasks!task_id ( ${LINKED_TASK_FIELDS} )
+        )
       `;
 // Added to the list select only when `label_id_in` is set: an inner join on the join table is how
 // PostgREST filters a task by the labels it carries.
@@ -95,6 +115,12 @@ const LABEL_FILTER_SELECT = 'task_labels!inner ( label_id )';
 // A `task_labels` row as `setLabels` reads and writes it: the label id, and the name the audit
 // entry records.
 const PINNED_LABEL_SELECT = 'label_id, label:labels ( name )';
+const linkedTasks = (rows, linkType) => rows
+    .filter((row) => row.link_type === linkType && row.task && !row.task.deleted)
+    .map((row) => {
+    const { deleted: _deleted, ...task } = row.task;
+    return task;
+});
 class TasksResource {
     supabase;
     auth;
@@ -106,7 +132,7 @@ class TasksResource {
         this.audit = audit;
         this.table = (0, audit_1.auditedTable)(supabase, auth, audit, {
             table: 'tasks',
-            entityType: enum_2.AuditTrailEntityType.TASKS,
+            entityType: enum_3.AuditTrailEntityType.TASKS,
         });
     }
     async list(query) {
@@ -196,10 +222,20 @@ class TasksResource {
             .select(TASK_DETAIL_SELECT)
             .eq('mms_id', taskId)
             .eq('organization_id', this.auth.organization_id)
+            .order('created_at', { referencedTable: 'links_out' })
+            .order('created_at', { referencedTable: 'links_in' })
             .single();
         if (error)
             throw (0, errors_1.mapPostgrestError)(error);
-        return data;
+        // Related is stored once, so it is read from both directions; a child link is the parent
+        // from the child's side and a subtask from the parent's.
+        const { links_out: linksOut, links_in: linksIn, ...task } = data;
+        return {
+            ...task,
+            parent: linkedTasks(linksIn, enum_1.TaskLinkType.CHILD)[0] ?? null,
+            subtasks: linkedTasks(linksOut, enum_1.TaskLinkType.CHILD),
+            related: [...linkedTasks(linksOut, enum_1.TaskLinkType.RELATED), ...linkedTasks(linksIn, enum_1.TaskLinkType.RELATED)],
+        };
     }
     async create(input) {
         // `label_ids` is not a tasks column: the labels are join rows, written after the task.
@@ -237,20 +273,126 @@ class TasksResource {
         return this.getById(taskId);
     }
     async archive(taskId) {
-        await this.table.transition(taskId, enum_2.AuditTrailAction.ARCHIVE, (0, transforms_1.archivedPatch)(this.auth.mms_id));
+        await this.table.transition(taskId, enum_3.AuditTrailAction.ARCHIVE, (0, transforms_1.archivedPatch)(this.auth.mms_id));
         return this.getById(taskId);
     }
     async unarchive(taskId) {
-        await this.table.transition(taskId, enum_2.AuditTrailAction.UNARCHIVE, (0, transforms_1.unarchivedPatch)(this.auth.mms_id));
+        await this.table.transition(taskId, enum_3.AuditTrailAction.UNARCHIVE, (0, transforms_1.unarchivedPatch)(this.auth.mms_id));
         return this.getById(taskId);
     }
     async restore(taskId) {
-        await this.table.transition(taskId, enum_2.AuditTrailAction.RESTORE, (0, transforms_1.restoredPatch)(this.auth.mms_id));
+        await this.table.transition(taskId, enum_3.AuditTrailAction.RESTORE, (0, transforms_1.restoredPatch)(this.auth.mms_id));
         return this.getById(taskId);
     }
     /** Soft delete - the row stays, flagged `deleted`, and `restore()` brings it back. */
     async remove(taskId) {
-        await this.table.transition(taskId, enum_2.AuditTrailAction.DELETE, (0, transforms_1.deletedPatch)(this.auth.mms_id));
+        await this.table.transition(taskId, enum_3.AuditTrailAction.DELETE, (0, transforms_1.deletedPatch)(this.auth.mms_id));
+    }
+    /**
+     * Links `input.linked_task_id` to the task as a subtask or as related. The rules live here,
+     * checked before the insert: not the task itself, both tasks this organization's and not
+     * deleted, and no loop of subtasks. The two a concurrent write could still break - one link
+     * per pair, one parent per task - are unique indexes, and their violations are turned into
+     * sentences. The link has no audit of its own: it is one UPDATE entry on the task it was
+     * made from.
+     */
+    async linkTask(taskId, input) {
+        const refuse = (status, message, cause) => new errors_1.SupabaseClientError({ status, error: 'Database Error', message }, { cause });
+        if (input.linked_task_id === taskId) {
+            throw refuse(400, 'A task cannot be linked to itself');
+        }
+        const { data: tasks, error: tasksError } = await this.supabase
+            .from('tasks')
+            .select('mms_id, task_number:task_id')
+            .eq('organization_id', this.auth.organization_id)
+            .eq('deleted', false)
+            .in('mms_id', [taskId, input.linked_task_id]);
+        if (tasksError)
+            throw (0, errors_1.mapPostgrestError)(tasksError);
+        const linked = tasks.find((task) => task.mms_id === input.linked_task_id);
+        if (tasks.length !== 2 || !linked) {
+            throw refuse(400, 'Invalid task for this organization');
+        }
+        if (input.link_type === enum_1.TaskLinkType.CHILD) {
+            // Walk up from the new parent: were the new subtask among its ancestors, the link would
+            // close a loop. The chain is a few links deep. `seen` stops the walk should two concurrent
+            // links ever have made a loop after all, rather than reading forever.
+            const seen = new Set();
+            let ancestor = taskId;
+            while (ancestor && !seen.has(ancestor)) {
+                const child = ancestor;
+                seen.add(child);
+                const { data: parentLink, error: parentError } = await this.supabase
+                    .from('task_links')
+                    .select('task_id')
+                    .eq('organization_id', this.auth.organization_id)
+                    .eq('linked_task_id', child)
+                    .eq('link_type', enum_1.TaskLinkType.CHILD)
+                    .maybeSingle();
+                if (parentError)
+                    throw (0, errors_1.mapPostgrestError)(parentError);
+                ancestor = parentLink?.task_id ?? null;
+                if (ancestor === input.linked_task_id) {
+                    throw refuse(400, 'A task cannot be a subtask of its own subtask');
+                }
+            }
+        }
+        const { error } = await this.supabase.from('task_links').insert({
+            task_id: taskId,
+            linked_task_id: input.linked_task_id,
+            link_type: input.link_type,
+            organization_id: this.auth.organization_id,
+            created_by: this.auth.mms_id,
+        });
+        if (error) {
+            // Postgres names the violated index: the primary key and the pair index both mean the two
+            // tasks are linked already.
+            if (error.code === '23505' && error.message.includes('uq_task_links_one_parent')) {
+                throw refuse(409, 'That task already has a parent', error);
+            }
+            if (error.code === '23505') {
+                throw refuse(409, 'These tasks are already linked', error);
+            }
+            throw (0, errors_1.mapPostgrestError)(error);
+        }
+        const role = input.link_type === enum_1.TaskLinkType.CHILD ? 'subtask' : 'related';
+        await this.audit.record({
+            action: enum_3.AuditTrailAction.UPDATE,
+            entityType: enum_3.AuditTrailEntityType.TASKS,
+            entityId: taskId,
+            changes: { linked_task: { old: null, new: `TASK-${linked.task_number} (${role})` } },
+        });
+        return this.getById(taskId);
+    }
+    /**
+     * Removes the link between the two tasks, whichever of them it was made from - a pair carries
+     * one link at most. Unlinked from the subtask's side, the entry names the other task as parent.
+     */
+    async unlinkTask(taskId, linkedTaskId) {
+        // Both ids are `@IsUUID('7')`-validated by TaskLinkIdDto, so interpolating them is safe.
+        const { data, error } = await this.supabase
+            .from('task_links')
+            .delete()
+            .eq('organization_id', this.auth.organization_id)
+            .or(`and(task_id.eq.${taskId},linked_task_id.eq.${linkedTaskId}),and(task_id.eq.${linkedTaskId},linked_task_id.eq.${taskId})`)
+            .select('task_id, link_type, from:tasks!task_id ( task_number:task_id ), to:tasks!linked_task_id ( task_number:task_id )');
+        if (error)
+            throw (0, errors_1.mapPostgrestError)(error);
+        const [row] = data;
+        if (!row) {
+            throw new errors_1.SupabaseClientError({ status: 404, error: 'Database Error', message: 'Link not found' });
+        }
+        const madeHere = row.task_id === taskId;
+        const role = row.link_type === enum_1.TaskLinkType.RELATED ? 'related' : madeHere ? 'subtask' : 'parent';
+        await this.audit.record({
+            action: enum_3.AuditTrailAction.UPDATE,
+            entityType: enum_3.AuditTrailEntityType.TASKS,
+            entityId: taskId,
+            changes: {
+                linked_task: { old: `TASK-${(madeHere ? row.to : row.from).task_number} (${role})`, new: null },
+            },
+        });
+        return this.getById(taskId);
     }
     /**
      * Refuses, before anything is written, a label id that is not one of this organization's task
@@ -265,7 +407,7 @@ class TasksResource {
             .from('labels')
             .select('mms_id', { count: 'exact', head: true })
             .eq('organization_id', this.auth.organization_id)
-            .eq('entity_type', enum_1.LabelEntityTypes.TASK)
+            .eq('entity_type', enum_2.LabelEntityTypes.TASK)
             .in('mms_id', labelIds);
         if (error)
             throw (0, errors_1.mapPostgrestError)(error);
@@ -326,8 +468,8 @@ class TasksResource {
                 throw (0, errors_1.mapPostgrestError)(error);
         }
         await this.audit.record({
-            action: enum_2.AuditTrailAction.UPDATE,
-            entityType: enum_2.AuditTrailEntityType.TASKS,
+            action: enum_3.AuditTrailAction.UPDATE,
+            entityType: enum_3.AuditTrailEntityType.TASKS,
             entityId: taskId,
             changes: {
                 labels: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@managemint-solutions/sdk",
-  "version": "0.42.0",
+  "version": "0.44.0",
   "description": "Typed Supabase data-access SDK for ManageMint Solutions",
   "license": "UNLICENSED",
   "author": "Scott Bebington <scottbebington@gmail.com>",
@@ -51,7 +51,7 @@
     "testEnvironment": "node"
   },
   "dependencies": {
-    "@managemint-solutions/entities": "^1.25.0"
+    "@managemint-solutions/entities": "^1.27.0"
   },
   "peerDependencies": {
     "@nestjs/common": "^11.0.0",