@mj-biz-apps/sales-core-entities-server 5.1.0

package/dist/LiveOrdersSeam.d.ts

@@ -0,0 +1,68 @@
+/**
+ * @fileoverview What is left of the orders handoff: a wrapper that forwards the CONTRACT route.
+ *
+ * ── THIS CLASS NO LONGER TOUCHES ORDERS, AND THAT IS THE POINT ─────────────────────────────────
+ *
+ * It used to create the order a won deal earned, through orders' own entity graph. Close-won does not
+ * create an order any more — the order is embedded on the deal from creation — so `CreateOrder` and
+ * `PreviewOrderMoney` described a route that no longer exists. Both are gone, along with the money
+ * boundary essay that justified them.
+ *
+ * ── ONE OF THEM WAS A LIVE HAZARD, NOT JUST DEAD WEIGHT ────────────────────────────────────────
+ *
+ * Both passed `DiscountPct` straight through to `OrderLine`, and the seam input documented no units.
+ * Orders stores a FRACTION (0.25 for a quarter off); the caller that used to feed this sent a rep-
+ * entered PERCENTAGE (25). With the caller deleted the conversion went with it, leaving two exported,
+ * typechecked methods that would write a 25% discount as 2500% — except `CK_OrderLine_DiscountPct`
+ * caps it, so the realistic outcome was a 50% discount recorded for a half-percent one, saved
+ * cleanly, with nothing to notice.
+ *
+ * That is precisely what `scripts/assert-discount-conversion.mjs` exists to prevent, and it was
+ * sitting on a published interface with no test on it since the CW and D check bundles were retired.
+ * Deleting the methods removes the hazard outright rather than documenting it.
+ *
+ * ── WHY THE CLASS SURVIVES AT ALL ──────────────────────────────────────────────────────────────
+ *
+ * `CreateContractFromDeal` and `RenewContractTerm` below are pure delegation to the contracts seam it
+ * is handed. So this is now a pass-through, and `CloseDealOperation.resolveSeam` could return that
+ * contracts seam directly instead of wrapping it — at which point this whole file goes. That edit is
+ * in `CloseDealOperation.ts`, which another session is merging, so it is left stated rather than made.
+ *
+ * `OrdersIsInstalled()` is the one thing here still doing work: the close reads it to decide whether
+ * an orders-dependent route is available at all.
+ *
+ * @module @mj-biz-apps/sales-core-entities-server
+ */
+import { RunView, type IMetadataProvider, type UserInfo } from '@memberjunction/core';
+import { type IDownstreamSeam, type ContractsCreateFromDealSeamInput, type ContractsRenewTermSeamInput, type ContractsSeamResult } from '@mj-biz-apps/sales-entities';
+/** Orders' entity names, in one place so a rename is one edit rather than a hunt. */
+declare const E_ORDER_HEADER = "MJ_BizApps_Orders: Order Headers";
+declare const E_ORDER_LINE = "MJ_BizApps_Orders: Order Lines";
+/**
+ * True when orders is actually installed here.
+ *
+ * Checked against METADATA rather than by catching an error, for the same reason the product picker
+ * does: `RunView`/`GetEntityObject` against an unregistered entity does not fail cleanly — it logs and
+ * leaves the caller guessing. Sales is designed to run standalone, so orders being absent is a
+ * supported state, not a fault.
+ */
+export declare function OrdersIsInstalled(): boolean;
+export declare class LiveOrdersSeam implements IDownstreamSeam {
+    private readonly user;
+    private readonly provider;
+    /**
+     * The contract half is delegated to whatever the composer supplied — the live contracts seam when
+     * contracts is installed, the stub when it is not. This class owns ORDERS and nothing else; it used
+     * to hold a private stub, which quietly made it the authority on a downstream it knows nothing
+     * about. See `CompositeDownstreamSeam`.
+     */
+    private readonly contracts;
+    constructor(user: UserInfo, provider: IMetadataProvider, contracts?: Pick<IDownstreamSeam, 'CreateContractFromDeal' | 'RenewContractTerm'>);
+    CreateContractFromDeal(input: ContractsCreateFromDealSeamInput): Promise<ContractsSeamResult>;
+    RenewContractTerm(input: ContractsRenewTermSeamInput): Promise<ContractsSeamResult>;
+}
+/** Anti-tree-shaking anchor, in the house style. */
+export declare function LoadLiveOrdersSeam(): void;
+/** Kept for the seam test, which reads back what orders actually wrote. */
+export { E_ORDER_HEADER, E_ORDER_LINE, RunView };
+//# sourceMappingURL=LiveOrdersSeam.d.ts.map

package/dist/LiveOrdersSeam.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"LiveOrdersSeam.d.ts","sourceRoot":"","sources":["../src/LiveOrdersSeam.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,OAAO,EAGH,OAAO,EAEP,KAAK,iBAAiB,EACtB,KAAK,QAAQ,EAChB,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EACH,KAAK,eAAe,EAEpB,KAAK,gCAAgC,EACrC,KAAK,2BAA2B,EAChC,KAAK,mBAAmB,EAE3B,MAAM,6BAA6B,CAAC;AAErC,qFAAqF;AACrF,QAAA,MAAM,cAAc,qCAAqC,CAAC;AAC1D,QAAA,MAAM,YAAY,mCAAmC,CAAC;AA0BtD;;;;;;;GAOG;AACH,wBAAgB,iBAAiB,IAAI,OAAO,CAG3C;AAED,qBAAa,cAAe,YAAW,eAAe;IAW9C,OAAO,CAAC,QAAQ,CAAC,IAAI;IACrB,OAAO,CAAC,QAAQ,CAAC,QAAQ;IAV7B;;;;;OAKG;IACH,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAwE;gBAG7E,IAAI,EAAE,QAAQ,EACd,QAAQ,EAAE,iBAAiB,EAC5C,SAAS,CAAC,EAAE,IAAI,CAAC,eAAe,EAAE,wBAAwB,GAAG,mBAAmB,CAAC;IAO9E,sBAAsB,CAAC,KAAK,EAAE,gCAAgC,GAAG,OAAO,CAAC,mBAAmB,CAAC;IAI7F,iBAAiB,CAAC,KAAK,EAAE,2BAA2B,GAAG,OAAO,CAAC,mBAAmB,CAAC;CAG7F;AAED,oDAAoD;AACpD,wBAAgB,kBAAkB,IAAI,IAAI,CAEzC;AAED,2EAA2E;AAC3E,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,OAAO,EAAE,CAAC"}

package/dist/LiveOrdersSeam.js

@@ -0,0 +1,80 @@
+/**
+ * @fileoverview What is left of the orders handoff: a wrapper that forwards the CONTRACT route.
+ *
+ * ── THIS CLASS NO LONGER TOUCHES ORDERS, AND THAT IS THE POINT ─────────────────────────────────
+ *
+ * It used to create the order a won deal earned, through orders' own entity graph. Close-won does not
+ * create an order any more — the order is embedded on the deal from creation — so `CreateOrder` and
+ * `PreviewOrderMoney` described a route that no longer exists. Both are gone, along with the money
+ * boundary essay that justified them.
+ *
+ * ── ONE OF THEM WAS A LIVE HAZARD, NOT JUST DEAD WEIGHT ────────────────────────────────────────
+ *
+ * Both passed `DiscountPct` straight through to `OrderLine`, and the seam input documented no units.
+ * Orders stores a FRACTION (0.25 for a quarter off); the caller that used to feed this sent a rep-
+ * entered PERCENTAGE (25). With the caller deleted the conversion went with it, leaving two exported,
+ * typechecked methods that would write a 25% discount as 2500% — except `CK_OrderLine_DiscountPct`
+ * caps it, so the realistic outcome was a 50% discount recorded for a half-percent one, saved
+ * cleanly, with nothing to notice.
+ *
+ * That is precisely what `scripts/assert-discount-conversion.mjs` exists to prevent, and it was
+ * sitting on a published interface with no test on it since the CW and D check bundles were retired.
+ * Deleting the methods removes the hazard outright rather than documenting it.
+ *
+ * ── WHY THE CLASS SURVIVES AT ALL ──────────────────────────────────────────────────────────────
+ *
+ * `CreateContractFromDeal` and `RenewContractTerm` below are pure delegation to the contracts seam it
+ * is handed. So this is now a pass-through, and `CloseDealOperation.resolveSeam` could return that
+ * contracts seam directly instead of wrapping it — at which point this whole file goes. That edit is
+ * in `CloseDealOperation.ts`, which another session is merging, so it is left stated rather than made.
+ *
+ * `OrdersIsInstalled()` is the one thing here still doing work: the close reads it to decide whether
+ * an orders-dependent route is available at all.
+ *
+ * @module @mj-biz-apps/sales-core-entities-server
+ */
+import { Metadata, RunView, } from '@memberjunction/core';
+import { StubDownstreamSeam, } from '@mj-biz-apps/sales-entities';
+/** Orders' entity names, in one place so a rename is one edit rather than a hunt. */
+const E_ORDER_HEADER = 'MJ_BizApps_Orders: Order Headers';
+const E_ORDER_LINE = 'MJ_BizApps_Orders: Order Lines';
+/**
+ * The operation that prices a draft — orders' CLASS REGISTRATION key, not its display name.
+ *
+ * Metadata calls it "Price Order"; the ClassFactory key is `Orders.PriceOrder`. Invoking by key keeps
+ * this a SOFT dependency: sales names a string, never imports orders' TypeScript, so a deployment
+ * without orders still compiles and runs.
+ */
+const OP_PRICE_ORDER = 'Orders.PriceOrder';
+/**
+ * True when orders is actually installed here.
+ *
+ * Checked against METADATA rather than by catching an error, for the same reason the product picker
+ * does: `RunView`/`GetEntityObject` against an unregistered entity does not fail cleanly — it logs and
+ * leaves the caller guessing. Sales is designed to run standalone, so orders being absent is a
+ * supported state, not a fault.
+ */
+export function OrdersIsInstalled() {
+    const md = new Metadata();
+    return md.Entities.some((e) => e.Name === E_ORDER_HEADER) && md.Entities.some((e) => e.Name === E_ORDER_LINE);
+}
+export class LiveOrdersSeam {
+    constructor(user, provider, contracts) {
+        this.user = user;
+        this.provider = provider;
+        this.contracts = contracts ?? new StubDownstreamSeam();
+    }
+    CreateContractFromDeal(input) {
+        return this.contracts.CreateContractFromDeal(input);
+    }
+    RenewContractTerm(input) {
+        return this.contracts.RenewContractTerm(input);
+    }
+}
+/** Anti-tree-shaking anchor, in the house style. */
+export function LoadLiveOrdersSeam() {
+    /* keeps the import alive */
+}
+/** Kept for the seam test, which reads back what orders actually wrote. */
+export { E_ORDER_HEADER, E_ORDER_LINE, RunView };
+//# sourceMappingURL=LiveOrdersSeam.js.map

