Skip to content

ci(github): restore Repo Assist workflow - #59

Open
Artic0din wants to merge 11 commits into
mainfrom
ci/complete-agent-workflow-20260731
Open

ci(github): restore Repo Assist workflow#59
Artic0din wants to merge 11 commits into
mainfrom
ci/complete-agent-workflow-20260731

Conversation

@Artic0din

@Artic0din Artic0din commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Repo Assist was lost during repository history changes.
This restores the current workflow with keyless Copilot authentication and persistent repository memory, and adds the missing shared agent and Cursor Cloud workflow rules.

Test plan

  • GH_REPO=Plaintext-Lab/PowerSync gh aw compile — 0 errors, 0 warnings
  • GH_REPO=Plaintext-Lab/PowerSync gh aw validate — 0 errors, 0 warnings
  • Weighted task selection — exactly 3 distinct tasks across 10,000 deterministic seeds
  • git diff --check — passed
  • gitleaks git --staged --redact --no-banner — no leaks found
  • Full python -m pytest exposed pre-existing default-branch failures: 2,219 passed, 63 failed, 16 errors. This PR has no source or test diff; follow-up test: restore full-suite isolation on Python 3.14 #60 records the baseline defect.

Release notes

Repo Assist and shared cloud-agent instructions are available again.

Copilot AI review requested due to automatic review settings July 31, 2026 13:59
@cursor

cursor Bot commented Jul 31, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Not ready to approve

.github/workflows/repo-assist.md contains multiple trailing-whitespace lines that can fail whitespace checks and should be cleaned up.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

Restores the repository’s GitHub Agentic Workflows “Repo Assist” automation and related shared agent configuration so maintainers regain daily + on-demand assistance (including keyless Copilot auth and persistent repo memory).

Changes:

  • Added Repo Assist workflow source (repo-assist.md) plus its generated lock workflow (repo-assist.lock.yml) and generated maintenance workflow.
  • Added shared agent/skill/config scaffolding for gh-aw usage (agents, skills, MCP config, actions lock).
  • Added AGENTS.md and Cursor Cloud workflow rules, and documented the restoration in the changelog.
File summaries
File Description
CHANGELOG.md Documents the restored Repo Assist/agent tooling.
AGENTS.md Adds repo-wide agent guidelines (purpose, boundaries, validation, tracking).
.github/workflows/repo-assist.md Restores the gh-aw workflow source definition for Repo Assist.
.github/workflows/repo-assist.lock.yml Adds the compiled workflow used by GitHub Actions.
.github/workflows/copilot-setup-steps.yml Adds Copilot Agent setup steps to install gh-aw tooling.
.github/workflows/agentics-maintenance.yml Adds generated maintenance workflow for agentic safe-outputs/cleanup operations.
.github/skills/agentic-workflows/SKILL.md Adds a routing skill for gh-aw workflow tasks.
.github/mcp.json Configures local MCP server entry for gh-aw tooling.
.github/aw/actions-lock.json Pins gh-aw helper actions to specific SHAs.
.github/agents/agentic-workflows.md Adds a dispatcher agent description for gh-aw tasks.
.gitattributes Marks lock workflows as generated and sets merge strategy to reduce conflicts.
.cursor/rules/canonical-cloud-workflow.mdc Adds canonical workflow rules for Cursor agents.
Review details

Suppressed comments (2)

.github/workflows/repo-assist.md:116

  • These target lines have trailing whitespace, which can break whitespace checks (e.g., git diff --check) and should be removed.
    allowed: [bug, enhancement, "help wanted", "good first issue", "spam", "off topic", documentation, question, duplicate, wontfix, "needs triage", "needs investigation", "breaking change", performance, security, refactor]
    max: 30
    target: "*" 
  remove-labels:
    allowed: [bug, enhancement, "help wanted", "good first issue", "spam", "off topic", documentation, question, duplicate, wontfix, "needs triage", "needs investigation", "breaking change", performance, security, refactor]
    max: 5
    target: "*" 

