@porulle/plugin-channel-connector 0.52.0 → 0.53.0

package/dist/index.d.ts

@@ -5,8 +5,6 @@ export { ChannelConnectorService, HERO_IMAGE_BYTE_CAP, selectImportImages, CATAL
 export { isValidCatalogMappingFieldPath, matchFieldPath, mergeCatalogFieldMapping, normalizeCatalogFieldMapping, compareCatalogFieldMappingSpecificity, providerCatalogFieldMappingDefaults, selectCatalogFieldMapping, validateCatalogMappingRow, } from "./catalog-field-mapping.js";
 export type { CatalogDeferredMedia, CatalogMediaFailure, CatalogMediaFailureReason, CatalogPageConvergence, ImportImageSelection, } from "./service.js";
 export type { CatalogFieldMapping, CatalogFieldMappingInput, CatalogFieldMappingRow, CatalogFieldTarget, } from "./catalog-field-mapping.js";
-/** ~320 Neon HTTP subrequests per product against a 10,000 per-invocation cap → hard ceiling near 31; 20 leaves margin for heavier products. */
-export declare const CHANNEL_IMPORT_MAX_ITEMS_PER_INVOCATION = 20;
 /**
  * How many bounded batches one sweep may walk before it refuses rather than loops.
  *

package/dist/index.js

@@ -10,8 +10,6 @@ import { oauthStateEventId, signState, verifyState } from "./oauth-state.js";
 export { mockChannelConnector } from "./mock-connector.js";
 export { ChannelConnectorService, HERO_IMAGE_BYTE_CAP, selectImportImages, CATALOG_OUTBOUND_SUPPRESSION_WINDOW_MS, CATALOG_PUSH_BATCH_SIZES, CATALOG_PUSH_MAX_ATTEMPTS, canCatalogPushTransition, canExportTransition, catalogPushConcurrencyKey, catalogPushRetryDelayMs, CHANNEL_INVENTORY_MAX_ITEMS_PER_INVOCATION, isCatalogPushBreakerOpen, } from "./service.js";
 export { isValidCatalogMappingFieldPath, matchFieldPath, mergeCatalogFieldMapping, normalizeCatalogFieldMapping, compareCatalogFieldMappingSpecificity, providerCatalogFieldMappingDefaults, selectCatalogFieldMapping, validateCatalogMappingRow, } from "./catalog-field-mapping.js";
-/** ~320 Neon HTTP subrequests per product against a 10,000 per-invocation cap → hard ceiling near 31; 20 leaves margin for heavier products. */
-export const CHANNEL_IMPORT_MAX_ITEMS_PER_INVOCATION = 20;
 /**
  * How many bounded batches one sweep may walk before it refuses rather than loops.
  *
@@ -169,67 +167,6 @@ export function channelConnectorPlugin(options = {}) {
                 return { output: { enqueued: stores.length } };
             },
         },
-        {
-            slug: "channel/import-catalog",
-            concurrency: { key: (input) => String(input.storeId), supersedes: true },
-            durableSteps: true,
-            handler: async ({ input, ctx }) => {
-                const service = new ChannelConnectorService(ctx.db, ctx.services, options);
-                const orgId = String(input.orgId);
-                const storeId = String(input.storeId);
-                const result = await walkBatches(ctx, "import-catalog", storeId, async () => {
-                    const page = await service.importCatalog(orgId, storeId, createSystemActor(orgId), {
-                        maxItems: CHANNEL_IMPORT_MAX_ITEMS_PER_INVOCATION,
-                    });
-                    if (!page.ok)
-                        throw new Error(page.error);
-                    return {
-                        exhausted: page.value.exhausted,
-                        counted: page.value.imported,
-                        cursor: page.value.cursor ?? null,
-                        ...(page.value.warnings ? { warnings: page.value.warnings } : {}),
-                        // UNCONDITIONAL, unlike `warnings` and `failures` beside it. The step's return value is
-                        // the only per-batch seam a host application has — `walkBatches` keeps just `last` — so
-                        // this is where a page gets its identity. Omitting it when empty would make "this batch
-                        // committed nothing" and "this plugin build does not report entities" the same
-                        // `undefined` at the seam, and a caller that collapses those enqueues nothing and
-                        // reports success. The service's own return declares it unconditional for the same
-                        // reason; dropping it here would have undone that one line later.
-                        entityIds: page.value.entityIds,
-                        ...(page.value.failures ? { failures: page.value.failures } : {}),
-                    };
-                });
-                const jobs = ctx.services.jobs;
-                // The catalog is always exhausted by the time the walk above returns, so this hand-off is
-                // unconditional. It is the ONE enqueue this task is allowed: it starts a DIFFERENT task
-                // once, spending a single level of the request chain's 32, rather than one per page the
-                // way the continuation it replaced did.
-                //
-                // A finished catalog is not a usable one. `importCatalog` writes entities, variants and
-                // prices and never touches `inventory_levels`, so a store whose sweep ends here has a
-                // catalog in which every variant rolls up as out of stock — which is what a consumer
-                // projection publishes and what a shopper is shown.
-                //
-                // Inventory used to arrive from `reconcile`, reachable only through the
-                // `channel/reconcile-sweep` cron. A deployment that removes its crons therefore loses a
-                // data-plane write silently, with every suite still green. Levelling inventory here keeps
-                // it inside the one operator action — "import this store" — instead of behind a second
-                // one somebody has to remember.
-                await jobs.enqueue("channel/sync-inventory", { orgId, storeId }, {
-                    organizationId: orgId,
-                    concurrencyKey: storeId,
-                });
-                return {
-                    output: {
-                        imported: result.counted,
-                        cursor: result.last.cursor ?? null,
-                        exhausted: true,
-                        batches: result.batches,
-                        ...(result.warnings.length > 0 ? { warnings: result.warnings } : {}),
-                    },
-                };
-            },
-        },
         {
             slug: "channel/backfill-catalog",
             concurrency: { key: (input) => String(input.storeId) },

package/dist/service.d.ts

@@ -392,7 +392,6 @@ export declare class ChannelConnectorService {
         storeDomain: string;
         webhookSecret?: string;
     }): Promise<PluginResult<PublicConnectedStore>>;
-    private get optionsJobs();
     disconnectStore(orgId: string, id: string): Promise<PluginResult<PublicConnectedStore>>;
     disconnectStoreSystem(orgId: string, id: string, redactDomain?: boolean): Promise<PluginResult<PublicConnectedStore>>;
     getStore(orgId: string, id: string): Promise<PluginResult<PublicConnectedStore>>;

package/dist/service.js

@@ -2126,19 +2126,10 @@ export class ChannelConnectorService {
                 return PluginErr(registration.error.message, "CONNECTOR_REGISTRATION_FAILED");
             }
         }
-        const jobs = this.optionsJobs;
-        if (jobs) {
-            await jobs.enqueue("channel/import-catalog", { orgId, storeId: rows[0].id }, {
-                organizationId: orgId,
-                concurrencyKey: rows[0].id,
-                supersedes: true,
-            });
-        }
+        // Connecting starts no import. The host's operator route starts one (and levels inventory after
+        // it); connect used to enqueue a second, sequential walk that ran beside the host's own.
         return Ok(redactStore(store));
     }
-    get optionsJobs() {
-        return this.jobs;
-    }
     async disconnectStore(orgId, id) {
         return this.disconnectStoreSystem(orgId, id);
     }