Skip to content

Commit b9eae1d

Browse files
authored
fix: unnecessarily recursive Draft type (#990)
1 parent 8e12c78 commit b9eae1d

File tree

4 files changed

+45
-1
lines changed

4 files changed

+45
-1
lines changed

__tests__/types/type-externals.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
import {isType, JSONTypes} from "type-plus"
2+
import {Draft} from "../../src/types/types-external"
3+
4+
describe("Draft<T>", () => {
5+
test("can use JSONTypes as T", () => {
6+
type A = Draft<JSONTypes>
7+
isType.equal<true, JSONTypes, A>()
8+
})
9+
})

package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@
9999
"spec.ts": "^1.1.0",
100100
"ts-jest": "^25.2.0",
101101
"tsdx": "^0.12.3",
102+
"type-plus": "^4.15.2",
102103
"typescript": "^4.2.3"
103104
}
104105
}

src/types/types-external.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,9 @@ export type IfAvailable<T, Fallback = void> =
3030
*/
3131
type WeakReferences = IfAvailable<WeakMap<any, any>> | IfAvailable<WeakSet<any>>
3232

33-
export type WritableDraft<T> = {-readonly [K in keyof T]: Draft<T[K]>}
33+
export type WritableDraft<T> = {
34+
-readonly [K in keyof T]: T[K] extends object ? Draft<T[K]> : T[K]
35+
}
3436

3537
/** Convert a readonly type into a mutable type, if possible */
3638
export type Draft<T> = T extends PrimitiveType

yarn.lock

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1791,6 +1791,11 @@ acorn@^7.1.0:
17911791
resolved "https://registry.yarnpkg.com/acorn/-/acorn-7.1.0.tgz#949d36f2c292535da602283586c2477c57eb2d6c"
17921792
integrity sha512-kL5CuoXA/dgxlBbVrflsflzQ3PAas7RYZB52NOm/6839iVYJgKMJ3cQJD+t2i5+qFa8h3MDpEOJiS64E8JLnSQ==
17931793

1794+
acorn@^8.8.0:
1795+
version "8.8.1"
1796+
resolved "https://registry.yarnpkg.com/acorn/-/acorn-8.8.1.tgz#0a3f9cbecc4ec3bea6f0a80b66ae8dd2da250b73"
1797+
integrity sha512-7zFpHzhnqYKrkYdUjF1HI1bzd0VygEGX8lFk4k5zVMqHEoES+P+7TKI+EvLO9WVMJ8eekdO0aDEK044xTXwPPA==
1798+
17941799
agent-base@4, agent-base@^4.3.0:
17951800
version "4.3.0"
17961801
resolved "https://registry.yarnpkg.com/agent-base/-/agent-base-4.3.0.tgz#8165f01c436009bccad0b1d122f05ed770efc6ee"
@@ -5854,6 +5859,11 @@ is-buffer@^1.1.5:
58545859
resolved "https://registry.yarnpkg.com/is-buffer/-/is-buffer-1.1.6.tgz#efaa2ea9daa0d7ab2ea13a97b2b8ad51fefbe8be"
58555860
integrity sha512-NcdALwpXkTm5Zvvbk7owOUSvVvBKDgKP5/ewfXEznmQFfs4ZRmanOeKBTjRVjka3QFoN6XJ+9F3USqfHqTaU5w==
58565861

5862+
is-buffer@^2.0.5:
5863+
version "2.0.5"
5864+
resolved "https://registry.yarnpkg.com/is-buffer/-/is-buffer-2.0.5.tgz#ebc252e400d22ff8d77fa09888821a24a658c191"
5865+
integrity sha512-i2R6zNFDwgEHJyQUtJEk0XFi1i0dPFn/oqjK3/vPCcDeJvW5NQ83V8QbicfF1SupOaB0h8ntgBC2YiE7dfyctQ==
5866+
58575867
is-callable@^1.1.4, is-callable@^1.2.0:
58585868
version "1.2.0"
58595869
resolved "https://registry.yarnpkg.com/is-callable/-/is-callable-1.2.0.tgz#83336560b54a38e35e3a2df7afd0454d691468bb"
@@ -11255,6 +11265,15 @@ terser@^4.1.2, terser@^4.6.2:
1125511265
source-map "~0.6.1"
1125611266
source-map-support "~0.5.12"
1125711267

