Correct the os-vault query skill for path direction, graph location, and label-only output #1564

New issue

Closed

opened 2026-11-05 14:19:06 +00:00 by jared · 2 comments

jared commented

2026-11-05 14:19:06 +00:00

Owner

Problem

The os-vault query skill gives commands that fail or mislead on the SecondBrain vault graph (1562 nodes, undirected).
A caller who follows the skill gets "No directed path found" and a false belief that the graph returns knowledge.
Evidence: /tmp/claude-1000/-home-jared-Documents-SecondBrain/8a73b51f-d8a3-4dfc-9de7-e72ea77eacac/scratchpad/graph-test-results.md (copied below; do not depend on the path).

Observed

graphify path "A" "B" is directed by default (--directed). The JSON directed:false flag is ignored.
On the undirected vault graph, every path call tried returned "No directed path found" and exit 0.
Question 4 (graphify path "Graceful Rate Limits Automation Standard" "Retry-After Header") gave "No directed path found". With --undirected it gave a 4-hop path.
The skill does not say that the graph path is relative to cwd (default graphify-out/graph.json). Outside the vault the caller must pass --graph.
Query output is a list of nodes and edges: labels, source file, community. It holds no note text and no summary. In the 12-question test, 0 of 12 outputs held the answer facts. The caller must open the note to get a fact.

Reproduce

n/a: the defects show in the skill text. Run graphify path "Graphify" "Qwen3.5 9B" in the vault without --undirected to see the directed default.

Approach

Edit the os-vault query skill. Show --undirected in every graphify path example. Add --graph <path> for use outside the vault. Add one line: query output holds labels only, so open the note for facts.

Required work:

Add --undirected to each graphify path command in plugins/os-vault/skills/query/SKILL.md.
Add a --graph instruction for use outside the vault to plugins/os-vault/skills/query/SKILL.md.
Add a line that output holds labels only, so a caller must open the note for facts, to plugins/os-vault/skills/query/SKILL.md.

Acceptance criteria

Given the query skill, when a reader looks at a graphify path example, then the example includes --undirected.
Given the query skill, when a caller works outside the vault, then the skill tells the caller to pass --graph with the vault graph path.
Given the query skill, when a caller reads the output rules, then a line states that output holds labels only and that the caller must open the note for facts.

Non-goals

Tag-scoped retrieval is unverified and stays out of this ticket. Adding summaries to output is a separate ticket.

Origin

Trigger: vault graph test of 12 questions, 2026-11-04 (8 PASS, 4 PARTIAL; strict facts-in-output 0 of 12).
Improvised this session: added --undirected to path calls by hand.
Chain: caller gets "No directed path found" ← graphify path defaults to --directed ← UPSTREAM-TOOL (graphify 0.9.102); skill omits the flag ← DESIGN (plugins/os-vault/skills/query/SKILL.md).
Root candidate: none found; searched backlog terms for this ticket (VERDICT: CREATE)
Where: plugins/os-vault/skills/query/SKILL.md
Session: b10276a0-3582-48b5-b0fe-f13795bdcc75
Transcript: /home/jared/.claude/projects/-home-jared-Documents-SecondBrain/b10276a0-3582-48b5-b0fe-f13795bdcc75.jsonl
Parent: #1540.

