ghost 6.54.0 → 6.54.1

package/core/server/services/members/import-export/import/importer.ts

@@ -0,0 +1,430 @@
+import moment from 'moment-timezone';
+import buildCompletionEmail from './completion-email';
+import type {Knex} from 'knex';
+import type {MemberImportRow, ImportErrorRow, ImportLabel, Label} from './row';
+import type {RowSpool, SpooledRows} from './spool';
+
+const metrics = require('@tryghost/metrics');
+const errors = require('@tryghost/errors');
+const tpl = require('@tryghost/tpl');
+const logging = require('@tryghost/logging');
+
+// The members CSV importer, sliced into one concern per method. Two entry points by
+// design: importCSV decides inline-vs-deferred by load, since a large import must not
+// hold a request open; importInline always runs now, for callers with no request to
+// protect (the Revue data import). The kernel takes a plain MemberImportRow array -- the
+// CSV reader and the deferred spool both produce one -- so a test can drive it from a
+// literal array. Collaborators are injected one per concern; knex is first-class, and no
+// Bookshelf model is referenced below the boundary that owns it.
+
+// The import service's arguments, built by the endpoint from the request frame.
+// requestUserEmail is null when the request carried no user, so the deferred path falls
+// back to the site owner.
+export interface ImportRequest {
+    filePath: string;
+    mapping?: Record<string, string>;
+    extraLabels?: Label[];
+    requestUserEmail: string | null;
+}
+
+interface VerificationTrigger {
+    testImportThreshold(): Promise<void>;
+}
+
+// A tier, narrowed to the id the import reads off it.
+export interface Tier {
+    id: {toString(): string};
+}
+
+// A member model, narrowed to what the import loop touches. Relations are loaded
+// via withRelated before the loop reads them, so related() is treated as present.
+interface MemberModel {
+    id: string;
+    name: string;
+    note: string;
+    related(relation: string): {toJSON(): Array<{name: string}>; length: number};
+}
+
+// Reads a source into import rows. The CSV reader is the only implementation today;
+// a test can inject one that returns a fixed rows array with no file involved.
+type ReadRows = (path: string, mapping?: Record<string, string>) => Promise<MemberImportRow[]>;
+
+// The members aggregate: the one thing the import writes. Its methods are the
+// member operations a row performs, plus the import label lookup, which is a
+// member-tagging concern rather than a store of its own. Hides the Member and
+// Label models behind the shape the import needs.
+export interface MembersRepository {
+    get(data: {email?: string}, options: object): Promise<MemberModel | null>;
+    create(values: object, options: object): Promise<MemberModel>;
+    update(values: object, options: object): Promise<MemberModel>;
+    getCustomerIdByEmail(email: string): Promise<string | undefined>;
+    linkStripeCustomer(data: {customer_id: string; member_id: string}, options: object): Promise<void>;
+    getImportLabel(name: string): Promise<{toJSON(): ImportLabel} | null>;
+}
+
+// Tiers: the tier catalog, an aggregate of its own, consulted to resolve the
+// default tier and a tier a row names.
+interface TiersRepository {
+    getDefault(): Promise<Tier>;
+    getByName(name: string): Promise<Tier | null>;
+}
+
+// Stripe: the external subscription work a row with Stripe data triggers.
+interface StripeSubscriptions {
+    forceStripeSubscriptionToProduct(
+        args: {customer_id: string; product_id?: string},
+        options: object
+    ): Promise<{isNewStripePrice: boolean; stripePriceId: string}>;
+    archivePrice(stripePriceId: string): Promise<void>;
+}
+
+// Gifts: reassigning a redeemed gift to the imported member.
+export interface GiftService {
+    reassignRedeemer(giftId: string, memberId: string, options: object): Promise<void>;
+}
+
+// The completion email concern: who it goes to, the links it carries, and sending
+// it. The default recipient is the site owner, used when the request had no user.
+export interface EmailNotifications {
+    send(options: object): Promise<unknown>;
+    getDefaultRecipient(): Promise<string>;
+    urlFor(type: string, data: unknown, absolute: boolean): string;
+}
+
+// The collaborators the import depends on, one per concern.
+interface ImporterDeps {
+    knex: Knex;
+    readRows: ReadRows;
+    spool: RowSpool;
+    members: MembersRepository;
+    tiers: TiersRepository;
+    stripe: StripeSubscriptions;
+    gifts: GiftService;
+    email: EmailNotifications;
+    addJob: (job: {job: () => Promise<void>; offloaded: boolean; name: string}) => void;
+    getTimezone: () => string;
+    getInlineThreshold: () => number;
+}
+
+// The values built for one row and handed to the members repository. Indexed off
+// MemberImportRow so a change to the import schema surfaces here: rename or retype a
+// field the kernel writes and this stops compiling until the mapping is updated. name
+// and note may be replaced with the existing member's values (both strings, already in
+// range); created_at may be clamped to now when the row dates a member in the future;
+// newsletters is opaque ORM relation JSON, an honest unknown at the model boundary.
+interface MemberImportValues {
+    email: MemberImportRow['email'];
+    name: MemberImportRow['name'];
+    note: MemberImportRow['note'];
+    subscribed: MemberImportRow['subscribed'];
+    created_at: MemberImportRow['created_at'] | Date;
+    labels: Label[];
+    newsletters?: unknown;
+}
+
+// The kernel does the importing, so it is what knows whether anything imported and,
+// if so, which label the members were tagged with (a label is only persisted once
+// attached to a member). The label is absent when nothing imported; the orchestrator
+// reads it as null and never fetches it itself.
+interface ImportResult {
+    imported: number;
+    errors: ImportErrorRow[];
+    importLabel?: ImportLabel;
+}
+
+// What the domain service returns: an import that ran inline carries its stats and
+// label for the response; a deferred one carries only the size accepted, because
+// the work -- and its email -- finishes after the request is answered.
+export type ImportOutcome =
+    | {deferred: false; originalImportSize: number; result: ImportResult}
+    | {deferred: true; originalImportSize: number};
+
+const messages = {
+    freeMemberNotAllowedImportTier: 'You cannot import a free member with a specified tier.',
+    invalidImportTier: '"{tier}" is not a valid tier.',
+    giftCannotCombineWithImportTier: 'Cannot specify both gift_id and import_tier.',
+    giftCannotCombineWithComplimentary: 'Cannot specify both gift_id and complimentary_plan.',
+    giftReassignFailed: 'Failed to reassign gift to member.'
+};
+
+// Columns whose presence makes a row slow to import (they reach out to Stripe), so
+// a file carrying any of them is deferred regardless of size. The routing decision
+// only ever sees the boolean this produces, never the column names themselves.
+const EXPENSIVE_COLUMNS = ['stripe_customer_id'];
+
+// The context each repository call runs under: the import identifies itself, and a
+// created member is additionally flagged as imported so downstream hooks can tell.
+const IMPORT_CONTEXT = {importer: true};
+const CREATE_CONTEXT = {import: true};
+
+// The import label members are tagged with, named for the request time. Shared by
+// the kernel (the label) and the deferred email (the error CSV filename).
+function buildImportLabelName(timezone: string): string {
+    return `Import ${moment().tz(timezone).format('YYYY-MM-DD HH:mm')}`;
+}
+
+function hasExpensiveColumns(rows: MemberImportRow[]): boolean {
+    return rows.some(row => EXPENSIVE_COLUMNS.some(column => !!row[column]));
+}
+
+// The routing decision, in terms it can reason about without knowing what a member
+// or a Stripe customer is: an empty or small, cheap file imports inline (an empty
+// one simply imports nothing); a large one, or one carrying expensive columns, is
+// too slow to hold a request open, so it is deferred to a background job.
+function canImportInline(rowCount: number, expensive: boolean, inlineThreshold: number): boolean {
+    return rowCount <= inlineThreshold && !expensive;
+}
+
+class MembersCSVImporter {
+    private _knex: Knex;
+    private _readRows: ReadRows;
+    private _spool: RowSpool;
+    private _members: MembersRepository;
+    private _tiers: TiersRepository;
+    private _stripe: StripeSubscriptions;
+    private _gifts: GiftService;
+    private _email: EmailNotifications;
+    private _addJob: (job: {job: () => Promise<void>; offloaded: boolean; name: string}) => void;
+    private _getTimezone: () => string;
+    private _getInlineThreshold: () => number;
+
+    constructor({knex, readRows, spool, members, tiers, stripe, gifts, email, addJob, getTimezone, getInlineThreshold}: ImporterDeps) {
+        this._knex = knex;
+        this._readRows = readRows;
+        this._spool = spool;
+        this._members = members;
+        this._tiers = tiers;
+        this._stripe = stripe;
+        this._gifts = gifts;
+        this._email = email;
+        this._addJob = addJob;
+        this._getTimezone = getTimezone;
+        this._getInlineThreshold = getInlineThreshold;
+    }
+
+    async importCSV(request: ImportRequest, verificationTrigger: VerificationTrigger): Promise<ImportOutcome> {
+        const rows = await this._readRows(request.filePath, request.mapping);
+        const labelName = buildImportLabelName(this._getTimezone());
+        const extraLabels = request.extraLabels ?? [];
+
+        if (canImportInline(rows.length, hasExpensiveColumns(rows), this._getInlineThreshold())) {
+            const result = await this.importRows(rows, labelName, extraLabels);
+            await verificationTrigger.testImportThreshold();
+            return {deferred: false, originalImportSize: rows.length, result};
+        }
+
+        await this.deferImport(rows, {labelName, extraLabels, requestUserEmail: request.requestUserEmail}, verificationTrigger);
+        return {deferred: true, originalImportSize: rows.length};
+    }
+
+    async importInline(request: ImportRequest, verificationTrigger: VerificationTrigger): Promise<ImportResult> {
+        const rows = await this._readRows(request.filePath, request.mapping);
+        const result = await this.importRows(rows, buildImportLabelName(this._getTimezone()), request.extraLabels ?? []);
+        await verificationTrigger.testImportThreshold();
+        return result;
+    }
+
+    private async deferImport(
+        rows: MemberImportRow[],
+        {labelName, extraLabels, requestUserEmail}: {labelName: string; extraLabels: Label[]; requestUserEmail: string | null},
+        verificationTrigger: VerificationTrigger
+    ): Promise<void> {
+        // Resolved here, not at the API boundary, so the owner lookup only runs when
+        // a request without a user actually reaches the deferred path.
+        const emailRecipient: string = requestUserEmail ?? await this._email.getDefaultRecipient();
+        const spooled = await this._spool.write(rows);
+
+        this._addJob({
+            job: () => this.runImportJob(spooled, {labelName, extraLabels, emailRecipient}, verificationTrigger),
+            offloaded: false,
+            name: 'members-import'
+        });
+    }
+
+    // Swallows its own errors: a failed import or email must not leave the queued job
+    // rejected, and the verification trigger must still run afterwards either way.
+    private async runImportJob(
+        spooled: SpooledRows,
+        {labelName, extraLabels, emailRecipient}: {labelName: string; extraLabels: Label[]; emailRecipient: string},
+        verificationTrigger: VerificationTrigger
+    ): Promise<void> {
+        try {
+            const spooledRows = await spooled.read();
+            const result = await this.importRows(spooledRows, labelName, extraLabels);
+            const importLabel = result.importLabel ?? null;
+            await this._email.send(buildCompletionEmail({
+                result,
+                recipient: emailRecipient,
+                labelName,
+                importLabel,
+                urlFor: this._email.urlFor
+            }));
+        } catch (e) {
+            logging.error('Error in members import job');
+            logging.error(e);
+        } finally {
+            await spooled.remove();
+        }
+
+        try {
+            await verificationTrigger.testImportThreshold();
+        } catch (e) {
+            logging.error('Error in members import job when testing import threshold');
+            logging.error(e);
+        }
+    }
+
+    // The members import kernel, shared by the inline and deferred paths. Each row is
+    // created or updated in its own transaction, so one row's failure rolls back only
+    // itself; failures are collected, and any Stripe prices an import tier created are
+    // archived at the end.
+    private async importRows(rows: MemberImportRow[], labelName: string, extraLabels: Label[]): Promise<ImportResult> {
+        const importLabel: Label = {name: labelName};
+        const globalLabels: Label[] = [importLabel, ...extraLabels];
+        const performStart = Date.now();
+        const defaultTier = await this._tiers.getDefault();
+        const tierIdCache = new Map();
+        const archivableStripePriceIds: string[] = [];
+        // Copied per row: the member model stamps ids and trims names onto these in
+        // place, and each row runs in its own transaction that can roll back. A
+        // caller can hand in a nameless label, which the model would drop anyway.
+        const cloneGlobalLabels = (): Label[] => globalLabels
+            .map(label => ({...label}))
+            .filter(label => label.name);
+
+        let imported = 0;
+        const importErrors: ImportErrorRow[] = [];
+        for (const row of rows) {
+            const trx = await this._knex.transaction(undefined, {doNotRejectOnRollback: false});
+            const options = {transacting: trx, context: IMPORT_CONTEXT};
+            try {
+                if (row.gift_id) {
+                    if (row.import_tier) {
+                        throw wrapGiftError(new errors.DataImportError({message: tpl(messages.giftCannotCombineWithImportTier)}));
+                    }
+                    if (row.complimentary_plan) {
+                        throw wrapGiftError(new errors.DataImportError({message: tpl(messages.giftCannotCombineWithComplimentary)}));
+                    }
+                }
+
+                const createdAt = (row.created_at && moment(row.created_at).isAfter(moment())) ? moment().toDate() : row.created_at;
+                const memberValues: MemberImportValues = {
+                    email: row.email,
+                    name: row.name,
+                    note: row.note,
+                    subscribed: row.subscribed,
+                    created_at: createdAt,
+                    labels: [...row.labels, ...cloneGlobalLabels()]
+                };
+                const existingMember = row.email
+                    ? await this._members.get({email: row.email}, {...options, withRelated: ['labels', 'newsletters']})
+                    : null;
+                let member;
+                if (existingMember) {
+                    const existingLabels = existingMember.related('labels').toJSON();
+                    const existingNewsletters = existingMember.related('newsletters');
+                    if (existingNewsletters.length > 0 && memberValues.subscribed) {
+                        memberValues.newsletters = existingNewsletters.toJSON();
+                    }
+                    if (!existingNewsletters.length && memberValues.subscribed) {
+                        memberValues.subscribed = false;
+                    }
+                    if (!row.name) {
+                        memberValues.name = existingMember.name;
+                    }
+                    if (!row.note) {
+                        memberValues.note = existingMember.note;
+                    }
+                    member = await this._members.update({
+                        ...memberValues,
+                        labels: existingLabels.concat(memberValues.labels)
+                    }, {...options, id: existingMember.id});
+                } else {
+                    member = await this._members.create(memberValues, Object.assign({}, options, {context: CREATE_CONTEXT}));
+                }
+
+                let importTierId;
+                if (row.import_tier) {
+                    if (!tierIdCache.has(row.import_tier)) {
+                        const tier = await this._tiers.getByName(row.import_tier);
+                        tierIdCache.set(row.import_tier, tier ? tier.id.toString() : null);
+                    }
+                    importTierId = tierIdCache.get(row.import_tier);
+                    if (!importTierId) {
+                        throw new errors.DataImportError({message: tpl(messages.invalidImportTier, {tier: row.import_tier})});
+                    }
+                }
+
+                if (row.stripe_customer_id) {
+                    let stripeCustomerId;
+                    if (row.stripe_customer_id.toLowerCase() === 'auto') {
+                        stripeCustomerId = row.email ? await this._members.getCustomerIdByEmail(row.email) : undefined;
+                    } else {
+                        stripeCustomerId = row.stripe_customer_id;
+                    }
+                    if (stripeCustomerId) {
+                        if (row.import_tier) {
+                            const {isNewStripePrice, stripePriceId} = await this._stripe.forceStripeSubscriptionToProduct({
+                                customer_id: stripeCustomerId,
+                                product_id: importTierId
+                            }, options);
+                            if (isNewStripePrice) {
+                                archivableStripePriceIds.push(stripePriceId);
+                            }
+                        }
+                        await this._members.linkStripeCustomer({customer_id: stripeCustomerId, member_id: member.id}, options);
+                    }
+                } else if (row.complimentary_plan) {
+                    const products = [];
+                    if (row.import_tier) {
+                        products.push({id: importTierId});
+                    } else {
+                        products.push({id: defaultTier.id.toString()});
+                    }
+                    await this._members.update({products}, {...options, id: member.id});
+                } else if (row.import_tier) {
+                    throw new errors.DataImportError({message: tpl(messages.freeMemberNotAllowedImportTier)});
+                }
+
+                if (row.gift_id) {
+                    try {
+                        await this._gifts.reassignRedeemer(row.gift_id, member.id, {transacting: trx});
+                    } catch (giftError) {
+                        throw wrapGiftError(giftError);
+                    }
+                }
+
+                await trx.commit();
+                imported += 1;
+            } catch (error) {
+                const errorList: unknown[] = Array.isArray(error) ? error : [error];
+                const errorMessage = errorList
+                    .map(e => (typeof e === 'object' && e !== null && 'message' in e ? e.message : undefined))
+                    .join(', ');
+                await trx.rollback();
+                importErrors.push({...row, error: errorMessage});
+            }
+        }
+
+        await Promise.all(archivableStripePriceIds.map(id => this._stripe.archivePrice(id)));
+        metrics.metric('members-import', {imported, errors: importErrors.length, value: Date.now() - performStart});
+
+        if (imported > 0) {
+            // The import label exists now that members carry it, so fetch it to report.
+            // Guarded against a null lookup in case a row is ever counted without it.
+            const importLabelModel = await this._members.getImportLabel(labelName);
+            return {imported, errors: importErrors, importLabel: importLabelModel?.toJSON()};
+        }
+        return {imported: 0, errors: importErrors};
+    }
+}
+
+function wrapGiftError(error: unknown) {
+    const err = error as {message?: unknown};
+    const message = (err && typeof err.message === 'string' && err.message) || tpl(messages.giftReassignFailed);
+    return new errors.DataImportError({
+        message: `Member cannot be assigned to a gift: ${message}`
+    });
+}
+
+export default MembersCSVImporter;

