Skip to content

fix(query): 'repos' keyword now returns all registered repositories#49

Merged
juice094 merged 1 commit into
mainfrom
fix/query-repos-list-all
May 15, 2026
Merged

fix(query): 'repos' keyword now returns all registered repositories#49
juice094 merged 1 commit into
mainfrom
fix/query-repos-list-all

Conversation

@juice094
Copy link
Copy Markdown
Owner

Problem

devbase query repos was parsed as Keyword('repos'), only matching repositories whose path/id/tags contained the literal string 'repos'. On a typical workspace this returned 1 result (the only repo whose path happened to contain \repos</code>), while sync --dry-run correctly saw 71+ repositories.

This created the illusion of a data-model split between the query layer and the sync layer, when in reality both read from the unified entities table.

Fix

Add special-case handling in query::run_json: when the expression is exactly repos, return all registered repositories unfiltered.

Verification

Before: query repos → count=1
After: query repos → count=72

Also includes the v0.21.0 architecture hardening plan document.

Previously 'query repos' was parsed as Keyword('repos'), only matching
repositories whose path/id/tags contained the literal string 'repos'.
Add special-case handling so that the bare 'repos' expression returns
all registered repositories unfiltered, aligning CLI behavior with user
intuition and the sync subsystem's view of the registry.

Includes v0.21.0 architecture hardening plan.
@juice094 juice094 merged commit 9e53c18 into main May 15, 2026
6 checks passed
juice094 added a commit that referenced this pull request May 15, 2026
…ories

Regression guard for the fix in PR #49: ensures that 'query repos'
short-circuits keyword filtering and returns the full registry list.
juice094 added a commit that referenced this pull request May 15, 2026
…ories

Regression guard for the fix in PR #49: ensures that 'query repos'
short-circuits keyword filtering and returns the full registry list.
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.

1 participant