fix(Search): missing PR details when searching (#163)

* fix(Search): missing PR details when searching * Bot - Fix Formatting --------- Co-authored-by: homie bot <[email protected]>
homie-gg · Jul 29, 2024 · e105b06 · e105b06
1 parent 07b633b
commit e105b06
Show file tree

Hide file tree

Showing 5 changed files with 17 additions and 13 deletions.
diff --git a/src/lib/ai/chat/tools/get-search-pull-requests-tool.ts b/src/lib/ai/chat/tools/get-search-pull-requests-tool.ts
@@ -102,9 +102,9 @@ export function getSearchPullRequestsTool(
           ),
         })
 
-        const rankedDocuments = reranked.results.filter(
-          (result) => result.relevanceScore > searchRelevanceThreshold,
-        )
+        const rankedDocuments = reranked.results
+          .filter((result) => result.relevanceScore > searchRelevanceThreshold)
+          .map((result) => matches[result.index].metadata) // return metadata as results
 
         logger.debug('Ranked and filtered results', {
           event: 'get_answer:search_pull_requests:ranked_results',

diff --git a/src/lib/ai/embed-code-change.ts b/src/lib/ai/embed-code-change.ts
@@ -44,6 +44,7 @@ export async function embedCodeChange(params: EmbedCodeChangeParams) {
       metadata: {
         ...metadata,
         text,
+        code_change: point,
       },
     }
 

diff --git a/src/lib/ai/embed-diff.ts b/src/lib/ai/embed-diff.ts
@@ -52,6 +52,9 @@ export async function embedDiff(params: EmbedDiffParams) {
         metadata: {
           ...metadata,
           text,
+          code_snippet: snippet,
+          pull_request_title: title,
+          pull_request_url: url,
         },
       }
 

diff --git a/src/lib/github/save-merged-pull-request.ts b/src/lib/github/save-merged-pull-request.ts
@@ -154,10 +154,10 @@ export async function saveMergedPullRequest(
   const wasMergedToDefaultBranch =
     pullRequest.base.ref === pullRequest.base.repo.default_branch
 
-  const embed_metadata = {
+  const embedMetadata = {
     type: 'pr_summary',
-    title: pullRequest.title,
-    url: pullRequest.html_url,
+    pull_request_title: pullRequest.title,
+    pull_request_url: pullRequest.html_url,
     ext_gh_pull_request_id: pullRequest.id,
     organization_id: organization.id,
     contributor_id: contributor.id,
@@ -182,7 +182,7 @@ export async function saveMergedPullRequest(
       merged_at: parseISO(pullRequest.merged_at),
       number: pullRequest.number,
       embed_value: summary,
-      embed_metadata,
+      embed_metadata: embedMetadata,
       source_branch: pullRequest.head.ref,
       target_branch: pullRequest.base.ref,
       was_merged_to_default_branch: wasMergedToDefaultBranch,
@@ -199,7 +199,7 @@ export async function saveMergedPullRequest(
         merged_at: parseISO(pullRequest.merged_at!),
         number: pullRequest.number,
         embed_value: summary,
-        embed_metadata,
+        embed_metadata: embedMetadata,
         source_branch: pullRequest.head.ref,
         target_branch: pullRequest.base.ref,
         was_merged_to_default_branch: wasMergedToDefaultBranch,
@@ -217,7 +217,7 @@ export async function saveMergedPullRequest(
     title: pullRequest.title,
     url: pullRequest.html_url,
     summary,
-    metadata: embed_metadata,
+    metadata: embedMetadata,
     contributor: pullRequest.user.login,
     mergedAt: pullRequestRecord.merged_at,
   })

diff --git a/src/lib/gitlab/save-merged-merge-request.ts b/src/lib/gitlab/save-merged-merge-request.ts
@@ -99,10 +99,10 @@ export async function saveMergedMergeRequest(
 
   const wasMergedToDefaultBranch = mergeRequest.target_branch === defaultBranch
 
-  const embed_metadata = {
+  const embedMetadata = {
     type: 'pr_summary',
-    title: mergeRequest.title,
-    url: mergeRequest.web_url,
+    pull_request_title: mergeRequest.title,
+    pull_request_url: mergeRequest.web_url,
     ext_gitlab_merge_request_id: mergeRequest.id,
     organization_id: organization.id,
     contributor_id: contributor.id,
@@ -161,7 +161,7 @@ export async function saveMergedMergeRequest(
     title: pullRequestRecord.title,
     url: pullRequestRecord.html_url,
     summary,
-    metadata: embed_metadata,
+    metadata: embedMetadata,
     contributor: mergeRequest.author.username,
     mergedAt: pullRequestRecord.merged_at,
   })
-Original file line number
+Diff line change
@@ Expand Up @@
           metadata: {
             ...metadata,
             text,
+            code_change: point,
           },
         }
@@ Expand Down @@