@ak--47/dungeon-master 1.8.0 → 1.8.1

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
@@ -0,0 +1,153 @@
1
+ # 1.8.1 Upgrade Guide
2
+
3
+ 1.8.1 repairs funnel counting, lifecycle timing, and emitted identity. Existing
4
+ configurations and call forms remain supported. No property renames or required
5
+ configuration changes are introduced. `applySessionShape` adds optional dataset
6
+ bounds; its existing unbounded calls retain their behavior.
7
+
8
+ Generated timestamps, identities, and report counts can change from 1.8.0 because
9
+ the previous output contained defects. Recheck saved story expectations rather
10
+ than expecting byte-identical output across versions.
11
+
12
+ ## Update the dependency
13
+
14
+ After 1.8.1 is published:
15
+
16
+ ```sh
17
+ npm install @ak--47/dungeon-master@1.8.1
18
+ ```
19
+
20
+ No additional runtime dependency is required. The package still requires Node.js
21
+ 20.20.0 or newer. This guide does not indicate that npm publication has occurred.
22
+
23
+ ## Funnel counts can change
24
+
25
+ - An ordered funnel whose first and last steps match now finalizes its conversion
26
+ correctly. With reentry enabled, the closing event can also start the next
27
+ attempt. Without reentry, it still completes and does not create another attempt.
28
+ - Ordinary completed histories consume events through the inclusive two-second
29
+ completion grace period. Those events do not become a second attempt. The
30
+ existing `graceperiod: false` option disables that wait.
31
+ - Hold-property-constant session windows use session boundaries from the full
32
+ stream before property filtering. Unrelated activity can keep a session open.
33
+ - First/last-touch segmentation merges properties across reached steps. A missing
34
+ property on the preferred step can fall back to another reached step. Explicit
35
+ step selection remains separate.
36
+
37
+ `countMode: 'totals'` still defaults to `reentry: false`. Specify `reentry: true`
38
+ when repeated histories are intended. This compatibility default differs from
39
+ Mixpanel general totals. Do not change existing report options implicitly when
40
+ comparing old and new results.
41
+
42
+ ## Lifecycle and identity output is corrected
43
+
44
+ Retention entry follows adjusted creation, retries precede the final onboarding
45
+ attempt, and usage follows onboarding completion. The engine reconciles its own
46
+ identity fields against surviving auth events after filtering. Engine-created
47
+ data-quality duplicates and world-event clones preserve identity provenance.
48
+
49
+ Explicit hook identity overrides and the synthetic experiment identity exception
50
+ remain supported. A later valid event carrying both IDs can provide identity
51
+ mapping evidence; this is not restricted to the first funnel.
52
+
53
+ Short windows retain partial output instead of introducing a required exception.
54
+ Inspect the existing `result.warnings` collection for:
55
+
56
+ | warning key | meaning |
57
+ | --- | --- |
58
+ | `lifecycle.firstFunnelClipped` | Configured timing cannot fit before dataset end; future steps are omitted without compressing TTC. |
59
+ | `lifecycle.emptyPreAuthAttempt` | Auth is the first step, so a failed pre-auth attempt emits no rows. |
60
+ | `lifecycle.strictAttemptBudget` | The strict event budget cannot retain every surviving attempt entry. |
61
+
62
+ Configured suppression and hook filtering remain authoritative. The engine does
63
+ not recreate intentionally removed events to satisfy an attempt count.
64
+
65
+ ## Bound session reshaping when the dataset end matters
66
+
67
+ Calls without bounds retain legacy full-UTC-day placement. They cannot infer the
68
+ dataset end from the last observed event and may still move events beyond it.
69
+ Pass known bounds to prevent that clipping:
70
+
71
+ ```js
72
+ import { applySessionShape } from '@ak--47/dungeon-master/hook-helpers';
73
+
74
+ const hook = (records, type, meta) => {
75
+ if (type === 'everything') {
76
+ applySessionShape(records, meta.profile.distinct_id, {
77
+ sessionsPerWeek: 3,
78
+ eventsPerSession: 5,
79
+ sessionMinutes: 10,
80
+ datasetStart: meta.datasetStart,
81
+ datasetEnd: meta.datasetEnd,
82
+ });
83
+ }
84
+ return records;
85
+ };
86
+ ```
87
+
88
+ Bounds accept ISO strings, Unix seconds, or Unix milliseconds. Either explicit
89
+ bound enables constrained placement; an omitted side uses the original UTC day
90
+ edge. Invalid bounds or insufficient per-week session capacity throw `RangeError`
91
+ before records are mutated. Two same-day sessions separated by more than thirty
92
+ minutes cannot fit in a twenty-minute interval. Adjust the requested cadence or
93
+ available window rather than swallowing that error.
94
+
95
+ The helper retimes the same records. It does not add or drop events. Unbounded
96
+ legacy requests remain nonthrowing, including overfull layouts where derived
97
+ sessions can merge.
98
+
99
+ ## Path injection remains append-only
100
+
101
+ `applyPathBias` selects the earliest chronological anchor and leaves original
102
+ traffic intact. `share: 1` selects all eligible users for injection. Competing
103
+ events can still interrupt the immediate branch, so it does not promise a 100%
104
+ Flows branch share. Recheck the observed branch, not only the injected count.
105
+
106
+ ## Revalidate the story, not just the schema
107
+
108
+ Run a pinned, seeded representative dungeon with ordinary standalone traffic and
109
+ competing funnels. Compare counts using explicit report settings. For deterministic
110
+ same-version comparisons, use `concurrency: 1` and strip only `insert_id`.
111
+
112
+ The new alignment checks establish selected source-derived Mixpanel contracts.
113
+ They do not execute Mixpanel's engine. Timezone/DST variants, list-valued HPC,
114
+ project-specific session exclusions, all parameter combinations, and arbitrary
115
+ hooks are not covered by a universal parity claim.
116
+
117
+ The recorded sweep produced 17.08 million events across 594 dungeons within ten
118
+ minutes. Its largest single dungeon had 281,751 events. Of 297 cells, 125 met their
119
+ evidence criteria and 172 had insufficient eligible populations. Increasing total
120
+ events is not a substitute for enough eligible users or converters.
121
+
122
+ ## Repository test workflows
123
+
124
+ These commands require a repository checkout with development dependencies.
125
+ The npm package ships this guide, but does not ship the test suites.
126
+
127
+ ```sh
128
+ node tests/alignment/run.mjs
129
+ node tests/alignment/run.mjs --sweep --timeout-ms=600000
130
+ ```
131
+
132
+ Both commands use macOS OS-level network denial and a hard maximum of ten minutes.
133
+ They fail closed on unsupported platforms. The editor's Testing panel is a separate
134
+ workflow: one serial editor config discovers unit, integration, E2E, and alignment
135
+ tests without global pruning. Direct-run engine scripts have opt-in tasks; their
136
+ full-sweep E2E wrapper is visible but skipped by default. Editor runs do not provide
137
+ an OS network sandbox. Use the named offline test tasks when network denial is
138
+ required, especially for E2E tests that can perform external operations.
139
+
140
+ `npm test` retains the default unit/integration/E2E suite and excludes alignment.
141
+ Its global setup prunes local `data` and `tmp`; preserve artifacts before using it.
142
+ The explicit `prune` task and default dungeon-run task retain their existing cleanup
143
+ behavior. None of the new offline test tasks depend on them.
144
+
145
+ ## Before publishing
146
+
147
+ Confirm package version and changelog are 1.8.1. Inspect `npm pack --dry-run --json`
148
+ and rerun the relevant release checks. Publishing to npm is a separate operator
149
+ action; neither test execution nor a Git commit publishes the package.
150
+
151
+ See the [changelog](../../CHANGELOG.md), [hook reference](../../HOOKS.md), and
152
+ [1.8.0 guide](1.8.0-upgrade-guide.md) for the unchanged standalone and warehouse
153
+ metric APIs.
@@ -17,6 +17,8 @@ import { dataLogger as logger } from "../utils/logger.js";
17
17
  // Keys that must never be nulled by data quality gremlins
