xhs-downloader

Security Findings (15)

• HighExcessive Permission Surface ×5

  • download_markdown:tool declares network permission
  • list_downloaded_files:tool declares network permission
  • set_download_directory:tool declares network permission
  • get_download_directory:tool declares network permission
  • create_subdirectory:tool declares network permission

  Rule: AS-002Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.
• MediumExcessive Permission Surface ×5

  • download_markdown:tool declares fs permission
  • list_downloaded_files:tool declares fs permission
  • set_download_directory:tool declares fs permission
  • get_download_directory:tool declares fs permission
  • create_subdirectory:tool declares fs permission

  Rule: AS-002Fix: Tool requests broad permissions (exec/fs/network). Validate input parameters using Enums where possible, and restrict file system operations to explicit allowed directories.
• LowDoS Resilience — Missing Rate Limit / Timeout ×5

  • download_markdown:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • list_downloaded_files:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • set_download_directory:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • get_download_directory:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration
  • create_subdirectory:tool performs network or execution operations but declares no rate-limit, timeout, or retry configuration

  Rule: AS-011Fix: Declare explicit rate-limit, timeout, and retry configuration for all network and execution tools. Implement exponential back-off and surface resource state to the calling agent.