feat: add GitHub Marketplace readiness, bug fixes, and expanded test coverage

.github/ISSUE_TEMPLATE/bug_report.yml

@@ -0,0 +1,90 @@
+name: Bug Report
+description: Report a bug or unexpected behavior in HF-MS Sync
+title: "[Bug]: "
+labels: ["bug"]
+assignees: []
+
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thanks for taking the time to report a bug! Please fill out the information below.
+
+  - type: textarea
+    id: description
+    attributes:
+      label: What happened?
+      description: A clear description of what the bug is.
+      placeholder: "When I run the sync, it fails with..."
+    validations:
+      required: true
+
+  - type: textarea
+    id: expected
+    attributes:
+      label: What did you expect?
+      description: What should have happened instead?
+    validations:
+      required: true
+
+  - type: textarea
+    id: reproduction
+    attributes:
+      label: Steps to reproduce
+      description: How can we reproduce this issue?
+      placeholder: |
+        1. Create config with...
+        2. Run workflow...
+        3. See error...
+    validations:
+      required: true
+
+  - type: textarea
+    id: config
+    attributes:
+      label: Configuration
+      description: |
+        Your `sync_config.yaml` (redact any sensitive tokens):
+      render: yaml
+
+  - type: textarea
+    id: logs
+    attributes:
+      label: Logs
+      description: |
+        Relevant log output from the GitHub Actions run:
+      render: text
+
+  - type: dropdown
+    id: direction
+    attributes:
+      label: Sync direction
+      options:
+        - hf_to_ms
+        - ms_to_hf
+        - bidirectional
+    validations:
+      required: true
+
+  - type: dropdown
+    id: version
+    attributes:
+      label: Action version
+      options:
+        - v1 (latest)
+        - main branch
+        - specific tag (specify below)
+    validations:
+      required: true
+
+  - type: input
+    id: runner
+    attributes:
+      label: Runner OS
+      placeholder: "ubuntu-latest, ubuntu-22.04, etc."
+
+  - type: textarea
+    id: additional
+    attributes:
+      label: Additional context
+      description: Any other information that might help.

.github/ISSUE_TEMPLATE/feature_request.yml

@@ -0,0 +1,55 @@
+name: Feature Request
+description: Suggest a new feature or enhancement for HF-MS Sync
+title: "[Feature]: "
+labels: ["enhancement"]
+assignees: []
+
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Have an idea for improving HF-MS Sync? We'd love to hear it!
+
+  - type: textarea
+    id: problem
+    attributes:
+      label: Problem or use case
+      description: What problem are you trying to solve? What's the use case?
+      placeholder: "I need to sync models but..."
+    validations:
+      required: true
+
+  - type: textarea
+    id: solution
+    attributes:
+      label: Proposed solution
+      description: How do you think this should be solved?
+    validations:
+      required: true
+
+  - type: textarea
+    id: alternatives
+    attributes:
+      label: Alternatives considered
+      description: Have you considered other approaches?
+
+  - type: dropdown
+    id: category
+    attributes:
+      label: Category
+      options:
+        - New sync direction or mode
+        - Performance improvement
+        - Better error handling
+        - Configuration enhancement
+        - Documentation
+        - CI/CD integration
+        - Other
+    validations:
+      required: true
+
+  - type: textarea
+    id: additional
+    attributes:
+      label: Additional context
+      description: Mockups, links, or any other relevant information.

.github/PULL_REQUEST_TEMPLATE.md

@@ -0,0 +1,39 @@
+## Description
+
+<!-- Briefly describe what this PR does and why -->
+
+## Type of change
+
+- [ ] Bug fix (non-breaking change that fixes an issue)
+- [ ] New feature (non-breaking change that adds functionality)
+- [ ] Breaking change (fix or feature that would cause existing functionality to change)
+- [ ] Documentation update
+- [ ] CI/CD or infrastructure change
+
+## Changes
+
+<!-- List the specific changes made -->
+
+-
+-
+-
+
+## Testing
+
+- [ ] Unit tests pass (`pytest tests/ -v`)
+- [ ] Lint passes (`ruff check src/ tests/`)
+- [ ] Added/updated tests for new functionality
+- [ ] Tested with dry-run mode
+- [ ] Tested E2E with a small model (if applicable)
+
+## Checklist
+
+- [ ] My code follows the project's style guidelines
+- [ ] I have performed a self-review of my code
+- [ ] I have updated the documentation (if applicable)
+- [ ] My changes generate no new warnings
+- [ ] New and existing tests pass with my changes
+
+## Screenshots / Logs
+
+<!-- If applicable, add screenshots or log output -->

.github/workflows/ci.yml

@@ -0,0 +1,151 @@
+name: CI
+
+on:
+  push:
+    branches: [main, master]
+  pull_request:
+    branches: [main, master]
+
+permissions:
+  contents: read
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - uses: actions/setup-python@v5
+        with:
+          python-version: '3.11'
+
+      - name: Install ruff
+        run: pip install ruff
+
+      - name: Ruff check
+        run: ruff check src/ tests/
+
+      - name: Ruff format check
+        run: ruff format --check src/ tests/
+
+  test:
+    name: Test (Python ${{ matrix.python-version }})
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ['3.10', '3.11', '3.12']
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - uses: actions/setup-python@v5
+        with:
+          python-version: ${{ matrix.python-version }}
+          cache: 'pip'
+
+      - name: Install dependencies
+        run: pip install -e ".[dev]"
+
+      - name: Run tests
+        run: pytest tests/ -v --tb=short --cov=src --cov-report=term-missing
+
+  action-validation:
+    name: Validate Action
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - name: Validate action.yml
+        run: |
+          echo "━━━ Checking action.yml ━━━"
+          # action.yml must exist
+          test -f action.yml || { echo "FAIL: action.yml not found"; exit 1; }
+
+          # Must have required fields
+          grep -q "^name:" action.yml || { echo "FAIL: missing name"; exit 1; }
+          grep -q "^description:" action.yml || { echo "FAIL: missing description"; exit 1; }
+          grep -q "^inputs:" action.yml || { echo "FAIL: missing inputs"; exit 1; }
+          grep -q "^outputs:" action.yml || { echo "FAIL: missing outputs"; exit 1; }
+          grep -q "using: 'docker'" action.yml || { echo "FAIL: not a docker action"; exit 1; }
+          grep -q "image: 'Dockerfile'" action.yml || { echo "FAIL: missing image"; exit 1; }
+
+          echo "action.yml validation PASSED"
+
+      - name: Validate Dockerfile
+        run: |
+          echo "━━━ Checking Dockerfile ━━━"
+          test -f Dockerfile || { echo "FAIL: Dockerfile not found"; exit 1; }
+          grep -q "ENTRYPOINT" Dockerfile || { echo "FAIL: missing ENTRYPOINT"; exit 1; }
+          grep -q "entrypoint.sh" Dockerfile || { echo "FAIL: missing entrypoint.sh"; exit 1; }
+
+          echo "Dockerfile validation PASSED"
+
+      - name: Validate entrypoint.sh
+        run: |
+          echo "━━━ Checking entrypoint.sh ━━━"
+          test -f entrypoint.sh || { echo "FAIL: entrypoint.sh not found"; exit 1; }
+          test -x entrypoint.sh || { echo "FAIL: entrypoint.sh not executable"; exit 1; }
+          head -1 entrypoint.sh | grep -q "^#!/bin/bash" || { echo "FAIL: missing shebang"; exit 1; }
+          grep -q "GITHUB_OUTPUT" entrypoint.sh || { echo "FAIL: missing GITHUB_OUTPUT"; exit 1; }
+
+          echo "entrypoint.sh validation PASSED"
+
+      - name: Validate LICENSE
+        run: |
+          test -f LICENSE || { echo "FAIL: LICENSE not found"; exit 1; }
+          echo "LICENSE validation PASSED"
+
+  docker-build:
+    name: Docker Build
+    runs-on: ubuntu-latest
+    needs: [lint, test, action-validation]
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          persist-credentials: false
+
+      - name: Build Docker image
+        run: docker build -t hf-ms-sync:ci .
+
+      - name: Verify Docker image
+        run: |
+          # Verify the image has the entrypoint (override ENTRYPOINT to run cat)
+          docker run --rm --entrypoint cat hf-ms-sync:ci /entrypoint.sh > /dev/null
+          echo "Docker image built and verified"
+
+      - name: Dry-run test in Docker
+        run: |
+          # Use ci_smoke_test.yaml (empty models/datasets) so the full
+          # dry-run flow completes without real API tokens.
+          docker run --rm \
+            -v ${{ github.workspace }}/config:/app/config \
+            -e INPUT_CONFIG=config/ci_smoke_test.yaml \
+            -e INPUT_DIRECTION=hf_to_ms \
+            -e INPUT_DRY_RUN=true \
+            -e INPUT_LOG_LEVEL=INFO \
+            -e INPUT_STATE_DIR=/tmp/sync_state \
+            -e INPUT_TARGET= \
+            -e INPUT_HF_TOKEN= \
+            -e INPUT_MODELSCOPE_TOKEN= \
+            -e GITHUB_OUTPUT=/tmp/github_output \
+            -e GITHUB_STEP_SUMMARY=/tmp/github_step_summary \
+            hf-ms-sync:ci 2>&1 | tee /tmp/docker_output.log
+
+          # Verify entrypoint started and parsed arguments correctly
+          grep -q "Config:.*ci_smoke_test.yaml" /tmp/docker_output.log \
+            || { echo "FAIL: entrypoint did not print config summary"; exit 1; }
+          grep -q "Dry run:.*true" /tmp/docker_output.log \
+            || { echo "FAIL: dry_run flag not propagated to entrypoint"; exit 1; }
+          grep -q "Direction:.*hf_to_ms" /tmp/docker_output.log \
+            || { echo "FAIL: direction flag not propagated"; exit 1; }
+          grep -q "No sync items configured" /tmp/docker_output.log \
+            || { echo "FAIL: sync engine did not reach empty-config path"; exit 1; }
+
+          echo "Docker smoke test PASSED"