.github/workflows/repo-assist.md:373

  • There is trailing whitespace at the end of this markdown line, which can cause git diff --check to fail and should be removed.
   ## Suggested Actions for Maintainer

   **Comprehensive list** of all pending actions requiring maintainer attention (excludes items already actioned and checked off). 
   - Reread the issue you're updating before you update it  -  there may be new checkbox adjustments since your last update that require you to adjust the suggested actions.
  • Files reviewed: 11/12 changed files
  • Comments generated: 1
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

Comment thread .github/workflows/repo-assist.md
@Artic0din
Artic0din requested a review from Copilot July 31, 2026 14:05
@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review latest head

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale comment

Comment thread .github/workflows/repo-assist.lock.yml
@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review latest head

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Not ready to approve

The scheduled-run PR-count guard currently fails the workflow run instead of cleanly skipping when the threshold is reached, which can create persistent red CI noise.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Review details

Suppressed comments (1)

.github/workflows/repo-assist.md:39

  • The pre-activation PR-count guard fails the entire scheduled workflow run when the limit is reached because the step exits non-zero and does not use continue-on-error. That makes the workflow appear as a failing run (red) rather than a graceful skip, even though the condition is expected behavior.
  steps:
    - id: check
      env:
        GH_TOKEN: ${{ github.token }}
      run: |
        MAX_OPEN_PRS=8
        if [[ "$GITHUB_EVENT_NAME" != "schedule" ]]; then exit 0; fi
        COUNT=$(gh pr list --repo "$GITHUB_REPOSITORY" --state open --search 'in:title "[repo-assist]"' --json number --jq 'length')
        [[ "$COUNT" -lt "$MAX_OPEN_PRS" ]]
  • Files reviewed: 11/12 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Verified the trailing-whitespace thread: on PR head 334ad2e9, .github/workflows/repo-assist.md has no trailing spaces (including the reported env: lines and the other locations), and git diff --check is clean against main. No further code change needed on that thread.

Open in Web View Automation 

Sent by Cursor Automation: Autofix PR review comments

Comment thread .github/workflows/repo-assist.md
@Artic0din

Copy link
Copy Markdown
Collaborator Author

Addressed the Copilot assessment in 1908d52. The PR-cap check now uses continue-on-error: true: the pre-activation job stays green, while steps.check.outcome remains failure so the agent job is skipped as intended.

@Artic0din
Artic0din requested a review from Copilot July 31, 2026 14:13
@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review latest head

Copy link
Copy Markdown
Collaborator Author

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • fast-track - for urgent changes, fast-track this PR to the front of the merge queue

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has required the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale comment

Comment thread .github/workflows/repo-assist.lock.yml

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Not ready to approve

.gitattributes sets merge=ours for lock files without defining the required custom merge driver, which can break merges for contributors.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Review details

Suppressed comments (1)

.gitattributes:1

  • merge=ours in .gitattributes relies on a custom merge driver named ours, but this repo doesn't define it anywhere, so contributors may hit merge errors/conflicts when Git tries to use an unknown driver. Since the file is already marked as generated, consider removing merge=ours (or alternatively add documented/checked-in configuration for the driver, but that would be a separate change).