package/dist/LiveOrdersSeam.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"LiveOrdersSeam.js","sourceRoot":"","sources":["../src/LiveOrdersSeam.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAkCG;AACH,OAAO,EAEH,QAAQ,EACR,OAAO,GAIV,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EAMH,kBAAkB,GACrB,MAAM,6BAA6B,CAAC;AAErC,qFAAqF;AACrF,MAAM,cAAc,GAAG,kCAAkC,CAAC;AAC1D,MAAM,YAAY,GAAG,gCAAgC,CAAC;AAEtD;;;;;;GAMG;AACH,MAAM,cAAc,GAAG,mBAAmB,CAAC;AAiB3C;;;;;;;GAOG;AACH,MAAM,UAAU,iBAAiB;IAC7B,MAAM,EAAE,GAAG,IAAI,QAAQ,EAAE,CAAC;IAC1B,OAAO,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,cAAc,CAAC,IAAI,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,YAAY,CAAC,CAAC;AAClH,CAAC;AAED,MAAM,OAAO,cAAc;IAUvB,YACqB,IAAc,EACd,QAA2B,EAC5C,SAAiF;QAFhE,SAAI,GAAJ,IAAI,CAAU;QACd,aAAQ,GAAR,QAAQ,CAAmB;QAG5C,IAAI,CAAC,SAAS,GAAG,SAAS,IAAI,IAAI,kBAAkB,EAAE,CAAC;IAC3D,CAAC;IAIM,sBAAsB,CAAC,KAAuC;QACjE,OAAO,IAAI,CAAC,SAAS,CAAC,sBAAsB,CAAC,KAAK,CAAC,CAAC;IACxD,CAAC;IAEM,iBAAiB,CAAC,KAAkC;QACvD,OAAO,IAAI,CAAC,SAAS,CAAC,iBAAiB,CAAC,KAAK,CAAC,CAAC;IACnD,CAAC;CACJ;AAED,oDAAoD;AACpD,MAAM,UAAU,kBAAkB;IAC9B,4BAA4B;AAChC,CAAC;AAED,2EAA2E;AAC3E,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,OAAO,EAAE,CAAC"}

package/dist/SaveDealOperation.d.ts

@@ -0,0 +1,45 @@
+/**
+ * @fileoverview `Sales.SaveDeal` — the server half of browser-side deal entry.
+ *
+ * The client mutates a `DealDraft`, serializes it with `ToSaveInput()`, and calls this. This file
+ * rehydrates that payload into a real `DealEntityServer` tree and saves it, which puts the write on the
+ * SAME path an Action or an agent would take — so every rule enforced in the entity subclass applies
+ * here too, for free.
+ *
+ * WHY REHYDRATION RATHER THAN A BULK WRITE. It would be faster to build the SQL directly. It would also
+ * be a second implementation of "what it means to save a deal", and the second one is the one that
+ * eventually disagrees with the first. Going through the entity objects means validation, the owner
+ * stamp, and later the close lock all happen without this file knowing they exist.
+ *
+ * PRICES NOTHING. Every figure it accepts is either an INPUT to the pricing engine
+ * (`RequestedDiscountPct`, `OverrideUnitPrice`) or a transcription of a signed document
+ * (`AnnualGrossFees`, `DiscountAmount`, `Total`). The four `Resolved*` columns are not writable through
+ * this operation at all — they come only from an `Orders.PreviewOrder` response, and accepting them
+ * here would be the hole through which locally computed money enters the app.
+ *
+ * @module @mj-biz-apps/sales-core-entities-server
+ */
+import { IMetadataProvider, UserInfo } from '@memberjunction/core';
+import { SalesSaveDealOperation as SalesSaveDealOperationBase, type SalesSaveDealInput, type SalesSaveDealOutput } from '@mj-biz-apps/sales-entities';
+export declare class SaveDealOperation extends SalesSaveDealOperationBase {
+    protected InternalExecute(input: SalesSaveDealInput, provider: IMetadataProvider, user: UserInfo): Promise<SalesSaveDealOutput>;
+    /** The pipeline owns the selling company. NOT NULL on Pipeline, so a successful read always has one. */
+    private resolvePipelineCompany;
+    /**
+     * Reconciles the submitted lines against what is stored.
+     *
+     * The array is the COMPLETE DESIRED SET: a stored line whose ID is absent from it is removed. That
+     * is what lets a workspace holding the whole tree just send its current state, with no delta
+     * bookkeeping in the browser — and `RemoveLine` queues the deletion so it happens inside the same
+     * transaction as the inserts.
+     */
+    private applyLines;
+    /** Identical semantics to {@link applyLines}. */
+    private applySchedule;
+    /**
+     * Rewrites a collection in place to match `ordered`. In place because the getters expose the live
+     * arrays the entity's `Save()` walks; replacing them would leave `Save()` writing the old order.
+     */
+    private reorder;
+}
+//# sourceMappingURL=SaveDealOperation.d.ts.map

package/dist/SaveDealOperation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"SaveDealOperation.d.ts","sourceRoot":"","sources":["../src/SaveDealOperation.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAGH,iBAAiB,EAGjB,QAAQ,EACX,MAAM,sBAAsB,CAAC;AAE9B,OAAO,EACH,sBAAsB,IAAI,0BAA0B,EAIpD,KAAK,kBAAkB,EACvB,KAAK,mBAAmB,EAG3B,MAAM,6BAA6B,CAAC;AA6ErC,qBACa,iBAAkB,SAAQ,0BAA0B;cAC7C,eAAe,CAC3B,KAAK,EAAE,kBAAkB,EACzB,QAAQ,EAAE,iBAAiB,EAC3B,IAAI,EAAE,QAAQ,GACf,OAAO,CAAC,mBAAmB,CAAC;IAiG/B,wGAAwG;YAC1F,sBAAsB;IAuBpC;;;;;;;OAOG;YACW,UAAU;IAgCxB,iDAAiD;YACnC,aAAa;IAgC3B;;;OAGG;IACH,OAAO,CAAC,OAAO;CAIlB"}

package/dist/SaveDealOperation.js

