Conversation
|
Resolves #57 |
|
To provide feedback, I recommend leaving inline comments for best results. Navigate to the Files changed tab and leave comments on the proposed code changes. Choose Start review for each comment, and then choose Request changes, and I'll propose revised changes. If creating PR-level comments, include as much detail as possible in your feedback (for example, file name, class name, line number). |
|
✅ I updated this pull request based on the pipeline log. To provide feedback on how Amazon Q Developer performed with code generation, create an issue in the Amazon Q Developer public repository. Note: This PR includes changes to Workflow contents under the |
🧪 CI InsightsHere's what we observed from your CI run for 689c1b0. 🟢 All jobs passed!But CI Insights is watching 👀 |
… review Adds comprehensive GitHub Actions workflows for: - Auto-labeling issues and PRs - Auto-assigning reviewers - Automated code cleanliness checks - Complete CI/CD review pipeline
|
✅ I updated this pull request based on the pipeline log. To provide feedback on how Amazon Q Developer performed with code generation, create an issue in the Amazon Q Developer public repository. Note: This PR includes changes to Workflow contents under the |
Add GitHub Actions workflows for automated testing using Playwright: - Test execution across multiple browsers - Code functionality and documentation review - Test coverage analysis - Automated PR creation for fixes
|
✅ I updated this pull request based on the pipeline log. To provide feedback on how Amazon Q Developer performed with code generation, create an issue in the Amazon Q Developer public repository. Note: This PR includes changes to Workflow contents under the |
This improves the robustness of workflow scripts by adding default values for undefined count variables and adds a check_issues script for monitoring workflow patterns.
|
🔴 I'm sorry, I can't create any more revisions for this pipeline failure. To provide feedback on how Amazon Q Developer performed with code generation, create an issue in the Amazon Q Developer public repository. |
|
/gemini review @cursor please review |
![gcassist[bot]](https://avatars.githubusercontent.com/u/223870169?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request refactors documentation by removing redundant sections in the README, renaming the implementation design document, and updating the Sphinx table of contents. It also introduces a utility script, check_issues.sh, to scan GitHub workflows for specific patterns. Review feedback highlights a broken documentation link in index.rst that should likely point to the implementation document. Additionally, it is recommended to move the new shell script to a dedicated directory and update its logic to return non-zero exit codes when issues are detected to ensure it functions correctly within a CI/CD pipeline.
|
|
||
| overview | ||
| getting_started | ||
| connection |
There was a problem hiding this comment.
The toctree entry connection refers to a file that does not exist in the repository (e.g., docs/connection.rst or docs/connection.md). This will cause the Sphinx documentation build to fail. Based on the file changes, it seems this might be a typo for implementation, which was recently moved to the docs/ directory. If a separate connection guide was intended, please ensure the file is included in the pull request.
| connection | |
| implementation |
| @@ -0,0 +1,16 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
This utility script is placed in the root directory. To maintain a clean project structure, it is recommended to move such scripts into a dedicated directory like scripts/ or tools/.
| grep -r '\$((.*))' .github/workflows/ || echo "No shell arithmetic found" | ||
|
|
||
| echo "" | ||
| echo "=== Searching for remaining GitHub Copilot actions ===" | ||
| grep -r 'github/copilot' .github/workflows/ || echo "No GitHub Copilot actions found" | ||
|
|
||
| echo "" | ||
| echo "=== Searching for remaining shell arithmetic patterns ===" | ||
| grep -r '\$(' .github/workflows/ | grep -E '(count|total).*=' || echo "No problematic patterns found" | ||
|
|
||
| echo "" | ||
| echo "=== Checking for any uses: statements with copilot ===" | ||
| grep -r 'uses:.*copilot' .github/workflows/ || echo "No copilot uses statements found" No newline at end of file |
There was a problem hiding this comment.
The script currently returns a success exit code (0) regardless of whether problematic patterns are found, because the || echo ... construct suppresses the non-zero exit code from grep. If this script is intended for use in CI/CD pipelines to validate code quality, it should be modified to exit with a non-zero status when issues are detected.
There was a problem hiding this comment.
Pull request overview
This PR updates project documentation and GitHub Actions automation around the newly introduced CDP I/O / multiplexing work, including adding navigation to connection docs and hardening several workflow scripts.
Changes:
- Updated docs structure (toctree) and refined the I/O/multiplexing implementation design document.
- Removed/disabled references to non-public GitHub Copilot Actions in CI workflows and added more defensive scripting/logging.
- Added a helper script (
check_issues.sh) to search workflows for certain patterns.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Removes redundant Sans-I/O section header text near the usage examples. |
| docs/index.rst | Adds connection to the Sphinx toctree. |
| docs/implementation.md | Renames the document title and adjusts formatting of the concluding requirement text. |
| check_issues.sh | Adds a local helper script to grep workflows for specific patterns. |
| .github/workflows/auto-label.yml | Wraps labeling call in try/catch with logging. |
| .github/workflows/auto-label-comment-prs.yml | Wraps label/comment calls in try/catch with logging. |
| .github/workflows/auto-copilot-test-review-playwright.yml | Removes a Copilot action step and replaces it with explanatory comments. |
| .github/workflows/auto-copilot-playwright-auto-test.yml | Removes Copilot agent steps and replaces them with explanatory comments. |
| .github/workflows/auto-copilot-org-playwright-loopv2.yaml | Removes Copilot agent steps and replaces them with explanatory comments. |
| .github/workflows/auto-copilot-org-playwright-loop.yaml | Removes Copilot agent steps and replaces them with explanatory comments. |
| .github/workflows/auto-copilot-functionality-docs-review.yml | Adds numeric-default guards before shell arithmetic; removes Copilot CLI action usage. |
| .github/workflows/auto-copilot-code-cleanliness-review.yml | Adds a numeric-default guard in the grep counting snippet; removes Copilot CLI action usage. |
| .github/workflows/auto-complete-cicd-review.yml | Adds error handling around issue listing/creation and update paths. |
| .github/workflows/auto-assign-pr.yml | Adds try/catch and clearer logging around assignment. |
Comments suppressed due to low confidence (1)
docs/implementation.md:184
- This quoted requirement text is unprofessional for project documentation and is likely to age poorly. Recommend paraphrasing it in neutral technical terms (and optionally linking to the issue) rather than quoting the original wording verbatim.
| overview | ||
| getting_started | ||
| connection | ||
| api |
There was a problem hiding this comment.
index.rst adds connection to the Sphinx toctree, but the doc is docs/connection.md and docs/conf.py does not enable Markdown support (e.g., myst_parser/recommonmark). This will likely break the docs build because Sphinx will look for connection.rst by default. Convert the doc to .rst or enable Markdown parsing in conf.py (and add the dependency) so the toctree entry resolves.
| ! -path "*/.venv/*" \ | ||
| ! -path "*/vendor/*" \ | ||
| -exec sh -c 'count=$(grep -c "$1" "$2" 2>/dev/null || echo 0); if [ "$count" -gt 20 ]; then echo "$count definitions in $2"; fi' _ "$pattern" {} \; \ | ||
| -exec sh -c 'count=$(grep -c "$1" "$2" 2>/dev/null || echo 0); count=${count:-0}; if [ "$count" -gt 20 ]; then echo "$count definitions in $2"; fi' _ "$pattern" {} \; \ |
There was a problem hiding this comment.
This grep -c is using a pattern with | alternation (e.g., ^def |^class ), but plain grep treats | literally (BRE) so the counts will be wrong. Use grep -cE (or escape as \|) so the complexity/size heuristic reports accurate numbers.
| -exec sh -c 'count=$(grep -c "$1" "$2" 2>/dev/null || echo 0); count=${count:-0}; if [ "$count" -gt 20 ]; then echo "$count definitions in $2"; fi' _ "$pattern" {} \; \ | |
| -exec sh -c 'count=$(grep -cE "$1" "$2" 2>/dev/null || echo 0); count=${count:-0}; if [ "$count" -gt 20 ]; then echo "$count definitions in $2"; fi' _ "$pattern" {} \; \ |
| // Add automated comment | ||
| await github.rest.issues.createComment({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: pr_number, | ||
| body: "Thanks for the PR! Copilot will assist with review." | ||
| }); No newline at end of file | ||
| try { | ||
| await github.rest.issues.createComment({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: pr_number, | ||
| body: "Thanks for the PR! Copilot will assist with review." | ||
| }); |
There was a problem hiding this comment.
This workflow runs on synchronize events, so the createComment call will post a new automated comment on every push to the PR, which can quickly spam PR timelines. Consider limiting commenting to opened/reopened, or check for an existing bot comment before creating a new one (and update/edit instead).
| #!/bin/bash | ||
|
|
||
| echo "=== Searching for remaining shell arithmetic operations ===" | ||
| grep -r '\$((.*))' .github/workflows/ || echo "No shell arithmetic found" |
There was a problem hiding this comment.
The grep pattern \$((.*)) is not a valid way to match $((...)) in grep BRE (parentheses are literals), so this check likely won’t find what it intends. If the goal is to detect shell arithmetic expansions, prefer a simpler/robust pattern like searching for the literal $(( (and optionally ))) or use grep -E with properly escaped parentheses.
| grep -r '\$((.*))' .github/workflows/ || echo "No shell arithmetic found" | |
| grep -rF '$((' .github/workflows/ || echo "No shell arithmetic found" |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request cleans up the documentation by removing redundant sections in the README, renaming the implementation summary to a design document, and updating the Sphinx index. Additionally, a new shell script check_issues.sh is introduced to audit GitHub workflows for specific patterns like shell arithmetic and Copilot actions. I have no feedback to provide.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Assign Copilot (or others) to new PRs | ||
| uses: actions/github-script@main | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: | | ||
| // Assign PRs to Copilot or other users | ||
| const copilotUsername = "copilot"; // <-- TUNE ME! | ||
| const assignees = [copilotUsername]; // Or: ["copilot","anotheruser"] | ||
| const currentAssignees = context.payload.pull_request.assignees.map(u => u.login); | ||
| if (!assignees.every(a => currentAssignees.includes(a))) { | ||
| console.log(`PR has assignees to add. Attempting to assign ${assignees.join(", ")}...`); | ||
|
|
||
| try { | ||
| await github.rest.issues.addAssignees({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: context.payload.pull_request.number, | ||
| assignees | ||
| }); | ||
| console.log(`✅ Assigned ${assignees.join(", ")} to PR #${context.payload.pull_request.number}`); |
| body: "Automated Playwright test fixes by Copilot Agent." | ||
| commit-message: "Copilot agent Playwright bugfixes" | ||
| continue-on-error: true | ||
|
|
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@main | ||
|
|
||
| - name: Setup Python | ||
| uses: actions/setup-python@main | ||
| with: | ||
| python-version: "3.11" | ||
|
|
||
| - name: Install dependencies | ||
| run: | | ||
| pip install -r requirements.txt | ||
| pip install pytest playwright pytest-playwright | ||
|
|
||
| - name: Install Playwright browsers | ||
| run: | | ||
| python -m playwright install | ||
|
|
||
| - name: Run Playwright Tests | ||
| run: | | ||
| pytest tests/ || exit 1 | ||
| continue-on-error: true | ||
|
|
||
| # Note: GitHub Copilot agent actions are not available as public actions | ||
| # This step would provide automated PR review | ||
| # For now, manual PR review is required | ||
|
|
||
| # Note: GitHub Copilot agent actions are not available as public actions | ||
| # This step would automatically fix failing Playwright tests | ||
| # For now, manual test fixing is required | ||
|
|
||
| - name: Create Pull Request for Automated Fixes | ||
| uses: peter-evans/create-pull-request@main | ||
| with: | ||
| branch: "copilot/playwright-fixes" | ||
| title: "Copilot: Auto-fix Playwright Tests" | ||
| body: "Automated Playwright test fixes by Copilot Agent." | ||
| commit-message: "Copilot agent Playwright bugfixes" | ||
| continue-on-error: true | ||
|
|
||
| - name: Automerge PR if checks pass | ||
| uses: pascalgn/automerge-action@main | ||
| with: | ||
| merge-method: squash | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| continue-on-error: true No newline at end of file |
| # For now, manual test fixing is required | ||
|
|
||
| - name: Create Pull Request for Automated Fixes | ||
| uses: peter-evans/create-pull-request@main |
| continue-on-error: true | ||
|
|
||
| - name: Automerge PR if checks pass | ||
| uses: pascalgn/automerge-action@main |
| title: "Copilot generated Playwright tests (auto-fixed)" | ||
| body: "Automated Playwright test generation/fix by Copilot agent." | ||
| commit-message: "Copilot agent Playwright tests and fixes" | ||
| continue-on-error: true |
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/github-script@main | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: | | ||
| const pr_number = context.payload.pull_request.number; | ||
|
|
||
| // Add label | ||
| try { | ||
| await github.rest.issues.addLabels({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: pr_number, | ||
| labels: ["needs-review", "copilot"] // <-- TUNE ME | ||
| }); | ||
| console.log(`✅ Added labels to PR #${pr_number}`); | ||
| } catch (error) { | ||
| console.log(`⚠️ Failed to add labels: ${error.message}`); | ||
| console.log("Note: This may be due to insufficient permissions or invalid label names."); | ||
| } | ||
|
|
||
| // Add automated comment | ||
| try { | ||
| await github.rest.issues.createComment({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: pr_number, | ||
| body: "Thanks for the PR! Copilot will assist with review." | ||
| }); | ||
| console.log(`✅ Added comment to PR #${pr_number}`); | ||
| } catch (error) { | ||
| console.log(`⚠️ Failed to add comment: ${error.message}`); | ||
| console.log("Note: This may be due to insufficient permissions."); | ||
| } No newline at end of file |
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Add labels | ||
| uses: actions/github-script@main | ||
| with: | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| script: | | ||
| // Add or tweak your labels here | ||
| const labels = ["triage", "copilot"]; // <-- TUNE ME! | ||
|
|
||
| try { | ||
| await github.rest.issues.addLabels({ | ||
| owner: context.repo.owner, | ||
| repo: context.repo.repo, | ||
| issue_number: context.issue.number, | ||
| labels | ||
| }); | ||
| console.log(`✅ Added labels [${labels.join(", ")}] to issue #${context.issue.number}`); | ||
| } catch (error) { | ||
| console.log(`⚠️ Failed to add labels: ${error.message}`); | ||
| console.log("Note: This may be due to insufficient permissions or invalid label names."); | ||
| } No newline at end of file |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 4 potential issues.
Bugbot Autofix prepared fixes for all 4 issues found in the latest run.
- ✅ Fixed: Cross-job step reference always resolves to empty
- I exposed
functionality-checkbuild output at the job level, addedneeds: functionality-checktodocumentation-review, and consumed it vianeeds.functionality-check.outputs.build_statusso the real build status is passed across jobs.
- I exposed
- ✅ Fixed: Quoted heredoc prevents date command expansion
- I changed the heredoc delimiter from
<< 'EOF'to<< EOFso$(date ...)is expanded when generating the report.
- I changed the heredoc delimiter from
- ✅ Fixed: Duplicate workflows race on same PR branch
- I removed the redundant
.github/workflows/auto-copilot-org-playwright-loopv2.yamlworkflow file so only one workflow targets the shared PR branch.
- I removed the redundant
- ✅ Fixed: Toctree references Markdown file without parser configured
- I enabled Markdown support by adding
myst_parserand.mdindocs/conf.pyand addingmyst-parsertodocs/requirements.txtsoconnection.mdresolves in the toctree.
- I enabled Markdown support by adding
Or push these changes by commenting:
@cursor push d682c57f0b
Preview (d682c57f0b)
diff --git a/.github/workflows/auto-complete-cicd-review.yml b/.github/workflows/auto-complete-cicd-review.yml
--- a/.github/workflows/auto-complete-cicd-review.yml
+++ b/.github/workflows/auto-complete-cicd-review.yml
@@ -283,7 +283,7 @@
mkdir -p /tmp/final-report
- cat > /tmp/final-report/complete-review.md << 'EOF'
+ cat > /tmp/final-report/complete-review.md << EOF
# Complete CI/CD Agent Review Report
**Review Date:** $(date -u +"%Y-%m-%d %H:%M:%S UTC")
diff --git a/.github/workflows/auto-copilot-functionality-docs-review.yml b/.github/workflows/auto-copilot-functionality-docs-review.yml
--- a/.github/workflows/auto-copilot-functionality-docs-review.yml
+++ b/.github/workflows/auto-copilot-functionality-docs-review.yml
@@ -17,6 +17,8 @@
jobs:
functionality-check:
runs-on: ubuntu-latest
+ outputs:
+ build_status: ${{ steps.build.outputs.BUILD_STATUS }}
steps:
- name: Checkout code
uses: actions/checkout@main
@@ -105,6 +107,8 @@
documentation-review:
runs-on: ubuntu-latest
+ needs: functionality-check
+ if: always()
steps:
- name: Checkout code
uses: actions/checkout@main
@@ -311,4 +315,4 @@
});
}
env:
- BUILD_STATUS: ${{ steps.build.outputs.BUILD_STATUS }}
+ BUILD_STATUS: ${{ needs['functionality-check'].outputs.build_status }}
diff --git a/.github/workflows/auto-copilot-org-playwright-loopv2.yaml b/.github/workflows/auto-copilot-org-playwright-loopv2.yaml
deleted file mode 100644
--- a/.github/workflows/auto-copilot-org-playwright-loopv2.yaml
+++ /dev/null
@@ -1,57 +1,0 @@
-name: "Org-wide: Copilot Playwright Test, Review, Auto-fix, PR, Merge"
-
-on:
- push:
- branches:
- - main
- - master
-
-jobs:
- playwright-review-fix:
- runs-on: ubuntu-latest
- steps:
- - name: Checkout code
- uses: actions/checkout@main
-
- - name: Setup Python
- uses: actions/setup-python@main
- with:
- python-version: "3.11"
-
- - name: Install dependencies
- run: |
- pip install -r requirements.txt
- pip install pytest playwright pytest-playwright
-
- - name: Install Playwright browsers
- run: |
- python -m playwright install
-
- - name: Run Playwright Tests
- run: |
- pytest tests/ || exit 1
- continue-on-error: true
-
- # Note: GitHub Copilot agent actions are not available as public actions
- # This step would provide automated PR review
- # For now, manual PR review is required
-
- # Note: GitHub Copilot agent actions are not available as public actions
- # This step would automatically fix failing Playwright tests
- # For now, manual test fixing is required
-
- - name: Create Pull Request for Automated Fixes
- uses: peter-evans/create-pull-request@main
- with:
- branch: "copilot/playwright-fixes"
- title: "Copilot: Auto-fix Playwright Tests"
- body: "Automated Playwright test fixes by Copilot Agent."
- commit-message: "Copilot agent Playwright bugfixes"
- continue-on-error: true
-
- - name: Automerge PR if checks pass
- uses: pascalgn/automerge-action@main
- with:
- merge-method: squash
- github-token: ${{ secrets.GITHUB_TOKEN }}
- continue-on-error: true
\ No newline at end of file
diff --git a/docs/conf.py b/docs/conf.py
--- a/docs/conf.py
+++ b/docs/conf.py
@@ -33,9 +33,15 @@
extensions = [
'sphinx.ext.autodoc',
'sphinx_autodoc_typehints',
+ 'myst_parser',
'sphinx_rtd_theme',
]
+source_suffix = {
+ '.rst': 'restructuredtext',
+ '.md': 'markdown',
+}
+
# Add any paths that contain templates here, relative to this directory.
templates_path = ['_templates']
diff --git a/docs/requirements.txt b/docs/requirements.txt
--- a/docs/requirements.txt
+++ b/docs/requirements.txt
@@ -3,4 +3,5 @@
sphinx
sphinx-autodoc-typehints
sphinx-rtd-theme
+myst-parser
.You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit a67eb46. Configure here.
| }); | ||
| } | ||
| env: | ||
| BUILD_STATUS: ${{ steps.build.outputs.BUILD_STATUS }} |
There was a problem hiding this comment.
Cross-job step reference always resolves to empty
Medium Severity
The documentation-review job references ${{ steps.build.outputs.BUILD_STATUS }} at the env level, but the build step (with id: build) is defined in the separate functionality-check job. In GitHub Actions, steps context is scoped to the current job. Since the documentation-review job has no step with id build, and no needs: functionality-check dependency, this expression always resolves to an empty string. The buildStatus variable will therefore always be 'unknown', and the report will never show the real build status.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit a67eb46. Configure here.
| cat > /tmp/final-report/complete-review.md << 'EOF' | ||
| # Complete CI/CD Agent Review Report | ||
|
|
||
| **Review Date:** $(date -u +"%Y-%m-%d %H:%M:%S UTC") |
There was a problem hiding this comment.
Quoted heredoc prevents date command expansion
Low Severity
The heredoc delimiter is single-quoted (<< 'EOF'), which disables all shell expansions within the heredoc body. The $(date -u +"%Y-%m-%d %H:%M:%S UTC") command substitution on the "Review Date" line will be written literally into the report file instead of being replaced with the actual date. The GitHub Actions expressions (${{ ... }}) still work since they're expanded before shell execution, but the date will appear as a raw command string.
Reviewed by Cursor Bugbot for commit a67eb46. Configure here.
| with: | ||
| merge-method: squash | ||
| github-token: ${{ secrets.GITHUB_TOKEN }} | ||
| continue-on-error: true No newline at end of file |
There was a problem hiding this comment.
Duplicate workflows race on same PR branch
Medium Severity
auto-copilot-org-playwright-loop.yaml and auto-copilot-org-playwright-loopv2.yaml are functionally identical — same workflow name, same triggers (push to main/master), same job name, same target branch (copilot/playwright-fixes), and same automerge step. Both will run simultaneously on every push, racing to create a PR on the same branch and automerge it, which can cause conflicts and unpredictable behavior. One of these files appears to be redundant.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit a67eb46. Configure here.
|
|
||
| overview | ||
| getting_started | ||
| connection |
There was a problem hiding this comment.
Toctree references Markdown file without parser configured
Medium Severity
The toctree adds a connection entry, but the corresponding file is docs/connection.md (Markdown). Sphinx's conf.py has no Markdown parser extension (myst-parser or recommonmark) in its extensions list, and no source_suffix is configured for .md files. Sphinx will look for connection.rst by default, which doesn't exist, causing the documentation build to fail with a "nonexisting document" error.
Reviewed by Cursor Bugbot for commit a67eb46. Configure here.
This pull request implements I/O and multiplexing support for the Chrome DevTools Protocol library. Key changes include:
Major Updates:
Documentation:
Infrastructure:
The changes maintain full backward compatibility while adding powerful new features for WebSocket-based communication and command multiplexing. All modifications follow best practices for security, performance, and error handling.
Note
High Risk
Adds multiple GitHub Actions with
writepermissions (issues/PRs/contents) including automated PR creation and auto-merge onmain/master, which can unintentionally change repo state or create workflow loops if misconfigured.Overview
Adds several new GitHub Actions workflows to auto-assign and auto-label PRs/issues, post automated PR comments, and run a scheduled/on-PR “complete review” pipeline that performs basic code-cleanliness checks, test execution (including Playwright), docs checks, and build attempts, then uploads artifacts and opens/updates a tracking issue.
Also introduces additional Playwright-centric workflows that can create follow-up PRs and attempt automerge when checks pass, plus a small
check_issues.shhelper script and minor docs tweaks (rename indocs/implementation.md, addconnectiontodocs/index.rst).Reviewed by Cursor Bugbot for commit a67eb46. Configure here.