Files
beardrive/cmd/bdrive/scope.go
T
9088127176 feat(sync): bdrive forget + sync --prune to take ignored paths off the hub (BEA-20) (#68)
Adding a path to .bdriveignore only stopped future uploads: anything that
synced before the rule existed stayed on the hub forever, with no command
that removed it without deleting it from local disk on every device.

Two engine changes make an explicit removal safe:

- materialize's delete loop now consults the filter. A cached path absent
  from the replayed target that the rules exclude is dropped from tracking
  instead of unlinked — without this, any delete op for a now-filtered path
  wipes every peer's local copy, which is the data loss this issue is about.
- the filter is reloaded mid-cycle from the pulled .bdriveignore, before
  materialize. A peer receiving the new rules and the deletes they justify
  in one batch would otherwise materialize with stale rules and the guard
  would never fire. materialize's write side is split into materializeFile
  so the ignore file can land on its own.

On top of that, Session.Prune journals a delete for every path the replayed
state still holds that the SHARED rules exclude — reconciling against the
replay, not the local cache, because a path filtered out in an earlier cycle
was dropped from the cache back then and is invisible locally today. The
rules are deliberately ignore-only: the include scope lives in each device's
own .bdrive/config.json and does not sync, so pruning against it would let a
narrow-scope device delete a whole-folder teammate's files.

Plain `bdrive sync` and the daemon are unchanged — pruning is never a side
effect of editing .bdriveignore.

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-07-28 07:22:11 +09:00

174 lines
4.7 KiB
Go

package main
import (
"fmt"
"os"
"path/filepath"
"strings"
"github.com/spf13/cobra"
"github.com/runbear-io/beardrive/internal/config"
)
// bdrive scope shows and edits the project's sync scope — the include list
// in .bdrive/config.json that `init --shared` seeds — so growing or
// shrinking what syncs never means hand-editing JSON. The daemon re-reads
// the config every tick, so changes apply within seconds.
func scopeCmd() *cobra.Command {
c := &cobra.Command{
Use: "scope",
Short: "Show or change which subfolders sync (the include list)",
Long: `Show or change the project's sync scope: the include list in
.bdrive/config.json, as set by init --shared. An empty list means the whole
folder syncs. Run from the mount root; the daemon picks changes up within
seconds.
Removing a folder stops syncing it but deletes nothing — local files stay,
and the hub keeps everything already synced. To take something off the hub
too, use ` + "`bdrive forget <path>`" + `.`,
Example: ` bdrive scope # show what syncs
bdrive scope add docs # also sync ./docs
bdrive scope rm docs # stop syncing ./docs (files stay everywhere)`,
Args: cobra.NoArgs,
RunE: func(cmd *cobra.Command, args []string) error {
folder, err := absFolder(nil)
if err != nil {
return err
}
proj, err := mustProject(folder)
if err != nil {
return err
}
printScope(proj)
return nil
},
}
c.AddCommand(scopeAddCmd(), scopeRmCmd())
return c
}
func scopeAddCmd() *cobra.Command {
return &cobra.Command{
Use: "add <dir>...",
Short: "Add shared subfolders to the sync scope",
Args: cobra.MinimumNArgs(1),
RunE: func(cmd *cobra.Command, args []string) error {
folder, err := absFolder(nil)
if err != nil {
return err
}
proj, err := mustProject(folder)
if err != nil {
return err
}
if len(proj.Include) == 0 {
return fmt.Errorf("this project syncs the whole folder; adding %s would narrow it to only that — re-run `bdrive init` with --shared if you want a scoped sync", strings.Join(args, ", "))
}
incs, err := cleanShared(args)
if err != nil {
return err
}
seen := map[string]bool{}
for _, i := range proj.Include {
seen[i] = true
}
added := 0
for _, inc := range incs {
if seen[inc] {
continue
}
if err := os.MkdirAll(filepath.Join(folder, filepath.FromSlash(strings.TrimSuffix(inc, "/"))), 0o755); err != nil {
return err
}
proj.Include = append(proj.Include, inc)
seen[inc] = true
added++
}
if added == 0 {
fmt.Println("already in the sync scope")
} else if _, err := config.SaveProject(folder, proj); err != nil {
return err
}
printScope(proj)
return nil
},
}
}
func scopeRmCmd() *cobra.Command {
return &cobra.Command{
Use: "rm <dir>...",
Short: "Remove shared subfolders from the sync scope (deletes nothing)",
Args: cobra.MinimumNArgs(1),
RunE: func(cmd *cobra.Command, args []string) error {
folder, err := absFolder(nil)
if err != nil {
return err
}
proj, err := mustProject(folder)
if err != nil {
return err
}
kept, err := scopeRemove(proj.Include, args)
if err != nil {
return err
}
if len(kept) == 0 {
return fmt.Errorf("removing the last shared folder would switch to syncing the whole folder; run `bdrive stop` to stop syncing instead")
}
proj.Include = kept
if _, err := config.SaveProject(folder, proj); err != nil {
return err
}
fmt.Println("removed from the sync scope — nothing was deleted, locally or on the hub")
printScope(proj)
return nil
},
}
}
// scopeRemove drops the named dirs from the include list, matching each
// argument literally, in normalized "/dir/" form, and in the pre-anchoring
// "dir/" form that configs written before the anchoring fix still hold
// (hand-edited configs may hold arbitrary patterns). Unknown entries are an
// error.
func scopeRemove(include, args []string) ([]string, error) {
remove := map[string]bool{}
for _, a := range args {
keys := map[string]bool{strings.TrimSpace(a): true}
if norm, err := cleanShared([]string{a}); err == nil {
keys[norm[0]] = true
keys[strings.TrimPrefix(norm[0], "/")] = true
}
found := false
for _, i := range include {
if keys[i] {
remove[i] = true
found = true
}
}
if !found {
return nil, fmt.Errorf("%q is not in the sync scope (see `bdrive scope`)", a)
}
}
var kept []string
for _, i := range include {
if !remove[i] {
kept = append(kept, i)
}
}
return kept, nil
}
func printScope(proj config.Project) {
if len(proj.Include) == 0 {
fmt.Println("the whole folder syncs (no include list)")
return
}
fmt.Println("syncing only:")
for _, i := range proj.Include {
fmt.Println(" ./" + strings.Trim(i, "/"))
}
}