Skip to content

Fix inaccuracies in LLM documentation (llms-full.txt + llms.txt)#11

Merged
MaksimShevtsov merged 2 commits into
001-llm-docsfrom
copilot/sub-pr-10
Feb 22, 2026
Merged

Fix inaccuracies in LLM documentation (llms-full.txt + llms.txt)#11
MaksimShevtsov merged 2 commits into
001-llm-docsfrom
copilot/sub-pr-10

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 22, 2026

Several factual inaccuracies in the LLM docs were flagged in review: wrong package name, misrepresented pooling behavior, undocumented no-op config fields, and an incorrect exception class count.

Changes

  • Package name: install commands corrected from row-queryrowquery
  • Code fence: install block changed from pythonbash (these are shell commands)
  • Pooling description: replaced "delegates to underlying drivers / no own pool" with accurate description of the current simple in-memory list-based pool (create_pool() returns a list, acquire_connection() does pool.pop())
  • pool_timeout / pool_recycle: marked as "Reserved for future use. Currently has no effect at runtime." — neither field is wired up in any adapter
  • Exception count: removed stale (18 classes) qualifier from llms.txt; exceptions.py defines 20 classes

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: MaksimShevtsov <20194438+MaksimShevtsov@users.noreply.github.com>
Copilot AI changed the title [WIP] Add LLM-optimized documentation for RowQuery API Fix inaccuracies in LLM documentation (llms-full.txt + llms.txt) Feb 22, 2026
Copilot AI requested a review from MaksimShevtsov February 22, 2026 17:20
@MaksimShevtsov MaksimShevtsov marked this pull request as ready for review February 22, 2026 18:21
@MaksimShevtsov MaksimShevtsov merged commit f00e6b1 into 001-llm-docs Feb 22, 2026
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