Cap or split hub notes so one note cannot crowd out query results #1570
Labels
No labels
P0
P1
P2
P3
bug
category/code-change
category/cross-repo
category/eval-run
category/investigation
category/no-test-target
category/owner-ruling
category/prose-artifact
category/recurring-chore
category/structure
category/umbrella
crc-refactor
create
delete
enhancement
filed-by/agent
filed-by/user
frozen
lint-rule
needs-info
needs-triage
next
plugin/cc-architect
plugin/os
plugin/os-adr
plugin/os-aidd-lint
plugin/os-backlog
plugin/os-context
plugin/os-doc-hygiene
plugin/os-sdlc
plugin/os-vault
project/cc-os
ready-for-agent
ready-for-human
recurring
review
rule-proposal
update
waiting
wayfinder:grilling
wayfinder:map
wayfinder:map
wayfinder:research
wayfinder:task
wayfinder:task
wontfix
worklist/deviations
worklist/lint-rule
worklist/new-implement-build
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
jared/cc-os#1570
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
One long note can crowd other notes out of query results.
reference/n8n-api-data-tables-403.mdyields 91 nodes, 5.8% of the vault graph (1562 nodes).Evidence: /tmp/claude-1000/-home-jared-Documents-SecondBrain/b10276a0-3582-48b5-b0fe-f13795bdcc75/scratchpad/vault-graph-weak-retrieval-evidence.md and /tmp/claude-1000/-home-jared-Documents-SecondBrain/8a73b51f-d8a3-4dfc-9de7-e72ea77eacac/scratchpad/graph-test-results.md (copied below; do not depend on the paths).
Observed
llm.py:31,700), so graphify slices it and sends several calls. The log shows "slice ... truncated at depth 0; splitting".Reproduce
Approach
Choose one of two designs and state the reason. Either cap the nodes per source note in query output, or split hub notes into focused notes of one topic each. The split also helps notes that now get one node. Test with question 1 and the 12 vault questions.
Required work: none beyond the acceptance criteria.
Orchestrator work: apply the split or the cap, rebuild the vault graph, and re-run the 12 vault questions.
Acceptance criteria
n8n-api-data-tables-403.md(or from its split notes taken together).Non-goals
Facts in output (see the summaries ticket).
Origin
The note-split half of this ticket moves to #1574 (revisit the vault-note process). This ticket keeps the query cap.
/os-backlog:ticket-decompose routed this to
/os-sdlc:ticket-takedownbecause the owner is a new object.One change: a per-source cap in query output. Owner: the same os-vault query wrapper proposed on #1571. Graphify has no per-file limit (
serve.py:1168-1245). Measured:reference/n8n-api-data-tables-403.mdholds 92 of 2524 nodes. The '40 noise nodes in Q1' claim is unverified; measure first, after #1568. A hub can be the right answer, so add a check that such a query still returns it.Resolution
Done: Closed by measurement. No cap built.
Evidence: Q1 still has 45 of 48 nodes from reference/n8n-api-data-tables-403.md, so the crowding is real, but the NOTE block lists distinct notes and the expected note is third. No question in the set fails from crowding. P-Q7 reaches note rank 14 only at budget 40000, past the NOTE block limit, so a cap would not surface it. results-after-1571, rulings.tsv 70.
Follow-ups: none: note split stays on #1574