di-sukharev / opencommit

Auto-generate impressive commits with AI in 1 second 🤯🔫
https://www.npmjs.com/package/opencommit
MIT License
5.66k stars 291 forks source link

[Bug]: └ ✖ Request failed with status code 404 #267

Open s95268 opened 8 months ago

s95268 commented 8 months ago

Opencommit Version

1.0.2

Node Version

v18.18.2

NPM Version

9.8.1

What OS are you seeing the problem on?

Other Linux Distro

What happened?

pinger@pinger:~/IdeaProjects/Mines$ oco version ┌ open-commit │ ◇ 2 staged files: README.md src/main/java/fr/pinger/mines/handlers/PapiHandler.java │ ◓ Generating the commit message
│ └ ✖ {"model":"gpt-4","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 as per the conventional commit convention and explain WHAT were the changes and mainly WHY the changes were done. I'll send you an output of 'git diff --staged' command, and you are to convert it into a commit message.\n Use GitMoji convention to preface the commit.\n Don't add any descriptions to the commit, only commit message.\n Use the present tense. Lines must not be longer than 74 characters. Use française for the commit message."},{"role":"user","content":"diff --git a/src/server.ts b/src/server.ts\n index ad4db42..f3b18a9 100644\n --- a/src/server.ts\n +++ b/src/server.ts\n @@ -10,7 +10,7 @@\n import {\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 @@\n app.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":"🐛 corriger(server.ts) : changer la casse de la variable de port de minuscules à majuscules (PORT)\n✨ fonctionnalité(server.ts) : ajouter la prise en charge de la variable d'environnement process.env.PORT\n"},{"role":"user","content":"diff --git a/README.md b/README.md\ndeleted file mode 100644\nindex db122c4..0000000\n--- a/README.md\n+++ /dev/null\n@@ -1 +0,0 @@\n-# Prison-Mine\ndiff --git a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\nindex a7832b8..ad6783d 100644\n--- a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n+++ b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n@@ -1,4 +1,3 @@\n-\n package fr.pinger.mines.handlers;\n \n import fr.pinger.mines.mine.Mine;\n@@ -7,46 +6,49 @@ import org.bukkit.entity.Player;\n import me.clip.placeholderapi.expansion.PlaceholderExpansion;\n \n public class PapiHandler extends PlaceholderExpansion {\n+\n+ @Override\n public String getIdentifier() {\n return \"Mines\";\n }\n \n+ @Override\n public String getAuthor() {\n return \"Pinger\";\n }\n \n+ @Override\n public String getVersion() {\n return \"1.0.0\";\n }\n \n+ @Override\n public String onPlaceholderRequest(final Player p, final String identifier) {\n for (final Mine mine : Mines.getSystem().getMines()) {\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_name\")) {\n- return mine.getName();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_timeuntilreset\")) {\n- return mine.getFormattedTimeUntilReset();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksbroken\")) {\n- return new StringBuilder().append(mine.getBlocksBroken()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksleft\")) {\n- return new StringBuilder().append(mine.getBlocksRemaining()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksprogressbar\")) {\n- return Mines.getAPI().getProgressBar(mine);\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentleft\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentLeft())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentbroken\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentMined())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentlefttoreset\")) {\n- return new StringBuilder().append(Math.round(100.0 - mine.getResetPercentage())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"percentbrokentoreset\")) {\n- return new StringBuilder().append(Math.round(mine.getResetPercentage())).toString();\n+ String mineName = mine.getName();\n+ if (identifier.startsWith(mineName + \"\")) {\n+ switch (identifier.substring(mineName.length() + 1)) {\n+ case \"name\":\n+ return mineName;\n+ case \"timeuntilreset\":\n+ return mine.getFormattedTimeUntilReset();\n+ case \"blocksbroken\":\n+ return String.valueOf(mine.getBlocksBroken());\n+ case \"blocksleft\":\n+ return String.valueOf(mine.getBlocksRemaining());\n+ case \"blocksprogressbar\":\n+ return Mines.getAPI().getProgressBar(mine);\n+ case \"percentleft\":\n+ return String.valueOf(Math.round(mine.getPercentLeft()));\n+ case \"percentbroken\":\n+ return String.valueOf(Math.round(mine.getPercentMined()));\n+ case \"percentlefttoreset\":\n+ return String.valueOf(Math.round(100.0 - mine.getResetPercentage()));\n+ case \"percentbrokentoreset\":\n+ return String.valueOf(Math.round(mine.getResetPercentage()));\n+ default:\n+ continue;\n+ }\n }\n }\n return null;"}],"temperature":0,"top_p":0.1,"max_tokens":500}