@@ -0,0 +1,268 @@
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+/**
+ * @fileoverview `Sales.SaveDeal` — the server half of browser-side deal entry.
+ *
+ * The client mutates a `DealDraft`, serializes it with `ToSaveInput()`, and calls this. This file
+ * rehydrates that payload into a real `DealEntityServer` tree and saves it, which puts the write on the
+ * SAME path an Action or an agent would take — so every rule enforced in the entity subclass applies
+ * here too, for free.
+ *
+ * WHY REHYDRATION RATHER THAN A BULK WRITE. It would be faster to build the SQL directly. It would also
+ * be a second implementation of "what it means to save a deal", and the second one is the one that
+ * eventually disagrees with the first. Going through the entity objects means validation, the owner
+ * stamp, and later the close lock all happen without this file knowing they exist.
+ *
+ * PRICES NOTHING. Every figure it accepts is either an INPUT to the pricing engine
+ * (`RequestedDiscountPct`, `OverrideUnitPrice`) or a transcription of a signed document
+ * (`AnnualGrossFees`, `DiscountAmount`, `Total`). The four `Resolved*` columns are not writable through
+ * this operation at all — they come only from an `Orders.PreviewOrder` response, and accepting them
+ * here would be the hole through which locally computed money enters the app.
+ *
+ * @module @mj-biz-apps/sales-core-entities-server
+ */
+import { BaseRemotableOperation, LogError, } from '@memberjunction/core';
+import { RegisterClass } from '@memberjunction/global';
+import { SalesSaveDealOperation as SalesSaveDealOperationBase, } from '@mj-biz-apps/sales-entities';
+const DEAL_ENTITY = 'MJ_BizApps_Sales: Deals';
+const PIPELINE_ENTITY = 'MJ_BizApps_Sales: Pipelines';
+/** Header fields copied straight through. Listed rather than spread so an unexpected key cannot ride in. */
+const HEADER_FIELDS = [
+    'Name',
+    'PipelineID',
+    'PipelineStageID',
+    'DealTypeID',
+    'DealStatusTypeID',
+    'AccountID',
+    'PrimaryContactID',
+    'BillingContactID',
+    'Amount',
+    'CurrencyID',
+    'TermMonths',
+    'EstimatedProjectWeeks',
+    'ExecutionDate',
+    'StartDate',
+    'ExpectedCloseDate',
+    'Probability',
+    'ForecastCategoryTypeID',
+    'LeadSourceTypeID',
+    'AutoRenew',
+    'AnnualIncreasePctOverride',
+    'CancellationNoticeDaysOverride',
+    'PaymentMethod',
+    'ContractVariances',
+    'Description',
+    'NextStep',
+    'NextStepDate',
+];
+/** Line fields copied straight through. Note the absence of every `Resolved*` column — see the header. */
+const LINE_FIELDS = [
+    'ProductID',
+    'ProductName',
+    'DealLineTypeID',
+    'Quantity',
+    'RequestedDiscountPct',
+    'OverrideUnitPrice',
+    'AnnualGrossFees',
+    'DiscountAmount',
+    'Total',
+    'TermMonths',
+    'ServicePeriodStart',
+    'ServicePeriodEnd',
+    'Description',
+];
+const SCHEDULE_FIELDS = ['PaymentDate', 'Amount', 'Description'];
+function issue(Section, Message, Field = null) {
+    return { Section, Field, ClientKey: null, Severity: 'error', Message };
+}
+/**
+ * Assigns a subset of fields from a plain payload onto an entity.
+ *
+ * `undefined` is SKIPPED and `null` is APPLIED, and the difference matters: omitting a key means "leave
+ * this alone" while sending null means "clear it". Collapsing the two would make a partial update wipe
+ * every field the caller did not mention.
+ */
+function assignFields(target, source, fields) {
+    const bag = source;
+    const entity = target;
+    for (const field of fields) {
+        if (bag[field] !== undefined) {
+            entity[field] = bag[field];
+        }
+    }
+}
+let SaveDealOperation = class SaveDealOperation extends SalesSaveDealOperationBase {
+    async InternalExecute(input, provider, user) {
+        const empty = {
+            Success: false,
+            Created: false,
+            Issues: [],
+            Lines: [],
+            PaymentSchedule: [],
+        };
+        if (!input?.Name?.trim()) {
+            return { ...empty, Issues: [issue('party', 'A deal needs a name.', 'Name')] };
+        }
+        if (!input.PipelineID) {
+            return { ...empty, Issues: [issue('party', 'A deal needs a pipeline.', 'PipelineID')] };
+        }
+        try {
+            const deal = await provider.GetEntityObject(DEAL_ENTITY, user);
+            const created = !input.ID;
+            if (created) {
+                deal.NewRecord();
+            }
+            else if (!(await deal.Load(input.ID))) {
+                return { ...empty, Issues: [issue('party', `Deal ${input.ID} was not found.`, 'ID')] };
+            }
+            assignFields(deal, input, HEADER_FIELDS);
+            // CompanyID must equal the pipeline's company — a CHECK cannot reach across the FK to
+            // compare them, so it is resolved here rather than trusted from the client. An explicitly
+            // supplied value is ignored in favour of the pipeline's, because the pipeline is the only
+            // source that can be right.
+            const companyID = await this.resolvePipelineCompany(input.PipelineID, provider, user);
+            if (!companyID) {
+                return {
+                    ...empty,
+                    Issues: [issue('party', 'That pipeline could not be read, so the selling company is unknown.', 'PipelineID')],
+                };
+            }
+            deal.CompanyID = companyID;
+            // Children are attached BEFORE Save(), because Save() is the transaction: the header and
+            // every child land together or not at all.
+            if (input.Lines !== undefined) {
+                await this.applyLines(deal, input.Lines, provider, user);
+            }
+            if (input.PaymentSchedule !== undefined) {
+                await this.applySchedule(deal, input.PaymentSchedule, provider, user);
+            }
+            if (!(await deal.Save())) {
+                return {
+                    ...empty,
+                    Issues: [
+                        issue('party', `The deal could not be saved: ${deal.LatestResult?.CompleteMessage ?? 'unknown error'}`),
+                    ],
+                };
+            }
+            // The owner is expressed as INTENT and materialized as a DealTeamMember row, from which the
+            // denormalized Deal.OwnerEmployeeID stamp is derived. It runs after the save because the
+            // team row needs a persisted DealID.
+            if (input.OwnerEmployeeID) {
+                await deal.SetOwner(input.OwnerEmployeeID, user);
+                if (!(await deal.Save())) {
+                    return {
+                        ...empty,
+                        DealID: deal.ID,
+                        Created: created,
+                        Issues: [issue('party', 'The deal saved, but the owner stamp could not be written.', 'OwnerEmployeeID')],
+                    };
+                }
+            }
+            return {
+                Success: true,
+                DealID: deal.ID,
+                DealNumber: deal.DealNumber ?? null,
+                Created: created,
+                Issues: [],
+                Lines: deal.Lines.map((line, i) => ({
+                    ClientKey: input.Lines?.[i]?.ClientKey ?? null,
+                    ID: line.ID,
+                    DisplayOrder: line.DisplayOrder,
+                })),
+                PaymentSchedule: deal.PaymentSchedule.map((row, i) => ({
+                    ClientKey: input.PaymentSchedule?.[i]?.ClientKey ?? null,
+                    ID: row.ID,
+                    DisplayOrder: row.DisplayOrder,
+                })),
+            };
+        }
+        catch (err) {
+            LogError(`Sales.SaveDeal failed: ${err}`);
+            return { ...empty, Issues: [issue('party', `The deal could not be saved: ${err}`)] };
+        }
+    }
+    /** The pipeline owns the selling company. NOT NULL on Pipeline, so a successful read always has one. */
+    async resolvePipelineCompany(pipelineID, provider, user) {
+        const view = provider;
+        const result = await view.RunView({
+            EntityName: PIPELINE_ENTITY,
+            ExtraFilter: `ID = '${pipelineID}'`,
+            ResultType: 'simple',
+            Fields: ['CompanyID'],
+        }, user);
+        if (!result.Success) {
+            LogError(`Sales.SaveDeal: pipeline lookup failed: ${result.ErrorMessage}`);
+            return null;
+        }
+        const row = (result.Results ?? [])[0];
+        return row?.CompanyID ?? null;
+    }
+    /**
+     * Reconciles the submitted lines against what is stored.
+     *
+     * The array is the COMPLETE DESIRED SET: a stored line whose ID is absent from it is removed. That
+     * is what lets a workspace holding the whole tree just send its current state, with no delta
+     * bookkeeping in the browser — and `RemoveLine` queues the deletion so it happens inside the same
+     * transaction as the inserts.
+     */
+    async applyLines(deal, lines, provider, user) {
+        await deal.LoadLines(user);
+        const keep = new Set(lines.map((l) => l.ID).filter((id) => !!id));
+        for (let i = deal.Lines.length - 1; i >= 0; i--) {
+            if (!keep.has(deal.Lines[i].ID)) {
+                deal.RemoveLine(i);
+            }
+        }
+        const byID = new Map(deal.Lines.map((l) => [l.ID, l]));
+        const ordered = [];
+        for (const incoming of lines) {
+            let entity = incoming.ID ? byID.get(incoming.ID) : undefined;
+            if (!entity) {
+                entity = await deal.CreateLine(user);
+            }
+            assignFields(entity, incoming, LINE_FIELDS);
+            ordered.push(entity);
+        }
+        // Rebuild the collection in the caller's order — the array's order is what DisplayOrder is
+        // derived from on save, so submitted order is the authority on how lines read.
+        this.reorder(deal.Lines, ordered);
+    }
+    /** Identical semantics to {@link applyLines}. */
+    async applySchedule(deal, rows, provider, user) {
+        await deal.LoadPaymentSchedule(user);
+        const keep = new Set(rows.map((r) => r.ID).filter((id) => !!id));
+        for (let i = deal.PaymentSchedule.length - 1; i >= 0; i--) {
+            if (!keep.has(deal.PaymentSchedule[i].ID)) {
+                deal.RemoveScheduleRow(i);
+            }
+        }
+        const byID = new Map(deal.PaymentSchedule.map((r) => [r.ID, r]));
+        const ordered = [];
+        for (const incoming of rows) {
+            let entity = incoming.ID ? byID.get(incoming.ID) : undefined;
+            if (!entity) {
+                entity = await deal.CreateScheduleRow(user);
+            }
+            assignFields(entity, incoming, SCHEDULE_FIELDS);
+            ordered.push(entity);
+        }
+        this.reorder(deal.PaymentSchedule, ordered);
+    }
+    /**
+     * Rewrites a collection in place to match `ordered`. In place because the getters expose the live
+     * arrays the entity's `Save()` walks; replacing them would leave `Save()` writing the old order.
+     */
+    reorder(live, ordered) {
+        live.length = 0;
+        live.push(...ordered);
+    }
+};
+SaveDealOperation = __decorate([
+    RegisterClass(BaseRemotableOperation, 'Sales.SaveDeal')
+], SaveDealOperation);
+export { SaveDealOperation };
+//# sourceMappingURL=SaveDealOperation.js.map

