testdriverai 7.9.0-test.41 → 7.9.0-test.43
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.
- package/agent/lib/sandbox.js +7 -1
- package/lib/sentry.js +2 -2
- package/package.json +1 -1
package/agent/lib/sandbox.js
CHANGED
|
@@ -117,7 +117,13 @@ const createSandbox = function (emitter, analytics, sessionInstance) {
|
|
|
117
117
|
|
|
118
118
|
this._sessionChannel = this._ably.channels.get(channelName);
|
|
119
119
|
|
|
120
|
-
|
|
120
|
+
// Explicitly attach the session channel BEFORE entering presence on the
|
|
121
|
+
// members channel. Entering members-presence triggers the API's waitpoint
|
|
122
|
+
// completion → claim-slot task → publishes slot-approved on the session
|
|
123
|
+
// channel. If the session channel isn't attached yet, that message lands
|
|
124
|
+
// before our attachment point and historyBeforeSubscribe() won't see it.
|
|
125
|
+
await this._sessionChannel.attach();
|
|
126
|
+
logger.debug(`[realtime] Channel attached: ${channelName}`);
|
|
121
127
|
|
|
122
128
|
// Enter presence on the team members channel so the API can count
|
|
123
129
|
// connected SDK clients with a single direct lookup per team.
|
package/lib/sentry.js
CHANGED
|
@@ -14,7 +14,7 @@ const Sentry = require("@sentry/node");
|
|
|
14
14
|
const crypto = require("crypto");
|
|
15
15
|
const os = require("os");
|
|
16
16
|
const { version } = require("../package.json");
|
|
17
|
-
const { sentryEnvironment } = require("./resolve-channel");
|
|
17
|
+
const { active, sentryEnvironment } = require("./resolve-channel");
|
|
18
18
|
const logger = require("../agent/lib/logger");
|
|
19
19
|
|
|
20
20
|
// Store trace contexts per session so concurrent tests don't overwrite each other.
|
|
@@ -34,7 +34,7 @@ const isEnabled = () => {
|
|
|
34
34
|
return false;
|
|
35
35
|
}
|
|
36
36
|
// Disable in test channel and PR previews
|
|
37
|
-
if (
|
|
37
|
+
if (active === 'test' || process.env.IS_PULL_REQUEST === 'true') {
|
|
38
38
|
return false;
|
|
39
39
|
}
|
|
40
40
|
return true;
|