Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

For TypeScript v5.2 patch #784

Merged
merged 1 commit into from
Aug 25, 2023
Merged

For TypeScript v5.2 patch #784

merged 1 commit into from
Aug 25, 2023

Conversation

samchon
Copy link
Owner

@samchon samchon commented Aug 25, 2023

type Intersection = { x: number } & Record<string | object | symbol, string>;

When intersection type with static object and dynamic object with symbol typed key comes, there had not been any problem until TypeScript@5.1. However, after TypeScript@5.2 patch, it became an error in typia. I think TypeScript may changed internal logic of compiler.

Anyway, I fixed that bug and during the implementation, I found another type of bug. When above Intersection type comes, the property x must not be doubly checked by validation function of typia like assert<T>(). However, typia had done it, and I fixed that bug, too.

```typescript
type Intersection = { x: number } & Record<string | object | symbol, string>;
```

When intersection type with static object and dynamic object with `symbol` typed key comes, there had not been any problem until `TypeScript@5.1`. However, after `TypeScript@5.2` patch, it became an error in `typia`. I think TypeScript may changed internal logic of compiler.

Anyway, I fixed that bug and during the implementation, I found another type of bug. When above `Intersection` type comes, the property `x` must not be doubly checked by validation function of typia like `assert<T>()`. However, `typia` had done it, and I fixed that bug, too.
@samchon samchon added bug Something isn't working enhancement New feature or request labels Aug 25, 2023
@samchon samchon self-assigned this Aug 25, 2023
@samchon samchon merged commit bbb353f into master Aug 25, 2023
2 checks passed
@samchon samchon deleted the features/intersection branch August 25, 2023 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant