hypaware-query skill: name the node and edge column traps - #1475
Open
bgmcmullen wants to merge 1 commit into
Open
hypaware-query skill: name the node and edge column traps#1475bgmcmullen wants to merge 1 commit into
bgmcmullen wants to merge 1 commit into
Conversation
The skill listed the column-name traps for ai_gateway_messages and none for the graph tables. In recorded sessions the same wrong name, `type` instead of `node_type`, failed in 11 sessions in one month on one machine and was retyped after every failure. Both shipped copies (claude and codex) now name the graph columns beside the messages traps: `node_type`, `natural_key`, `label` on node; `edge_type`, `src_type`, `dst_type` on edge. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds one bullet to the hypaware-query skill, in both the claude and codex copies, naming the column-name traps on the graph tables beside the existing traps for
ai_gateway_messages: neithernodenoredgehas atypecolumn. A node's kind isnode_type, its identitynatural_key, its display textlabel; an edge's kind isedge_type, withsrc_typeanddst_typefor its endpoints.Why
Measured in recorded sessions on one machine:
Column "type" not foundon the node table in 11 sessions over one month, each time retyped after the failure. It was the most frequent real mistake in the record once permission prompts were set aside, and the skill's trap list did not cover it because the list was written for the messages table only.Found by the recommendation ask on the
first-ask-recommendbranch, which proposed a per-user hook for it; the lasting fix is in the shipped skill, so it is split out here. The same two lines are on that branch and will merge cleanly.Review notes
Docs only, no runtime change.
npm testpasses. CPU and memory: no concern, nothing executable changed.🤖 Generated with Claude Code