Skip to content

feat: Issue #34 - switching test setup to bun #35

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.

Sign up for GitHub

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

Open
ai16z-demirix wants to merge 9 commits into
base: 1.x
Choose a base branch
from

fixing due to suggestion/bug from cursor - fetching single post

948e73c
Select commit
Loading
Failed to load commit list.
Open

feat: Issue #34 - switching test setup to bun #35

fixing due to suggestion/bug from cursor - fetching single post
948e73c
Select commit
Loading
Failed to load commit list.
Cursor / Cursor BugBot completed Jul 18, 2025 in 2m 37s

BugBot Review

BugBot Analysis Progress (2m 38s elapsed)

✅ Gathered PR context (1s)
✅ Analyzed code changes (1s)
✅ Completed bug detection — 1 potential bug found (2m 32s)
✅ Validation and filtering completed (0s)
✅ Posted analysis results — 1 bug reported (3s)
✅ Analysis completed successfully (0s)

Final Result: BugBot completed review and found 1 potential issue

Request ID: serverGenReqId_f3fb5ee2-fbfb-492b-9801-d384ea0cc157

Details

Bug: Test Fails Due to Missing Environment Variable

The test "should prioritize config over runtime over env" cannot fully verify the config > runtime > env priority chain. The vi.stubEnv call for TWITTER_API_KEY is commented out, with a note that Bun uses process.env directly, but no alternative is provided to set the environment variable for the test.

src/__tests__/environment.test.ts#L156-L161

plugin-twitter/src/__tests__/environment.test.ts

Lines 156 to 161 in 948e73c

test("should prioritize config over runtime over env", async () => {
// Bun doesn't need stubEnv - will use process.env directly
// vi.stubEnv("TWITTER_API_KEY", "env-api-key");
mockRuntime.getSetting = mock((key) => {

Fix in Cursor • Fix in Web

Was this report helpful? Give feedback by reacting with 👍 or 👎

View more details on Cursor