queryEvidence keeps the five-second signal while the traversal signal moved to thirty
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 25/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- javascript, sql
- Domain
- databases
Research direction
Read hypaware-core/plugins-workspace/context-graph/src/query.js around queryEvidence and queryNeighbors, then review the traversal budget tests in test/plugins/context-graph-query.test.js:178-201. The issue is explicitly backlog-only until an evidence-lookup surface exists; when one is added, decide and test the budget on both sides, and update the recorded split if needed.
Written by the indexing model from the issue text.
Description
Source: #2180 at 074860a333b58dce2d6125ba6d83fca15bbbeb1c
Location: hypaware-core/plugins-workspace/context-graph/src/query.js:352
Severity: low
Evidence
At head 074860a3, queryEvidence arms AbortSignal.timeout(5000) at query.js:352 while queryNeighbors arms AbortSignal.timeout(30_000) at :143. The PR's rationale (sequential reads on cold or unindexed storage) applies to queryEvidence identically: it issues up to three sequential executeQuerySql reads over node/edge and then ai_gateway_messages/enrichment_committed, which are much larger tables.
Why deferral is safe
Safe outside this PR because the path is unreachable by users today: git grep for queryEvidence outside test/ returns only the function's own definition at query.js:349, and the plugin registers only the graph neighbors verb / graph_neighbors MCP tool (verb.js:22-28), so no user can hit the 5s abort. LLP 0428's blockquote at head now records the split as deliberate ('Evidence lookups keep the five-second signal'), so the corpus is coherent. Raising a budget on an unreached path is speculative consistency cleanup, not an observed misbehavior.
Acceptance condition
When an evidence-lookup surface (CLI verb or MCP tool) is added, decide the queryEvidence signal budget deliberately, cover it on both sides like the traversal budget tests at test/plugins/context-graph-query.test.js:178-201, and update the LLP 0428/0431 recorded split if the value changes.
Review preference; backlog only. A human may explicitly delegate it with neutral:fix.
- Dominant language
- JavaScript
- Stars
- 11
- Forks
- 0
- Avg merge
- 2h 55m
- Merged PRs (30d)
- 294
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from hyparam/hypaware
-
neutral:fix
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
report_commands.js runtime output still says 'server' in four places; one names the destinationOpen
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day
-
Difficulty 1/5 Under an hour Newbie friendliness 76/100
Maintainers usually reply within 1 day
All issues in hyparam/hypaware
Similar issues
-
good first issue
Difficulty 2/5 3-5 days Newbie friendliness 62/100
paipai1999/POS-system#3 ·
-
good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
r-afael/better-yt-recommendations#36 ·
Maintainers usually reply within 1 day
-
needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
microsoft/playwright#43875 · 1 comment ·
Maintainers usually reply within 1 day
-
cli good first issue
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
Maintainers usually reply within 1 day
-
Run the dependency drift checkPossibly taken @spaceninja claimed this today. Open
Difficulty 1/5 Under an hour Newbie friendliness 82/100
cloudfour/stylelint-config-cloudfour#683 ·
Maintainers usually reply within 1 day