fix: correct README multi-path claim, cover repeated --remote-path refusal - #402
fix: correct README multi-path claim, cover repeated --remote-path refusal#402khaliqgant wants to merge 1 commit into
Conversation
…fusal README documented multi-allowlist scoping as available via "the relayfile CLI directly", but the CLI is also single-root while scoped mount operator surfaces are unavailable. Corrects the README to state that constraint plainly instead of overclaiming, and adds a real regression test proving a repeated --remote-path with --local-layout=scoped is refused before any network or filesystem side effects (zero server requests, mirror directory never created). Fixes #379. Verified: - go build ./... - go test ./cmd/relayfile-cli/... -run TestMountRefusesRepeatedRemotePathsWithScopedLayoutBeforeSideEffects -v -count=1 - go test ./cmd/relayfile-cli/... -count=1 (full package suite)
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Relayfile Eval ReviewRun: Passed: 4 | Needs human: 0 | Reviewable: 0 | Missing output: 0 | Failed: 0 | Skipped: 0 Human Review CasesNo reviewable human-review cases captured Relayfile output. |
1 participant
Fixes #379.
README documented multi-allowlist scoping as available via "the relayfile CLI directly", but the CLI is also single-root while scoped mount operator surfaces are unavailable. Corrects the README to state that constraint plainly instead of overclaiming, and adds a real regression test proving a repeated `--remote-path` with `--local-layout=scoped` is refused before any network or filesystem side effects (zero server requests, mirror directory never created).
Test plan