package/core/server/services/members/import-export/import/reader.js

@@ -0,0 +1,32 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = readMemberRows;
+const csv_1 = require("../csv");
+const row_1 = require("./row");
+// Each accepted CSV header (the key) maps to the member field it fills (the value):
+// a "subscribed_to_emails" column fills the "subscribed" field, and so on.
+const FIELD_BY_HEADER = {
+    email: 'email',
+    name: 'name',
+    note: 'note',
+    subscribed_to_emails: 'subscribed',
+    created_at: 'created_at',
+    complimentary_plan: 'complimentary_plan',
+    stripe_customer_id: 'stripe_customer_id',
+    labels: 'labels',
+    import_tier: 'import_tier',
+    gift_id: 'gift_id'
+};
+// Resolve a caller mapping's target to the member field it fills. The target is usually
+// a field already, but a caller may name a default header (e.g. "subscribed_to_emails"),
+// which resolves to its field. hasOwn keeps a prototype name from matching a method.
+function toMemberField(target) {
+    return Object.hasOwn(FIELD_BY_HEADER, target) ? FIELD_BY_HEADER[target] : target;
+}
+async function readMemberRows(path, mapping) {
+    const headerToField = mapping
+        ? Object.fromEntries(Object.entries(mapping).map(([header, target]) => [header, toMemberField(target)]))
+        : FIELD_BY_HEADER;
+    const rows = await (0, csv_1.parse)(path, headerToField);
+    return rows.map(row => row_1.memberImportRowSchema.parse(row));
+}

