Skip to main content

jj_lib/
simple_op_store.rs

1// Copyright 2020 The Jujutsu Authors
2//
3// Licensed under the Apache License, Version 2.0 (the "License");
4// you may not use this file except in compliance with the License.
5// You may obtain a copy of the License at
6//
7// https://www.apache.org/licenses/LICENSE-2.0
8//
9// Unless required by applicable law or agreed to in writing, software
10// distributed under the License is distributed on an "AS IS" BASIS,
11// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12// See the License for the specific language governing permissions and
13// limitations under the License.
14
15#![expect(missing_docs)]
16
17use std::collections::BTreeMap;
18use std::collections::HashMap;
19use std::collections::HashSet;
20use std::fmt::Debug;
21use std::fs;
22use std::io;
23use std::io::ErrorKind;
24use std::io::Write as _;
25use std::path::Path;
26use std::path::PathBuf;
27use std::time::SystemTime;
28
29use async_trait::async_trait;
30use itertools::Itertools as _;
31use pollster::FutureExt as _;
32use prost::Message as _;
33use smallvec::SmallVec;
34use tempfile::NamedTempFile;
35use thiserror::Error;
36
37use crate::backend::BackendInitError;
38use crate::backend::CommitId;
39use crate::backend::MillisSinceEpoch;
40use crate::backend::Timestamp;
41use crate::content_hash::blake2b_hash;
42use crate::dag_walk;
43use crate::file_util::IoResultExt as _;
44use crate::file_util::PathError;
45use crate::file_util::persist_content_addressed_temp_file;
46use crate::merge::Merge;
47use crate::object_id::HexPrefix;
48use crate::object_id::ObjectId;
49use crate::object_id::PrefixResolution;
50use crate::op_store;
51use crate::op_store::OpStore;
52use crate::op_store::OpStoreError;
53use crate::op_store::OpStoreResult;
54use crate::op_store::Operation;
55use crate::op_store::OperationId;
56use crate::op_store::OperationMetadata;
57use crate::op_store::RefTarget;
58use crate::op_store::RemoteRef;
59use crate::op_store::RemoteRefState;
60use crate::op_store::RemoteView;
61use crate::op_store::RootOperationData;
62use crate::op_store::TimestampRange;
63use crate::op_store::View;
64use crate::op_store::ViewId;
65use crate::ref_name::GitRefNameBuf;
66use crate::ref_name::RefNameBuf;
67use crate::ref_name::RemoteNameBuf;
68use crate::ref_name::WorkspaceName;
69use crate::ref_name::WorkspaceNameBuf;
70
71// BLAKE2b-512 hash length in bytes
72const OPERATION_ID_LENGTH: usize = 64;
73const VIEW_ID_LENGTH: usize = 64;
74
75/// Error that may occur during [`SimpleOpStore`] initialization.
76#[derive(Debug, Error)]
77#[error("Failed to initialize simple operation store")]
78pub struct SimpleOpStoreInitError(#[from] pub PathError);
79
80impl From<SimpleOpStoreInitError> for BackendInitError {
81    fn from(err: SimpleOpStoreInitError) -> Self {
82        Self(err.into())
83    }
84}
85
86#[derive(Debug)]
87pub struct SimpleOpStore {
88    path: PathBuf,
89    root_data: RootOperationData,
90    root_operation_id: OperationId,
91    root_view_id: ViewId,
92}
93
94impl SimpleOpStore {
95    pub fn name() -> &'static str {
96        "simple_op_store"
97    }
98
99    /// Creates an empty OpStore. Returns error if it already exists.
100    pub fn init(
101        store_path: &Path,
102        root_data: RootOperationData,
103    ) -> Result<Self, SimpleOpStoreInitError> {
104        let store = Self::new(store_path, root_data);
105        store.init_base_dirs()?;
106        Ok(store)
107    }
108
109    /// Load an existing OpStore
110    pub fn load(store_path: &Path, root_data: RootOperationData) -> Self {
111        Self::new(store_path, root_data)
112    }
113
114    fn new(store_path: &Path, root_data: RootOperationData) -> Self {
115        Self {
116            path: store_path.to_path_buf(),
117            root_data,
118            root_operation_id: OperationId::from_bytes(&[0; OPERATION_ID_LENGTH]),
119            root_view_id: ViewId::from_bytes(&[0; VIEW_ID_LENGTH]),
120        }
121    }
122
123    fn init_base_dirs(&self) -> Result<(), PathError> {
124        for dir in [self.views_dir(), self.operations_dir()] {
125            fs::create_dir(&dir).context(&dir)?;
126        }
127        Ok(())
128    }
129
130    fn views_dir(&self) -> PathBuf {
131        self.path.join("views")
132    }
133
134    fn operations_dir(&self) -> PathBuf {
135        self.path.join("operations")
136    }
137}
138
139#[async_trait]
140impl OpStore for SimpleOpStore {
141    fn name(&self) -> &str {
142        Self::name()
143    }
144
145    fn root_operation_id(&self) -> &OperationId {
146        &self.root_operation_id
147    }
148
149    async fn read_view(&self, id: &ViewId) -> OpStoreResult<View> {
150        if *id == self.root_view_id {
151            return Ok(View::make_root(self.root_data.root_commit_id.clone()));
152        }
153
154        let path = self.views_dir().join(id.hex());
155        let buf = fs::read(&path)
156            .context(&path)
157            .map_err(|err| io_to_read_error(err, id))?;
158
159        let proto = crate::protos::simple_op_store::View::decode(&*buf)
160            .map_err(|err| to_read_error(err.into(), id))?;
161        view_from_proto(proto).map_err(|err| to_read_error(err.into(), id))
162    }
163
164    async fn write_view(&self, view: &View) -> OpStoreResult<ViewId> {
165        let dir = self.views_dir();
166        let temp_file = NamedTempFile::new_in(&dir)
167            .context(&dir)
168            .map_err(|err| io_to_write_error(err, "view"))?;
169
170        let proto = view_to_proto(view);
171        temp_file
172            .as_file()
173            .write_all(&proto.encode_to_vec())
174            .context(temp_file.path())
175            .map_err(|err| io_to_write_error(err, "view"))?;
176
177        let id = ViewId::new(blake2b_hash(view).to_vec());
178
179        let new_path = dir.join(id.hex());
180        persist_content_addressed_temp_file(temp_file, &new_path)
181            .context(&new_path)
182            .map_err(|err| io_to_write_error(err, "view"))?;
183        Ok(id)
184    }
185
186    async fn read_operation(&self, id: &OperationId) -> OpStoreResult<Operation> {
187        if *id == self.root_operation_id {
188            return Ok(Operation::make_root(self.root_view_id.clone()));
189        }
190
191        let path = self.operations_dir().join(id.hex());
192        let buf = fs::read(&path)
193            .context(&path)
194            .map_err(|err| io_to_read_error(err, id))?;
195
196        let proto = crate::protos::simple_op_store::Operation::decode(&*buf)
197            .map_err(|err| to_read_error(err.into(), id))?;
198        let mut operation =
199            operation_from_proto(proto).map_err(|err| to_read_error(err.into(), id))?;
200        if operation.parents.is_empty() {
201            // Repos created before we had the root operation will have an operation without
202            // parents.
203            operation.parents.push(self.root_operation_id.clone());
204        }
205        Ok(operation)
206    }
207
208    async fn write_operation(&self, operation: &Operation) -> OpStoreResult<OperationId> {
209        assert!(!operation.parents.is_empty());
210        let dir = self.operations_dir();
211        let temp_file = NamedTempFile::new_in(&dir)
212            .context(&dir)
213            .map_err(|err| io_to_write_error(err, "operation"))?;
214
215        let proto = operation_to_proto(operation);
216        temp_file
217            .as_file()
218            .write_all(&proto.encode_to_vec())
219            .context(temp_file.path())
220            .map_err(|err| io_to_write_error(err, "operation"))?;
221
222        let id = OperationId::new(blake2b_hash(operation).to_vec());
223
224        let new_path = dir.join(id.hex());
225        persist_content_addressed_temp_file(temp_file, &new_path)
226            .context(&new_path)
227            .map_err(|err| io_to_write_error(err, "operation"))?;
228        Ok(id)
229    }
230
231    async fn resolve_operation_id_prefix(
232        &self,
233        prefix: &HexPrefix,
234    ) -> OpStoreResult<PrefixResolution<OperationId>> {
235        let op_dir = self.operations_dir();
236        let find = || -> io::Result<_> {
237            let matches_root = prefix.matches(&self.root_operation_id);
238            let hex_prefix = prefix.hex();
239            if hex_prefix.len() == OPERATION_ID_LENGTH * 2 {
240                // Fast path for full-length ID
241                if matches_root || op_dir.join(hex_prefix).try_exists()? {
242                    let id = OperationId::from_bytes(prefix.as_full_bytes().unwrap());
243                    return Ok(PrefixResolution::SingleMatch(id));
244                } else {
245                    return Ok(PrefixResolution::NoMatch);
246                }
247            }
248
249            let mut matched = matches_root.then(|| self.root_operation_id.clone());
250            for entry in op_dir.read_dir()? {
251                let Ok(name) = entry?.file_name().into_string() else {
252                    continue; // Skip invalid UTF-8
253                };
254                if !name.starts_with(&hex_prefix) {
255                    continue;
256                }
257                let Some(id) = OperationId::try_from_hex(&name) else {
258                    continue; // Skip invalid hex
259                };
260                if matched.is_some() {
261                    return Ok(PrefixResolution::AmbiguousMatch);
262                }
263                matched = Some(id);
264            }
265            if let Some(id) = matched {
266                Ok(PrefixResolution::SingleMatch(id))
267            } else {
268                Ok(PrefixResolution::NoMatch)
269            }
270        };
271        find()
272            .context(&op_dir)
273            .map_err(|err| OpStoreError::Other(err.into()))
274    }
275
276    #[tracing::instrument(skip(self))]
277    async fn gc(&self, head_ids: &[OperationId], keep_newer: SystemTime) -> OpStoreResult<()> {
278        let to_op_id = |entry: &fs::DirEntry| -> Option<OperationId> {
279            let name = entry.file_name().into_string().ok()?;
280            OperationId::try_from_hex(name)
281        };
282        let to_view_id = |entry: &fs::DirEntry| -> Option<ViewId> {
283            let name = entry.file_name().into_string().ok()?;
284            ViewId::try_from_hex(name)
285        };
286        let remove_file_if_not_new = |entry: &fs::DirEntry| -> Result<(), PathError> {
287            let path = entry.path();
288            // Check timestamp, but there's still TOCTOU problem if an existing
289            // file is renewed.
290            let metadata = entry.metadata().context(&path)?;
291            let mtime = metadata.modified().expect("unsupported platform?");
292            if mtime > keep_newer {
293                tracing::trace!(?path, "not removing");
294                Ok(())
295            } else {
296                tracing::trace!(?path, "removing");
297                fs::remove_file(&path).context(&path)
298            }
299        };
300
301        // Reachable objects are resolved without considering the keep_newer
302        // parameter. We could collect ancestors of the "new" operations here,
303        // but more files can be added anyway after that.
304        let read_op = |id: &OperationId| {
305            self.read_operation(id)
306                .block_on()
307                .map(|data| (id.clone(), data))
308        };
309        let reachable_ops: HashMap<OperationId, Operation> = dag_walk::dfs_ok(
310            head_ids.iter().map(read_op),
311            |(id, _)| id.clone(),
312            |(_, data)| data.parents.iter().map(read_op).collect_vec(),
313        )
314        .try_collect()?;
315        let reachable_views: HashSet<&ViewId> =
316            reachable_ops.values().map(|data| &data.view_id).collect();
317        tracing::info!(
318            reachable_op_count = reachable_ops.len(),
319            reachable_view_count = reachable_views.len(),
320            "collected reachable objects"
321        );
322
323        let prune_ops = || -> Result<(), PathError> {
324            let op_dir = self.operations_dir();
325            for entry in op_dir.read_dir().context(&op_dir)? {
326                let entry = entry.context(&op_dir)?;
327                let Some(id) = to_op_id(&entry) else {
328                    tracing::trace!(?entry, "skipping invalid file name");
329                    continue;
330                };
331                if reachable_ops.contains_key(&id) {
332                    continue;
333                }
334                // If the operation was added after collecting reachable_views,
335                // its view mtime would also be renewed. So there's no need to
336                // update the reachable_views set to preserve the view.
337                remove_file_if_not_new(&entry)?;
338            }
339            Ok(())
340        };
341        prune_ops().map_err(|err| OpStoreError::Other(err.into()))?;
342
343        let prune_views = || -> Result<(), PathError> {
344            let view_dir = self.views_dir();
345            for entry in view_dir.read_dir().context(&view_dir)? {
346                let entry = entry.context(&view_dir)?;
347                let Some(id) = to_view_id(&entry) else {
348                    tracing::trace!(?entry, "skipping invalid file name");
349                    continue;
350                };
351                if reachable_views.contains(&id) {
352                    continue;
353                }
354                remove_file_if_not_new(&entry)?;
355            }
356            Ok(())
357        };
358        prune_views().map_err(|err| OpStoreError::Other(err.into()))?;
359
360        Ok(())
361    }
362}
363
364fn io_to_read_error(err: PathError, id: &impl ObjectId) -> OpStoreError {
365    if err.source.kind() == ErrorKind::NotFound {
366        OpStoreError::ObjectNotFound {
367            object_type: id.object_type(),
368            hash: id.hex(),
369            source: Box::new(err),
370        }
371    } else {
372        to_read_error(err.into(), id)
373    }
374}
375
376fn to_read_error(
377    source: Box<dyn std::error::Error + Send + Sync>,
378    id: &impl ObjectId,
379) -> OpStoreError {
380    OpStoreError::ReadObject {
381        object_type: id.object_type(),
382        hash: id.hex(),
383        source,
384    }
385}
386
387fn io_to_write_error(err: PathError, object_type: &'static str) -> OpStoreError {
388    OpStoreError::WriteObject {
389        object_type,
390        source: Box::new(err),
391    }
392}
393
394#[derive(Debug, Error)]
395enum PostDecodeError {
396    #[error("Invalid hash length (expected {expected} bytes, got {actual} bytes)")]
397    InvalidHashLength { expected: usize, actual: usize },
398    #[error("Invalid remote ref state value {0}")]
399    InvalidRemoteRefStateValue(i32),
400    #[error("Invalid number of ref target terms {0}")]
401    EvenNumberOfRefTargetTerms(usize),
402}
403
404fn operation_id_from_proto(bytes: Vec<u8>) -> Result<OperationId, PostDecodeError> {
405    if bytes.len() != OPERATION_ID_LENGTH {
406        Err(PostDecodeError::InvalidHashLength {
407            expected: OPERATION_ID_LENGTH,
408            actual: bytes.len(),
409        })
410    } else {
411        Ok(OperationId::new(bytes))
412    }
413}
414
415fn view_id_from_proto(bytes: Vec<u8>) -> Result<ViewId, PostDecodeError> {
416    if bytes.len() != VIEW_ID_LENGTH {
417        Err(PostDecodeError::InvalidHashLength {
418            expected: VIEW_ID_LENGTH,
419            actual: bytes.len(),
420        })
421    } else {
422        Ok(ViewId::new(bytes))
423    }
424}
425
426fn timestamp_to_proto(timestamp: &Timestamp) -> crate::protos::simple_op_store::Timestamp {
427    crate::protos::simple_op_store::Timestamp {
428        millis_since_epoch: timestamp.timestamp.0,
429        tz_offset: timestamp.tz_offset,
430    }
431}
432
433fn timestamp_from_proto(proto: crate::protos::simple_op_store::Timestamp) -> Timestamp {
434    Timestamp {
435        timestamp: MillisSinceEpoch(proto.millis_since_epoch),
436        tz_offset: proto.tz_offset,
437    }
438}
439
440fn operation_metadata_to_proto(
441    metadata: &OperationMetadata,
442) -> crate::protos::simple_op_store::OperationMetadata {
443    crate::protos::simple_op_store::OperationMetadata {
444        start_time: Some(timestamp_to_proto(&metadata.time.start)),
445        end_time: Some(timestamp_to_proto(&metadata.time.end)),
446        description: metadata.description.clone(),
447        hostname: metadata.hostname.clone(),
448        username: metadata.username.clone(),
449        is_snapshot: metadata.is_snapshot,
450        workspace_name: metadata.workspace_name.clone().map(Into::into),
451        tags: metadata.tags.clone(),
452    }
453}
454
455fn operation_metadata_from_proto(
456    proto: crate::protos::simple_op_store::OperationMetadata,
457) -> OperationMetadata {
458    let time = TimestampRange {
459        start: timestamp_from_proto(proto.start_time.unwrap_or_default()),
460        end: timestamp_from_proto(proto.end_time.unwrap_or_default()),
461    };
462    let workspace_name = proto.workspace_name.map(Into::into);
463    OperationMetadata {
464        time,
465        description: proto.description,
466        hostname: proto.hostname,
467        username: proto.username,
468        is_snapshot: proto.is_snapshot,
469        workspace_name,
470        tags: proto.tags,
471    }
472}
473
474fn commit_predecessors_map_to_proto(
475    map: &BTreeMap<CommitId, Vec<CommitId>>,
476) -> Vec<crate::protos::simple_op_store::CommitPredecessors> {
477    map.iter()
478        .map(
479            |(commit_id, predecessor_ids)| crate::protos::simple_op_store::CommitPredecessors {
480                commit_id: commit_id.to_bytes(),
481                predecessor_ids: predecessor_ids.iter().map(|id| id.to_bytes()).collect(),
482            },
483        )
484        .collect()
485}
486
487fn commit_predecessors_map_from_proto(
488    proto: Vec<crate::protos::simple_op_store::CommitPredecessors>,
489) -> BTreeMap<CommitId, Vec<CommitId>> {
490    proto
491        .into_iter()
492        .map(|entry| {
493            let commit_id = CommitId::new(entry.commit_id);
494            let predecessor_ids = entry
495                .predecessor_ids
496                .into_iter()
497                .map(CommitId::new)
498                .collect();
499            (commit_id, predecessor_ids)
500        })
501        .collect()
502}
503
504fn operation_to_proto(operation: &Operation) -> crate::protos::simple_op_store::Operation {
505    let (commit_predecessors, stores_commit_predecessors) = match &operation.commit_predecessors {
506        Some(map) => (commit_predecessors_map_to_proto(map), true),
507        None => (vec![], false),
508    };
509    let parents = operation.parents.iter().map(|id| id.to_bytes()).collect();
510    crate::protos::simple_op_store::Operation {
511        view_id: operation.view_id.as_bytes().to_vec(),
512        parents,
513        metadata: Some(operation_metadata_to_proto(&operation.metadata)),
514        commit_predecessors,
515        stores_commit_predecessors,
516    }
517}
518
519fn operation_from_proto(
520    proto: crate::protos::simple_op_store::Operation,
521) -> Result<Operation, PostDecodeError> {
522    let parents = proto
523        .parents
524        .into_iter()
525        .map(operation_id_from_proto)
526        .try_collect()?;
527    let view_id = view_id_from_proto(proto.view_id)?;
528    let metadata = operation_metadata_from_proto(proto.metadata.unwrap_or_default());
529    let commit_predecessors = proto
530        .stores_commit_predecessors
531        .then(|| commit_predecessors_map_from_proto(proto.commit_predecessors));
532    Ok(Operation {
533        view_id,
534        parents,
535        metadata,
536        commit_predecessors,
537    })
538}
539
540fn view_to_proto(view: &View) -> crate::protos::simple_op_store::View {
541    let wc_commit_ids = view
542        .wc_commit_ids
543        .iter()
544        .map(|(name, id)| (name.into(), id.to_bytes()))
545        .collect();
546    let head_ids = view.head_ids.iter().map(|id| id.to_bytes()).collect();
547
548    let bookmarks = bookmark_views_to_proto_legacy(&view.local_bookmarks, &view.remote_views);
549
550    let local_tags = view
551        .local_tags
552        .iter()
553        .map(|(name, target)| crate::protos::simple_op_store::Tag {
554            name: name.into(),
555            target: ref_target_to_proto(target),
556        })
557        .collect();
558
559    let remote_views = remote_views_to_proto(&view.remote_views);
560
561    let git_refs = view
562        .git_refs
563        .iter()
564        .map(|(name, target)| {
565            #[expect(deprecated)]
566            crate::protos::simple_op_store::GitRef {
567                name: name.into(),
568                commit_id: Default::default(),
569                target: ref_target_to_proto(target),
570            }
571        })
572        .collect();
573
574    let git_head = ref_target_to_proto(&view.git_head);
575
576    #[expect(deprecated)]
577    crate::protos::simple_op_store::View {
578        head_ids,
579        wc_commit_id: Default::default(),
580        wc_commit_ids,
581        bookmarks,
582        local_tags,
583        remote_views,
584        git_refs,
585        git_head_legacy: Default::default(),
586        git_head,
587        // New/loaded view should have been migrated to the latest format
588        has_git_refs_migrated_to_remote_tags: true,
589    }
590}
591
592fn view_from_proto(proto: crate::protos::simple_op_store::View) -> Result<View, PostDecodeError> {
593    // TODO: validate commit id length?
594    // For compatibility with old repos before we had support for multiple working
595    // copies
596    let mut wc_commit_ids = BTreeMap::new();
597    #[expect(deprecated)]
598    if !proto.wc_commit_id.is_empty() {
599        wc_commit_ids.insert(
600            WorkspaceName::DEFAULT.to_owned(),
601            CommitId::new(proto.wc_commit_id),
602        );
603    }
604    for (name, commit_id) in proto.wc_commit_ids {
605        wc_commit_ids.insert(WorkspaceNameBuf::from(name), CommitId::new(commit_id));
606    }
607    let head_ids = proto.head_ids.into_iter().map(CommitId::new).collect();
608
609    let (local_bookmarks, mut remote_views) = bookmark_views_from_proto_legacy(proto.bookmarks)?;
610
611    let local_tags = proto
612        .local_tags
613        .into_iter()
614        .map(|tag_proto| {
615            let name: RefNameBuf = tag_proto.name.into();
616            (name, ref_target_from_proto(tag_proto.target))
617        })
618        .collect();
619
620    let git_refs: BTreeMap<_, _> = proto
621        .git_refs
622        .into_iter()
623        .map(|git_ref| {
624            let name: GitRefNameBuf = git_ref.name.into();
625            let target = if git_ref.target.is_some() {
626                ref_target_from_proto(git_ref.target)
627            } else {
628                // Legacy format
629                #[expect(deprecated)]
630                RefTarget::normal(CommitId::new(git_ref.commit_id))
631            };
632            (name, target)
633        })
634        .collect();
635
636    // Use legacy remote_views only when new data isn't available (jj < 0.34)
637    if !proto.remote_views.is_empty() {
638        remote_views = remote_views_from_proto(proto.remote_views)?;
639    }
640
641    #[cfg(feature = "git")]
642    if !proto.has_git_refs_migrated_to_remote_tags {
643        tracing::info!("migrating Git-tracking tags");
644        let git_tags: BTreeMap<_, _> = git_refs
645            .iter()
646            .filter_map(|(full_name, target)| {
647                let name = full_name.as_str().strip_prefix("refs/tags/")?;
648                assert!(!name.is_empty());
649                let name: RefNameBuf = name.into();
650                let remote_ref = RemoteRef {
651                    target: target.clone(),
652                    state: RemoteRefState::Tracked,
653                };
654                Some((name, remote_ref))
655            })
656            .collect();
657        if !git_tags.is_empty() {
658            let git_view = remote_views
659                .entry(crate::git::REMOTE_NAME_FOR_LOCAL_GIT_REPO.to_owned())
660                .or_default();
661            assert!(git_view.tags.is_empty());
662            git_view.tags = git_tags;
663        }
664    }
665
666    #[expect(deprecated)]
667    let git_head = if proto.git_head.is_some() {
668        ref_target_from_proto(proto.git_head)
669    } else if !proto.git_head_legacy.is_empty() {
670        RefTarget::normal(CommitId::new(proto.git_head_legacy))
671    } else {
672        RefTarget::absent()
673    };
674
675    Ok(View {
676        head_ids,
677        local_bookmarks,
678        local_tags,
679        remote_views,
680        git_refs,
681        git_head,
682        wc_commit_ids,
683    })
684}
685
686fn bookmark_views_to_proto_legacy(
687    local_bookmarks: &BTreeMap<RefNameBuf, RefTarget>,
688    remote_views: &BTreeMap<RemoteNameBuf, RemoteView>,
689) -> Vec<crate::protos::simple_op_store::Bookmark> {
690    op_store::merge_join_ref_views(local_bookmarks, remote_views, |view| &view.bookmarks)
691        .map(|(name, bookmark_target)| {
692            let local_target = ref_target_to_proto(bookmark_target.local_target);
693            // TODO: Drop serialization to the old format in jj 0.40 or so.
694            let remote_bookmarks = bookmark_target
695                .remote_refs
696                .iter()
697                .map(
698                    |&(remote_name, remote_ref)| crate::protos::simple_op_store::RemoteBookmark {
699                        remote_name: remote_name.into(),
700                        target: ref_target_to_proto(&remote_ref.target),
701                        state: Some(remote_ref_state_to_proto(remote_ref.state)),
702                    },
703                )
704                .collect();
705            #[expect(deprecated)]
706            crate::protos::simple_op_store::Bookmark {
707                name: name.into(),
708                local_target,
709                remote_bookmarks,
710            }
711        })
712        .collect()
713}
714
715type BookmarkViews = (
716    BTreeMap<RefNameBuf, RefTarget>,
717    BTreeMap<RemoteNameBuf, RemoteView>,
718);
719
720fn bookmark_views_from_proto_legacy(
721    bookmarks_legacy: Vec<crate::protos::simple_op_store::Bookmark>,
722) -> Result<BookmarkViews, PostDecodeError> {
723    let mut local_bookmarks: BTreeMap<RefNameBuf, RefTarget> = BTreeMap::new();
724    let mut remote_views: BTreeMap<RemoteNameBuf, RemoteView> = BTreeMap::new();
725    for bookmark_proto in bookmarks_legacy {
726        let bookmark_name: RefNameBuf = bookmark_proto.name.into();
727        let local_target = ref_target_from_proto(bookmark_proto.local_target);
728        #[expect(deprecated)]
729        let remote_bookmarks = bookmark_proto.remote_bookmarks;
730        for remote_bookmark in remote_bookmarks {
731            let remote_name: RemoteNameBuf = remote_bookmark.remote_name.into();
732            let state = match remote_bookmark.state {
733                Some(n) => remote_ref_state_from_proto(n)?,
734                // Legacy view saved by jj < 0.11. The proto field is not
735                // changed to non-optional type because that would break forward
736                // compatibility. Zero may be omitted if the field is optional.
737                None => RemoteRefState::New,
738            };
739            let remote_view = remote_views.entry(remote_name).or_default();
740            let remote_ref = RemoteRef {
741                target: ref_target_from_proto(remote_bookmark.target),
742                state,
743            };
744            remote_view
745                .bookmarks
746                .insert(bookmark_name.clone(), remote_ref);
747        }
748        if local_target.is_present() {
749            local_bookmarks.insert(bookmark_name, local_target);
750        }
751    }
752    Ok((local_bookmarks, remote_views))
753}
754
755fn remote_views_to_proto(
756    remote_views: &BTreeMap<RemoteNameBuf, RemoteView>,
757) -> Vec<crate::protos::simple_op_store::RemoteView> {
758    remote_views
759        .iter()
760        .map(|(name, view)| crate::protos::simple_op_store::RemoteView {
761            name: name.into(),
762            bookmarks: remote_refs_to_proto(&view.bookmarks),
763            tags: remote_refs_to_proto(&view.tags),
764        })
765        .collect()
766}
767
768fn remote_views_from_proto(
769    remote_views_proto: Vec<crate::protos::simple_op_store::RemoteView>,
770) -> Result<BTreeMap<RemoteNameBuf, RemoteView>, PostDecodeError> {
771    remote_views_proto
772        .into_iter()
773        .map(|proto| {
774            let name: RemoteNameBuf = proto.name.into();
775            let view = RemoteView {
776                bookmarks: remote_refs_from_proto(proto.bookmarks)?,
777                tags: remote_refs_from_proto(proto.tags)?,
778            };
779            Ok((name, view))
780        })
781        .collect()
782}
783
784fn remote_refs_to_proto(
785    remote_refs: &BTreeMap<RefNameBuf, RemoteRef>,
786) -> Vec<crate::protos::simple_op_store::RemoteRef> {
787    remote_refs
788        .iter()
789        .map(
790            |(name, remote_ref)| crate::protos::simple_op_store::RemoteRef {
791                name: name.into(),
792                target_terms: ref_target_to_terms_proto(&remote_ref.target),
793                state: remote_ref_state_to_proto(remote_ref.state),
794            },
795        )
796        .collect()
797}
798
799fn remote_refs_from_proto(
800    remote_refs_proto: Vec<crate::protos::simple_op_store::RemoteRef>,
801) -> Result<BTreeMap<RefNameBuf, RemoteRef>, PostDecodeError> {
802    remote_refs_proto
803        .into_iter()
804        .map(|proto| {
805            let name: RefNameBuf = proto.name.into();
806            let remote_ref = RemoteRef {
807                target: ref_target_from_terms_proto(proto.target_terms)?,
808                state: remote_ref_state_from_proto(proto.state)?,
809            };
810            Ok((name, remote_ref))
811        })
812        .collect()
813}
814
815fn ref_target_to_terms_proto(
816    value: &RefTarget,
817) -> Vec<crate::protos::simple_op_store::RefTargetTerm> {
818    value
819        .as_merge()
820        .iter()
821        .map(|term| term.as_ref().map(|id| id.to_bytes()))
822        .map(|value| crate::protos::simple_op_store::RefTargetTerm { value })
823        .collect()
824}
825
826fn ref_target_from_terms_proto(
827    proto: Vec<crate::protos::simple_op_store::RefTargetTerm>,
828) -> Result<RefTarget, PostDecodeError> {
829    let terms: SmallVec<[_; 1]> = proto
830        .into_iter()
831        .map(|crate::protos::simple_op_store::RefTargetTerm { value }| value.map(CommitId::new))
832        .collect();
833    if terms.len().is_multiple_of(2) {
834        Err(PostDecodeError::EvenNumberOfRefTargetTerms(terms.len()))
835    } else {
836        Ok(RefTarget::from_merge(Merge::from_vec(terms)))
837    }
838}
839
840fn ref_target_to_proto(value: &RefTarget) -> Option<crate::protos::simple_op_store::RefTarget> {
841    let term_to_proto =
842        |term: &Option<CommitId>| crate::protos::simple_op_store::ref_conflict::Term {
843            value: term.as_ref().map(|id| id.to_bytes()),
844        };
845    let merge = value.as_merge();
846    let conflict_proto = crate::protos::simple_op_store::RefConflict {
847        removes: merge.removes().map(term_to_proto).collect(),
848        adds: merge.adds().map(term_to_proto).collect(),
849    };
850    let proto = crate::protos::simple_op_store::RefTarget {
851        value: Some(crate::protos::simple_op_store::ref_target::Value::Conflict(
852            conflict_proto,
853        )),
854    };
855    Some(proto)
856}
857
858#[expect(deprecated)]
859#[cfg(test)]
860fn ref_target_to_proto_legacy(
861    value: &RefTarget,
862) -> Option<crate::protos::simple_op_store::RefTarget> {
863    if let Some(id) = value.as_normal() {
864        let proto = crate::protos::simple_op_store::RefTarget {
865            value: Some(crate::protos::simple_op_store::ref_target::Value::CommitId(
866                id.to_bytes(),
867            )),
868        };
869        Some(proto)
870    } else if value.has_conflict() {
871        let ref_conflict_proto = crate::protos::simple_op_store::RefConflictLegacy {
872            removes: value.removed_ids().map(|id| id.to_bytes()).collect(),
873            adds: value.added_ids().map(|id| id.to_bytes()).collect(),
874        };
875        let proto = crate::protos::simple_op_store::RefTarget {
876            value: Some(
877                crate::protos::simple_op_store::ref_target::Value::ConflictLegacy(
878                    ref_conflict_proto,
879                ),
880            ),
881        };
882        Some(proto)
883    } else {
884        assert!(value.is_absent());
885        None
886    }
887}
888
889fn ref_target_from_proto(
890    maybe_proto: Option<crate::protos::simple_op_store::RefTarget>,
891) -> RefTarget {
892    // TODO: Delete legacy format handling when we decide to drop support for views
893    // saved by jj <= 0.8.
894    let Some(proto) = maybe_proto else {
895        // Legacy absent id
896        return RefTarget::absent();
897    };
898    match proto.value.unwrap() {
899        #[expect(deprecated)]
900        crate::protos::simple_op_store::ref_target::Value::CommitId(id) => {
901            // Legacy non-conflicting id
902            RefTarget::normal(CommitId::new(id))
903        }
904        #[expect(deprecated)]
905        crate::protos::simple_op_store::ref_target::Value::ConflictLegacy(conflict) => {
906            // Legacy conflicting ids
907            let removes = conflict.removes.into_iter().map(CommitId::new);
908            let adds = conflict.adds.into_iter().map(CommitId::new);
909            RefTarget::from_legacy_form(removes, adds)
910        }
911        crate::protos::simple_op_store::ref_target::Value::Conflict(conflict) => {
912            let term_from_proto = |term: crate::protos::simple_op_store::ref_conflict::Term| {
913                term.value.map(CommitId::new)
914            };
915            let removes = conflict.removes.into_iter().map(term_from_proto);
916            let adds = conflict.adds.into_iter().map(term_from_proto);
917            RefTarget::from_merge(Merge::from_removes_adds(removes, adds))
918        }
919    }
920}
921
922fn remote_ref_state_to_proto(state: RemoteRefState) -> i32 {
923    let proto_state = match state {
924        RemoteRefState::New => crate::protos::simple_op_store::RemoteRefState::New,
925        RemoteRefState::Tracked => crate::protos::simple_op_store::RemoteRefState::Tracked,
926    };
927    proto_state as i32
928}
929
930fn remote_ref_state_from_proto(proto_value: i32) -> Result<RemoteRefState, PostDecodeError> {
931    let proto_state = proto_value
932        .try_into()
933        .map_err(|prost::UnknownEnumValue(n)| PostDecodeError::InvalidRemoteRefStateValue(n))?;
934    let state = match proto_state {
935        crate::protos::simple_op_store::RemoteRefState::New => RemoteRefState::New,
936        crate::protos::simple_op_store::RemoteRefState::Tracked => RemoteRefState::Tracked,
937    };
938    Ok(state)
939}
940
941#[cfg(test)]
942mod tests {
943    use insta::assert_snapshot;
944    use itertools::Itertools as _;
945    use maplit::btreemap;
946    use maplit::hashmap;
947    use maplit::hashset;
948
949    use super::*;
950    use crate::hex_util;
951    use crate::tests::TestResult;
952    use crate::tests::new_temp_dir;
953
954    fn create_view() -> View {
955        let new_remote_ref = |target: &RefTarget| RemoteRef {
956            target: target.clone(),
957            state: RemoteRefState::New,
958        };
959        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
960            target: target.clone(),
961            state: RemoteRefState::Tracked,
962        };
963        let head_id1 = CommitId::from_hex("aaa111");
964        let head_id2 = CommitId::from_hex("aaa222");
965        let bookmark_main_local_target = RefTarget::normal(CommitId::from_hex("ccc111"));
966        let bookmark_main_origin_target = RefTarget::normal(CommitId::from_hex("ccc222"));
967        let bookmark_deleted_origin_target = RefTarget::normal(CommitId::from_hex("ccc333"));
968        let tag_v1_local_target = RefTarget::normal(CommitId::from_hex("ddd111"));
969        let tag_v1_origin_target = RefTarget::normal(CommitId::from_hex("ddd222"));
970        let tag_deleted_origin_target = RefTarget::normal(CommitId::from_hex("ddd333"));
971        let git_refs_main_target = RefTarget::normal(CommitId::from_hex("fff111"));
972        let git_refs_feature_target = RefTarget::from_legacy_form(
973            [CommitId::from_hex("fff111")],
974            [CommitId::from_hex("fff222"), CommitId::from_hex("fff333")],
975        );
976        let default_wc_commit_id = CommitId::from_hex("abc111");
977        let test_wc_commit_id = CommitId::from_hex("abc222");
978        View {
979            head_ids: hashset! {head_id1, head_id2},
980            local_bookmarks: btreemap! {
981                "main".into() => bookmark_main_local_target,
982            },
983            local_tags: btreemap! {
984                "v1.0".into() => tag_v1_local_target,
985            },
986            remote_views: btreemap! {
987                "origin".into() => RemoteView {
988                    bookmarks: btreemap! {
989                        "main".into() => tracked_remote_ref(&bookmark_main_origin_target),
990                        "deleted".into() => new_remote_ref(&bookmark_deleted_origin_target),
991                    },
992                    tags: btreemap! {
993                        "v1.0".into() => tracked_remote_ref(&tag_v1_origin_target),
994                        "deleted".into() => new_remote_ref(&tag_deleted_origin_target),
995                    },
996                },
997            },
998            git_refs: btreemap! {
999                "refs/heads/main".into() => git_refs_main_target,
1000                "refs/heads/feature".into() => git_refs_feature_target,
1001            },
1002            git_head: RefTarget::normal(CommitId::from_hex("fff111")),
1003            wc_commit_ids: btreemap! {
1004                WorkspaceName::DEFAULT.to_owned() => default_wc_commit_id,
1005                "test".into() => test_wc_commit_id,
1006            },
1007        }
1008    }
1009
1010    fn create_operation() -> Operation {
1011        let pad_id_bytes = |hex: &str, len: usize| {
1012            let mut bytes = hex_util::decode_hex(hex).unwrap();
1013            bytes.resize(len, b'\0');
1014            bytes
1015        };
1016        Operation {
1017            view_id: ViewId::new(pad_id_bytes("aaa111", VIEW_ID_LENGTH)),
1018            parents: vec![
1019                OperationId::new(pad_id_bytes("bbb111", OPERATION_ID_LENGTH)),
1020                OperationId::new(pad_id_bytes("bbb222", OPERATION_ID_LENGTH)),
1021            ],
1022            metadata: OperationMetadata {
1023                time: TimestampRange {
1024                    start: Timestamp {
1025                        timestamp: MillisSinceEpoch(123456789),
1026                        tz_offset: 3600,
1027                    },
1028                    end: Timestamp {
1029                        timestamp: MillisSinceEpoch(123456800),
1030                        tz_offset: 3600,
1031                    },
1032                },
1033                description: "check out foo".to_string(),
1034                hostname: "some.host.example.com".to_string(),
1035                username: "someone".to_string(),
1036                is_snapshot: false,
1037                workspace_name: Some(WorkspaceNameBuf::from("test")),
1038                tags: hashmap! {
1039                    "key1".to_string() => "value1".to_string(),
1040                    "key2".to_string() => "value2".to_string(),
1041                },
1042            },
1043            commit_predecessors: Some(btreemap! {
1044                CommitId::from_hex("111111") => vec![],
1045                CommitId::from_hex("222222") => vec![
1046                    CommitId::from_hex("333333"),
1047                    CommitId::from_hex("444444"),
1048                ],
1049            }),
1050        }
1051    }
1052
1053    #[test]
1054    fn test_hash_view() {
1055        // Test exact output so we detect regressions in compatibility
1056        assert_snapshot!(
1057            ViewId::new(blake2b_hash(&create_view()).to_vec()).hex(),
1058            @"2c0b174d117ca85e7faa96f6d997362403105e8eb31e7f82ac9abd3dc48ae62683e9a76ef5d117ebc8a743d17e1945236df9ccefd7574f7e4b5336a63796b967"
1059        );
1060    }
1061
1062    #[test]
1063    fn test_hash_operation() {
1064        // Test exact output so we detect regressions in compatibility
1065        assert_snapshot!(
1066            OperationId::new(blake2b_hash(&create_operation()).to_vec()).hex(),
1067            @"f5963c593a63bb852061a86ad919c12c6ba1940eeef30a832524c39ccea6a9f768aa2aa53becec34d379eb291ec6726837c4113857849cb9dcc62dbe0a517176"
1068        );
1069    }
1070
1071    #[test]
1072    fn test_read_write_view() -> TestResult {
1073        let temp_dir = new_temp_dir();
1074        let root_data = RootOperationData {
1075            root_commit_id: CommitId::from_hex("000000"),
1076        };
1077        let store = SimpleOpStore::init(temp_dir.path(), root_data)?;
1078        let view = create_view();
1079        let view_id = store.write_view(&view).block_on()?;
1080        let read_view = store.read_view(&view_id).block_on()?;
1081        assert_eq!(read_view, view);
1082        Ok(())
1083    }
1084
1085    #[test]
1086    fn test_read_write_operation() -> TestResult {
1087        let temp_dir = new_temp_dir();
1088        let root_data = RootOperationData {
1089            root_commit_id: CommitId::from_hex("000000"),
1090        };
1091        let store = SimpleOpStore::init(temp_dir.path(), root_data)?;
1092        let operation = create_operation();
1093        let op_id = store.write_operation(&operation).block_on()?;
1094        let read_operation = store.read_operation(&op_id).block_on()?;
1095        assert_eq!(read_operation, operation);
1096        Ok(())
1097    }
1098
1099    #[test]
1100    fn test_remote_views_legacy_roundtrip() {
1101        let mut view = create_view();
1102        assert!(!view.remote_views.is_empty());
1103        for remote_view in view.remote_views.values_mut() {
1104            // remote tags cannot be preserved in "legacy" format
1105            remote_view.tags.clear();
1106        }
1107        let mut proto = view_to_proto(&view);
1108        proto.remote_views.clear(); // drop "new" format
1109        let view_reconstructed = view_from_proto(proto).unwrap();
1110        assert_eq!(view.remote_views, view_reconstructed.remote_views);
1111    }
1112
1113    #[test]
1114    fn test_remote_views_new_roundtrip() {
1115        let view = create_view();
1116        assert!(!view.remote_views.is_empty());
1117        let mut proto = view_to_proto(&view);
1118        for bookmark in &mut proto.bookmarks {
1119            #[expect(deprecated)]
1120            bookmark.remote_bookmarks.clear(); // drop "legacy" format
1121        }
1122        let view_reconstructed = view_from_proto(proto).unwrap();
1123        assert_eq!(view.remote_views, view_reconstructed.remote_views);
1124    }
1125
1126    #[test]
1127    fn test_migrate_git_refs_to_remote_tags() {
1128        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
1129            target: target.clone(),
1130            state: RemoteRefState::Tracked,
1131        };
1132        let git_ref_to_proto = |name: &str, ref_target| crate::protos::simple_op_store::GitRef {
1133            name: name.to_owned(),
1134            #[expect(deprecated)]
1135            commit_id: Default::default(),
1136            target: ref_target_to_proto(ref_target),
1137        };
1138        let v1_target = RefTarget::normal(CommitId::from_hex("111111"));
1139        let main_target = RefTarget::normal(CommitId::from_hex("222222"));
1140        let orig_remote_views = btreemap! {
1141            "git".into() => RemoteView {
1142                bookmarks: btreemap! {
1143                    "main".into() => tracked_remote_ref(&main_target),
1144                },
1145                tags: btreemap! {},
1146            },
1147        };
1148        let proto = crate::protos::simple_op_store::View {
1149            remote_views: remote_views_to_proto(&orig_remote_views),
1150            git_refs: vec![
1151                git_ref_to_proto("refs/tags/v1.0", &v1_target),
1152                git_ref_to_proto("refs/heads/main", &main_target),
1153            ],
1154            has_git_refs_migrated_to_remote_tags: false,
1155            ..Default::default()
1156        };
1157
1158        let view = view_from_proto(proto).unwrap();
1159        if cfg!(feature = "git") {
1160            assert_eq!(
1161                view.remote_views,
1162                btreemap! {
1163                    "git".into() => RemoteView {
1164                        bookmarks: btreemap! {
1165                            "main".into() => tracked_remote_ref(&main_target),
1166                        },
1167                        tags: btreemap! {
1168                            "v1.0".into() => tracked_remote_ref(&v1_target),
1169                        },
1170                    },
1171                }
1172            );
1173        } else {
1174            assert_eq!(view.remote_views, orig_remote_views);
1175        }
1176
1177        // Once migrated, "git" remote tags shouldn't be populated again.
1178        let mut proto = view_to_proto(&view);
1179        assert!(proto.has_git_refs_migrated_to_remote_tags);
1180        for view_proto in &mut proto.remote_views {
1181            view_proto.tags.clear();
1182        }
1183        let view = view_from_proto(proto).unwrap();
1184        assert_eq!(view.remote_views, orig_remote_views);
1185    }
1186
1187    #[test]
1188    fn test_bookmark_views_legacy_roundtrip() {
1189        let new_remote_ref = |target: &RefTarget| RemoteRef {
1190            target: target.clone(),
1191            state: RemoteRefState::New,
1192        };
1193        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
1194            target: target.clone(),
1195            state: RemoteRefState::Tracked,
1196        };
1197        let local_bookmark1_target = RefTarget::normal(CommitId::from_hex("111111"));
1198        let local_bookmark3_target = RefTarget::normal(CommitId::from_hex("222222"));
1199        let git_bookmark1_target = RefTarget::normal(CommitId::from_hex("333333"));
1200        let remote1_bookmark1_target = RefTarget::normal(CommitId::from_hex("444444"));
1201        let remote2_bookmark2_target = RefTarget::normal(CommitId::from_hex("555555"));
1202        let remote2_bookmark4_target = RefTarget::normal(CommitId::from_hex("666666"));
1203        let local_bookmarks = btreemap! {
1204            "bookmark1".into() => local_bookmark1_target.clone(),
1205            "bookmark3".into() => local_bookmark3_target.clone(),
1206        };
1207        let remote_views = btreemap! {
1208            "git".into() => RemoteView {
1209                bookmarks: btreemap! {
1210                    "bookmark1".into() => tracked_remote_ref(&git_bookmark1_target),
1211                },
1212                tags: btreemap! {},
1213            },
1214            "remote1".into() => RemoteView {
1215                bookmarks: btreemap! {
1216                    "bookmark1".into() => tracked_remote_ref(&remote1_bookmark1_target),
1217                },
1218                tags: btreemap! {},
1219            },
1220            "remote2".into() => RemoteView {
1221                bookmarks: btreemap! {
1222                    // "bookmark2" is non-tracking. "bookmark4" is tracking, but locally deleted.
1223                    "bookmark2".into() => new_remote_ref(&remote2_bookmark2_target),
1224                    "bookmark4".into() => tracked_remote_ref(&remote2_bookmark4_target),
1225                },
1226                tags: btreemap! {},
1227            },
1228        };
1229
1230        let bookmarks_legacy = bookmark_views_to_proto_legacy(&local_bookmarks, &remote_views);
1231        assert_eq!(
1232            bookmarks_legacy
1233                .iter()
1234                .map(|proto| &proto.name)
1235                .sorted()
1236                .collect_vec(),
1237            vec!["bookmark1", "bookmark2", "bookmark3", "bookmark4"],
1238        );
1239
1240        let (local_bookmarks_reconstructed, remote_views_reconstructed) =
1241            bookmark_views_from_proto_legacy(bookmarks_legacy).unwrap();
1242        assert_eq!(local_bookmarks_reconstructed, local_bookmarks);
1243        assert_eq!(remote_views_reconstructed, remote_views);
1244    }
1245
1246    #[test]
1247    fn test_ref_target_change_delete_order_roundtrip() {
1248        let target = RefTarget::from_merge(Merge::from_removes_adds(
1249            vec![Some(CommitId::from_hex("111111"))],
1250            vec![Some(CommitId::from_hex("222222")), None],
1251        ));
1252        let maybe_proto = ref_target_to_proto(&target);
1253        assert_eq!(ref_target_from_proto(maybe_proto), target);
1254
1255        // If it were legacy format, order of None entry would be lost.
1256        let target = RefTarget::from_merge(Merge::from_removes_adds(
1257            vec![Some(CommitId::from_hex("111111"))],
1258            vec![None, Some(CommitId::from_hex("222222"))],
1259        ));
1260        let maybe_proto = ref_target_to_proto(&target);
1261        assert_eq!(ref_target_from_proto(maybe_proto), target);
1262    }
1263
1264    #[test]
1265    fn test_ref_target_legacy_roundtrip() {
1266        let target = RefTarget::absent();
1267        let maybe_proto = ref_target_to_proto_legacy(&target);
1268        assert_eq!(ref_target_from_proto(maybe_proto), target);
1269
1270        let target = RefTarget::normal(CommitId::from_hex("111111"));
1271        let maybe_proto = ref_target_to_proto_legacy(&target);
1272        assert_eq!(ref_target_from_proto(maybe_proto), target);
1273
1274        // N-way conflict
1275        let target = RefTarget::from_legacy_form(
1276            [CommitId::from_hex("111111"), CommitId::from_hex("222222")],
1277            [
1278                CommitId::from_hex("333333"),
1279                CommitId::from_hex("444444"),
1280                CommitId::from_hex("555555"),
1281            ],
1282        );
1283        let maybe_proto = ref_target_to_proto_legacy(&target);
1284        assert_eq!(ref_target_from_proto(maybe_proto), target);
1285
1286        // Change-delete conflict
1287        let target = RefTarget::from_legacy_form(
1288            [CommitId::from_hex("111111")],
1289            [CommitId::from_hex("222222")],
1290        );
1291        let maybe_proto = ref_target_to_proto_legacy(&target);
1292        assert_eq!(ref_target_from_proto(maybe_proto), target);
1293    }
1294}