18
18
  const NULL_EXEMPT_KEYS = new Set(['event', 'time', 'insert_id', 'user_id', 'device_id', 'distinct_id', '_drop', '_anomaly', '_persona']);
19
19
 
20
+ export const engineIdentity = Symbol('engineIdentity');
21
+
20
22
 
21
23
  /**
22
24
  * Creates a Mixpanel event with a flat shape
@@ -293,6 +295,9 @@ export async function makeEvent(
293
295
 
294
296
  eventTemplate.insert_id = randomUUID();
295
297
 
298
+ const originalIdentity = { user_id: eventTemplate.user_id, device_id: eventTemplate.device_id };
299
+ Object.defineProperty(eventTemplate, engineIdentity, { value: originalIdentity, configurable: true });
300
+
296
301
  // Call hook if configured (hooks override everything — they are the final authority)
297
302
  const { hook } = config;
298
303
  if (hook) {
@@ -305,6 +310,7 @@ export async function makeEvent(
305
310
  });
306
311
  // If hook returns a modified event, use it; otherwise use original
307
312
  if (hookedEvent && typeof hookedEvent === 'object') {
313
+ Object.defineProperty(hookedEvent, engineIdentity, { value: originalIdentity, configurable: true });
308
314
  return hookedEvent;
309
315
  }
310
316
  }
@@ -331,6 +331,21 @@ export async function makeFunnel(context, funnel, user, firstEventTime, profile
331
331
  dataQuality: featureCtx.dataQuality || context.config.dataQuality || null,
332
332
  latestTime: Number.isFinite(featureCtx.latestTime) ? featureCtx.latestTime : undefined,
333
333
  };
334
+ if (attemptInfo.isFirstFunnel && attemptInfo.isBorn && funnelEventsWithTiming.length) {
335
+ const spanMs = Math.max(0, ...funnelEventsWithTiming.map(event => event.relativeTimeMs || 0));
336
+ const latestStart = Math.min(funnelFeatureCtx.latestTime ?? context.FIXED_NOW, context.FIXED_NOW - spanMs / 1000);
337
+ if (firstEventTime > latestStart) {
338
+ context.addWarning({
339
+ key: 'lifecycle.firstFunnelClipped',
340
+ requested: spanMs,
341
+ applied: Math.max(0, (context.FIXED_NOW - firstEventTime) * 1000),
342
+ reason: 'First-funnel timing exceeds the remaining dataset window; emit the in-window prefix and omit future steps without compressing configured TTC.',
343
+ severity: 'warn',
344
+ });
345
+ }
346
+ funnelFeatureCtx.latestTime = Math.max(firstEventTime, latestStart);
347
+ if (featureCtx.pinFirstTime) funnelFeatureCtx.fixedTimeMs = firstEventTime * 1000;
348
+ }
334
349
 
335
350
  // Pre-compute per-step stamping modes for execution order. For isFirstFunnel + isBorn
336
351
  // runs, the first event in execution order whose config has `isAuthEvent: true` is
@@ -114,6 +114,9 @@ export function applyLifecycleWave(events, uid, opts) {
114
114
  * For users where `hashFloat(uid) < share`, injects the `path` sequence —
115
115
  * each step cloned from the user's OWN existing event of that name — right
116
116
  * after the user's FIRST `anchor` occurrence, with tight monotonic gaps.
117
+ * This is append-only: original traffic is never moved or removed. Events
118
+ * already between the anchor and clones can interrupt the immediate branch.
119
+ * `share` selects injection recipients, not the measured Flows branch share.
117
120
  *
118
121
  * Why these rules (HOOKS.md §2.17): Flows' unique mode reads only the FIRST
119
122
  * flow per user, so the injection anchors on the first occurrence; gaps are
@@ -192,12 +195,23 @@ export function applyPathBias(events, uid, opts) {
192
195
  * Boundary guarantees (what makes derived sessions deterministic against
193
196
  * jitter — HOOKS.md §2.13): intra-session gaps stay well under Mixpanel's
194
197
  * 30-min timeout (even spacing capped at 20min + bounded jitter, worst case
195
- * <28min); inter-session gaps stay well over it (sessions land on distinct
196
- * days when possible; same-day sessions are centered in equal partitions of
197
- * the day, spaced ≥¼ partition guaranteed >30min up to 8 sessions/day);
198
+ * <28min); with explicit bounds, inter-session gaps stay strictly over it (sessions land on distinct
199
+ * days when possible; same-day slots reserve at least 30min + 1ms between
200
+ * clusters, compressing their spans when necessary);
198
201
  * and no engineered session crosses UTC midnight (a day-boundary split would
199
202
  * cut it — `session_query.cpp` daySplit). Valid precisely because of P2.1:
200
203
  * session_ids are re-derived after the everything hook.
204
+ * With both bounds omitted, placement keeps the legacy full-UTC-day slots
205
+ * between the user's first and last active days. Overfull legacy requests
206
+ * still run, but their clusters can merge under the 30-minute timeout.
207
+ * Optional bounds are additive: supply known dataset limits to constrain
208
+ * placement; the helper cannot infer datasetEnd from a user's last event.
209
+ * An omitted side defaults to the corresponding full UTC day edge. Partial
210
+ * days compress clusters, including zero-span clusters. Only explicit-bound
211
+ * mode throws for insufficient per-week capacity, before any record changes.
212
+ * The helper never silently reduces the cluster target or drops events;
213
+ * legacy full-day placement can exceed an unknown dataset end and be clipped
214
+ * later by the engine. Pass metadata bounds to prevent that clipping.
201
215
  *
202
216
  * @param {Array<Object>} events - Full user event array (everything hook).
203
217
  * @param {string} uid - Unused for hashing here; kept for atom-signature
@@ -207,10 +221,15 @@ export function applyPathBias(events, uid, opts) {
207
221
  * @param {number} opts.sessionsPerWeek - Target clusters per week (≥1).
208
222
  * @param {number} opts.eventsPerSession - Target events per cluster (≥1).
209
223
  * @param {number} opts.sessionMinutes - Max cluster span in minutes.
224
+ * @param {string|number} [opts.datasetStart] - Inclusive lower bound (ISO,
225
+ * unix seconds, or unix milliseconds); accepts meta.datasetStart directly.
226
+ * @param {string|number} [opts.datasetEnd] - Inclusive upper bound (same units);
227
+ * accepts meta.datasetEnd directly.
228
+ * @throws {RangeError} Invalid explicit bounds or insufficient bounded 30-minute-session capacity.
210
229
  * @returns {Array<Object>} The SAME array, timestamps rewritten.
211
230
  */