package/dist/SaveDealOperation.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"SaveDealOperation.js","sourceRoot":"","sources":["../src/SaveDealOperation.ts"],"names":[],"mappings":";;;;;;AAAA;;;;;;;;;;;;;;;;;;;;GAoBG;AACH,OAAO,EAEH,sBAAsB,EAGtB,QAAQ,GAEX,MAAM,sBAAsB,CAAC;AAC9B,OAAO,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAC;AACvD,OAAO,EACH,sBAAsB,IAAI,0BAA0B,GAQvD,MAAM,6BAA6B,CAAC;AAIrC,MAAM,WAAW,GAAG,yBAAyB,CAAC;AAC9C,MAAM,eAAe,GAAG,6BAA6B,CAAC;AAEtD,4GAA4G;AAC5G,MAAM,aAAa,GAAG;IAClB,MAAM;IACN,YAAY;IACZ,iBAAiB;IACjB,YAAY;IACZ,kBAAkB;IAClB,WAAW;IACX,kBAAkB;IAClB,kBAAkB;IAClB,QAAQ;IACR,YAAY;IACZ,YAAY;IACZ,uBAAuB;IACvB,eAAe;IACf,WAAW;IACX,mBAAmB;IACnB,aAAa;IACb,wBAAwB;IACxB,kBAAkB;IAClB,WAAW;IACX,2BAA2B;IAC3B,gCAAgC;IAChC,eAAe;IACf,mBAAmB;IACnB,aAAa;IACb,UAAU;IACV,cAAc;CACR,CAAC;AAEX,0GAA0G;AAC1G,MAAM,WAAW,GAAG;IAChB,WAAW;IACX,aAAa;IACb,gBAAgB;IAChB,UAAU;IACV,sBAAsB;IACtB,mBAAmB;IACnB,iBAAiB;IACjB,gBAAgB;IAChB,OAAO;IACP,YAAY;IACZ,oBAAoB;IACpB,kBAAkB;IAClB,aAAa;CACP,CAAC;AAEX,MAAM,eAAe,GAAG,CAAC,aAAa,EAAE,QAAQ,EAAE,aAAa,CAAU,CAAC;AAE1E,SAAS,KAAK,CAAC,OAAkC,EAAE,OAAe,EAAE,QAAuB,IAAI;IAC3F,OAAO,EAAE,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,OAAO,EAAE,CAAC;AAC3E,CAAC;AAED;;;;;;GAMG;AACH,SAAS,YAAY,CAAmB,MAAkB,EAAE,MAAS,EAAE,MAAyB;IAC5F,MAAM,GAAG,GAAG,MAA4C,CAAC;IACzD,MAAM,MAAM,GAAG,MAA4C,CAAC;IAC5D,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QACzB,IAAI,GAAG,CAAC,KAAK,CAAC,KAAK,SAAS,EAAE,CAAC;YAC3B,MAAM,CAAC,KAAK,CAAC,GAAG,GAAG,CAAC,KAAK,CAAC,CAAC;QAC/B,CAAC;IACL,CAAC;AACL,CAAC;AAGM,IAAM,iBAAiB,GAAvB,MAAM,iBAAkB,SAAQ,0BAA0B;IACnD,KAAK,CAAC,eAAe,CAC3B,KAAyB,EACzB,QAA2B,EAC3B,IAAc;QAEd,MAAM,KAAK,GAAwB;YAC/B,OAAO,EAAE,KAAK;YACd,OAAO,EAAE,KAAK;YACd,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,EAAE;YACT,eAAe,EAAE,EAAE;SACtB,CAAC;QAEF,IAAI,CAAC,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,CAAC;YACvB,OAAO,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,sBAAsB,EAAE,MAAM,CAAC,CAAC,EAAE,CAAC;QAClF,CAAC;QACD,IAAI,CAAC,KAAK,CAAC,UAAU,EAAE,CAAC;YACpB,OAAO,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,0BAA0B,EAAE,YAAY,CAAC,CAAC,EAAE,CAAC;QAC5F,CAAC;QAED,IAAI,CAAC;YACD,MAAM,IAAI,GAAG,MAAM,QAAQ,CAAC,eAAe,CAAmB,WAAW,EAAE,IAAI,CAAC,CAAC;YACjF,MAAM,OAAO,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;YAE1B,IAAI,OAAO,EAAE,CAAC;gBACV,IAAI,CAAC,SAAS,EAAE,CAAC;YACrB,CAAC;iBAAM,IAAI,CAAC,CAAC,MAAM,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,EAAY,CAAC,CAAC,EAAE,CAAC;gBAChD,OAAO,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,QAAQ,KAAK,CAAC,EAAE,iBAAiB,EAAE,IAAI,CAAC,CAAC,EAAE,CAAC;YAC3F,CAAC;YAED,YAAY,CAAC,IAAI,EAAE,KAAK,EAAE,aAAa,CAAC,CAAC;YAEzC,sFAAsF;YACtF,0FAA0F;YAC1F,0FAA0F;YAC1F,4BAA4B;YAC5B,MAAM,SAAS,GAAG,MAAM,IAAI,CAAC,sBAAsB,CAAC,KAAK,CAAC,UAAU,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YACtF,IAAI,CAAC,SAAS,EAAE,CAAC;gBACb,OAAO;oBACH,GAAG,KAAK;oBACR,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,qEAAqE,EAAE,YAAY,CAAC,CAAC;iBAChH,CAAC;YACN,CAAC;YACD,IAAI,CAAC,SAAS,GAAG,SAAS,CAAC;YAE3B,yFAAyF;YACzF,2CAA2C;YAC3C,IAAI,KAAK,CAAC,KAAK,KAAK,SAAS,EAAE,CAAC;gBAC5B,MAAM,IAAI,CAAC,UAAU,CAAC,IAAI,EAAE,KAAK,CAAC,KAAK,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YAC7D,CAAC;YACD,IAAI,KAAK,CAAC,eAAe,KAAK,SAAS,EAAE,CAAC;gBACtC,MAAM,IAAI,CAAC,aAAa,CAAC,IAAI,EAAE,KAAK,CAAC,eAAe,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;YAC1E,CAAC;YAED,IAAI,CAAC,CAAC,MAAM,IAAI,CAAC,IAAI,EAAE,CAAC,EAAE,CAAC;gBACvB,OAAO;oBACH,GAAG,KAAK;oBACR,MAAM,EAAE;wBACJ,KAAK,CAAC,OAAO,EAAE,gCAAgC,IAAI,CAAC,YAAY,EAAE,eAAe,IAAI,eAAe,EAAE,CAAC;qBAC1G;iBACJ,CAAC;YACN,CAAC;YAED,4FAA4F;YAC5F,yFAAyF;YACzF,qCAAqC;YACrC,IAAI,KAAK,CAAC,eAAe,EAAE,CAAC;gBACxB,MAAM,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,eAAe,EAAE,IAAI,CAAC,CAAC;gBACjD,IAAI,CAAC,CAAC,MAAM,IAAI,CAAC,IAAI,EAAE,CAAC,EAAE,CAAC;oBACvB,OAAO;wBACH,GAAG,KAAK;wBACR,MAAM,EAAE,IAAI,CAAC,EAAE;wBACf,OAAO,EAAE,OAAO;wBAChB,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,2DAA2D,EAAE,iBAAiB,CAAC,CAAC;qBAC3G,CAAC;gBACN,CAAC;YACL,CAAC;YAED,OAAO;gBACH,OAAO,EAAE,IAAI;gBACb,MAAM,EAAE,IAAI,CAAC,EAAE;gBACf,UAAU,EAAE,IAAI,CAAC,UAAU,IAAI,IAAI;gBACnC,OAAO,EAAE,OAAO;gBAChB,MAAM,EAAE,EAAE;gBACV,KAAK,EAAE,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;oBAChC,SAAS,EAAE,KAAK,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC,EAAE,SAAS,IAAI,IAAI;oBAC9C,EAAE,EAAE,IAAI,CAAC,EAAE;oBACX,YAAY,EAAE,IAAI,CAAC,YAAY;iBAClC,CAAC,CAAC;gBACH,eAAe,EAAE,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC;oBACnD,SAAS,EAAE,KAAK,CAAC,eAAe,EAAE,CAAC,CAAC,CAAC,EAAE,SAAS,IAAI,IAAI;oBACxD,EAAE,EAAE,GAAG,CAAC,EAAE;oBACV,YAAY,EAAE,GAAG,CAAC,YAAY;iBACjC,CAAC,CAAC;aACN,CAAC;QACN,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACX,QAAQ,CAAC,0BAA0B,GAAG,EAAE,CAAC,CAAC;YAC1C,OAAO,EAAE,GAAG,KAAK,EAAE,MAAM,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,gCAAgC,GAAG,EAAE,CAAC,CAAC,EAAE,CAAC;QACzF,CAAC;IACL,CAAC;IAED,wGAAwG;IAChG,KAAK,CAAC,sBAAsB,CAChC,UAAkB,EAClB,QAA2B,EAC3B,IAAc;QAEd,MAAM,IAAI,GAAG,QAAuC,CAAC;QACrD,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,OAAO,CAC7B;YACI,UAAU,EAAE,eAAe;YAC3B,WAAW,EAAE,SAAS,UAAU,GAAG;YACnC,UAAU,EAAE,QAAQ;YACpB,MAAM,EAAE,CAAC,WAAW,CAAC;SACxB,EACD,IAAI,CACP,CAAC;QACF,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;YAClB,QAAQ,CAAC,2CAA2C,MAAM,CAAC,YAAY,EAAE,CAAC,CAAC;YAC3E,OAAO,IAAI,CAAC;QAChB,CAAC;QACD,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,CAAC,CAAuC,CAAC;QAC5E,OAAO,GAAG,EAAE,SAAS,IAAI,IAAI,CAAC;IAClC,CAAC;IAED;;;;;;;OAOG;IACK,KAAK,CAAC,UAAU,CACpB,IAAsB,EACtB,KAA2B,EAC3B,QAA2B,EAC3B,IAAc;QAEd,MAAM,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC;QAE3B,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,EAAgB,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;QAChF,KAAK,IAAI,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC9C,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC;gBAC9B,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;YACvB,CAAC;QACL,CAAC;QAED,MAAM,IAAI,GAAG,IAAI,GAAG,CAAuC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QAC7F,MAAM,OAAO,GAAmC,EAAE,CAAC;QAEnD,KAAK,MAAM,QAAQ,IAAI,KAAK,EAAE,CAAC;YAC3B,IAAI,MAAM,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;YAC7D,IAAI,CAAC,MAAM,EAAE,CAAC;gBACV,MAAM,GAAG,MAAM,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC;YACzC,CAAC;YACD,YAAY,CAAC,MAAM,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC;YAC5C,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACzB,CAAC;QAED,2FAA2F;QAC3F,+EAA+E;QAC/E,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,EAAE,OAAO,CAAC,CAAC;IACtC,CAAC;IAED,iDAAiD;IACzC,KAAK,CAAC,aAAa,CACvB,IAAsB,EACtB,IAAqC,EACrC,QAA2B,EAC3B,IAAc;QAEd,MAAM,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAAC;QAErC,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,EAAgB,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;QAC/E,KAAK,IAAI,CAAC,GAAG,IAAI,CAAC,eAAe,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YACxD,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,CAAC;gBACxC,IAAI,CAAC,iBAAiB,CAAC,CAAC,CAAC,CAAC;YAC9B,CAAC;QACL,CAAC;QAED,MAAM,IAAI,GAAG,IAAI,GAAG,CAChB,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAC7C,CAAC;QACF,MAAM,OAAO,GAA8C,EAAE,CAAC;QAE9D,KAAK,MAAM,QAAQ,IAAI,IAAI,EAAE,CAAC;YAC1B,IAAI,MAAM,GAAG,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;YAC7D,IAAI,CAAC,MAAM,EAAE,CAAC;gBACV,MAAM,GAAG,MAAM,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;YAChD,CAAC;YACD,YAAY,CAAC,MAAM,EAAE,QAAQ,EAAE,eAAe,CAAC,CAAC;YAChD,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACzB,CAAC;QAED,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,eAAe,EAAE,OAAO,CAAC,CAAC;IAChD,CAAC;IAED;;;OAGG;IACK,OAAO,CAAI,IAAS,EAAE,OAAY;QACtC,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC;QAChB,IAAI,CAAC,IAAI,CAAC,GAAG,OAAO,CAAC,CAAC;IAC1B,CAAC;CACJ,CAAA;AA/MY,iBAAiB;IAD7B,aAAa,CAAC,sBAAsB,EAAE,gBAAgB,CAAC;GAC3C,iBAAiB,CA+M7B"}

