Hacktoberfest 2026: as issues que os mantenedores marcaram para outubro, abertas e boas para iniciantes. Ver issues do Hacktoberfest

[Bug]: manual nodes are persisted only by the desktop app, so a headless nvpair-tui node loses them on every restart

Aberta
#58 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 5 dias

@sylvesterkaczmarek já está trabalhando nisso.

Desde 10/9/2026.

  • #60 de @sylvesterkaczmarek — aberto

Avaliação

Dificuldade
4/5
Tempo estimado
3-5 dias
Facilidade para iniciantes
65/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
go, typescript
Domínio
backend, desktop

Direção de pesquisa

Comece com services/nvpair-ui-broker/manualnodes.go e services/nvpair-tui/ui/manualnodes.go e, em seguida, compare-os com desktop/src/electron/service-bridge/manual-nodes-store.ts e o comportamento de replay do supervisor. Reproduza a reinicialização com o comando systemctl documentado e inspecione o README de node-settings para encontrar o caminho dos dados por usuário. Está concluído quando os nós manuais são persistidos e reproduzidos novamente após ready para os front ends de TUI, desktop e service-mode.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

PAIR version or commit

Debian package nvpair 0.1.1-463 (amd64). nvpair-ui-broker --version reports 0.40.2.

Affected component

Discovery or pairing

Environment
OS:            Ubuntu 26.04 LTS, x86_64
Front end:     nvpair-tui only (started by a `systemd --user` unit inside tmux, linger enabled).
               The Electron desktop app is not installed on these nodes.
GPU / driver:  consumer NVIDIA GPUs, driver 595.91.07
Engine:        Ollama 0.33.3, adopted by PAIR as the node engine
Cluster size:  5 nodes joined to one cluster, split across two routed subnets
Steps to reproduce
  1. On a node whose only front end is nvpair-tui, open the Manual tab, press a, enter a peer address, press Enter.
  2. Confirm the entry is listed as OLLAMA yes / NODE yes, and that the peer appears in Proxies -> Ollama upstreams.
  3. Restart the front end and its broker: systemctl --user restart nvpair-tui.
  4. Open the Manual tab again.
Expected behavior

The manual node list survives a restart on any supported front end.

This matters because on a cluster split across routed subnets, manual nodes are the only way to
pair peers at all (mDNS does not cross subnets, and #1 additionally prevents reflectors from
relaying the records). Losing the list on every restart makes such a cluster impractical to
operate without the desktop app.

Actual behavior

After the restart the Manual tab shows No manual nodes. and the peer is gone from
Ollama upstreams. Nothing was written to disk: configs/manual-nodes.json never appears
in the per-user data dir, and the configs/ directory is not created.

Reading the source, the durable list appears to be implemented only in the Electron app:

  • desktop/src/electron/service-bridge/manual-nodes-store.ts reads and writes
    <userData>/configs/manual-nodes.json with entries { id, address, name }.
  • desktop/src/electron/service-bridge/modular-supervisor.ts notes that
    "PAIR UI owns the durable list (manual-nodes.json) and replays it once".
  • services/nvpair-ui-broker/manualnodes.go performs no file I/O, and
    services/nvpair-tui/ui/manualnodes.go neither reads nor writes that file.
  • services/nvpair-manual-nodes is stateless by design — its README documents
    node/add, node/remove, nodes/list and no state or config-path flag.

So the replay is desktop-only, even though services/nvpair-node-settings/README.md already
describes manual-nodes.json as living in the per-user data dir, "same directory as
settings.json" — which reads as if the file is meant to be a per-node artifact rather than a
private detail of the desktop app.

Suggested fix

Move ownership of the durable list, and the one-shot replay after ready, into
nvpair-ui-broker. The broker already supervises nvpair-manual-nodes over JSON-RPC and
already has node/add available, so every front end — desktop, TUI, and any future
service mode (cf. #49, #32) — would inherit persistence without its own copy of the logic.

Sanitized logs

Manual tab, immediately after adding the entry:

 ID                    ADDRESS                       OLLAMA    NODE
 manual:peer-a.…       peer-a.example.internal       yes       yes

Per-user data dir at the same moment (no configs/ directory is created):

$ ls "$HOME/.config/Nvidia Corporation/Personal AI Router/configs/"
ls: cannot access '.../configs/': No such file or directory

After systemctl --user restart nvpair-tui:

No manual nodes. Press a to add one by address.
a add node • r remove node • tab next • shift+tab prev • ? help • q quit
$ ls "$HOME/.config/Nvidia Corporation/Personal AI Router/configs/"
ls: cannot access '.../configs/': No such file or directory
Related
  • #1 — mDNS answers sent from an ephemeral source port, so reflectors never relay node records
    across subnets. Together with this issue, a routed multi-subnet cluster cannot be kept paired
    from a headless node.
  • #49 — Build a service version on Linux.
  • #32 — Run a services-only PAIR node as a container or Kubernetes pod.
Confirmations
  • I searched existing issues for duplicates.
  • This is not a security vulnerability.
  • I agree to follow the Code of Conduct.
Linguagem predominante
Go
Estrelas
1.6k
Forks
266
Merge médio
3d 6h
PRs com merge (30d)
22

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de NVIDIA/Personal-AI-Router

Todas as issues de NVIDIA/Personal-AI-Router

Issues semelhantes

Mais issues de Go

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.