Address feedback

Author: pamelafox

.github/workflows/python.yaml

@@ -23,6 +23,6 @@ jobs:
         - name: Lint with ruff
           run: |
             uv run ruff check .
-        - name: Check formatting with black
+        - name: Check formatting with ruff
           run: |
-            uv run black . --check --verbose
+            uv run ruff format . --check

.pre-commit-config.yaml

@@ -6,7 +6,7 @@ repos:
     -   id: end-of-file-fixer
     -   id: trailing-whitespace
 -   repo: https://github.com/astral-sh/ruff-pre-commit
-    rev: v0.1.0
+    rev: v0.14.7
     hooks:
         # Run the linter.
         - id: ruff

infra/core/host/container-registry.bicep

@@ -2,6 +2,7 @@ param name string
 param location string = resourceGroup().location
 param tags object = {}
 
+// It would be better security-wise if this was false, but that is not yet working with the VNet+azd integration
 param adminUserEnabled bool = true
 param anonymousPullEnabled bool = false
 param dataEndpointEnabled bool = false

infra/main.bicep

@@ -162,7 +162,7 @@ module containerAppsNSG 'br/public:avm/res/network/network-security-group:0.5.1'
     name: '${prefix}-container-apps-nsg'
     location: location
     tags: tags
-    securityRules: usePrivateIngress
+    securityRules: !usePrivateIngress
       ? [
           {
             name: 'AllowHttpsInbound'

servers/basic_mcp_stdio.py

@@ -57,7 +57,7 @@ async def add_expense(
             if not file_exists:
                 writer.writerow(["date", "amount", "category", "description", "payment_method"])
 
-            writer.writerow([date_iso, amount, category.value, description, payment_method.name])
+            writer.writerow([date_iso, amount, category.value, description, payment_method.value])
 
         return f"Successfully added expense: ${amount} for {description} on {date_iso}"