pixivflow 3.4.0 → 3.4.2

package/dist/commands/SchedulerCommand.js

@@ -12,6 +12,7 @@ const ScheduleTriggerServer_1 = require("../scheduler/ScheduleTriggerServer");
 const SlotCoordinator_1 = require("../scheduler/SlotCoordinator");
 const ManualJobAdmission_1 = require("../scheduler/ManualJobAdmission");
 const ManualJobService_1 = require("../scheduler/ManualJobService");
+const JobCancellation_1 = require("../scheduler/JobCancellation");
 const ManualRefetchAdapter_1 = require("../scheduler/ManualRefetchAdapter");
 const schedules_1 = require("../scheduler/schedules");
 const targetRoutes_1 = require("../delivery/targetRoutes");
@@ -100,6 +101,13 @@ class SchedulerCommand extends Command_1.BaseCommand {
                     database: runtime.database,
                     config: resolveConfig,
                     admission,
+                    // A consumer cancel must bite the WORK, not only the ledger: the ledger
+                    // write already happened inside `cancelJob`, so this only stops the
+                    // in-flight download loop (and the `Scheduler.running` it holds, which
+                    // otherwise refuses every later admission for the same schedule).
+                    onCancel: (slotId) => {
+                        runtime.cancelSlot(slotId, JobCancellation_1.CANCELLED_BY_CONSUMER_MESSAGE);
+                    },
                 });
                 // Durable dispatch. The trigger endpoint records the occurrence FIRST,
                 // then hands it to the shared scheduler and answers immediately. It must

package/dist/commands/scheduler-runtime.d.ts

