Repository navigation
feat: run rstack-editor E2E in the rstest and rslint suites - #115
Merged
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3dcf038509
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
fi3ework
force-pushed
the
feat/rstack-editor-e2e
branch
from
November 5, 2026 06:26
3dcf038 to
b4fe4bc
Compare
The editor's E2E fixtures install published packages, so the suites inject the local build through RSTACK_E2E_TOOLCHAIN_OVERRIDES: a tarball of @rstest/core or of @rslint/core plus its host native package. Both rslint packages come from getRslintPackages so the platform tuple is never hardcoded. The VS Code download is cached outside the checkout, which the suite cleans with git clean -fdxq.
The rstest execute jobs had no timeout (GitHub's default is 360 min); the rstack-editor job with E2E measured ~2 min. Prepare Rslint Binding measured 36-48 min in the last runs against a 60 min limit.
fi3ework
force-pushed
the
feat/rstack-editor-e2e
branch
from
November 5, 2026 06:37
b4fe4bc to
a8f6da9
Compare
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
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.
Summary
The
rstack-editorsuites in both stacks ran onlytest:unit+lint. The editor's E2E fixtures are their own pnpm workspace roots that install published packages from npm, so eco-ci's consumer-side override never reaches them.tests/shared/rstack-editor.tsadds the build step and the E2E step; the stack files are thin re-exports.rstestfor the rstest suite,smoke+lintfor rslint.RSTACK_E2E_TOOLCHAIN_OVERRIDESmakes the editor pack each package into a tarball and install it into every fixture, so the E2E runs the unreleased build. Rslint needs@rslint/coreplus the host@rslint/native-<tuple>fromgetRslintPackages(the platform tuple is never hardcoded); the rstest core is found by package name through the cachedgetMonorepoPackagesscan.VSCODE_TEST_CACHE_PATHkeeps the VS Code download outside the consumer, becauserunInReporunsgit clean -fdxqin it before each suite.utils.execanow mergesoptions.envinto thesetupEnvironmentsnapshot instead of replacing it, so the E2E command keepsCI,ECOSYSTEM_CI,NODE_OPTIONSandTURBO_FORCE.Prepare Rslint Bindinggoes 60 → 90 min after measuring 36–48 min in the last runs.Evidence
Before: the
rstack-editorsuites rantest:unit+lintonly; the E2E slices were never exercised against the unreleased toolchain.After: selected runs on this branch with
suiteRef=ci/eco-e2e:success; all 4 fixtures reported
resolved through .toolchain; main and dependency-recovery launches exited 0; E2E step 62 s, job ~2 min.success; 6 fixtures ×
@rslint/core+@rslint/native-linux-x64-gnu@0.10.0-fresh.41961300325.1resolved through.toolchain;[smoke] OK; 21 lint launches exited 0; E2E step 324 s, suite job 6.7 min.prepare-bindingtook 48 min, unchanged.Merge Danger
Door: two-way
Reverting removes the E2E step and the timeout changes; no consumer behavior is affected.
Blast Radius: ecosystem-ci
Only the
rstack-editorsuites in the rstest and rslint workflows change. Both stacks gain a VS Code download and a few minutes of E2E time per suite job; the 30 min rstest budget covers the observed 2 min, and the 90 min binding budget covers the observed 36–48 min.Depends on rstackjs/rstack-editor#126 (merged).