@devrev/airsync-sdk 2.0.0-beta.10 → 2.0.0-beta.11

package/dist/http/client.d.ts

@@ -1,3 +1,4 @@
-declare const axiosClient: import("axios").AxiosInstance;
+import { AxiosInstance } from 'axios';
+declare const axiosClient: Pick<AxiosInstance, 'get' | 'post'>;
 export { axiosClient };
 //# sourceMappingURL=client.d.ts.map

package/dist/http/client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"client.d.ts","sourceRoot":"","sources":["../../src/http/client.ts"],"names":[],"mappings":"AAGA,QAAA,MAAM,WAAW,+BAEf,CAAC;AAmFH,OAAO,EAAE,WAAW,EAAE,CAAC"}
+{"version":3,"file":"client.d.ts","sourceRoot":"","sources":["../../src/http/client.ts"],"names":[],"mappings":"AAAA,OAAc,EAAc,aAAa,EAAE,MAAM,OAAO,CAAC;AAmGzD,QAAA,MAAM,WAAW,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,GAAG,MAAM,CAGpD,CAAC;AAEF,OAAO,EAAE,WAAW,EAAE,CAAC"}

package/dist/http/client.js

@@ -6,64 +6,75 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.axiosClient = void 0;
 const axios_1 = __importDefault(require("axios"));
 const axios_retry_1 = __importDefault(require("axios-retry"));
-const axiosClient = axios_1.default.create({
-    timeout: 30 * 1000,
-});
+function createAxiosClient() {
+    const client = axios_1.default.create({
+        timeout: 30 * 1000,
+    });
+    (0, axios_retry_1.default)(client, {
+        retries: 5,
+        shouldResetTimeout: true,
+        retryDelay: (retryCount, error) => {
+            var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
+            let delay;
+            if (((_a = error.response) === null || _a === void 0 ? void 0 : _a.status) === 429) {
+                const retryAfter = ((_c = (_b = error.response) === null || _b === void 0 ? void 0 : _b.headers) === null || _c === void 0 ? void 0 : _c['retry-after']) ||
+                    ((_e = (_d = error.response) === null || _d === void 0 ? void 0 : _d.headers) === null || _e === void 0 ? void 0 : _e['Retry-After']);
+                delay = parseInt(retryAfter, 10) * 1000;
+            }
+            else {
+                delay = axios_retry_1.default.exponentialDelay(retryCount, error, 1000);
+            }
+            const requestId = ((_g = (_f = error.config) === null || _f === void 0 ? void 0 : _f.headers) === null || _g === void 0 ? void 0 : _g['x-request-id']) ||
+                ((_j = (_h = error.config) === null || _h === void 0 ? void 0 : _h.headers) === null || _j === void 0 ? void 0 : _j['X-Request-ID']);
+            const delayInSeconds = Math.round(delay / 1000);
+            console.warn(`Retrying request to ${(_k = error.config) === null || _k === void 0 ? void 0 : _k.url} in ${delayInSeconds}s due to ${(_m = (_l = error.response) === null || _l === void 0 ? void 0 : _l.status) !== null && _m !== void 0 ? _m : 'unknown'} error.`, Object.assign(Object.assign({ method: (_o = error.config) === null || _o === void 0 ? void 0 : _o.method }, (retryCount && { retryCount })), (requestId && { requestId })));
+            return delay;
+        },
+        retryCondition: (error) => {
+            var _a, _b, _c, _d, _e, _f, _g;
+            const retryAfter = ((_b = (_a = error.response) === null || _a === void 0 ? void 0 : _a.headers) === null || _b === void 0 ? void 0 : _b['retry-after']) ||
+                ((_d = (_c = error.response) === null || _c === void 0 ? void 0 : _c.headers) === null || _d === void 0 ? void 0 : _d['Retry-After']);
+            if (((_e = error.response) === null || _e === void 0 ? void 0 : _e.status) && error.response.status >= 500) {
+                return true;
+            }
+            // 429 only when a valid non-negative Retry-After header is present
+            else if (((_f = error.response) === null || _f === void 0 ? void 0 : _f.status) &&
+                error.response.status === 429 &&
+                retryAfter &&
+                !isNaN(Number(retryAfter)) &&
+                Number(retryAfter) >= 0) {
+                return true;
+            }
+            // Network errors for idempotent requests; 429 already handled above
+            else if (axios_retry_1.default.isNetworkOrIdempotentRequestError(error) &&
+                ((_g = error.response) === null || _g === void 0 ? void 0 : _g.status) !== 429) {
+                return true;
+            }
+            // axios-retry excludes ECONNABORTED (client-side timeout only, never a
+            // server response) from isNetworkError, so handle it here separately.
+            else if (error.code === 'ECONNABORTED') {
+                return true;
+            }
+            else {
+                return false;
+            }
+        },
+        onMaxRetryTimesExceeded(error) {
+            var _a, _b, _c, _d, _e, _f;
+            (_b = (_a = error.config) === null || _a === void 0 ? void 0 : _a.headers) === null || _b === void 0 ? true : delete _b.authorization;
+            (_d = (_c = error.config) === null || _c === void 0 ? void 0 : _c.headers) === null || _d === void 0 ? true : delete _d.Authorization;
+            (_e = error.request) === null || _e === void 0 ? true : delete _e._header;
+            console.error(`Request to ${(_f = error.config) === null || _f === void 0 ? void 0 : _f.url} failed after max retries. Error`, error);
+        },
+    });
+    return client;
+}
+let instance;
+const getInstance = () => (instance !== null && instance !== void 0 ? instance : (instance = createAxiosClient()));
+// Created on first use so that loading the SDK never calls axios.create(),
+// which breaks connector tests that mock axios.
+const axiosClient = {
+    get: async (...args) => getInstance().get(...args),
+    post: async (...args) => getInstance().post(...args),
+};
 exports.axiosClient = axiosClient;
-(0, axios_retry_1.default)(axiosClient, {
-    retries: 5,
-    shouldResetTimeout: true,
-    retryDelay: (retryCount, error) => {
-        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l, _m, _o;
-        let delay;
-        if (((_a = error.response) === null || _a === void 0 ? void 0 : _a.status) === 429) {
-            const retryAfter = ((_c = (_b = error.response) === null || _b === void 0 ? void 0 : _b.headers) === null || _c === void 0 ? void 0 : _c['retry-after']) ||
-                ((_e = (_d = error.response) === null || _d === void 0 ? void 0 : _d.headers) === null || _e === void 0 ? void 0 : _e['Retry-After']);
-            delay = parseInt(retryAfter, 10) * 1000;
-        }
-        else {
-            delay = axios_retry_1.default.exponentialDelay(retryCount, error, 1000);
-        }
-        const requestId = ((_g = (_f = error.config) === null || _f === void 0 ? void 0 : _f.headers) === null || _g === void 0 ? void 0 : _g['x-request-id']) ||
-            ((_j = (_h = error.config) === null || _h === void 0 ? void 0 : _h.headers) === null || _j === void 0 ? void 0 : _j['X-Request-ID']);
-        const delayInSeconds = Math.round(delay / 1000);
-        console.warn(`Retrying request to ${(_k = error.config) === null || _k === void 0 ? void 0 : _k.url} in ${delayInSeconds}s due to ${(_m = (_l = error.response) === null || _l === void 0 ? void 0 : _l.status) !== null && _m !== void 0 ? _m : 'unknown'} error.`, Object.assign(Object.assign({ method: (_o = error.config) === null || _o === void 0 ? void 0 : _o.method }, (retryCount && { retryCount })), (requestId && { requestId })));
-        return delay;
-    },
-    retryCondition: (error) => {
-        var _a, _b, _c, _d, _e, _f, _g;
-        const retryAfter = ((_b = (_a = error.response) === null || _a === void 0 ? void 0 : _a.headers) === null || _b === void 0 ? void 0 : _b['retry-after']) ||
-            ((_d = (_c = error.response) === null || _c === void 0 ? void 0 : _c.headers) === null || _d === void 0 ? void 0 : _d['Retry-After']);
-        if (((_e = error.response) === null || _e === void 0 ? void 0 : _e.status) && error.response.status >= 500) {
-            return true;
-        }
-        // 429 only when a valid non-negative Retry-After header is present
-        else if (((_f = error.response) === null || _f === void 0 ? void 0 : _f.status) &&
-            error.response.status === 429 &&
-            retryAfter &&
-            !isNaN(Number(retryAfter)) &&
-            Number(retryAfter) >= 0) {
-            return true;
-        }
-        // Network errors for idempotent requests; 429 already handled above
-        else if (axios_retry_1.default.isNetworkOrIdempotentRequestError(error) &&
-            ((_g = error.response) === null || _g === void 0 ? void 0 : _g.status) !== 429) {
-            return true;
-        }
-        // axios-retry excludes ECONNABORTED (client-side timeout only, never a
-        // server response) from isNetworkError, so handle it here separately.
-        else if (error.code === 'ECONNABORTED') {
-            return true;
-        }
-        else {
-            return false;
-        }
-    },
-    onMaxRetryTimesExceeded(error) {
-        var _a, _b, _c, _d, _e, _f;
-        (_b = (_a = error.config) === null || _a === void 0 ? void 0 : _a.headers) === null || _b === void 0 ? true : delete _b.authorization;
-        (_d = (_c = error.config) === null || _c === void 0 ? void 0 : _c.headers) === null || _d === void 0 ? true : delete _d.Authorization;
-        (_e = error.request) === null || _e === void 0 ? true : delete _e._header;
-        console.error(`Request to ${(_f = error.config) === null || _f === void 0 ? void 0 : _f.url} failed after max retries. Error`, error);
-    },
-});

