@sanity/assist 1.2.2 → 1.2.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/assist",
-  "version": "1.2.2",
+  "version": "1.2.4",
   "description": "",
   "keywords": [
     "sanity",

package/src/assistDocument/components/AssistDocumentForm.tsx

@@ -28,7 +28,7 @@ import {
 import {BackToInstructionListLink} from './instruction/BackToInstructionsLink'
 import {useAiPaneRouter} from '../../assistInspector/helpers'
 import {SelectedFieldContextProvider, SelectedFieldContextValue} from './SelectedFieldContext'
-import {Stack} from '@sanity/ui'
+import {Card, Stack, Text} from '@sanity/ui'
 import {documentTypeFromAiDocumentId} from '../../helpers/ids'
 
 const EMPTY_FIELDS: AssistField[] = []
@@ -36,6 +36,16 @@ const EMPTY_FIELDS: AssistField[] = []
 export const TypePathContext = createContext<string | undefined>(undefined)
 
 export function AssistDocumentForm(props: ObjectInputProps) {
+  if (props.readOnly) {
+    return (
+      <Card border tone="caution" padding={2}>
+        <Text size={1}> You do not have sufficient permissions to manage instructions.</Text>
+      </Card>
+    )
+  }
+  return <AssistDocumentFormEditable {...props} />
+}
+function AssistDocumentFormEditable(props: ObjectInputProps) {
   const {onChange} = props
   const value = props.value as AssistDocument | undefined
   const id = value?._id

package/src/assistDocument/hooks/useStudioAssistDocument.ts

@@ -26,11 +26,13 @@ import {maxHistoryVisibilityMs} from '../../constants'
 interface UseAssistDocumentProps {
   documentId: string
   schemaType: ObjectSchemaType
+  initDoc?: boolean
 }
 
 export function useStudioAssistDocument({
   documentId,
   schemaType,
+  initDoc,
 }: UseAssistDocumentProps): StudioAssistDocument | undefined {
   const documentTypeName = schemaType.name
   const currentUser = useCurrentUser()
@@ -48,13 +50,17 @@ export function useStudioAssistDocument({
   const client = useClient({apiVersion: '2023-01-01'})
 
   useEffect(() => {
-    if (!assistDocument) {
-      client.createIfNotExists({
-        _id: assistDocumentId(documentTypeName),
-        _type: assistDocumentTypeName,
-      })
+    if (!assistDocument && initDoc) {
+      client
+        .createIfNotExists({
+          _id: assistDocumentId(documentTypeName),
+          _type: assistDocumentTypeName,
+        })
+        .catch((e) => {
+          // best effort
+        })
     }
-  }, [client, assistDocument, documentTypeName])
+  }, [client, assistDocument, documentTypeName, initDoc])
 
   return useMemo(() => {
     if (!assistDocument) {

package/src/assistInspector/AssistInspector.tsx

@@ -213,7 +213,7 @@ export function AssistInspector(props: DocumentInspectorProps) {
 
   const aiDocId = assistDocumentId(documentType)
 
-  const assistDocument = useStudioAssistDocument({documentId, schemaType})
+  const assistDocument = useStudioAssistDocument({documentId, schemaType, initDoc: true})
   const assistField = assistDocument?.fields?.find((f) => f.path === typePath)
   const instruction = assistField?.instructions?.find((i) => i._key === instructionKey)
   const tasks = useMemo(

package/src/schemas/serialize/serializeSchema.test.ts

@@ -392,6 +392,36 @@ describe('serializeSchema', () => {
     ])
   })
 
+  test('should serialize list values in string array', () => {
+    const schema = Schema.compile({
+      name: 'test',
+      types: [
+        defineType({
+          type: 'array',
+          name: 'list',
+          of: [{type: 'string'}],
+          options: {
+            list: [
+              {title: 'A', value: 'a'},
+              {title: 'B', value: 'b'},
+            ],
+          },
+        }),
+      ],
+    })
+
+    const serializedTypes = serializeSchema(schema, {leanFormat: true})
+
+    expect(serializedTypes).toEqual([
+      {
+        name: 'list',
+        type: 'array',
+        of: [{type: 'string', name: 'string', title: 'String'}],
+        values: ['a', 'b'],
+      },
+    ])
+  })
+
   test('should exclude truthy hidden and readonly', () => {
     const schema = Schema.compile({
       name: 'test',

package/src/schemas/serialize/serializeSchema.ts

@@ -83,10 +83,11 @@ function getBaseFields(
           imagePromptField: imagePromptField,
         }
       : undefined,
-    values:
-      type.jsonType === 'string' && type?.options?.list
-        ? type?.options?.list.map((v) => (typeof v === 'string' ? v : v.value ?? `${v.title}`))
-        : undefined,
+    values: type?.options?.list
+      ? type?.options?.list.map((v: string | {value: string; title: string}) =>
+          typeof v === 'string' ? v : v.value ?? `${v.title}`
+        )
+      : undefined,
     of: 'of' in type && typeName === 'array' ? arrayOf(type, schema, options) : undefined,
     to:
       'to' in type && typeName === 'reference'

package/src/useApiClient.ts

@@ -26,7 +26,6 @@ export interface InstructStatus {
 }
 
 const basePath = '/assist/tasks/instruction'
-const editorialAiFieldActionFeature = 'editorialAiFieldActions'
 
 export function useApiClient(customApiClient?: (defaultClient: SanityClient) => SanityClient) {
   const client = useClient({apiVersion: '2023-06-05'})
@@ -91,33 +90,22 @@ export function useGenerateCaption(apiClient: SanityClient) {
 
 export function useGetInstructStatus(apiClient: SanityClient) {
   const [loading, setLoading] = useState(true)
-  const projectClient = useClient({apiVersion: '2023-06-05'})
 
   const getInstructStatus = useCallback(async () => {
     setLoading(true)
 
     const projectId = apiClient.config().projectId
     try {
-      const features = await projectClient.request<string[]>({
-        method: 'GET',
-        url: `/projects/${projectId}/features`,
-      })
-
-      const enabled = features.some((f) => f === editorialAiFieldActionFeature)
-
-      const status = await apiClient.request<Omit<InstructStatus, 'enabled'>>({
+      const status = await apiClient.request<InstructStatus>({
         method: 'GET',
         url: `${basePath}/${apiClient.config().dataset}/status?projectId=${projectId}`,
       })
 
-      return {
-        ...status,
-        enabled,
-      }
+      return status
     } finally {
       setLoading(false)
     }
-  }, [setLoading, apiClient, projectClient])
+  }, [setLoading, apiClient])
 
   return {
     loading,