chore(api): bump maximum memory (#451)

* chore(api): bump maximum memory

* chore(build): commit build artefacts

.github/workflows/main.yml

@@ -14,7 +14,7 @@ jobs:
       - name: Get yarn cache directory path
         id: yarn-cache-dir-path
         run: echo "dir=$(yarn config get cacheFolder)" >> $GITHUB_OUTPUT
-      - uses: actions/cache/restore@v3
+      - uses: actions/cache/restore@v4
         id: cache-restore
         with:
           path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
@@ -25,12 +25,12 @@ jobs:
           cache: 'yarn'
       - name: Yarn Install
         run: yarn install
-      - uses: actions/cache/save@v3
+      - uses: actions/cache/save@v4
         id: cache-save
         with:
           path: ${{ steps.yarn-cache-dir-path.outputs.dir }}
           key: ${{ runner.os }}-yarn-${{ hashFiles('**/yarn.lock') }}
-      - uses: pre-commit/action@v3.0.0
+      - uses: pre-commit/action@v3.0.1
       - name: Tests
         run: yarn test
         continue-on-error: true # ignore test failures for now

README.md

@@ -37,9 +37,18 @@ Associates this new version with the given ID of a Speckle Function.
 
 Your Speckle Token must have write permissions for the Speckle Function with this ID, otherwise the publish will fail.
 
+#### `speckle_function_input_schema_file_path`
+
+*Optional.* The path to the JSON Schema file that describes the input schema for this version of the Speckle Function. This file is used to define the input form that will be presented to users when they compose an Automation based on this Function. If not provided, no input form will be presented to users.
+This can be an absolute path, or a path relative to the Home directory of the GitHub Action runner.
+
+#### `speckle_function_command`
+
+The command to run when this version of the Speckle Function is invoked. This command must be a valid command for the Docker image that contains the Speckle Function. This command must be a single string.
+
 #### `speckle_function_release_tag`
 
-The release tag for this version of the Speckle Function. This is intended to provide a more human understandable name for this version, and we recommend using the [Semver](https://semver.org) naming conventions. The name must conform to the following:
+The release tag for this version of the Speckle Function. This is intended to provide a more human understandable name for this version, and we recommend using the Git SHA of the commit used to generate this function version. The name must conform to the following:
 
 - A minimum of 1 character is required.
 - A maximum of 128 characters are permitted.

package.json

@@ -29,26 +29,26 @@
     "zod": "^3.22.3"
   },
   "devDependencies": {
-    "@types/js-yaml": "^4.0.5",
-    "@types/node": "^18.18.8",
-    "@typescript-eslint/eslint-plugin": "^6.10.0",
-    "@typescript-eslint/parser": "^6.10.0",
-    "@vercel/ncc": "^0.36.1",
-    "@vitest/coverage-istanbul": "^0.34.6",
-    "eslint": "^8.53.0",
+    "@types/js-yaml": "^4.0.9",
+    "@types/node": "^18.19.17",
+    "@typescript-eslint/eslint-plugin": "^7.0.1",
+    "@typescript-eslint/parser": "^7.9.0",
+    "@vercel/ncc": "^0.38.1",
+    "@vitest/coverage-istanbul": "^1.6.0",
+    "eslint": "^8.56.0",
     "eslint-import-resolver-typescript": "^3.6.1",
     "eslint-plugin-eslint-comments": "^3.2.0",
     "eslint-plugin-filenames": "latest",
     "eslint-plugin-github": "^4.10.1",
     "eslint-plugin-i18n-text": "^1.0.1",
-    "eslint-plugin-import": "^2.29.0",
+    "eslint-plugin-import": "^2.29.1",
     "eslint-plugin-no-only-tests": "^3.1.0",
-    "eslint-plugin-prettier": "^5.0.1",
-    "eslint-plugin-vitest": "^0.3.9",
-    "msw": "^2.0.5",
-    "prettier": "^3.0.3",
+    "eslint-plugin-prettier": "^5.1.3",
+    "eslint-plugin-vitest": "^0.3.22",
+    "msw": "^2.3.0",
+    "prettier": "^3.2.5",
     "typescript": "^5.2.2",
-    "vite": "^4.5.0",
-    "vitest": "^0.34.6"
+    "vite": "^5.1.8",
+    "vitest": "^1.6.0"
   }
 }

src/main.ts

@@ -3,7 +3,7 @@ import { ZodError, z } from 'zod'
 import fetch from 'node-fetch'
 import { retry } from '@lifeomic/attempt'
 import { readFileSync } from 'node:fs'