package/dist/SequenceService.d.ts

@@ -0,0 +1,39 @@
+/**
+ * @fileoverview `SequenceService` — calls the DB-level atomic numbering sproc from TypeScript, so
+ * `DealEntityServer` can assign `DealNumber` before `super.Save()` commits the row.
+ *
+ * WHY THE COUNTER IS IN SQL AND THIS IS THE ONLY SQL-SHAPED THING HERE. `spAssignNextDealNumber` needs
+ * `HOLDLOCK, UPDLOCK` read-modify-write semantics that have no application-level equivalent — two deals
+ * created in the same instant must not take the same number, and no amount of careful TypeScript can
+ * promise that. Everything else about numbering (when to assign, what to do when it fails) is here.
+ *
+ * PROVIDER: INJECTED AND REQUIRED, with no global fallback. The caller is the entity server, which
+ * passes its own `ProviderToUse`, so the sproc call rides the SAME connection context — and therefore
+ * the same transaction — as the save it is numbering. A global provider would take the number on a
+ * different connection, and a rolled-back save would then leave the number consumed. Copied deliberately
+ * from accounting's `SequenceService`, including that constraint.
+ *
+ * @module @mj-biz-apps/sales-core-entities-server
+ */
+import { IMetadataProvider, UserInfo } from '@memberjunction/core';
+/**
+ * This app's schema, in ONE place in this package.
+ *
+ * Exported because `DealEntityServer` needs it too, for the targeted `UPDATE` that maintains the
+ * `Deal.Amount` cache. A second copy of the literal is a second thing to change when the schema is
+ * renamed for the PostgreSQL conversion, and the one that gets missed is the one nobody tested.
+ */
+export declare const SALES_SCHEMA = "__mj_BizAppsSales";
+/**
+ * Atomically increments the singleton deal counter and returns the formatted `DEAL-{seq:000000}`.
+ *
+ * Singleton rather than per-company, matching contracts' `ContractSequence` and orders'
+ * `OrderSequence` — see the migration comment on `DealSequence` for why accounting's per-company,
+ * per-fiscal-year scope is the exception and not the pattern.
+ *
+ * @throws when the sproc returns nothing, rather than letting a null number reach the row. A deal with
+ *         no number is recoverable; a deal that silently stored `null` where a number was expected is
+ *         the kind of thing found weeks later by a report that skips it.
+ */
+export declare function getNextDealNumber(contextUser: UserInfo, provider: IMetadataProvider): Promise<string>;
+//# sourceMappingURL=SequenceService.d.ts.map

