Compare commits

..

2 Commits

Author SHA1 Message Date
di-sukharev
e465d59b64 0.0.22 2023-03-06 22:39:39 +08:00
di-sukharev
01e6646b30 * 🎨 style(README.md): update OpenCommit logo filename
* 📝 docs(README.md): add examples section
The OpenCommit logo filename has been updated to match the new filename. The README.md file now includes an examples section that provides a link to the commits on the OpenCommit GitHub repository. This section also mentions that the emoji and long commit description text are configurable.

* 📝 chore(TODO.md): update TODO list
* 🐛 fix(TODO.md): add missing checkbox for a TODO item
The TODO list has been updated with new items. The bundle size can be reduced by properly configuring esbuild. The code has some TODOs that need to be addressed. Small files can be batched in one request to improve performance. Tests need to be added to ensure the code works as expected. Finally, a hook needs to be made to work.

* ⬆️ chore(package.json): update version to 0.0.21
* 🐛 fix(package.json): change cli file extension from .mjs to .cjs
* 🚀 feat(package.json): add repository and deploy scripts
The version number has been updated to 0.0.21. The cli file extension has been changed from .mjs to .cjs to improve compatibility with Node.js. A repository field has been added to the package.json file to specify the location of the project's repository. A deploy script has been added to build the project, increment the patch version number, and publish the package to the npm registry.

* 🎨 style(api.ts): remove unused import and commented code
* 🐛 fix(api.ts): remove prompt for OPENAI_API_KEY and add instructions to set it manually
The unused import of `ChatCompletionRequestMessage` and `ChatCompletionResponseMessage` has been removed. The commented code for prompting the user for `OPENAI_API_KEY` has also been removed. Instead, instructions have been added to set the `OPENAI_API_KEY` manually by running `oc config set OPENAI_API_KEY=<your token>`. This improves the user experience by providing clear instructions on how to set the `OPENAI_API_KEY` and removes the unnecessary prompt.

* 🐛 fix(cli.ts): remove assert statement from packageJSON import
The assert statement is not necessary for importing the package.json file. Removing it improves the readability of the code.

* 🐛 fix(githook.ts): use __filename instead of fileURLToPath(new URL('cli.mjs', import.meta.url))
*  feat(githook.ts): add error handling for fs.realpath call
The fileURLToPath function is not necessary as __filename already provides the path to the current file. The error handling for the fs.realpath call is added to prevent the program from crashing if the symlink does not exist.

* 🐛 fix(generateCommitMessageFromGitDiff.ts): fix length calculation for prompt messages
* 💄 style(generateCommitMessageFromGitDiff.ts): add whitespace to improve readability
The length calculation for the prompt messages was incorrect, causing the maximum request token limit to be exceeded. This has been fixed by correctly calculating the length of the prompt messages. Additionally, whitespace has been added to improve the readability of the code.

* 🐛 fix(generateCommitMessageWithChatCompletion): fix commit message length check
* 🚀 chore(generateCommitMessageWithChatCompletion): add TODO for edge case handling
The commit message length check has been fixed to include the length of the initial messages prompt. This ensures that the commit message length is correctly calculated and prevents the commit message from exceeding the maximum required tokens. A TODO has been added to handle an edge case.

* 🐛 fix(tsconfig.json): remove commented out rootDir property
* 🚀 chore(tsconfig.json): update compiler options
The rootDir property was commented out and not being used, so it was removed. The compiler options were updated to target ES2020, use CommonJS module, and allow importing of JSON files. The outDir property was set to "./out". The strict option was enabled along with noUnusedLocals and noUnusedParameters. The skipLibCheck option was set to true.
2023-03-06 22:39:18 +08:00
6 changed files with 36 additions and 39 deletions

View File

