Skip to content

Papayne/project assistant transfer ids #586

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

Merged

Conversation

payneio
Copy link
Contributor

@payneio payneio commented May 20, 2025

This pull request introduces significant changes to the project-assistant codebase, focusing on renaming and refactoring the "Context Transfer" feature to "Knowledge Transfer," improving logging practices by reducing verbosity, and cleaning up unused or redundant code. These updates enhance clarity, maintainability, and alignment with the new terminology.

Renaming and Refactoring "Context Transfer" to "Knowledge Transfer":

  • Renamed all instances of ContextTransfer to KnowledgeTransfer, including class names, file names, and variables. For example, ContextTransferConfigModel was renamed to KnowledgeTransferConfigModel, and the file context_transfer.py was renamed to knowledge_transfer.py. [1] [2] [3] [4]
  • Updated references to context-specific assets (e.g., context_transfer_card_content.md and context_transfer_whiteboard_prompt.txt) to their knowledge-specific counterparts. [1] [2]

Logging Improvements:

  • Reduced logging verbosity by changing many logger.info statements to logger.debug, especially for routine operations like file handling and conversation association. [1] [2] [3]
  • Removed redundant logging statements for operations like storing, updating, and deleting coordinator files. [1] [2] [3]

Code Cleanup and Simplification:

  • Removed the ConfigurationTemplate class and the get_template function from project_common.py, as they were no longer necessary after the refactor.
  • Consolidated imports and removed unused ones across several files to improve code readability. [1] [2]

Configuration Updates:

  • Updated the PromptConfig schema to reflect the new structure, replacing fields like instruction_prompt with coordinator_role and team_role.
  • Adjusted the assistant configuration to use the new KnowledgeTransferConfigModel instead of the deprecated ContextTransferConfigModel. [1] [2]

Asset and Template Updates:

  • Updated the content_evaluator_factory and dashboard cards to use the new KNOWLEDGE_TRANSFER_TEMPLATE_ID and associated assets. [1] [2]

These changes collectively streamline the codebase, align terminology with the updated feature name, and improve maintainability.

payneio added 2 commits May 20, 2025 13:51
…ment new configurations for knowledge sharing assistant
…t and clarity. Standardize on template_id constant handling.
@payneio payneio merged commit 67bd849 into microsoft:main May 20, 2025
4 checks passed
@payneio payneio deleted the papayne/project-assistant-transfer-ids branch July 8, 2025 23:53
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