package/dist/SequenceService.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"SequenceService.d.ts","sourceRoot":"","sources":["../src/SequenceService.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AACH,OAAO,EAAE,iBAAiB,EAAE,QAAQ,EAAE,MAAM,sBAAsB,CAAC;AAGnE;;;;;;GAMG;AACH,eAAO,MAAM,YAAY,sBAAsB,CAAC;AAEhD;;;;;;;;;;GAUG;AACH,wBAAsB,iBAAiB,CAAC,WAAW,EAAE,QAAQ,EAAE,QAAQ,EAAE,iBAAiB,GAAG,OAAO,CAAC,MAAM,CAAC,CAsB3G"}

package/dist/SequenceService.js

@@ -0,0 +1,44 @@
+/**
+ * This app's schema, in ONE place in this package.
+ *
+ * Exported because `DealEntityServer` needs it too, for the targeted `UPDATE` that maintains the
+ * `Deal.Amount` cache. A second copy of the literal is a second thing to change when the schema is
+ * renamed for the PostgreSQL conversion, and the one that gets missed is the one nobody tested.
+ */
+export const SALES_SCHEMA = '__mj_BizAppsSales';
+/**
+ * Atomically increments the singleton deal counter and returns the formatted `DEAL-{seq:000000}`.
+ *
+ * Singleton rather than per-company, matching contracts' `ContractSequence` and orders'
+ * `OrderSequence` — see the migration comment on `DealSequence` for why accounting's per-company,
+ * per-fiscal-year scope is the exception and not the pattern.
+ *
+ * @throws when the sproc returns nothing, rather than letting a null number reach the row. A deal with
+ *         no number is recoverable; a deal that silently stored `null` where a number was expected is
+ *         the kind of thing found weeks later by a report that skips it.
+ */
+export async function getNextDealNumber(contextUser, provider) {
+    const sqlProvider = getSqlServerProvider(provider);
+    const sql = `
+        DECLARE @dealNumber NVARCHAR(50);
+        EXEC ${SALES_SCHEMA}.spAssignNextDealNumber
+            @DealNumber = @dealNumber OUTPUT;
+        SELECT @dealNumber AS DealNumber;
+    `;
+    // ExecuteSQL binds an OBJECT of parameters BY NAME. An array is treated as positional (p0), which
+    // would not match a named @-param — accounting's service records learning this the hard way, and the
+    // note is kept here because the sproc taking no inputs makes it easy to reintroduce later.
+    const rows = await sqlProvider.ExecuteSQL(sql, {}, { isMutation: true, description: 'spAssignNextDealNumber' }, contextUser);
+    const value = rows?.[0]?.DealNumber;
+    if (!value || typeof value !== 'string') {
+        throw new Error('SequenceService.getNextDealNumber: spAssignNextDealNumber returned no value');
+    }
+    return value;
+}
+function getSqlServerProvider(provider) {
+    if (!provider) {
+        throw new Error('SequenceService: an IMetadataProvider must be injected — there is no global fallback');
+    }
+    return provider;
+}
+//# sourceMappingURL=SequenceService.js.map

