Skip to content

[EIS] Add Decision models to the sheet sync and supported models page - #9035

Draft
theletterf wants to merge 2 commits into
mainfrom
eis-decision-models
Draft

theletterf wants to merge 2 commits into
mainfrom
eis-decision-models

Conversation

@theletterf

Copy link
Copy Markdown
Member

Sean Handley let me know today that the EIS catalog is adding decision models, under a new Decision task type. The first one, a Jina Decision model, launches next month. This PR gets the sheet-to-docs automation ready for it.

The sync filters the Type column by exact match, so Decision rows would have been dropped silently. Sean confirmed it's a new type value and not a new column, so the fix is small.

  • config.yml: adds a decision-models.csv output that filters on Decision.
  • eis-supported-models.md: adds a "Decision models" section with a csv-include.
  • decision-models.csv: a header-only seed file, so the page builds before the sync generates real data.
  • sync_sheet.py: write_csv now writes a header row when a filter matches nothing. Its comment already said so, but it wrote a 0-byte file.

I built the docs locally with docs-builder (0 errors, 0 warnings). With no rows, the section renders a table with only its header row. I tested the write_csv change in isolation. I have not run the workflow end to end.

The Type string Decision comes from Sean's message. The sheet has no Decision rows yet, so I couldn't confirm it there. Please check it when the first row lands.

I opened this as a draft because merging today would publish an empty table. I'd merge it when the first Decision model row appears in the sheet, then run the workflow with dry_run to confirm the CSV fills in. I left the unused models.csv alone; that cleanup can be a separate PR.

Related to elastic/docs-eng-team#1130

🤖 Generated with Claude Code

theletterf and others added 2 commits October 26, 2026 14:08
Add a decision-models.csv output filtered on the Decision type, a Decision
models section on the supported models page, and write a header-only CSV
when a filter matches no rows.

Related to elastic/docs-eng-team#1130

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

@github-actions

Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@theletterf

Copy link
Copy Markdown
Member Author

+CC @elastic/developer-docs @seanhandley Ready to merge when the new Decision models come out.

This branch was successfully deployed

1 active deployment
docs-preview — cbaae045 Deployed Oct 26, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants