[v0.1] Add pre-merge lint check and lint all existing code

Feat/lint workflow

Author: ArnavBalyan

.github/workflows/lint.yml

@@ -0,0 +1,33 @@
+name: Lint
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version: "3.10"
+          
+      - name: Install dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install ruff
+          
+      - name: Check for linting errors
+        run: |
+          ruff check .
+          
+      - name: Check formatting
+        run: |
+          ruff format --check .

bin/client.py

@@ -1 +1 @@
-# todo: client wrapper
+# todo: client wrapper

bin/server.py

@@ -1,18 +1,18 @@
 import sys
 from pathlib import Path
+
 sys.path.insert(0, str(Path(__file__).parent.parent / "src"))
 sys.path.insert(0, str(Path(__file__).parent.parent))
 
+from examples.simple_stock import StockWorkflow
+
 from concierge.core.registry import register_workflow
 from concierge.server import start_server
-
-from examples.simple_stock import StockWorkflow
 from examples.zillow.workflow import ZillowWorkflow
 
 # Example code, not invoked through concierge serve.
 if __name__ == "__main__":
     register_workflow(StockWorkflow)
     register_workflow(ZillowWorkflow)
-    
-    start_server()
 
+    start_server()

examples/__init__.py

@@ -1,2 +1 @@
 """Concierge Examples Package"""
-

examples/ecommerce/__init__.py

@@ -1,4 +1,3 @@
 from .workflow import EcommerceWorkflow
 
 __all__ = ["EcommerceWorkflow"]
-

examples/ecommerce/workflow.py

@@ -1,12 +1,12 @@
-from concierge.core import workflow, stage, task, State
+from concierge.core import State, stage, task, workflow
 
 
 @stage(name="discovery")
 class DiscoveryStage:
     @task(description="Search for products by keyword")
     def search_products(self, state: State, query: str) -> dict:
         return {"results": []}
-    
+
     @task(description="Filter by price range")
     def filter_by_price(self, state: State, min_price: float, max_price: float) -> dict:
         return {"filtered_count": 0}
@@ -17,7 +17,7 @@ class ProductStage:
     @task(description="View product details")
     def view_details(self, state: State, product_id: str) -> dict:
         return {"product": {}}
-    
+
     @task(description="Read customer reviews")
     def read_reviews(self, state: State, product_id: str) -> dict:
         return {"reviews": []}
@@ -28,11 +28,11 @@ class CartStage:
     @task(description="Add item to cart")
     def add_to_cart(self, state: State, product_id: str, quantity: int) -> dict:
         return {"cart_size": 1}
-    
+
     @task(description="Update quantity")
     def update_quantity(self, state: State, product_id: str, quantity: int) -> dict:
         return {"updated": True}
-    
+
     @task(description="Remove item from cart")
     def remove_item(self, state: State, product_id: str) -> dict:
         return {"removed": True}
@@ -43,7 +43,7 @@ class CheckoutStage:
     @task(description="Apply coupon code")
     def apply_coupon(self, state: State, code: str) -> dict:
         return {"discount": 0}
-    
+
     @task(description="Complete purchase")
     def complete_purchase(self, state: State) -> dict:
         return {"order_id": "ORD123"}
@@ -55,11 +55,5 @@ class EcommerceWorkflow:
     product = ProductStage
     cart = CartStage
     checkout = CheckoutStage
-    
-    transitions = {
-        discovery: [product, cart],
-        product: [cart, discovery],
-        cart: [checkout, discovery],
-        checkout: []
-    }
 
+    transitions = {discovery: [product, cart], product: [cart, discovery], cart: [checkout, discovery], checkout: []}

examples/food_delivery/__init__.py

@@ -1,4 +1,3 @@
 from .workflow import FoodDeliveryWorkflow
 
 __all__ = ["FoodDeliveryWorkflow"]
-

examples/food_delivery/workflow.py

@@ -1,12 +1,12 @@
-from concierge.core import workflow, stage, task, State
+from concierge.core import State, stage, task, workflow
 
 
 @stage(name="restaurants")
 class RestaurantsStage:
     @task(description="Search restaurants by cuisine")
     def search_restaurants(self, state: State, cuisine: str, location: str) -> dict:
         return {"restaurants": []}
-    
+
     @task(description="Filter by rating or delivery time")
     def filter_restaurants(self, state: State, min_rating: float) -> dict:
         return {"filtered": []}
@@ -17,11 +17,11 @@ class MenuStage:
     @task(description="View restaurant menu")
     def view_menu(self, state: State, restaurant_id: str) -> dict:
         return {"menu_items": []}
-    
+
     @task(description="Add item to order")
     def add_item(self, state: State, item_id: str, quantity: int) -> dict:
         return {"order_size": 1}
-    
+
     @task(description="Customize item")
     def customize_item(self, state: State, item_id: str, modifications: list) -> dict:
         return {"customized": True}
@@ -32,7 +32,7 @@ class CartStage:
     @task(description="Review order and total")
     def review_order(self, state: State) -> dict:
         return {"total": 0, "items": []}
-    
+
     @task(description="Apply promo code")
     def apply_promo(self, state: State, code: str) -> dict:
         return {"discount": 0}
@@ -43,7 +43,7 @@ class CheckoutStage:
     @task(description="Set delivery instructions")
     def set_instructions(self, state: State, instructions: str) -> dict:
         return {"saved": True}
-    
+
     @task(description="Place order")
     def place_order(self, state: State) -> dict:
         return {"order_id": "DD123", "eta": 30}
@@ -55,11 +55,5 @@ class FoodDeliveryWorkflow:
     menu = MenuStage
     cart = CartStage
     checkout = CheckoutStage
-    
-    transitions = {
-        restaurants: [menu],
-        menu: [cart, restaurants],
-        cart: [checkout, menu],
-        checkout: []
-    }
 
+    transitions = {restaurants: [menu], menu: [cart, restaurants], cart: [checkout, menu], checkout: []}

examples/payment/__init__.py

@@ -1,4 +1,3 @@
 from .workflow import PaymentWorkflow
 
 __all__ = ["PaymentWorkflow"]
-

examples/payment/workflow.py

@@ -1,12 +1,12 @@
-from concierge.core import workflow, stage, task, State
+from concierge.core import State, stage, task, workflow
 
 
 @stage(name="transaction")
 class TransactionStage:
     @task(description="Create payment request")
     def create_payment(self, state: State, amount: float, recipient: str) -> dict:
         return {"payment_id": "PAY123"}
-    
+
     @task(description="Request money")
     def request_money(self, state: State, amount: float, recipient: str, note: str) -> dict:
         return {"request_id": "REQ123"}
@@ -17,11 +17,11 @@ class ReviewStage:
     @task(description="Review payment details")
     def review_payment(self, state: State) -> dict:
         return {"details": {}}
-    
+
     @task(description="Add payment note")
     def add_note(self, state: State, note: str) -> dict:
         return {"saved": True}
-    
+
     @task(description="Select funding source")
     def select_source(self, state: State, source: str) -> dict:
         return {"source": source}
@@ -32,7 +32,7 @@ class VerificationStage:
     @task(description="Verify with 2FA code")
     def verify_2fa(self, state: State, code: str) -> dict:
         return {"verified": True}
-    
+
     @task(description="Verify with biometric")
     def verify_biometric(self, state: State) -> dict:
         return {"verified": True}
@@ -43,7 +43,7 @@ class ConfirmationStage:
     @task(description="Confirm and send payment")
     def confirm_payment(self, state: State) -> dict:
         return {"transaction_id": "TXN123", "status": "completed"}
-    
+
     @task(description="Cancel payment")
     def cancel_payment(self, state: State) -> dict:
         return {"cancelled": True}
@@ -55,11 +55,10 @@ class PaymentWorkflow:
     review = ReviewStage
     verification = VerificationStage
     confirmation = ConfirmationStage
-    
+
     transitions = {
         transaction: [review],
         review: [verification, transaction],
         verification: [confirmation],
-        confirmation: []
+        confirmation: [],
     }
-