-import { join } from 'node:path'
+import { isAbsolute, join } from 'node:path'
 
 const InputVariablesSchema = z.object({
   speckleAutomateUrl: z.string().url().min(1),
@@ -29,7 +29,7 @@ const InputVariablesSchema = z.object({
     .int()
     .finite()
     .gte(1)
-    .lte(8000)
+    .lte(60000)
     .optional()
 })
 
@@ -40,12 +40,18 @@ const parseInputs = (): InputVariables => {
   core.setSecret(speckleTokenRaw)
 
   const rawInputSchemaPath = core.getInput('speckle_function_input_schema_file_path')
-  const homeDir = process.env['HOME']
-  if (!homeDir)
-    throw new Error('The home directory is not defined, cannot load inputSchema')
+
+  let rawInputSchemaPathAbsolute = rawInputSchemaPath
+  if (!isAbsolute(rawInputSchemaPath)) {
+    const homeDir = process.env['HOME']
+    if (!homeDir)
+      throw new Error('The home directory is not defined, cannot load inputSchema')
+    rawInputSchemaPathAbsolute = join(homeDir, rawInputSchemaPath)
+  }
+
   let speckleFunctionInputSchema: Record<string, unknown> | null = null
-  if (rawInputSchemaPath) {
-    const rawInputSchema = readFileSync(join(homeDir, rawInputSchemaPath), 'utf-8')
+  if (rawInputSchemaPathAbsolute) {
+    const rawInputSchema = readFileSync(rawInputSchemaPathAbsolute, 'utf-8')
     speckleFunctionInputSchema = JSON.parse(rawInputSchema)
   }
 
@@ -173,9 +179,20 @@ const registerNewVersionForTheSpeckleAutomateFunction = async (
     if (parsedResult.success) return parsedResult.data
     throw parsedResult.error
   } catch (err) {
-    throw Error('Failed to register new function version to the automate server', {
-      cause: err
-    })
+    if (err instanceof Error) {
+      throw Error(
+        `Failed to register new function version to the automate server. ${err.message}`,
+        {
+          cause: err
+        }
+      )
+    }
+    throw Error(
+      `Failed to register new function version to the automate server. ${err}`,
+      {
+        cause: err
+      }
+    )
   }
 }
 

tests/main.spec.ts

@@ -20,6 +20,49 @@ describe('Register new version', () => {
   let tmpDir: string
   let countHappyPath = 0
   let count500Errors = 0
+  let count422Errors = 0
+
+  const error422 = {
+    type: 'H3Error',
+    message: 'Body parsing failed',
+    stack: `Error: Body parsing failed
+            at createError...`,
+    statusCode: 422,
+    fatal: false,
+    unhandled: false,
+    statusMessage: 'Body parsing failed',
+    data: {
+      type: 'ZodError',
+      message:
+        '[\n  {\n    "code": "custom",\n    "message": "Invalid JSON schema: strict mode: unknown keyword: \\"IAmInvalid\\"",\n    "path": [\n      "inputSchema"\n    ]\n  }\n]',
+      stack: {
+        ZodError: [
+          {
+            code: 'custom',
+            message: 'Invalid JSON schema: strict mode: unknown keyword: "IAmInvalid"',
+            path: ['inputSchema']
+          }
+        ]
+      },
+      aggregateErrors: [
+        {
+          type: 'Object',
+          message: 'Invalid JSON schema: strict mode: unknown keyword: "IAmInvalid"',
+          stack: {},
+          code: 'custom',
+          path: ['inputSchema']
+        }
+      ],
+      issues: [
+        {
+          code: 'custom',
+          message: 'Invalid JSON schema: strict mode: unknown keyword: "IAmInvalid"',
+          path: ['inputSchema']
+        }
+      ],
+      name: 'ZodError'
+    }
+  }
 
   const server = setupServer(
     http.post(
@@ -44,15 +87,29 @@ describe('Register new version', () => {
         return HttpResponse.error() // simulates a network error
       }
     ),
+    http.post(
+      'http://myfakeautomate.speckle.internal/api/v1/functions/422_response/versions',
+      async ({ request }) => {
+        const parseResult = FunctionVersionRequestSchema.safeParse(await request.json())
+        expect(parseResult.success).to.be.true
+        count422Errors++
+        return HttpResponse.json(error422, {
+          status: 422
+        })
+      }
+    ),
     http.post(
       'http://myfakeautomate.speckle.internal/api/v1/functions/500_response/versions',
       async ({ request }) => {
         const parseResult = FunctionVersionRequestSchema.safeParse(await request.json())
         expect(parseResult.success).to.be.true
         count500Errors++
-        return new HttpResponse(null, {
-          status: 500
-        })
+        return HttpResponse.json(
+          {},
+          {
+            status: 500
+          }
+        )
       }
     )
   )
@@ -123,6 +180,24 @@ describe('Register new version', () => {
     expect(count500Errors).to.toBeGreaterThan(1) // we expect the action to retry the request
     count500Errors = 0
   })
+  it('handles 422 responses', async () => {
+    writeFileSync(join(tmpDir, 'schema.json'), '{}')
+    vi.stubEnv('INPUT_SPECKLE_FUNCTION_ID', '422_response')
+    vi.stubEnv('INPUT_SPECKLE_TOKEN', '{token}')
+    vi.stubEnv('INPUT_SPECKLE_FUNCTION_COMMAND', 'echo "hello automate"')
+    vi.stubEnv('HOME', tmpDir) // the input schema file path is assumed to be relative to the home directory
+    vi.stubEnv('INPUT_SPECKLE_FUNCTION_INPUT_SCHEMA_FILE_PATH', './schema.json')
+    vi.stubEnv('INPUT_SPECKLE_FUNCTION_RELEASE_TAG', 'v1.0.0')
+    vi.stubEnv('INPUT_SPECKLE_AUTOMATE_URL', 'http://myfakeautomate.speckle.internal')
+    vi.stubEnv('GITHUB_SHA', 'commitSha')
+    vi.stubEnv('GITHUB_REF_TYPE', 'commit')
+    vi.stubEnv('GITHUB_REF_NAME', 'version')
+    await expect(run()).rejects.toThrow(
+      'Failed to register new function version to the automate server'
+    )
+    expect(count422Errors).to.eq(1) // we expect the action not to retry the request
+    count422Errors = 0 // reset the count after the test
+  })
   it('errors if the token is empty', async () => {
     writeFileSync(join(tmpDir, 'schema.json'), '{}')
     vi.stubEnv('INPUT_SPECKLE_FUNCTION_ID', 'fake_function_id')