Repository navigation
tsk setup: name both installed skills (T331) - #264
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
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.
T331.
tsk setupprinted only…/tsk-cli/SKILL.mdper agent, so it read as if only one skill was installed.tsk-coordinator/SKILL.mdwas written beside it but never shown.Change
tsk setup agents [--yes], interactivetsk setup,tsk setup <agent>,--skill-dir: one row per agent,<skills folder> (tsk-cli, tsk-coordinator). The per-agent table stays; rows are about as wide as before (folder is 17 chars shorter than the file path).Install or update the tsk skills (tsk-cli, tsk-coordinator) for cursor? [y/N].tsk setuplisting shows each agent's skills folder plus one line naming both skills.--json(single and batch): addsskills_rootandskills(both file paths). Existing fields unchanged (pathis still the tsk-cli file).--skill-states/--detected-ids(verified byte-identical shape below), installers, skills (no version bump needed).tsk setup --helpaction rows say "both skills".cli.mdsetup section (both skills, output line, example). CHANGELOG Unreleased → Changed.Proof (release build, isolated
HOME,TSK_STATE_DIR,XDG_*, PATH;$TMP= scratch dir)Files on disk:
sd/,.claude/skills/,.cursor/skills/each holdtsk-cli/SKILL.mdandtsk-coordinator/SKILL.md.Checks
fmt --check, clippy-D warnings,cargo test): pass.cargo clippy --all-targets --target x86_64-pc-windows-gnu -- -D warnings: pass (compile/lint only).userprofile_only_drives_windows_skill_install_and_listing, updated to the new listing and install line): covered by CI Windows Verify only, not run locally.cd site && npm test: 58 pass.skill_dir_writes_full_skill_prints_path_exits_0andsetup_agent_batch_prints_padded_rows_for_each_outcome.Not in this PR
Install the tsk skill for them?,Updated the tsk skill for …) still say "skill" singular; changing them runs the Installer workflow and packaging tests. Separate task if wanted.