ci: Split GitHub Actions into separate workflows (#7)

* docs: Remove Benefits of create-with-session section from README

The create-with-session command and its 'new' alias are already in
upstream/main, so the benefits section is no longer needed.

* ci: Split GitHub Actions into three separate workflows

- **lint.yml**: Run linter (ruff) and type checking (mypy) on every push and PR
- **tests.yml**: Run basic tests that don't require email sending on every push and PR
- **email-tests.yml**: Run tests that require Postmark email sending only on manual trigger

Benefits:
- Linting runs fast on every PR without waiting for long test runs
- Basic API tests run on every PR without requiring Postmark secrets
- Email sending tests only run when manually triggered by maintainer with secrets
- Uses pytest markers ("flow") to separate tests by dependencies

This fixes the issue where PRs from forks fail due to missing Postmark secrets.

* fix: Remove mypy from lint workflow (not installed)

mypy is not available in the project, so remove it from lint workflow.
Ruff provides sufficient linting capabilities.

* fix: Add @pytest.mark.flow to tests that require Postmark credentials

- test_email_sender_initialization() creates PostmarkEmailSender without session
- test_wait_for_email_async() sends test email using PostmarkEmailSender
- test_wait_for_email_filtered_async() sends test emails using PostmarkEmailSender

These tests now properly excluded from basic tests workflow via -m "not flow"

---------

Co-authored-by: clawdbot-silly-waddle <clawdbot-silly-waddle@users.noreply.github.com>

Author: clawdbot-silly-waddle

.github/workflows/email-tests.yml

@@ -0,0 +1,49 @@
+name: Email Tests (Requires Postmark Credentials)
+
+on:
+  workflow_dispatch:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v4
+
+      - name: Set up Python
+        run: uv python install 3.12
+
+      - name: Install dependencies
+        run: uv sync --group dev
+
+      - name: Check Postmark credentials
+        env:
+          POSTMARK_API_TOKEN: ${{ secrets.POSTMARK_API_TOKEN }}
+          SENDER_EMAIL: ${{ secrets.SENDER_EMAIL }}
+        run: |
+          if [ -z "$POSTMARK_API_TOKEN" ]; then
+            echo "Error: POSTMARK_API_TOKEN is not set in repository secrets"
+            echo "This workflow requires Postmark credentials to run email tests"
+            echo "Please add POSTMARK_API_TOKEN and SENDER_EMAIL to repository secrets"
+            exit 1
+          fi
+          if [ -z "$SENDER_EMAIL" ]; then
+            echo "Error: SENDER_EMAIL is not set in repository secrets"
+            exit 1
+          fi
+          echo "Postmark credentials are properly configured"
+
+      - name: Run email sending tests
+        env:
+          POSTMARK_API_TOKEN: ${{ secrets.POSTMARK_API_TOKEN }}
+          SENDER_EMAIL: ${{ secrets.SENDER_EMAIL }}
+        run: uv run pytest tests/ -v -m "flow"
+
+      - name: Run async email sending tests
+        env:
+          POSTMARK_API_TOKEN: ${{ secrets.POSTMARK_API_TOKEN }}
+          SENDER_EMAIL: ${{ secrets.SENDER_EMAIL }}
+        run: uv run pytest tests/test_tacomail_async.py -v -m "flow"

.github/workflows/lint.yml

@@ -0,0 +1,26 @@
+name: Lint and Type Check
+
+on:
+  push:
+    branches: [main]
+  pull_request:
+    branches: [main]
+
+jobs:
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - name: Install uv
+        uses: astral-sh/setup-uv@v4
+
+      - name: Set up Python
+        run: uv python install 3.12
+
+      - name: Install dependencies
+        run: uv sync --group dev
+
+      - name: Run linter
+        run: uv run ruff check .

.github/workflows/tests.yml

@@ -1,4 +1,4 @@
-name: Tests
+name: Basic Tests
 
 on:
   push:
@@ -9,39 +9,21 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
-    
+
     steps:
       - uses: actions/checkout@v4
-      
+
       - name: Install uv
         uses: astral-sh/setup-uv@v4
-      
+
       - name: Set up Python
         run: uv python install 3.12
-      
+
       - name: Install dependencies
         run: uv sync --group dev
-      
-      - name: Run linter
-        run: uv run ruff check .
-      
-      - name: Check Postmark credentials
-        env:
-          POSTMARK_API_TOKEN: ${{ secrets.POSTMARK_API_TOKEN }}
-          SENDER_EMAIL: ${{ secrets.SENDER_EMAIL }}
-        run: |
-          if [ -z "$POSTMARK_API_TOKEN" ]; then
-            echo "Error: POSTMARK_API_TOKEN is not set in repository secrets"
-            exit 1
-          fi
-          if [ -z "$SENDER_EMAIL" ]; then
-            echo "Error: SENDER_EMAIL is not set in repository secrets"
-            exit 1
-          fi
-          echo "Postmark credentials are properly configured"
-      
-      - name: Run tests
-        env:
-          POSTMARK_API_TOKEN: ${{ secrets.POSTMARK_API_TOKEN }}
-          SENDER_EMAIL: ${{ secrets.SENDER_EMAIL }}
-        run: uv run pytest tests/ -v
+
+      - name: Run basic tests (no email sending required)
+        run: uv run pytest tests/ -v -m "not flow"
+
+      - name: Run async basic tests
+        run: uv run pytest tests/test_tacomail_async.py -v -m "not flow"

README.md

@@ -170,17 +170,6 @@ To receive emails, you can use either approach:
 2. ✅ `tacomail create-session <email>` - Create session (REQUIRED)
 3. ✅ `tacomail wait <email>` - Monitor inbox for incoming emails
 
-### Benefits of create-with-session
-
-The `create-with-session` command (and its short alias `new`) provides several advantages:
-
-- **⚡ Faster workflow**: One command instead of two
-- **🎯 Reduced errors**: No need to copy-paste email between commands
-- **📋 Complete information**: Shows both email and session details at once
-- **🔄 Works in both modes**: Supports both sync and async clients
-- **🎨 Better UX**: Clear next steps displayed after creation
-- **⚙️ Flexible options**: Still supports domain and username customization
-
 ### Common Workflows
 
 **Workflow 1: Quick setup for testing**

tests/test_tacomail.py

@@ -66,6 +66,7 @@ def test_fetch_empty_inbox(client: TacomailClient):
     assert len(emails) == 0
 
 
+@pytest.mark.flow
 def test_email_sender_initialization():
     sender = PostmarkEmailSender()
     assert sender.api_token is not None

tests/test_tacomail_async.py

@@ -49,6 +49,7 @@ async def test_delete_session_async(
     await client.delete_session(username, domain)
 
 
+@pytest.mark.flow
 @pytest.mark.asyncio
 async def test_wait_for_email_async(
     client_generator: AsyncGenerator[AsyncTacomailClient, None],
@@ -81,6 +82,7 @@ async def test_wait_for_email_async(
     assert received_email.body.text.strip() == test_body
 
 
+@pytest.mark.flow
 @pytest.mark.asyncio
 async def test_wait_for_email_filtered_async(
     client_generator: AsyncGenerator[AsyncTacomailClient, None],