Skip to content

Refactor context transfer to knowledge transfer terminology (the code this time) #585

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

payneio
Copy link
Contributor

@payneio payneio commented May 20, 2025

This pull request introduces a comprehensive renaming and refactoring effort to replace the term "Context Transfer" with "Knowledge Transfer" across the codebase for better alignment with the assistant's functionality. The changes include updates to identifiers, file names, configuration models, and logic handling.

Terminology Update: "Context Transfer" → "Knowledge Transfer"

  • Renamed CONTEXT_TRANSFER_TEMPLATE_ID to KNOWLEDGE_TRANSFER_TEMPLATE_ID and updated its usage in assistants/project-assistant/assistant/chat.py and related files. [1] [2] [3]
  • Replaced is_context_transfer_assistant with is_knowledge_transfer_assistant for determining assistant roles in multiple files, including chat.py and state_inspector.py. [1] [2]

Configuration and Model Changes

  • Renamed ContextTransferConfigModel to KnowledgeTransferConfigModel and updated all references, including imports and usage in config.py and configs/__init__.py. [1] [2]
  • Refactored context_transfer.py to knowledge_transfer.py and renamed related classes (ContextTransferPromptConfig, ContextTransferCoordinatorConfig, ContextTransferTeamConfig) to reflect the new terminology. [1] [2] [3]

Prompt and Template Adjustments

  • Updated prompt configuration fields in PromptConfig to use more descriptive names like coordinator_role and team_role instead of generic terms like instruction_prompt.
  • Adjusted file references for loading text templates, such as context_transfer_whiteboard_prompt.txtknowledge_transfer_whiteboard_prompt.txt. [1] [2]

Enum and Role Logic Updates

  • Renamed CONTEXT_TRANSFER_ASSISTANT to KNOWLEDGE_TRANSFER_ASSISTANT in the ConfigurationTemplate enum and updated logic in get_template and related methods. [1] [2]

State Inspector and Markdown Updates

  • Modified logic in state_inspector.py to reflect the new terminology, including updates to display names and descriptions for knowledge transfer mode. [1] [2]
  • Adjusted markdown generation to use "Knowledge" instead of "Context" in section headers and descriptions. [1] [2]

These changes ensure consistency across the codebase while improving clarity and alignment with the assistant's purpose.…ment new configurations for knowledge sharing assistant

…ment new configurations for knowledge sharing assistant
@payneio payneio closed this Jun 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant