index_notes: dot-files (e.g. .draft.md, .trash.md) are indexed despite BRAIN_INDEX_HIDDEN default
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- Meno di un'ora
- Idoneità per principianti
- 90/100
Direzione di ricerca
Inizia da src/sqlite_graph_memory/index_notes.py ed esamina come iter_notes() filtra i percorsi nascosti. Verifica il comportamento predefinito con la riproduzione nell’issue, poi controlla se i test esistenti coprono i file il cui nome inizia con un punto. Il lavoro è completato quando questi file vengono esclusi per impostazione predefinita e il comportamento documentato di BRAIN_INDEX_HIDDEN=1 continua a funzionare.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Steps to reproduce
- Create a markdown folder with a hidden file (e.g.
.secret.mdornotes/.draft.md). - Run
iter_notes(vault_dir)withBRAIN_INDEX_HIDDENunset (default behavior):from sqlite_graph_memory.index_notes import iter_notes iter_notes(vault_dir)
What happened
iter_notes() returns the dot-file:
['.secret.md']
In src/sqlite_graph_memory/index_notes.py:88:
if not include_hidden and any(part.startswith('.') for part in rel.parts[:-1]):
continue
rel.parts[:-1] inspects parent directories, but excludes rel.parts[-1] (the filename itself). Consequently, dot-files are included in the index.
What you expected instead
Per the docstring ("Skips dot-directories and Syncthing conflict copies. Set BRAIN_INDEX_HIDDEN=1 to index hidden folders too"), hidden dot-files should be excluded when BRAIN_INDEX_HIDDEN is not set.
Environment
OS: Windows 11 / Linux / macOS
Runtime: Python 3.12.7
This repo at: 9308a91 (v0.3.0)
Anything you already tried
Changing rel.parts[:-1] to rel.parts:
if not include_hidden and any(part.startswith('.') for part in rel.parts):
continue
Before you submit
- I searched the open and closed issues and this is not a duplicate.
- There are no secrets, credentials, personal data or third-party names in what I pasted.
- I would like to fix this myself.
- Lingua principale
- Python
- Stelle
- 9
- Fork
- 6
- Merge medio
- 20h 2m
- PR unite (30g)
- 14
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di tonydzi/sqlite-graph-memory
-
accepted help wanted
Difficoltà 3/5 Mezza giornata Idoneità per principianti 68/100
tonydzi/sqlite-graph-memory#21 ·
I maintainer di solito rispondono entro 1 giorno
-
accepted help wanted
Difficoltà 3/5 1-2 giorni Idoneità per principianti 50/100
tonydzi/sqlite-graph-memory#19 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
accepted help wanted
Difficoltà 4/5 1-2 giorni Idoneità per principianti 62/100
tonydzi/sqlite-graph-memory#17 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di tonydzi/sqlite-graph-memory
Issue simili
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
yunaremaia/proto-drift#22 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
yunaremaia/memwatch#96 ·
-
bug good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
yunaremaia/ci-gate-watch#34 ·
-
bug good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
yunaremaia/agent-capability-attestation#64 ·
I maintainer di solito rispondono entro 1 giorno
-
bug good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
yunaremaia/agent-call-graph#43 ·
I maintainer di solito rispondono entro 1 giorno