Repository navigation
feat(setup): card-based skill library with linkable sources and auto-update - #78
Conversation
Phone widths hide the Cards/Table switch (zero-size) and collapse the skills chipbar (display:none); the audit clicked both raw. Use the visible skills.agents-all rendering, seed Table view per combo (fresh profile), expand collapsed system groups before rows-vs-cards comparison, and wait attached for hidden table rows.
|
Hi @0libote, thank you for this! The skill inspector is a really nice addition. One click shows a skill's agents, source and location, it works well from the keyboard, and focus comes back to the card when it closes. The Skills switch also fits naturally next to the other categories under Automatic updates. And thanks for I don't think it's ready to merge yet, though. Most of it comes down to two things: automatic updates can overwrite local edits, and the OpenCode change. Before merging
DesignThe cards are easy to scan, but as the default view they drop a lot of what the table showed:
My suggestion would be to keep Table (and the phone list) as the default, with Cards as an option. Alternatively, cards could show their folder and any per-agent exceptions, grouped by folder. Smaller notes
None of this is a "no". The direction makes sense, and the inspector in particular is a clear improvement. Happy to help with any of these, including splitting out the OpenCode part, if that's useful. Thanks again! |
Phones hide the Cards/Table switch (zero-size) and the skills chipbar (display:none); the audit clicked both raw. Use the visible skills.agents-all rendering, assert cards (not table rows) on phones with self-calibrating counts from the tab and chip badges, wait attached for hidden table rows, and expand groups through whichever group toggles are visible.
…rces Idle auto-installs now skip skills with local changes instead of overwriting them; the update stays available in Updates for an explicit review. Linking a source defaults to manual-only, and linking a single differently named skill requires confirmation. Tracked replacements carry auto-update flags over instead of resetting them. Also fixes footer/docs wording and em dashes.
The OpenCode 2.x watcher fix changes persistence for existing containers and belongs in its own PR. This reverts de53d31, a3851c1 and caae2ab; the reworked approach will follow separately. Revert "fix(opencode): use real config dirs, strip invalid skills key" Revert "fix(opencode): tolerate unset T3_INFRA_NODE under set -u in persist test" Revert "fix(tests): isolate OpenCode config resolution from runner env"
|
Done, split as suggested: the OpenCode change is reverted out of here and reworked as #80, which exports On the blocking items:
Smaller notes are fixed too: em dashes removed from the new strings, the library footer reads "Click a card"/"Click a row" per view, and the docs no longer describe a View menu or card menu. Two things I deliberately left: the cards-vs-table default (your call — happy to flip the default or add folder/exception info to cards as a follow-up), and the CI is running on both heads (pushed just now); will report back on green. |
Cards are the default view, so the library readiness gate and the skill-row click targets must also match cards, not just table and phone-list rows.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Follow-up to the review on dizys#78. Library: - Table stays the default. Cards are an option beside View, keep the folder groups (so same-named skills stay apart and bundled skills stay collapsed), and name each agent that limits a skill. - Phones keep the grouped list in either layout. - Agent scope and bundled skills go back to the View menu, so there is no bare +N toggle or amber Bundled button. - The inspector drops the Access & usage section, which repeated the agent list, file count and auto-update setting above it, and the Link source box gets proper padding. Auto-updates: - The idle pass installs only what the scheduled check found, instead of fetching every opted-in source on each tick. - It frees each review slot afterwards, so Install from source stays available. - A skill linked from a copy that already differed from its source waits for one reviewed update, since Setup can't tell an older copy from a customized one. - A reinstall from a different source starts opted out. - Turning on only the Skills switch no longer logs "Auto-update turned off". Docs describe the layout switch, linking and the two opt-ins; the audits are back to main's View menu with a new Cards check. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The old Skills view put skills in rows and providers in columns. Locating a skill or comparing agent access meant scanning a wide matrix. Repository tracking and manual update checks already worked for skills installed from a source; the gap was linking an existing editable skill after installation and choosing automatic updates per skill.
The default view is now one card per installed skill. Each card shows its name, description and how many agents can use it. Click a card to see its locations, agent access and source details. Skills with the same name from different locations remain separate entries.
The inspector lets you link an existing editable skill to a repository, opt it into automatic updates, or unlink the source. Updates → Automatic updates now has a Skills switch: it checks tracked sources on schedule and installs updates for opted-in skills when the server is idle. Manual checks remain available. Cards/Table, agent scope and bundled-skill visibility are inline controls that remember their settings.
All comparison images use the same 1440×900 frame.
Skill inspector
The same
frontend-designskill appears in both panels. Source tracking was already present; the updated inspector adds per-skill auto-update and unlink controls.Validation:
node --test tests/*.test.mjs: 1070 pass, 0 fail (updated the View-menu test for the inline controls and the auto-update view test for the new policy).Follow-up before merge
I merged main into this branch and finished the open points from the review in 76cd06e. Some of the description above no longer matches. Table is the default again, and agent scope and bundled skills are back in the View menu.
Library
Auto-updates
Docs cover the layout switch, linking, and the two opt-ins. The browser audits are back to main's View menu, with a new Cards check.
Screenshots (main vs. this branch, same fixtures, light)
Checks
node --test tests/*.test.mjs: 1089 pass, 0 fail.skills-ui-audit(all six layouts, both themes, and the tab frames),skills-hooks-ui-audit,skills-update-ui-audit,auto-update-ui-auditandharness-ui-auditpass against the fixture console.The OpenCode config change is split out as #80 and isn't part of this PR.
🤖 Generated with Claude Code