code-review-assistant_skill

Overview

This skill performs structured, actionable code reviews for Python projects, with a focus on modules that parse URLs to extract emails and LinkedIn links. It produces grouped findings—strengths, issues/risks, and prioritized suggestions—so developers can quickly apply fixes. Feedback emphasizes correctness, security, performance, and maintainability.

How this skill works

I read the full code to understand intent, data shapes, and external interactions, then evaluate logic, style, and edge cases against common Python best practices. The review checks parsing correctness for email and LinkedIn extraction, error handling for network and HTML parsing, dependency usage, and potential security risks like injection or credential leaks. Results include concise examples and code snippets when needed and prioritize fixes by impact.

When to use it

• Before merging changes that add or modify URL scraping or parsing logic
• When evaluating a new parser/scraper for extracting emails and LinkedIn profiles
• During code audits to ensure compliance with security and privacy rules
• When onboarding contributors to ensure consistent style and test coverage
• Before deploying scraping code to production or scheduled jobs

Best practices

• Validate and sanitize all external inputs and HTTP responses before parsing
• Use robust HTML parsers (e.g., BeautifulSoup, lxml) and avoid brittle regex-only extraction
• Isolate networking and parsing for easy unit testing and mocking
• Avoid storing secrets or API keys in code; read from environment or secure vaults
• Add clear rate-limiting/backoff and respect robots.txt for ethical scraping

Example use cases

• Review a new Python module that crawls websites and extracts contact emails and LinkedIn URLs
• Audit regex patterns used for email detection and replace them with validated parsers
• Assess error handling for network failures, timeouts, and malformed HTML
• Identify performance bottlenecks in large-scale scraping jobs and suggest batching/async
• Confirm tests cover edge cases like obfuscated email formats and redirected LinkedIn links