-
Notifications
You must be signed in to change notification settings - Fork 2.7k
Enable Chrome Browser Takeover Mode with Remote Debugging Integration #12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from 27 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
5bb4dda
Updating usage of pupetteer and readme udpates
a8trejo 075c062
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo a0654dd
Order package json scripts by alphabet
a8trejo 3630fe3
Not overwriting 2.0.1
a8trejo ffd3562
Added changeset to RooCline and respective workflow, docs updates
a8trejo fcfb6a6
Draft updates
a8trejo 22ce1de
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo fe1d823
Updates to connect puppetter to browser with remote-debugging-port by…
a8trejo 4763c4e
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo b0e48c5
Updates to make browser remote port dynamic from prompt
a8trejo 3ad3bdc
Fixing package scripts
a8trejo b347ed6
Reusing composite action
a8trejo 2565b74
Updating caching
a8trejo 9c27b12
Tmp fixes
a8trejo d4f0d9f
fixing path
a8trejo d317c21
Installing with registry.npmjs
a8trejo 0784970
Removing unused file
a8trejo f48707f
Ellipsis comments
a8trejo 8d73ad7
Adding npmrc to repo to avoid code artifact issues
a8trejo 7028670
Misc updates
a8trejo 7d35ded
Updating README
a8trejo eae3626
PR comments
a8trejo cbdbd9b
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo 4ad27f2
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo eb710d7
Minor workflows updates
a8trejo 1a7392c
Updating README instructions
a8trejo 0ada2d5
Ellipsis quiet mode
a8trejo 2e9d6b6
PR self-review
a8trejo 4ebd331
Marketplace publish compability with changeset
a8trejo 1cdf981
Minor updates
a8trejo 000eed6
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo b361bbb
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo 4d53bd2
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo f049fb4
Updates to enter interactive mode from the extension settings instead…
a8trejo ef6f0a2
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo ba3b027
Updates to launch a new browser if remote debugging browser is not ru…
a8trejo 8e9218c
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo ea6eae7
Updates from main
a8trejo b38e1f9
Updating extension name in extension.ts
a8trejo 59e976a
Adding new explicit browser action snapshot to work with interactive …
a8trejo 4e0cc31
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo a1b48fa
Merge branch 'main' of https://github.com/RooVetGit/Roo-Cline into fe…
a8trejo File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| # Changesets | ||
|
|
||
| Hello and welcome! This folder has been automatically generated by `@changesets/cli`, a build tool that works | ||
| with multi-package repos, or single-package repos to help you version and publish your code. You can | ||
| find the full documentation for it [in our repository](https://github.com/changesets/changesets) | ||
|
|
||
| We have a quick list of common questions to get you started engaging with this project in | ||
| [our documentation](https://github.com/changesets/changesets/blob/main/docs/common-questions.md) |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "roo-cline": minor | ||
| --- | ||
|
|
||
| Enabled Roo Cline to use puppeteer and connect to a current browser which is already authenticated to multiple sites. |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| { | ||
| "$schema": "https://unpkg.com/@changesets/[email protected]/schema.json", | ||
| "changelog": "@changesets/cli/changelog", | ||
| "commit": false, | ||
| "fixed": [], | ||
| "linked": [], | ||
| "access": "restricted", | ||
| "baseBranch": "main", | ||
| "updateInternalDependencies": "patch", | ||
| "ignore": [] | ||
| } |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,79 @@ | ||
| name: Release | ||
| run-name: Release ${{ github.event_name == 'push' && '- Create Changeset PR' || '- Approve & Merge Changeset PR' }} | ||
|
|
||
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| pull_request_target: | ||
|
|
||
| concurrency: ${{ github.workflow }}-${{ github.ref }} | ||
|
|
||
| env: | ||
| AWS_REGION: us-east-1 | ||
|
|
||
| jobs: | ||
| release: | ||
| name: Release | ||
| if: ${{ github.event_name == 'push' && github.actor != 'R00-B0T'}} | ||
| runs-on: ubuntu-latest | ||
| permissions: | ||
| contents: write | ||
| pull-requests: write | ||
| steps: | ||
| - name: Checkout Repo | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| fetch-depth: 0 | ||
| - name: Setup Node.js 20 | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: 20 | ||
| - id: install-aws-cli | ||
| uses: unfor19/install-aws-cli-action@v1 | ||
| with: | ||
| version: 2 | ||
| verbose: false | ||
| arch: amd64 | ||
| - name: Configure AWS Credentials | ||
| uses: aws-actions/configure-aws-credentials@v4 | ||
| with: | ||
| aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }} | ||
| aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }} | ||
| aws-region: ${{ env.AWS_REGION }} | ||
| - name: NPM Install | ||
| run: | | ||
| npm run install:all | ||
| npm run build | ||
|
|
||
| - name: Changeset Pull Request | ||
| id: changesets | ||
| uses: changesets/action@v1 | ||
| with: | ||
| commit: "changeset version bump" | ||
| title: "Changeset version bump" | ||
| version: npm run version-packages # This performs the changeset version bump | ||
| # publish: npm run publish # Revisit possible refactor along with npm-publish.yml | ||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.CROSS_REPO_ACCESS_TOKEN }} | ||
|
|
||
| auto-approve-and-merge: | ||
| name: Auto approve and merge Bump version PRs | ||
| runs-on: ubuntu-latest | ||
| if: github.actor == 'R00-B0T' && contains(github.event.pull_request.title, 'Changeset version bump') | ||
| permissions: | ||
| pull-requests: write | ||
|
|
||
| steps: | ||
| - name: Checkout Repo | ||
| uses: actions/checkout@v4 | ||
| with: | ||
| token: ${{ secrets.CROSS_REPO_ACCESS_TOKEN }} | ||
| - name: Auto approve PR | ||
| uses: hmarr/auto-approve-action@v4 | ||
| with: | ||
| review-message: "I'm approving since it's a bump version PR" | ||
| - name: Enable automerge on PR | ||
| run: gh pr merge --squash --auto ${{ github.event.pull_request.number }} | ||
| env: | ||
| GH_TOKEN: ${{ secrets.CROSS_REPO_ACCESS_TOKEN }} |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,4 +5,5 @@ node_modules | |
|
|
||
| .DS_Store | ||
|
|
||
| .npmrc | ||
| # Builds | ||
| bin | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| registry=https://registry.npmjs.org/ |
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,12 @@ | ||
| Remember to rebuild the extension and install it using the cli specified on the `Roo-Cline/README.md` file, also remember to change directory to the `Roo-Cline` folder before running the commands from the readme. | ||
|
|
||
| Consider that any `export` or `install` command has already been run, so exclude those.ˆ | ||
|
|
||
| Before editing the file 'Roo-Cline/src/core/Cline.ts', since it's a really large file, read the original file to understand its full content. | ||
| Then create a new file with the -updates suffix containing ONLY the changes you've made to the original file. | ||
|
|
||
|
|
||
| In interactive mode, browser actions are disabled (if (!this.isInteractive) check in doAction method) | ||
| The navigateToUrl method uses doAction, which means navigation won't work in interactive mode | ||
|
|
||
| Can we analyze 'Roo-Cline/src/core/Cline.ts' to see how it handles browser actions and if there's some logic preventing it from using 'doAction' |
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| Could you navigate to this URL in interactive mode (browserPort = 7333) | ||
a8trejo marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| https://cloud.cypress.io/projects/hkawvm/runs/22055/overview/b890f243-ce56-4047-9252-c99469dfbbea?roarHideRunsWithDiffGroupsAndTags=1&interactive=true | ||
|
|
||
| Please analyze the failure and let me know if you can fix it. | ||
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -7,7 +7,7 @@ pr_review: | |
|
|
||
| # If quiet mode is enabled, Ellipsis will only leave reviews when it has comments, so “Looks good to me” reviews | ||
| # will be skipped. This can reduce clutter. | ||
| quiet: false | ||
| quiet: true | ||
|
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. let me know if anyone has strong objections against this, but I find it to be too much spam every time I get a "Looks good to me" email, not to mention how long it makes a PR conversation |
||
|
|
||
| # You can disable automatic code review using auto_review_enabled. This will override any global settings you | ||
| # have configured via the web UI. | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you give me a rundown on how the release process works once we have this new file?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh yeah, will update the readme, but basically the same as
Roo-Node-Typeswherechangesettakes care of it