skills: move and update BrainX install skill#119
Conversation
Reviewer's GuideRelocates the BrainX install skill from docs into the top-level skills directory while updating its workflow to handle explicit uninstall/cleanup routing, stricter release selection using compatibility metadata, and clearer environment and package-scoping rules, plus tightening the BrainX compatibility matrix constraints. Flow diagram for updated BrainX install/uninstall skill routingflowchart TD
A[User BrainX request] --> B{Request type}
B -->|uninstall / remove / cleanup| C[Open references/uninstall-and-cleanup.md]
C --> D[Follow uninstall_and_cleanup workflow]
B -->|install / upgrade / downgrade / migrate / device / compatibility| E{Existing BrainX packages or pinned version?}
E -->|no| F[Resolve latest Python-compatible BrainX release via package index]
F --> G[Confirm device extra and Target BrainX release]
E -->|yes| H[Open references/compatibility-and-release-matching.md]
H --> I[Select one BrainX release from compatibility matrix]
G --> J[Inspect environments and prefer existing suitable venv]
I --> J
J --> K[Present complete specification with Target BrainX release]
K --> L{User confirms?}
L -->|yes| M[Execute install workflow]
L -->|no| N[Revise specification or stop]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- In the 'Select One Release' section, the sentence starting with 'First case, When no BrainX...' is awkwardly capitalized and punctuated; consider simplifying to two sentences (e.g., 'In the first case, when no BrainX...' without the comma) to improve clarity.
- The final note in 'Reference Routing' still instructs to 'use the unpinned quick-install path' for clean installs, but earlier sections now require resolving the exact latest release from package-index metadata; aligning this wording with the updated latest-release resolution behavior would avoid conflicting guidance.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- In the 'Select One Release' section, the sentence starting with 'First case, When no BrainX...' is awkwardly capitalized and punctuated; consider simplifying to two sentences (e.g., 'In the first case, when no BrainX...' without the comma) to improve clarity.
- The final note in 'Reference Routing' still instructs to 'use the unpinned quick-install path' for clean installs, but earlier sections now require resolving the exact latest release from package-index metadata; aligning this wording with the updated latest-release resolution behavior would avoid conflicting guidance.
## Individual Comments
### Comment 1
<location path="skills/brainx-install/SKILL.md" line_range="80" />
<code_context>
## 3. Select One Release
-When no BrainX meta-package or ecosystem component is installed and the user has not requested a specific version, do not open the compatibility reference. Prefer the unpinned official quick-install form that matches the confirmed device target: `BrainX`, `BrainX[cpu]`, `BrainX[cuda12]`, `BrainX[cuda13]`, or `BrainX[tpu]`.
+First case, When no BrainX meta-package or ecosystem component is installed and the user has not requested a specific version, do not open the compatibility reference. Use read-only official package-index metadata with the proposed interpreter to identify the exact latest Python-compatible BrainX release, then report it in the form `latest (vYYYY.M.D)`. For example, report `latest (v2026.7.9)` when `2026.7.9` is the verified current release.
-When BrainX packages are already installed or the user requests a specific pinned version, follow **Reference Routing** at the end of this skill, then apply the matching procedure below.
</code_context>
<issue_to_address>
**nitpick (typo):** Fix capitalization in "First case, When no" for grammatical correctness.
"First case, When no BrainX meta-package..." is grammatically off due to the capitalized "When" after a comma. Please rephrase, e.g. "First case: when no BrainX meta-package..." or "First, when no BrainX meta-package..." to improve flow and capitalization.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| ## 3. Select One Release | ||
|
|
||
| When no BrainX meta-package or ecosystem component is installed and the user has not requested a specific version, do not open the compatibility reference. Prefer the unpinned official quick-install form that matches the confirmed device target: `BrainX`, `BrainX[cpu]`, `BrainX[cuda12]`, `BrainX[cuda13]`, or `BrainX[tpu]`. | ||
| First case, When no BrainX meta-package or ecosystem component is installed and the user has not requested a specific version, do not open the compatibility reference. Use read-only official package-index metadata with the proposed interpreter to identify the exact latest Python-compatible BrainX release, then report it in the form `latest (vYYYY.M.D)`. For example, report `latest (v2026.7.9)` when `2026.7.9` is the verified current release. |
There was a problem hiding this comment.
nitpick (typo): Fix capitalization in "First case, When no" for grammatical correctness.
"First case, When no BrainX meta-package..." is grammatically off due to the capitalized "When" after a comma. Please rephrase, e.g. "First case: when no BrainX meta-package..." or "First, when no BrainX meta-package..." to improve flow and capitalization.
Summary
brainx-installfromdocs/skillsto the repository-levelskillsdirectory, parallel todocsThis supersedes the layout introduced by merged PR #118.
Verification
Summary by Sourcery
Relocate and expand the BrainX install skill to support both installation and uninstall/cleanup workflows with stricter release selection and environment-handling rules.
New Features:
Enhancements:
Documentation: