Adding strategy for migrating Duo Chat tools to blueprint
Why is this change being made?
Architectural Blueprint for Prompts Migration (!7311 - merged) adds a blueprint for prompt migration. It contains Design and implementation details
section that lists strategies per-feature. Currently, it's Code Completion and Code Generations.
This MR adds a strategy for Duo Chat tools migration.
PoC:
- Execute Duo Chat explain code tool via agents (gitlab-org/gitlab!160252 - merged)
- feat(agents): add prompts for explain code tool (gitlab-org/modelops/applied-ml/code-suggestions/ai-assist!1132 - merged)
Commits
- Adding strategy for migrating Duo Chat tools to blueprint
Merge request reports
Activity
changed milestone to %17.3
added Architecture Evolution Design Doc HandbookContent labels
assigned to @igor.drozdov
1 Message 📖 This merge request might require a review from a Coach Engineer. Architecture Evolution Review
This merge request might require a review from a Coach Engineer.
The following files, which might require the additional review, have been changed:
content/handbook/engineering/architecture/design-documents/prompts_migration/_index.md
If needed, you can retry the
🔁 danger-review
job that generated this comment.Generated by
🚫 DangerEdited by ****- Resolved by Igor Drozdov
- Resolved by Igor Drozdov
@brytannia @alejandro @achueshev I've extracted the section related to the prompt migration of Duo Chat Tools into a separate MR to have a focused discussion about it. I've added the links to the description: link to the general blueprint and PoC of migrating explain code. Does the strategy itself make sense to you, could you please have a look?
/cc @oregand
requested review from @brytannia, @alejandro, and @achueshev
- Resolved by Igor Drozdov
- Resolved by Igor Drozdov
- Resolved by Igor Drozdov
thanks @igor.drozdov for the PoC and this MR! Looks good
✨ I left two questions
added 321 commits
-
89b67f0c...77479064 - 320 commits from branch
main
- 48b70cda - Adding strategy for migrating Duo Chat tools to blueprint
-
89b67f0c...77479064 - 320 commits from branch
added 1 commit
- 21ac2548 - Adding strategy for migrating Duo Chat tools to blueprint
added 112 commits
-
21ac2548...e4092aae - 111 commits from branch
main
- e5a77660 - Adding strategy for migrating Duo Chat tools to blueprint
-
21ac2548...e4092aae - 111 commits from branch
mentioned in merge request gitlab-org/gitlab!160252 (merged)
@mksionek it's related to Execute Duo Chat explain code tool via agents (gitlab-org/gitlab!160252 - merged), could you please have a look as well?
requested review from @mksionek
mentioned in merge request gitlab-org/modelops/applied-ml/code-suggestions/ai-assist!1132 (merged)
mentioned in issue gitlab-org/gitlab#475050 (closed)
mentioned in issue gitlab-org/gitlab#475048 (closed)
mentioned in issue gitlab-org/gitlab#475049 (closed)
mentioned in issue gitlab-org/gitlab#475047 (closed)
mentioned in issue gitlab-org/gitlab#475046 (closed)
mentioned in issue gitlab-org/gitlab#475045 (closed)
mentioned in issue gitlab-org/gitlab#475044 (closed)
mentioned in issue gitlab-org/modelops/applied-ml/code-suggestions/ai-assist#595 (closed)
mentioned in epic gitlab-org&14431 (closed)
removed review request for @brytannia
started a merge train
mentioned in commit 282fd6bf