[Bug]: Request failed error code 400

[Kevdome3000]

Opencommit Version

1.1.48

Node Version

19.8.1

NPM Version

9.63

What OS are you seeing the problem on?

Mac

What happened?

used opencommit from cli with a small commit and worked perfectly. Followed up with a larger commit, and received error stating commit was too large. So I reduced the size of the commit and tried again, and received this error: Error: Request failed with status code 400

I've received the request failed error every commit since.

For what it's worth, aicommits exhibits the same behavior

Expected Behavior

I'd expect it to be able to recover from an error received from open ai when a prompt contains too many tokens

Current Behavior

get's stuck in request failed loop

Possible Solution

No response

Steps to Reproduce

No response

Relevant log output

No response

[suryaelidanto]

Same here, can anyone resolve it?

[TaQuangKhoi]

it happen when the files i staged is very big..

[di-sukharev]

could anyone pls share output of git diff?

[kamushadenes]

That's because the model context window is exceeded. Implementing https://github.com/di-sukharev/opencommit/issues/114 would fix this as well.

[di-sukharev]

@kamushadenes thanks bro! we already split large files into smaller commits, im not sure why the 400 is still happening, do you want to investigate and PR?

[kamushadenes]

Can you post the git diff output? It could be something more complex like binary files maybe?

[crsepulv]

I have a lot of not committed files and failed, I git commit as usual ...and following OC's commits worked, I think that too much commited files exhaust openai maximum input size for api messages.

[github-actions[bot]]

Stale issue message

[AlanNevill]

Request failed with status code 400.

{"model":"gpt-3.5-turbo","messages":[{"role":"system","content":"You are to act as the author of a commit message in git. Your mission is to create clean and comprehensive commit messages in the conventional commit convention and explain WHAT were the changes and WHY the changes were done. I'll send you an output of 'git diff --staged' command, and you convert it into a commit message.\nDo not preface the commit with anything.\nDon't add any descriptions to the commit, only commit message.\nUse the present tense. Lines must not be longer than 74 characters. Use english to answer."},{"role":"user","content":"diff --git a/src/server.ts b/src/server.ts\nindex ad4db42..f3b18a9 100644\n--- a/src/server.ts\n+++ b/src/server.ts\n@@ -10,7 +10,7 @@\nimport {\n initWinstonLogger();\n \n const app = express();\n -const port = 7799;\n +const PORT = 7799;\n \n app.use(express.json());\n \n@@ -34,6 +34,6 @@\napp.use((_, res, next) => {\n // ROUTES\n app.use(PROTECTED_ROUTER_URL, protectedRouter);\n \n -app.listen(port, () => {\n - console.log(Server listening on port ${port});\n +app.listen(process.env.PORT || PORT, () => {\n + console.log(Server listening on port ${PORT});\n });"},{"role":"assistant","content":"fix(server.ts): change port variable case from lowercase port to uppercase PORT to improve semantics\nfeat(server.ts): add support for process.env.PORT environment variable to be able to run app on a configurable port\n"},{"role":"user","content":"diff --git a/Pages/Tcat1sList.razor b/Pages/Tcat1sList.razor\nindex cc50ede..c9f8771 100644\n--- a/Pages/Tcat1sList.razor\n+++ b/Pages/Tcat1sList.razor\n@@ -4,9 +4,9 @@\n

Tcat1sList

\n Tcat1sList\n \n+@inject FinanceContext financeContext\n @inject TcatService tcatService\n @inject ToastService toastService\n-@inject IDbContextFactory DbFactory\n @inject NavigationManager navigationManager\n @inject IJSRuntime jsRuntime\n \n@@ -22,8 +22,8 @@ else\n <div class=\"row\">\n <div class=\"col-5\">\n <div class=\"e-statustext\">Main categories