│ │ ◇ 📝 Commit message generated │ └ ✖ Request failed with status code 404

Expected Behavior

pinger@pinger:~/IdeaProjects/Mines$ oco version ┌ open-commit │ ◇ 2 staged files: README.md src/main/java/fr/pinger/mines/handlers/PapiHandler.java │ ◓ Generating the commit message
│ └ ✖ {"model":"gpt-4","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 as per the conventional commit convention and explain WHAT were the changes and mainly WHY the changes were done. I'll send you an output of 'git diff --staged' command, and you are to convert it into a commit message.\n Use GitMoji convention to preface the commit.\n Don't add any descriptions to the commit, only commit message.\n Use the present tense. Lines must not be longer than 74 characters. Use française for the commit message."},{"role":"user","content":"diff --git a/src/server.ts b/src/server.ts\n index ad4db42..f3b18a9 100644\n --- a/src/server.ts\n +++ b/src/server.ts\n @@ -10,7 +10,7 @@\n import {\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 @@\n app.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":"🐛 corriger(server.ts) : changer la casse de la variable de port de minuscules à majuscules (PORT)\n✨ fonctionnalité(server.ts) : ajouter la prise en charge de la variable d'environnement process.env.PORT\n"},{"role":"user","content":"diff --git a/README.md b/README.md\ndeleted file mode 100644\nindex db122c4..0000000\n--- a/README.md\n+++ /dev/null\n@@ -1 +0,0 @@\n-# Prison-Mine\ndiff --git a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\nindex a7832b8..ad6783d 100644\n--- a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n+++ b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n@@ -1,4 +1,3 @@\n-\n package fr.pinger.mines.handlers;\n \n import fr.pinger.mines.mine.Mine;\n@@ -7,46 +6,49 @@ import org.bukkit.entity.Player;\n import me.clip.placeholderapi.expansion.PlaceholderExpansion;\n \n public class PapiHandler extends PlaceholderExpansion {\n+\n+ @Override\n public String getIdentifier() {\n return \"Mines\";\n }\n \n+ @Override\n public String getAuthor() {\n return \"Pinger\";\n }\n \n+ @Override\n public String getVersion() {\n return \"1.0.0\";\n }\n \n+ @Override\n public String onPlaceholderRequest(final Player p, final String identifier) {\n for (final Mine mine : Mines.getSystem().getMines()) {\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_name\")) {\n- return mine.getName();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_timeuntilreset\")) {\n- return mine.getFormattedTimeUntilReset();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksbroken\")) {\n- return new StringBuilder().append(mine.getBlocksBroken()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksleft\")) {\n- return new StringBuilder().append(mine.getBlocksRemaining()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksprogressbar\")) {\n- return Mines.getAPI().getProgressBar(mine);\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentleft\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentLeft())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentbroken\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentMined())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentlefttoreset\")) {\n- return new StringBuilder().append(Math.round(100.0 - mine.getResetPercentage())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"percentbrokentoreset\")) {\n- return new StringBuilder().append(Math.round(mine.getResetPercentage())).toString();\n+ String mineName = mine.getName();\n+ if (identifier.startsWith(mineName + \"\")) {\n+ switch (identifier.substring(mineName.length() + 1)) {\n+ case \"name\":\n+ return mineName;\n+ case \"timeuntilreset\":\n+ return mine.getFormattedTimeUntilReset();\n+ case \"blocksbroken\":\n+ return String.valueOf(mine.getBlocksBroken());\n+ case \"blocksleft\":\n+ return String.valueOf(mine.getBlocksRemaining());\n+ case \"blocksprogressbar\":\n+ return Mines.getAPI().getProgressBar(mine);\n+ case \"percentleft\":\n+ return String.valueOf(Math.round(mine.getPercentLeft()));\n+ case \"percentbroken\":\n+ return String.valueOf(Math.round(mine.getPercentMined()));\n+ case \"percentlefttoreset\":\n+ return String.valueOf(Math.round(100.0 - mine.getResetPercentage()));\n+ case \"percentbrokentoreset\":\n+ return String.valueOf(Math.round(mine.getResetPercentage()));\n+ default:\n+ continue;\n+ }\n }\n }\n return null;"}],"temperature":0,"top_p":0.1,"max_tokens":500}

│ │ ◇ 📝 Commit message generated │ └ ✖ Request failed with status code 404

Current Behavior

