winebox 0.1.3__tar.gz → 0.1.5__tar.gz

{winebox-0.1.3 → winebox-0.1.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: winebox
-Version: 0.1.3
+Version: 0.1.5
 Summary: Wine Cellar Management Application with OCR label scanning
 Project-URL: Homepage, https://github.com/jdrumgoole/winebox
 Project-URL: Repository, https://github.com/jdrumgoole/winebox
@@ -33,6 +33,7 @@ Requires-Dist: pydantic>=2.0.0
 Requires-Dist: pytesseract>=0.3.10
 Requires-Dist: python-jose[cryptography]>=3.3.0
 Requires-Dist: python-multipart>=0.0.6
+Requires-Dist: slowapi>=0.1.9
 Requires-Dist: sqlalchemy>=2.0.0
 Requires-Dist: uvicorn[standard]>=0.27.0
 Provides-Extra: dev
@@ -41,6 +42,8 @@ Requires-Dist: httpx>=0.26.0; extra == 'dev'
 Requires-Dist: invoke>=2.2.0; extra == 'dev'
 Requires-Dist: myst-parser>=2.0.0; extra == 'dev'
 Requires-Dist: pytest-asyncio>=0.23.0; extra == 'dev'
+Requires-Dist: pytest-playwright>=0.4.0; extra == 'dev'
+Requires-Dist: pytest-xdist>=3.5.0; extra == 'dev'
 Requires-Dist: pytest>=8.0.0; extra == 'dev'
 Requires-Dist: sphinx>=7.0.0; extra == 'dev'
 Description-Content-Type: text/markdown

{winebox-0.1.3 → winebox-0.1.5}/docs/index.md

@@ -162,6 +162,8 @@ winebox/
 │   ├── routers/      # API endpoints
 │   ├── services/     # Business logic
 │   └── static/       # Web interface
+├── scripts/          # Utility scripts
+│   └── migrations/   # Database migration system
 ├── tests/            # Test suite
 ├── docs/             # Documentation
 ├── data/             # Database and images

{winebox-0.1.3 → winebox-0.1.5}/docs/user-guide.md

@@ -41,6 +41,11 @@ The check-in process adds bottles to your cellar inventory.
 6. **Add notes** (optional):
    - Where you purchased it, price, occasion, etc.
 7. Click **Check In Wine**
+8. **Review in confirmation dialog**:
+   - A confirmation dialog appears with all editable fields
+   - Make any final adjustments to wine details
+   - View raw label text by expanding the "Raw Label Text" section
+   - Click **Confirm** to save or **Cancel** to return to the form
 
 ### Using the API
 
@@ -178,3 +183,62 @@ Your data is stored in:
 - **Images**: `data/images/`
 
 Back up these files regularly to preserve your cellar records.
+
+## Development
+
+### Running Tests
+
+WineBox has both unit tests and end-to-end (E2E) browser tests.
+
+```bash
+# Run all tests (unit + E2E)
+invoke test
+
+# Run only unit tests (fast, no server required)
+invoke test-unit
+
+# Run only E2E tests (requires running server)
+invoke test-e2e
+
+# Run E2E tests with more workers for faster execution
+invoke test-e2e --workers 8
+```
+
+**Note**: E2E tests use Playwright for browser automation and create unique test users for parallel execution. The server must be running (`invoke start-background`) before running E2E tests.
+
+### Invoke Tasks
+
+Common development tasks:
+
+```bash
+# Server management
+invoke start              # Start server in foreground
+invoke start-background   # Start server in background
+invoke stop               # Stop the server
+invoke restart            # Restart the server
+invoke status             # Check server status
+invoke logs               # View server logs
+
+# Database management
+invoke init-db            # Initialize database
+invoke purge --force      # Delete database and images
+invoke purge-wines --force # Delete wines but keep users
+
+# Database migrations
+uv run python -m scripts.migrations.runner status      # Show version
+uv run python -m scripts.migrations.runner up          # Migrate to latest
+uv run python -m scripts.migrations.runner down --to 0 # Revert to version
+uv run python -m scripts.migrations.runner history     # Show history
+
+# User management
+invoke add-user <username> --password <pass>
+invoke remove-user <username> --force
+invoke list-users
+invoke disable-user <username>
+invoke enable-user <username>
+invoke passwd <username> --password <newpass>
+
+# Documentation
+invoke docs-build         # Build Sphinx documentation
+invoke docs-serve         # Build and serve docs locally
+```

{winebox-0.1.3 → winebox-0.1.5}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "winebox"
-version = "0.1.3"
+version = "0.1.5"
 description = "Wine Cellar Management Application with OCR label scanning"
 readme = "README.md"
 requires-python = ">=3.11"
@@ -37,6 +37,7 @@ dependencies = [
     "bcrypt>=4.0.0,<4.1.0",
     "python-jose[cryptography]>=3.3.0",
     "anthropic>=0.40.0",
+    "slowapi>=0.1.9",
 ]
 
 [project.urls]
@@ -48,6 +49,8 @@ Issues = "https://github.com/jdrumgoole/winebox/issues"
 dev = [
     "pytest>=8.0.0",
     "pytest-asyncio>=0.23.0",
+    "pytest-playwright>=0.4.0",
+    "pytest-xdist>=3.5.0",
     "httpx>=0.26.0",
     "invoke>=2.2.0",
     "sphinx>=7.0.0",

winebox-0.1.5/scripts/__init__.py

@@ -0,0 +1 @@
+"""Scripts package for WineBox utility scripts."""

winebox-0.1.5/scripts/migrations/__init__.py

@@ -0,0 +1,20 @@
+"""Database migration system for WineBox.
+
+This package provides versioned database migrations with forward and reverse support.
+
+Usage:
+    # Show current version and available migrations
+    uv run python -m scripts.migrations.runner status
+
+    # Migrate to latest version
+    uv run python -m scripts.migrations.runner up
+
+    # Migrate to specific version
+    uv run python -m scripts.migrations.runner up --to 2
+
+    # Revert to specific version
+    uv run python -m scripts.migrations.runner down --to 0
+
+    # Show migration history
+    uv run python -m scripts.migrations.runner history
+"""

winebox-0.1.5/scripts/migrations/db_migrate_0_to_1.py

@@ -0,0 +1,46 @@
+#!/usr/bin/env python3
+"""Migration: Add full_name and anthropic_api_key columns to users table.
+
+This migration adds user settings columns:
+- full_name: Optional display name for the user
+- anthropic_api_key: Optional API key for Claude Vision (per-user override)
+
+Version: 0 -> 1
+"""
+
+import sqlite3
+
+# Migration metadata
+SOURCE_VERSION = 0
+TARGET_VERSION = 1
+DESCRIPTION = "Add full_name and anthropic_api_key to users table"
+
+
+def migrate(cursor: sqlite3.Cursor) -> None:
+    """Apply the forward migration.
+
+    Adds full_name and anthropic_api_key columns to the users table.
+    These are nullable columns, so existing users will have NULL values.
+    """
+    # Check existing columns
+    cursor.execute("PRAGMA table_info(users)")
+    columns = {row[1] for row in cursor.fetchall()}
+
+    # Add full_name column if it doesn't exist
+    if "full_name" not in columns:
+        cursor.execute("ALTER TABLE users ADD COLUMN full_name VARCHAR(255)")
+
+    # Add anthropic_api_key column if it doesn't exist
+    if "anthropic_api_key" not in columns:
+        cursor.execute("ALTER TABLE users ADD COLUMN anthropic_api_key VARCHAR(255)")
+
+
+def validate(cursor: sqlite3.Cursor) -> bool:
+    """Validate the migration was successful.
+
+    Returns True if both columns exist in the users table.
+    """
+    cursor.execute("PRAGMA table_info(users)")
+    columns = {row[1] for row in cursor.fetchall()}
+
+    return "full_name" in columns and "anthropic_api_key" in columns

winebox-0.1.5/scripts/migrations/db_revert_1_to_0.py

@@ -0,0 +1,108 @@
+#!/usr/bin/env python3
+"""Revert migration: Remove full_name and anthropic_api_key columns from users table.
+
+This revert removes the user settings columns added in version 1.
+WARNING: Data in these columns will be lost!
+
+Version: 1 -> 0
+"""
+
+import sqlite3
+
+# Migration metadata
+SOURCE_VERSION = 1
+TARGET_VERSION = 0
+DESCRIPTION = "Remove full_name and anthropic_api_key from users table"
+
+
+def migrate(cursor: sqlite3.Cursor) -> None:
+    """Apply the reverse migration (revert).
+
+    Removes full_name and anthropic_api_key columns from the users table.
+
+    Since SQLite doesn't support DROP COLUMN in older versions, we use
+    the table rebuild pattern:
+    1. Create new table without the columns
+    2. Copy data from old table
+    3. Drop old table
+    4. Rename new table
+    5. Recreate indexes
+
+    WARNING: Data in full_name and anthropic_api_key columns will be lost!
+    """
+    # Check if columns exist (may have already been removed)
+    cursor.execute("PRAGMA table_info(users)")
+    columns = {row[1] for row in cursor.fetchall()}
+
+    if "full_name" not in columns and "anthropic_api_key" not in columns:
+        # Columns already removed, nothing to do
+        return
+
+    # Disable foreign key checks during table rebuild
+    cursor.execute("PRAGMA foreign_keys = OFF")
+
+    try:
+        # 1. Create new table without full_name and anthropic_api_key
+        cursor.execute("""
+            CREATE TABLE users_new (
+                id CHAR(36) PRIMARY KEY,
+                username VARCHAR(50) NOT NULL UNIQUE,
+                email VARCHAR(255) UNIQUE,
+                hashed_password VARCHAR(255) NOT NULL,
+                is_active BOOLEAN DEFAULT 1 NOT NULL,
+                is_admin BOOLEAN DEFAULT 0 NOT NULL,
+                created_at DATETIME DEFAULT CURRENT_TIMESTAMP NOT NULL,
+                updated_at DATETIME DEFAULT CURRENT_TIMESTAMP NOT NULL,
+                last_login DATETIME
+            )
+        """)
+
+        # 2. Copy data from old table (excluding removed columns)
+        cursor.execute("""
+            INSERT INTO users_new (
+                id, username, email, hashed_password,
+                is_active, is_admin, created_at, updated_at, last_login
+            )
+            SELECT
+                id, username, email, hashed_password,
+                is_active, is_admin, created_at, updated_at, last_login
+            FROM users
+        """)
+
+        # 3. Drop old table
+        cursor.execute("DROP TABLE users")
+
+        # 4. Rename new table
+        cursor.execute("ALTER TABLE users_new RENAME TO users")
+
+        # 5. Recreate indexes
+        cursor.execute(
+            "CREATE UNIQUE INDEX IF NOT EXISTS ix_users_username ON users (username)"
+        )
+        cursor.execute(
+            "CREATE UNIQUE INDEX IF NOT EXISTS ix_users_email ON users (email)"
+        )
+
+    finally:
+        # Re-enable foreign key checks
+        cursor.execute("PRAGMA foreign_keys = ON")
+
+
+def validate(cursor: sqlite3.Cursor) -> bool:
+    """Validate the revert was successful.
+
+    Returns True if both columns have been removed from the users table.
+    """
+    cursor.execute("PRAGMA table_info(users)")
+    columns = {row[1] for row in cursor.fetchall()}
+
+    # Verify columns are removed
+    if "full_name" in columns or "anthropic_api_key" in columns:
+        return False
+
+    # Verify essential columns still exist
+    required_columns = {
+        "id", "username", "email", "hashed_password",
+        "is_active", "is_admin", "created_at", "updated_at", "last_login"
+    }
+    return required_columns.issubset(columns)