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

(6 评论) (1 反应) (0 负责人)Rust (19,787 star) (862 fork)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
议题类型: 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.