@@ -13,7 +13,7 @@
## Examples
Look into [the commits](https://github.com/di-sukharev/opencommit/commit/4795a695800686c42e7c72fca7569103d21cd510) to see how OpenCommit works. Emoji and long commit description text is configurable.
Look into [the commits](https://github.com/di-sukharev/opencommit/commit) to see how OpenCommit works. Emoji and long commit description text is configurable.
## Setup
@@ -83,7 +83,7 @@ To remove description:
opencommit config set description=false
```
<!-- ## Git hook
## Git hook
You can set opencommit as Git [`prepare-commit-msg`](https://git-scm.com/docs/githooks#_prepare_commit_msg) hook. Hook integrates with you IDE Source Control and allows you edit the message before commit.
@@ -106,7 +106,7 @@ To use the hook:
git commit
```
Or follow the process of your IDE Source Control feature, when it calls `git commit` command — OpenCommit will integrate into the flow. -->
Or follow the process of your IDE Source Control feature, when it calls `git commit` command — OpenCommit will integrate into the flow.
## Payments

4
package-lock.json generated
View File

@@ -1,12 +1,12 @@
{
"name": "open-commit",
"version": "0.0.12",
"version": "0.0.22",
"lockfileVersion": 3,
"requires": true,
"packages": {
"": {
"name": "open-commit",
"version": "0.0.12",
"version": "0.0.22",
"license": "ISC",
"dependencies": {
"@clack/prompts": "^0.6.1",

View File

@@ -1,6 +1,6 @@
{
"name": "opencommit",
"version": "0.0.12",
"version": "0.0.22",
"description": "AI generates conventional commits with mind-blowing accuracy.",
"keywords": [
"git",
@@ -37,7 +37,7 @@
"start": "node ./out/cli.cjs",
"dev": "ts-node ./src/cli.ts",
"build": "rimraf out && esbuild ./src/cli.ts --bundle --outfile=out/cli.cjs --format=cjs --platform=node",
"patch": "npm version patch",
"deploy": "npm run build && npm version patch && npm publish",
"lint": "eslint src --ext ts && tsc --noEmit"
},
"devDependencies": {

2
src/cli.ts Normal file → Executable file
View File

@@ -1,7 +1,7 @@
#!/usr/bin/env node
import { cli } from 'cleye';
import packageJSON from '../package.json' assert { type: 'json' };
import packageJSON from '../package.json';
import { configCommand } from './commands/config';
import { hookCommand, isHookCalled } from './commands/githook.js';

View File

@@ -20,7 +20,7 @@ export const hookCommand = command(
parameters: ['<set/unset>']
},
async (argv) => {
const HOOK_PATH = fileURLToPath(new URL('./out/cli.cjs', import.meta.url));
const HOOK_URL = __filename;
try {
await assertGitRepo();
@@ -29,10 +29,17 @@ export const hookCommand = command(
if (mode === 'set') {
intro(`setting opencommit as '${HOOK_NAME}' hook`);
if (isHookExists) {
const realPath = await fs.realpath(SYMLINK_URL);
if (realPath === HOOK_PATH)
if (isHookExists) {
let realPath;
try {
realPath = await fs.realpath(SYMLINK_URL);
} catch (error) {
outro(error as string);
realPath = null;
}
if (realPath === HOOK_URL)
return outro(`opencommit is already set as '${HOOK_NAME}'`);
throw new Error(
@@ -41,7 +48,7 @@ export const hookCommand = command(
}
await fs.mkdir(path.dirname(SYMLINK_URL), { recursive: true });
await fs.symlink(HOOK_PATH, SYMLINK_URL, 'file');
await fs.symlink(HOOK_URL, SYMLINK_URL, 'file');
await fs.chmod(SYMLINK_URL, 0o755);
return outro(`${chalk.green('✔')} Hook set`);
@@ -49,6 +56,7 @@ export const hookCommand = command(
if (mode === 'unset') {
intro(`unsetting opencommit as '${HOOK_NAME}' hook`);
if (!isHookExists) {
return outro(
`opencommit wasn't previously set as '${HOOK_NAME}' hook, nothing to remove`
@@ -56,7 +64,7 @@ export const hookCommand = command(
}
const realpath = await fs.realpath(SYMLINK_URL);
if (realpath !== HOOK_PATH) {
if (realpath !== HOOK_URL) {
return outro(
`opencommit wasn't previously set as '${HOOK_NAME}' hook, but different hook was, if you want to remove it — do it manually`
);

View File

@@ -1,36 +1,25 @@
{
"compilerOptions": {
/* Projects */
"target": "ES2020",
"lib": ["ES5"],
/* Language and Environment */
"target": "ESNext" /* Set the JavaScript language version for emitted JavaScript and include compatible library declarations. */,
"lib": [
"ES6"
] /* Specify a set of bundled library declaration files that describe the target runtime environment. */,
"module": "CommonJS",
// "rootDir": "./src",
"moduleResolution": "node",
"resolveJsonModule": true,
/* Modules */
"module": "ESNext" /* Specify what module code is generated. */,
// "rootDir": "./src" /* Specify the root folder within your source files. */,
"moduleResolution": "node" /* Specify how TypeScript looks up a file from a given module specifier. */,
"resolveJsonModule": true /* Enable importing .json files. */,
"allowJs": true,
/* JavaScript Support */
"allowJs": true /* Allow JavaScript files to be a part of your program. Use the 'checkJS' option to get errors from these files. */,
"outDir": "./out",
/* Emit */
"outDir": "./out" /* Specify an output folder for all emitted files. */,
"esModuleInterop": true,
"forceConsistentCasingInFileNames": true,
/* Interop Constraints */
"esModuleInterop": true /* Emit additional JavaScript to ease support for importing CommonJS modules. This enables 'allowSyntheticDefaultImports' for type compatibility. */,
"forceConsistentCasingInFileNames": true /* Ensure that casing is correct in imports. */,
"strict": true,
"noUnusedLocals": true,
"noUnusedParameters": true,
/* Type Checking */
"strict": true /* Enable all strict type-checking options. */,
"noUnusedLocals": true /* Enable error reporting when local variables aren't read. */,
"noUnusedParameters": true /* Raise an error when a function parameter isn't read. */,
/* Completeness */
"skipLibCheck": true /* Skip type checking all .d.ts files. */
"skipLibCheck": true
},
"exclude": ["node_modules"],
"ts-node": {