cokit-cli 1.0.8 → 1.2.0

package/skills/ck-shopify/scripts/tests/test_shopify_init.py

@@ -1,385 +0,0 @@
-"""
-Tests for shopify_init.py
-
-Run with: pytest test_shopify_init.py -v --cov=shopify_init --cov-report=term-missing
-"""
-
-import os
-import sys
-import json
-import pytest
-import subprocess
-from pathlib import Path
-from unittest.mock import Mock, patch, mock_open, MagicMock
-
-# Add parent directory to path
-sys.path.insert(0, str(Path(__file__).parent.parent))
-
-from shopify_init import EnvLoader, EnvConfig, ShopifyInitializer
-
-
-class TestEnvLoader:
-    """Test EnvLoader class."""
-
-    def test_load_env_file_success(self, tmp_path):
-        """Test loading valid .env file."""
-        env_file = tmp_path / ".env"
-        env_file.write_text("""
-SHOPIFY_API_KEY=test_key
-SHOPIFY_API_SECRET=test_secret
-SHOP_DOMAIN=test.myshopify.com
-# Comment line
-SCOPES=read_products,write_products
-""")
-
-        result = EnvLoader.load_env_file(env_file)
-
-        assert result['SHOPIFY_API_KEY'] == 'test_key'
-        assert result['SHOPIFY_API_SECRET'] == 'test_secret'
-        assert result['SHOP_DOMAIN'] == 'test.myshopify.com'
-        assert result['SCOPES'] == 'read_products,write_products'
-
-    def test_load_env_file_with_quotes(self, tmp_path):
-        """Test loading .env file with quoted values."""
-        env_file = tmp_path / ".env"
-        env_file.write_text("""
-SHOPIFY_API_KEY="test_key"
-SHOPIFY_API_SECRET='test_secret'
-""")
-
-        result = EnvLoader.load_env_file(env_file)
-
-        assert result['SHOPIFY_API_KEY'] == 'test_key'
-        assert result['SHOPIFY_API_SECRET'] == 'test_secret'
-
-    def test_load_env_file_nonexistent(self, tmp_path):
-        """Test loading non-existent .env file."""
-        result = EnvLoader.load_env_file(tmp_path / "nonexistent.env")
-        assert result == {}
-
-    def test_load_env_file_invalid_format(self, tmp_path):
-        """Test loading .env file with invalid lines."""
-        env_file = tmp_path / ".env"
-        env_file.write_text("""
-VALID_KEY=value
-INVALID_LINE_NO_EQUALS
-ANOTHER_VALID=test
-""")
-
-        result = EnvLoader.load_env_file(env_file)
-
-        assert result['VALID_KEY'] == 'value'
-        assert result['ANOTHER_VALID'] == 'test'
-        assert 'INVALID_LINE_NO_EQUALS' not in result
-
-    def test_get_env_paths(self, tmp_path):
-        """Test getting .env file paths."""
-        # Create directory structure
-        claude_dir = tmp_path / ".claude"
-        skills_dir = claude_dir / "skills"
-        skill_dir = skills_dir / "shopify"
-
-        skill_dir.mkdir(parents=True)
-
-        # Create .env files
-        (skill_dir / ".env").write_text("SKILL=1")
-        (skills_dir / ".env").write_text("SKILLS=1")
-        (claude_dir / ".env").write_text("CLAUDE=1")
-
-        paths = EnvLoader.get_env_paths(skill_dir)
-
-        assert len(paths) == 3
-        assert skill_dir / ".env" in paths
-        assert skills_dir / ".env" in paths
-        assert claude_dir / ".env" in paths
-
-    def test_load_config_priority(self, tmp_path, monkeypatch):
-        """Test configuration loading priority."""
-        skill_dir = tmp_path / "skill"
-        skills_dir = tmp_path
-        claude_dir = tmp_path.parent
-
-        skill_dir.mkdir(parents=True)
-
-        # Create .env files with different values
-        (skill_dir / ".env").write_text("SHOPIFY_API_KEY=skill_key")
-        (skills_dir / ".env").write_text("SHOPIFY_API_KEY=skills_key\nSHOP_DOMAIN=skills.myshopify.com")
-
-        # Override with process env
-        monkeypatch.setenv("SHOPIFY_API_KEY", "process_key")
-
-        config = EnvLoader.load_config(skill_dir)
-
-        # Process env should win
-        assert config.shopify_api_key == "process_key"
-        # Shop domain from skills/.env
-        assert config.shop_domain == "skills.myshopify.com"
-
-    def test_load_config_no_files(self, tmp_path):
-        """Test configuration loading with no .env files."""
-        config = EnvLoader.load_config(tmp_path)
-
-        assert config.shopify_api_key is None
-        assert config.shopify_api_secret is None
-        assert config.shop_domain is None
-        assert config.scopes is None
-
-
-class TestShopifyInitializer:
-    """Test ShopifyInitializer class."""
-
-    @pytest.fixture
-    def config(self):
-        """Create test config."""
-        return EnvConfig(
-            shopify_api_key="test_key",
-            shopify_api_secret="test_secret",
-            shop_domain="test.myshopify.com",
-            scopes="read_products,write_products"
-        )
-
-    @pytest.fixture
-    def initializer(self, config):
-        """Create initializer instance."""
-        return ShopifyInitializer(config)
-
-    def test_prompt_with_default(self, initializer):
-        """Test prompt with default value."""
-        with patch('builtins.input', return_value=''):
-            result = initializer.prompt("Test", "default_value")
-            assert result == "default_value"
-
-    def test_prompt_with_input(self, initializer):
-        """Test prompt with user input."""
-        with patch('builtins.input', return_value='user_input'):
-            result = initializer.prompt("Test", "default_value")
-            assert result == "user_input"
-
-    def test_select_option_valid(self, initializer):
-        """Test select option with valid choice."""
-        options = ['app', 'extension', 'theme']
-        with patch('builtins.input', return_value='2'):
-            result = initializer.select_option("Choose", options)
-            assert result == 'extension'
-
-    def test_select_option_invalid_then_valid(self, initializer):
-        """Test select option with invalid then valid choice."""
-        options = ['app', 'extension']
-        with patch('builtins.input', side_effect=['5', 'invalid', '1']):
-            result = initializer.select_option("Choose", options)
-            assert result == 'app'
-
-    def test_check_cli_installed_success(self, initializer):
-        """Test CLI installed check - success."""
-        mock_result = Mock()
-        mock_result.returncode = 0
-
-        with patch('subprocess.run', return_value=mock_result):
-            assert initializer.check_cli_installed() is True
-
-    def test_check_cli_installed_failure(self, initializer):
-        """Test CLI installed check - failure."""
-        with patch('subprocess.run', side_effect=FileNotFoundError):
-            assert initializer.check_cli_installed() is False
-
-    def test_create_app_config(self, initializer, tmp_path):
-        """Test creating app configuration file."""
-        initializer.create_app_config(tmp_path, "test-app", "read_products")
-
-        config_file = tmp_path / "shopify.app.toml"
-        assert config_file.exists()
-
-        content = config_file.read_text()
-        assert 'name = "test-app"' in content
-        assert 'scopes = "read_products"' in content
-        assert 'client_id = "test_key"' in content
-
-    def test_create_extension_config(self, initializer, tmp_path):
-        """Test creating extension configuration file."""
-        initializer.create_extension_config(tmp_path, "test-ext", "checkout")
-
-        config_file = tmp_path / "shopify.extension.toml"
-        assert config_file.exists()
-
-        content = config_file.read_text()
-        assert 'name = "test-ext"' in content
-        assert 'purchase.checkout.block.render' in content
-
-    def test_create_extension_config_admin_action(self, initializer, tmp_path):
-        """Test creating admin action extension config."""
-        initializer.create_extension_config(tmp_path, "admin-ext", "admin_action")
-
-        config_file = tmp_path / "shopify.extension.toml"
-        content = config_file.read_text()
-        assert 'admin.product-details.action.render' in content
-
-    def test_create_readme(self, initializer, tmp_path):
-        """Test creating README file."""
-        initializer.create_readme(tmp_path, "app", "Test App")
-
-        readme_file = tmp_path / "README.md"
-        assert readme_file.exists()
-
-        content = readme_file.read_text()
-        assert '# Test App' in content
-        assert 'shopify app dev' in content
-
-    @patch('builtins.input')
-    @patch('builtins.print')
-    def test_init_app(self, mock_print, mock_input, initializer, tmp_path, monkeypatch):
-        """Test app initialization."""
-        monkeypatch.chdir(tmp_path)
-
-        # Mock user inputs
-        mock_input.side_effect = ['my-app', 'read_products,write_products']
-
-        initializer.init_app()
-
-        # Check directory created
-        app_dir = tmp_path / "my-app"
-        assert app_dir.exists()
-
-        # Check files created
-        assert (app_dir / "shopify.app.toml").exists()
-        assert (app_dir / "README.md").exists()
-        assert (app_dir / "package.json").exists()
-
-        # Check package.json content
-        package_json = json.loads((app_dir / "package.json").read_text())
-        assert package_json['name'] == 'my-app'
-        assert 'dev' in package_json['scripts']
-
-    @patch('builtins.input')
-    @patch('builtins.print')
-    def test_init_extension(self, mock_print, mock_input, initializer, tmp_path, monkeypatch):
-        """Test extension initialization."""
-        monkeypatch.chdir(tmp_path)
-
-        # Mock user inputs: type selection (1 = checkout), name
-        mock_input.side_effect = ['1', 'my-extension']
-
-        initializer.init_extension()
-
-        # Check directory and files created
-        ext_dir = tmp_path / "my-extension"
-        assert ext_dir.exists()
-        assert (ext_dir / "shopify.extension.toml").exists()
-        assert (ext_dir / "README.md").exists()
-
-    @patch('builtins.input')
-    @patch('builtins.print')
-    def test_init_theme(self, mock_print, mock_input, initializer):
-        """Test theme initialization."""
-        mock_input.return_value = 'my-theme'
-
-        # Should just print instructions
-        initializer.init_theme()
-
-        # Verify print was called (instructions shown)
-        assert mock_print.called
-
-    @patch('builtins.print')
-    def test_run_no_cli(self, mock_print, initializer):
-        """Test run when CLI not installed."""
-        with patch.object(initializer, 'check_cli_installed', return_value=False):
-            with pytest.raises(SystemExit) as exc_info:
-                initializer.run()
-            assert exc_info.value.code == 1
-
-    @patch.object(ShopifyInitializer, 'check_cli_installed', return_value=True)
-    @patch.object(ShopifyInitializer, 'init_app')
-    @patch('builtins.input')
-    @patch('builtins.print')
-    def test_run_app_selected(self, mock_print, mock_input, mock_init_app, mock_cli_check, initializer):
-        """Test run with app selection."""
-        mock_input.return_value = '1'  # Select app
-
-        initializer.run()
-
-        mock_init_app.assert_called_once()
-
-    @patch.object(ShopifyInitializer, 'check_cli_installed', return_value=True)
-    @patch.object(ShopifyInitializer, 'init_extension')
-    @patch('builtins.input')
-    @patch('builtins.print')
-    def test_run_extension_selected(self, mock_print, mock_input, mock_init_ext, mock_cli_check, initializer):
-        """Test run with extension selection."""
-        mock_input.return_value = '2'  # Select extension
-
-        initializer.run()
-
-        mock_init_ext.assert_called_once()
-
-
-class TestMain:
-    """Test main function."""
-
-    @patch('shopify_init.ShopifyInitializer')
-    @patch('shopify_init.EnvLoader')
-    def test_main_success(self, mock_loader, mock_initializer):
-        """Test main function success path."""
-        from shopify_init import main
-
-        mock_config = Mock()
-        mock_loader.load_config.return_value = mock_config
-
-        mock_init_instance = Mock()
-        mock_initializer.return_value = mock_init_instance
-
-        with patch('builtins.print'):
-            main()
-
-        mock_init_instance.run.assert_called_once()
-
-    @patch('shopify_init.ShopifyInitializer')
-    @patch('sys.exit')
-    def test_main_keyboard_interrupt(self, mock_exit, mock_initializer):
-        """Test main function with keyboard interrupt."""
-        from shopify_init import main
-
-        mock_initializer.return_value.run.side_effect = KeyboardInterrupt
-
-        with patch('builtins.print'):
-            main()
-
-        mock_exit.assert_called_with(0)
-
-    @patch('shopify_init.ShopifyInitializer')
-    @patch('sys.exit')
-    def test_main_exception(self, mock_exit, mock_initializer):
-        """Test main function with exception."""
-        from shopify_init import main
-
-        mock_initializer.return_value.run.side_effect = Exception("Test error")
-
-        with patch('builtins.print'):
-            main()
-
-        mock_exit.assert_called_with(1)
-
-
-class TestEnvConfig:
-    """Test EnvConfig dataclass."""
-
-    def test_env_config_defaults(self):
-        """Test EnvConfig default values."""
-        config = EnvConfig()
-
-        assert config.shopify_api_key is None
-        assert config.shopify_api_secret is None
-        assert config.shop_domain is None
-        assert config.scopes is None
-
-    def test_env_config_with_values(self):
-        """Test EnvConfig with values."""
-        config = EnvConfig(
-            shopify_api_key="key",
-            shopify_api_secret="secret",
-            shop_domain="test.myshopify.com",
-            scopes="read_products"
-        )
-
-        assert config.shopify_api_key == "key"
-        assert config.shopify_api_secret == "secret"
-        assert config.shop_domain == "test.myshopify.com"
-        assert config.scopes == "read_products"

