Fix skills install flow and improve bundle export handling#19
Merged
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates curated skill installation and bundle export behavior, with supporting documentation for a future binary-safe .a7bundle format.
Changes:
- Updates the curated skills registry URLs and adds a local registry fallback.
- Adjusts package skill association scanning and bundle export dependency collection.
- Adds tests for package helper association handling, flat tool dependency export, and binary rejection.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
docs/registry.json |
Refreshes curated skill entries and raw install URLs. |
docs/a7bundle-v2.md |
Adds the proposed v2 bundle/archive design. |
crates/web/src/api.rs |
Adds registry fallback and changes package skill association extraction. |
crates/web/frontend/src/views/SkillsView.vue |
Shows import/install status in the Browse tab. |
crates/sharing/src/lib.rs |
Adds text-only asset reading errors and same-stem flat tool dependency export. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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\n- fix curated skills registry install URLs and surface install status in the Browse tab\n- remove ghost unresolved refs from package helper files in export UI associations\n- export same-stem sibling dependency folders for flat tools\n- reject binary assets clearly for current .a7bundle v1 text-only export path\n- add .a7bundle v2 and artifact versioning design doc\n\n## Verification\n- cargo check -p agent007-sharing -p agent007-web\n- cargo test -p agent007-web package_skill_helper_files_do_not_become_ghost_script_refs -- --nocapture\n- cargo test -p agent007-sharing builder_flat_tool_export_includes_same_stem_dependency_dir -- --nocapture\n- cargo test -p agent007-sharing builder_rejects_binary_tool_assets_with_clear_error -- --nocapture