Skip to content

Feature/base#4

Merged
gauthamp10 merged 3 commits into
mainfrom
feature/base
Mar 19, 2026
Merged

Feature/base#4
gauthamp10 merged 3 commits into
mainfrom
feature/base

Conversation

@gauthamp10

Copy link
Copy Markdown
Owner

No description provided.

The global /search endpoint requires authentication and returned 401
for unauthenticated (guest) requests, silently showing no results.

Changes:
- In guest mode, the "Projects" scope now falls back to
  GET /projects?search=...&visibility=public which works without a token.
- Issues, Merge Requests, Commits and Code scopes are disabled (greyed
  out with a lock icon) in guest mode; clicking shows an amber notice
  with a Sign in link.
- Added isError handling so API failures surface a readable message
  instead of silently showing an empty state.
- Empty-state description is contextualised for guest vs authenticated.

Made-with: Cursor
…pping

The guest-mode queryFn returned a plain object whose item type didn't
include optional GitLabSearchResult fields (state, filename, path, data),
causing TypeScript to widen the union and reject property accesses in the
render tree.

Fix: annotate the mapped items as GitLabSearchResult so the return type
is consistently PagedResult<GitLabSearchResult> in both branches.

Made-with: Cursor
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@gauthamp10 gauthamp10 merged commit 93c4afe into main Mar 19, 2026
15 checks passed
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