package/dist/state/extraction-state.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"extraction-state.d.ts","sourceRoot":"","sources":["../../src/state/extraction-state.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AACzC,OAAO,EAAsB,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAGxE,sGAAsG;AACtG,qBAAa,eAAe,CAAC,cAAc,CAAE,SAAQ,SAAS,CAAC,cAAc,CAAC;gBAChE,MAAM,EAAE,cAAc,CAAC,cAAc,CAAC;IAIlD;;;;OAIG;IACH,uBAAuB,IAAI,IAAI;CAyHhC;AAED,wBAAsB,qBAAqB,CAAC,cAAc,EAAE,EAC1D,KAAK,EACL,YAAY,EACZ,oBAAoB,EACpB,OAAO,GACR,EAAE,cAAc,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,eAAe,CAAC,cAAc,CAAC,CAAC,CAkB3E"}
+{"version":3,"file":"extraction-state.d.ts","sourceRoot":"","sources":["../../src/state/extraction-state.ts"],"names":[],"mappings":"AAOA,OAAO,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AACzC,OAAO,EAAsB,cAAc,EAAE,MAAM,oBAAoB,CAAC;AAGxE,sGAAsG;AACtG,qBAAa,eAAe,CAAC,cAAc,CAAE,SAAQ,SAAS,CAAC,cAAc,CAAC;gBAChE,MAAM,EAAE,cAAc,CAAC,cAAc,CAAC;IAIlD;;;;OAIG;IACH,uBAAuB,IAAI,IAAI;CA4HhC;AAED,wBAAsB,qBAAqB,CAAC,cAAc,EAAE,EAC1D,KAAK,EACL,YAAY,EACZ,oBAAoB,EACpB,OAAO,GACR,EAAE,cAAc,CAAC,cAAc,CAAC,GAAG,OAAO,CAAC,eAAe,CAAC,cAAc,CAAC,CAAC,CAkB3E"}

package/dist/state/extraction-state.js

@@ -21,6 +21,7 @@ class ExtractionState extends base_state_1.BaseState {
      * all other events reuse those cached boundaries. Validates extract_from < extract_to.
      */
     resolveExtractionWindow() {
+        var _a, _b;
         const sdkState = this.sdkState;
         const eventContext = this.event.payload.event_context;
         const overrideExtractFrom = (value) => {
@@ -29,10 +30,15 @@ class ExtractionState extends base_state_1.BaseState {
             }
             eventContext.extract_from = value;
         };
-        if (this.event.payload.event_type === extraction_1.EventType.StartExtractingData &&
-            !sdkState.lastSyncStarted) {
-            sdkState.lastSyncStarted = new Date().toISOString();
-            console.log(`Setting lastSyncStarted to ${sdkState.lastSyncStarted}.`);
+        if (this.event.payload.event_type === extraction_1.EventType.StartExtractingData) {
+            if (!sdkState.lastSyncStarted) {
+                sdkState.lastSyncStarted = new Date().toISOString();
+                console.log(`Setting lastSyncStarted to ${sdkState.lastSyncStarted}.`);
+            }
+            // A sync that failed before attachments must not leak its SSOR records into this one.
+            if ((_b = (_a = sdkState.toDevRev) === null || _a === void 0 ? void 0 : _a.deferredSsorArtifacts) === null || _b === void 0 ? void 0 : _b.length) {
+                sdkState.toDevRev.deferredSsorArtifacts = [];
+            }
         }
         if (this.event.payload.event_type === extraction_1.EventType.StartExtractingMetadata) {
             const timeFields = [

package/docs/MIGRATION.md

@@ -35,7 +35,9 @@ v2; templates are copied into new connectors, so stale comments spread.
 2. **Start from green latest v1.** If behind the latest stable 1.x, upgrade and
    apply the release notes of every release in between that mentions a
    migration (the last ones were 1.16.0–1.18.0), then get tsc/lint/build/test
-   green on v1. Record failures that remain as **Pre-existing**.
+   green on v1. If the upgrade alone breaks tsc with axios type mismatches,
+   bump the connector's `axios` to the version that SDK release depends on.
+   Record failures that remain as **Pre-existing**.
 3. **Install v2.**
    ```bash
    npm uninstall @devrev/ts-adaas
@@ -51,15 +53,19 @@ v2; templates are copied into new connectors, so stale comments spread.
 5. **Verify**: `npm install && npx tsc --noEmit && npm run lint && npm run build && npm test`.
    If tsconfig excludes folders the connector still runs (`test/`, a local
    runner), type-check them too: use the repo's `tsconfig.test.json` if it has
-   one, otherwise a temporary tsconfig that extends `tsconfig.json` with
-   `"include": ["src", "test"]` and `npx tsc --noEmit -p` it. Fix and rerun,
+   one, otherwise a temporary tsconfig that extends `tsconfig.json` and
+   overrides `"include": ["src", "test"]`, `"exclude": ["node_modules", "dist"]`,
+   `"rootDir": "."` and `"noEmit": true` (the base `exclude` and `rootDir` still
+   apply otherwise), then `npx tsc -p` it. Fix and rerun,
    at most 4 attempts; the migration must add no new failures compared to
    step 2.
 6. **Sweep.** This must return nothing except legitimate hits (an
    `EventEmitter.emit`, the connector's own `axiosClient` from §9,
-   `@devrev/typescript-sdk` in `package.json` from §1, `git config --local`):
+   `@devrev/typescript-sdk` in `package.json` from §1, `git config --local`,
+   legacy event names in fields the SDK doesn't read, such as
+   `event_type_adaas`):
    ```bash
-   grep -rnE "@devrev/ts-adaas|airsync-sdk/dist|\bprocessTask\b|WorkerAdapter|\.emit\(|hasWorkerEmitted|Airdrop(Event|Message)|axiosClient|(format|serialize)AxiosError|workerPath:|overridenOptions|skipConfirmation|isLocalDevelopment|--local|-- local\b|@devrev/typescript-sdk|(Extractor|Loader)EventType|ExtractionCommonError|ERROR_CODE=|sdkState|onUpload|uploadedArtifacts|currentEventDataLength|EXTRACTION_(DATA|METADATA|ATTACHMENTS|EXTERNAL_SYNC_UNITS)_" \
+   grep -rnE "@devrev/ts-adaas|airsync-sdk/dist|\bprocessTask\b|\bWorkerAdapter\b|\.emit\(|hasWorkerEmitted|\bAirdrop(Event|Message)\b|axiosClient|(format|serialize)AxiosError|workerPath:|overridenOptions|skipConfirmation|isLocalDevelopment|--local|-- local\b|@devrev/typescript-sdk|(Extractor|Loader)EventType|ExtractionCommonError|ERROR_CODE=|sdkState|onUpload|uploadedArtifacts|currentEventDataLength|EXTRACTION_(DATA|METADATA|ATTACHMENTS|EXTERNAL_SYNC_UNITS)_" \
      --include='*.ts' --include='*.js' --include='*.json' --include='*.yml' --include='*.yaml' --include='*.md' --include='*.mdc' --include='*.sh' . \
      | grep -vE 'node_modules|/dist/|/coverage/|package-lock.json|MIGRATION_REVIEW.md'
    ```
@@ -80,7 +86,7 @@ v2; templates are copied into new connectors, so stale comments spread.
    helpers (§4), paths that emitted nothing (§4), `onTimeout` bodies kept (§4),
    delay values that look like milliseconds or headers (§4), the cursor decision
    (§8), ESU repo rewrites (§7), mapper error handling (§10), and a v1 baseline
-   upgrade.
+   upgrade. Record the exact installed versions.
    ```markdown
    # v2 Migration Review
    Migrated `@devrev/ts-adaas` 1.20.2 → `@devrev/airsync-sdk` 2.0.0.
@@ -134,7 +140,10 @@ The shapes are unchanged. `AirSyncEvent.context` now also declares `user_id`,
 `payload.event_context`). Delete a hand-rolled context type that added these
 (e.g. `interface CustomAirdropEvent extends AirdropEvent { context: CustomContext }`):
 a narrower `context` no longer satisfies `AirSyncEvent`. Keep only the fields
-the SDK type lacks.
+the SDK type lacks. These fields are required, so typed mock events must set
+them. `execution_metadata` is typed `{ devrev_endpoint }`; a custom type that
+widened it with `@devrev/typescript-sdk` fields goes too, along with mock values
+for fields nothing reads.
 
 Deprecated `EventContext` aliases are removed. Use the current names:
 
@@ -147,7 +156,10 @@ Deprecated `EventContext` aliases are removed. Use the current names:
 | `sync_run`, `sync_run_id` | `run_id` |
 | `uuid` | `request_id_adaas` |
 
-Rename these keys in fixture and test `event_context` objects too. Untyped
+Rename these keys in fixture and test `event_context` objects too. When an
+object already has the new key (or has two old keys for one new one, like
+`dev_user` and `dev_user_id`), keep the existing new-name value, otherwise the
+first old one, and drop the rest. Untyped
 JSON still compiles, but the SDK reads only the current names (e.g. state
 requests carry `request_id_adaas`).
 
@@ -246,14 +258,14 @@ processExtractionTask({
 |----|----|
 | `emit(XxxDone)` | `return { status: 'success' }` |
 | `emit(XxxProgress, { progress })` | `return { status: 'progress' }` |
-| `emit(XxxDelayed, { delay })` | `return { status: 'delay', delaySeconds: delay }` — keep the same number; the platform always read it as seconds. If it is clearly milliseconds or a raw `Retry-After` header, keep the behavior (`Number(header)`) and flag it in the review |
+| `emit(XxxDelayed, { delay })` | `return { status: 'delay', delaySeconds: delay }` — keep the same number; the platform always read it as seconds. A raw `Retry-After` header stays `Number(header)`. If it is clearly milliseconds (a `*_MS` constant, a value in the thousands, "ms" in the message), keep the number, flag it and list it under **Unresolved**: v1 waited that many seconds too, and fixing it changes behavior |
 | `emit(XxxDelayed)` with no `delay` | `delaySeconds` is required; use the connector's existing default retry constant, else `0` (what v1 effectively sent), and flag it |
 | `emit(XxxError, { error })` | `return { status: 'error', error }` — `error` is required; fall back to `{ message: '<phase> failed' }` |
 | `emit(XxxDone, { error })` (finished with warnings) | `return { status: 'success' }` and log the error. Returning `error` would fail the phase |
 | `emit(ExternalSyncUnitExtractionDone, { external_sync_units })` | push to a repo, then `success` (§7) |
 | `emit(MetadataExtractionDone, { pre_extraction_item_counts })` | `adapter.preExtractionItemCounts = counts`, then `success` |
 | a path that **emitted nothing** (a `catch` that only logs, a branch that falls through) | `{ status: 'error', error: { message } }`, or rethrow. In v1 the SDK reported these as a phase error ("Worker exited without emitting"); never turn them into `success` or `progress` |
-| any `emit(XxxDone)` after `loadItemTypes` / `loadAttachments` / `streamAttachments`, guarded (`!adapter.hasWorkerEmitted && !adapter.isTimeout`) or not | `return adapter.loadItemTypes(...)` (or the other call). Never discard their result and return `success`: v1 emitted their `delay` / `error` itself. Drop the guard and any module-level "already emitted" flags |
+| any `emit(XxxDone)` after `loadItemTypes` / `loadAttachments` / `streamAttachments`, guarded (`!adapter.hasWorkerEmitted && !adapter.isTimeout`) or not | `return adapter.loadItemTypes(...)` (or the other call). Never discard their result and return `success`: v1 emitted their `delay` / `error` itself. Drop the guard and any module-level "already emitted" flags. If v1 did more work after the call and then emitted `Done` with an `error`, return the call's result and log that error |
 
 In non-resumable phases (external sync units, state deletion), `progress` and
 `delay` become `*Error`.
@@ -282,7 +294,16 @@ If a helper **emitted and then kept going** (no throw), the caller often
 emitted again later, e.g. `DataExtractionDone` after the loop, and v1 dropped
 every emit after the first. Record the first result the same way, stop the
 work that followed it (return out of the loop or extractor), and return it
-from the task. Unawaited emits inside loops count too.
+from the task. Unawaited emits inside loops count too. When the emitting helper
+is deep in the call stack, throw an error class that carries the `TaskResult`
+instead (`class TaskResultError extends Error { constructor(readonly result: TaskResult) { super(); } }`),
+stop the extractor, and return `e.result` from the task's outermost `catch`.
+A free-function handler that must keep only the first result stores it per
+adapter (`WeakMap<adapter, TaskResult>`) and the task reads it back.
+
+A helper that returned `{ delay?, error? }` and emitted only when one was set
+needs an explicit result when neither is set: `success` if the caller carried
+on, otherwise `error`.
 
 If a helper (often the HTTP client's 429 handler) **emitted and then threw a
 sentinel error** that call sites rethrow, keep the throw, drop the emit, and map
@@ -321,6 +342,10 @@ helpers, assert on the return value.
     custom error. Make its body safe to run after a completed task. A custom
     "Lambda timeout" `onTimeout` on a task that emits on every path is
     deleted; v1 never ran it.
+  - `onTimeout` runs only after the task has returned, in v1 and v2, so
+    cleanup there (cancelling a rate limiter, aborting requests) never
+    interrupted a stuck task. Move it into the task's `isTimeout` /
+    `timeoutSignal` handling if it matters, otherwise delete it.
 - Don't call `process.exit()` or `postState()` in `onTimeout`; the SDK saves
   state and exits.
 
@@ -355,7 +380,9 @@ async function extractList(adapter: ExtractionAdapter<State>) { ... }
 | `event`, `state`, `mappers`, `postState`, `uploadArtifact`, `downloadArtifact`, `isTimeout`, `extractionScope` | both |
 
 - The adapters and `Repo` are **type-only**: `new ExtractionAdapter(...)` and
-  `class X extends LoadingAdapter<S>` don't compile; `interface X extends …`
+  `class X extends LoadingAdapter<S>` or `extends Repo` (test doubles that
+  subclassed `Repo` through `dist/`) don't compile; use a plain object with
+  the methods the code calls. `interface X extends …`
   does. SDK internals (`emit`, `sdkState`, `options`, constructors, …) aren't
   public. `isTimeout` and `artifacts` are read-only.
 - `adapter.mappers` works in extraction as in v1; keep those calls and only
@@ -423,7 +450,9 @@ return { status: 'success' }; // was a bare `return;`
   as `skip` are dropped; every `success` creates or updates a mapper record.
 - Shared helpers typed as these responses change too. Callers that read
   `result.delay` or interpolate `${result.error}` (now an object) switch on
-  `result.status`.
+  `result.status`. A helper that only builds delays and errors returns the
+  narrowed type (`Extract<…, { status: 'delay' | 'error' }>`), so callers and
+  tests can read `delaySeconds` and `error` without a check.
 - An error `code` returned from the stream callback is only logged; the pool
   skips that attachment.
 - `httpStream` is typed `HttpStreamResponse` (`{ data, headers }`); an axios
@@ -439,6 +468,14 @@ its state fields and tests. Workarounds typically override the
 `ssor_attachment` repo's `onUpload`, splice `uploadedArtifacts`, adjust
 `currentEventDataLength`, or stash artifact refs in state.
 
+- A worker whose only job was re-injecting those refs returns
+  `{ status: 'success' }` (or its remaining work); keep its phase wired.
+- A connector's own `{ itemType: 'ssor_attachment' }` repo entry can stay.
+- Tests that mocked the deep `Uploader` use adapter doubles instead:
+  `{ uploadArtifact: jest.fn(), downloadArtifact: jest.fn() }`.
+- If the workaround stashed refs in connector state, syncs between data done and
+  attachments done lose them after the upgrade; drain them first.
+
 ## 7. External sync units go through a repo
 
 `EventData.external_sync_units` is removed. Push them to a repo:
@@ -479,8 +516,13 @@ the emit.
 **`lastSyncStarted` / `lastSuccessfulSyncStarted`: rename or repoint.**
 Getting this wrong compiles but silently breaks incremental sync.
 
-- **Not in the connector's `State` interface** (it came from the SDK): delete
-  writes, and repoint reads to `extract_from`.
+These rules cover **top-level** state keys only; the v1 migration moves only
+those. A cursor nested in the connector's own object (`state.extractor.lastSuccessfulSyncStarted`)
+is connector-owned: leave it alone.
+
+- **Not in the connector's `State` interface**, or declared but never written
+  by the connector (it came from the SDK): delete writes, and repoint reads to
+  `extract_from`.
 - **Declared and used as the connector's own cursor** (derived from data, or
   read alongside `extract_from`): **rename** it (e.g. →
   `lastSuccessfulWindowStart`) everywhere, including fixtures. Otherwise the v1
@@ -490,6 +532,14 @@ Getting this wrong compiles but silently breaks incremental sync.
   it is read next to `extract_from`**. Renaming would leave it empty after the
   upgrade, because the v1 value moves into SDK state.
 
+After repointing, branches that chose between the two
+(`reset_extract_from ? extract_from : lastSuccessfulSyncStarted`) read the same
+value; collapse them and drop the `reset_extract_from` read. If the connector
+already used `extract_from` for something else (e.g. a skip filter) and the
+cursor for a second window (e.g. "re-emit and bump the floor"), repointing
+merges the two, so code below the window may become dead. Flag that low
+confidence.
+
 `extract_from` equals the last successful sync start only when the platform
 sends no time window. If a repointed read feeds logic beyond "skip items older
 than this" (e.g. distinguishing first syncs), flag it as low confidence; there
@@ -514,7 +564,7 @@ import axios from 'axios';
 import axiosRetry from 'axios-retry';
 
 export const axiosClient = axios.create();
-axiosRetry(axiosClient, {
+axiosRetry(axiosClient as Parameters<typeof axiosRetry>[0], {
   retries: 5,
   retryDelay: (retryCount, error) => axiosRetry.exponentialDelay(retryCount, error, 1000),
   retryCondition: (error) =>
@@ -528,6 +578,9 @@ axiosRetry(axiosClient, {
 });
 ```
 
+The cast is needed under ts-jest, which can resolve axios's `.d.ts` and
+`.d.cts` to different types.
+
 A deep import of the SDK's internal client (`dist/http/axios-client-internal`)
 relied on more: a 30 s timeout, retries of 429s that send `Retry-After` (waiting
 that long) and of `ECONNABORTED`. Reproduce what the connector depends on with
@@ -674,7 +727,9 @@ change.
 - For a deleted `onTimeout`, drop the tests of its body and assert it's
   `undefined`. A factory like `processExtractionTask: (_config) => jest.fn()`
   discards the config; use `processExtractionTask: jest.fn()` and read
-  `mocked.mock.calls[0][0].onTimeout`. Tests of a deleted SSOR workaround go with it.
+  `mocked.mock.calls[0][0].onTimeout`. Capture it in a module-level `const`
+  right after importing the worker: `jest.clearAllMocks()` in a `beforeEach`
+  wipes the calls. Tests of a deleted SSOR workaround go with it.
 
 ## 14. Smaller changes
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devrev/airsync-sdk",
-  "version": "2.0.0-beta.10",
+  "version": "2.0.0-beta.11",
   "description": "Typescript SDK for building AirSync snap-ins on the DevRev platform.",
   "type": "commonjs",
   "main": "./dist/index.js",