Repository navigation
Fix native skill discovery paths and explain missing skills - #772
Merged
Merged
Conversation
Contributor
Author
|
@greptileai review |
Contributor
[Medium impact] The PR appears safe to merge; no actionable issues remain from this review. SummaryThe PR gives each harness explicit native skill search roots and adds copyable detection details to Customize.
No new actionable issues were found.
|
Contributor
Author
|
@greptileai review |
sox8502
marked this pull request as ready for review
October 23, 2026 22:45
Contributor
Author
|
@greptileai review |
Contributor
Author
|
@greptileai review |
Contributor
Author
|
@greptileai review |
Contributor
Author
|
@greptileai review |
3 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skills use explicit native search roots. Cursor discovery, installation and isolated launch share the same root, and invocation reads the selected native file even when Cursor’s own loader ignores
CURSOR_CONFIG_DIR. Antigravity accepts a directory-name fallback whennameis absent. Customize adds collapsed, copyable detection details from the same catalog scan.Consolidated catalog selection and diagnostics, removed duplicate scanning and test scaffolding, and fixed retirement of stale managed Cursor links. The cleanup removes 156 source lines net across both passes, including the invocation fix and its regression coverage. Native ownership, upload precedence, exclusions and collision safeguards are preserved. Related to #685; the reporter’s exact cause remains unconfirmed.
Validation:
d3a6b1b5: Customize diagnostics/copy feedback, refresh after add/remove, upload precedence/exclusions, harness-filtered menus, native Cursor default/configured-root invocations, and unnamed BOM/CRLF Antigravity invocation.d3a6b1b5: catalog/diagnostics and actual native Cursor and unnamed Antigravity invocations; 70 skill regressions and managed-link regression pass.d4644a0fGreptile 5/5 with no unresolved actionable findings.3.17.8/ Antigravity1.4.0(unavailable).Final tests used Cursor CLI
2026.09.28-64d2043(macOS) /2026.10.22-27dcf5f(Windows), and Antigravity1.4.7on Windows. Screenshots and native invocation records were captured; video capture is unavailable. Shared-folder compatibility and recursive/plugin inventories remain deferred. CI results are shown in Checks.