`but branch list|show` use their own version of a persisted CliId mapping · gitbutlerapp/gitbutler#11276

(6 comments) (1 reaction) (0 assignees)Rust (19,787 stars) (862 forks)batch import

help wanted

説明

What it does is to just assign a number to each of them, and each time but branch list is run, it stores a mapping from CliId to branch name in .git/gitbutler/branch_id_map.json.

This isn't what we want, as we really don't want to create more loose files. Nor is the idea of CliIds that they must be persisted to work.

These commands should be cleaned up to use a standard CliId that can be generated from context, so persistence can be completely removed. See load_id_map and store_id_map for reference.

コントリビューターガイド

技術スタック: rustshell
領域: clibackenddeveloper experience
Issue 種別: refactor
難度: 3
推定時間: 1-3 hours
活動状況: fresh
明確さ: mostly clear
前提条件: Rust knowledgefamiliarity with git internals
初心者向け度: 40
調査方針: Examine the current implementation of `but branch list|show` and the CliId persistence in `load id map` and `store id map` functions. Determine how to derive CliId from context (e.g., branch name, reference) to eliminate the need for the persistent mapping file. Review related comments and the `branch id map.json` usage to ensure the solution aligns with the project's architecture.