Skip to content

fix: add actionable warning for background requests during replay#76

Merged
sohil-kshirsagar merged 2 commits intomainfrom
fix/replay-background-request-warning
Mar 16, 2026
Merged

fix: add actionable warning for background requests during replay#76
sohil-kshirsagar merged 2 commits intomainfrom
fix/replay-background-request-warning

Conversation

@sohil-kshirsagar
Copy link
Contributor

Upgrades the no-op background request log from debug to warn with actionable guidance. When background jobs, scheduled tasks, or middleware (e.g., rate limiters, AMQP consumers) fire outside a test trace during replay, the log now tells users to disable these services when TUSK_DRIFT_MODE=REPLAY.

TODO: Have the setup agent detect background services and automatically configure the app to skip them during replay.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@tusk-dev
Copy link

tusk-dev bot commented Mar 16, 2026

Skipped due to new commit on branch

View output

Tip

New to Tusk Unit Tests? Learn more here.

View check history

Commit Status Output Created (UTC)
ebb7ee3 Generated 3 tests - 3 passed Tests Mar 16, 2026 3:54PM
01edca4 Generated 3 tests - 3 passed Tests Mar 16, 2026 4:00PM
47158a6 No tests generated Output Mar 16, 2026 4:04PM
6aa2aab Skipped due to new commit on branch Output Mar 16, 2026 10:12PM

@sohil-kshirsagar sohil-kshirsagar force-pushed the fix/replay-background-request-warning branch from ebb7ee3 to 01edca4 Compare March 16, 2026 16:00
@sohil-kshirsagar sohil-kshirsagar force-pushed the fix/replay-background-request-warning branch from 01edca4 to 47158a6 Compare March 16, 2026 16:03
Copy link

@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

@sohil-kshirsagar sohil-kshirsagar requested a review from jy-tan March 16, 2026 22:46
@sohil-kshirsagar sohil-kshirsagar merged commit 409bb88 into main Mar 16, 2026
27 checks passed
@sohil-kshirsagar sohil-kshirsagar deleted the fix/replay-background-request-warning branch March 16, 2026 22:53
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