Validate SDK development and ship creator release assets - #4030
Closed
TomCC7 wants to merge 2 commits into
Closed
Conversation
TomCC7
added this pull request to stack #4031
September 9, 2026 19:59
❌ 1 Tests Failed:
View the top 1 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
TomCC7
force-pushed
the
test/installer-acceptance
branch
from
September 9, 2026 23:59
6ad059f to
78e9b88
Compare
This was referenced Sep 10, 2026
Closed
Closed
Closed
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.
Superseded by the replacement Python dimup stack: #4042 → #4043 → #4044 → #4045 → #4046 → #4047. See the onboarding PRD for the current review map.
Adds end-to-end acceptance for SDK development and a versioned curl bootstrap for the one-shot workspace creator. CI builds native creator binaries on Ubuntu 22.04 x86_64/ARM64. Existing DimOS releases receive the binaries, bootstrap, and checksums; Python publishing explicitly downloads only Python artifacts. No release is published by this change.
Acceptance covers both profiles on Ubuntu 22.04/24.04: contributor setup, SDK creation from the checkout's wheel, editable hello execution/tests, source edits, adding a dependency, restoration, and a fresh clone. The hello test uses isolated transport/config/state. Shell tests cover Bash/Zsh activation and direnv load/unload; bootstrap tests cover architecture, checksums, argument forwarding, and cleanup.
Removes the old Bash installer and documents the workspace-first flow. The setup diagram is a small Mermaid generator rather than a browser application.
Local validation: full manipulation acceptance passed; 8 current Rust tests and standalone Clippy passed; 11 lightweight Python/shell/bootstrap tests passed (direnv unavailable locally, exercised in CI); Ruff, focused Mypy, lockfile validation, and native release packaging passed. The repository-wide Clippy hook still traverses an unrelated DimSLAM flake issue; standalone installer Clippy is used.
The workspace-creator CI run for 55ee524a36 passed: all four Ubuntu/profile acceptance jobs, native x86_64/ARM64 release builds, Rust checks, and all 12 lightweight tests including direnv load/unload. The minimal test job explicitly avoids runtime conftest imports and uses importlib collection.
Pending: dedicated AGX Orin/JetPack 6.2 runner provisioning/results and team verification of robot workflows. Separately, the base PR's repository-wide CI hit
test_create_or_attach_splits_owner_and_reader; its automatic retry passed but the original job remained red. This is outside the installer paths. Public curl promotion stays withdrawn until acceptance; existing releases do not contain the new assets automatically.