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`

オープン
#611 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
4/5
見積もり時間
3〜5日
初心者へのやさしさ
35/100
issue の種類
バグ
明瞭さ
おおむね明確
活発さ
活発
技術スタック
rust
領域
backend

調査の方向性

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.

索引モデルが issue の本文から書いたものです。

説明

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.

主要言語
Rust
スター
2.1k
フォーク
110
平均マージ
23時間 52分
マージ済み PR(30日)
138

環境構築

このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Ephemeral-AI-Lab/layerfs のほかの issue

Ephemeral-AI-Lab/layerfs の issue をすべて見る

似ている issue

Rust の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。