Skip to content

refactor: Split docs_tools.py into focused modules by responsibility#6

Open
juansepl wants to merge 1 commit intomainfrom
refactor/gdocs-split-by-responsibility
Open

refactor: Split docs_tools.py into focused modules by responsibility#6
juansepl wants to merge 1 commit intomainfrom
refactor/gdocs-split-by-responsibility

Conversation

@juansepl
Copy link
Copy Markdown
Collaborator

  • Split monolithic 1,302-line file into 4 focused modules
  • cache.py: Document caching logic (~50 lines)
  • utils.py: Helper utilities for URL parsing (~100 lines)
  • content_processor.py: Document parsing & formatting (~450 lines)
  • tools.py: MCP tool definitions (~750 lines)

Benefits:

  • Improved AI context efficiency (load only relevant modules)
  • Better maintainability (changes isolated by concern)
  • Clearer dependencies (explicit imports)
  • Easier navigation and testing

All functionality preserved. Original file backed up as docs_tools.py.backup

…bility

- Split monolithic 1,302-line file into 4 focused modules
- cache.py: Document caching logic (~50 lines)
- utils.py: Helper utilities for URL parsing (~100 lines)
- content_processor.py: Document parsing & formatting (~450 lines)
- tools.py: MCP tool definitions (~750 lines)

Benefits:
- Improved AI context efficiency (load only relevant modules)
- Better maintainability (changes isolated by concern)
- Clearer dependencies (explicit imports)
- Easier navigation and testing

All functionality preserved. Original file backed up as docs_tools.py.backup
@juansepl juansepl self-assigned this Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant