[SavedObjects] use ComposerQuery for esql pipeline option

[rudolf]

Summary

Replaces the raw pipeline: string option on SavedObjectsEsqlOptions with pipeline: ComposerQuery from @elastic/esql.

Developers now use the esql tagged template to build pipelines:

import { esql } from '@elastic/esql';

const result = await savedObjectsClient.esql({
  type: 'dashboard',
  namespaces: ['default'],
  pipeline: esql`WHERE dashboard.title LIKE ${{ title: searchTerm }} | LIMIT 100`,
});

Template holes (${{ name: value }}) are promoted to ES|QL named parameters — values are forwarded to Elasticsearch at the protocol level and never appear in the query string, eliminating the injection risk of string interpolation.

Changes

@kbn/core-saved-objects-api-server

• SavedObjectsEsqlOptions.pipeline changed from string to ComposerQuery
• Removed the @elastic/esql import in the params example (params are now extracted automatically from the template)

@kbn/core-saved-objects-api-server-internal

• Removed SOURCE_COMMAND_PATTERN guard (the type system now prevents raw strings; the esql tag's parser handles source-command detection)
• performEsql calls .toRequest() unconditionally to extract the pipeline string and named params
• ComposerQuery params are merged with any explicitly provided params before the request

@kbn/eslint-plugin-esql

• no_invalid_esql_template rule gains a pipelinePropertyNames option (default: ['pipeline'])
• When an esql template is used as the value of a pipeline: property, the missing-FROM warning is suppressed — the FROM clause is supplied by the callee

Consumers updated

• examples/saved_objects/server/esql_example_routes.ts
• src/core/server/integration_tests/saved_objects/service/lib/esql.test.ts
• dev_docs/tutorials/saved_objects_esql.mdx

Test plan

• Unit tests: node scripts/jest src/core/packages/saved-objects/api-server-internal/src/lib/apis/esql.test.ts (24 tests)
• ESLint rule tests: node scripts/jest packages/kbn-eslint-plugin-esql (36 tests)
• Type check: node scripts/type_check --project src/core/packages/saved-objects/api-server-internal/tsconfig.json
• Integration tests: node scripts/jest_integration src/core/server/integration_tests/saved_objects/service/lib/esql.test.ts

Made with Cursor

[achyutjhunjhunwala]

@rudolf I believe the changes to the ES Lint were not pushed ?

[kibanamachine]

💔 Build Failed

• Buildkite Build
• Commit: a8412d1
• Build duration: 33 mins

Failed CI Steps

• Check Types
• Local Check

Metrics [docs]

✅ unchanged

History

• 💛 Build #448115 was flaky d5bd87a

[stratoula]

We should not do the change in the esql pipeline, is there a reason you did that?

[stratoula]

Why did you change this? esql_grammar_sync.sh doesnt exist at the repo anymore, it is at the esql-js one