chore(web): trim verbose comments from review fixes

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: whoisthey

packages/web/src/ee/features/chat/components/chatThread/referencedFileSourceListItemContainer.tsx

@@ -52,10 +52,9 @@ const ReferencedFileSourceListItemContainerComponent = ({
                 ref: fetchRef,
             });
 
-            // The pinned commit can disappear (e.g. a force-push + GC prunes it),
-            // which surfaces as an unresolvable git ref. Only that case falls
-            // back to the symbolic ref; other errors (repo/path/access) are
-            // surfaced as-is so we don't silently render the wrong revision.
+            // A gone pinned commit (e.g. force-push + GC) surfaces as an
+            // unresolvable ref. Only then fall back to the symbolic ref; other
+            // errors are surfaced as-is rather than silently showing latest.
             if (
                 isServiceError(pinned) &&
                 pinned.errorCode === ErrorCode.INVALID_GIT_REF &&

packages/web/src/features/git/getFileSourceApi.test.ts

@@ -92,9 +92,7 @@ describe('getFileSourceForRepo', () => {
         mockSimpleGit.mockReturnValue({ cwd: mockCwd });
         mockFindFirst.mockResolvedValue(MOCK_REPO);
 
-        // Default: ref resolves to a concrete sha, file show succeeds, and
-        // .gitattributes is absent. The SUT resolves the ref first (rev-parse),
-        // then reads content + .gitattributes at the resolved sha.
+        // ref resolves to a sha, file show succeeds, .gitattributes is absent.
         mockGitRaw.mockImplementation(async (args: string[]) => {
             if (args[0] === 'rev-parse') {
                 return 'resolvedsha\n';
@@ -203,7 +201,7 @@ describe('getFileSourceForRepo', () => {
             // pr_payload.head_sha as ref, but the bare clone hasn't fetched it yet.
             mockGitRaw.mockRejectedValue(
                 new Error("fatal: ambiguous argument 'deadbeef': unknown revision or path not in the working tree"),
-            ); // rejects rev-parse (swallowed) and the show, which drives the result
+            );
 
             const result = await getFileSourceForRepo(
                 { path: 'src/index.ts', repo: 'github.com/owner/repo', ref: 'deadbeef' },

packages/web/src/features/git/getFileSourceApi.ts

@@ -47,15 +47,14 @@ export const getFileSourceForRepo = async (
 
     const gitRef = ref ?? repo.defaultBranch ?? 'HEAD';
 
-    // Resolve the symbolic ref to a concrete commit up front so the content,
-    // language, and commitSha all come from the same revision even if the ref
-    // moves mid-request. `^{commit}` peels annotated tags. Reads below fall back
-    // to the symbolic ref when resolution fails.
+    // Resolve to a concrete commit up front so content, language, and commitSha
+    // all come from one revision even if the ref moves mid-request. `^{commit}`
+    // peels annotated tags. Reads below fall back to the symbolic ref.
     let commitSha: string | undefined;
     try {
         commitSha = (await git.raw(['rev-parse', `${gitRef}^{commit}`])).trim();
     } catch {
-        // Leave unpinned; the reads below use the symbolic ref.
+        // Leave unpinned.
     }
     const readRef = commitSha ?? gitRef;
 

packages/web/src/features/tools/findSymbolDefinitions.ts

@@ -63,9 +63,8 @@ export const findSymbolDefinitionsDefinition: ToolDefinition<
             codeHostType: repoInfoResult.codeHostType,
         };
 
-        // Pin to the indexed commit carried by the same search snapshot that
-        // produced these matches, rather than a follow-up repo-info lookup that
-        // could drift if the index advances in between.
+        // Pin from the indexed commit in this search's snapshot, not a separate
+        // repo-info lookup that could drift if the index advances in between.
         const indexedCommitShaByRepo = new Map(
             response.repositoryInfo.map((info) => [info.name, info.indexedCommitHash]),
         );

packages/web/src/features/tools/findSymbolReferences.ts

@@ -74,9 +74,8 @@ export const findSymbolReferencesDefinition: ToolDefinition<
             codeHostType: repoInfoResult.codeHostType,
         };
 
-        // Pin to the indexed commit carried by the same search snapshot that
-        // produced these matches, rather than a follow-up repo-info lookup that
-        // could drift if the index advances in between.
+        // Pin from the indexed commit in this search's snapshot, not a separate
+        // repo-info lookup that could drift if the index advances in between.
         const indexedCommitShaByRepo = new Map(
             response.repositoryInfo.map((info) => [info.name, info.indexedCommitHash]),
         );