.github/workflows/*.lock.yml linguist-generated=true merge=ours
  • Files reviewed: 11/12 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@graphite-app

graphite-app Bot commented Jul 31, 2026

Copy link
Copy Markdown

Merge activity

  • Jul 31, 2:17 PM UTC: Artic0din added this pull request to the Graphite merge queue.
  • Jul 31, 2:21 PM UTC: The Graphite merge queue couldn't merge this PR because it failed for an unknown reason (All comments in this PR must be resolved before merging. Once you've resolved all open comment threads, you can retry your merge.).

Comment thread .github/workflows/repo-assist.lock.yml
## Summary

Repo Assist was lost during repository history changes.
This restores the current workflow with keyless Copilot authentication and persistent repository memory, and adds the missing shared agent and Cursor Cloud workflow rules.

## Test plan

- `GH_REPO=Plaintext-Lab/PowerSync gh aw compile` — 0 errors, 0 warnings
- `GH_REPO=Plaintext-Lab/PowerSync gh aw validate` — 0 errors, 0 warnings
- `git diff --check`
- `gitleaks git --staged --redact=100 --no-banner` — no leaks found

## Release notes

Repo Assist and shared cloud-agent instructions are available again.
@graphite-app
graphite-app Bot force-pushed the ci/complete-agent-workflow-20260731 branch from 1908d52 to d0d0fce Compare July 31, 2026 14:19

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1908d526ea

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/repo-assist.md
Comment thread .github/workflows/repo-assist.lock.yml
Comment thread .github/workflows/repo-assist.md Outdated
Comment thread .gitattributes
Comment thread AGENTS.md Outdated
Comment thread .github/workflows/repo-assist.md Outdated
Comment thread .github/workflows/repo-assist.md Outdated
Generate conventional PR titles, allow required changelog edits, and correct Repo Assist discovery and agent routing.
@Artic0din
Artic0din requested a review from Copilot July 31, 2026 14:27
@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review latest head

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Human review recommended

It introduces/reenables broad, write-capable automation in GitHub Actions (Repo Assist + repo memory) and should get a final human security/ops review of triggers and permissions.

Review details
  • Files reviewed: 11/12 changed files
  • Comments generated: 0 new
  • Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale comment

Comment thread .github/workflows/repo-assist.lock.yml

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

@codex review

@copilot review

Removed merge=ours from .gitattributes in commit fix(ci): remove merge=ours from .gitattributes lock-file rule. The rule now reads:

.github/workflows/*.lock.yml linguist-generated=true

Lock-file merges will now either succeed normally or produce an explicit conflict, rather than silently keeping the base-branch version and discarding PR updates.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@Artic0din

Copy link
Copy Markdown
Collaborator Author

Review fixes are on the current head.

@codex review

@copilot review

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

Review fixes are on the current head.

@codex review...

Reviewed the current head. No issues found:

  • git diff --check — passes, no trailing whitespace
  • .gitattributesmerge=ours on *.lock.yml is correctly in place per the gh-aw canonical pattern (thread resolved)
  • No production source or test changes in this PR

The branch looks good to merge.

@Artic0din

Copy link
Copy Markdown
Collaborator Author

Latest review findings are resolved on the current head.

@codex review

@copilot review

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

Latest review findings are resolved on the current head.

@codex review...

Reviewed the current head. All previous findings are resolved:

  • git diff --check — passes, no trailing whitespace
  • .gitattributesmerge=ours on *.lock.yml is intentional per gh-aw canonical pattern (thread resolved)
  • External gh-aw prompt URLs pinned to commit SHA bbb8042878 — good reproducibility hardening
  • No production source or test changes

No further issues found. Ready to merge.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@Artic0din

Copy link
Copy Markdown
Collaborator Author

@copilot review

@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Stale comment

Completed orchestrated security triage for the current PR head and posted only net-new, unacknowledged findings after deduplication against prior automation threads and human replies.

Open in Web View Automation 

Sent by Cursor Security Agent: Security Reviewer

Comment thread .github/workflows/repo-assist.lock.yml

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

@copilot review

No changes since the last review — branch is clean, git diff --check passes, and all previous findings remain resolved. Ready to merge.

@Artic0din

Copy link
Copy Markdown
Collaborator Author

@copilot review

@Artic0din

Copy link
Copy Markdown
Collaborator Author

@codex review

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown

@copilot review

Reviewed the current head (633d599). All findings remain resolved and the new commit is a positive security improvement — GitHub MCP server access is now scoped to plaintext-lab/powersync only (previously "all"), both in the lock file and source .md. git diff --check passes, working tree is clean. Ready to merge.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 633d5992b8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/repo-assist.md Outdated
Comment thread .github/workflows/repo-assist.lock.yml Outdated
Comment thread .github/workflows/repo-assist.md Outdated
Comment thread .github/workflows/repo-assist.md Outdated
Enable close-issue for monthly summary rollover, count unlabelled PRs
in Task 1 weighting, give Task 5 a create-issue path on schedules, and
require GH_AW_GITHUB_TOKEN so generated PRs trigger validation.

Co-authored-by: Artic0din <Artic0din@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants