Skip to content

Correct stale facts in llms.txt and pin its collector counts - #4157

Merged
erikdarlingdata merged 1 commit into
devfrom
feature/llms-txt-stale-facts
Sep 24, 2026
Merged

erikdarlingdata merged 1 commit into
devfrom
feature/llms-txt-stale-facts

Conversation

@erikdarlingdata

Copy link
Copy Markdown
Owner

What does this PR do?

It corrects the facts in llms.txt that were out of date. It also adds a test so that the collector counts in that file cannot drift again.

Changes to llms.txt:

  • Azure SQL Database is supported by Lite and Darling. The file said "Lite only".
  • The SQL Server collector count is 42. The file said "41 T-SQL collectors".
  • The file did not mention PostgreSQL targets. It now says that Darling monitors self-hosted PostgreSQL, AWS RDS, and Aurora with 29 PostgreSQL collectors.
  • The Darling service runs on Windows or Linux. Its viewers are a Windows desktop app and a web dashboard. The file said "a detached viewer".
  • Alerts also go to Microsoft Teams, Slack, PagerDuty, and generic webhooks. The file listed only tray notifications and email.
  • The download count is 15,000+. The file said 4,300+. The new number counts the installer, ZIP, and Linux tarball assets of all releases. It does not count the files that the auto-updater downloads.
  • SolarWinds renamed SentryOne to SQL Sentry (SolarWinds page). The file now says "SolarWinds SQL Sentry (formerly SentryOne)" and spells "SolarWinds" correctly.
  • The documentation list now links the Darling operator guide.

Changes to CITATION.cff:

  • The version ("2.3.0") and date-released ("2026-03-18") fields are removed. No release step updates them, because the version bump is one line in Directory.Build.props. They still showed the March release.
  • The postgresql keyword is added.

New test: LlmsTxtCollectorCensusTests compares both collector counts in llms.txt with CollectorCatalog. The tool range in the same file already has this kind of test (LlmsTxtToolCensus_SpansTheTwoCurrentEditions).

Which component(s) does this affect?

  • Lite
  • Darling
  • Lite Tests
  • Darling Tests
  • SQL collection scripts
  • Documentation
  • Full Dashboard (deprecated)
  • CLI Installer (deprecated)

How was this tested?

  • Lite.Tests builds with 0 warnings.
  • LlmsTxtCollectorCensusTests and CrossAppMcpToolInventoryPinTests pass (8 tests).
  • To prove that the new test catches drift, I changed 42 to 41 and 29 to 28 in llms.txt. Both cases failed with the expected message. Then I restored the file, and both cases passed.
  • The download count comes from the GitHub releases API.
  • The product page URL returns HTTP 200.

Checklist

  • I have read the contributing guide
  • My code builds with zero warnings (dotnet build -c Debug)
  • I have tested my changes against at least one SQL Server version (not applicable, because no collector or query changed)
  • I have not introduced any hardcoded credentials or server names

🤖 Generated with Claude Code

https://claude.ai/code/session_01TnwcVMUfUSqBGy8yWVEDQH

llms.txt:
- Azure SQL Database is supported by Lite and Darling, not Lite only.
- 42 SQL Server collectors (it said 41 T-SQL collectors).
- Darling monitors PostgreSQL (self-hosted, AWS RDS, Aurora) with 29
  collectors. The file never mentioned PostgreSQL targets.
- The Darling service runs on Windows or Linux, and its viewers are a
  Windows desktop app and a web dashboard.
- Alerts also go to Teams, Slack, PagerDuty and generic webhooks.
- 15,000+ downloads (it said 4,300+). Counts installer, ZIP and tarball
  assets only, not auto-updater traffic.
- SentryOne is SolarWinds SQL Sentry now. Solarwinds -> SolarWinds.
- Link the Darling operator guide.

CITATION.cff: remove version 2.3.0 and date-released 2026-03-18. No
release step updates them (the version bump is Directory.Build.props
only), so they have been stale since March. Add the postgresql keyword.

LlmsTxtCollectorCensusTests compares both collector counts with
CollectorCatalog, the same guard the file's tool range already has.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TnwcVMUfUSqBGy8yWVEDQH
@erikdarlingdata
erikdarlingdata enabled auto-merge (squash) September 24, 2026 10:51
@erikdarlingdata
erikdarlingdata merged commit 61615f7 into dev Sep 24, 2026
15 of 16 checks passed
@erikdarlingdata
erikdarlingdata deleted the feature/llms-txt-stale-facts branch September 24, 2026 10:56
erikdarlingdata added a commit that referenced this pull request Sep 26, 2026
…4438)

Adds the missing [Unreleased] CHANGELOG entries for nine merged PRs. Two more need none.

- Fixed: #3588, #3886, #3889, #3900, #3911, #4016, #4032 and #4042.
- Changed: #4157. llms.txt and CITATION.cff now match the shipped product.
- None:
  - #4332 adds RawChunkIntervalPlanner without wiring it in; the later PR that wires it carries its own entry.
  - #4398 changes tests only.
- Each entry was written from its PR's diff, with a [#N] label linking the pull request.
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.

1 participant