Skip to content

Conversation

@nickanderson
Copy link
Member

No description provided.

@nickanderson
Copy link
Member Author

@cf-bottom jenkins please

@cf-bottom
Copy link

Copy link
Contributor

@craigcomstock craigcomstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any reason why this should be changed. I would expect something in the commit to describe why this change was needed, what problem it is solving, ...

@nickanderson
Copy link
Member Author

Well, the WHY that caused me to take action is because the improper casing was sending my LLMs off in bad directions with my vibe coding over the weekend.

But, in fact I think the reason is because what was there was not accurate.

Views were called Tables, and their names were incorrect, which would not work if you tried to use them as documented in queries. I can update the commit message.

When I was vibe coding over the weekend, the inaccuracy of the documentation was
sending my agent off in the wrong direction. It kept trying to use queries with
the documented capitalization as I fed it that context for the schema.

This change fixes the documentation to correctly reflect the actual state of the
database with proper capitalization and identification of View vs Table.
@nickanderson nickanderson force-pushed the SqlSchemaFixCase/master branch from a7d5b5b to f16372d Compare November 25, 2025 18:05
@nickanderson nickanderson merged commit 37b624b into cfengine:master Nov 25, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants