Skip to content

fix: remove HTML from acceptable content types for trace recording#147

Merged
sohankshirsagar merged 1 commit intomainfrom
sohan/remove-html-content-type
Mar 25, 2026
Merged

fix: remove HTML from acceptable content types for trace recording#147
sohankshirsagar merged 1 commit intomainfrom
sohan/remove-html-content-type

Conversation

@sohankshirsagar
Copy link
Contributor

Summary

Removes DecodedType.HTML from ACCEPTABLE_CONTENT_TYPES. HTML responses are typically static pages or server-rendered views that are large, noisy, and poor candidates for regression testing

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

@tusk-dev
Copy link

tusk-dev bot commented Mar 25, 2026

No tests generated

View output

Tip

New to Tusk Unit Tests? Learn more here.

No symbols were selected for unit test generation because the change is limited to a constant (ACCEPTABLE_CONTENT_TYPES) with no associated logic or computation. The modification is purely a data update, and the actual filtering behavior occurs in unchanged files. Existing unit tests already cover the relevant decoding and encoding functions, and testing the constant itself would offer minimal algorithmic value.

View check history

Commit Status Output Created (UTC)
d8f0b19 No tests generated Output Mar 25, 2026 11:40PM

@sohankshirsagar sohankshirsagar merged commit b1ad466 into main Mar 25, 2026
19 checks passed
@sohankshirsagar sohankshirsagar deleted the sohan/remove-html-content-type branch March 25, 2026 23:48
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.

2 participants