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_async;
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_async::dfs(
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        attributes: metadata
452            .attributes
453            .iter()
454            .map(|(k, v)| (k.clone(), v.clone()))
455            .collect(),
456    }
457}
458
459fn operation_metadata_from_proto(
460    proto: crate::protos::simple_op_store::OperationMetadata,
461) -> OperationMetadata {
462    let time = TimestampRange {
463        start: timestamp_from_proto(proto.start_time.unwrap_or_default()),
464        end: timestamp_from_proto(proto.end_time.unwrap_or_default()),
465    };
466    let workspace_name = proto.workspace_name.map(Into::into);
467    OperationMetadata {
468        time,
469        description: proto.description,
470        hostname: proto.hostname,
471        username: proto.username,
472        is_snapshot: proto.is_snapshot,
473        workspace_name,
474        attributes: proto.attributes.into_iter().collect(),
475    }
476}
477
478fn commit_predecessors_map_to_proto(
479    map: &BTreeMap<CommitId, Vec<CommitId>>,
480) -> Vec<crate::protos::simple_op_store::CommitPredecessors> {
481    map.iter()
482        .map(
483            |(commit_id, predecessor_ids)| crate::protos::simple_op_store::CommitPredecessors {
484                commit_id: commit_id.to_bytes(),
485                predecessor_ids: predecessor_ids.iter().map(|id| id.to_bytes()).collect(),
486            },
487        )
488        .collect()
489}
490
491fn commit_predecessors_map_from_proto(
492    proto: Vec<crate::protos::simple_op_store::CommitPredecessors>,
493) -> BTreeMap<CommitId, Vec<CommitId>> {
494    proto
495        .into_iter()
496        .map(|entry| {
497            let commit_id = CommitId::new(entry.commit_id);
498            let predecessor_ids = entry
499                .predecessor_ids
500                .into_iter()
501                .map(CommitId::new)
502                .collect();
503            (commit_id, predecessor_ids)
504        })
505        .collect()
506}
507
508fn operation_to_proto(operation: &Operation) -> crate::protos::simple_op_store::Operation {
509    let (commit_predecessors, stores_commit_predecessors) = match &operation.commit_predecessors {
510        Some(map) => (commit_predecessors_map_to_proto(map), true),
511        None => (vec![], false),
512    };
513    let parents = operation.parents.iter().map(|id| id.to_bytes()).collect();
514    crate::protos::simple_op_store::Operation {
515        view_id: operation.view_id.as_bytes().to_vec(),
516        parents,
517        metadata: Some(operation_metadata_to_proto(&operation.metadata)),
518        commit_predecessors,
519        stores_commit_predecessors,
520    }
521}
522
523fn operation_from_proto(
524    proto: crate::protos::simple_op_store::Operation,
525) -> Result<Operation, PostDecodeError> {
526    let parents = proto
527        .parents
528        .into_iter()
529        .map(operation_id_from_proto)
530        .try_collect()?;
531    let view_id = view_id_from_proto(proto.view_id)?;
532    let metadata = operation_metadata_from_proto(proto.metadata.unwrap_or_default());
533    let commit_predecessors = proto
534        .stores_commit_predecessors
535        .then(|| commit_predecessors_map_from_proto(proto.commit_predecessors));
536    Ok(Operation {
537        view_id,
538        parents,
539        metadata,
540        commit_predecessors,
541    })
542}
543
544fn view_to_proto(view: &View) -> crate::protos::simple_op_store::View {
545    let wc_commit_ids = view
546        .wc_commit_ids
547        .iter()
548        .map(|(name, id)| (name.into(), id.to_bytes()))
549        .collect();
550    let head_ids = view.head_ids.iter().map(|id| id.to_bytes()).collect();
551
552    let bookmarks = bookmark_views_to_proto_legacy(&view.local_bookmarks, &view.remote_views);
553
554    let local_tags = view
555        .local_tags
556        .iter()
557        .map(|(name, target)| crate::protos::simple_op_store::Tag {
558            name: name.into(),
559            target: ref_target_to_proto(target),
560        })
561        .collect();
562
563    let remote_views = remote_views_to_proto(&view.remote_views);
564
565    let git_refs = view
566        .git_refs
567        .iter()
568        .map(|(name, target)| {
569            #[expect(deprecated)]
570            crate::protos::simple_op_store::GitRef {
571                name: name.into(),
572                commit_id: Default::default(),
573                target: ref_target_to_proto(target),
574            }
575        })
576        .collect();
577
578    let git_head = ref_target_to_proto(&view.git_head);
579
580    #[expect(deprecated)]
581    crate::protos::simple_op_store::View {
582        head_ids,
583        wc_commit_id: Default::default(),
584        wc_commit_ids,
585        bookmarks,
586        local_tags,
587        remote_views,
588        git_refs,
589        git_head_legacy: Default::default(),
590        git_head,
591        // New/loaded view should have been migrated to the latest format
592        has_git_refs_migrated_to_remote_tags: true,
593    }
594}
595
596fn view_from_proto(proto: crate::protos::simple_op_store::View) -> Result<View, PostDecodeError> {
597    // TODO: validate commit id length?
598    // For compatibility with old repos before we had support for multiple working
599    // copies
600    let mut wc_commit_ids = BTreeMap::new();
601    #[expect(deprecated)]
602    if !proto.wc_commit_id.is_empty() {
603        wc_commit_ids.insert(
604            WorkspaceName::DEFAULT.to_owned(),
605            CommitId::new(proto.wc_commit_id),
606        );
607    }
608    for (name, commit_id) in proto.wc_commit_ids {
609        wc_commit_ids.insert(WorkspaceNameBuf::from(name), CommitId::new(commit_id));
610    }
611    let head_ids = proto.head_ids.into_iter().map(CommitId::new).collect();
612
613    let (local_bookmarks, mut remote_views) = bookmark_views_from_proto_legacy(proto.bookmarks)?;
614
615    let local_tags = proto
616        .local_tags
617        .into_iter()
618        .map(|tag_proto| {
619            let name: RefNameBuf = tag_proto.name.into();
620            (name, ref_target_from_proto(tag_proto.target))
621        })
622        .collect();
623
624    let git_refs: BTreeMap<_, _> = proto
625        .git_refs
626        .into_iter()
627        .map(|git_ref| {
628            let name: GitRefNameBuf = git_ref.name.into();
629            let target = if git_ref.target.is_some() {
630                ref_target_from_proto(git_ref.target)
631            } else {
632                // Legacy format
633                #[expect(deprecated)]
634                RefTarget::normal(CommitId::new(git_ref.commit_id))
635            };
636            (name, target)
637        })
638        .collect();
639
640    // Use legacy remote_views only when new data isn't available (jj < 0.34)
641    if !proto.remote_views.is_empty() {
642        remote_views = remote_views_from_proto(proto.remote_views)?;
643    }
644
645    #[cfg(feature = "git")]
646    if !proto.has_git_refs_migrated_to_remote_tags {
647        tracing::info!("migrating Git-tracking tags");
648        let git_tags: BTreeMap<_, _> = git_refs
649            .iter()
650            .filter_map(|(full_name, target)| {
651                let name = full_name.as_str().strip_prefix("refs/tags/")?;
652                assert!(!name.is_empty());
653                let name: RefNameBuf = name.into();
654                let remote_ref = RemoteRef {
655                    target: target.clone(),
656                    state: RemoteRefState::Tracked,
657                };
658                Some((name, remote_ref))
659            })
660            .collect();
661        if !git_tags.is_empty() {
662            let git_view = remote_views
663                .entry(crate::git::REMOTE_NAME_FOR_LOCAL_GIT_REPO.to_owned())
664                .or_default();
665            assert!(git_view.tags.is_empty());
666            git_view.tags = git_tags;
667        }
668    }
669
670    #[expect(deprecated)]
671    let git_head = if proto.git_head.is_some() {
672        ref_target_from_proto(proto.git_head)
673    } else if !proto.git_head_legacy.is_empty() {
674        RefTarget::normal(CommitId::new(proto.git_head_legacy))
675    } else {
676        RefTarget::absent()
677    };
678
679    Ok(View {
680        head_ids,
681        local_bookmarks,
682        local_tags,
683        remote_views,
684        git_refs,
685        git_head,
686        wc_commit_ids,
687    })
688}
689
690fn bookmark_views_to_proto_legacy(
691    local_bookmarks: &BTreeMap<RefNameBuf, RefTarget>,
692    remote_views: &BTreeMap<RemoteNameBuf, RemoteView>,
693) -> Vec<crate::protos::simple_op_store::Bookmark> {
694    op_store::merge_join_ref_views(local_bookmarks, remote_views, |view| &view.bookmarks)
695        .map(|(name, bookmark_target)| {
696            let local_target = ref_target_to_proto(bookmark_target.local_target);
697            // TODO: Drop serialization to the old format in jj 0.40 or so.
698            let remote_bookmarks = bookmark_target
699                .remote_refs
700                .iter()
701                .map(
702                    |&(remote_name, remote_ref)| crate::protos::simple_op_store::RemoteBookmark {
703                        remote_name: remote_name.into(),
704                        target: ref_target_to_proto(&remote_ref.target),
705                        state: Some(remote_ref_state_to_proto(remote_ref.state)),
706                    },
707                )
708                .collect();
709            #[expect(deprecated)]
710            crate::protos::simple_op_store::Bookmark {
711                name: name.into(),
712                local_target,
713                remote_bookmarks,
714            }
715        })
716        .collect()
717}
718
719type BookmarkViews = (
720    BTreeMap<RefNameBuf, RefTarget>,
721    BTreeMap<RemoteNameBuf, RemoteView>,
722);
723
724fn bookmark_views_from_proto_legacy(
725    bookmarks_legacy: Vec<crate::protos::simple_op_store::Bookmark>,
726) -> Result<BookmarkViews, PostDecodeError> {
727    let mut local_bookmarks: BTreeMap<RefNameBuf, RefTarget> = BTreeMap::new();
728    let mut remote_views: BTreeMap<RemoteNameBuf, RemoteView> = BTreeMap::new();
729    for bookmark_proto in bookmarks_legacy {
730        let bookmark_name: RefNameBuf = bookmark_proto.name.into();
731        let local_target = ref_target_from_proto(bookmark_proto.local_target);
732        #[expect(deprecated)]
733        let remote_bookmarks = bookmark_proto.remote_bookmarks;
734        for remote_bookmark in remote_bookmarks {
735            let remote_name: RemoteNameBuf = remote_bookmark.remote_name.into();
736            let state = match remote_bookmark.state {
737                Some(n) => remote_ref_state_from_proto(n)?,
738                // Legacy view saved by jj < 0.11. The proto field is not
739                // changed to non-optional type because that would break forward
740                // compatibility. Zero may be omitted if the field is optional.
741                None => RemoteRefState::New,
742            };
743            let remote_view = remote_views.entry(remote_name).or_default();
744            let remote_ref = RemoteRef {
745                target: ref_target_from_proto(remote_bookmark.target),
746                state,
747            };
748            remote_view
749                .bookmarks
750                .insert(bookmark_name.clone(), remote_ref);
751        }
752        if local_target.is_present() {
753            local_bookmarks.insert(bookmark_name, local_target);
754        }
755    }
756    Ok((local_bookmarks, remote_views))
757}
758
759fn remote_views_to_proto(
760    remote_views: &BTreeMap<RemoteNameBuf, RemoteView>,
761) -> Vec<crate::protos::simple_op_store::RemoteView> {
762    remote_views
763        .iter()
764        .map(|(name, view)| crate::protos::simple_op_store::RemoteView {
765            name: name.into(),
766            bookmarks: remote_refs_to_proto(&view.bookmarks),
767            tags: remote_refs_to_proto(&view.tags),
768        })
769        .collect()
770}
771
772fn remote_views_from_proto(
773    remote_views_proto: Vec<crate::protos::simple_op_store::RemoteView>,
774) -> Result<BTreeMap<RemoteNameBuf, RemoteView>, PostDecodeError> {
775    remote_views_proto
776        .into_iter()
777        .map(|proto| {
778            let name: RemoteNameBuf = proto.name.into();
779            let view = RemoteView {
780                bookmarks: remote_refs_from_proto(proto.bookmarks)?,
781                tags: remote_refs_from_proto(proto.tags)?,
782            };
783            Ok((name, view))
784        })
785        .collect()
786}
787
788fn remote_refs_to_proto(
789    remote_refs: &BTreeMap<RefNameBuf, RemoteRef>,
790) -> Vec<crate::protos::simple_op_store::RemoteRef> {
791    remote_refs
792        .iter()
793        .map(
794            |(name, remote_ref)| crate::protos::simple_op_store::RemoteRef {
795                name: name.into(),
796                target_terms: ref_target_to_terms_proto(&remote_ref.target),
797                state: remote_ref_state_to_proto(remote_ref.state),
798            },
799        )
800        .collect()
801}
802
803fn remote_refs_from_proto(
804    remote_refs_proto: Vec<crate::protos::simple_op_store::RemoteRef>,
805) -> Result<BTreeMap<RefNameBuf, RemoteRef>, PostDecodeError> {
806    remote_refs_proto
807        .into_iter()
808        .map(|proto| {
809            let name: RefNameBuf = proto.name.into();
810            let remote_ref = RemoteRef {
811                target: ref_target_from_terms_proto(proto.target_terms)?,
812                state: remote_ref_state_from_proto(proto.state)?,
813            };
814            Ok((name, remote_ref))
815        })
816        .collect()
817}
818
819fn ref_target_to_terms_proto(
820    value: &RefTarget,
821) -> Vec<crate::protos::simple_op_store::RefTargetTerm> {
822    value
823        .as_merge()
824        .iter()
825        .map(|term| term.as_ref().map(|id| id.to_bytes()))
826        .map(|value| crate::protos::simple_op_store::RefTargetTerm { value })
827        .collect()
828}
829
830fn ref_target_from_terms_proto(
831    proto: Vec<crate::protos::simple_op_store::RefTargetTerm>,
832) -> Result<RefTarget, PostDecodeError> {
833    let terms: SmallVec<[_; 1]> = proto
834        .into_iter()
835        .map(|crate::protos::simple_op_store::RefTargetTerm { value }| value.map(CommitId::new))
836        .collect();
837    if terms.len().is_multiple_of(2) {
838        Err(PostDecodeError::EvenNumberOfRefTargetTerms(terms.len()))
839    } else {
840        Ok(RefTarget::from_merge(Merge::from_vec(terms)))
841    }
842}
843
844fn ref_target_to_proto(value: &RefTarget) -> Option<crate::protos::simple_op_store::RefTarget> {
845    let term_to_proto =
846        |term: &Option<CommitId>| crate::protos::simple_op_store::ref_conflict::Term {
847            value: term.as_ref().map(|id| id.to_bytes()),
848        };
849    let merge = value.as_merge();
850    let conflict_proto = crate::protos::simple_op_store::RefConflict {
851        removes: merge.removes().map(term_to_proto).collect(),
852        adds: merge.adds().map(term_to_proto).collect(),
853    };
854    let proto = crate::protos::simple_op_store::RefTarget {
855        value: Some(crate::protos::simple_op_store::ref_target::Value::Conflict(
856            conflict_proto,
857        )),
858    };
859    Some(proto)
860}
861
862#[expect(deprecated)]
863#[cfg(test)]
864fn ref_target_to_proto_legacy(
865    value: &RefTarget,
866) -> Option<crate::protos::simple_op_store::RefTarget> {
867    if let Some(id) = value.as_normal() {
868        let proto = crate::protos::simple_op_store::RefTarget {
869            value: Some(crate::protos::simple_op_store::ref_target::Value::CommitId(
870                id.to_bytes(),
871            )),
872        };
873        Some(proto)
874    } else if value.has_conflict() {
875        let ref_conflict_proto = crate::protos::simple_op_store::RefConflictLegacy {
876            removes: value.removed_ids().map(|id| id.to_bytes()).collect(),
877            adds: value.added_ids().map(|id| id.to_bytes()).collect(),
878        };
879        let proto = crate::protos::simple_op_store::RefTarget {
880            value: Some(
881                crate::protos::simple_op_store::ref_target::Value::ConflictLegacy(
882                    ref_conflict_proto,
883                ),
884            ),
885        };
886        Some(proto)
887    } else {
888        assert!(value.is_absent());
889        None
890    }
891}
892
893fn ref_target_from_proto(
894    maybe_proto: Option<crate::protos::simple_op_store::RefTarget>,
895) -> RefTarget {
896    // TODO: Delete legacy format handling when we decide to drop support for views
897    // saved by jj <= 0.8.
898    let Some(proto) = maybe_proto else {
899        // Legacy absent id
900        return RefTarget::absent();
901    };
902    match proto.value.unwrap() {
903        #[expect(deprecated)]
904        crate::protos::simple_op_store::ref_target::Value::CommitId(id) => {
905            // Legacy non-conflicting id
906            RefTarget::normal(CommitId::new(id))
907        }
908        #[expect(deprecated)]
909        crate::protos::simple_op_store::ref_target::Value::ConflictLegacy(conflict) => {
910            // Legacy conflicting ids
911            let removes = conflict.removes.into_iter().map(CommitId::new);
912            let adds = conflict.adds.into_iter().map(CommitId::new);
913            RefTarget::from_legacy_form(removes, adds)
914        }
915        crate::protos::simple_op_store::ref_target::Value::Conflict(conflict) => {
916            let term_from_proto = |term: crate::protos::simple_op_store::ref_conflict::Term| {
917                term.value.map(CommitId::new)
918            };
919            let removes = conflict.removes.into_iter().map(term_from_proto);
920            let adds = conflict.adds.into_iter().map(term_from_proto);
921            RefTarget::from_merge(Merge::from_removes_adds(removes, adds))
922        }
923    }
924}
925
926fn remote_ref_state_to_proto(state: RemoteRefState) -> i32 {
927    let proto_state = match state {
928        RemoteRefState::New => crate::protos::simple_op_store::RemoteRefState::New,
929        RemoteRefState::Tracked => crate::protos::simple_op_store::RemoteRefState::Tracked,
930    };
931    proto_state as i32
932}
933
934fn remote_ref_state_from_proto(proto_value: i32) -> Result<RemoteRefState, PostDecodeError> {
935    let proto_state = proto_value
936        .try_into()
937        .map_err(|prost::UnknownEnumValue(n)| PostDecodeError::InvalidRemoteRefStateValue(n))?;
938    let state = match proto_state {
939        crate::protos::simple_op_store::RemoteRefState::New => RemoteRefState::New,
940        crate::protos::simple_op_store::RemoteRefState::Tracked => RemoteRefState::Tracked,
941    };
942    Ok(state)
943}
944
945#[cfg(test)]
946mod tests {
947    use insta::assert_snapshot;
948    use itertools::Itertools as _;
949    use maplit::btreemap;
950    use maplit::hashset;
951
952    use super::*;
953    use crate::hex_util;
954    use crate::tests::TestResult;
955    use crate::tests::new_temp_dir;
956
957    fn create_view() -> View {
958        let new_remote_ref = |target: &RefTarget| RemoteRef {
959            target: target.clone(),
960            state: RemoteRefState::New,
961        };
962        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
963            target: target.clone(),
964            state: RemoteRefState::Tracked,
965        };
966        let head_id1 = CommitId::from_hex("aaa111");
967        let head_id2 = CommitId::from_hex("aaa222");
968        let bookmark_main_local_target = RefTarget::normal(CommitId::from_hex("ccc111"));
969        let bookmark_main_origin_target = RefTarget::normal(CommitId::from_hex("ccc222"));
970        let bookmark_deleted_origin_target = RefTarget::normal(CommitId::from_hex("ccc333"));
971        let tag_v1_local_target = RefTarget::normal(CommitId::from_hex("ddd111"));
972        let tag_v1_origin_target = RefTarget::normal(CommitId::from_hex("ddd222"));
973        let tag_deleted_origin_target = RefTarget::normal(CommitId::from_hex("ddd333"));
974        let git_refs_main_target = RefTarget::normal(CommitId::from_hex("fff111"));
975        let git_refs_feature_target = RefTarget::from_legacy_form(
976            [CommitId::from_hex("fff111")],
977            [CommitId::from_hex("fff222"), CommitId::from_hex("fff333")],
978        );
979        let default_wc_commit_id = CommitId::from_hex("abc111");
980        let test_wc_commit_id = CommitId::from_hex("abc222");
981        View {
982            head_ids: hashset! {head_id1, head_id2},
983            local_bookmarks: btreemap! {
984                "main".into() => bookmark_main_local_target,
985            },
986            local_tags: btreemap! {
987                "v1.0".into() => tag_v1_local_target,
988            },
989            remote_views: btreemap! {
990                "origin".into() => RemoteView {
991                    bookmarks: btreemap! {
992                        "main".into() => tracked_remote_ref(&bookmark_main_origin_target),
993                        "deleted".into() => new_remote_ref(&bookmark_deleted_origin_target),
994                    },
995                    tags: btreemap! {
996                        "v1.0".into() => tracked_remote_ref(&tag_v1_origin_target),
997                        "deleted".into() => new_remote_ref(&tag_deleted_origin_target),
998                    },
999                },
1000            },
1001            git_refs: btreemap! {
1002                "refs/heads/main".into() => git_refs_main_target,
1003                "refs/heads/feature".into() => git_refs_feature_target,
1004            },
1005            git_head: RefTarget::normal(CommitId::from_hex("fff111")),
1006            wc_commit_ids: btreemap! {
1007                WorkspaceName::DEFAULT.to_owned() => default_wc_commit_id,
1008                "test".into() => test_wc_commit_id,
1009            },
1010        }
1011    }
1012
1013    fn create_operation() -> Operation {
1014        let pad_id_bytes = |hex: &str, len: usize| {
1015            let mut bytes = hex_util::decode_hex(hex).unwrap();
1016            bytes.resize(len, b'\0');
1017            bytes
1018        };
1019        Operation {
1020            view_id: ViewId::new(pad_id_bytes("aaa111", VIEW_ID_LENGTH)),
1021            parents: vec![
1022                OperationId::new(pad_id_bytes("bbb111", OPERATION_ID_LENGTH)),
1023                OperationId::new(pad_id_bytes("bbb222", OPERATION_ID_LENGTH)),
1024            ],
1025            metadata: OperationMetadata {
1026                time: TimestampRange {
1027                    start: Timestamp {
1028                        timestamp: MillisSinceEpoch(123456789),
1029                        tz_offset: 3600,
1030                    },
1031                    end: Timestamp {
1032                        timestamp: MillisSinceEpoch(123456800),
1033                        tz_offset: 3600,
1034                    },
1035                },
1036                description: "check out foo".to_string(),
1037                hostname: "some.host.example.com".to_string(),
1038                username: "someone".to_string(),
1039                is_snapshot: false,
1040                workspace_name: Some(WorkspaceNameBuf::from("test")),
1041                attributes: btreemap! {
1042                    "key1".to_string() => "value1".to_string(),
1043                    "key2".to_string() => "value2".to_string(),
1044                },
1045            },
1046            commit_predecessors: Some(btreemap! {
1047                CommitId::from_hex("111111") => vec![],
1048                CommitId::from_hex("222222") => vec![
1049                    CommitId::from_hex("333333"),
1050                    CommitId::from_hex("444444"),
1051                ],
1052            }),
1053        }
1054    }
1055
1056    #[test]
1057    fn test_hash_view() {
1058        // Test exact output so we detect regressions in compatibility
1059        assert_snapshot!(
1060            ViewId::new(blake2b_hash(&create_view()).to_vec()).hex(),
1061            @"2c0b174d117ca85e7faa96f6d997362403105e8eb31e7f82ac9abd3dc48ae62683e9a76ef5d117ebc8a743d17e1945236df9ccefd7574f7e4b5336a63796b967"
1062        );
1063    }
1064
1065    #[test]
1066    fn test_hash_operation() {
1067        // Test exact output so we detect regressions in compatibility
1068        assert_snapshot!(
1069            OperationId::new(blake2b_hash(&create_operation()).to_vec()).hex(),
1070            @"f5963c593a63bb852061a86ad919c12c6ba1940eeef30a832524c39ccea6a9f768aa2aa53becec34d379eb291ec6726837c4113857849cb9dcc62dbe0a517176"
1071        );
1072    }
1073
1074    #[test]
1075    fn test_read_write_view() -> TestResult {
1076        let temp_dir = new_temp_dir();
1077        let root_data = RootOperationData {
1078            root_commit_id: CommitId::from_hex("000000"),
1079        };
1080        let store = SimpleOpStore::init(temp_dir.path(), root_data)?;
1081        let view = create_view();
1082        let view_id = store.write_view(&view).block_on()?;
1083        let read_view = store.read_view(&view_id).block_on()?;
1084        assert_eq!(read_view, view);
1085        Ok(())
1086    }
1087
1088    #[test]
1089    fn test_read_write_operation() -> TestResult {
1090        let temp_dir = new_temp_dir();
1091        let root_data = RootOperationData {
1092            root_commit_id: CommitId::from_hex("000000"),
1093        };
1094        let store = SimpleOpStore::init(temp_dir.path(), root_data)?;
1095        let operation = create_operation();
1096        let op_id = store.write_operation(&operation).block_on()?;
1097        let read_operation = store.read_operation(&op_id).block_on()?;
1098        assert_eq!(read_operation, operation);
1099        Ok(())
1100    }
1101
1102    #[test]
1103    fn test_remote_views_legacy_roundtrip() {
1104        let mut view = create_view();
1105        assert!(!view.remote_views.is_empty());
1106        for remote_view in view.remote_views.values_mut() {
1107            // remote tags cannot be preserved in "legacy" format
1108            remote_view.tags.clear();
1109        }
1110        let mut proto = view_to_proto(&view);
1111        proto.remote_views.clear(); // drop "new" format
1112        let view_reconstructed = view_from_proto(proto).unwrap();
1113        assert_eq!(view.remote_views, view_reconstructed.remote_views);
1114    }
1115
1116    #[test]
1117    fn test_remote_views_new_roundtrip() {
1118        let view = create_view();
1119        assert!(!view.remote_views.is_empty());
1120        let mut proto = view_to_proto(&view);
1121        for bookmark in &mut proto.bookmarks {
1122            #[expect(deprecated)]
1123            bookmark.remote_bookmarks.clear(); // drop "legacy" format
1124        }
1125        let view_reconstructed = view_from_proto(proto).unwrap();
1126        assert_eq!(view.remote_views, view_reconstructed.remote_views);
1127    }
1128
1129    #[test]
1130    fn test_migrate_git_refs_to_remote_tags() {
1131        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
1132            target: target.clone(),
1133            state: RemoteRefState::Tracked,
1134        };
1135        let git_ref_to_proto = |name: &str, ref_target| crate::protos::simple_op_store::GitRef {
1136            name: name.to_owned(),
1137            #[expect(deprecated)]
1138            commit_id: Default::default(),
1139            target: ref_target_to_proto(ref_target),
1140        };
1141        let v1_target = RefTarget::normal(CommitId::from_hex("111111"));
1142        let main_target = RefTarget::normal(CommitId::from_hex("222222"));
1143        let orig_remote_views = btreemap! {
1144            "git".into() => RemoteView {
1145                bookmarks: btreemap! {
1146                    "main".into() => tracked_remote_ref(&main_target),
1147                },
1148                tags: btreemap! {},
1149            },
1150        };
1151        let proto = crate::protos::simple_op_store::View {
1152            remote_views: remote_views_to_proto(&orig_remote_views),
1153            git_refs: vec![
1154                git_ref_to_proto("refs/tags/v1.0", &v1_target),
1155                git_ref_to_proto("refs/heads/main", &main_target),
1156            ],
1157            has_git_refs_migrated_to_remote_tags: false,
1158            ..Default::default()
1159        };
1160
1161        let view = view_from_proto(proto).unwrap();
1162        if cfg!(feature = "git") {
1163            assert_eq!(
1164                view.remote_views,
1165                btreemap! {
1166                    "git".into() => RemoteView {
1167                        bookmarks: btreemap! {
1168                            "main".into() => tracked_remote_ref(&main_target),
1169                        },
1170                        tags: btreemap! {
1171                            "v1.0".into() => tracked_remote_ref(&v1_target),
1172                        },
1173                    },
1174                }
1175            );
1176        } else {
1177            assert_eq!(view.remote_views, orig_remote_views);
1178        }
1179
1180        // Once migrated, "git" remote tags shouldn't be populated again.
1181        let mut proto = view_to_proto(&view);
1182        assert!(proto.has_git_refs_migrated_to_remote_tags);
1183        for view_proto in &mut proto.remote_views {
1184            view_proto.tags.clear();
1185        }
1186        let view = view_from_proto(proto).unwrap();
1187        assert_eq!(view.remote_views, orig_remote_views);
1188    }
1189
1190    #[test]
1191    fn test_bookmark_views_legacy_roundtrip() {
1192        let new_remote_ref = |target: &RefTarget| RemoteRef {
1193            target: target.clone(),
1194            state: RemoteRefState::New,
1195        };
1196        let tracked_remote_ref = |target: &RefTarget| RemoteRef {
1197            target: target.clone(),
1198            state: RemoteRefState::Tracked,
1199        };
1200        let local_bookmark1_target = RefTarget::normal(CommitId::from_hex("111111"));
1201        let local_bookmark3_target = RefTarget::normal(CommitId::from_hex("222222"));
1202        let git_bookmark1_target = RefTarget::normal(CommitId::from_hex("333333"));
1203        let remote1_bookmark1_target = RefTarget::normal(CommitId::from_hex("444444"));
1204        let remote2_bookmark2_target = RefTarget::normal(CommitId::from_hex("555555"));
1205        let remote2_bookmark4_target = RefTarget::normal(CommitId::from_hex("666666"));
1206        let local_bookmarks = btreemap! {
1207            "bookmark1".into() => local_bookmark1_target.clone(),
1208            "bookmark3".into() => local_bookmark3_target.clone(),
1209        };
1210        let remote_views = btreemap! {
1211            "git".into() => RemoteView {
1212                bookmarks: btreemap! {
1213                    "bookmark1".into() => tracked_remote_ref(&git_bookmark1_target),
1214                },
1215                tags: btreemap! {},
1216            },
1217            "remote1".into() => RemoteView {
1218                bookmarks: btreemap! {
1219                    "bookmark1".into() => tracked_remote_ref(&remote1_bookmark1_target),
1220                },
1221                tags: btreemap! {},
1222            },
1223            "remote2".into() => RemoteView {
1224                bookmarks: btreemap! {
1225                    // "bookmark2" is non-tracking. "bookmark4" is tracking, but locally deleted.
1226                    "bookmark2".into() => new_remote_ref(&remote2_bookmark2_target),
1227                    "bookmark4".into() => tracked_remote_ref(&remote2_bookmark4_target),
1228                },
1229                tags: btreemap! {},
1230            },
1231        };
1232
1233        let bookmarks_legacy = bookmark_views_to_proto_legacy(&local_bookmarks, &remote_views);
1234        assert_eq!(
1235            bookmarks_legacy
1236                .iter()
1237                .map(|proto| &proto.name)
1238                .sorted()
1239                .collect_vec(),
1240            vec!["bookmark1", "bookmark2", "bookmark3", "bookmark4"],
1241        );
1242
1243        let (local_bookmarks_reconstructed, remote_views_reconstructed) =
1244            bookmark_views_from_proto_legacy(bookmarks_legacy).unwrap();
1245        assert_eq!(local_bookmarks_reconstructed, local_bookmarks);
1246        assert_eq!(remote_views_reconstructed, remote_views);
1247    }
1248
1249    #[test]
1250    fn test_ref_target_change_delete_order_roundtrip() {
1251        let target = RefTarget::from_merge(Merge::from_removes_adds(
1252            vec![Some(CommitId::from_hex("111111"))],
1253            vec![Some(CommitId::from_hex("222222")), None],
1254        ));
1255        let maybe_proto = ref_target_to_proto(&target);
1256        assert_eq!(ref_target_from_proto(maybe_proto), target);
1257
1258        // If it were legacy format, order of None entry would be lost.
1259        let target = RefTarget::from_merge(Merge::from_removes_adds(
1260            vec![Some(CommitId::from_hex("111111"))],
1261            vec![None, Some(CommitId::from_hex("222222"))],
1262        ));
1263        let maybe_proto = ref_target_to_proto(&target);
1264        assert_eq!(ref_target_from_proto(maybe_proto), target);
1265    }
1266
1267    #[test]
1268    fn test_ref_target_legacy_roundtrip() {
1269        let target = RefTarget::absent();
1270        let maybe_proto = ref_target_to_proto_legacy(&target);
1271        assert_eq!(ref_target_from_proto(maybe_proto), target);
1272
1273        let target = RefTarget::normal(CommitId::from_hex("111111"));
1274        let maybe_proto = ref_target_to_proto_legacy(&target);
1275        assert_eq!(ref_target_from_proto(maybe_proto), target);
1276
1277        // N-way conflict
1278        let target = RefTarget::from_legacy_form(
1279            [CommitId::from_hex("111111"), CommitId::from_hex("222222")],
1280            [
1281                CommitId::from_hex("333333"),
1282                CommitId::from_hex("444444"),
1283                CommitId::from_hex("555555"),
1284            ],
1285        );
1286        let maybe_proto = ref_target_to_proto_legacy(&target);
1287        assert_eq!(ref_target_from_proto(maybe_proto), target);
1288
1289        // Change-delete conflict
1290        let target = RefTarget::from_legacy_form(
1291            [CommitId::from_hex("111111")],
1292            [CommitId::from_hex("222222")],
1293        );
1294        let maybe_proto = ref_target_to_proto_legacy(&target);
1295        assert_eq!(ref_target_from_proto(maybe_proto), target);
1296    }
1297}