package/core/server/services/members/import-export/import/reader.ts

@@ -0,0 +1,33 @@
+import {parse as parseCSV} from '../csv';
+import {memberImportRowSchema, type MemberImportRow} from './row';
+
+// Each accepted CSV header (the key) maps to the member field it fills (the value):
+// a "subscribed_to_emails" column fills the "subscribed" field, and so on.
+const FIELD_BY_HEADER: Record<string, string> = {
+    email: 'email',
+    name: 'name',
+    note: 'note',
+    subscribed_to_emails: 'subscribed',
+    created_at: 'created_at',
+    complimentary_plan: 'complimentary_plan',
+    stripe_customer_id: 'stripe_customer_id',
+    labels: 'labels',
+    import_tier: 'import_tier',
+    gift_id: 'gift_id'
+};
+
+// Resolve a caller mapping's target to the member field it fills. The target is usually
+// a field already, but a caller may name a default header (e.g. "subscribed_to_emails"),
+// which resolves to its field. hasOwn keeps a prototype name from matching a method.
+function toMemberField(target: string): string {
+    return Object.hasOwn(FIELD_BY_HEADER, target) ? FIELD_BY_HEADER[target] : target;
+}
+
+export default async function readMemberRows(path: string, mapping?: Record<string, string>): Promise<MemberImportRow[]> {
+    const headerToField: Record<string, string> = mapping
+        ? Object.fromEntries(Object.entries(mapping).map(([header, target]): [string, string] => [header, toMemberField(target)]))
+        : FIELD_BY_HEADER;
+
+    const rows = await parseCSV(path, headerToField);
+    return rows.map(row => memberImportRowSchema.parse(row));
+}

