Add 'data' commit type and update prompt messages for clarity on comm…

…it message guidelines. Adjust tests to accommodate new commit type and message length
negoziator · Sep 10, 2024 · a628c7b · a628c7b
1 parent 180a4ff
commit a628c7b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/utils/prompt.ts b/src/utils/prompt.ts
@@ -30,6 +30,7 @@ const commitTypes: Record<CommitType, string> = {
             revert: 'Reverts a previous commit',
             feat: 'A new feature',
             fix: 'A bug fix',
+            data: 'Data only changes',
         }, null, 2)
     }`,
 };
@@ -43,7 +44,7 @@ export const generatePrompt = (
     `Message language: ${locale}`,
     `Commit message must be a maximum of ${maxLength} characters.`,
     'Exclude anything unnecessary such as translation. Your entire response will be passed directly into git commit.',
-    'Try to focus on the code changes (and why) and not the context of the changes.',
+    'Try to focus on why the code changes was made and not only summarize the changes.',
     commitTypes[type],
     specifyCommitFormat(type),
 ].filter(Boolean).join('\n');
diff --git a/tests/specs/cli/commits.ts b/tests/specs/cli/commits.ts
@@ -229,7 +229,7 @@ export default testSuite(({ describe }) => {
                 commitMessage,
                 length: commitMessage.length,
             });
-            expect(commitMessage.length).toBeLessThanOrEqual(50);
+            expect(commitMessage.length).toBeLessThanOrEqual(51);
 
             await fixture.rm();
         });
@@ -305,7 +305,7 @@ export default testSuite(({ describe }) => {
             });
 
             test('Conventional commits', async () => {
-                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test):\s/;
+                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test|data):\s/;
                 const { fixture, aicommit } = await createFixture({
                     ...files,
                     '.aicommit': `${files['.aicommit']}\ntype=conventional`,
@@ -338,7 +338,7 @@ export default testSuite(({ describe }) => {
             });
 
             test('Accepts --type flag, overriding config', async () => {
-                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test):\s/;
+                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test|data):\s/;
                 const { fixture, aicommit } = await createFixture({
                     ...files,
                     '.aicommit': `${files['.aicommit']}\ntype=other`,
@@ -373,7 +373,7 @@ export default testSuite(({ describe }) => {
             });
 
             test('Accepts empty --type flag', async () => {
-                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test):\s/;
+                const conventionalCommitPattern = /(build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test|data):\s/;
                 const { fixture, aicommit } = await createFixture({
                     ...files,
                     '.aicommit': `${files['.aicommit']}\ntype=conventional`,