Create action that reads comments from docs

[liv-s]

No description provided.

[greptile-apps]

Greptile Overview

Greptile Summary

This PR adds a new action readCommentsOnDoc to fetch comments and replies from Google Docs using the Google Drive API. The implementation includes proper schema definitions, type generation, and test coverage.

Key Changes:

• New read-only action with appropriate OAuth scopes (drive.readonly)
• Supports pagination via pageSize and nextPageToken
• Returns comment metadata including author info, timestamps, and quoted content
• Properly integrated into the action mapper registry

Critical Issue:

• The implementation maps API response fields incorrectly - uses c.commentId and r.replyId instead of c.id and r.id, causing all IDs to be undefined in the output

Confidence Score: 1/5

• This PR has a critical bug that will cause the feature to malfunction in production
• The incorrect field mapping (c.commentId instead of c.id) means all comment and reply IDs will be undefined, breaking any functionality that depends on these IDs
• readCommentsOnDoc.ts requires immediate fixes to the field mapping logic

Important Files Changed

Filename	Overview
src/actions/providers/google-oauth/readCommentsOnDoc.ts	New action to read Google Doc comments - critical bug: uses wrong field names (commentId/replyId instead of id) causing all IDs to be undefined

[greptile-apps]

_{1 file reviewed, 3 comments}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

Google Drive API returns id not commentId. This will cause all comment IDs to be undefined.

Suggested change

	res.data.comments?.map((c: any) => ({
	commentId: c.commentId,
	res.data.comments?.map((c: any) => ({
	commentId: c.id,

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/actions/providers/google-oauth/readCommentsOnDoc.ts
Line: 40:41

Comment:
Google Drive API returns `id` not `commentId`. This will cause all comment IDs to be undefined.

```suggestion
      res.data.comments?.map((c: any) => ({
        commentId: c.id,
```

How can I resolve this? If you propose a fix, please make it concise.

[rb0000]

? is this true?

[greptile-apps]

Yes, you're correct. The Google Drive API returns id not commentId. Both line 40 and line 55 need to be fixed:

Suggested change

	res.data.comments?.map((c: any) => ({
	commentId: c.commentId,
	commentId: c.id,
	content: c.content,
	createdTime: c.createdTime,
	modifiedTime: c.modifiedTime,
	resolved: c.resolved,
	author: c.author
	? {
	displayName: c.author.displayName,
	emailAddress: c.author.emailAddress,
	me: c.author.me,
	}
	: undefined,
	quotedFileContent: c.quotedFileContent,
	replies:
	c.replies?.map((r: any) => ({
	replyId: r.id,

The API response uses id for both comments and replies, but your schema correctly expects commentId and replyId in the output.

[greptile-apps]

Google Drive API returns id not replyId for replies.

Suggested change

	replyId: r.replyId,
	replyId: r.id,

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/actions/providers/google-oauth/readCommentsOnDoc.ts
Line: 56:56

Comment:
Google Drive API returns `id` not `replyId` for replies.

```suggestion
            replyId: r.id,
```

How can I resolve this? If you propose a fix, please make it concise.

[greptile-apps]

Error object may contain sensitive information and should be handled carefully. Consider logging only non-sensitive error details.

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/actions/providers/google-oauth/readCommentsOnDoc.ts
Line: 76:76

Comment:
Error object may contain sensitive information and should be handled carefully. Consider logging only non-sensitive error details.

How can I resolve this? If you propose a fix, please make it concise.