queryEvidence keeps the five-second signal while the traversal signal moved to thirty

Aperta
  • Qualcosa non va?
#2,184 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
25/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
javascript, sql
Ambito
databases

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
JavaScript
Stelle
11
Fork
0
Merge medio
2h 49m
PR unite (30g)
297

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di hyparam/hypaware

Tutte le issue di hyparam/hypaware

Issue simili

Altre issue su JavaScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.