mirror of
https://github.com/runbear-io/beardrive.git
synced 2026-08-25 08:08:08 +02:00
Snow LeeandClaude Opus 5
62c241d3d6
Agent platforms read hook config only from the directory a session starts in — never a parent, never a subfolder. Project-level hooks therefore fired only for sessions that happened to start at the mount, and, living inside a synced folder, they replicated one machine's agent config to the whole team (a second writer of a file bdrive already owns). Claude Code additionally ignores project hooks until the folder is trusted, so in practice they were often inert without any visible sign. Hooks now go to each platform's user config, once per machine, covering every session in every folder; the existing shell guard keeps them a no-op outside BearDrive projects. Install migrates away blocks older versions wrote into projects, and `bdrive hooks uninstall` removes ours while leaving foreign hooks untouched. Setup is also one command now. init absorbs the skill install, prints the hub link, and takes --server, so connecting to a named hub no longer needs a separate login; the runbook forbids preflight and command chaining, since each distinct command costs the user a permission prompt. For plugin users a PreToolUse hook auto-approves bdrive's own setup subcommands — narrowly: any shell operator in the command disqualifies it. Also drops --shared in favor of `init . --only wiki,docs`, which writes a managed block of .bdriveignore rules instead of a second scope mechanism. Because those rules sync, `sync --prune` now refuses on a scoped project rather than stripping everything outside the scope from the hub for everyone. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_016aYntCWwdUhpzUfEk3ddyJ
96 lines
3.0 KiB
Go
96 lines
3.0 KiB
Go
package main
|
|
|
|
import (
|
|
"bytes"
|
|
"os"
|
|
"path/filepath"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/runbear-io/beardrive/internal/config"
|
|
)
|
|
|
|
// bdrive url computes the permission-walled viewer link locally: hub origin
|
|
// + project id from the mount's remote, path segments percent-encoded with
|
|
// literal "/" separators, unsynced paths refused.
|
|
func TestURLCommand(t *testing.T) {
|
|
t.Setenv("BDRIVE_HOME", t.TempDir())
|
|
folder := t.TempDir()
|
|
folder, _ = filepath.EvalSymlinks(folder)
|
|
if _, err := config.SaveProject(folder, config.Project{
|
|
Volume: "wiki",
|
|
Remote: "https://hub.example.com/p/p-12345678",
|
|
}); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
os.MkdirAll(filepath.Join(folder, "wiki notes"), 0o755)
|
|
os.WriteFile(filepath.Join(folder, "wiki notes", "a report.md"), []byte("x"), 0o644)
|
|
os.WriteFile(filepath.Join(folder, ".bdriveignore"), []byte("drafts/\n"), 0o644)
|
|
|
|
run := func(args ...string) (string, error) {
|
|
c := urlCmd()
|
|
var out bytes.Buffer
|
|
c.SetOut(&out)
|
|
c.SetArgs(args)
|
|
err := c.Execute()
|
|
return strings.TrimSpace(out.String()), err
|
|
}
|
|
|
|
// A file: segments encoded, slashes literal.
|
|
got, err := run(filepath.Join(folder, "wiki notes", "a report.md"))
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
want := "https://hub.example.com/p-12345678/wiki%20notes/a%20report.md"
|
|
if got != want {
|
|
t.Fatalf("url = %q, want %q", got, want)
|
|
}
|
|
|
|
// The project root: the home page.
|
|
if got, err = run(folder); err != nil || got != "https://hub.example.com/p-12345678" {
|
|
t.Fatalf("root url = %q, %v", got, err)
|
|
}
|
|
|
|
// An ignored path is refused — the link would 404 for everyone.
|
|
if _, err = run(filepath.Join(folder, "drafts", "wip.md")); err == nil || !strings.Contains(err.Error(), "not synced") {
|
|
t.Fatalf("ignored path: err = %v, want 'not synced'", err)
|
|
}
|
|
|
|
// Outside the project entirely.
|
|
if _, err = run(filepath.Join(t.TempDir(), "elsewhere.md")); err == nil {
|
|
t.Fatal("outside path should error")
|
|
}
|
|
}
|
|
|
|
// A legacy include-list mount refuses links outside the scope.
|
|
func TestURLCommandIncludeScope(t *testing.T) {
|
|
t.Setenv("BDRIVE_HOME", t.TempDir())
|
|
folder := t.TempDir()
|
|
folder, _ = filepath.EvalSymlinks(folder)
|
|
if _, err := config.SaveProject(folder, config.Project{
|
|
Volume: "wiki",
|
|
Remote: "https://hub.example.com/p/p-12345678",
|
|
Include: []string{"wiki/"},
|
|
}); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
os.MkdirAll(filepath.Join(folder, "wiki"), 0o755)
|
|
os.WriteFile(filepath.Join(folder, "wiki", "a.md"), []byte("x"), 0o644)
|
|
os.WriteFile(filepath.Join(folder, "code.go"), []byte("x"), 0o644)
|
|
|
|
run := func(arg string) (string, error) {
|
|
c := urlCmd()
|
|
var out bytes.Buffer
|
|
c.SetOut(&out)
|
|
c.SetArgs([]string{arg})
|
|
err := c.Execute()
|
|
return strings.TrimSpace(out.String()), err
|
|
}
|
|
if got, err := run(filepath.Join(folder, "wiki", "a.md")); err != nil || got != "https://hub.example.com/p-12345678/wiki/a.md" {
|
|
t.Fatalf("in-scope = %q, %v", got, err)
|
|
}
|
|
if _, err := run(filepath.Join(folder, "code.go")); err == nil || !strings.Contains(err.Error(), "not synced") {
|
|
t.Fatalf("out-of-scope: err = %v, want 'not synced'", err)
|
|
}
|
|
}
|