Skip to content

Auto-discover local repos; declutter Connected Accounts and check-in start - #95

Merged
qiyundai merged 1 commit into
mainfrom
feature/repos-auto-discovery
Jul 24, 2026
Merged

Auto-discover local repos; declutter Connected Accounts and check-in start#95
qiyundai merged 1 commit into
mainfrom
feature/repos-auto-discovery

Conversation

@qiyundai

Copy link
Copy Markdown
Collaborator

Summary

  • Repos: replaced manual "add repo by path" with auto-discovery — the user picks one or more parent "code root" folders in Settings, and mypa scans them for git checkouts, deriving the GitHub repo/default branch and filtering to the configured GitHub-org scope. Authoring stays off by default and is opt-in per repo, since enabling it lets mypa open real PRs against that checkout.
  • Connected Accounts: the "About You" surface list now only renders identity surfaces backed by an enabled MCP server — no more "not connected" placeholder rows for services the user never enabled (e.g. Linear/Notion).
  • Check-in: starting a check-in now shows the existing thinking-dots indicator immediately, instead of an empty card until the opening briefing's first streamed chunk arrives.

Reviewed by three parallel review passes (correctness, security, conventions) before push; fixes applied from that review:

  • Reconciliation no longer creates a duplicate discovered RepoLink when a scanned path already has a manual link.
  • rescanRepos() now queues a follow-up scan when called mid-scan, so addCodeRoots/removeCodeRoot always return results reflecting their own just-written config, not a stale in-flight scan.
  • addCodeRoots rejects filesystem/drive roots and the home directory outright, and the scanner yields to the event loop periodically, so a bad root can't hang or repeatedly re-scan the whole disk.
  • Removed the now-unreachable legacy addRepoLink/removeRepoLink functions rather than keeping them as unused back-compat code.
  • Updated docs-dev/renderer.md, which the first pass had missed.

Test plan

  • npm run typecheck passes
  • npm run build passes
  • Manual: add a code root in Settings, confirm repos are discovered with authoring off by default; toggle authoring/edit Jira keys, rescan, confirm edits persist; remove a root and confirm its repos disappear
  • Manual: confirm Connected Accounts only shows enabled surfaces
  • Manual: start a check-in and confirm the thinking indicator shows immediately

🤖 Generated with Claude Code

…eck-in start indicator

Reduces setup friction: repos are now discovered by scanning user-chosen code
root folders instead of manual path entry (authoring stays opt-in per repo),
Connected Accounts no longer lists unenabled identity surfaces as "not
connected", and starting a check-in shows the thinking indicator immediately
instead of an empty card while the opening briefing spins up.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@qiyundai
qiyundai merged commit 8784ef1 into main Jul 24, 2026
3 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