@carthooks/arcubase-cli 0.1.22 → 0.1.24

package/src/tests/execute_validation.test.ts

@@ -1095,6 +1095,16 @@ test('entry fixed app tables query cannot be disabled by query file', async () =
 })
 
 test('dashboard and widget commands map to approved endpoints', async () => {
+  const userAppGet = await executeCLI(
+    'user',
+    ['app', 'get', '--app-id', 'app_1'],
+    env as any,
+    async (url, init) => new Response(JSON.stringify({ url, method: init?.method }), { status: 200 }),
+  )
+  assert.equal(userAppGet.kind, 'result')
+  assert.equal(userAppGet.data.url, 'https://arcubase.example.com/api/apps/app_1')
+  assert.equal(userAppGet.data.method, 'GET')
+
   const adminWidgetList = await executeCLI(
     'admin',
     ['widget', 'list', '--app-id', 'app_1', '--type', 'dashboard', '--dashboard-id', '100'],
@@ -1131,6 +1141,61 @@ test('dashboard and widget commands map to approved endpoints', async () => {
   assert.deepEqual(userWidgetData.data.body, { dashboard_id: 100, type: 1 })
 })
 
+test('widget create ingress mistakes suggest ingress widget shape before request', async () => {
+  let called = false
+  await assert.rejects(async () => {
+    await executeCLI(
+      'admin',
+      ['widget', 'create', '--app-id', 'app_1', '--body-json', '{"type":"list","options":{"entity_id":2188891001}}'],
+      env as any,
+      async () => {
+        called = true
+        return new Response('{}', { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'BODY_VALIDATION_FAILED')
+    assert.equal(error.details?.requestType, 'AppNewWidgetsReqVO')
+    assert.ok((error.details?.commonMistakes ?? []).includes('ingress widgets use options.ingress_id'))
+    assert.ok((error.details?.commonMistakes ?? []).includes('after creating an ingress widget, use widget update-ingress-options to adjust display configuration'))
+    return true
+  })
+  assert.equal(called, false)
+})
+
+test('widget update ingress list options points to update-ingress-options before request', async () => {
+  let called = false
+  await assert.rejects(async () => {
+    await executeCLI(
+      'admin',
+      [
+        'widget',
+        'update',
+        '--app-id',
+        'app_1',
+        '--widget-id',
+        'widget_1',
+        '--body-json',
+        '{"ingress_id":2188893001,"options":{"list":{"default_cols":[1001,1002],"page_size":10}}}',
+      ],
+      env as any,
+      async () => {
+        called = true
+        return new Response('{}', { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'BODY_VALIDATION_FAILED')
+    assert.equal(error.details?.requestType, 'AppUpdateWidgetsReqVO')
+    assert.ok((error.details?.commonMistakes ?? []).includes('ingress/list display settings use widget update-ingress-options, not widget update'))
+    assert.ok((error.details?.suggestions ?? []).some((item) => item.includes('widget update-ingress-options')))
+    return true
+  })
+  assert.equal(called, false)
+})
+
 test('widget preview-data rejects unsupported aggregate before request', async () => {
   let called = false
   await assert.rejects(async () => {
@@ -1191,6 +1256,38 @@ test('widget preview-data rejects field dimensions without IsField true before r
   assert.equal(called, false)
 })
 
+test('widget preview-data filter mistakes suggest chart conditions before request', async () => {
+  let called = false
+  await assert.rejects(async () => {
+    await executeCLI(
+      'admin',
+      [
+        'widget',
+        'preview-data',
+        '--app-id',
+        'app_1',
+        '--body-json',
+        '{"dataSource":2188891001,"dataSourceType":"entity","dimensions":[{"IsField":true,"FieldID":1004,"Name":"Source"}],"filters":[{"FieldID":1006,"Operator":"$eq","Value":"yes"}],"measures":[{"Name":"Distinct Customers","FieldID":1002,"Aggregate":"distinct_count"}]}',
+      ],
+      env as any,
+      async () => {
+        called = true
+        return new Response('{}', { status: 200 })
+      },
+    )
+  }, (error: unknown) => {
+    assert.ok(error instanceof CLIError)
+    assert.equal(error.code, 'BODY_VALIDATION_FAILED')
+    assert.equal(error.details?.requestType, 'AppPreviewWidgetsDataReqVO')
+    assert.ok((error.details?.issues ?? []).some((item) => item.path === 'body'))
+    assert.ok((error.details?.commonMistakes ?? []).includes('use conditions, not filters, for chart data filtering'))
+    assert.ok((error.details?.commonMistakes ?? []).includes('do not retry without conditions and then hand-calculate filtered statistics'))
+    assert.ok((error.details?.suggestions ?? []).some((item) => item.includes('"conditions":{"mode":"and","conditions":[{"column":":1006","operator":"$eq","value":"yes"}]}')))
+    return true
+  })
+  assert.equal(called, false)
+})
+
 test('widget update rejects chart field dimensions without IsField true before request', async () => {
   let called = false
   await assert.rejects(async () => {

package/src/tests/help.test.ts

@@ -136,6 +136,12 @@ test('dashboard and widget help exposes agent-readable body docs and flags', asy
   assert.match(userWidgetData.text, /body: FetchWidgetsDataReqVO via --body-json/)
   assert.match(userWidgetData.text, /^types:$/m)
   assert.match(userWidgetData.text, /^docs:$/m)
+
+  const userAppGet = await executeCLI('user', ['app', 'get', '--help'], env as any, async () => new Response('{}'))
+  assert.equal(userAppGet.kind, 'help')
+  assert.match(userAppGet.text, /arcubase app get/)
+  assert.match(userAppGet.text, /endpoint: \/apps\/:id/)
+  assert.match(userAppGet.text, /path flags: --app-id/)
 })
 
 test('admin access-rule help gives update whitelist and assign-users body-json examples', async () => {
@@ -214,6 +220,17 @@ test('admin dashboard update-layout help-examples show top-level layout array',
   assert.match(out.text, /do not wrap the array/)
 })
 
+test('admin dashboard update-options help-examples show user visibility scope', async () => {
+  const out = await executeCLI('admin', ['dashboard', 'update-options', '--help-examples'], env as any, async () => new Response('{}'))
+  assert.equal(out.kind, 'help')
+  assert.match(out.text, /arcubase-admin dashboard update-options --help-examples/)
+  assert.match(out.text, /"changed":\["users"\]/)
+  assert.match(out.text, /"users":\[\{"type":"user","id":2188889901\}\]/)
+  assert.match(out.text, /currentSpeaker\.arcubaseUserId/)
+  assert.match(out.text, /verify from the user side with arcubase app get/)
+  assert.doesNotMatch(out.text, /"type":"member"/)
+})
+
 test('admin widget preview-data help-examples show chart query body without widget id', async () => {
   const out = await executeCLI('admin', ['widget', 'preview-data', '--help-examples'], env as any, async () => new Response('{}'))
   assert.equal(out.kind, 'help')
@@ -221,6 +238,8 @@ test('admin widget preview-data help-examples show chart query body without widg
   assert.match(out.text, /"dataSource":2188891001/)
   assert.match(out.text, /"dataSourceType":"entity"/)
   assert.match(out.text, /"Aggregate":"sum"/)
+  assert.match(out.text, /"conditions":\{"mode":"and","conditions":\[\{"column":":1006","operator":"\$eq","value":"yes"\}\]\}/)
+  assert.match(out.text, /use conditions, not filters/)
   assert.match(out.text, /preview-data does not accept --widget-id/)
   assert.match(out.text, /do not send widget_id or id/)
   assert.doesNotMatch(out.text, /--widget-id <widget_id>/)
@@ -236,6 +255,14 @@ test('admin widget create help-examples avoid manual options type', async () =>
   assert.doesNotMatch(out.text, /"options":\{"dashboard_id":2188892001,"type":/)
 })
 
+test('admin widget update-ingress-options help-examples show ingress display body', async () => {
+  const out = await executeCLI('admin', ['widget', 'update-ingress-options', '--help-examples'], env as any, async () => new Response('{}'))
+  assert.equal(out.kind, 'help')
+  assert.match(out.text, /arcubase-admin widget update-ingress-options --help-examples/)
+  assert.match(out.text, /"cond_follow":true/)
+  assert.match(out.text, /verify with widget get/)
+})
+
 test('user row bulk-update help-examples show ids and condition selection bodies', async () => {
   const out = await executeCLI('user', ['row', 'bulk-update', '--help-examples'], env as any, async () => new Response('{}'))
   assert.equal(out.kind, 'help')