Greptile Summary

This PR adds 5 unit tests for parse_vercel_url() to cover the logId, selectedLog, deploymentId, and deployment_id query-parameter aliases, plus a whitespace-trimming assertion — all of which are correct against the current implementation.

• The first four new test functions are separated by only one blank line instead of two, which will fail ruff E302 and block make lint / make format-check (both are CI gates). Running make format locally will fix this automatically.

Confidence Score: 4/5

Safe to merge once the blank-line formatting is fixed; the test logic itself is correct.

All five new assertions correctly reflect the implementation. The only blocking issue is a formatting violation (single blank lines between top-level functions) that will fail the ruff CI gate — easily fixed with make format.

tests/app/remote/test_vercel_poller.py — fix blank lines between the four new test functions before the whitespace test.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["parse_vercel_url(url)"] --> B["cleaned = url.strip()"]
    B --> C["urlparse(cleaned)"]
    C --> D["Extract path segments\n(team_slug, project_slug, deployment_id)"]
    C --> E["parse_qs(query)"]
    E --> F{"selected_log_id?"}
    F -->|"selectedLogId"| G["selected_log_id = selectedLogId"]
    F -->|"logId"| H["selected_log_id = logId"]
    F -->|"selectedLog"| I["selected_log_id = selectedLog"]
    E --> J{"deployment_id?"}
    J -->|"path segment"| K["deployment_id from path"]
    J -->|"deploymentId"| L["deployment_id = deploymentId"]
    J -->|"deployment_id"| M["deployment_id = deployment_id"]
    G & H & I --> N["ParsedVercelUrl\n(original_url=cleaned, ...)"]
    K & L & M --> N
    D --> N
    style G fill:#d4edda
    style H fill:#d4edda
    style I fill:#d4edda
    style L fill:#d4edda
    style M fill:#d4edda

_{Reviews (1): Last reviewed commit: "test: add unit tests for parse_vercel_ur..." | Re-trigger Greptile}

🍵 @zeesshhh0 made tea, opened a PR, and merged before it cooled. No notes. ☕

👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.