## Problem The os-vault query skill gives commands that fail or mislead on the SecondBrain vault graph (1562 nodes, undirected). A caller who follows the skill gets "No directed path found" and a false belief that the graph returns knowledge. Evidence: /tmp/claude-1000/-home-jared-Documents-SecondBrain/8a73b51f-d8a3-4dfc-9de7-e72ea77eacac/scratchpad/graph-test-results.md (copied below; do not depend on the path). ## Observed - `graphify path "A" "B"` is directed by default (`--directed`). The JSON `directed:false` flag is ignored. - On the undirected vault graph, every `path` call tried returned "No directed path found" and exit 0. - Question 4 (`graphify path "Graceful Rate Limits Automation Standard" "Retry-After Header"`) gave "No directed path found". With `--undirected` it gave a 4-hop path. - The skill does not say that the graph path is relative to cwd (default `graphify-out/graph.json`). Outside the vault the caller must pass `--graph`. - Query output is a list of nodes and edges: labels, source file, community. It holds no note text and no summary. In the 12-question test, 0 of 12 outputs held the answer facts. The caller must open the note to get a fact. ## Reproduce n/a: the defects show in the skill text. Run `graphify path "Graphify" "Qwen3.5 9B"` in the vault without `--undirected` to see the directed default. ## Approach Edit the os-vault query skill. Show `--undirected` in every `graphify path` example. Add `--graph <path>` for use outside the vault. Add one line: query output holds labels only, so open the note for facts. Required work: 1. Add `--undirected` to each `graphify path` command in `plugins/os-vault/skills/query/SKILL.md`. 2. Add a `--graph` instruction for use outside the vault to `plugins/os-vault/skills/query/SKILL.md`. 3. Add a line that output holds labels only, so a caller must open the note for facts, to `plugins/os-vault/skills/query/SKILL.md`. ## Acceptance criteria 1. Given the query skill, when a reader looks at a `graphify path` example, then the example includes `--undirected`. 2. Given the query skill, when a caller works outside the vault, then the skill tells the caller to pass `--graph` with the vault graph path. 3. Given the query skill, when a caller reads the output rules, then a line states that output holds labels only and that the caller must open the note for facts. ## Non-goals Tag-scoped retrieval is unverified and stays out of this ticket. Adding summaries to output is a separate ticket. ## Origin - Trigger: vault graph test of 12 questions, 2026-11-04 (8 PASS, 4 PARTIAL; strict facts-in-output 0 of 12). - Improvised this session: added `--undirected` to `path` calls by hand. - Chain: caller gets "No directed path found" ← `graphify path` defaults to `--directed` ← UPSTREAM-TOOL (graphify 0.9.102); skill omits the flag ← DESIGN (plugins/os-vault/skills/query/SKILL.md). - Root candidate: none found; searched backlog terms for this ticket (VERDICT: CREATE) - Where: plugins/os-vault/skills/query/SKILL.md - Session: b10276a0-3582-48b5-b0fe-f13795bdcc75 - Transcript: /home/jared/.claude/projects/-home-jared-Documents-SecondBrain/b10276a0-3582-48b5-b0fe-f13795bdcc75.jsonl - Parent: #1540.

jared added the

plugin/os-vault

filed-by/user

category/code-change

labels

2026-11-05 14:19:06 +00:00

jared commented

2026-11-05 14:33:52 +00:00

Author

Owner

Work started on branch ticket-1564 (worktree .claude/worktrees/ticket-1564).

jared commented

2026-11-05 14:34:59 +00:00

Author

Owner

Resolution

Done: Query skill now shows --undirected on graphify path, --graph for use outside the vault, and a labels-only output line

Evidence: Merged to main (commit bc2f801f); installed ~/.claude/plugins/os-vault skill identical to main; graphify path fails directed and returns 2-hop path with --undirected on the vault graph; aidd-lint Tier 1 findings unchanged by edit

Follow-ups: none

## Resolution **Done:** Query skill now shows --undirected on graphify path, --graph for use outside the vault, and a labels-only output line **Evidence:** Merged to main (commit bc2f801f); installed ~/.claude/plugins/os-vault skill identical to main; graphify path fails directed and returns 2-hop path with --undirected on the vault graph; aidd-lint Tier 1 findings unchanged by edit **Follow-ups:** none

jared closed this issue

2026-11-05 14:34:59 +00:00

jared referenced this issue from a commit

2026-11-05 16:11:53 +00:00

fix(os-vault): query skill uses --undirected, --graph, and labels-only note (#1564)