@carthooks/arcubase-cli 0.1.3 → 0.1.4

package/src/tests/execute_validation.test.ts

@@ -429,6 +429,189 @@ test('selection action rejects selectAll-only condition for archive action', asy
   })
 })
 
+test('insert entity rejects local file path for file field and points to arcubase upload', async () => {
+  const bodyFile = tempJSONFile({
+    fields: {
+      1001: 'Lead A',
+      1007: './contract.pdf',
+    },
+  })
+  const calls: Array<{ url: string; method?: string }> = []
+
+  await assert.rejects(async () => {
+    await executeCLI(
+      'user',
+      ['workflow', 'insert-entity', '--app-id', 'app_1', '--entity-id', 'entity_1', '--body-file', bodyFile],
+      env as any,
+      async (url, init) => {
+        calls.push({ url: String(url), method: init?.method })
+        if (String(url) === 'https://arcubase.example.com/entity/app_1/entity_1' && init?.method === 'GET') {
+          return new Response(JSON.stringify({
+            data: {
+              id: 1,
+              app_id: 1,
+              fields: [
+                { id: 1001, type: 'text', label: 'Lead Name' },
+                { id: 1007, type: 'file', label: 'Contract File' },
+              ],
+            },
+          }), { status: 200 })
+        }
+        return new Response('{}', { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'BODY_VALIDATION_FAILED')
+    assert.ok((error.details?.issues ?? []).some((item) => item.path === 'body.fields.1007'))
+    assert.match(error.message, /arcubase upload \.\/contract\.pdf/)
+    assert.ok((error.details?.docHints ?? []).some((item) => item.file.endsWith('/docs/runtime-reference/uploads.md')))
+    return true
+  })
+
+  assert.deepEqual(calls, [
+    { url: 'https://arcubase.example.com/entity/app_1/entity_1', method: 'GET' },
+  ])
+})
+
+test('update row rejects object value for image field and requires upload array', async () => {
+  const bodyFile = tempJSONFile({
+    changed_fields: [1008],
+    data: {
+      1008: {
+        upload_id: 123456,
+        file: 'lead-photo.jpg',
+        file_name: 'lead-photo.jpg',
+        meta: {},
+      },
+    },
+  })
+
+  await assert.rejects(async () => {
+    await executeCLI(
+      'user',
+      ['workflow', 'update-entity-row', '--app-id', 'app_1', '--entity-id', 'entity_1', '--row-id', 'row_1', '--body-file', bodyFile],
+      env as any,
+      async (url, init) => {
+        if (String(url) === 'https://arcubase.example.com/entity/app_1/entity_1' && init?.method === 'GET') {
+          return new Response(JSON.stringify({
+            data: {
+              id: 1,
+              app_id: 1,
+              fields: [
+                { id: 1008, type: 'image', label: 'Lead Photo' },
+              ],
+            },
+          }), { status: 200 })
+        }
+        return new Response('{}', { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'BODY_VALIDATION_FAILED')
+    assert.ok((error.details?.issues ?? []).some((item) => item.path === 'body.data.1008'))
+    assert.match(error.message, /must be an array/)
+    assert.match(error.message, /arcubase upload <local-file>/)
+    return true
+  })
+})
+
+test('insert entity accepts upload array for file field after schema preflight', async () => {
+  const bodyFile = tempJSONFile({
+    fields: {
+      1001: 'Lead A',
+      1007: [
+        {
+          upload_id: 123456,
+          file: 'contract.pdf',
+          file_name: 'contract.pdf',
+          meta: {},
+        },
+      ],
+    },
+  })
+  const calls: Array<{ url: string; method?: string }> = []
+
+  const out = await executeCLI(
+    'user',
+    ['workflow', 'insert-entity', '--app-id', 'app_1', '--entity-id', 'entity_1', '--body-file', bodyFile],
+    env as any,
+    async (url, init) => {
+      calls.push({ url: String(url), method: init?.method })
+      if (String(url) === 'https://arcubase.example.com/entity/app_1/entity_1' && init?.method === 'GET') {
+        return new Response(JSON.stringify({
+          data: {
+            id: 1,
+            app_id: 1,
+            fields: [
+              { id: 1001, type: 'text', label: 'Lead Name' },
+              { id: 1007, type: 'file', label: 'Contract File' },
+            ],
+          },
+        }), { status: 200 })
+      }
+      return new Response(String(init?.body ?? '{}'), { status: 200 })
+    },
+  )
+
+  assert.equal(out.kind, 'result')
+  assert.deepEqual(calls, [
+    { url: 'https://arcubase.example.com/entity/app_1/entity_1', method: 'GET' },
+    { url: 'https://arcubase.example.com/entity/app_1/entity_1/insert', method: 'POST' },
+  ])
+  assert.equal(out.data.fields['1007'][0].upload_id, 123456)
+})
+
+test('insert entity decorates upload bind failures with actionable guidance', async () => {
+  const bodyFile = tempJSONFile({
+    fields: {
+      1001: 'Lead A',
+      1007: [
+        {
+          upload_id: 123456,
+          file: 'contract.pdf',
+          file_name: 'contract.pdf',
+          meta: {},
+        },
+      ],
+    },
+  })
+
+  await assert.rejects(async () => {
+    await executeCLI(
+      'user',
+      ['workflow', 'insert-entity', '--app-id', 'app_1', '--entity-id', 'entity_1', '--body-file', bodyFile],
+      env as any,
+      async (url, init) => {
+        if (String(url) === 'https://arcubase.example.com/entity/app_1/entity_1' && init?.method === 'GET') {
+          return new Response(JSON.stringify({
+            data: {
+              fields: [{ id: 1007, type: 'file', label: 'Contract File' }],
+            },
+          }), { status: 200 })
+        }
+        return new Response(JSON.stringify({
+          error: {
+            message: 'failed to bind upload to assets: Head \"http://test-bucket.test.aliyuncs.com/upload/123456/contract.pdf\": dial tcp: lookup test-bucket.test.aliyuncs.com: no such host',
+            type: 'system',
+            key: 'error.system',
+          },
+          trace_id: 'req_upload_bind',
+        }), { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'UPSTREAM_BODY_ERROR')
+    assert.equal(error.details?.traceId, 'req_upload_bind')
+    assert.equal(error.details?.reason, 'the row payload used a file/image upload entry, but Arcubase could not bind that upload into an asset record')
+    assert.match(error.details?.hint ?? '', /rerun arcubase upload/)
+    assert.ok((error.details?.docHints ?? []).some((item) => item.file.endsWith('/docs/runtime-reference/uploads.md')))
+    return true
+  })
+})
+
 test('selection action rejects condition selection without selectAll', async () => {
   const bodyFile = tempJSONFile({
     selection: {

package/src/tests/help.test.ts

@@ -54,7 +54,7 @@ test('command help prints runtime doc hints for request body commands', async ()
   assert.equal(out.kind, 'help')
   assert.match(out.text, /docs:/)
   assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/row-crud\.md/)
-  assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/examples\/oms-01\/README\.md/)
+  assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/examples\/README\.md/)
 })
 
 test('command help prints query-file support', async () => {
@@ -67,5 +67,26 @@ test('command help without request body can still point to lifecycle and example
   const out = await executeCLI('admin', ['entity', 'admin-get-entity-info', '--help'], undefined, async () => new Response('{}'))
   assert.equal(out.kind, 'help')
   assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/table-lifecycle\.md/)
-  assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/examples\/oms-01\/README\.md/)
+  assert.match(out.text, /\/opt\/arcubase-sdk\/docs\/runtime-reference\/examples\/README\.md/)
+})
+
+test('admin-save-entity help prints row-operation next step', async () => {
+  const out = await executeCLI('admin', ['entity', 'admin-save-entity', '--help'], undefined, async () => new Response('{}'))
+  assert.equal(out.kind, 'help')
+  assert.match(out.text, /switch to arcubase for row operations/)
+  assert.match(out.text, /arcubase workflow insert-entity/)
+  assert.match(out.text, /arcubase workflow query-entity/)
+})
+
+test('wrong admin workflow command points to arcubase row command', async () => {
+  await assert.rejects(
+    () => executeCLI('admin', ['workflow', 'insert-entity'], env as any, async () => new Response('{}')),
+    (error: any) => {
+      assert.equal(error.code, 'UNKNOWN_COMMAND')
+      assert.equal(error.details?.reason, 'workflow insert-entity is a row operation, not an arcubase-admin command')
+      assert.equal(error.details?.hint, 'use: arcubase workflow insert-entity')
+      assert.ok((error.details?.suggestions ?? []).includes('arcubase workflow insert-entity'))
+      return true
+    }
+  )
 })

package/src/tests/upload.test.ts

@@ -0,0 +1,133 @@
+import test from 'node:test'
+import assert from 'node:assert/strict'
+import fs from 'fs'
+import os from 'os'
+import path from 'path'
+import { executeCLI } from '../runtime/execute.js'
+
+const env = {
+  ARCUBASE_BASE_URL: 'https://arcubase.example.com',
+  ARCUBASE_TENANT_ID: 'tenant_1',
+  ARCUBASE_ACCESS_TOKEN: 'tok',
+  ARCUBASE_TRACE_ID: '',
+  ARCUBASE_SUBJECT_TYPE: '',
+  ARCUBASE_SUBJECT_ID: '',
+  ARCUBASE_TOKEN_EXPIRES_AT: '',
+}
+
+test('upload help works without runtime env', async () => {
+  const out = await executeCLI('user', ['upload', '--help'], undefined, async () => new Response('{}'))
+  assert.equal(out.kind, 'help')
+  assert.match(out.text, /arcubase upload <local-file>/)
+  assert.match(out.text, /uploads\.md/)
+})
+
+test('upload returns file field value array for row payloads', async () => {
+  const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'arcubase-upload-'))
+  const sourceFile = path.join(tempDir, 'contract.pdf')
+  fs.writeFileSync(sourceFile, 'fake pdf bytes')
+
+  const calls: Array<{ url: string; method?: string }> = []
+
+  const result = await executeCLI(
+    'user',
+    ['upload', sourceFile],
+    env as any,
+    async (input, init) => {
+      const url = String(input)
+      calls.push({ url, method: init?.method })
+      if (url === 'https://arcubase.example.com/upload/token') {
+        return new Response(
+          JSON.stringify({
+            data: {
+              id: 123456,
+              mode: 'aliyun-oss',
+              token_data: {
+                dir: 'upload/123456/',
+                policy: 'policy',
+                accessid: 'accessid',
+                signature: 'signature',
+                host: 'https://uploads.example.com',
+              },
+            },
+          }),
+          { status: 200, headers: { 'Content-Type': 'application/json' } },
+        )
+      }
+      if (url === 'https://uploads.example.com') {
+        return new Response('', { status: 200 })
+      }
+      throw new Error(`unexpected fetch url: ${url}`)
+    },
+  )
+
+  assert.equal(result.kind, 'result')
+  assert.deepEqual(calls, [
+    { url: 'https://arcubase.example.com/upload/token', method: 'POST' },
+    { url: 'https://uploads.example.com', method: 'POST' },
+  ])
+  assert.deepEqual(result.data, [
+    {
+      upload_id: 123456,
+      file: 'contract.pdf',
+      file_name: 'contract.pdf',
+      meta: {},
+    },
+  ])
+})
+
+test('upload supports s3-post-policy tokens', async () => {
+  const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'arcubase-upload-s3-'))
+  const sourceFile = path.join(tempDir, 'contract.pdf')
+  fs.writeFileSync(sourceFile, 'fake pdf bytes')
+
+  const calls: Array<{ url: string; method?: string }> = []
+
+  const result = await executeCLI(
+    'user',
+    ['upload', sourceFile],
+    env as any,
+    async (input, init) => {
+      const url = String(input)
+      calls.push({ url, method: init?.method })
+      if (url === 'https://arcubase.example.com/upload/token') {
+        return new Response(
+          JSON.stringify({
+            data: {
+              id: 654321,
+              mode: 's3-post-policy',
+              token_data: {
+                dir: 'upload/654321/',
+                policy: 'policy',
+                host: 'https://uploads.example.com/private-bucket',
+                x_amz_algorithm: 'AWS4-HMAC-SHA256',
+                x_amz_credential: 'test/20260504/us-east-1/s3/aws4_request',
+                x_amz_date: '20260504T010203Z',
+                x_amz_signature: 'abc123',
+              },
+            },
+          }),
+          { status: 200, headers: { 'Content-Type': 'application/json' } },
+        )
+      }
+      if (url === 'https://uploads.example.com/private-bucket') {
+        return new Response('', { status: 200 })
+      }
+      throw new Error(`unexpected fetch url: ${url}`)
+    },
+  )
+
+  assert.equal(result.kind, 'result')
+  assert.deepEqual(calls, [
+    { url: 'https://arcubase.example.com/upload/token', method: 'POST' },
+    { url: 'https://uploads.example.com/private-bucket', method: 'POST' },
+  ])
+  assert.deepEqual(result.data, [
+    {
+      upload_id: 654321,
+      file: 'contract.pdf',
+      file_name: 'contract.pdf',
+      meta: {},
+    },
+  ])
+})