package/dist/SequenceService.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"SequenceService.js","sourceRoot":"","sources":["../src/SequenceService.ts"],"names":[],"mappings":"AAoBA;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,YAAY,GAAG,mBAAmB,CAAC;AAEhD;;;;;;;;;;GAUG;AACH,MAAM,CAAC,KAAK,UAAU,iBAAiB,CAAC,WAAqB,EAAE,QAA2B;IACtF,MAAM,WAAW,GAAG,oBAAoB,CAAC,QAAQ,CAAC,CAAC;IACnD,MAAM,GAAG,GAAG;;eAED,YAAY;;;KAGtB,CAAC;IACF,kGAAkG;IAClG,qGAAqG;IACrG,2FAA2F;IAC3F,MAAM,IAAI,GAAG,MAAM,WAAW,CAAC,UAAU,CACrC,GAAG,EACH,EAAE,EACF,EAAE,UAAU,EAAE,IAAI,EAAE,WAAW,EAAE,wBAAwB,EAAE,EAC3D,WAAW,CACd,CAAC;IACF,MAAM,KAAK,GAAG,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,UAAU,CAAC;IACpC,IAAI,CAAC,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QACtC,MAAM,IAAI,KAAK,CAAC,6EAA6E,CAAC,CAAC;IACnG,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC;AAED,SAAS,oBAAoB,CAAC,QAA2B;IACrD,IAAI,CAAC,QAAQ,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CAAC,sFAAsF,CAAC,CAAC;IAC5G,CAAC;IACD,OAAO,QAAiC,CAAC;AAC7C,CAAC"}