Conversation
There was a problem hiding this comment.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
| "envFile": "${workspaceFolder}/.env", | ||
| "console": "internalConsole", | ||
| "internalConsoleOptions": "openOnSessionStart" | ||
| }, |
There was a problem hiding this comment.
Missing TEST_PROFILE_NAME environment variable in launch configuration
Medium Severity
The "Test Profile: host.docker.internal" launch configuration doesn't set the TEST_PROFILE_NAME environment variable. Unlike other test profile configurations (e.g., "default-horizon", "custody") that explicitly set TEST_PROFILE_NAME in their env block, this configuration is missing it entirely. Without TEST_PROFILE_NAME set to host-docker-internal, the configuration will run the default profile instead of the intended host-docker-internal profile, causing unexpected behavior for developers who expect the configuration name to match the actual test profile being executed.
Description
ServiceRunnerfrom Java to Kotlin, maintaining full functional parity and Java interoperabilityobject(singleton pattern).vscode/launch.json,.vscode/settings.json,.vscode/tasks.json).vscode/launch.json: Pre-configured launch configurations for all Anchor Platform services and test profiles (18 configurations total).vscode/settings.json: Java development settings (note: contains empty strings forjava.jdt.ls.java.homeandjava.debug.settings.vmArgswhich may need to be configured per developer).vscode/tasks.json: Empty file for future task definitions.vscodefrom.gitignoreto allow committing VS Code configuration filesscripts/docker-start.shexecutable (set executable file permissions)Context
ServiceRunner.javato Kotlin works around the issue.Testing
./gradlew test- All existing tests passDocumentation
N/A
Known limitations
NA