2.0: `commit list main` is refused, a short Commit id to `commit diff` is reported as a Branch, `project list` has no `has_more`

Aperta
  • Qualcosa non va?
#611 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
3-5 giorni
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
backend

Direzione di ricerca

Read core/docs/issues/533/HANDOFF.md and compare its three reported gaps with the CLI behavior at commit c421d9dba. Trace the commit list, commit diff, and project list command paths and their tests; the issue identifies no implementation files or tests. Before changing behavior, resolve the open questions about main and short commit prefixes; done means the intended behaviors are decided and covered by tests.

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

Descrizione

bug

Three of the "Known gaps found late" in core/docs/issues/533/HANDOFF.md still reproduce at c421d9dba (macOS arm64, native machine, debug build, fresh home).

1. commit list on main is refused

$ layerfs branch list gaps/p1        → items include {"name":"main", ...}
$ layerfs commit list gaps/p1 main   → {"code":"NoSuchBranch","message":"Branch main does not exist"}

S §3.3 says "On main, a Layer id is accepted wherever a Commit is", so I expected main's Layers, newest first.

2. A short Commit id to commit diff is reported as a Branch

$ layerfs commit diff gaps/p1 <66-char id> <66-char id> --paths-only   → ok
$ layerfs commit diff gaps/p1 12e0edc1c2f9 126f10bdec46                → {"code":"NoSuchBranch","message":"Branch 12e0edc1c2f9 does not exist"}
$ layerfs commit get gaps/p1 a 126f10bdec46                             → ok (commit.get accepts prefixes of 8+ characters)

3. project list has no has_more

$ layerfs commit list gaps/p1 a --page-size 1   → {"items":[...], "has_more":true, "next_page_pos":1}
$ layerfs project list --page-size 1            → {"items":[...], "next_page_pos":1}

Questions:

  • For 2: should commit.diff accept the same 8+ character prefixes as commit.get, or refuse a short id with a Commit error instead of NoSuchBranch?
  • For 1: is listing main's Layers the intended answer, and is it wanted for 2.0?

I can send small PRs against layerfs_v2_0_0 for 2 and 3, one each with a test. I'll wait for a go-ahead in case these are already in progress.

Lingua principale
Rust
Stelle
2.1k
Fork
110
Merge medio
23h 52m
PR unite (30g)
138

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 Ephemeral-AI-Lab/layerfs

Tutte le issue di Ephemeral-AI-Lab/layerfs

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.