Merge pull request #72 from actions/mr/linters

Author: maraisr

.markdown-lint.yml renamed to .github/linters/.markdown-lint.yml

@@ -83,8 +83,7 @@ jobs:
         run: echo "hello" > prompt.txt
 
       - name: Create System Prompt File
-        run:
-          echo "You are a helpful AI assistant for testing." > system-prompt.txt
+        run: echo "You are a helpful AI assistant for testing." > system-prompt.txt
 
       - name: Test Local Action with Prompt File
         id: test-action-prompt-file

.yaml-lint.yml renamed to .github/linters/.yaml-lint.yml

@@ -11,8 +11,7 @@ permissions:
 
 jobs:
   update_tag:
-    name:
-      Update the major tag to include the ${{ github.event.release.tag_name }}
+    name: Update the major tag to include the ${{ github.event.release.tag_name }}
       changes
     runs-on: ubuntu-latest
     steps:

.github/workflows/ci.yml

@@ -83,8 +83,7 @@ model: openai/gpt-4o
 ```yaml
 messages:
   - role: system
-    content:
-      You are a helpful assistant that describes animals using JSON format
+    content: You are a helpful assistant that describes animals using JSON format
   - role: user
     content: |-
       Describe a {{animal}}

.github/workflows/release-new-action-version.yml

@@ -1,5 +1,5 @@
 import type * as core from '@actions/core'
-import { vi } from 'vitest'
+import {vi} from 'vitest'
 
 export const debug = vi.fn<typeof core.debug>()
 export const error = vi.fn<typeof core.error>()

.prettierrc.yml

@@ -1,7 +1,6 @@
 messages:
   - role: system
-    content:
-      You are a helpful assistant that describes animals using JSON format
+    content: You are a helpful assistant that describes animals using JSON format
   - role: user
     content: |-
       Describe a {{animal}}

README.md

@@ -1,3 +1,3 @@
-import { vi } from 'vitest'
+import {vi} from 'vitest'
 
 export const wait = vi.fn<typeof import('../src/wait.js').wait>()

__fixtures__/core.ts

@@ -1,41 +1,37 @@
-import { describe, it, expect } from 'vitest'
-import {
-  buildMessages,
-  buildResponseFormat,
-  buildInferenceRequest
-} from '../src/helpers'
-import { PromptConfig } from '../src/prompt'
+import {describe, it, expect} from 'vitest'
+import {buildMessages, buildResponseFormat, buildInferenceRequest} from '../src/helpers'
+import {PromptConfig} from '../src/prompt'
 
 describe('helpers.ts - inference request building', () => {
   describe('buildMessages', () => {
     it('should build messages from prompt config', () => {
       const promptConfig: PromptConfig = {
         messages: [
-          { role: 'system', content: 'System message' },
-          { role: 'user', content: 'User message' }
-        ]
+          {role: 'system', content: 'System message'},
+          {role: 'user', content: 'User message'},
+        ],
       }
 
       const result = buildMessages(promptConfig)
       expect(result).toEqual([
-        { role: 'system', content: 'System message' },
-        { role: 'user', content: 'User message' }
+        {role: 'system', content: 'System message'},
+        {role: 'user', content: 'User message'},
       ])
     })
 
     it('should build messages from legacy format', () => {
       const result = buildMessages(undefined, 'System prompt', 'User prompt')
       expect(result).toEqual([
-        { role: 'system', content: 'System prompt' },
-        { role: 'user', content: 'User prompt' }
+        {role: 'system', content: 'System prompt'},
+        {role: 'user', content: 'User prompt'},
       ])
     })
 
     it('should use default system prompt when none provided', () => {
       const result = buildMessages(undefined, undefined, 'User prompt')
       expect(result).toEqual([
-        { role: 'system', content: 'You are a helpful assistant' },
-        { role: 'user', content: 'User prompt' }
+        {role: 'system', content: 'You are a helpful assistant'},
+        {role: 'user', content: 'User prompt'},
       ])
     })
   })
@@ -47,24 +43,24 @@ describe('helpers.ts - inference request building', () => {
         responseFormat: 'json_schema',
         jsonSchema: JSON.stringify({
           name: 'test_schema',
-          schema: { type: 'object' }
-        })
+          schema: {type: 'object'},
+        }),
       }
 
       const result = buildResponseFormat(promptConfig)
       expect(result).toEqual({
         type: 'json_schema',
         json_schema: {
           name: 'test_schema',
-          schema: { type: 'object' }
-        }
+          schema: {type: 'object'},
+        },
       })
     })
 
     it('should return undefined for text format', () => {
       const promptConfig: PromptConfig = {
         messages: [],
-        responseFormat: 'text'
+        responseFormat: 'text',
       }
 
       const result = buildResponseFormat(promptConfig)
@@ -73,7 +69,7 @@ describe('helpers.ts - inference request building', () => {
 
     it('should return undefined when no response format specified', () => {
       const promptConfig: PromptConfig = {
-        messages: []
+        messages: [],
       }
 
       const result = buildResponseFormat(promptConfig)
@@ -84,27 +80,25 @@ describe('helpers.ts - inference request building', () => {
       const promptConfig: PromptConfig = {
         messages: [],
         responseFormat: 'json_schema',
-        jsonSchema: 'invalid json'
+        jsonSchema: 'invalid json',
       }
 
-      expect(() => buildResponseFormat(promptConfig)).toThrow(
-        'Invalid JSON schema'
-      )
+      expect(() => buildResponseFormat(promptConfig)).toThrow('Invalid JSON schema')
     })
   })
 
   describe('buildInferenceRequest', () => {
     it('should build complete inference request from prompt config', () => {
       const promptConfig: PromptConfig = {
         messages: [
-          { role: 'system', content: 'System message' },
-          { role: 'user', content: 'User message' }
+          {role: 'system', content: 'System message'},
+          {role: 'user', content: 'User message'},
         ],
         responseFormat: 'json_schema',
         jsonSchema: JSON.stringify({
           name: 'test_schema',
-          schema: { type: 'object' }
-        })
+          schema: {type: 'object'},
+        }),
       }
 
       const result = buildInferenceRequest(
@@ -114,13 +108,13 @@ describe('helpers.ts - inference request building', () => {
         'gpt-4',
         100,
         'https://api.test.com',
-        'test-token'
+        'test-token',
       )
 
       expect(result).toEqual({
         messages: [
-          { role: 'system', content: 'System message' },
-          { role: 'user', content: 'User message' }
+          {role: 'system', content: 'System message'},
+          {role: 'user', content: 'User message'},
         ],
         modelName: 'gpt-4',
         maxTokens: 100,
@@ -130,9 +124,9 @@ describe('helpers.ts - inference request building', () => {
           type: 'json_schema',
           json_schema: {
             name: 'test_schema',
-            schema: { type: 'object' }
-          }
-        }
+            schema: {type: 'object'},
+          },
+        },
       })
     })
 
@@ -144,19 +138,19 @@ describe('helpers.ts - inference request building', () => {
         'gpt-4',
         100,
         'https://api.test.com',
-        'test-token'
+        'test-token',
       )
 
       expect(result).toEqual({
         messages: [
-          { role: 'system', content: 'System prompt' },
-          { role: 'user', content: 'User prompt' }
+          {role: 'system', content: 'System prompt'},
+          {role: 'user', content: 'User prompt'},
         ],
         modelName: 'gpt-4',
         maxTokens: 100,
         endpoint: 'https://api.test.com',
         token: 'test-token',
-        responseFormat: undefined
+        responseFormat: undefined,
       })
     })
   })