Skip to content

Extract sanitizer module#169

Merged
Reversean merged 1 commit intomasterfrom
refactor/core-sanitizer
Mar 17, 2026
Merged

Extract sanitizer module#169
Reversean merged 1 commit intomasterfrom
refactor/core-sanitizer

Conversation

@Reversean
Copy link
Member

@Reversean Reversean commented Mar 8, 2026

  • Sanitizer moved to @hawk.so/core (except Element handling since it depends on jsdom)
  • Sanitizer.isPlainObject removed (use function from type-guards.ts)
  • type guard private methods moved to type-guards.ts as public functions
  • SanitizerTypeHandler feature added
  • SanitizerTypeHandler for browser Element added

Copy link
Member

@neSpecc neSpecc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • move type checking methods to utils, cover by tests

@Reversean Reversean force-pushed the refactor/core-sanitizer branch 3 times, most recently from 7f705de to f4adbb5 Compare March 16, 2026 18:03
@Reversean Reversean force-pushed the refactor/core-validation branch from 2c57a08 to 0ab3946 Compare March 16, 2026 18:39
@Reversean Reversean force-pushed the refactor/core-sanitizer branch from f4adbb5 to 0a536c0 Compare March 16, 2026 18:44
@Reversean Reversean force-pushed the refactor/core-validation branch from 0ab3946 to 906d976 Compare March 16, 2026 18:57
@Reversean Reversean force-pushed the refactor/core-sanitizer branch from 0a536c0 to bf5545f Compare March 16, 2026 18:57
@Reversean
Copy link
Member Author

  • move type checking methods to utils, cover by tests

Done.

@Reversean Reversean force-pushed the refactor/core-validation branch from 906d976 to 835a79f Compare March 17, 2026 09:21
@Reversean Reversean force-pushed the refactor/core-sanitizer branch from bf5545f to 6e0faf5 Compare March 17, 2026 09:23
Base automatically changed from refactor/core-validation to master March 17, 2026 09:24
@Reversean Reversean force-pushed the refactor/core-sanitizer branch from 6e0faf5 to c605b06 Compare March 17, 2026 11:03
- Sanitizer moved to @hawk.so/core (except Element handling)
- SanitizerTypeHandler feature added
- browser Element SanitizerTypeHandler added
- type-guards utils moved from sanitizer
- type-guards utils covered with tests
Copy link
Member

@neSpecc neSpecc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good job

@Reversean Reversean merged commit af39f39 into master Mar 17, 2026
3 checks passed
@Reversean Reversean deleted the refactor/core-sanitizer branch March 17, 2026 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants