Skip to content

Update git-co-authors extension#26090

Merged
raycastbot merged 23 commits intoraycast:mainfrom
mrclsu:ext/git-co-authors
Mar 17, 2026
Merged

Update git-co-authors extension#26090
raycastbot merged 23 commits intoraycast:mainfrom
mrclsu:ext/git-co-authors

Conversation

@mrclsu
Copy link
Copy Markdown
Contributor

@mrclsu mrclsu commented Mar 6, 2026

Description

This PR Includes the following changes to the git-co-authors extension:

  • Updated Choose Co-Author action with: frecency based sorting, and empty list view
  • New action for loading authors from folders
  • New action for clearing authors
  • Add Windows support

Screencast

Load Authors Action:
image

Clear Authors Action:
image

Checklist

@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: git-co-authors Issues related to the git-co-authors extension platform: macOS platform: Windows labels Mar 6, 2026
@raycastbot
Copy link
Copy Markdown
Collaborator

raycastbot commented Mar 6, 2026

Thank you for your contribution! 🎉

🔔 @nickradford @mateusjunges you might want to have a look.

You can use this guide to learn how to check out the Pull Request locally in order to test it.

📋 Quick checkout commands
BRANCH="ext/git-co-authors"
FORK_URL="https://github.com/mrclsu/raycast-extensions.git"
EXTENSION_NAME="git-co-authors"
REPO_NAME="raycast-extensions"

git clone -n --depth=1 --filter=tree:0 -b $BRANCH $FORK_URL
cd $REPO_NAME
git sparse-checkout set --no-cone "extensions/$EXTENSION_NAME"
git checkout
cd "extensions/$EXTENSION_NAME"
npm install && npm run dev

We're currently experiencing a high volume of incoming requests. As a result, the initial review may take up to 10-15 business days.

@mrclsu mrclsu force-pushed the ext/git-co-authors branch 2 times, most recently from 9dcc7ae to b9bdd34 Compare March 6, 2026 15:35
@mrclsu mrclsu marked this pull request as ready for review March 6, 2026 15:40
@mrclsu mrclsu requested a review from pernielsentikaer as a code owner March 6, 2026 15:40
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 6, 2026

Greptile Summary

This PR significantly expands the git-co-authors extension with frecency-based sorting in the author picker, a new bulk-import flow (LoadCoAuthorsFromFolder + AuthorsSelector), a standalone clear-authors command, and Windows platform support. The implementation is solid — race conditions in the async folder scan are guarded with a scanIdRef, the empty-list and empty-selection states are handled, platform-specific keyboard shortcuts are applied consistently, and the ESLint config is correctly migrated to the ESLint v9 flat-config format.

  • All previous review concerns (shallow-copy mutation, array-index React key, missing empty-state, shortcut Windows compatibility, etc.) have been addressed.
  • load-co-authors.tsx mixes the deprecated two-argument showToast(style, title) form with the newer object form — minor consistency issue.
  • The README keyboard shortcuts table only lists macOS symbols; Windows equivalents are absent despite Windows now being a supported platform.

Confidence Score: 4/5

  • This PR is safe to merge — no logic bugs introduced and all previously flagged issues have been resolved.
  • The implementation is well-structured and the previous round of review concerns have been thoroughly addressed. The only remaining items are minor: inconsistent showToast API usage in one file, and missing Windows keyboard shortcut documentation in the README. Neither affects runtime behaviour.
  • No files require special attention.

Important Files Changed

Filename Overview
extensions/git-co-authors/src/choose-coauthor.tsx Main command view updated with frecency-based sorting via useFrecencySorting, empty list view, and platform-specific keyboard shortcuts for both macOS and Windows. Logic looks correct; previous concerns around shortcuts and missing empty state are fully resolved.
extensions/git-co-authors/src/authors-selector.tsx New component for previewing and selecting loaded authors before saving. Previous concerns (array-index key, shallow-copy mutation, misleading success HUD on empty selection) are all addressed in this version.
extensions/git-co-authors/src/load-co-authors.tsx New view command that scans a directory for git repos and bulk-imports co-authors. Race conditions are handled with scanIdRef. Minor: inconsistent use of deprecated two-argument showToast(style, title) alongside the newer object-form API on line 43.
extensions/git-co-authors/src/utils.ts Added findGitReposInDir and getCoAuthorsForDir helpers. The redundant stderr check from the previous iteration is gone, and directory-read failures now surface a proper toast to the user instead of silently returning.
extensions/git-co-authors/README.md Significantly expanded with feature list, commands table, and keyboard shortcuts. The shortcuts table only documents macOS symbols; Windows equivalents are absent despite Windows now being a supported platform.

Comments Outside Diff (1)

  1. extensions/git-co-authors/README.md, line 83-91 (link)

    Windows keyboard shortcuts missing from README

    The keyboard shortcuts table only documents macOS symbols (, ). Since this PR explicitly adds "Windows" to the supported platforms, Windows users reading this table won't know their equivalents (Ctrl, Alt).

    Consider adding a Windows column or a separate Windows section, for example:

    Action macOS Windows
    Copy selected authors Ctrl Enter
    Paste into window Ctrl Shift Enter
    Add new author N Ctrl N
    Edit author E Ctrl E
    Remove author Ctrl Backspace
    Clear all authors Ctrl Alt Backspace

Last reviewed commit: 91820e7

mrclsu and others added 4 commits March 9, 2026 11:30
- Bump all dependencies
- Fix linter errors
- Improve error handling
- Unify merge co-authors from folder, and repo into a single action
- Readme improvements
- Fix casing
- Merge pull request raycast#3 from mrclsu/emptyview-no-authors
- Merge pull request raycast#4 from mrclsu/add-clear-authors-command
- Merge branch \'main\' into add-clear-authors-command
- Merge pull request raycast#2 from mrclsu/fix-load-co-authors-state-and-dedup
- Merge pull request raycast#1 from mrclsu/add-load-from-folder-command
- Add clear authors command that clears all imported authors
- Add empty view when there are no authors imported yet
- Fix state, dedup, and error handling in load-co-authors
- fix recursive repo scanning
- add load from folder command
- Update raycast api versions
- Initial implementation of load from repo

Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu mrclsu force-pushed the ext/git-co-authors branch from c1de7d8 to af8222d Compare March 9, 2026 10:30
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

mrclsu and others added 2 commits March 9, 2026 13:39
Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

mrclsu and others added 3 commits March 9, 2026 13:49
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

mrclsu and others added 3 commits March 9, 2026 14:09
Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

mrclsu and others added 2 commits March 9, 2026 14:33
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

Co-authored-by: Jonatan Bakucz <jonatan.mark.bakucz@emarsys.com>
@mrclsu
Copy link
Copy Markdown
Contributor Author

mrclsu commented Mar 9, 2026

@greptileai

@pernielsentikaer pernielsentikaer self-assigned this Mar 17, 2026
@raycastbot raycastbot added the OP is contributor The OP of the PR is a contributor of the extension label Mar 17, 2026
Copy link
Copy Markdown
Collaborator

@pernielsentikaer pernielsentikaer left a comment

Choose a reason for hiding this comment

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

Looks good to me, approved 🔥

@raycastbot raycastbot merged commit e459586 into raycast:main Mar 17, 2026
2 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Published to the Raycast Store:
https://raycast.com/nickradford/git-co-authors

@raycastbot
Copy link
Copy Markdown
Collaborator

🎉 🎉 🎉

We've rewarded your Raycast account with some credits. You will soon be able to exchange them for some swag.

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

Labels

extension fix / improvement Label for PRs with extension's fix improvements extension: git-co-authors Issues related to the git-co-authors extension OP is contributor The OP of the PR is a contributor of the extension platform: macOS platform: Windows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants