genai comments #605
genai comments #605
Conversation
|
/test |
|
/genai-foobar |
1 similar comment
|
/genai-foobar |
|
/genai-foo |
|
/genai-foobar |
1 similar comment
|
/genai-foobar |
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| ...rest, | ||
| timestamp: new Date(timestamp), | ||
| reactions: reactions.map((r) => ({ ...r, iconPath: "" })), | ||
| mode: vscode.CommentMode.Preview, |
There was a problem hiding this comment.
The iconPath for the reactions is set to an empty string. This might lead to missing icons for the reactions in the comments.
generated by pr-review-commit
missing_icon_path
| thread.canReply = true | ||
| thread.state = resolved | ||
| ? vscode.CommentThreadState.Resolved | ||
| : vscode.CommentThreadState.Unresolved |
There was a problem hiding this comment.
The state of the comment thread is either set to Resolved or Unresolved. There might be other states that are not handled.
generated by pr-review-commit
unhandled_state
| ? vscode.CommentThreadState.Resolved | ||
| : vscode.CommentThreadState.Unresolved | ||
| } | ||
| }) |
There was a problem hiding this comment.
There is no error handling in the workspace change event. If an error occurs while processing the entries, it might lead to unexpected behavior.
generated by pr-review-commit
missing_error_handling
| @@ -57,6 +58,7 @@ async function resolveExpansionVars( | |||
| secrets[secret] = value | |||
| } else trace.error(`secret \`${secret}\` not found`) | |||
| } | |||
| const comments = await commentsForSource(template.id) | |||
There was a problem hiding this comment.
You might have forgotten to use 'await' before calling the async function 'commentsForSource'.
generated by pr-review-commit
missing_await
| const { reactions, timestamp, ...rest } = comment | ||
| return { | ||
| ...rest, | ||
| timestamp: new Date(timestamp), |
There was a problem hiding this comment.
You should validate the 'timestamp' before converting it to a Date object to avoid potential runtime errors.
generated by pr-review-commit
missing_date_validation
| const r = new JSONLineCache<K, V>(name) | ||
| host.userState[key] = r | ||
| if (!snapshot) host.userState[key] = r | ||
| return r |
There was a problem hiding this comment.
The function byName does not handle the case when options is undefined. This could lead to a TypeError when trying to destructure snapshot from options. Consider adding a default value for options in the function parameters or a check for options before destructuring. 🛠️
generated by pr-review-commit
missing_error_handling
| const cache = commentsCache() | ||
| const entries = await cache.entries() | ||
| return entries.map(({ val }) => val).filter((c) => c.source === source) | ||
| } |
There was a problem hiding this comment.
The function commentsCache does not handle the case when options is undefined. This could lead to a TypeError when passing options to JSONLineCache.byName. Consider adding a default value for options in the function parameters or a check for options before passing it to JSONLineCache.byName. 🛠️
generated by pr-review-commit
missing_error_handling
No description provided.