@@ -62,8 +62,14 @@ export declare function withDeliveryMode<T extends {
  *  - `shutdown`: the process is going away. The Slot is deliberately left
  *    non-terminal so recovery resumes the same occurrence after restart; no
  *    worker survives to duplicate it.
+ *  - `consumer`: an operator/consumer cancelled THIS job through the protocol
+ *    `POST /jobs/{job_id}/cancel`. The ledger already took the Slot terminal in
+ *    one transaction (`cancelConsumerJob`, reason `cancelled_by_consumer`), so
+ *    the abort path must not roll it up a second time — doing so would overwrite
+ *    the cancel verdict with a generic failure and flip the protocol status from
+ *    `cancelled` back to `failed`.
  */
-export type CancelOrigin = 'timeout' | 'shutdown';
+export type CancelOrigin = 'timeout' | 'shutdown' | 'consumer';
 /**
  * Decide a scheduled run's Slot fate when `runAllTargets()` aborts abnormally
  * instead of reporting per-target failures.
@@ -90,6 +96,21 @@ export interface SchedulerRuntime {
      * (`shutdown`: leave the Slot non-terminal so recovery resumes it).
      */
     cancelActive(reason: string, origin?: CancelOrigin): void;
+    /**
+     * Cancel the in-flight run **only if it is the one owning `slotId`**.
+     *
+     * This is what makes a protocol consumer cancel bite: `cancelConsumerJob`
+     * terminalises the ledger in one transaction, but the work itself is an
+     * in-process download loop that knows nothing about the ledger. Without this
+     * call the cancelled job keeps consuming until it finishes on its own, and the
+     * holding `Scheduler.running` keeps refusing every later admission for that
+     * schedule (`scheduler_busy`) for the whole remaining run.
+     *
+     * Returns true when this call cancelled that Slot's live run. A slot another
+     * run owns (or a job that is only queued) is untouched — its ledger terminal is
+     * all there is to do.
+     */
+    cancelSlot(slotId: string, reason: string): boolean;
     /**
      * The active run never settled after its timeout and drain window. Stop
      * renewing its lease and take its Slot terminal so recovery cannot re-dispatch

package/dist/commands/scheduler-runtime.js

@@ -98,6 +98,11 @@ function shouldTerminaliseAbortedSlot(origin, slotAbandoned) {
     // only overwrite it when the wedged job finally settles.
     if (slotAbandoned)
         return false;
+    // A consumer cancel is already terminal in the ledger (`cancelled_by_consumer`,
+    // written by cancelConsumerJob in one transaction). Finishing it again would
+    // replace that verdict with a generic failure.
+    if (origin === 'consumer')
+        return false;
     // Shutdown is not a failure: recovery is meant to resume this occurrence.
     return origin !== 'shutdown';
 }
@@ -291,6 +296,12 @@ async function createSchedulerRuntime(configPathArg) {
      * `null` while no cancellation has been requested for the current run.
      */
     let activeAbortOrigin = null;
+    /**
+     * The Slot the in-flight run owns, if any. Set where the lease is claimed and
+     * cleared where it is released, so `cancelSlot()` can tell "this job is the one
+     * running right now" from "this job is merely queued behind another run".
+     */
+    let activeSlotId = null;
     // Independently-pumped durable outbox (content + notifications). Started in
     // the long-running scheduler daemon; run-once drains explicitly before exit.
     const deliveryDispatcher = new DeliveryDispatcher_1.DeliveryDispatcher(config.delivery, buildProxyUrl(config.network));
@@ -452,6 +463,7 @@ async function createSchedulerRuntime(configPathArg) {
             // This run owns the Slot now: any cancellation recorded against a previous
             // run must not decide how THIS run's abort is handled.
             activeAbortOrigin = null;
+            activeSlotId = activeSlot.slotId;
             let cancelled = false;
             const heartbeat = setInterval(() => {
                 // A cancelled/timed-out run must stop renewing its lease. An infinitely
@@ -468,6 +480,7 @@ async function createSchedulerRuntime(configPathArg) {
             varReleaseLease = () => {
                 clearInterval(heartbeat);
                 activeLeaseHooks = null;
+                activeSlotId = null;
                 coordinator.releaseRunLease(activeSlot.slotId, runOwner);
             };
             activeLeaseHooks = {
@@ -497,6 +510,7 @@ async function createSchedulerRuntime(configPathArg) {
                     database.slots.markSlotStatus(activeSlot.slotId, 'failed', `abandoned after scheduler timeout; no delivery (${reason})`);
                     coordinator.releaseRunLease(activeSlot.slotId, runOwner);
                     activeLeaseHooks = null;
+                    activeSlotId = null;
                 },
             };
         }
@@ -778,6 +792,15 @@ async function createSchedulerRuntime(configPathArg) {
         // or left recoverable is decided by `origin` in the abort path of runJob.
         activeLeaseHooks?.stopHeartbeat();
     };
+    const cancelSlot = (slotId, reason) => {
+        // A cancellation is only meaningful for the run that owns THIS Slot. Every
+        // other case (queued behind another run, or already terminal) has nothing to
+        // abort: the durable ledger write is the whole effect.
+        if (!activeSlotId || activeSlotId !== slotId)
+            return false;
+        cancelActive(reason, 'consumer');
+        return true;
+    };
     const abandonActiveRun = (reason) => {
         // Cancellation did not take effect inside the drain window — a request that
         // ignores the abort. This run will never settle, so finish the lease
@@ -800,6 +823,7 @@ async function createSchedulerRuntime(configPathArg) {
         tokenMaintenance,
         runJob,
         cancelActive,
+        cancelSlot,
         abandonActiveRun,
         activeExecutionCount: () => activeExecutions,
         startOutboxWorker: () => outboxWorker.start(),

package/dist/config/types.d.ts

@@ -945,10 +945,19 @@ export interface StandaloneConfig {
          * classified. 'skip' (default) is safe mode: never ship an unclassifiable
          * cover, and log `coverType=unknown` so a future Pixiv format change is
          * visible instead of silently leaking designs again. 'keep' prefers
-         * availability over certainty. A failed probe always keeps the cover.
+         * availability over certainty.
+         *
+         * `probeFailed` governs the separate case where the cover probe itself
+         * failed (network / auth / rate limit) and the bytes were never seen.
+         * 'skip' (default) is safe mode for the same reason: most novel covers ARE
+         * generated designs, so keeping an unseen cover re-ships what the 640x900
+         * classifier exists to suppress; the cost is one illustration when a real
+         * cover is transiently unreachable. 'keep' restores availability-first.
+         * Either way the `coverType=probe_failed` warning stays visible.
          */
         novelCover?: {
             unknown?: 'skip' | 'keep';
+            probeFailed?: 'skip' | 'keep';
         };
     };
 }

package/dist/config/validation.js

@@ -715,6 +715,11 @@ function validateConfig(config, location, databasePath) {
             config.download.novelCover.unknown !== 'keep') {
             errors.push(`download.novelCover.unknown: Must be "skip" or "keep" (got ${String(config.download.novelCover.unknown)})`);
         }
+        if (config.download.novelCover?.probeFailed !== undefined &&
+            config.download.novelCover.probeFailed !== 'skip' &&
+            config.download.novelCover.probeFailed !== 'keep') {
+            errors.push(`download.novelCover.probeFailed: Must be "skip" or "keep" (got ${String(config.download.novelCover.probeFailed)})`);
+        }
     }
     // The bounded candidate scan is what stops a page full of duplicates from
     // burning a whole scheduled slot, so a non-integer value is reported here.

package/dist/delivery/DeliveryService.d.ts

@@ -47,6 +47,13 @@ export interface RefetchOutcomePayload {
     requestId: string;
     /** 'no_alternative' | 'failed' (replacement success rides the submission). */
     disposition: 'no_alternative' | 'failed';
+    /**
+     * The cross-boundary, closed-vocabulary protocol error code (protocol/v1
+     * `$defs/Error.code`). The consumer writes it into its `failure_code` CODE
+     * column, so raw upstream text must never appear here.
+     */
+    reasonCode?: string;
+    /** Bounded, single-line, human-readable business message (never a raw body). */
     reason?: string;
     workId?: string;
     /** Bounded candidate-scan bookkeeping for diagnostics (spec-compatible). */

package/dist/delivery/HttpMultipartDelivery.js

@@ -173,6 +173,11 @@ class HttpMultipartDelivery {
             ? {
                 request_id: outcome.requestId,
                 disposition: outcome.disposition,
+                // The closed-vocabulary code goes to TelePost's `failure_code` CODE
+                // column; `reason` is only the bounded human business message it
+                // sanitizes into `terminal_reason`. Raw upstream text (an nginx 502
+                // HTML page) must never appear on the wire.
+                reason_code: outcome.reasonCode,
                 reason: outcome.reason,
                 work_id: outcome.workId,
                 scanned: outcome.scanned,

package/dist/delivery/types.d.ts

@@ -154,6 +154,12 @@ export interface DeliveryNotificationRequest {
     refetchOutcome?: {
         requestId: string;
         disposition: 'no_alternative' | 'failed';
+        /**
+         * The cross-boundary, closed-vocabulary protocol error code (protocol/v1
+         * `$defs/Error.code`); raw upstream text never crosses the boundary.
+         */
+        reasonCode?: string;
+        /** Bounded, single-line, human-readable business message (never a raw body). */
         reason?: string;
         workId?: string;
         scanned?: number;

package/dist/domain/media/NovelCoverPolicy.d.ts

@@ -15,14 +15,36 @@ export interface NovelCoverPolicy {
      * 'keep': prefer availability over certainty.
      */
     unknownCover: 'skip' | 'keep';
+    /**
+     * What to do when the cover probe itself failed (network / auth / rate
+     * limit), so the bytes were never fetched and no content type exists.
+     *
+     * 'skip' (default): safe mode — most novel covers ARE Pixiv's generated
+     * designs, so keeping an unseen probe failure re-ships exactly the covers
+     * the 640x900 classifier exists to suppress. The cost is one illustration
+     * when a real author cover happens to be transiently unreachable.
+     * 'keep': prefer availability over certainty — a failed probe never costs a
+     * cover, at the price of re-shipping generated designs whenever the CDN or
+     * the account is unreachable.
+     */
+    probeFailed: 'skip' | 'keep';
 }
-/** Production-safe default: an unclassifiable cover is never shipped. */
+/**
+ * Production-safe default: an unclassifiable cover is never shipped, and a
+ * probe that never produced bytes is not treated as evidence that the cover is
+ * real content.
+ */
 export declare const DEFAULT_NOVEL_COVER_POLICY: NovelCoverPolicy;
 /**
  * Classify fetched cover bytes. Returns `unknown` when the image header cannot
  * be read (no decoder in this project, so only the header is inspected).
  */
 export declare function classifyNovelCover(cover: ArrayBuffer | Uint8Array | null | undefined): NovelCoverType;
-/** The delivery decision for one classified cover under a policy. */
-export declare function coverDeliveryDecision(policy: NovelCoverPolicy, type: NovelCoverType): 'deliver' | 'skip';
+/**
+ * Everything `coverDeliveryDecision` can be asked about: the three results
+ * `classifyNovelCover` returns, plus the probe that never produced bytes.
+ */
+export type NovelCoverOutcome = NovelCoverType | 'probe_failed';
+/** The delivery decision for one cover outcome under a policy. */
+export declare function coverDeliveryDecision(policy: NovelCoverPolicy, outcome: NovelCoverOutcome): 'deliver' | 'skip';
 //# sourceMappingURL=NovelCoverPolicy.d.ts.map

package/dist/domain/media/NovelCoverPolicy.js

@@ -31,8 +31,15 @@ exports.coverDeliveryDecision = coverDeliveryDecision;
  * shipping generated covers again.
  *
  * A FAILED probe (network / auth / rate limit) is deliberately NOT `unknown`:
- * it is `probe_failed` and always keeps the cover, because a transient fetch
- * error must never cost a real one.
+ * it is `probe_failed` — the bytes were never seen, so no content type exists.
+ * It is a policy decision like `unknown` (`download.novelCover.probeFailed`),
+ * and the production-safe default is `skip`: the majority of novel covers ARE
+ * Pixiv's generated designs, so a probe failure that keeps the cover re-ships
+ * exactly the payloads the 640x900 classifier exists to suppress. `keep` stays
+ * available for an operator who prefers availability over certainty (a
+ * transient fetch error must never cost a real cover). Either way the loud
+ * `coverType=probe_failed` log line stays, so the decision is observable
+ * instead of silent.
  */
 const imageDimensions_1 = require("../../utils/imageDimensions");
 /**
@@ -43,8 +50,15 @@ const imageDimensions_1 = require("../../utils/imageDimensions");
  */
 exports.PIXIV_GENERATED_COVER_WIDTH = 640;
 exports.PIXIV_GENERATED_COVER_HEIGHT = 900;
-/** Production-safe default: an unclassifiable cover is never shipped. */
-exports.DEFAULT_NOVEL_COVER_POLICY = { unknownCover: 'skip' };
+/**
+ * Production-safe default: an unclassifiable cover is never shipped, and a
+ * probe that never produced bytes is not treated as evidence that the cover is
+ * real content.
+ */
+exports.DEFAULT_NOVEL_COVER_POLICY = {
+    unknownCover: 'skip',
+    probeFailed: 'skip',
+};
 /**
  * Classify fetched cover bytes. Returns `unknown` when the image header cannot
  * be read (no decoder in this project, so only the header is inspected).
@@ -59,12 +73,14 @@ function classifyNovelCover(cover) {
     }
     return 'custom';
 }
-/** The delivery decision for one classified cover under a policy. */
-function coverDeliveryDecision(policy, type) {
-    if (type === 'pixiv_generated')
+/** The delivery decision for one cover outcome under a policy. */
+function coverDeliveryDecision(policy, outcome) {
+    if (outcome === 'pixiv_generated')
         return 'skip';
-    if (type === 'unknown')
+    if (outcome === 'unknown')
         return policy.unknownCover === 'keep' ? 'deliver' : 'skip';
+    if (outcome === 'probe_failed')
+        return policy.probeFailed === 'keep' ? 'deliver' : 'skip';
     return 'deliver';
 }
 //# sourceMappingURL=NovelCoverPolicy.js.map

package/dist/download/DownloadManager.js

@@ -132,6 +132,7 @@ class DownloadManager {
             : MaterializationPolicy_1.DEFAULT_MATERIALIZATION_POLICY;
         const novelCoverPolicy = {
             unknownCover: config.download?.novelCover?.unknown ?? NovelCoverPolicy_1.DEFAULT_NOVEL_COVER_POLICY.unknownCover,
+            probeFailed: config.download?.novelCover?.probeFailed ?? NovelCoverPolicy_1.DEFAULT_NOVEL_COVER_POLICY.probeFailed,
         };
         this.novelDownloader = new NovelDownloader_1.NovelDownloader(client, database, fileService, database, undefined, materialization, novelCoverPolicy);
         this.planner = new DownloadPlanner_1.DownloadPlanner(database, {

package/dist/download/NovelDownloader.d.ts

@@ -34,8 +34,11 @@ export declare class NovelDownloader {
      *  - unknown         → policy decision, default 'skip' (safe mode), so a future
      *                      Pixiv cover-format change surfaces as a loud
      *                      `coverType=unknown` log instead of leaking silently
-     * A FAILED probe (network / auth / rate limit) is a separate case and always
-     * keeps the cover: a transient fetch error must never cost a real one.
+     *  - probe_failed    → policy decision, default 'skip': the bytes were never
+     *                      seen, and most novel covers are generated designs, so
+     *                      keeping an unseen cover re-ships what this classifier
+     *                      exists to suppress. `download.novelCover.probeFailed:
+     *                      keep` restores the availability-first behaviour.
      */
     private resolveCoverUrl;
 }

package/dist/download/NovelDownloader.js

@@ -418,8 +418,11 @@ class NovelDownloader {
      *  - unknown         → policy decision, default 'skip' (safe mode), so a future
      *                      Pixiv cover-format change surfaces as a loud
      *                      `coverType=unknown` log instead of leaking silently
-     * A FAILED probe (network / auth / rate limit) is a separate case and always
-     * keeps the cover: a transient fetch error must never cost a real one.
+     *  - probe_failed    → policy decision, default 'skip': the bytes were never
+     *                      seen, and most novel covers are generated designs, so
+     *                      keeping an unseen cover re-ships what this classifier
+     *                      exists to suppress. `download.novelCover.probeFailed:
+     *                      keep` restores the availability-first behaviour.
      */
     async resolveCoverUrl(novelId, coverUrl) {
         const normalized = (0, novelCover_1.normalizeNovelCoverUrl)(coverUrl);
@@ -430,13 +433,29 @@ class NovelDownloader {
             cover = await this.client.downloadImage(normalized);
         }
         catch (error) {
-            logger_1.logger.warn(`Novel ${novelId} cover probe failed; keeping the cover (coverType=probe_failed)`, {
+            const reason = error instanceof Error ? error.message : String(error);
+            // A failed probe is a policy decision, not an automatic keep: most novel
+            // covers ARE Pixiv's generated designs, so silently shipping an unseen
+            // cover re-ships exactly what the 640x900 classifier exists to suppress.
+            const probeDecision = (0, NovelCoverPolicy_1.coverDeliveryDecision)(this.novelCoverPolicy, 'probe_failed');
+            if (probeDecision === 'deliver') {
+                logger_1.logger.warn(`Novel ${novelId} cover probe failed; keeping the cover per novelCover.probeFailed=keep (coverType=probe_failed)`, {
+                    novelId,
+                    coverUrl: normalized,
+                    coverType: 'probe_failed',
+                    policy: this.novelCoverPolicy.probeFailed,
+                    reason,
+                });
+                return normalized;
+            }
+            logger_1.logger.warn(`Novel ${novelId} cover probe failed; skipping the cover per novelCover.probeFailed=skip (coverType=probe_failed)`, {
                 novelId,
                 coverUrl: normalized,
                 coverType: 'probe_failed',
-                reason: error instanceof Error ? error.message : String(error),
+                policy: this.novelCoverPolicy.probeFailed,
+                reason,
             });
-            return normalized;
+            return null;
         }
         const coverType = (0, NovelCoverPolicy_1.classifyNovelCover)(cover);
         const decision = (0, NovelCoverPolicy_1.coverDeliveryDecision)(this.novelCoverPolicy, coverType);

package/dist/notification/NotificationPolicy.js

@@ -4,6 +4,7 @@ exports.NotificationPolicy = void 0;
 const DeliveryService_1 = require("../delivery/DeliveryService");
 const targetRoutes_1 = require("../delivery/targetRoutes");
 const TargetOutcome_1 = require("../scheduler/TargetOutcome");
+const ProtocolErrors_1 = require("../scheduler/ProtocolErrors");
 const SlotCoordinator_1 = require("../scheduler/SlotCoordinator");
 const logger_1 = require("../logger");
 /**
@@ -227,6 +228,20 @@ class NotificationPolicy {
         const name = this.targetName(target);
         if (!name)
             return;
+        // A consumer/operator cancel is NOT a refetch failure (finding D). The
+        // cancelled cell keeps its terminal `cancelled_by_consumer` verdict, and no
+        // `disposition:"failed"` outcome may ever be emitted for it — the operator
+        // deliberately stopped the work, and the cancellation is already surfaced
+        // through the job status / events channel, never as a failed refetch report.
+        const cancelledCell = target.id ? this.database.slots.getCell(slot.slotId, target.id) : null;
+        if (cancelledCell?.terminalReasonCode === ProtocolErrors_1.CANCELLED_BY_CONSUMER) {
+            logger_1.logger.info('Refetch outcome suppressed: job was cancelled by the consumer', {
+                slot: slot.slotId,
+                target: target.id,
+                requestId,
+            });
+            return;
+        }
         const deliveryTarget = this.config.delivery?.targets?.[name];
         if (!deliveryTarget || deliveryTarget.type !== 'httpMultipart')
             return;
@@ -239,12 +254,22 @@ class NotificationPolicy {
             });
             return;
         }
+        // Translate the internal outcome ONCE, through the existing classifier:
+        // the wire carries the closed-vocabulary protocol code plus the short
+        // Chinese business message. The raw technical text (`outcome.error` /
+        // `outcome.reason`) never crosses the boundary — a live acceptance run
+        // shipped a 220-char multi-line nginx 502 HTML page into TelePost's
+        // `failure_code` CODE column that way.
+        const normalized = (0, TargetOutcome_1.terminalReasonFor)(outcome);
+        const reasonCode = (0, TargetOutcome_1.protocolErrorCodeForTerminalReason)(normalized?.code) ?? undefined;
+        const reason = (normalized?.message ?? '').trim() || undefined;
         let payload;
         if (outcome.kind === 'no_candidate' || outcome.kind === 'duplicate') {
             payload = {
                 requestId,
                 disposition: 'no_alternative',
-                reason: outcome.reason,
+                reasonCode,
+                reason,
                 workId: outcome.kind === 'duplicate' ? outcome.workId : undefined,
                 ...scanCounts(outcome.scan),
             };
@@ -253,7 +278,8 @@ class NotificationPolicy {
             payload = {
                 requestId,
                 disposition: 'failed',
-                reason: outcome.error,
+                reasonCode,
+                reason,
                 ...scanCounts(outcome.scan),
             };
         }

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "type": "commonjs",
   "name": "pixivflow",
-  "version": "3.4.0",
+  "version": "3.4.2",
   "private": true
 }

package/dist/scheduler/ManualJobService.d.ts

@@ -25,6 +25,16 @@ export interface ManualJobServiceDeps {
     config(): StandaloneConfig;
     /** The shared admission path — the same instance the legacy shim uses. */
     admission: ManualJobAdmission;
+    /**
+     * Ask the runtime to abort the in-flight run when the cancelled job is the one
+     * executing right now.
+     *
+     * The ledger write is the source of truth and happens first; this only stops
+     * the in-process download loop from running to completion after it has been
+     * cancelled. A job that is merely queued behind another run has no live run to
+     * abort, and the hook is then never called.
+     */
+    onCancel?(slotId: string): void;
     /** Injected clock, for tests. */
     now?(): number;
 }

package/dist/scheduler/ManualJobService.js

@@ -78,10 +78,15 @@ class ManualJobService {
      * that terminalises the work and stops further deliveries.
      */
     cancelJob(jobId) {
-        (0, JobCancellation_1.cancelConsumerJob)(this.deps.database, jobId, this.now());
+        const result = (0, JobCancellation_1.cancelConsumerJob)(this.deps.database, jobId, this.now());
         // The cancellation is now durable, so its terminal event must be too — a
         // consumer that polls after cancelling must never see an unterminated stream.
         this.events.reconcile(jobId);
+        // Only a cancel that actually transitioned the ledger has a live run worth
+        // aborting. The ledger write came first, so the abort path cannot resurrect
+        // the job; it only makes the running download loop stop consuming.
+        if (result.cancelled)
+            this.deps.onCancel?.(jobId);
         return this.viewBySlotId(jobId);
     }
     viewBySlotId(slotId) {

package/dist/scheduler/SlotCoordinator.js

@@ -245,6 +245,13 @@ class SlotCoordinator {
         const cell = this.database.slots.getCell(slotId, targetId);
         if (!cell)
             return;
+        // A consumer/operator cancel is FINAL (see JobCancellation.ts). The abort of
+        // the interrupted run still lands here through the normal per-target outcome
+        // path; a late `failed` must never re-verdict a cancelled cell, or the
+        // protocol status flips from `cancelled` back to `failed` ~10s after the
+        // operator's cancel (finding D). Cancel verdicts are never re-written.
+        if (cell.terminalReasonCode === ProtocolErrors_1.CANCELLED_BY_CONSUMER)
+            return;
         switch (outcome.kind) {
             case 'submitted':
                 this.database.slots.lockCellWork(slotId, targetId, outcome.workId, outcome.workType);

package/dist/version.js

@@ -2,5 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BUILD = void 0;
 // GENERATED by scripts/write-version.js — do not edit manually.
-exports.BUILD = { version: '3.4.0', commit: '5d231179a9b3' };
+exports.BUILD = { version: '3.4.2', commit: '4db0bf21a785' };
 //# sourceMappingURL=version.js.map

package/dist/webui/package.json

@@ -1,6 +1,6 @@
 {
   "type": "commonjs",
   "name": "pixivflow-webui-backend",
-  "version": "3.4.0",
+  "version": "3.4.2",
   "description": "PixivFlow WebUI Backend - CommonJS module"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pixivflow",
-  "version": "3.4.0",
+  "version": "3.4.2",
   "description": "🎨 Pixiv 下载、筛选与自动收集工具 - 批量下载插画和小说、按标签/热度/日期筛选、定时任务与可靠 HTTP 交付 | Pixiv downloader and automation toolkit with filtering, scheduling and reliable HTTP delivery",
   "repository": {
     "type": "git",