Skip to content

Comments

fix(deepagents): CompositeBackend grepRaw/globInfo respect path→backend mapping (#241)#247

Open
Christian Bromann (christian-bromann) wants to merge 2 commits intomainfrom
cb/composite-fix
Open

fix(deepagents): CompositeBackend grepRaw/globInfo respect path→backend mapping (#241)#247
Christian Bromann (christian-bromann) wants to merge 2 commits intomainfrom
cb/composite-fix

Conversation

@christian-bromann

Fixes #241.

  • grepRaw and globInfo in CompositeBackend no longer fan out to every routed backend when the search path doesn't match a route prefix. They now delegate only to the default backend, matching the behavior of all other operations (read, write, edit, lsInfo).
  • Removed the now-unused private routes field (only sortedRoutes is needed).
  • Updated tests to assert the corrected routing: searching at / only queries the default backend; searching with an explicit route prefix (e.g. /memories/) correctly queries the routed backend.

Motivation

When a CompositeBackend is configured with path-based routing like:

new CompositeBackend(workspaceFs, {
    'memory/': configsFs,
    'skills/': configsFs,
})

@changeset-bot
Copy link

changeset-bot bot commented Feb 23, 2026

🦋 Changeset detected

Latest commit: 1f62273

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
deepagents Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Composite backend doesn't respect path->backend mapping

1 participant