Skip to content

Add Gaffer section to llms.txt#967

Merged
George-Payne merged 1 commit into
masterfrom
george/ui-1633-make-gaffer-discoverable-via-llmstxt-docs-section
Jun 4, 2026
Merged

Add Gaffer section to llms.txt#967
George-Payne merged 1 commit into
masterfrom
george/ui-1633-make-gaffer-discoverable-via-llmstxt-docs-section

Conversation

@George-Payne
Copy link
Copy Markdown
Member

@George-Payne George-Payne commented Jun 4, 2026

Adds a ## Gaffer (Projection Development Toolkit) section to the curated llms.txt (docs/.vuepress/configs/llms.ts), peer to Client Libraries / Cloud.

Gaffer is a separate Astro/Starlight site (gaffer.kurrent.io), so the section is a static list of external links. It leads with a pointer to gaffer's own llms.txt / llms-full.txt so tooling can fetch gaffer's full machine-readable index, plus a few key human-facing pages.

see: kurrent-io/gaffer#138

@George-Payne George-Payne requested a review from a team as a code owner June 4, 2026 07:54
@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 4, 2026

UI-1633

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

Docs build

@qodo-code-review
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Add Gaffer section to llms.txt documentation index

✨ Enhancement 📝 Documentation

Grey Divider

Walkthroughs

Description
• Adds Gaffer section to curated llms.txt for AI tool discovery
• Points to Gaffer's own llms.txt and key documentation pages
• Gaffer is separate Astro/Starlight site with external URLs
• Defers to Gaffer's own index to avoid content duplication
Diagram
flowchart LR
  llmsConfig["llms.ts config"] -- "adds gaffer getter" --> gafferGetter["gaffer TemplateGetter"]
  gafferGetter -- "returns markdown links" --> gafferLinks["External Gaffer URLs"]
  llmsConfig -- "registers in template" --> llmsTemplate["llms.txt template"]
  llmsTemplate -- "includes gaffer section" --> output["Curated llms.txt output"]

Loading

Grey Divider

File Changes

1. docs/.vuepress/configs/llms.ts 📝 Documentation +17/-0

Add Gaffer section with external documentation links

• Adds gaffer TemplateGetter function returning markdown links to Gaffer documentation
• Links include main Gaffer site, llms.txt/llms-full.txt indexes, install guide, MCP server setup,
 and testing docs
• Adds ## Gaffer (Projection Development Toolkit) section to llms.txt template with {gaffer}
 placeholder
• Registers gaffer in the template variables array between clients and connectorsSinks

docs/.vuepress/configs/llms.ts


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown
Contributor

qodo-code-review Bot commented Jun 4, 2026

Code Review by Qodo

🐞 Bugs (1) 📘 Rule violations (0) 📎 Requirement gaps (0) 🔗 Cross-repo conflicts (0)

Context used

Grey Divider


Advisory comments

1. Outdated section-order comment 🐞 Bug ⚙ Maintainability
Description
The comment above CURATED_TEMPLATE still documents the curated section order as “overview →
concepts → clients → APIs → …”, but the template now inserts the new Gaffer section between Clients
and APIs. This mismatch can mislead future edits to section ordering and placeholder registration.
Code

docs/.vuepress/configs/llms.ts[R574-576]

+## Gaffer (Projection Development Toolkit)
+{gaffer}
+
Evidence
The comment explicitly states the section order, but the template content below now includes a
Gaffer section between {clients} and {apis}, contradicting that documented order.

docs/.vuepress/configs/llms.ts[546-550]
docs/.vuepress/configs/llms.ts[569-581]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
A comment above `CURATED_TEMPLATE` claims the section order is `overview → concepts → clients → APIs → ...`, but the template now includes a `Gaffer` section between `clients` and `apis`. Update the comment so it accurately reflects the current template order.

## Issue Context
This comment is used as a quick guide for future maintainers; if it diverges from the actual template, future changes risk being made incorrectly.

## Fix Focus Areas
- docs/.vuepress/configs/llms.ts[546-550]

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

Qodo Logo

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 4, 2026

Deploying documentation with  Cloudflare Pages  Cloudflare Pages

Latest commit: 742ddcb
Status: ✅  Deploy successful!
Preview URL: https://2d69b23e.documentation-21k.pages.dev
Branch Preview URL: https://george-ui-1633-make-gaffer-d.documentation-21k.pages.dev

View logs

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

Docs build

George-Payne added a commit to kurrent-io/gaffer that referenced this pull request Jun 4, 2026
Adds the
[`starlight-llms-txt`](https://github.com/delucis/starlight-llms-txt)
plugin to the gaffer docs site so it generates its own `llms.txt`
(curated index), `llms-full.txt` (complete docs in one file), and
`llms-small.txt` (abridged).

docs.kurrent.io cross-links to gaffer's own AI index rather than
duplicating gaffer's page list, so gaffer owns and maintains its index
from its real page tree.

see: kurrent-io/documentation#967
@George-Payne George-Payne force-pushed the george/ui-1633-make-gaffer-discoverable-via-llmstxt-docs-section branch from 17e7b80 to 6967fb4 Compare June 4, 2026 08:24
- Add a `## Gaffer` section to the curated llms.txt, peer to Client Libraries
- Points LLM tooling at gaffer's own llms.txt / llms-full.txt plus key pages

Gaffer is a separate Astro/Starlight site, so the section references
external gaffer.kurrent.io URLs and defers to gaffer's own AI index
rather than duplicating its page list (UI-1633).
@George-Payne George-Payne force-pushed the george/ui-1633-make-gaffer-discoverable-via-llmstxt-docs-section branch from 6967fb4 to 742ddcb Compare June 4, 2026 08:29
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

Docs Preview

  • Status: ⏳ Creating preview…

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2026

Docs build

@George-Payne George-Payne merged commit b7fa5c9 into master Jun 4, 2026
3 checks passed
@George-Payne George-Payne deleted the george/ui-1633-make-gaffer-discoverable-via-llmstxt-docs-section branch June 4, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants