@managemint-solutions/sdk 0.24.2 → 0.26.0

package/README.md

@@ -38,12 +38,19 @@ const supabase = createSupabaseClient({
 
 // Inputs and outputs are the shared types from @managemint-solutions/entities.
 const statuses: StatusEntity[] = await supabase.statuses.list(StatusEntityTypes.PROJECT);
+// `is_complete` marks the finished state and is optional — it defaults to false on create and
+// is left unchanged on update when omitted. `is_fail` and `is_triage` are read back on every
+// status but are not settable through the SDK yet.
 const created = await supabase.statuses.create({
   label: 'In progress',
   colour: '#2ecc71',
   entity: StatusEntityTypes.PROJECT,
 });
-await supabase.statuses.update(created.mms_id, { label: 'Doing', colour: '#2ecc71' });
+await supabase.statuses.update(created.mms_id, {
+  label: 'Done',
+  colour: '#2ecc71',
+  is_complete: true,
+});
 await supabase.statuses.remove(created.mms_id);
 ```
 
@@ -137,7 +144,9 @@ const page = await supabase.projects.list({
   client_id: clientId, // a UUID, or the base64-JSON option value a select sends
 });
 // `search` matches name/specification; `created_*`, `start_date_*` and `end_date_*` bound the
-// date columns, and `status_id_in` filters by status.
+// date columns, and `status_id_in` filters by status. `status_id_not_in` is the exclusion form —
+// it keeps rows with no status, which `status_id_in` cannot express. `completed` is carried but
+// not acted on here: the api resolves it to status ids against the statuses catalogue first.
 
 const project = await supabase.projects.create({
   name: 'Website rebuild',
@@ -171,7 +180,9 @@ const page = await supabase.tasks.list({
   priority_in: [1, 3],
 });
 // `search` matches name/description, and `client_id`, `assigned_to_id`, the date ranges and
-// `status_id_in` narrow it further.
+// `status_id_in` narrow it further. `status_id_not_in` excludes statuses while keeping rows with
+// no status; `completed` is carried but not acted on here — the api resolves it to status ids
+// against the statuses catalogue first.
 
 const task = await supabase.tasks.create({
   name: 'Draft the proposal',

package/dist/clients/index.js

@@ -31,7 +31,10 @@ const CLIENT_LIST_SELECT = `
         status:statuses!status_id (
           mms_id,
           colour,
-          label
+          label,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         created_by:user_profiles!created_by (*)
       `;
@@ -44,7 +47,10 @@ const CLIENT_DETAIL_SELECT = `
         status:statuses!status_id (
           mms_id,
           label,
-          colour
+          colour,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         contract_client,
         contracted_hours,

package/dist/projects/dto.d.ts

@@ -1,5 +1,5 @@
 import type { CreateProjectDto as CreateProjectDtoType, GetProjectsDto as GetProjectsDtoType, ProjectIdDto as ProjectIdDtoType, UpdateProjectDto as UpdateProjectDtoType } from '@managemint-solutions/entities/projects/dto';
-export declare class GetProjectsDto implements Omit<GetProjectsDtoType, 'status_id_in'> {
+export declare class GetProjectsDto implements Omit<GetProjectsDtoType, 'status_id_in' | 'status_id_not_in'> {
     readonly search?: string | null;
     readonly page: number;
     readonly page_size: number;
@@ -10,6 +10,8 @@ export declare class GetProjectsDto implements Omit<GetProjectsDtoType, 'status_
     readonly end_date_after?: string | null;
     readonly end_date_before?: string | null;
     readonly status_id_in?: string[] | null;
+    readonly status_id_not_in?: string[] | null;
+    readonly completed?: boolean | null;
     readonly client_id?: string | null;
     readonly archive: boolean;
 }

package/dist/projects/dto.js

@@ -16,9 +16,10 @@ const transforms_1 = require("../transforms");
 const validators_1 = require("../validators");
 // class-validator versions of the shared DTO types from @managemint-solutions/entities, so the
 // wire contract cannot drift.
-// `status_id_in` is omitted from the implemented type on purpose: the entities type carries the
-// CSV wire value (`string | null`), while this class carries the `@Transform`-parsed
-// `string[] | null` the query builder feeds to `.in('status_id', ...)`.
+// `status_id_in` and `status_id_not_in` are omitted from the implemented type on purpose: the
+// entities type carries the CSV wire values (`string | null`), while this class carries the
+// `@Transform`-parsed `string[] | null` the query builder feeds to `.in('status_id', ...)` and to
+// the `status_id.is.null,status_id.not.in.(...)` exclusion.
 class GetProjectsDto {
     search;
     page = 1;
@@ -30,6 +31,10 @@ class GetProjectsDto {
     end_date_after;
     end_date_before;
     status_id_in;
+    status_id_not_in;
+    // Tri-state: true = only completed rows, false = only rows that are not completed, omitted = no
+    // completion filter. The api resolves it to status ids before it reaches `list`.
+    completed;
     client_id;
     archive = false;
 }
@@ -99,6 +104,20 @@ __decorate([
     (0, class_validator_1.IsUUID)('7', { each: true, message: 'Each status ID must be a valid UUID v7' }),
     __metadata("design:type", Object)
 ], GetProjectsDto.prototype, "status_id_in", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, validators_1.IsNullable)(),
+    (0, class_transformer_1.Transform)(({ value }) => value ? value.split(',').map((id) => id.trim()) : null, { toClassOnly: true }),
+    (0, class_validator_1.IsUUID)('7', { each: true, message: 'Each status ID must be a valid UUID v7' }),
+    __metadata("design:type", Object)
+], GetProjectsDto.prototype, "status_id_not_in", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, validators_1.IsNullable)(),
+    (0, class_transformer_1.Transform)(({ value }) => value === 'true' ? true : value === 'false' ? false : value, { toClassOnly: true }),
+    (0, class_validator_1.IsBoolean)({ message: 'Completed must be true or false' }),
+    __metadata("design:type", Object)
+], GetProjectsDto.prototype, "completed", void 0);
 __decorate([
     (0, class_validator_1.IsOptional)(),
     (0, validators_1.IsNullable)(),

package/dist/projects/index.js

@@ -32,7 +32,10 @@ const PROJECT_LIST_SELECT = `
         status:statuses!status_id (
           mms_id,
           colour,
-          label
+          label,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         created_by:user_profiles!created_by (*),
         manager:user_profiles!manager_id (*),
@@ -46,7 +49,10 @@ const PROJECT_DETAIL_SELECT = `
         status:statuses!status_id (
           mms_id,
           label,
-          colour
+          colour,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         has_budget,
         start_date,
@@ -118,6 +124,15 @@ class ProjectsResource {
         if (query.status_id_in && query.status_id_in.length > 0) {
             projectsQuery = projectsQuery.in('status_id', query.status_id_in);
         }
+        // Excluding statuses has to be spelled out as "no status OR not one of these": PostgREST can
+        // only filter a parent row by an embedded column with `!inner`, which would drop projects that
+        // have no status at all — and a project with no status is not complete, so it belongs in the
+        // default view. SQL's `NOT IN` drops nulls for the same reason, hence the explicit null arm.
+        // The ids are `@IsUUID('7')`-validated by the DTO, so interpolating them is safe. Repeated
+        // `or` params are ANDed by PostgREST, so this composes with the search filter above.
+        if (query.status_id_not_in && query.status_id_not_in.length > 0) {
+            projectsQuery = projectsQuery.or(`status_id.is.null,status_id.not.in.(${query.status_id_not_in.join(',')})`);
+        }
         if (query.client_id) {
             projectsQuery = projectsQuery.eq('client_id', query.client_id);
         }

package/dist/statuses/dto.d.ts

@@ -10,8 +10,10 @@ export declare class AddStatusDto implements AddStatusDtoType {
     readonly label: string;
     readonly colour: string;
     readonly entity: StatusEntityTypes;
+    readonly is_complete?: boolean;
 }
 export declare class UpdateStatusDto implements UpdateStatusDtoType {
     readonly label: string;
     readonly colour: string;
+    readonly is_complete?: boolean;
 }

package/dist/statuses/dto.js

@@ -36,6 +36,8 @@ class AddStatusDto {
     colour;
     // `entity` is the wire name; the SDK maps it to the `entity_type` column.
     entity;
+    // Marks the status as the finished state; omitted means false.
+    is_complete;
 }
 exports.AddStatusDto = AddStatusDto;
 __decorate([
@@ -55,10 +57,17 @@ __decorate([
     (0, class_validator_1.IsEnum)(enum_1.StatusEntityTypes, { message: 'Invalid entity type' }),
     __metadata("design:type", String)
 ], AddStatusDto.prototype, "entity", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, class_validator_1.IsBoolean)({ message: 'is_complete must be true or false' }),
+    __metadata("design:type", Boolean)
+], AddStatusDto.prototype, "is_complete", void 0);
 class UpdateStatusDto {
     label;
     // Colour must be a valid hex code (e.g., #FFFFFF)
     colour;
+    // Marks the status as the finished state; omitted leaves the flag unchanged.
+    is_complete;
 }
 exports.UpdateStatusDto = UpdateStatusDto;
 __decorate([
@@ -74,3 +83,8 @@ __decorate([
     (0, class_validator_1.Matches)(/^#[0-9A-Fa-f]{6}$/, { message: 'Colour must be a valid hex code' }),
     __metadata("design:type", String)
 ], UpdateStatusDto.prototype, "colour", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, class_validator_1.IsBoolean)({ message: 'is_complete must be true or false' }),
+    __metadata("design:type", Boolean)
+], UpdateStatusDto.prototype, "is_complete", void 0);

package/dist/statuses/index.js

@@ -22,9 +22,16 @@ __exportStar(require("./dto"), exports);
 // The public contract is the shared StatusEntity from @managemint-solutions/entities; the
 // `as StatusEntity` casts declare the shape the select string below produces (the columns are
 // defined in the api's supabase/migrations).
-const STATUS_SELECT = 'mms_id, label, colour';
+const STATUS_SELECT = 'mms_id, label, colour, is_complete, is_fail, is_triage';
 // The audited write returns the whole row; the wire shape is the STATUS_SELECT columns.
-const toStatus = (row) => ({ mms_id: row.mms_id, label: row.label, colour: row.colour });
+const toStatus = (row) => ({
+    mms_id: row.mms_id,
+    label: row.label,
+    colour: row.colour,
+    is_complete: row.is_complete,
+    is_fail: row.is_fail,
+    is_triage: row.is_triage,
+});
 class StatusesResource {
     supabase;
     auth;
@@ -52,6 +59,7 @@ class StatusesResource {
             label: input.label,
             colour: input.colour,
             entity_type: input.entity,
+            is_complete: input.is_complete ?? false,
             organization_id: this.auth.organization_id,
             created_by: this.auth.mms_id,
         });
@@ -61,6 +69,7 @@ class StatusesResource {
         const row = await this.table.update(statusId, {
             label: input.label,
             colour: input.colour,
+            ...(input.is_complete !== undefined && { is_complete: input.is_complete }),
             updated_at: new Date().toISOString(),
             last_updated_by: this.auth.mms_id,
         });

package/dist/tasks/dto.d.ts

@@ -2,7 +2,7 @@ import type { CreateTaskDto as CreateTaskDtoType, GetTasksDto as GetTasksDtoType
 export declare class TaskIdDto implements TaskIdDtoType {
     readonly taskId: string;
 }
-export declare class GetTasksDto implements Omit<GetTasksDtoType, 'status_id_in' | 'priority_in'> {
+export declare class GetTasksDto implements Omit<GetTasksDtoType, 'status_id_in' | 'status_id_not_in' | 'priority_in'> {
     readonly search?: string | null;
     readonly page: number;
     readonly page_size: number;
@@ -16,6 +16,8 @@ export declare class GetTasksDto implements Omit<GetTasksDtoType, 'status_id_in'
     readonly end_date_after?: string | null;
     readonly end_date_before?: string | null;
     readonly status_id_in?: string[] | null;
+    readonly status_id_not_in?: string[] | null;
+    readonly completed?: boolean | null;
     readonly priority_in?: number[] | null;
     readonly archive: boolean;
 }

package/dist/tasks/dto.js

@@ -25,9 +25,10 @@ __decorate([
     ,
     __metadata("design:type", String)
 ], TaskIdDto.prototype, "taskId", void 0);
-// `status_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', ...)` / `.in('priority', ...)`.
+// `status_id_in`, `status_id_not_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', ...)` /
+// `.in('priority', ...)` and to the `status_id.is.null,status_id.not.in.(...)` exclusion.
 class GetTasksDto {
     search;
     page = 1;
@@ -42,6 +43,10 @@ class GetTasksDto {
     end_date_after;
     end_date_before;
     status_id_in;
+    status_id_not_in;
+    // Tri-state: true = only completed rows, false = only rows that are not completed, omitted = no
+    // completion filter. The api resolves it to status ids before it reaches `list`.
+    completed;
     priority_in;
     archive = false;
 }
@@ -135,6 +140,20 @@ __decorate([
     (0, class_validator_1.IsUUID)('7', { each: true, message: 'Invalid status ID format' }),
     __metadata("design:type", Object)
 ], GetTasksDto.prototype, "status_id_in", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, validators_1.IsNullable)(),
+    (0, class_transformer_1.Transform)(({ value }) => value ? value.split(',').map((id) => id.trim()) : null, { toClassOnly: true }),
+    (0, class_validator_1.IsUUID)('7', { each: true, message: 'Invalid status ID format' }),
+    __metadata("design:type", Object)
+], GetTasksDto.prototype, "status_id_not_in", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, validators_1.IsNullable)(),
+    (0, class_transformer_1.Transform)(({ value }) => value === 'true' ? true : value === 'false' ? false : value, { toClassOnly: true }),
+    (0, class_validator_1.IsBoolean)({ message: 'Completed must be true or false' }),
+    __metadata("design:type", Object)
+], GetTasksDto.prototype, "completed", void 0);
 __decorate([
     (0, class_validator_1.IsOptional)(),
     (0, validators_1.IsNullable)(),

package/dist/tasks/index.js

@@ -31,7 +31,10 @@ const TASK_LIST_SELECT = `
         status:statuses!status_id (
           mms_id,
           colour,
-          label
+          label,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         priority,
         assigned_to:user_profiles!assigned_to_id (*)
@@ -49,7 +52,10 @@ const TASK_DETAIL_SELECT = `
         status:statuses!status_id (
           mms_id,
           colour,
-          label
+          label,
+          is_complete,
+          is_fail,
+          is_triage
         ),
         priority,
         assigned_to:user_profiles!assigned_to_id (*),
@@ -127,6 +133,15 @@ class TasksResource {
         if (query.status_id_in && query.status_id_in.length > 0) {
             tasksQuery = tasksQuery.in('status_id', query.status_id_in);
         }
+        // Excluding statuses has to be spelled out as "no status OR not one of these": PostgREST can
+        // only filter a parent row by an embedded column with `!inner`, which would drop tasks that
+        // have no status at all — and a task with no status is not complete, so it belongs in the
+        // default view. SQL's `NOT IN` drops nulls for the same reason, hence the explicit null arm.
+        // The ids are `@IsUUID('7')`-validated by the DTO, so interpolating them is safe. Repeated
+        // `or` params are ANDed by PostgREST, so this composes with the search filter above.
+        if (query.status_id_not_in && query.status_id_not_in.length > 0) {
+            tasksQuery = tasksQuery.or(`status_id.is.null,status_id.not.in.(${query.status_id_not_in.join(',')})`);
+        }
         if (query.priority_in && query.priority_in.length > 0) {
             tasksQuery = tasksQuery.in('priority', query.priority_in);
         }

package/package.json

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