package/core/server/services/members/import-export/import/row.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.memberImportRowSchema = void 0;
+const zod_1 = require("zod");
+function splitLabels(cell) {
+    return cell ? cell.split(',').map(name => ({ name })) : [];
+}
+// An empty cell (or the literal 'undefined') reads as absent, not as a value -- so an
+// empty created_at is a missing date, not the invalid empty string.
+const optionalCell = zod_1.z.string()
+    .transform(cell => (cell === '' || cell === 'undefined' ? undefined : cell))
+    .optional();
+// Present cells are lenient in opposite directions: subscribed unless the cell reads
+// 'false', comped only when it reads 'true'. An absent column stays undefined (optional,
+// not defaulted), so a CSV lacking these columns leaves the member's state untouched.
+const isSubscribed = (cell) => cell.toLowerCase() !== 'false';
+const isComplimentary = (cell) => cell.toLowerCase() === 'true';
+// The member import vocabulary as a schema: the single source of truth that coerces the
+// raw string Row into typed member fields. Unknown columns (custom_fields.*, and anything
+// else a source carries) pass through via .loose() so nothing is lost before the kernel.
+exports.memberImportRowSchema = zod_1.z.object({
+    id: optionalCell,
+    email: optionalCell,
+    name: optionalCell,
+    note: optionalCell,
+    subscribed: zod_1.z.string().transform(isSubscribed).optional(),
+    complimentary_plan: zod_1.z.string().transform(isComplimentary).optional(),
+    stripe_customer_id: optionalCell,
+    created_at: optionalCell,
+    import_tier: optionalCell,
+    gift_id: optionalCell,
+    labels: zod_1.z.string().default('').transform(splitLabels)
+}).loose();