package/skills/ck-skill-creator/SKILL.md

@@ -1,263 +0,0 @@
----
-name: 'ck-skill-creator'
-description: 'Guide for creating effective skills, adding skill references, skill scripts or optimizing existing skills. This skill should be used when users want to create a new skill (or update an existing skill) that extends Claude\'s capabilities with specialized knowledge, workflows, frameworks, libraries or plugins usage, or API and tool integrations.'
----
-
-# Skill Creator
-
-This skill provides guidance for creating effective skills.
-
-## About Skills
-
-Skills are modular, self-contained packages that extend Claude's capabilities by providing
-specialized knowledge, workflows, and tools. Think of them as "onboarding guides" for specific
-domains or tasks—they transform Claude from a general-purpose agent into a specialized agent
-equipped with procedural knowledge that no model can fully possess.
-
-**IMPORTANT:**
-- Skills are not documentation, they are practical instructions for GitHub Copilot to use the tools, packages, plugins or APIs to achieve the tasks.
-- Each skill teaches GitHub Copilot how to perform a specific development task, not what a tool does.
-- GitHub Copilot can activate multiple skills automatically to achieve the user's request.
-
-### What Skills Provide
-
-1. Specialized workflows - Multi-step procedures for specific domains
-2. Tool integrations - Instructions for working with specific file formats or APIs
-3. Domain expertise - Company-specific knowledge, schemas, business logic
-4. Bundled resources - Scripts, references, and assets for complex and repetitive tasks
-
-### Anatomy of a Skill
-
-Every skill consists of a required SKILL.md file and optional bundled resources:
-
-```
-
-└── skill-name/
-    ├── SKILL.md (required)
-    │   ├── YAML frontmatter metadata (required)
-    │   │   ├── name: (required)
-    │   │   ├── description: (required)
-    │   │   ├── license: (optional)
-    │   │   └── version: (optional)
-    │   └── Markdown instructions (required)
-    └── Bundled Resources (optional)
-        ├── scripts/          - Executable code (Python/Bash/etc.)
-        ├── references/       - Documentation intended to be loaded into context as needed
-        └── assets/           - Files used in output (templates, icons, fonts, etc.)
-```
-
-#### Requirements (**IMPORTANT**)
-
-- Skill should be combined into specific topics, for example: `cloudflare`, `cloudflare-r2`, `cloudflare-workers`, `docker`, `gcloud` should be combined into `devops`
-- `SKILL.md` should be **less than 100 lines** and include the references of related markdown files and scripts.
-- Each script or referenced markdown file should be also **less than 100 lines**, remember that you can always split them into multiple files (**progressive disclosure** principle).
-- Descriptions in metadata of `SKILL.md` files should be both concise and still contains enough usecases of the references and scripts, this will help skills can be activated automatically during the implementation process of GitHub Copilot.
-- **Referenced markdowns**:
-  - Sacrifice grammar for the sake of concision when writing these files.
-  - Can reference other markdown files or scripts as well.
-- **Referenced scripts**:
-  - Prefer nodejs or python scripts instead of bash script, because bash scripts are not well-supported on Windows.
-  - If you're going to write python scripts, make sure you have `requirements.txt`
-  - Make sure scripts respect `.env` file follow this order: `process.env` >  >  >
-  - Create `.env.example` files to show the required environment variables.
-  - Always write tests for these scripts.
-
-**IMPORTANT:**
-- Always keep in mind that `SKILL.md` and reference files should be token consumption efficient, so that **progressive disclosure** can be leveraged at best.
-- `SKILL.md` should be **less than 100 lines**
-- Referenced markdown files should be also **less than 100 lines**, remember that you can always split them into multiple files (**progressive disclosure** principle).
-- Referenced scripts: no limit on length, just make sure it works, no compile issues, no runtime issues, no dependencies issues, no environment issues, no platform issues.
-
-**Why?**
-Better **context engineering**: leverage **progressive disclosure** technique of Agent Skills, when agent skills are activated, GitHub Copilot will consider to load only relevant files into the context, instead of reading all long `SKILL.md` as before.
-
-#### SKILL.md (required)
-
-**File name:** `SKILL.md` (uppercase)
-**File size:** Under 100 lines, if you need more, plit it to multiple files in `references` folder.
-`SKILL.md` is always short and concise, straight to the point, treat it as a quick reference guide.
-
-**Metadata Quality:** The `name` and `description` in YAML frontmatter determine when Claude will use the skill. Be specific about what the skill does and when to use it. Use the third-person (e.g. "This skill should be used when..." instead of "Use this skill when...").
-
-#### Bundled Resources (optional)
-
-##### Scripts (`scripts/`)
-
-Executable code (Python/Bash/etc.) for tasks that require deterministic reliability or are repeatedly rewritten.
-
-- **When to include**: When the same code is being rewritten repeatedly or deterministic reliability is needed
-- **Example**: `scripts/rotate_pdf.py` for PDF rotation tasks
-- **Benefits**: Token efficient, deterministic, may be executed without loading into context
-- **Note**: Scripts may still need to be read by Claude for patching or environment-specific adjustments
-
-**IMPORTANT:**
-- Write tests for scripts.
-- Run tests and make sure it works, if tests fail, fix them and run tests again, repeat until tests pass.
-- Run scripts manually with some usecases to make sure it works.
-- Make sure scripts respect `.env` file follow this order: `process.env` >  >  >
-
-##### References (`references/`)
-
-Documentation and reference material intended to be loaded as needed into context to inform Claude's process and thinking.
-
-- **When to include**: For documentation that Claude should reference while working
-- **Examples**: `references/finance.md` for financial schemas, `references/mnda.md` for company NDA template, `references/policies.md` for company policies, `references/api_docs.md` for API specifications
-- **Use cases**: Database schemas, API documentation, domain knowledge, company policies, detailed workflow guides
-- **Benefits**: Keeps SKILL.md lean, loaded only when Claude determines it's needed
-- **Best practice**: If files are large (>100 lines), include grep search patterns in SKILL.md
-- **Avoid duplication**: Information should live in either SKILL.md or references files, not both. Prefer references files for detailed information unless it's truly core to the skill—this keeps SKILL.md lean while making information discoverable without hogging the context window. Keep only essential procedural instructions and workflow guidance in SKILL.md; move detailed reference material, schemas, and examples to references files.
-
-**IMPORTANT:**
-- Referenced markdown files should be also **less than 100 lines**, remember that you can always split them into multiple files (**progressive disclosure** principle).
-- Referenced markdown files are practical instructions for GitHub Copilot to use the tools, packages, plugins or APIs to achieve the tasks.
-- Each skill teaches GitHub Copilot how to perform a specific development task, not what a tool does.
-
-##### Assets (`assets/`)
-
-Files not intended to be loaded into context, but rather used within the output Claude produces.
-
-- **When to include**: When the skill needs files that will be used in the final output
-- **Examples**: `assets/logo.png` for brand assets, `assets/slides.pptx` for PowerPoint templates, `assets/frontend-template/` for HTML/React boilerplate, `assets/font.ttf` for typography
-- **Use cases**: Templates, images, icons, boilerplate code, fonts, sample documents that get copied or modified
-- **Benefits**: Separates output resources from documentation, enables Claude to use files without loading them into context
-
-### Progressive Disclosure Design Principle
-
-Skills use a three-level loading system to manage context efficiently:
-
-1. **Metadata (name + description)** - Always in context (~100 words)
-2. **SKILL.md body** - When skill triggers (<5k words)
-3. **Bundled resources** - As needed by Claude (Unlimited*)
-
-*Unlimited because scripts can be executed without reading into context window.
-
-## Skill Creation Process
-
-To create a skill, follow the "Skill Creation Process" in order, skipping steps only if there is a clear reason why they are not applicable.
-
-### Step 1: Understanding the Skill with Concrete Examples
-
-Skip this step only when the skill's usage patterns are already clearly understood. It remains valuable even when working with an existing skill.
-
-To create an effective skill, clearly understand concrete examples of how the skill will be used. This understanding can come from either direct user examples or generated examples that are validated with user feedback.
-
-For example, when building an image-editor skill, relevant questions include:
-
-- "What functionality should the image-editor skill support? Editing, rotating, anything else?"
-- "Can you give some examples of how this skill would be used?"
-- "I can imagine users asking for things like 'Remove the red-eye from this image' or 'Rotate this image'. Are there other ways you imagine this skill being used?"
-- "What would a user say that should trigger this skill?"
-
-To avoid overwhelming users, avoid asking too many questions in a single message. Start with the most important questions and follow up as needed for better effectiveness.
-
-Conclude this step when there is a clear sense of the functionality the skill should support.
-
-### Step 2: Planning the Reusable Skill Contents
-
-To turn concrete examples into an effective skill, analyze each example by:
-
-1. Considering how to execute on the example from scratch
-2. Identifying what scripts, references, and assets would be helpful when executing these workflows repeatedly
-
-Example: When building a `pdf-editor` skill to handle queries like "Help me rotate this PDF," the analysis shows:
-
-1. Rotating a PDF requires re-writing the same code each time
-2. A `scripts/rotate_pdf.py` script would be helpful to store in the skill
-
-Example: When designing a `frontend-webapp-builder` skill for queries like "Build me a todo app" or "Build me a dashboard to track my steps," the analysis shows:
-
-1. Writing a frontend webapp requires the same boilerplate HTML/React each time
-2. An `assets/hello-world/` template containing the boilerplate HTML/React project files would be helpful to store in the skill
-
-Example: When building a `big-query` skill to handle queries like "How many users have logged in today?" the analysis shows:
-
-1. Querying BigQuery requires re-discovering the table schemas and relationships each time
-2. A `references/schema.md` file documenting the table schemas would be helpful to store in the skill
-
-To establish the skill's contents, analyze each concrete example to create a list of the reusable resources to include: scripts, references, and assets.
-
-- Make sure scripts respect `.env` file follow this order: `process.env` >  >  >
-- Make sure scripts have tests.
-
-### Step 3: Initializing the Skill
-
-At this point, it is time to actually create the skill.
-
-Skip this step only if the skill being developed already exists, and iteration or packaging is needed. In this case, continue to the next step.
-
-When creating a new skill from scratch, always run the `init_skill.py` script. The script conveniently generates a new template skill directory that automatically includes everything a skill requires, making the skill creation process much more efficient and reliable.
-
-Usage:
-
-```bash
-scripts/init_skill.py <skill-name> --path <output-directory>
-```
-
-The script:
-
-- Creates the skill directory at the specified path
-- Generates a SKILL.md template with proper frontmatter and TODO placeholders
-- Creates example resource directories: `scripts/`, `references/`, and `assets/`
-- Adds example files in each directory that can be customized or deleted
-
-After initialization, customize or remove the generated SKILL.md and example files as needed.
-
-### Step 4: Edit the Skill
-
-When editing the (newly-generated or existing) skill, remember that the skill is being created for another instance of Claude to use. Focus on including information that would be beneficial and non-obvious to Claude. Consider what procedural knowledge, domain-specific details, or reusable assets would help another Claude instance execute these tasks more effectively.
-
-#### Start with Reusable Skill Contents
-
-To begin implementation, start with the reusable resources identified above: `scripts/`, `references/`, and `assets/` files. Note that this step may require user input. For example, when implementing a `brand-guidelines` skill, the user may need to provide brand assets or templates to store in `assets/`, or documentation to store in `references/`.
-
-Also, delete any example files and directories not needed for the skill. The initialization script creates example files in `scripts/`, `references/`, and `assets/` to demonstrate structure, but most skills won't need all of them.
-
-#### Update SKILL.md
-
-**Writing Style:** Write the entire skill using **imperative/infinitive form** (verb-first instructions), not second person. Use objective, instructional language (e.g., "To accomplish X, do Y" rather than "You should do X" or "If you need to do X"). This maintains consistency and clarity for AI consumption.
-
-To complete SKILL.md, answer the following questions:
-
-1. What is the purpose of the skill, in a few sentences?
-2. When should the skill be used?
-3. In practice, how should Claude use the skill? All reusable skill contents developed above should be referenced so that Claude knows how to use them.
-
-### Step 5: Packaging a Skill
-
-Once the skill is ready, it should be packaged into a distributable zip file that gets shared with the user. The packaging process automatically validates the skill first to ensure it meets all requirements:
-
-```bash
-scripts/package_skill.py <path/to/skill-folder>
-```
-
-Optional output directory specification:
-
-```bash
-scripts/package_skill.py <path/to/skill-folder> ./dist
-```
-
-The packaging script will:
-
-1. **Validate** the skill automatically, checking:
-   - YAML frontmatter format and required fields
-   - Skill naming conventions and directory structure
-   - Description completeness and quality
-   - File organization and resource references
-
-2. **Package** the skill if validation passes, creating a zip file named after the skill (e.g., `my-skill.zip`) that includes all files and maintains the proper directory structure for distribution.
-
-If validation fails, the script will report the errors and exit without creating a package. Fix any validation errors and run the packaging command again.
-
-### Step 6: Iterate
-
-After testing the skill, users may request improvements. Often this happens right after using the skill, with fresh context of how the skill performed.
-
-**Iteration workflow:**
-1. Use the skill on real tasks
-2. Notice struggles or inefficiencies
-3. Identify how SKILL.md or bundled resources should be updated
-4. Implement changes and test again
-
-## References
-- [Agent Skills Spec](https://github.com/anthropics/awesome-claude-prompts/blob/main/CONTRIBUTING.md)
-- [GitHub Copilot Documentation](https://docs.github.com/en/copilot)