Add Cursor Cloud Agent development environment - #219
Merged
Merged
Conversation
Install the CI-pinned Dart SDK (2.19.6) and fetch pub dependencies for the w_common and w_common_tools packages. Chrome (needed for browser tests) is already present in the base image, and all dependencies are public pub.dev packages, so no wk/Artifactory auth is required. Co-authored-by: Edward Weymouth <edwardweymouth-wk@users.noreply.github.com>
edwardweymouth-wk
marked this pull request as ready for review
September 9, 2026 21:19
Move the self-contained Dart SDK install + pub get into the environment.json install field and remove the separate .cursor/install.sh. This makes the config a single source of truth and lets a promotable default-branch environment build be produced (a build is required for the dashboard Save flow to attach a valid environment). Co-authored-by: Edward Weymouth <edwardweymouth-wk@users.noreply.github.com>
edwardweymouth-wk
requested review from
a team,
lauracallahan-wk and
sorenthompson-wk
September 9, 2026 21:37
sorenthompson-wk
approved these changes
Sep 9, 2026
|
QA +1
@Workiva/release-management-p |
andrewmumm-wk
approved these changes
Sep 10, 2026
andrewmumm-wk
left a comment
andrewmumm-wk
left a comment
There was a problem hiding this comment.
QA+1
@Workiva/release-management-p
btr-rmconsole-2
Bot
added
Merge Requirements Met
and removed
Merge Requirements Unmet
labels
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
This repository had no Cursor Cloud Agent environment configuration, so a fresh agent started with no Dart toolchain and no dependencies installed. This PR adds a repository-managed environment so future agents can build, analyze, format, and test both packages out of the box.
Changes
.cursor/environment.jsonwith a self-containedinstallstep that:2.19.6(matches.github/workflows/dart_ci.yml) to/usr/lib/dart-sdkand symlinks it onto the defaultPATHat/usr/local/bin/dart(no shell-profile mutation), anddart pub getfor bothw_commonandw_common_tools.The install command is inlined directly in
environment.json(single source of truth), which also allows a promotable default-branch environment build to be produced for the dashboard Save flow.No custom Dockerfile is used: the default base image already ships Chrome (required for the
chrome-platform browser tests) plusgit/curl/unzip. All dependencies resolve from public pub.dev, so nowk/Artifactory authentication is required. Nostartstep is needed because the SDK,PATHsymlink, and pub cache persist in the environment build snapshot.Release Notes
Add Cursor Cloud Agent development environment configuration.
Review
See CONTRIBUTING.md for more details on review types (+1 / QA +1 / +10) and code review process.
Please review:
QA Checklist
dart pub get,dart run dependency_validator,dart analyze,dart format --output=none --set-exit-if-changed ., anddart testfor both packages (271 + 25 tests pass), plus thecompile_sassCLI against SCSS fixtures. Also verified via a fresh Cloud Agent booted from the built environment.Merge Checklist
While we perform many automated checks before auto-merging, some manual checks are needed: