feat(skill): add zip skill parsing util

[fang-tech]

AgentScope-Java Version

[The version of AgentScope-Java you are working on, e.g. 1.0.8, check your pom.xml dependency version or run mvn dependency:tree | grep agentscope-parent:pom(only mac/linux)]

Description

[Please describe the background, purpose, changes made, and how to test this PR]

Checklist

Please check the following items before code is ready to be reviewed.

• Code has been formatted with mvn spotless:apply
• All tests are passing (mvn test)
• Javadoc comments are complete and follow project conventions
• Related documentation has been updated (e.g. links, examples, etc.)
• Code is ready for review

[gemini-code-assist]

Summary of Changes

Hello @fang-tech, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request significantly enhances the AgentSkill loading mechanism by introducing the ability to parse skills from compressed zip archives. This feature allows for more organized and portable skill definitions, where a skill and its associated resources can be bundled together. Additionally, it improves the debugging experience by providing a clearer string representation for AgentSkill objects.

Highlights

• New Zip Skill Parsing Utility: Introduced a comprehensive set of createFromZip methods in SkillUtil to parse AgentSkill instances from zip files. These methods support input as byte arrays, file paths, and input streams, allowing for flexible skill loading from packaged sources.
• Robust Zip Validation: The new zip parsing logic includes robust validation to ensure the integrity of skill packages. It enforces a single root directory structure, verifies the presence and correct placement of the SKILL.md file, and handles cases of multiple or incorrectly located SKILL.md entries.
• Enhanced AgentSkill Representation: Added a toString() method to the AgentSkill class, providing a more informative string representation that includes the skill's name, description, content, and source.
• Refactored and Expanded Tests: Existing SkillUtil tests for Markdown parsing have been refactored into a nested class, and extensive new unit tests have been added for the zip parsing functionality. These tests cover various scenarios, including successful parsing, missing SKILL.md, incorrect zip structures, and different input types.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

This pull request introduces a useful feature for parsing agent skills from zip archives. The implementation includes overloads for handling byte arrays, paths, and input streams, which is good for flexibility. The accompanying tests are well-structured and cover various success and failure scenarios, ensuring the new functionality is robust.

My review focuses on a few areas for improvement:

1. Memory Efficiency: The current implementation for handling zip files reads the entire zip file and then each of its entries into memory. This could lead to OutOfMemoryError with large skill packages. I've provided suggestions to improve this by using streams.
2. toString() Implementation: I've suggested an improvement to the new toString() method in AgentSkill to make it more informative and safer by not logging potentially large content.

Overall, this is a solid contribution. Addressing the memory concerns will make it even better.

[Copilot]

Pull request overview

Adds support for creating AgentSkill instances from a zipped “skill package” (containing SKILL.md plus optional resource files), along with expanded unit test coverage for markdown parsing and the new zip-based loading APIs.

Changes:

• Added SkillUtil.createFromZip(...) overloads to load skills from byte[], Path, or InputStream.
• Expanded SkillUtilTest to organize markdown tests and add comprehensive zip package tests.
• Added AgentSkill.toString() implementation.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File	Description
agentscope-core/src/main/java/io/agentscope/core/skill/util/SkillUtil.java	Introduces zip package parsing APIs and supporting helpers.
agentscope-core/src/test/java/io/agentscope/core/skill/SkillUtilTest.java	Adds/organizes tests for markdown parsing and new zip parsing behaviors.
agentscope-core/src/main/java/io/agentscope/core/skill/AgentSkill.java	Adds a toString() implementation for easier debugging/logging.

[codecov]

Codecov Report

❌ Patch coverage is 66.19718% with 24 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
.../java/io/agentscope/core/skill/util/SkillUtil.java	67.14%	14 Missing and 9 partials ⚠️
...main/java/io/agentscope/core/skill/AgentSkill.java	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!