@carthooks/arcubase-cli 0.1.7 → 0.1.9

package/src/runtime/zod_registry.ts

@@ -67,19 +67,12 @@ const examplesIndexDocHint: RuntimeDocHint = {
   file: '/opt/arcubase-sdk/docs/runtime-reference/examples/README.md',
 }
 
-const appDiscoveryDocHint: RuntimeDocHint = {
-  title: 'App discovery',
-  file: '/opt/arcubase-sdk/docs/runtime-reference/app-discovery.md',
-}
-
 const docHintIndex: Record<string, RuntimeDocHint[]> = {
   AppCreateByTenantsReqVO: [
-    appDiscoveryDocHint,
     { title: 'Table lifecycle', file: '/opt/arcubase-sdk/docs/runtime-reference/table-lifecycle.md' },
     examplesIndexDocHint,
   ],
   AppCreateEntityReqVO: [
-    appDiscoveryDocHint,
     { title: 'Table lifecycle', file: '/opt/arcubase-sdk/docs/runtime-reference/table-lifecycle.md' },
     examplesIndexDocHint,
   ],
@@ -155,25 +148,14 @@ export function getDocHints(typeName: string): RuntimeDocHint[] {
 
 const commandDocHintIndex: Record<string, RuntimeDocHint[]> = {
   'admin.app.createAppByTenants': [
-    appDiscoveryDocHint,
     { title: 'Table lifecycle', file: '/opt/arcubase-sdk/docs/runtime-reference/table-lifecycle.md' },
     examplesIndexDocHint,
   ],
-  'admin.app.listApps': [
-    appDiscoveryDocHint,
-    examplesIndexDocHint,
-  ],
-  'admin.app.inventory': [
-    appDiscoveryDocHint,
-    examplesIndexDocHint,
-  ],
   'admin.app.createEntity': [
-    appDiscoveryDocHint,
     { title: 'Table lifecycle', file: '/opt/arcubase-sdk/docs/runtime-reference/table-lifecycle.md' },
     examplesIndexDocHint,
   ],
   'admin.entity.adminGetEntityInfo': [
-    appDiscoveryDocHint,
     { title: 'Table lifecycle', file: '/opt/arcubase-sdk/docs/runtime-reference/table-lifecycle.md' },
     examplesIndexDocHint,
   ],
@@ -183,34 +165,34 @@ const commandDocHintIndex: Record<string, RuntimeDocHint[]> = {
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.insertEntity': [
+  'user.workflow.insertEntity': [
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     { title: 'Uploads', file: '/opt/arcubase-sdk/docs/runtime-reference/uploads.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.queryEntity': [
+  'user.workflow.queryEntity': [
     { title: 'Search and bulk actions', file: '/opt/arcubase-sdk/docs/runtime-reference/search-and-bulk-actions.md' },
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.getEntityRow': [
+  'user.workflow.getEntityRow': [
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.updateEntityRow': [
+  'user.workflow.updateEntityRow': [
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     { title: 'Uploads', file: '/opt/arcubase-sdk/docs/runtime-reference/uploads.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.queryEntitySelection': [
+  'user.workflow.queryEntitySelection': [
     { title: 'Search and bulk actions', file: '/opt/arcubase-sdk/docs/runtime-reference/search-and-bulk-actions.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.deleteEntityRow': [
+  'user.workflow.deleteEntityRow': [
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     examplesIndexDocHint,
   ],
-  'user.rows.restoreEntityRow': [
+  'user.workflow.restoreEntityRow': [
     { title: 'Row CRUD', file: '/opt/arcubase-sdk/docs/runtime-reference/row-crud.md' },
     examplesIndexDocHint,
   ],

package/src/tests/command_registry.test.ts

@@ -2,41 +2,91 @@ import test from 'node:test'
 import assert from 'node:assert/strict'
 
 import { adminCommands, userCommands } from '../generated/command_registry.generated.js'
-import { findCommandSuggestions } from '../runtime/command_registry.js'
 
-test('admin app create-entity command exists', () => {
-  assert.ok(adminCommands.some((c) => c.commandPath.join(' ') === 'app create-entity'))
-})
-
-test('admin app list-apps command exists', () => {
-  assert.ok(adminCommands.some((c) => c.commandPath.join(' ') === 'app list-apps'))
-})
-
-test('admin app inventory command exists', () => {
-  const suggestions = findCommandSuggestions('app', 'inventory', 20)
-  assert.ok(suggestions.some((item) => item.binary === 'arcubase-admin' && item.moduleName === 'app' && item.commandName === 'inventory'))
-})
-
-test('user rows query-entity command exists', () => {
-  assert.ok(userCommands.some((c) => c.commandPath.join(' ') === 'rows query-entity'))
-})
+function commandPaths(commands: ReadonlyArray<{ commandPath: readonly string[] }>): string[] {
+  return commands.map((item) => item.commandPath.join(' ')).sort()
+}
 
-test('inline request type keeps full object text instead of truncated brace', () => {
-  const command = adminCommands.find((c) => c.commandPath.join(' ') === 'dashboard clone-dashboard')
-  assert.ok(command)
-  const typed = command!
-  assert.match(String(typed.requestType), /^\{\s*name:\s*string\s*\}$/)
+test('admin surface is restricted to reset white list', () => {
+  const actual = commandPaths(adminCommands)
+  const expected = [
+    'app create',
+    'app delete',
+    'app get',
+    'app list',
+    'app update',
+    'table create',
+    'table delete',
+    'table get',
+    'table list',
+    'table rename',
+    'table update-schema',
+    'access-rule assign-users',
+    'access-rule create',
+    'access-rule delete',
+    'access-rule get',
+    'access-rule list',
+    'access-rule update',
+    'workflow create-version',
+    'workflow deploy',
+    'workflow get',
+    'workflow update',
+  ].sort()
+  assert.deepEqual(actual, expected)
 })
 
-test('tag commands no longer emit truncated inline request type', () => {
-  const command = adminCommands.find((c) => c.commandPath.join(' ') === 'entity-tags batch-add-entity-tags')
-  assert.ok(command)
-  const typed = command!
-  assert.match(String(typed.requestType), /^\{\s*policy_id:\s*string.*selection:\s*any.*tag_names:\s*string\[\]\s*\}$/)
-  assert.equal(typed.scalarParams.length, 0)
+test('user surface is restricted to reset white list', () => {
+  const actual = commandPaths(userCommands)
+  const expected = [
+    'entry',
+    'profile get',
+    'profile update',
+    'row bulk-update',
+    'row create',
+    'row delete',
+    'row get',
+    'row query',
+    'row selection-action',
+    'row update',
+    'table dataset',
+    'table get',
+    'table relation-options',
+    'upload check-mobile',
+    'upload mobile',
+    'upload preview',
+    'upload token',
+    'workflow approve',
+    'workflow countersign',
+    'workflow forward',
+    'workflow process-map',
+    'workflow query-approve-users',
+    'workflow query-nodes',
+    'workflow resolve',
+    'workflow retract',
+    'workflow rollback',
+  ].sort()
+  assert.deepEqual(actual, expected)
 })
 
-test('suggestions can point wrong admin delete guess to user rows delete command', () => {
-  const suggestions = findCommandSuggestions('entity', 'admin-delete-entity-row')
-  assert.ok(suggestions.some((item) => item.binary === 'arcubase' && item.moduleName === 'rows' && item.commandName === 'delete-entity-row'))
+test('black list commands are removed from registry', () => {
+  const all = new Set([...commandPaths(adminCommands), ...commandPaths(userCommands)])
+  const forbidden = [
+    'app inventory',
+    'app list-apps',
+    'app create-entity',
+    'entity admin-get-entity-info',
+    'entity admin-save-entity',
+    'ingress create-entity-ingress',
+    'workflow query-entity',
+    'workflow insert-entity',
+    'workflow update-entity-row',
+    'workflow delete-entity-row',
+    'workflow restore-entity-row',
+    'workflow query-entity-selection',
+    'profile change-password',
+    'profile auth-channels',
+  ]
+  for (const item of forbidden) {
+    assert.equal(all.has(item), false, `${item} should not exist`)
+  }
 })