## Review guidelines

### Route Configuration

1. **Example Format**: The `example` field must start with `/` and be a working RSSHub route path (e.g., `/example/route`), not a full URL or source website URL.

2. **Route Name**: Do not repeat the namespace name in the route name. The namespace is already defined in `namespace.ts`.

3. **Radar Source Format**: Use relative paths without `https://` prefix in `radar[].source`. Example: `source: ['www.example.com/path']` instead of `source: ['https://www.example.com/path']`.

4. **Radar Target**: The `radar[].target` may be empty. If present, it must match the route path and its parameters.

5. **Namespace URL**: In `namespace.ts`, the `url` field should not include the `https://` protocol prefix.

6. **Single Category**: Provide only one category in the `categories` array, not multiple.

7. **Unnecessary Files**: Do not create separate `README.md` or `radar.ts` files. Put descriptions in `Route['description']` and radar rules in `Route['radar']`.

8. **Features Accuracy**: Set `requirePuppeteer: true` only if your route actually uses browser automation (`@/utils/playwright`). Do not mismatch feature flags.

9. **Maintainer GitHub ID**: The `maintainers` field must contain valid GitHub usernames. Verify that the username exists before adding it.

### Code Style

10. **Naming Convention**: Use `camelCase` for variable names in JavaScript/TypeScript. Avoid `snake_case` (e.g., use `videoUrl` instead of `video_url`).

11. **Import Sorting**: Keep imports sorted. Run autofix if linter reports import order issues.

12. **Avoid Loading HTML Twice**: Do not call `load()` from cheerio multiple times on the same content. Reuse the initial `$` object.

13. **Async/Await in Close**: When closing a Playwright page or context, use `await page.close()`, `await context.close()`, or `await destroy()` instead of non-awaited calls.

14. **No Explicit Null**: No need to explicitly set a property to `null` if it does not exist - just omit it.

15. **Valid Item Properties**: Only use properties defined in [lib/types.ts](https://github.com/DIYgod/RSSHub/blob/master/lib/types.ts). Custom properties like `avatar`, `bio` will be ignored by RSSHub.

16. **String Methods**: Use `startsWith()` instead of `includes()` when checking if a string begins with a specific prefix.

### Data Handling

17. **Use Cache**: Always [cache](https://docs.rsshub.app/joinus/advanced/use-cache) the returned results when fetching article details in a loop using `cache.tryGet()`.

18. **Description Content**: The `description` field should contain only the main article content. Do not include `title`, `author`, `pubDate`, or tags in `description` - they have their own dedicated fields.

19. **Category Field**: Extract tags/categories from articles and place them in the `category` field, not in `description`.

20. **pubDate Field**: Always include `pubDate` when the source provides date/time information. Use the `parseDate` utility function.

21. **No Fake Dates**: Do not use `new Date()` as a fallback for `pubDate`. If no date is available, leave it undefined. See [No Date documentation](https://docs.rsshub.app/joinus/advanced/pub-date#no-date).

22. **No Title Trimming**: Do not manually trim or truncate titles. RSSHub core handles title processing automatically.

23. **Unique Links**: Ensure each item's `link` is unique as it will be used as `guid`. Avoid fallback URLs that could cause duplicate `guid` values.

24. **Human-Readable Links**: The feed `link` field should point to a human-readable webpage URL, not an API endpoint URL.

### API and Data Fetching

25. **Prefer APIs Over Scraping**: When the target website has an API (often found by scrolling pages or checking network requests), use the API endpoint instead of HTML scraping.

26. **JSON Parsing**: When using `ofetch`, `JSON.parse` is automatically applied. Do not manually decode JSON escape sequences like `\u003C`.

27. **No Page Turning**: RSS feeds should only request the first page of content. Do not implement pagination parameters for users.

28. **Use Common Parameters**: Use RSSHub's built-in common parameters like [`limit`](https://docs.rsshub.app/guide/parameters#limit-entries) instead of implementing custom query parameters for limiting entries.

29. **No Custom Query Parameters**: Avoid using querystring parameters for route configuration. Use path parameters (`:param`) instead.

30. **No Custom Filtering**: Do not implement custom tag/category filtering in routes. Users can apply filtering using [common parameters](https://docs.rsshub.app/guide/parameters).

31. **Avoid Dynamic Hashes**: If an API requires a hash that changes across builds, extract it dynamically from the webpage rather than hardcoding it.

### Media and Enclosures

32. **Valid MIME Types**: The `enclosure_type` must be a valid MIME type as defined in RFC specifications. For example, `video/youtube` is not valid - use actual video file URLs with proper types like `video/mp4`.

33. **Direct Media URLs**: `enclosure_url` must point directly to downloadable media files (e.g., `.mp4`, `.mp3`), not to web pages containing media.

34. **Video Poster**: Use the HTML5 `<video>` element's `poster` attribute for video thumbnails instead of adding separate `<img>` elements.

35. **No Referrer Policy in Routes**: Do not add `referrerpolicy` attributes to images/videos - RSSHub middleware handles this automatically.

### Browser Automation

36. **Limit Request Types**: Do not allow every type of request through the browser. Use `page.route()` with an explicit allowlist of resource types (e.g., `document`) and abort everything else. Do not use a blocklist that aborts only known types such as images or fonts.

37. **Use Selectors, Not Delays**: Do not use fixed `setTimeout` delays. Use `page.waitForSelector()` instead to wait for specific elements.

38. **Avoid Multiple Sessions**: Do not call `playwright()` or `getPlaywrightPage()` inside `Promise.all()` loops - this creates multiple browser sessions and dramatically increases resource usage.

39. **Do Not Bypass Empty Checks**: Do not return empty arrays with custom messages to bypass RSSHub's [internal checks](https://github.com/DIYgod/RSSHub/blob/master/lib/middleware/parameter.ts) for empty items. This makes it hard for users and maintainers to know if a feed is broken.

### Default Values and Examples

40. **Preserve Default Values**: Do not change documented default values for existing route parameters unless the current default is broken.

41. **Preserve Working Examples**: Do not modify existing route examples unless they no longer work.

42. **Route Parameters**: The `parameters` object keys must match the actual path parameters defined in the route path. Do not add non-existent parameters.

### Error Handling

43. **Error Messages**: Use clear, actionable error messages that help users understand what went wrong.

44. **Resolve All Review Comments**: Before requesting re-review, ensure all previous review comments are addressed, not just some of them.

### Code Organization

45. **Move Functions Up**: Move function definitions to the highest possible scope. Avoid defining functions inside loops or callbacks when they can be defined at module level.

46. **No Await in Loops**: Avoid using `await` inside loops when possible. Use `Promise.all()` with proper concurrency control instead.

47. **Check URL Validity**: Before using URLs from config files or namespaces, verify they don't return 404 errors.

48. **Comments Language**: Write code comments in English for consistency and accessibility.

49. **Parentheses in Arrow Functions**: Always use parentheses around arrow function parameters, even for single parameters.

50. **No Tests for Routes**: Do not write test files for routes under `lib/routes/`. Route tests only encode the author's assumptions about the target site and go stale when it changes. Routes are verified through automatic route testing on pull requests, which runs the route examples against the live site.