feat: add dependency file and cicd

Author: jason.hsu

.github/workflows/ci.yml

@@ -14,6 +14,48 @@ on:
       - 'releases/*'
 
 jobs:
+  # ============================================
+  # DEPENDENCY CHECK JOB
+  # Ensures all imports are satisfied by requirements.txt
+  # ============================================
+  dependency-check:
+    name: Dependency Check
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.11"
+
+      - name: Install only from requirements.txt (clean environment)
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+
+      - name: Verify all production imports work
+        run: |
+          # This catches missing dependencies in requirements.txt
+          python -c "
+          import okx
+          from okx import Account, Trade, Funding, MarketData, PublicData
+          from okx import SubAccount, Convert, BlockTrading, CopyTrading
+          from okx import SpreadTrading, Grid, TradingData, Status
+          from okx.websocket import WsPublicAsync, WsPrivateAsync
+          print('✅ All imports successful')
+          print(f'   okx version: {okx.__version__}')
+          "
+
+      - name: Install dev dependencies and verify test imports
+        run: |
+          pip install -r requirements-dev.txt
+          python -c "
+          import pytest
+          import unittest
+          print('✅ Dev imports successful')
+          "
+
   # ============================================
   # LINT JOB
   # ============================================
@@ -31,11 +73,11 @@ jobs:
       - name: Install linting tools
         run: |
           python -m pip install --upgrade pip
-          pip install flake8
+          pip install ruff
 
-      - name: Run flake8
+      - name: Run ruff
         run: |
-          flake8 okx/ --max-line-length=120 --ignore=E501,W503,E203 --count --show-source --statistics
+          ruff check okx/ --ignore=E501
         continue-on-error: true  # Set to false once codebase is cleaned up
 
   # ============================================
@@ -44,10 +86,12 @@ jobs:
   test:
     name: Test (Python ${{ matrix.python-version }})
     runs-on: ubuntu-latest
+    needs: [dependency-check]  # Only run tests if dependencies are valid
     strategy:
-      fail-fast: false
+      fail-fast: true
+      max-parallel: 1
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12"]
+        python-version: ["3.9", "3.12"]
 
     steps:
       - uses: actions/checkout@v4
@@ -61,23 +105,23 @@ jobs:
         uses: actions/cache@v4
         with:
           path: ~/.cache/pip
-          key: ${{ runner.os }}-pip-${{ matrix.python-version }}-${{ hashFiles('setup.py') }}
+          key: ${{ runner.os }}-pip-${{ matrix.python-version }}-${{ hashFiles('requirements*.txt') }}
           restore-keys: |
             ${{ runner.os }}-pip-${{ matrix.python-version }}-
             ${{ runner.os }}-pip-
 
       - name: Install dependencies
         run: |
           python -m pip install --upgrade pip
+          pip install -r requirements-dev.txt
           pip install -e .
-          pip install pytest pytest-cov pytest-asyncio websockets certifi
 
       - name: Run tests
         run: |
           python -m pytest test/unit/ -v --cov=okx --cov-report=term-missing --cov-report=xml
 
       - name: Upload coverage to Codecov
-        if: matrix.python-version == '3.11'
+        if: matrix.python-version == '3.12'
         uses: codecov/codecov-action@v4
         with:
           file: ./coverage.xml
@@ -110,10 +154,20 @@ jobs:
       - name: Check package
         run: twine check dist/*
 
+      - name: Test install from wheel (clean environment)
+        run: |
+          # Create a fresh venv and install the built wheel
+          python -m venv /tmp/test-install
+          /tmp/test-install/bin/pip install dist/*.whl
+          /tmp/test-install/bin/python -c "
+          import okx
+          from okx import Account, Trade, Funding
+          print(f'✅ Package installs correctly: okx {okx.__version__}')
+          "
+
       - name: Upload build artifacts
         uses: actions/upload-artifact@v4
         with:
           name: dist
           path: dist/
           retention-days: 7
-

requirements-dev.txt

@@ -0,0 +1,19 @@
+# Development dependencies
+-r requirements.txt
+
+# Testing
+pytest>=7.0.0
+pytest-asyncio>=0.21.0
+pytest-cov>=4.0.0
+
+# Linting & Formatting
+ruff>=0.1.0
+mypy>=1.0.0
+
+# Environment
+python-dotenv>=1.0.0
+
+# Build tools
+build>=1.0.0
+twine>=4.0.0
+

requirements.txt

@@ -0,0 +1,7 @@
+# Core dependencies for python-okx
+httpx[http2]>=0.24.0
+requests>=2.25.0
+websockets>=10.0
+certifi>=2021.0.0
+loguru>=0.7.0
+

setup.py

@@ -1,8 +1,36 @@
 import setuptools
+from pathlib import Path
+
+# Read version from package
 import okx
-with open("README.md", "r",encoding="utf-8") as fh:
+
+# Read README
+with open("README.md", "r", encoding="utf-8") as fh:
     long_description = fh.read()
 
+
+def parse_requirements(filename):  # type: (str) -> list
+    """Parse requirements from a requirements file."""
+    requirements_path = Path(__file__).parent / filename
+    requirements = []
+    
+    if not requirements_path.exists():
+        return requirements
+    
+    with open(requirements_path, "r", encoding="utf-8") as f:
+        for line in f:
+            line = line.strip()
+            # Skip empty lines, comments, and -r includes
+            if not line or line.startswith("#") or line.startswith("-r"):
+                continue
+            # Handle inline comments
+            if "#" in line:
+                line = line.split("#")[0].strip()
+            requirements.append(line)
+    
+    return requirements
+
+
 setuptools.setup(
     name="python-okx",
     version=okx.__version__,
@@ -12,21 +40,21 @@
     long_description=long_description,
     long_description_content_type="text/markdown",
     url="https://okx.com/docs-v5/",
-    packages=setuptools.find_packages(),
+    packages=setuptools.find_packages(exclude=["test", "test.*", "example"]),
+    python_requires=">=3.7",
     classifiers=[
         "Programming Language :: Python :: 3",
+        "Programming Language :: Python :: 3.7",
+        "Programming Language :: Python :: 3.8",
+        "Programming Language :: Python :: 3.9",
+        "Programming Language :: Python :: 3.10",
+        "Programming Language :: Python :: 3.11",
+        "Programming Language :: Python :: 3.12",
         "License :: OSI Approved :: MIT License",
         "Operating System :: OS Independent",
     ],
-    install_requires=[
-        "importlib-metadata",
-        "httpx[http2]",
-        "keyring",
-        "loguru",
-        "requests",
-        "Twisted",
-        "pyOpenSSL",
-        "websockets",
-        "certifi"
-    ]
-)
+    install_requires=parse_requirements("requirements.txt"),
+    extras_require={
+        "dev": parse_requirements("requirements-dev.txt"),
+    },
+)