Skip to content

feat(settings): add repo context search#583

Merged
matt2e merged 5 commits intomainfrom
repo-settings-search
Apr 8, 2026
Merged

feat(settings): add repo context search#583
matt2e merged 5 commits intomainfrom
repo-settings-search

Conversation

@matt2e
Copy link
Copy Markdown
Contributor

@matt2e matt2e commented Apr 8, 2026

Summary

  • add a repo search field to the Actions settings sidebar
  • match search terms across repo name and subpath tokens in any order
  • add tests for repo context search matching behavior

@matt2e matt2e requested review from baxen and wesbillman as code owners April 8, 2026 01:23
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 41f8b758dd

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

const subpath = context.subpath?.toLowerCase() ?? '';
const subpathParts = subpath.split('/').filter(Boolean);

return [githubRepo, org, repoName, subpath, ...subpathParts].filter(Boolean);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Include full repo context string in search terms

The matcher never indexes the combined githubRepo/subpath label, so a query like block/builderbot/apps/staged (which matches how the repo is displayed in the sidebar/title) returns no results even for an exact visible context. This makes exact copy/paste searches fail unless users manually insert spaces between segments, which is a functional mismatch for the new search UX.

Useful? React with 👍 / 👎.

@matt2e matt2e merged commit bcafdb7 into main Apr 8, 2026
5 checks passed
@matt2e matt2e deleted the repo-settings-search branch April 8, 2026 01:31
loganj added a commit that referenced this pull request Apr 9, 2026
…ktree-dis

* origin/main: (45 commits)
  feat(penpal): persist recent files activity across restarts (#595)
  feat: persist open windows, tabs, and geometry across app restarts (#588)
  feat(staged): generate next prefill text when generating notes (#594)
  feat(staged): add "Run again" button to action output dialog (#593)
  feat: queue repo sessions in project MCP with async cancellation fix (#591)
  fix(staged): lowercase "building" subtitle for consistency (#592)
  fix(staged): use completedAt for commit prefill timeline ranking (#590)
  fix: add per-branch mutex to prevent worktree setup race condition (#589)
  feat: add diff commit session launcher (#585)
  fix: avoid error state for empty titled code reviews (#587)
  fix: cancel in-flight auto reviews before manual sessions (#584)
  feat(settings): add repo context search (#583)
  fix(staged): sort timeline items by completion time so queued items appear before completed ones (#568)
  fix: drain queued branch sessions from backend lifecycle (#579)
  fix: restore resume session metadata for resumed actions (#580)
  feat(staged): add purple diff comment theme (#578)
  fix(ui): hide branch actions during setup (#572)
  fix: validate parsed PR URLs before extraction (#577)
  fix: centralize staged relative time updates (#576)
  fix: avoid UI freezes when starting background sessions (#575)
  ...

# Conflicts:
#	apps/penpal/internal/watcher/watcher.go
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