Repository navigation
Add optional GitHub connection checks and resumable onboarding - #28
Merged
Merged
Conversation
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (15)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
ask-pro --githubnow checks the signed-in ChatGPT account's GitHub connection before uploading the context bundle or sending the prompt. If setup is needed or the UI cannot confirm a connection, it opens the GitHub plugin page and returns a resumable handoff. After the human connects the account,--resumechecks again and submits the original request; ordinary consults do not require GitHub.The option persists with the session, defaults to normal ChatGPT, and asks Pro for read-only repository evidence with citations and revision information. The maintained skill prefers GitHub for committed code and reserves source-file bundles for local changes or evidence the app cannot retrieve. The current ChatGPT GitHub plugin advertises write capabilities: this is not an enforced read-only permission mode. ask-pro does not collect GitHub credentials, automate consent, select repositories, approve app actions, or prove repository access from connection status alone.
Validation:
rvw_e3036c1dcompleted with both reviewers clean at6bb682e.NEEDS_GITHUB_CONNECTIONbefore submission. After human setup, resume passed the connection check, selected Pro, and returned independently verified facts from the requestedprofilePaths.tscommit, with no source files uploaded. The answer reports GitHub file retrieval and the requested revision, but the harvested citation link was absent and the activity panel was not independently inspected. This proves handoff/resume and answer accuracy; it does not independently prove the tool trace or a disconnected initial account. Final recovery-only fixes were covered by tests/CI after the live run started.b9506ed. Publication was skipped for the PR. The only change after clean review was the manual-test evidence note.No dependencies added. README, maintained skill, Windows notes, changelog, and manual smoke steps cover the new flow and permission limits.