package/core/server/services/members/import-export/import/row.ts

@@ -0,0 +1,52 @@
+import {z} from 'zod';
+
+// Labels reach the import as a comma-separated cell and are split into objects here.
+export interface Label {
+    name: string;
+}
+
+function splitLabels(cell: string): Label[] {
+    return cell ? cell.split(',').map(name => ({name})) : [];
+}
+
+// An empty cell (or the literal 'undefined') reads as absent, not as a value -- so an
+// empty created_at is a missing date, not the invalid empty string.
+const optionalCell = z.string()
+    .transform(cell => (cell === '' || cell === 'undefined' ? undefined : cell))
+    .optional();
+
+// Present cells are lenient in opposite directions: subscribed unless the cell reads
+// 'false', comped only when it reads 'true'. An absent column stays undefined (optional,
+// not defaulted), so a CSV lacking these columns leaves the member's state untouched.
+const isSubscribed = (cell: string): boolean => cell.toLowerCase() !== 'false';
+const isComplimentary = (cell: string): boolean => cell.toLowerCase() === 'true';
+
+// The member import vocabulary as a schema: the single source of truth that coerces the
+// raw string Row into typed member fields. Unknown columns (custom_fields.*, and anything
+// else a source carries) pass through via .loose() so nothing is lost before the kernel.
+export const memberImportRowSchema = z.object({
+    id: optionalCell,
+    email: optionalCell,
+    name: optionalCell,
+    note: optionalCell,
+    subscribed: z.string().transform(isSubscribed).optional(),
+    complimentary_plan: z.string().transform(isComplimentary).optional(),
+    stripe_customer_id: optionalCell,
+    created_at: optionalCell,
+    import_tier: optionalCell,
+    gift_id: optionalCell,
+    labels: z.string().default('').transform(splitLabels)
+}).loose();
+
+export type MemberImportRow = z.infer<typeof memberImportRowSchema>;
+
+// A row that failed to import, carrying the raw message that stopped it. The message is
+// left raw here; turning it into human copy is a presentation concern.
+export type ImportErrorRow = MemberImportRow & {error: string};
+
+// The persisted import label as plain data -- taken off the Bookshelf model before it
+// leaves the kernel, so the result and the email carry only what they show.
+export interface ImportLabel {
+    name: string;
+    slug: string;
+}

