Skip to content

fix: support MCP user data dir env#272

Closed
Bortlesboat wants to merge 1 commit into
D4Vinci:mainfrom
Bortlesboat:codex/mcp-user-data-dir-env
Closed

fix: support MCP user data dir env#272
Bortlesboat wants to merge 1 commit into
D4Vinci:mainfrom
Bortlesboat:codex/mcp-user-data-dir-env

Conversation

@Bortlesboat

Copy link
Copy Markdown
Contributor

Summary

  • read SCRAPLING_USER_DATA_DIR in the MCP server
  • pass that profile path into dynamic and stealthy browser sessions created by open_session, fetch, and stealthy_fetch
  • add unit coverage with fake browser sessions so the env plumbing is tested without launching Playwright

Fixes #269.

Tests

  • py -3 -m pytest tests/ai/test_ai_mcp.py -q -k user_data_dir
  • py -3 -m pytest tests/ai/test_ai_mcp.py -q
  • py -3 -m compileall -q scrapling/core/ai.py tests/ai/test_ai_mcp.py
  • git diff --check

@D4Vinci D4Vinci added the AI-Slop A lack of knowledge of the library, with the AI used poorly to create this ticket. label May 8, 2026
@D4Vinci D4Vinci closed this May 8, 2026
@D4Vinci D4Vinci added not planned and removed AI-Slop A lack of knowledge of the library, with the AI used poorly to create this ticket. labels May 8, 2026
@D4Vinci

D4Vinci commented May 8, 2026

Copy link
Copy Markdown
Owner

Hi @Bortlesboat
Sorry, this feature is not planned for addition. I will close this issue too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature Request]: MCP server should support a config option for persistent cookie storage via user_data_dir

2 participants