Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Always support JSON mode #82

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Always support JSON mode #82

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
8dcc039
Update main.ts
jdriscoll98 Jul 26, 2024
4dc001a
Merge pull request #1 from jdriscoll98/jdriscoll98/always-support-jso…
jdriscoll98 Jul 26, 2024
a31be9e
add check for finsih reason
jdriscoll98 Jul 26, 2024
77fb263
add check for # of files
jdriscoll98 Jul 26, 2024
62fdb56
log the response
jdriscoll98 Jul 27, 2024
f238c27
Fix edge case for json comment
jdriscoll98 Jul 27, 2024
ea67fbb
update prompt
jdriscoll98 Jul 29, 2024
8295118
tweak prompt
jdriscoll98 Jul 30, 2024
64f5674
tweak prompt
jdriscoll98 Jul 30, 2024
b44283c
tweak prompt
jdriscoll98 Jul 31, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix edge case for json comment
  • Loading branch information
@jdriscoll98
jdriscoll98 committed Jul 27, 2024
commit f238c27c26c4917930d8283bd331bd42135b1171
6 changes: 5 additions & 1 deletion src/main.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,11 @@ async function getAIResponse(prompt: string): Promise<Array<{
}

const res = response.choices[0].message?.content?.trim() || "{}";
return JSON.parse(res).reviews;
if (res.startsWith("```json")) {
return JSON.parse(res.slice(7, -3)).reviews
} else {
return JSON.parse(res).reviews;
}
} catch (error) {
console.error("Error:", error, response?.choices[0].message?.content);
return null;
Expand Down