pinger@pinger:~/IdeaProjects/Mines$ oco version ┌ open-commit │ ◇ 2 staged files: README.md src/main/java/fr/pinger/mines/handlers/PapiHandler.java │ ◓ Generating the commit message
│ └ ✖ {"model":"gpt-4","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 as per the conventional commit convention and explain WHAT were the changes and mainly WHY the changes were done. I'll send you an output of 'git diff --staged' command, and you are to convert it into a commit message.\n Use GitMoji convention to preface the commit.\n Don't add any descriptions to the commit, only commit message.\n Use the present tense. Lines must not be longer than 74 characters. Use française for the commit message."},{"role":"user","content":"diff --git a/src/server.ts b/src/server.ts\n index ad4db42..f3b18a9 100644\n --- a/src/server.ts\n +++ b/src/server.ts\n @@ -10,7 +10,7 @@\n import {\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 @@\n app.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":"🐛 corriger(server.ts) : changer la casse de la variable de port de minuscules à majuscules (PORT)\n✨ fonctionnalité(server.ts) : ajouter la prise en charge de la variable d'environnement process.env.PORT\n"},{"role":"user","content":"diff --git a/README.md b/README.md\ndeleted file mode 100644\nindex db122c4..0000000\n--- a/README.md\n+++ /dev/null\n@@ -1 +0,0 @@\n-# Prison-Mine\ndiff --git a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\nindex a7832b8..ad6783d 100644\n--- a/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n+++ b/src/main/java/fr/pinger/mines/handlers/PapiHandler.java\n@@ -1,4 +1,3 @@\n-\n package fr.pinger.mines.handlers;\n \n import fr.pinger.mines.mine.Mine;\n@@ -7,46 +6,49 @@ import org.bukkit.entity.Player;\n import me.clip.placeholderapi.expansion.PlaceholderExpansion;\n \n public class PapiHandler extends PlaceholderExpansion {\n+\n+ @Override\n public String getIdentifier() {\n return \"Mines\";\n }\n \n+ @Override\n public String getAuthor() {\n return \"Pinger\";\n }\n \n+ @Override\n public String getVersion() {\n return \"1.0.0\";\n }\n \n+ @Override\n public String onPlaceholderRequest(final Player p, final String identifier) {\n for (final Mine mine : Mines.getSystem().getMines()) {\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_name\")) {\n- return mine.getName();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_timeuntilreset\")) {\n- return mine.getFormattedTimeUntilReset();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksbroken\")) {\n- return new StringBuilder().append(mine.getBlocksBroken()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksleft\")) {\n- return new StringBuilder().append(mine.getBlocksRemaining()).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_blocksprogressbar\")) {\n- return Mines.getAPI().getProgressBar(mine);\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentleft\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentLeft())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentbroken\")) {\n- return new StringBuilder().append(Math.round(mine.getPercentMined())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"_percentlefttoreset\")) {\n- return new StringBuilder().append(Math.round(100.0 - mine.getResetPercentage())).toString();\n- }\n- if (identifier.equalsIgnoreCase(String.valueOf(mine.getName()) + \"percentbrokentoreset\")) {\n- return new StringBuilder().append(Math.round(mine.getResetPercentage())).toString();\n+ String mineName = mine.getName();\n+ if (identifier.startsWith(mineName + \"\")) {\n+ switch (identifier.substring(mineName.length() + 1)) {\n+ case \"name\":\n+ return mineName;\n+ case \"timeuntilreset\":\n+ return mine.getFormattedTimeUntilReset();\n+ case \"blocksbroken\":\n+ return String.valueOf(mine.getBlocksBroken());\n+ case \"blocksleft\":\n+ return String.valueOf(mine.getBlocksRemaining());\n+ case \"blocksprogressbar\":\n+ return Mines.getAPI().getProgressBar(mine);\n+ case \"percentleft\":\n+ return String.valueOf(Math.round(mine.getPercentLeft()));\n+ case \"percentbroken\":\n+ return String.valueOf(Math.round(mine.getPercentMined()));\n+ case \"percentlefttoreset\":\n+ return String.valueOf(Math.round(100.0 - mine.getResetPercentage()));\n+ case \"percentbrokentoreset\":\n+ return String.valueOf(Math.round(mine.getResetPercentage()));\n+ default:\n+ continue;\n+ }\n }\n }\n return null;"}],"temperature":0,"top_p":0.1,"max_tokens":500}

│ │ ◇ 📝 Commit message generated │ └ ✖ Request failed with status code 404

Possible Solution

No response

Steps to Reproduce

No response

Relevant log output

No response

ThomAille commented 7 months ago

@WePico I have the same problem, can you find a solution?

eallion commented 7 months ago

Same

JoseAFlores777 commented 6 months ago

Is there any solution to this?

github-actions[bot] commented 5 months ago

Stale issue message

di-sukharev commented 4 months ago

reopening, will take a look at this