[8.15] @kbn/io-ts-utils: strictKeys: validate objects in arrays (elas…

…tic#191607) (elastic#191797)

# Backport

This will backport the following commits from `main` to `8.15`:
- [@kbn/io-ts-utils: strictKeys: validate objects in arrays
(elastic#191607)](elastic#191607)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Dario
Gieselaar","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-08-30T06:00:42Z","message":"@kbn/io-ts-utils:
strictKeys: validate objects in arrays (elastic#191607)\n\nMakes sure keys from
plain objects in arrays are validated as
well.\r\n\r\n---------\r\n\r\nCo-authored-by: Søren Louv-Jansen
<[email protected]>\r\nCo-authored-by: Carlos Crespo
<[email protected]>","sha":"8ade3da14d67aabb3a3bd4172187579559e610ff","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Obs
AI
Assistant","ci:project-deploy-observability","v8.16.0","v8.15.1"],"title":"@kbn/io-ts-utils:
make sure keys from objects in arrays are
validated","number":191607,"url":"https://github.com/elastic/kibana/pull/191607","mergeCommit":{"message":"@kbn/io-ts-utils:
strictKeys: validate objects in arrays (elastic#191607)\n\nMakes sure keys from
plain objects in arrays are validated as
well.\r\n\r\n---------\r\n\r\nCo-authored-by: Søren Louv-Jansen
<[email protected]>\r\nCo-authored-by: Carlos Crespo
<[email protected]>","sha":"8ade3da14d67aabb3a3bd4172187579559e610ff"}},"sourceBranch":"main","suggestedTargetBranches":["8.15"],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/191607","number":191607,"mergeCommit":{"message":"@kbn/io-ts-utils:
strictKeys: validate objects in arrays (elastic#191607)\n\nMakes sure keys from
plain objects in arrays are validated as
well.\r\n\r\n---------\r\n\r\nCo-authored-by: Søren Louv-Jansen
<[email protected]>\r\nCo-authored-by: Carlos Crespo
<[email protected]>","sha":"8ade3da14d67aabb3a3bd4172187579559e610ff"}},{"branch":"8.15","label":"v8.15.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Dario Gieselaar <[email protected]>

packages/kbn-io-ts-utils/src/strict_keys_rt/index.test.ts

@@ -205,7 +205,38 @@ describe('strictKeysRt', () => {
       {
         type: t.array(t.type({ foo: t.string })),
         passes: [[{ foo: 'bar' }], [{ foo: 'baz' }, { foo: 'bar' }]],
-        fails: [],
+        fails: [{ foo: 'bar', bar: 'foo' }],
+      },
+      {
+        type: t.type({
+          nestedArray: t.array(
+            t.type({
+              bar: t.string,
+            })
+          ),
+        }),
+        passes: [
+          {
+            nestedArray: [],
+          },
+          {
+            nestedArray: [
+              {
+                bar: 'foo',
+              },
+            ],
+          },
+        ],
+        fails: [
+          {
+            nestedArray: [
+              {
+                bar: 'foo',
+                foo: 'bar',
+              },
+            ],
+          },
+        ],
       },
     ];
 

packages/kbn-io-ts-utils/src/strict_keys_rt/index.ts

@@ -8,7 +8,7 @@
 
 import * as t from 'io-ts';
 import { either, isRight } from 'fp-ts/lib/Either';
-import { difference, isPlainObject, forEach } from 'lodash';
+import { difference, isPlainObject, forEach, isArray, castArray } from 'lodash';
 import { MergeType } from '../merge_rt';
 
 /*
@@ -100,11 +100,31 @@ function getHandledKeys<T extends Record<string, unknown>>(
       keys.handled.add(ownPrefix);
     }
 
+    const processObject = (typeForObject: t.Mixed, objectToProcess: Record<string, unknown>) => {
+      const nextKeys = getHandledKeys(typeForObject, objectToProcess, ownPrefix);
+      nextKeys.all.forEach((k) => keys.all.add(k));
+      nextKeys.handled.forEach((k) => keys.handled.add(k));
+    };
+
     if (isPlainObject(value)) {
-      handlingTypes.forEach((i) => {
-        const nextKeys = getHandledKeys(i, value as Record<string, unknown>, ownPrefix);
-        nextKeys.all.forEach((k) => keys.all.add(k));
-        nextKeys.handled.forEach((k) => keys.handled.add(k));
+      handlingTypes.forEach((typeAtIndex) => {
+        processObject(typeAtIndex, value as Record<string, unknown>);
+      });
+    }
+
+    if (isArray(value)) {
+      handlingTypes.forEach((typeAtIndex) => {
+        if (!isParsableType(typeAtIndex) || typeAtIndex._tag !== 'ArrayType') {
+          return;
+        }
+
+        const innerType = typeAtIndex.type;
+
+        castArray(value).forEach((valueAtIndex) => {
+          if (isPlainObject(valueAtIndex)) {
+            processObject(innerType, valueAtIndex as Record<string, unknown>);
+          }
+        });
       });
     }
   });

x-pack/plugins/observability_solution/observability_ai_assistant/common/types.ts

@@ -101,6 +101,7 @@ export interface UserInstruction {
 export type UserInstructionOrPlainText = string | UserInstruction;
 
 export interface ObservabilityAIAssistantScreenContextRequest {
+  starterPrompts?: StarterPrompt[];
   screenDescription?: string;
   data?: Array<{
     name: string;

x-pack/plugins/observability_solution/observability_ai_assistant/server/routes/runtime_types.ts

@@ -14,6 +14,7 @@ import {
   type Message,
   MessageRole,
   type ObservabilityAIAssistantScreenContextRequest,
+  type StarterPrompt,
 } from '../../common/types';
 
 const serializeableRt = t.any;
@@ -129,6 +130,12 @@ export const functionRt = t.intersection([
   }),
 ]);
 
+export const starterPromptRt: t.Type<StarterPrompt> = t.type({
+  title: t.string,
+  prompt: t.string,
+  icon: t.any,
+});
+
 export const screenContextRt: t.Type<ObservabilityAIAssistantScreenContextRequest> = t.partial({
   description: t.string,
   data: t.array(
@@ -139,4 +146,6 @@ export const screenContextRt: t.Type<ObservabilityAIAssistantScreenContextReques
     })
   ),
   actions: t.array(functionRt),
+  screenDescription: t.string,
+  starterPrompts: t.array(starterPromptRt),
 });