Skip to content

Commit 78563db

Browse files
authored
Standardize SDK and foundation runtime package contracts (#362)
* feat: standardize foundation runtime packages * fix: accept browser-safe SDK HTTP clients * fix: resolve foundation wave quality findings * fix: normalize lookup invocation failures * test: stabilize foundation coverage gates * test: close foundation patch coverage gaps
1 parent a76db01 commit 78563db

225 files changed

Lines changed: 13463 additions & 9115 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Lines changed: 1 addition & 85 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"schemaVersion": 1,
33
"recordedAt": "2026-07-26",
4-
"findingCount": 45,
4+
"findingCount": 24,
55
"findings": [
66
{
77
"id": ".::missing-script::format:check",
@@ -39,34 +39,6 @@
3939
"id": "docs-site::missing-script::test",
4040
"message": "Profile documentation-site requires script test"
4141
},
42-
{
43-
"id": "packages/network/ts-p2p::missing-script::format:check",
44-
"message": "Profile node-library requires script format:check"
45-
},
46-
{
47-
"id": "packages/network/ts-p2p::missing-script::lint",
48-
"message": "Profile node-library requires script lint"
49-
},
50-
{
51-
"id": "packages/network/ts-p2p::missing-script::pack:check",
52-
"message": "Profile node-library requires script pack:check"
53-
},
54-
{
55-
"id": "packages/network/ts-p2p::missing-script::typecheck",
56-
"message": "Profile node-library requires script typecheck"
57-
},
58-
{
59-
"id": "packages/overlays/gasp-core::missing-script::format:check",
60-
"message": "Profile dual-runtime-library requires script format:check"
61-
},
62-
{
63-
"id": "packages/overlays/gasp-core::missing-script::pack:check",
64-
"message": "Profile dual-runtime-library requires script pack:check"
65-
},
66-
{
67-
"id": "packages/overlays/gasp-core::missing-script::typecheck",
68-
"message": "Profile dual-runtime-library requires script typecheck"
69-
},
7042
{
7143
"id": "packages/overlays/overlay-discovery-services::missing-script::format:check",
7244
"message": "Profile node-library requires script format:check"
@@ -115,50 +87,6 @@
11587
"id": "packages/overlays/topics::missing-script::typecheck",
11688
"message": "Profile node-library requires script typecheck"
11789
},
118-
{
119-
"id": "packages/sdk::missing-script::format:check",
120-
"message": "Profile browser-library requires script format:check"
121-
},
122-
{
123-
"id": "packages/sdk::missing-script::pack:check",
124-
"message": "Profile browser-library requires script pack:check"
125-
},
126-
{
127-
"id": "packages/sdk::missing-script::test:browser",
128-
"message": "Profile browser-library requires script test:browser"
129-
},
130-
{
131-
"id": "packages/sdk::missing-script::typecheck",
132-
"message": "Profile browser-library requires script typecheck"
133-
},
134-
{
135-
"id": "packages/verifast::missing-script::format:check",
136-
"message": "Profile wasm-library requires script format:check"
137-
},
138-
{
139-
"id": "packages/verifast::missing-script::lint",
140-
"message": "Profile wasm-library requires script lint"
141-
},
142-
{
143-
"id": "packages/verifast::missing-script::pack:check",
144-
"message": "Profile wasm-library requires script pack:check"
145-
},
146-
{
147-
"id": "packages/wallet/ts-wallet-relay::missing-script::format:check",
148-
"message": "Profile cli-library requires script format:check"
149-
},
150-
{
151-
"id": "packages/wallet/ts-wallet-relay::missing-script::lint",
152-
"message": "Profile cli-library requires script lint"
153-
},
154-
{
155-
"id": "packages/wallet/ts-wallet-relay::missing-script::pack:check",
156-
"message": "Profile cli-library requires script pack:check"
157-
},
158-
{
159-
"id": "packages/wallet/ts-wallet-relay::missing-script::test:coverage",
160-
"message": "Profile cli-library requires script test:coverage"
161-
},
16290
{
16391
"id": "packages/wallet/wallet-toolbox-examples::missing-script::format:check",
16492
"message": "Profile examples requires script format:check"
@@ -170,18 +98,6 @@
17098
{
17199
"id": "packages/wallet/wallet-toolbox-examples::mutating-check-script::lint",
172100
"message": "lint must not modify the working tree"
173-
},
174-
{
175-
"id": "packages/wallet/wallet-toolbox::missing-script::format:check",
176-
"message": "Profile node-library requires script format:check"
177-
},
178-
{
179-
"id": "packages/wallet/wallet-toolbox::missing-script::pack:check",
180-
"message": "Profile node-library requires script pack:check"
181-
},
182-
{
183-
"id": "packages/wallet/wallet-toolbox::missing-script::typecheck",
184-
"message": "Profile node-library requires script typecheck"
185101
}
186102
]
187103
}
Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,14 @@
11
'use strict'
22

33
/**
4-
* The SDK's browser-safe default HTTP client includes a guarded CommonJS
5-
* `require("node:https")` fallback for old Node runtimes. Webpack leaves that
6-
* unreachable string in UMD output, where the browser composition gate must
7-
* reject it. Browser clients use fetch, so remove only this exact fallback.
4+
* Older SDK artifacts included a guarded CommonJS `require("node:https")`
5+
* fallback for old Node runtimes. Webpack left that unreachable string in UMD
6+
* output, where the browser composition gate rejected it. Browser clients use
7+
* fetch, so remove only this exact legacy fallback when it is present.
8+
*
9+
* Newer SDK artifacts already hide the optional Node module identifier from
10+
* static browser analysis. Those inputs are safe and require no transformation.
811
*/
912
module.exports = function removeNodeHttpsFallback(source) {
10-
const transformed = source.replace(/require\((['"])node:https\1\)/g, 'undefined')
11-
if (transformed === source) {
12-
throw new Error('Expected the SDK node:https fallback in the browser build input.')
13-
}
14-
return transformed
13+
return source.replace(/require\((['"])node:https\1\)/g, 'undefined')
1514
}

0 commit comments

Comments
 (0)