11268+
tersify@^3.10.2:
11269+
version "3.10.5"
11270+
resolved "https://registry.yarnpkg.com/tersify/-/tersify-3.10.5.tgz#eb2b230e7725481129b7fa349f2ce2683bd8a9ec"
11271+
integrity sha512-dLgsTbGVH/6Z+KU4d+NqMmsMFRwRmTFNxsl5vKGkUmaSQ0TTAwnv+Hck5S8Pdn5bcZSyjR3aTvvetJ7ZUs+ZYg==
11272+
dependencies:
11273+
acorn "^8.8.0"
11274+
is-buffer "^2.0.5"
11275+
unpartial "^1.0.0"
11276+
1125811277
test-exclude@^5.2.3:
1125911278
version "5.2.3"
1126011279
resolved "https://registry.yarnpkg.com/test-exclude/-/test-exclude-5.2.3.tgz#c3d3e1e311eb7ee405e092dac10aefd09091eac0"
@@ -11607,6 +11626,14 @@ type-fest@^0.8.1:
1160711626
resolved "https://registry.yarnpkg.com/type-fest/-/type-fest-0.8.1.tgz#09e249ebde851d3b1e48d27c105444667f17b83d"
1160811627
integrity sha512-4dbzIzqvjtgiM5rw1k5rEHtBANKmdudhGyBEajN01fEyhaAIhsoKNy6y7+IN93IfpFtwY9iqi7kD+xwKhQsNJA==
1160911628

11629+
type-plus@^4.15.2:
11630+
version "4.15.2"
11631+
resolved "https://registry.yarnpkg.com/type-plus/-/type-plus-4.15.2.tgz#2d97273bde0c921845ae70a0d1cef075a7c0cfc1"
11632+
integrity sha512-5hEzFXMHOK/p5hVhNmgy7CIxUjTIK9a/NOw0pDbc4tb7LXm9JLwLEH0QdGWU5fGPp5l05rtPZfjdSOPqfAyX9w==
11633+
dependencies:
11634+
tersify "^3.10.2"
11635+
unpartial "^1.0.3"
11636+
1161011637
typedarray-to-buffer@^3.1.5:
1161111638
version "3.1.5"
1161211639
resolved "https://registry.yarnpkg.com/typedarray-to-buffer/-/typedarray-to-buffer-3.1.5.tgz#a97ee7a9ff42691b9f783ff1bc5112fe3fca9080"
@@ -11725,6 +11752,11 @@ universalify@^2.0.0:
1172511752
resolved "https://registry.yarnpkg.com/universalify/-/universalify-2.0.0.tgz#75a4984efedc4b08975c5aeb73f530d02df25717"
1172611753
integrity sha512-hAZsKq7Yy11Zu1DE0OzWjw7nnLZmJZYTDZZyEFHZdUhV8FkH5MCfoU1XMaxXovpyW5nq5scPqq0ZDP9Zyl04oQ==
1172711754

11755+
unpartial@^1.0.0, unpartial@^1.0.3:
11756+
version "1.0.4"
11757+
resolved "https://registry.yarnpkg.com/unpartial/-/unpartial-1.0.4.tgz#56490ed475105110cee61890a80c5da394741833"
11758+
integrity sha512-xY8319WOQcRDRVFWvTty2YJXKvM8XRPehqWZUd7k/BatpbuitI4Jd+2xlwjTZbZScSvT75TnUsUmy5uwFa3o0g==
11759+
1172811760
unpipe@~1.0.0:
1172911761
version "1.0.0"
1173011762
resolved "https://registry.yarnpkg.com/unpipe/-/unpipe-1.0.0.tgz#b2bf4ee8514aae6165b4817829d21b2ef49904ec"

0 commit comments

Comments
 (0)