212
231
  export function applySessionShape(events, uid, opts) {
213
- const { sessionsPerWeek, eventsPerSession, sessionMinutes } = opts || {};
232
+ const { sessionsPerWeek, eventsPerSession, sessionMinutes, datasetStart, datasetEnd } = opts || {};
214
233
  if (!Array.isArray(events) || !events.length) return events;
215
234
  if (!isPos(sessionsPerWeek) || !isPos(eventsPerSession) || !isPos(sessionMinutes)) return events;
216
235
 
@@ -221,6 +240,13 @@ export function applySessionShape(events, uid, opts) {
221
240
  const N = timed.length;
222
241
  const firstMs = toMs(timed[0].time);
223
242
  const lastMs = toMs(timed[N - 1].time);
243
+ const bounded = datasetStart !== undefined || datasetEnd !== undefined;
244
+ const lowerMs = datasetStart === undefined ? Math.floor(firstMs / DAY_MS) * DAY_MS : toMs(datasetStart);
245
+ const upperMs = datasetEnd === undefined ? (Math.floor(lastMs / DAY_MS) + 1) * DAY_MS - 1 : toMs(datasetEnd);
246
+ if (!Number.isFinite(lowerMs) || !Number.isFinite(upperMs) || lowerMs > upperMs ||
247
+ Math.abs(lowerMs) > 8.64e15 || Math.abs(upperMs) > 8.64e15) {
248
+ throw new RangeError('applySessionShape: invalid dataset bounds');
249
+ }
224
250
  const weeks = Math.max(1, Math.ceil((lastMs - firstMs + 1) / WEEK_MS));
225
251
  const numSessions = Math.max(1, Math.min(
226
252
  Math.floor(sessionsPerWeek) * weeks,
@@ -235,6 +261,15 @@ export function applySessionShape(events, uid, opts) {
235
261
  const chance = getChance();
236
262
  const firstDay = Math.floor(firstMs / DAY_MS);
237
263
  const lastDay = Math.floor(lastMs / DAY_MS);
264
+ const separationMs = 30 * MIN_MS + 1;
265
+ const perDayCount = new Map();
266
+ const dayBounds = day => [Math.ceil(Math.max(day * DAY_MS, lowerMs)),
267
+ Math.floor(Math.min((day + 1) * DAY_MS - 1, upperMs))];
268
+ const remainingCapacity = day => {
269
+ if (!bounded) return Infinity;
270
+ const [start, end] = dayBounds(day);
271
+ return (end >= start ? Math.floor((end - start) / separationMs) + 1 : 0) - (perDayCount.get(day) || 0);
272
+ };
238
273
 
239
274
  // Pick one day per session, week by week: prefer the user's original
240
275
  // active days in the week, fill from the rest of the week's days, and
@@ -251,22 +286,38 @@ export function applySessionShape(events, uid, opts) {
251
286
  const t = toMs(ev.time);
252
287
  if (t >= weekStartMs && t < weekStartMs + WEEK_MS) originalDays.add(Math.floor(t / DAY_MS));
253
288
  }
254
- const pool = [...originalDays].filter(d => d >= dayLo && d <= dayHi);
289
+ const pool = [...originalDays].filter(d => d >= dayLo && d <= dayHi && remainingCapacity(d) > 0);
255
290
  const others = [];
256
- for (let d = dayLo; d <= dayHi; d++) if (!originalDays.has(d)) others.push(d);
291
+ for (let d = dayLo; d <= dayHi; d++) if (!originalDays.has(d) && remainingCapacity(d) > 0) others.push(d);
257
292
  const picked = chance.pickset(pool, Math.min(need, pool.length));
258
293
  if (picked.length < need) picked.push(...chance.pickset(others, Math.min(need - picked.length, others.length)));
259
- let i = 0;
260
- while (picked.length < need) picked.push(picked[i++ % Math.max(1, picked.length)]); // reuse days: > days/week sessions
294
+ for (const day of picked) perDayCount.set(day, (perDayCount.get(day) || 0) + 1);
295
+ const candidates = [...pool, ...others];
296
+ while (picked.length < need) {
297
+ if (!bounded) {
298
+ let reuseIndex = 0;
299
+ while (picked.length < need) {
300
+ const day = picked[reuseIndex++ % Math.max(1, picked.length)];
301
+ picked.push(day);
302
+ perDayCount.set(day, (perDayCount.get(day) || 0) + 1);
303
+ }
304
+ break;
305
+ }
306
+ const available = candidates.filter(day => remainingCapacity(day) > 0);
307
+ if (!available.length) {
308
+ throw new RangeError(`applySessionShape: insufficient session capacity in week ${w + 1} for ${need} clusters`);
309
+ }
310
+ for (const day of available) {
311
+ picked.push(day);
312
+ perDayCount.set(day, (perDayCount.get(day) || 0) + 1);
313
+ if (picked.length === need) break;
314
+ }
315
+ }
261
316
  picked.sort((a, b) => a - b);
262
317
  sessionDays.push(...picked);
263
318
  }
264
319
 
265
- // Place sessions within days. Same-day sessions get equal partitions of
266
- // the day; the session is centered in its partition with bounded jitter,
267
- // which guarantees the inter-session and midnight invariants above.
268
- const perDayCount = new Map();
269
- for (const d of sessionDays) perDayCount.set(d, (perDayCount.get(d) || 0) + 1);
320
+ // Bound each day's slots and reserve the strict timeout gap between them.
270
321
  const perDaySeen = new Map();
271
322
  const sesMs = sessionMinutes * MIN_MS;
272
323
 
@@ -280,10 +331,15 @@ export function applySessionShape(events, uid, opts) {
280
331
  const j = perDaySeen.get(day) || 0;
281
332
  perDaySeen.set(day, j + 1);
282
333
 
283
- const seg = DAY_MS / m;
284
- const span = Math.min(sesMs, seg * 0.5);
285
- const center = day * DAY_MS + j * seg + (seg - span) / 2;
286
- const q = Math.floor((seg - span) / 4);
334
+ const [dayStart, dayEnd] = dayBounds(day);
335
+ const seg = (dayEnd - dayStart - (m - 1) * separationMs) / m;
336
+ const slotStart = Math.floor(dayStart + j * (seg + separationMs));
337
+ const slotEnd = Math.floor(dayStart + (j + 1) * seg + j * separationMs);
338
+ const slotWidth = bounded ? slotEnd - slotStart : DAY_MS / m;
339
+ const span = bounded ? Math.floor(Math.min(sesMs, slotWidth * 0.5)) : Math.min(sesMs, slotWidth * 0.5);
340
+ const center = bounded ? slotStart + Math.floor((slotWidth - span) / 2)
341
+ : day * DAY_MS + j * slotWidth + (slotWidth - span) / 2;
342
+ const q = Math.floor((slotWidth - span) / 4);
287
343
  const start = center + (q > 0 ? chance.integer({ min: -q, max: q }) : 0);
288
344
 
289
345
  if (size === 1) {
@@ -11,7 +11,7 @@ import pLimit from 'p-limit';
11
11
  import os from 'os';
12
12
  import * as u from "../utils/utils.js";
13
13
  import * as t from 'ak-tools';
14
- import { makeEvent } from "../generators/events.js";
14
+ import { makeEvent, engineIdentity } from "../generators/events.js";
15
15
  import { makeFunnel } from "../generators/funnels.js";
16
16
  import { makeUserProfile } from "../generators/profiles.js";
17
17
  import { makeSCD } from "../generators/scd.js";
@@ -403,6 +403,8 @@ export async function userLoop(context) {
403
403
  }
404
404
 
405
405
  let userFirstEventTime;
406
+ let userUsageStartTime = context.FIXED_BEGIN;
407
+ const promisedAttemptEntries = new Set();
406
408
 
407
409
  // ── v1.5 Active-day scheduling ──
408
410
  // When `avgActiveDaysPerUser` is set, build a per-user day plan: a list
@@ -493,13 +495,9 @@ export async function userLoop(context) {
493
495
  // Active-day mode: anchor the first funnel on a picked day so the user's
494
496
  // signup lands within their planned active window. Legacy mode: anchor at
495
497
  // adjustedCreated minus a noise offset.
496
- let cursor;
497
- if (dayPlan) {
498
- const bounds = nextDayBounds();
499
- cursor = bounds ? bounds.earliest : adjustedCreated.subtract(noise(), 'seconds').unix();
500
- } else {
501
- cursor = adjustedCreated.subtract(noise(), 'seconds').unix();
502
- }
498
+ const lifecycleStart = Math.max(adjustedCreated.valueOf() / 1000, context.FIXED_BEGIN);
499
+ const firstDayBounds = dayPlan && !hasRetentionCurve ? nextDayBounds() : null;
500
+ let cursor = Math.max(lifecycleStart, firstDayBounds?.earliest ?? lifecycleStart);
503
501
 
504
502
  // Resolve attempts plan. `attempts.{min,max}` count FAILED PRIORS; total
505
503
  // passes = failedPriors + 1. Validator coerced bounds; default both 0.
@@ -509,6 +507,7 @@ export async function userLoop(context) {
509
507
  const failedPriors = (maxA > 0) ? chance.integer({ min: minA, max: maxA }) : 0;
510
508
  const totalAttempts = failedPriors + 1;
511
509
  let firstAttemptFirstEventTime = null;
510
+ if (failedPriors > 0) cursor = lifecycleStart;
512
511
 
513
512
  for (let attemptNum = 1; attemptNum <= totalAttempts; attemptNum++) {
514
513
  const isFinal = attemptNum === totalAttempts;
@@ -528,16 +527,23 @@ export async function userLoop(context) {
528
527
  truncateBeforeAuth: !isFinal,
529
528
  devicePool: userDevicePool,
530
529
  };
530
+ const firstFeatureCtx = {
531
+ ...featureCtx,
532
+ latestTime: firstDayBounds && failedPriors === 0 ? firstDayBounds.latest : context.FIXED_NOW,
533
+ pinFirstTime: hasRetentionCurve || failedPriors > 0,
534
+ };
531
535
  const [data, converted, authMs] = await makeFunnel(
532
- context, funnelToRun, user, cursor, profile, userSCD, persona, featureCtx, attemptMeta
536
+ context, funnelToRun, user, cursor, profile, userSCD, persona, firstFeatureCtx, attemptMeta
533
537
  );
534
538
  if (isFinal) userConverted = converted;
535
539
  if (data && data.length) {
540
+ if (failedPriors > 0) promisedAttemptEntries.add(data[0].insert_id);
536
541
  if (firstAttemptFirstEventTime === null) {
537
542
  firstAttemptFirstEventTime = dayjs(data[0].time).unix();
538
543
  }
539
544
  // Advance the cursor for the next attempt by a small abandon-and-retry gap.
540
- const lastTime = dayjs(data[data.length - 1].time).unix();
545
+ const lastTime = Math.max(...data.map(event => Date.parse(event.time))) / 1000;
546
+ userUsageStartTime = Math.max(userUsageStartTime, lastTime + 0.001);
541
547
  cursor = lastTime + chance.integer({ min: 60, max: 30 * 60 }); // 1–30 min later
542
548
  numEventsPreformed += data.length;
543
549
  usersEvents = usersEvents.concat(data);
@@ -547,6 +553,13 @@ export async function userLoop(context) {
547
553
  if (userAuthTimeMs === null) userAuthTimeMs = authMs;
548
554
  userAuthed = true;
549
555
  }
556
+ if (!isFinal && !data.length && config.events.find(event => event.event === firstFunnel.sequence[0])?.isAuthEvent) {
557
+ context.addWarning({
558
+ key: 'lifecycle.emptyPreAuthAttempt',
559
+ requested: 1, applied: 0, severity: 'warn',
560
+ reason: 'A failed prior has no pre-auth step because auth starts the funnel; the prior emits no rows and the final attempt still runs.',
561
+ });
562
+ }
550
563
  }
551
564
 
552
565
  userFirstEventTime = firstAttemptFirstEventTime !== null
@@ -614,6 +627,8 @@ export async function userLoop(context) {
614
627
  // gets re-anchored to the picked day's start (subsequent funnel steps
615
628
  // spill within `timeToConvert` hours; this is intentional).
616
629
  const dayBounds = dayPlan ? nextDayBounds() : null;
630
+ const usageEarliest = Math.max(dayBounds?.earliest ?? userFirstEventTime, userUsageStartTime);
631
+ if (usageEarliest > (dayBounds?.latest ?? context.FIXED_NOW)) continue;
617
632
  // Compute step1's `latestTime` so the funnel's relative span fits before
618
633
  // FIXED_NOW. Without this, born-late users + long-ttc funnels generate
619
634
  // large numbers of `_drop`'d events that consume budget cycles. The
@@ -639,7 +654,7 @@ export async function userLoop(context) {
639
654
  const ttcSec = (currentFunnel.timeToConvert || 0) * 3600;
640
655
  // Anchor cursor at picked day's start when active-day mode is on,
641
656
  // otherwise pass userFirstEventTime (constant). NO cursor accumulation.
642
- const funnelCursor = dayBounds ? dayBounds.earliest : userFirstEventTime;
657
+ const funnelCursor = usageEarliest;
643
658
  // Constrain funnel step1's TimeSoup latestTime so the full funnel fits in
644
659
  // window. Without this, late steps spill past FIXED_NOW and get `_drop`'d.
645
660
  //
@@ -683,7 +698,7 @@ export async function userLoop(context) {
683
698
  // — passing `true` here would pin every standalone event to the same
684
699
  // `earliestTime`, which the now-deleted bunchIntoSessions used to paper
685
700
  // over. With bunchIntoSessions removed, TimeSoup is the time source.
686
- const standaloneEarliest = dayBounds ? dayBounds.earliest : userFirstEventTime;
701
+ const standaloneEarliest = usageEarliest;
687
702
  // v1.5.1: pass `config.superProps` so standalone events get super-property
688
703
  // stamping. Pre-1.5.1, standalone events received `{}` here, but the
689
704
  // validator's auto-funnel (catch-all) consumed all non-strict events so
@@ -763,6 +778,8 @@ export async function userLoop(context) {
763
778
  for (const ev of usersEvents) {
764
779
  if (chance.bool({ likelihood: dataQuality.duplicateRate * 100 })) {
765
780
  const dupe = { ...ev };
781
+ const identityDescriptor = Object.getOwnPropertyDescriptor(ev, engineIdentity);
782
+ if (identityDescriptor) Object.defineProperty(dupe, engineIdentity, identityDescriptor);
766
783
  dupe.time = dayjs(ev.time).add(chance.integer({ min: 1, max: 60 }), 'seconds').toISOString();
767
784
  dupe.insert_id = randomUUID();
768
785
  dupes.push(dupe);
@@ -828,6 +845,13 @@ export async function userLoop(context) {
828
845
  profile._drop = true;
829
846
  }
830
847
 
848
+ const identityBeforeEverything = new Map(usersEvents.map(event => [event.insert_id, {
849
+ user_id: event.user_id, device_id: event.device_id,
850
+ original: event[engineIdentity],
851
+ }]));
852
+ const profileDropBeforeEverything = profile._drop;
853
+ let profileDropOverridden = false;
854
+
831
855
  // Hook for processing all user events (hooks override everything)
832
856
  if (config.hook) {
833
857
  // `meta.isPreAuth(event)` predicate bound to this user's auth state.
@@ -843,7 +867,16 @@ export async function userLoop(context) {
843
867
  return Number.isFinite(t) ? t < userAuthTimeMsLocal : false;
844
868
  };
845
869
  const newEvents = await config.hook(usersEvents, "everything", {
846
- profile,
870
+ profile: new Proxy(profile, {
871
+ deleteProperty(target, key) {
872
+ if (key === '_drop') profileDropOverridden = true;
873
+ return Reflect.deleteProperty(target, key);
874
+ },
875
+ set(target, key, value) {
876
+ if (key === '_drop') profileDropOverridden = true;
877
+ return Reflect.set(target, key, value);
878
+ },
879
+ }),
847
880
  scd: userSCD,
848
881
  config,
849
882
  userIsBornInDataset,
@@ -944,13 +977,44 @@ export async function userLoop(context) {
944
977
  // stream to the remaining room with a seeded uniform sample (keeps the
945
978
  // time shape; preserves array order). Only under the flag.
946
979
  if (strictEventCount && usersEvents.length > 0) {
980
+ const promisedEntries = usersEvents.filter(event => promisedAttemptEntries.has(event.insert_id));
947
981
  const room = numEvents - context.getStoredEventCount();
948
982
  if (room <= 0) {
949
983
  usersEvents = [];
950
984
  } else if (usersEvents.length > room) {
951
- const keep = new Set(chance.pickset(usersEvents, room));
985
+ const reserved = promisedEntries.slice(0, room);
986
+ const others = usersEvents.filter(event => !promisedAttemptEntries.has(event.insert_id));
987
+ const keep = new Set([...reserved, ...chance.pickset(others, Math.max(0, room - reserved.length))]);
952
988
  usersEvents = usersEvents.filter(e => keep.has(e));
953
989
  }
990
+ if (promisedEntries.some(event => !usersEvents.includes(event))) {
991
+ context.addWarning({
992
+ key: 'lifecycle.strictAttemptBudget',
993
+ requested: promisedEntries.length, applied: Math.max(0, room), severity: 'warn',
994
+ reason: 'The remaining strict event budget is smaller than the surviving first-funnel attempt entries; strictEventCount takes precedence and later entries are omitted.',
995
+ });
996
+ }
997
+ }
998
+
999
+ if (userIsBornInDataset && userDevicePool?.length) {
1000
+ const authNames = new Set(config.events.filter(event => event.isAuthEvent).map(event => event.event));
1001
+ const stitches = usersEvents.filter(event => authNames.has(event.event) && event.user_id && event.device_id);
1002
+ const emittedAuthTime = stitches.length ? Math.min(...stitches.map(event => Date.parse(event.time))) : null;
1003
+ for (const event of usersEvents) {
1004
+ if (event.event === '$experiment_started') continue;
1005
+ if (emittedAuthTime === null || Date.parse(event.time) < emittedAuthTime) {
1006
+ const before = identityBeforeEverything.get(event.insert_id);
1007
+ const original = event[engineIdentity] || before?.original;
1008
+ if (!original) continue;
1009
+ const explicitUser = event.user_id !== original.user_id;
1010
+ const explicitDevice = before
1011
+ ? event.device_id !== before.device_id || (!before.device_id && !!original.device_id)
1012
+ : event.device_id !== original.device_id;
1013
+ if (!explicitUser) delete event.user_id;
1014
+ if (!explicitUser && !explicitDevice) event.device_id ||= userDevicePool[0];
1015
+ }
1016
+ }
1017
+ if (emittedAuthTime === null && !profileDropOverridden && profile._drop === profileDropBeforeEverything) profile._drop = true;
954
1018
  }
955
1019
 
956
1020
  // Store all user data (skip profile push when a hook returned null
@@ -1108,7 +1172,10 @@ export function amplifyWorldEvents(events, worldEvents, chance, fixedNow) {
1108
1172
  if (frac > 0 && chance.bool({ likelihood: frac * 100 })) copies++;
1109
1173
  for (let c = 0; c < copies; c++) {
1110
1174
  const t = chance.integer({ min: windowStart, max: windowEnd - 1 });
1111
- clones.push({ ...ev, time: dayjs.unix(t).toISOString(), insert_id: randomUUID() });
1175
+ const clone = { ...ev, time: dayjs.unix(t).toISOString(), insert_id: randomUUID() };
1176
+ const identityDescriptor = Object.getOwnPropertyDescriptor(ev, engineIdentity);
1177
+ if (identityDescriptor) Object.defineProperty(clone, engineIdentity, identityDescriptor);
1178
+ clones.push(clone);
1112
1179
  }
1113
1180
  }
1114
1181
  }