package/core/server/services/members/import-export/import/spool.js

@@ -0,0 +1,30 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.createRowSpool = createRowSpool;
+const node_os_1 = __importDefault(require("node:os"));
+const node_path_1 = __importDefault(require("node:path"));
+const node_crypto_1 = __importDefault(require("node:crypto"));
+const fs_extra_1 = __importDefault(require("fs-extra"));
+// Spools import rows to a private JSON file under the OS temp dir, so a deferred
+// import can hand them to a background job and read them back after the request has
+// already returned. The rows go in and come out as MemberImportRow, so nothing but
+// the import's own row shape crosses this boundary.
+function createRowSpool() {
+    return {
+        async write(rows) {
+            const spoolPath = node_path_1.default.join(node_os_1.default.tmpdir(), `members-import-${node_crypto_1.default.randomUUID()}.json`);
+            await fs_extra_1.default.writeFile(spoolPath, JSON.stringify(rows), { mode: 0o600 });
+            return {
+                async read() {
+                    return JSON.parse(await fs_extra_1.default.readFile(spoolPath, 'utf8'));
+                },
+                async remove() {
+                    await fs_extra_1.default.remove(spoolPath).catch(() => { });
+                }
+            };
+        }
+    };
+}

package/core/server/services/members/import-export/import/spool.ts

