Skip to content

Commit 322bd22

Browse files
committed
Merge branch 'main' into hm/update-get-preferences
2 parents 881992c + a0e023a commit 322bd22

File tree

12 files changed

+27
-24
lines changed

12 files changed

+27
-24
lines changed

.github/workflows/update-pull-request.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -197,6 +197,8 @@ jobs:
197197
- prepare
198198
- dedupe-yarn-lock
199199
steps:
200+
- name: Checkout repository
201+
uses: actions/checkout@v4
200202
- name: Restore yarn.lock
201203
uses: actions/cache/restore@v4
202204
with:

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@
9494
"@typescript-eslint/eslint-plugin": "^5.42.1",
9595
"@typescript-eslint/parser": "^6.21.0",
9696
"@yarnpkg/types": "^4.0.0",
97-
"chromedriver": "^133.0.0",
97+
"chromedriver": "^133.0.1",
9898
"depcheck": "^1.4.7",
9999
"eslint": "^8.27.0",
100100
"eslint-config-prettier": "^8.5.0",

packages/snaps-controllers/src/services/AbstractExecutionService.test.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,9 @@ import type { ExecutionServiceArgs } from './AbstractExecutionService';
88
import { NodeThreadExecutionService } from './node';
99

1010
class MockExecutionService extends NodeThreadExecutionService {
11-
constructor({ messenger, setupSnapProvider }: ExecutionServiceArgs) {
11+
constructor({ messenger, setupSnapProvider, ...args }: ExecutionServiceArgs) {
1212
super({
13+
...args,
1314
messenger,
1415
setupSnapProvider,
1516
initTimeout: inMilliseconds(5, Duration.Second),

packages/snaps-controllers/src/services/iframe/IframeExecutionService.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,10 @@ export class IframeExecutionService extends AbstractExecutionService<Window> {
1919
iframeUrl,
2020
messenger,
2121
setupSnapProvider,
22+
...args
2223
}: IframeExecutionEnvironmentServiceArgs) {
2324
super({
25+
...args,
2426
messenger,
2527
setupSnapProvider,
2628
});

packages/snaps-controllers/src/services/offscreen/OffscreenExecutionService.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,10 @@ export class OffscreenExecutionService extends ProxyExecutionService {
2525
messenger,
2626
setupSnapProvider,
2727
offscreenPromise,
28+
...args
2829
}: OffscreenExecutionEnvironmentServiceArgs) {
2930
super({
31+
...args,
3032
messenger,
3133
setupSnapProvider,
3234
stream: new BrowserRuntimePostMessageStream({

packages/snaps-controllers/src/services/proxy/ProxyExecutionService.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,10 @@ export class ProxyExecutionService extends AbstractExecutionService<string> {
3030
stream,
3131
messenger,
3232
setupSnapProvider,
33+
...args
3334
}: ProxyExecutionEnvironmentServiceArgs) {
3435
super({
36+
...args,
3537
messenger,
3638
setupSnapProvider,
3739
usePing: false,

packages/snaps-controllers/src/services/webview/WebViewExecutionService.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,10 @@ export class WebViewExecutionService extends ProxyExecutionService {
1414
messenger,
1515
setupSnapProvider,
1616
getWebView,
17+
...args
1718
}: WebViewExecutionServiceArgs) {
1819
super({
20+
...args,
1921
messenger,
2022
setupSnapProvider,
2123
stream: new WebViewMessageStream({

packages/snaps-controllers/src/services/webworker/WebWorkerExecutionService.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,10 @@ export class WebWorkerExecutionService extends AbstractExecutionService<string>
3737
documentUrl,
3838
messenger,
3939
setupSnapProvider,
40+
...args
4041
}: WebWorkerExecutionEnvironmentServiceArgs) {
4142
super({
43+
...args,
4244
messenger,
4345
setupSnapProvider,
4446
});

packages/snaps-simulation/src/helpers.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -141,9 +141,7 @@ export type SnapHelpers = {
141141
* @param keyringRequest - Keyring request.
142142
* @returns The response.
143143
*/
144-
onKeyringRequest(
145-
keyringRequest: KeyringOptions,
146-
): Promise<SnapResponseWithoutInterface>;
144+
onKeyringRequest(keyringRequest: KeyringOptions): SnapRequest;
147145

148146
/**
149147
* Get the response from the Snap's `onInstall` handler.
@@ -270,12 +268,10 @@ export function getHelpers({
270268
});
271269
};
272270

273-
const onKeyringRequest = async (
274-
request: KeyringOptions,
275-
): Promise<SnapResponseWithoutInterface> => {
271+
const onKeyringRequest = (request: KeyringOptions) => {
276272
log('Sending keyring request %o.', request);
277273

278-
const response = await handleRequest({
274+
return handleRequest({
279275
snapId,
280276
store,
281277
executionService,
@@ -284,8 +280,6 @@ export function getHelpers({
284280
handler: HandlerType.OnKeyringRequest,
285281
request,
286282
});
287-
288-
return response;
289283
};
290284

291285
return {

scripts/install-chrome.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@ set -u
55
set -o pipefail
66

77
# To get the latest version, run `yarn update-chrome`
8-
CHROME_VERSION='133.0.6943.53-1'
8+
CHROME_VERSION='133.0.6943.98-1'
99
CHROME_BINARY="google-chrome-stable_${CHROME_VERSION}_amd64.deb"
1010
CHROME_BINARY_URL="https://dl.google.com/linux/chrome/deb/pool/main/g/google-chrome-stable/${CHROME_BINARY}"
1111

1212
# To retrieve this checksum, run the `wget` and `shasum` commands below
13-
CHROME_BINARY_SHA512SUM='5e071dacaa0e7c5c1c858d2389c872c4d45f1568436a9d1a30dbbd34ecbd69f71119154360edfb03682a50704d41202db051b40e6afcb47ea249a558c0c9eb08'
13+
CHROME_BINARY_SHA512SUM='51475f9b22e1cb0a1411398b437523a80bafb1d22033806ddc51e2f2c2e289a193995f5a1ff6c15a47c9738aff403261f25b50480c58b8dfb290c8e8df4ddc9b'
1414

1515
wget -O "${CHROME_BINARY}" -t 5 "${CHROME_BINARY_URL}"
1616

0 commit comments

Comments
 (0)