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`
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
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 の本文から書いたものです。
説明
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.diffaccept the same 8+ character prefixes ascommit.get, or refuse a short id with a Commit error instead ofNoSuchBranch? - 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 を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Ephemeral-AI-Lab/layerfs のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
Ephemeral-AI-Lab/layerfs#689 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Ephemeral-AI-Lab/layerfs#643 ·
メンテナーはふだん 1 日以内に返信
-
`branch.publish` accepts a stale base and overwrites `main` (lost update)対応中かも @mengmengmengqiang が今日担当しました。 オープンapi/calls bug core/history
Ephemeral-AI-Lab/layerfs#691 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 8/100
Ephemeral-AI-Lab/layerfs#684 · コメント 10 件 ·
メンテナーはふだん 1 日以内に返信
-
docs
難易度 5/5 1週間以上 初心者へのやさしさ 15/100
Ephemeral-AI-Lab/layerfs#683 ·
メンテナーはふだん 1 日以内に返信
Ephemeral-AI-Lab/layerfs の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
pop-os/cosmic-settings#2296 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
jonathanong/no-mistakes#3431 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Eyevinn/open-live-ingest#36 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信