@@ -0,0 +1,38 @@
+import os from 'node:os';
+import path from 'node:path';
+import crypto from 'node:crypto';
+import fs from 'fs-extra';
+import type {MemberImportRow} from './row';
+
+// A handle to rows written to a spool: the deferred job reads them back once it
+// runs, then removes the file. Removal swallows errors -- a failed cleanup must
+// never fail the import it was cleaning up after.
+export interface SpooledRows {
+    read(): Promise<MemberImportRow[]>;
+    remove(): Promise<void>;
+}
+
+export interface RowSpool {
+    write(rows: MemberImportRow[]): Promise<SpooledRows>;
+}
+
+// Spools import rows to a private JSON file under the OS temp dir, so a deferred
+// import can hand them to a background job and read them back after the request has
+// already returned. The rows go in and come out as MemberImportRow, so nothing but
+// the import's own row shape crosses this boundary.
+export function createRowSpool(): RowSpool {
+    return {
+        async write(rows) {
+            const spoolPath = path.join(os.tmpdir(), `members-import-${crypto.randomUUID()}.json`);
+            await fs.writeFile(spoolPath, JSON.stringify(rows), {mode: 0o600});
+            return {
+                async read() {
+                    return JSON.parse(await fs.readFile(spoolPath, 'utf8'));
+                },
+                async remove() {
+                    await fs.remove(spoolPath).catch(() => {});
+                }
+            };
+        }
+    };
+}